Skip to content

[codex] Add accept-all diff review action#160

Merged
BunsDev merged 2 commits into
mainfrom
okcode/accept-all-changes
Apr 1, 2026
Merged

[codex] Add accept-all diff review action#160
BunsDev merged 2 commits into
mainfrom
okcode/accept-all-changes

Conversation

@BunsDev

@BunsDev BunsDev commented Apr 1, 2026

Copy link
Copy Markdown
Member

Summary

  • add a header-level Accept All action to the diff review panel
  • reuse shared diff review state logic for bulk acceptance instead of duplicating UI-only behavior
  • cover the bulk acceptance path with review state tests

Why

Reviewing multi-file changes one file at a time is unnecessarily slow when the user wants to accept the whole visible diff selection.

Impact

Users can accept every visible file change in one action, and the button disables once all visible files are already accepted.

Validation

  • bun fmt
  • bun lint
  • bun typecheck

BunsDev added 2 commits March 31, 2026 23:52
- Add icon, tag chips, and improved command/path presentation
- Make dialog content and skills page scrollable within available height
@vercel

vercel Bot commented Apr 1, 2026

Copy link
Copy Markdown

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 7:32am

@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
@BunsDev BunsDev marked this pull request as ready for review April 1, 2026 07:33
@BunsDev BunsDev merged commit 6fd6396 into main Apr 1, 2026
13 of 14 checks passed
@BunsDev BunsDev deleted the okcode/accept-all-changes branch April 1, 2026 08:58
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