Skip to content

Commit 81666c1

Browse files
committed
docs(tui): refresh stale comments for the input-required strip
The collapse-order comment still called worker_wait the last optional cut after input_required followed it, and the chrome compose-key comment claimed an exhaustive input list that omitted the strip text it now composes.
1 parent 9ad0032 commit 81666c1

2 files changed

Lines changed: 25 additions & 3 deletions

File tree

‎src/tui/geometry/zones.ts‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -213,8 +213,9 @@ export const COLLAPSE_ORDER = [
213213
// banner and strip.
214214
"pending",
215215
// A parked worker is standing state the operator cannot see anywhere else
216-
// once the wake turn scrolls away, so its single row is the last optional
217-
// cut, just ahead of prompt growth reclaim.
216+
// once the wake turn scrolls away, so its single row is one of the last
217+
// optional cuts, just ahead of prompt growth reclaim. The input-required
218+
// strip below it is newer chrome, so it is the very last optional cut.
218219
"worker_wait",
219220
"input_required",
220221
// prompt growth reclaimed next (handled specially; never below PROMPT_BASE_ROWS)

‎src/tui/shell/chrome.ts‎

Lines changed: 22 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -281,7 +281,28 @@ function operatorInputRequiredChunks(
281281
}
282282

283283
/**
284-
* Key over every input a chrome paint reads; a miss leaves stale chrome.
284+
* Every input the chrome compose paths read, as one comparable key. A missed
285+
* input here means stale chrome, so this list is exhaustive:
286+
*
287+
* - notice row: the composed `noticeText` output (folds in the in-flight
288+
* tool and its start time, `lockupNowMs` as the waiting-on clock, the
289+
* interrupt flash, transcript pin state, the status flash, and pending
290+
* attachment count)
291+
* - pending column: the session queue's items as composed row text, plus the
292+
* selected item id - nav repaints the highlight without touching row text
293+
* - border geometry: `layout.contentWidth`
294+
* - top rule: MCP-needs-auth presence, plugin-needs-attention, `modelLabel`
295+
* - bottom rule: workspace cwd and branch, `homedir()` (label compression)
296+
* - both rules' lockup slot: `lockupNowMs`, `lockupAnimating`, `lockupPhase`,
297+
* `lockupChangedMs`, `lockupRampPhase`, `lockupStalledForMs`
298+
* - cost meter: band, percent label, cost label (or absence)
299+
* - landing suggestions: whether the prompt has text
300+
* - WORKER WAITING strip: its composed text at the current width (folds in
301+
* the selected identity, its preview, and the additional count)
302+
* - INPUT REQUIRED strip: its composed text at the current width (folds in
303+
* the selected question preview and the additional count)
304+
*
305+
* Landing and zone paints read their own state and do not pass through here.
285306
*/
286307
function chromeComposeKey(
287308
shell: AppShell,

0 commit comments

Comments
 (0)