Skip to content

web-ui: approval card with a pager - #1056

Closed
time-attack wants to merge 1 commit into
bui/03-chat-rows-headerfrom
bui/04-approval-card
Closed

web-ui: approval card with a pager#1056
time-attack wants to merge 1 commit into
bui/03-chat-rows-headerfrom
bui/04-approval-card

Conversation

@time-attack

@time-attack time-attack commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

The composer's approval panel becomes the reference card — title and reason
badge, summary and purpose, the command in a mono block, a "Show full command"
disclosure — showing one pending approval at a time with a ‹ 1/N › pager when
several are queued, ghost Deny / Allow once / Allow always pills and a primary
Allow for session. The transcript marker is the compact version. The pager
index resets with the conversation.

Demo

A real pending approval (a pipe-to-shell command): the card with its reason badge, "Triggered by" snippet and the Deny / Allow pills.

05-approval-card.gif

Verification

Typecheck, the full plugins/web-ui suite, eslint and prettier pass at this level of the stack (each level is verified on its own, on top of the previous one). Live QA of the finished stack ran in a browser-only dev-instance against a seeded conversation (tool run, markdown table, TypeScript and diff fences, a pending approval, ⌘K, list pages, settings), in light and dark and at phone width.

Stack

Merge in order; each PR is based on the one above it and retargets automatically when its base merges.

# PR Change
1 #1053 Beautiful UI foundation — tokens, fonts, styles scaffold
2 #1054 loading state and the live thinking trace
3 #1055 chat rows, session header and empty state
4 #1056 (this) approval card with a pager
5 #1057 tool chips and code blocks
6 #1058 records tables and filter lists
7 #1059 search palette
8 #1060 settings as inspector cards
9 #1061 selection actions toolbar
10 #1062 recommendation card

The composer's approval panel becomes the reference card — title and reason
badge, summary and purpose, the command in a mono block, a "Show full command"
disclosure — showing one pending approval at a time with a ‹ 1/N › pager when
several are queued, ghost Deny / Allow once / Allow always pills and a primary
Allow for session. The transcript marker is the compact version. The pager
index resets with the conversation.
@time-attack

Copy link
Copy Markdown
Collaborator Author

Closing per review: holding off on importing Beautiful UI wholesale for now (interactions read as too stimulating and too Conductor-like). A subset may come back as our own take in collaboration with Eve. Branch bui/04-approval-card is kept.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant