feat: rename the stack diff layout to unified - #945
Conversation
|
PR author is not in the allowed authors list. |
|
@saadjs is attempting to deploy a commit to the Modem Team on Vercel. A member of the Team first needs to authorize it. |
|
@saadjs Hey, question, what got you to open this PR? |
Help needed + good first issue labels 🙂 |
|
Hello, just tried to test out this PR, and it seems like its working! Here's a recording of when I got this working locally! pr945-unified-layout.mp4 |
fcc1358 to
73388dc
Compare
|
Rebased onto the current |
73388dc to
7cbee61
Compare
|
Final audit found and fixed three extension compatibility gaps before merge: pre-v21 |
|
Follow-up compatibility adjustment in |
|
Correction in |
7f346af to
c5a0944
Compare
c5a0944 to
c332fc1
Compare
Summary
Renames Hunk's single-column diff layout from
stacktounifiedacross the CLI, config, extension and OpenTUI APIs, renderer internals, tests, documentation, website copy, examples, benchmarks, and maintained media. Closes #915.Existing public integrations remain compatible through narrow deprecated aliases and event fields:
--mode stackandmode = "stack"normalize tounifiedhunk.view.layoutStackaliaseshunk.view.layoutUnifiedlayout="stack""stack"shape, while additive canonical types expose"unified"stack, with canonical v22 values available ascanonicalModeandcanonicalLayouthunk.view.layoutStackas their stablecommandId, withhunk.view.layoutUnifiedavailable ascanonicalCommandIdThis is a terminology change only. Rendering, navigation, responsive behavior, wrapping, copying, notes, and scrolling are unchanged.
Rebase Notes
mainat5ecc4ddamainand advanced the session daemon protocol to v15stackoccurrences as deprecated compatibility coverage, historical records/metrics, or unrelated call-stack, STML, CSS, and prose usesValidation
Passed on Linux with Bun 1.4.2:
bun run typecheckbun run lintbun run format:checkbun run deps:checkbun run test(4,085 passed, 12 skipped)bun run test:integration(156 passed, 1 macOS-only test skipped)bun run test:tty-smoke(10 passed)bun test ./test/review-conformance(111 passed)bun run test:session-broker-node(4 passed)bun run test:theme-contrast(7 passed)bun run check:docsbun run check:changelogbun run build:npmbun run check:packbun run website:checkbun run website:buildbun run website:linksbun run website:test:browser(101 passed, 1 desktop-only test skipped on mobile)bun run bench:render-layoutbun run install:bindiff --help,show,patch, andpagerinvocationsInstalled-artifact acceptance additionally covered full-screen
stack/unifiedparity at 88, 140, and 220 columns; legacy CLI, config, keybinding, OpenTUI, and extension inputs; exhaustive legacy and canonical packed declarations; pre-v22 extension event behavior; hunk navigation; explicit layout switching; auto-layout resizing; and current menu wording. The split and unified screenshots were refreshed from the current TUI.Platform Notes
Tested locally on Linux. CI provides the remaining platform coverage.