Repository navigation
feat(tui): show accepted-but-unread queued input - #1395
Merged
TheGreatAxios merged 2 commits intoOct 13, 2026
Merged
TheGreatAxios merged 2 commits into
TheGreatAxios merged 2 commits into
Conversation
bwachman
changed the base branch from
cl-10200-make-parked-worker-ask_director-questions-visible-and
to
cl-10189-show-required-input-for-primary-session-ask_operator-at-the
October 11, 2026 18:39
bwachman
force-pushed
the
cl-10437-show-accepted-but-unread-queued-input-while-the-dispatcher
branch
from
October 11, 2026 18:39
16660d8 to
3899e2e
Compare
bwachman
force-pushed
the
cl-10437-show-accepted-but-unread-queued-input-while-the-dispatcher
branch
from
October 11, 2026 20:50
3899e2e to
7e4fdd5
Compare
bwachman
changed the base branch from
cl-10189-show-required-input-for-primary-session-ask_operator-at-the
to
cl-10432-codify-outcome-aware-escalation-policy-for-blocked-worker
October 11, 2026 20:50
TheGreatAxios
requested changes
Oct 12, 2026
TheGreatAxios
left a comment
Collaborator
There was a problem hiding this comment.
Ledger lifecycle verified against the reactor — accepted-result entry, message.run.started read signal, wake-origin exclusion, and lifecycle clears all confirmed (206/0 focused suite). One item: the body is stale — base is cl-10432 (not cl-10200), the PR is one squashed commit (body says 5), focused suite reports 206 (body says 203). Refresh the body.
bwachman
force-pushed
the
cl-10432-codify-outcome-aware-escalation-policy-for-blocked-worker
branch
from
October 13, 2026 00:50
5c7743b to
fd6416e
Compare
bwachman
force-pushed
the
cl-10437-show-accepted-but-unread-queued-input-while-the-dispatcher
branch
from
October 13, 2026 00:50
7e4fdd5 to
9cfed0e
Compare
Contributor
Author
|
The PR body is refreshed: base is now cl-10432 (was cl-10200), the PR is one commit (was 5), and the focused suite count is 206 pass / 0 fail (was 203), re-run at this head. |
TheGreatAxios
force-pushed
the
cl-10432-codify-outcome-aware-escalation-policy-for-blocked-worker
branch
from
October 13, 2026 01:42
fd6416e to
2c73ecd
Compare
TheGreatAxios
force-pushed
the
cl-10437-show-accepted-but-unread-queued-input-while-the-dispatcher
branch
from
October 13, 2026 01:51
9cfed0e to
864a948
Compare
TheGreatAxios
changed the base branch from
cl-10432-codify-outcome-aware-escalation-policy-for-blocked-worker
to
main
October 13, 2026 01:51
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
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.
Note on base
This branch is stacked on the
cl-10432branch, which sits on top of the stack that provides the worker-wait / parked-question machinery. The diff is the CL-10437 change only: the queued-input strip on top of cl-10432. The base must staycl-10432-codify-outcome-aware-escalation-policy-for-blocked-workerso the PR shows only the single CL-10437 commit.Summary
Shows accepted-but-unread queued operator input in the TUI so the operator can see that the dispatcher has accepted input that has not been read yet. When a queued delivery is accepted (an accepted final result arrives), the ledger entry appears as a QUEUED INPUT row and stays visible until the director reads the message, which is signaled by the existing upstream
message.run.startedevent (Option A read signal). Worker wakes never enter the ledger. The strip occupies a one-row geometry zone, coexists with the WORKER WAITING zone, and collapses and restores cleanly.Verification
After the rebase onto main and the review fixes, CI passes on 9cfed0e; the results below are from the original verification.
bun test src/tui/queued-input.test.ts src/tui/runtime-bridge.test.ts src/tui/queued-delivery-hop.test.ts src/tui/queued-input-chrome.test.ts src/tui/worker-wait-chrome.test.ts src/tui/geometry.test.ts src/tui/geometry/resolve.test.ts src/tui/stream-event-map.test.ts)message.run.startedevent (notmessage.received), so the row paints until the director reads the messageAC to code and test mapping
src/tui/queued-input.ts+queued-input-chrome.test.tsmessage.run.started(Option A read signal):src/tui/runtime-bridge.ts+stream-event-map.ts+queued-input-chrome.test.tsmessageIdidentity/dedup, FIFO, out-of-order consume; no content fallback:src/tui/queued-input.tssrc/tui/runner/submit.ts+delivery-queue.ts+queued-input-chrome.test.tsclearQueuedDelivery/dispose/teardown), stale accepts inert:src/tui/runtime-bridge.ts+ testssrc/tui/geometry/zones.ts+geometry/resolve.ts+geometry/resolve.test.tsdocs/TUI.mdupdated for the strip and stacking:docs/TUI.mdDeferred / Notes
message.receivedcorrelation confirmed in probe (deferred; the read signal usesmessage.run.started).Refs CL-10437