fix(cli): make the session banner two lines, not a table - #385
Open
agjs wants to merge 1 commit into
Open
Conversation
The aligned label/detail table shipped in 0.53.8 reads well on a wide
pane and badly on a narrow one, which is the case that was reported. The
label column is fixed overhead: on a ~34-column main pane (a normal
terminal at a large font) it left ~20 columns for the value, so every
detail wrapped raggedly and the banner ran to ten rows of it.
Measured rather than guessed: instrumenting seedPaneLanding showed
mainInnerCols() returning 83 on a 150-column terminal — correct. The
width was never wrong; the layout was wrong for narrow widths. A table
was my invention here, and it was the wrong shape for the job.
Capabilities are a short list, so they now share ONE flowing
`·`-separated line, and the details ride in parentheses only while the
whole line fits on a row. Below that the names stand alone — the names
are what a reader scans for, the details are a nicety, and three wrapped
rows of parentheticals is exactly the wall of text this banner is meant
to avoid.
At ~34 columns:
◆ plan mode · approve to build
github · image · delegation
and at 83:
◆ plan mode · approve to build
github (git + PR review) · image (read + generate) · delegation (×4, cap 1)
Specialist names left the banner (discoverable via /agents), paths moved
to notes rather than bloating the capability list, and the headline lost
"reply " so it fits a narrow pane on one row. Wrapped rows still hang-
indent deeper than the first, so nothing snaps back to column 0.
Verified in a 60-column pty: the parentheticals are correctly dropped,
each chip appears exactly once, and the run-meta line zero times.
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
tsforge | 7e3f82c | Commit Preview URL Branch Preview URL |
Sep 07 2026, 10:18 PM |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #382. The aligned label/detail table it shipped reads well wide and badly narrow — which is the case that got reported.
What was wrong
The label column is fixed overhead. On a ~34-column main pane — a normal terminal at a large font — it left ~20 columns for the value, so every detail wrapped raggedly:
Measured, not guessed: instrumenting
seedPaneLandingshowedmainInnerCols()returning 83 on a 150-column terminal — correct. The width was never wrong; the layout was wrong for narrow panes. The table was my invention in #382 and it was the wrong shape for the job.Now
Capabilities share one flowing
·-separated line. Details ride in parentheses only while the whole line fits on a row; below that the names stand alone — the names are what you scan for, the details are a nicety, and three wrapped rows of parentheticals is the wall of text this banner exists to avoid.At ~34 columns:
At 83:
Also: specialist names left the banner (discoverable via
/agents), paths moved to notes instead of bloating the capability list, and the headline lostreplyso it fits a narrow pane on one row. Wrapped rows still hang-indent deeper than the first, so nothing snaps back to column 0.Verification
bun test packages— 5908 pass, 0 failbun run e2e:pty— 0[FAIL]