Skip to content

feat: add diff-so-fancy-style diff readability - #36

Merged
HartBrook merged 4 commits into
mainfrom
codex/wingdiff/integrate-diff-so-fancy
Nov 2, 2026
Merged

HartBrook merged 4 commits into
mainfrom
codex/wingdiff/integrate-diff-so-fancy

Conversation

@pike00

@pike00 pike00 commented Nov 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add exact intraline emphasis for paired deletion/addition lines.
  • Replace raw unified-diff hunk headers with readable @ path:line @ context headers.
  • Preserve syntax highlighting, Unicode code points, line selection, fingerprints, and review anchors.
  • Normalize the demo fixture and add Playwright coverage for emphasis, headers, and line selection.
  • Add a forced-colors underline fallback for accessibility.

Verification

  • npm test: 151 server tests and 66 web tests passed.
  • npm run typecheck: passed.
  • npm run build: passed.
  • npm run test:e2e: 8 tests passed.
  • Independent task and whole-branch reviews approved.

Notes

The diff-so-fancy terminal renderer is not bundled or invoked. Wingdiff keeps Git's structured diff authoritative and applies the presentation improvements natively in React.

@pike00
pike00 requested a review from HartBrook as a code owner November 2, 2026 03:32

@HartBrook HartBrook left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@HartBrook
HartBrook merged commit 0c507c4 into main Nov 2, 2026
9 checks passed
@HartBrook
HartBrook deleted the codex/wingdiff/integrate-diff-so-fancy branch November 2, 2026 14:06
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.

2 participants