Repository navigation
feat: simplify review navigation; capture-only shell policy - #112
Merged
Merged
Conversation
- Remove stream inbox and header "N new" button - Header search opens review picker (Cmd/Ctrl+K) - Repo rail mark and list name open latest stream; chevron toggles - Header repo/branch switchers are filterable comboboxes (new ui/combobox primitive) - Bootstrap declares local input (checkouts|capture); capture shows one diff, no navigation - Drop piped-specific UI outside capture mode
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.
StreamInbox.tsx, inbox state inuse-stream-nav.ts, filter helpers inproject-nav.ts, CSS) and header "N new" button: four overlapping ways to switch reviews collapse to three (Reviews column, header comboboxes, header search). Unread marks on rail/list remain the arrival signal.SearchReviewsButton, Cmd/Ctrl+K) and opens the existing picker.latestStream); the chevron alone toggles. Newcomponents/disclosure-row.tsxprimitive holds the two-target row. Closed-group state now saves synchronously, since opening a stream remounts the column and dropped the save effect.components/ui/combobox.tsx(Base UI combobox, menu tokens shared viamenuPopupClassName/menuItemClassName). Branch filter (streamMatches) searches branch, worktree, host and sessions. Gallery page and goldens added.Bootstrapgains requiredinput: checkouts | capture(OpenAPI, generated client,readBootstrap,Capabilities.Input()). Webshell-policy.tsderivesreviewNavigation; a capture (piped stdin,dev --fixture) shows only its diff and file sidebar: no column, search, Cmd+K, picker, comboboxes or sync.withReviewNavigation); new pipe-mode test; local-workspace specs rewritten for new behaviors. Docs updated (DESIGN.md, docs/system.md, docs/development.md, PRODUCT.md, docs-website).Validation:
mise run checkpasses (76 unit, Go, 117 Playwright incl. gallery).