Skip to content

feat: add explicit comment resolution - #4

Merged
quantk merged 1 commit into
mainfrom
agent/comment-resolution
Jul 28, 2026
Merged

feat: add explicit comment resolution#4
quantk merged 1 commit into
mainfrom
agent/comment-resolution

Conversation

@quantk

@quantk quantk commented Jul 28, 2026

Copy link
Copy Markdown
Owner

What changed

  • add explicit open/resolved state for saved human comments
  • keep F7 limited to open comments without changing their state
  • keep detached open comments visible after agent edits
  • place detached comments beneath a unique exact former text match, including deleted lines
  • make detached cards navigable with j/k and actionable with e, x, and d d
  • document the workflow in README and agent-guide.md

Why

F7 inserts an unsubmitted draft, so treating insertion as delivery would lose feedback if the user never sends it. At the same time, old comments must not be resent forever. Explicit resolution lets the reviewer decide when feedback is complete while retaining comment history.

Validation

  • npm run version:check
  • npm test (42 Node tests, 12 Bun UI tests)
  • npm run check

@quantk
quantk marked this pull request as ready for review July 28, 2026 15:02
@quantk
quantk merged commit 34eea81 into main Jul 28, 2026
7 checks passed
@quantk
quantk deleted the agent/comment-resolution branch July 28, 2026 15:03
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