Fix permission and operator overlays not showing their choices on the landing screen - #344
Merged
TheGreatAxios merged 1 commit intoAug 7, 2026
Conversation
The gate can fire before any transcript row exists, while the landing composition still owns the screen. The float only asked the landing split for enough headroom to fit one choice row, so a request with more choices than that got its list cut down to whatever the even top/bottom split happened to leave above the prompt box — the title and body painted, most or all of the choices did not, leaving the operator with no visible way to answer.
TheGreatAxios
force-pushed
the
cl-5560-approval-and-human-request-ui-boxes-do-not-appear
branch
from
August 7, 2026 04:38
a3361c5 to
3734a57
Compare
TheGreatAxios
deleted the
cl-5560-approval-and-human-request-ui-boxes-do-not-appear
branch
August 7, 2026 06:36
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.
Summary
Verification
bun run typecheck,bun run build, andbun testall pass (one pre-existing, unrelated flake inmarkdown-rows.test.tsaround fenced-code timing, confirmed present onorigin/mainbefore this change)createAppShell+wireGates, real render passes, captured frames) with a realistic permission request (arun_shellcall with one scope) at 80x24 and 120x40: before the fix, "Always allow npm install" — the one choice the operator actually needs to grant durable trust — was clipped off screen on the very first turn; after the fix, all three choices ("Reject", "Accept once", "Always allow npm install") renderCloses CL-5560