Skip to content

Restore patch-only diff review state#151

Merged
BunsDev merged 2 commits into
mainfrom
okcode/revert-diff-viewer-toggle
Apr 1, 2026
Merged

Restore patch-only diff review state#151
BunsDev merged 2 commits into
mainfrom
okcode/revert-diff-viewer-toggle

Conversation

@BunsDev
Copy link
Copy Markdown
Member

@BunsDev BunsDev commented Apr 1, 2026

Summary

  • Removes the per-file patch/full context toggle from the diff panel.
  • Simplifies diff review state to track only accepted and collapsed.
  • Drops the full-context loading and fallback UI, always rendering the existing file diff.
  • Updates unit tests to match the reduced review state shape.

Testing

  • Not run (changes reviewed from diff only).
  • Existing test coverage updated in apps/web/src/lib/diffFileReviewState.test.ts to reflect the new state model.

- Drop stored patch/full context mode from diff review state
- Simplify diff panel rendering to always use patch context
- Update review-state tests for the slimmer state shape
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 1, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Apr 1, 2026 5:03am

@github-actions github-actions Bot added size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Apr 1, 2026
- Add the diff viewer package to the monorepo
- Refresh lockfile to reflect the new dependency set
@BunsDev BunsDev merged commit e7b3924 into main Apr 1, 2026
8 of 9 checks passed
@BunsDev BunsDev deleted the okcode/revert-diff-viewer-toggle branch April 1, 2026 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant