cmux-next: fix the two Chief title reds from #23391
Merged
TE
16d ago+6+6+0123456789-3-3-0123456789
Fixes two base reds that came in with #23391 (reported by chief-sidebar):
ChiefWorkspaceRowsTests.theTaskTitleDropsOnlyASubagentID:ChiefWorkspaceName.taskTitletrimmed blanks on both ends before looking for theaN ·id, so"a3 · "became"a3 ·", the" · "match failed, and it returned"a3 ·"instead of nil. It now drops only the leading blanks before the match; the trailing ones still go after.ChiefSidebarItemTests.dedupedSessionNamesKeepTheirRoleAndNumber: the test still expected the internala3/a10ids as row titles. #23391 hides those ids on purpose (Leo's ask), and an untitled lane reads "Subagent". The test now gives the lanes task titles and checks that they keep their subagent-number order.
Example failing run: https://github.com/manaflow-ai/cmux/actions/runs/39635743092/job/119347539209
No capture: a string helper and a test, with no visible change beyond the "a3 · " edge case.
Agent session: cosmic-gray
🤖 Generated with Claude Code
![View with [code]smith](https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v3-3x.png)
![Autofix with [code]smith](https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark-3x.png)
Need help on this PR? Tag @codesmith-bot with what you need.
Summary by cubic
Fixes two Chief title test failures that came in with #23391.
ChiefWorkspaceName.taskTitlenow drops only leading whitespace before matching the subagent id, so"a3 · "keeps its" · "marker and no longer returns"a3 ·"instead of nil.- The deduped-session test now gives lanes task titles and checks their subagent-number order, instead of expecting the internal
a3/a10ids as row titles.