Skip to content

Improve file and diff inspection - #28

Merged
powerfooI merged 1 commit into
mainfrom
feature/file-diff-viewer-improvements
Aug 20, 2026
Merged

Improve file and diff inspection#28
powerfooI merged 1 commit into
mainfrom
feature/file-diff-viewer-improvements

Conversation

@powerfooI

@powerfooI powerfooI commented Aug 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • share one Highlight.js language-resolution and token-coloring pipeline between File Preview and Diff Viewer
  • automatically collapse generated, truncated, and 1,000+ changed-line diffs while keeping manual expansion available and skipping collapsed rendering/highlighting work
  • detect .gitattributes linguist-generated markers in diff summaries and prefer an ordinary file for the initial Diff selection

Verification

  • bun run format:check
  • bun run lint
  • cd web && bun run typecheck
  • cd server && bun run typecheck
  • cd web && bun run build
  • TMPDIR=/tmp bun test server/src/workspace/git-diff.test.ts web/src/components/diffAutoCollapse.test.ts web/src/components/syntaxHighlighting.test.ts (12 pass)
  • full suite excluding server/src/worktree/remove.test.ts (576 pass, 1 skip)

Known test environment issue

TMPDIR=/tmp bun test server/src/worktree/remove.test.ts has one unrelated failure: stops a stable process whose cwd is inside the checkout times out while cleaning up its spawned process. The test file is unchanged from main; its other 20 tests pass.

Screenshots

Not included: these changes affect syntax rendering and automatic behavior inside workspace-backed Files/Diff views, and no Herdr workspace fixture is available in the submission environment.

@powerfooI
powerfooI force-pushed the feature/file-diff-viewer-improvements branch from 8957b3c to f309665 Compare August 20, 2026 16:59
@powerfooI
powerfooI merged commit d5ee54d into main Aug 20, 2026
1 check passed
@powerfooI
powerfooI deleted the feature/file-diff-viewer-improvements branch August 20, 2026 17:05
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