Skip to content

Clear the inline diff review automatically after a successful send - #1236

Merged
h0x91b merged 1 commit into
mainfrom
feat/dev3-auto-reset-review-after-send
Aug 3, 2026
Merged

Clear the inline diff review automatically after a successful send#1236
h0x91b merged 1 commit into
mainfrom
feat/dev3-auto-reset-review-after-send

Conversation

@h0x91b

@h0x91b h0x91b commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Feedback from Alexander Kiselyov: the review cycle was always four steps — write comments, "Send to Agent", "Reset review", confirm a destructive dialog. After a successful send the comments are already delivered, so the reset and its confirmation were pure friction.

  • A successful send now clears the inline review comments (and their persisted entry) automatically — no extra click, no confirmation dialog.
  • A failed send changes nothing: the comments survive so the reviewer can retry.
  • The manual "Reset review" button keeps its confirmation for the "changed my mind, drop the comments without sending" case.
  • The success toast now says the review was sent and cleared, so the disappearance is not a surprise (en/ru/es).

@h0x91b
h0x91b enabled auto-merge (squash) August 3, 2026 09:56
@h0x91b
h0x91b merged commit 52aabef into main Aug 3, 2026
9 checks passed
@h0x91b
h0x91b deleted the feat/dev3-auto-reset-review-after-send branch August 3, 2026 09:57
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