Repository navigation
Keep repositories visible on Inbox rows without an issue - #1823
Merged
Merged
Conversation
This was referenced Nov 10, 2026
Closed
# Conflicts: # src/console/assets/console.css # src/console/row.html
Rows without an issue now show the repository before the Command (#1821), so the subject's text no longer reads '<Command> on <repository>'. The subject's title attribute still does.
This was referenced Nov 10, 2026
Merged
Merged
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
Inbox rows without an issue or pull request show their repository above the Command name. Security runs, Triage runs and Custom processes can be identified without opening Command details. Long repositories wrap on desktop and phone.
Closes #1821
Evidence
Execution evidence
Test seam: the rendered Console Inbox, served over HTTP from fixture Command records and read in Chromium.
PLAYWRIGHT_BROWSERS_PATH=/home/jstephens/.cache/ms-playwright node --test --test-name-pattern='Inbox shows the repository' console/tests/inbox.test.cjsfailed all 12 viewport/Command/repository cases:repository must be readable in the row. In the example Security row, the subject ended at x=569 but its repository extended to x=730.After: the same command passed:
tests 13,pass 13,fail 0(12 cases plus the parent test).PLAYWRIGHT_BROWSERS_PATH=/home/jstephens/.cache/ms-playwright node --test --test-name-pattern='Inbox shows a full Command name' console/tests/inbox.test.cjsfailed before withWithout JavaScript the full Command name must be readable: {"width":341,"fullWidth":525}and passed after:tests 1,pass 1,fail 0. The reviewer's original standalone command also failed before and passed after (width:341,fullWidth:341,whiteSpace:"normal",overflow:"visible"). The test is retained inconsole/tests/inbox.test.cjs.PLAYWRIGHT_BROWSERS_PATH=/home/jstephens/.cache/ms-playwright node --test --test-name-pattern='Inbox shows|Inbox keeps issue numbers' console/tests/inbox.test.cjs— 19 passed, 0 failed.PLAYWRIGHT_BROWSERS_PATH=/home/jstephens/.cache/ms-playwright scripts/affected-tests origin/main— build passed; all 153 Console browser tests passed in 132.03s. The Rust run (367.77s, including 148s compilation) passed 1,654 unit tests, then stopped atbug_originsbecause the host's default Python 3.9 lackstomllib. Re-running that exact failing test with Python 3.11 on a private PATH passed:PATH=/tmp/issue-1821-tools.etObqR:$PATH cargo test --test bug_origins settlement::historical_review_and_reclosed_origin_regressions(1 passed, 0 failed). Full Rust CI for the published head is in progress.origin/mainis the fetchedmaintip,a7d76bbb; the localmainis stale and checked out elsewhere. Both independent reviews used an isolated checkout withmainpinned to that fetched tip andgit diff main...HEAD.Visual evidence
1440×900 desktop, Needs you with Command details closed:
390×844 phone, the same folder and state:
Review coverage
Standards raised 2 findings: 1 fixed and 1 declined below. Spec raised 0. Each reviewer read all 4 changed files:
console/README.md,console/tests/inbox.test.cjs,src/console/assets/console.css,src/console/row.html. Unread changed files: none. Generated-output exclusions: none.Unaddressed findings
Standards
src/console/row.html:2: declined the suggestion to introduce structured subject fields solely for this layout. The sole unlinked-subject formatter insrc/logs/status/inbox.rs:394–418guarantees<Command> on <repository>; splitting its finalonalso handles Command labels containing that text. The template already follows this presentation-splitting pattern for linked#subjects. This is a documented implementation tradeoff, with no demonstrated behavioral failure or decision deferred to the Spec or Day shift.Spec
None.
Merge Danger
Door: two-way
Reverting restores the previous row rendering; no stored records change.
Blast Radius: Console
Changes the repository and Command-name layout on Inbox and History rows without a linked issue or pull request. Linked subjects retain their existing rendering.
Built with claude 2.1.322 · claude-opus-5-5 · medium