Repository navigation
Conversation
KieranP
force-pushed
the
unnified-diff
branch
from
November 14, 2026 10:56
f35a323 to
17dedf0
Compare
KieranP
force-pushed
the
unnified-diff
branch
2 times, most recently
from
November 14, 2026 23:52
bea756f to
d9871b5
Compare
Reviewing what an agent changed across twenty files meant opening twenty file tabs and stepping through them one by one. The All changes tab shows every file in the Changes list, in its order and scope, as one scrolling diff with review comments on its lines. It is a third tab kind rather than a file tab holding many paths, so every switch on the kind has to say what it does. Its rows are one text with a map back to each file, so find, copy, the step arrows and the rotor see rows only, and headers, folds and notices sit in row gaps, hosted only near the screen: hosting all of them for 1,802 files took the app to 1.43 GB. Unified and Split, in file tabs and this tab, now draw on one scroller over a canvas holding each column's gutter and text and the views in the gaps, so all of it scrolls natively. Split had been two scroll views locked by offset, which pulled each other back as one clamped, and every view laid over them was moved on each scroll. - Open it beside the Changes filter or with tabs open-all-changes; a Changes row click shows the tab at that file - Fold unchanged runs, collapse files, push a sticky header up as the next file's arrives, and highlight only the files near the screen - Share review cards, the composer and Send with file tabs; review add quotes only through a file tab - In Split, show a file all added or all removed full width - Count each row's height with the gap under it, and start TextKit's container under the gap above the first row, so rows on screen are laid out however many headers stack above them - Keep the scroller in a margin as wide as it grows under the pointer - Send Cmd+Up and Cmd+Down to the document's top and end - Ask git once a read for the merge base, HEAD, committed sizes, ignored files and the index, skip the diffs of untracked new files and keep a steady window of reads, so 300 files read in 0.4 s rather than 3.3 s Closes #42 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
KieranP
force-pushed
the
unnified-diff
branch
from
November 15, 2026 00:13
d9871b5 to
090bee5
Compare
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.
Closes #42
Reviewing what an agent changed across twenty files meant opening twenty file tabs and stepping through them one by one. The All changes tab shows every file in the Changes list, in its order and Modified/Branch scope, as one scrolling diff with review comments on its lines. It follows the owner's picks from the mockups on the issue.
tabs open-all-changes; a Changes row click shows the tab at that file, and the file at the top lights its rowtabs step/findand the rotor work on it; headers, folds and notices sit in row gaps, hosted only near the screenreview addreaches the tabDocs/design/files/all-changes-tab.md; status and owner feedback:PROGRESS.mdMemory, measured over 1,802 changed files plus 191 added: 191 MB against 78 MB with the tab shut (1.43 GB before hosting only what is near the screen).
Known limits, queued in
PROGRESS.md: in Split, find, the arrows and the rotor skip the rows of added and removed files, and Split is two locked scroll views, to be reworked as one; every file's rows are still read and held, capped at 200,000.Tests:
make test, 2,999 passing;swift format lint --strictandswiftlintclean.🤖 Generated with Claude Code