refactor(sidebar): name the label map for what it is keyed by

The map moved from worktree id to host-qualified identity in the previous
commit; the parameter name still said 'ByWorktreeId', which is exactly the
collapse this change exists to prevent.
This commit is contained in:
Brennan Benson
2026-08-16 21:59:42 -07:00
parent 021de1a556
commit d413c3d463
3 changed files with 7 additions and 7 deletions
@@ -131,7 +131,7 @@ export function buildRows(
collapsedGroups,
groupDepth: 0,
sectionKey: ALL_GROUP_KEY,
hostContextLabelByWorktreeId: mixedWorktreeHostContextLabels,
hostContextLabelByWorktreeIdentity: mixedWorktreeHostContextLabels,
cyclicLineageIds
})
}
@@ -160,7 +160,7 @@ export function appendOrderedGroups(
// be registered on two hosts — then every row in the group shares a repo id
// and the per-repo label cannot tell them apart. Fall back to the per-row
// host labels, which are keyed by host-qualified identity.
const hostContextLabelByWorktreeId =
const hostContextLabelByWorktreeIdentity =
groupBy === 'repo' && hostContextLabelByRepoId ? undefined : mixedWorktreeHostContextLabels
appendWorktreeRows(result, items, repoMap, lineageById, worktreeMap, {
nestLineage,
@@ -168,7 +168,7 @@ export function appendOrderedGroups(
groupDepth: projectGroupDepth,
sectionKey: key,
hostContextLabelByRepoId,
hostContextLabelByWorktreeId,
hostContextLabelByWorktreeIdentity,
cyclicLineageIds
})
}
@@ -174,7 +174,7 @@ export function appendWorktreeRows(
groupDepth: number
sectionKey: string
hostContextLabelByRepoId?: ReadonlyMap<string, string>
hostContextLabelByWorktreeId?: ReadonlyMap<string, string>
hostContextLabelByWorktreeIdentity?: ReadonlyMap<string, string>
cyclicLineageIds: ReadonlySet<string>
}
): void {
@@ -184,7 +184,7 @@ export function appendWorktreeRows(
groupDepth,
sectionKey,
hostContextLabelByRepoId,
hostContextLabelByWorktreeId,
hostContextLabelByWorktreeIdentity,
cyclicLineageIds
} = options
if (!nestLineage) {
@@ -200,7 +200,7 @@ export function appendWorktreeRows(
lineageChildCount: 0,
lineageCollapsed: false,
hostContextLabel:
hostContextLabelByWorktreeId?.get(getWorktreeHostIdentity(worktree)) ??
hostContextLabelByWorktreeIdentity?.get(getWorktreeHostIdentity(worktree)) ??
hostContextLabelByRepoId?.get(worktree.repoId)
})
)
@@ -250,7 +250,7 @@ export function appendWorktreeRows(
lineageChildCount: children.length,
lineageCollapsed,
hostContextLabel:
hostContextLabelByWorktreeId?.get(getWorktreeHostIdentity(worktree)) ??
hostContextLabelByWorktreeIdentity?.get(getWorktreeHostIdentity(worktree)) ??
hostContextLabelByRepoId?.get(worktree.repoId)
})
)