Skip to content

Add change-level review UI for ask-before-edits#59

Merged
samhu1 merged 1 commit intosamhu1:masterfrom
anvita-g:issue-27-diff-review
Apr 5, 2026
Merged

Add change-level review UI for ask-before-edits#59
samhu1 merged 1 commit intosamhu1:masterfrom
anvita-g:issue-27-diff-review

Conversation

@anvita-g
Copy link
Copy Markdown
Contributor

@anvita-g anvita-g commented Apr 3, 2026

Summary

This resolves issue #27 by adding a structured edit diff in ask-before-edits mode before applying file changes.

  • Introduces selective review for multi-change edits via per-change toggles and Apply Selected.
  • Keeps the existing fast path with Allow All for one-click full approval.
  • Refactors shared diff logic into reusable utilities used by both tool output and permission review UI.

Why this change

  • Improves reviewability by showing exactly what will change before approval.
  • Reduces risk from all-or-nothing approvals by enabling partial acceptance of edits, especially for large refactors.
  • Preserves speed for users who prefer immediate full approval.

Testing

  • pnpm build
  • pnpm exec tsc --noEmit
  • Manual verification completed

Screenshots (if UI changes)

image image

Contributor Recognition

  • Preferred public name (optional): Anvita Gollu
  • Preferred link (optional):
  • OK to feature in CONTRIBUTORS.md: [ ] Yes

Checklist

  • Scope is focused
  • Docs updated if needed
  • No secrets or private data added

@anvita-g anvita-g requested a review from samhu1 as a code owner April 3, 2026 03:53
@samhu1 samhu1 merged commit 479bf80 into samhu1:master Apr 5, 2026
3 of 4 checks passed
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