Skip to content

Commit 05577a2

Browse files
refactor: address PR comments.
1 parent 0227f7e commit 05577a2

1 file changed

Lines changed: 9 additions & 27 deletions

File tree

src/modules/app-core/workspace-context-status-controller.js

Lines changed: 9 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
const hasTokenValue = token => typeof token === 'string' && token.trim().length > 0
2-
31
const createWorkspaceContextStatusController = ({
42
statusNode,
53
toNonEmptyWorkspaceText,
@@ -12,9 +10,6 @@ const createWorkspaceContextStatusController = ({
1210
getWorkspaceRepositoryFullName,
1311
getSelectedRepositoryFullName,
1412
}) => {
15-
let hasValidatedGitHubPat = false
16-
let hasCompletedRepositoryLoad = false
17-
1813
const getWorkspaceName = () => {
1914
const workspaceScope =
2015
toNonEmptyWorkspaceText(getWorkspaceScopeMarker?.()).toLowerCase() || 'local'
@@ -58,13 +53,12 @@ const createWorkspaceContextStatusController = ({
5853
const workspaceName = getWorkspaceName()
5954
const workspaceScope =
6055
toNonEmptyWorkspaceText(getWorkspaceScopeMarker?.()).toLowerCase() || 'local'
61-
const shouldShowRepositoryContext =
62-
hasValidatedGitHubPat && workspaceScope !== 'local'
63-
const repository = shouldShowRepositoryContext
64-
? toNonEmptyWorkspaceText(getWorkspaceRepositoryFullName?.()) ||
65-
toNonEmptyWorkspaceText(getSelectedRepositoryFullName?.()) ||
66-
'unknown'
67-
: 'local'
56+
const repository =
57+
workspaceScope === 'local'
58+
? 'local'
59+
: toNonEmptyWorkspaceText(getWorkspaceRepositoryFullName?.()) ||
60+
toNonEmptyWorkspaceText(getSelectedRepositoryFullName?.()) ||
61+
'unknown'
6862

6963
statusNode.textContent = `${workspaceName}${repository}`
7064
}
@@ -74,25 +68,13 @@ const createWorkspaceContextStatusController = ({
7468
}
7569

7670
const syncTokenState = token => {
77-
if (!hasTokenValue(token)) {
78-
hasValidatedGitHubPat = false
79-
hasCompletedRepositoryLoad = false
80-
} else if (hasCompletedRepositoryLoad) {
81-
hasValidatedGitHubPat = true
82-
}
83-
71+
void token
8472
render()
8573
}
8674

8775
const syncWritableRepositoriesState = ({ token, isLoadingRepositories = false }) => {
88-
if (!isLoadingRepositories) {
89-
hasCompletedRepositoryLoad = true
90-
}
91-
92-
if (hasTokenValue(token) && !isLoadingRepositories) {
93-
hasValidatedGitHubPat = true
94-
}
95-
76+
void token
77+
void isLoadingRepositories
9678
render()
9779
}
9880

0 commit comments

Comments
 (0)