Skip to content

docs: fix review resolution states, add human-side interactions#768

Merged
selfcontained merged 1 commit into
mainfrom
agt_8dc23326528d/job-doc-ter-342206fc
Jul 14, 2026
Merged

docs: fix review resolution states, add human-side interactions#768
selfcontained merged 1 commit into
mainfrom
agt_8dc23326528d/job-doc-ter-342206fc

Conversation

@selfcontained

Copy link
Copy Markdown
Owner

Summary

  • tools.tsx: dispatch_review_resolve resolution options updated from fixed/ignored/wont_fix to fixed/dismissed (matching PR Fix extractHunkAroundLines bug for top-of-file hunks, add 19 tests #767's DB migration and MCP handler changes)
  • agents.tsx: Documented that humans can now reply to threads, resolve items, and reopen feedback directly from the UI — both inline in the diff and from the Reviews sidebar tab
  • tips.ts: Updated diff-review tip to reflect bidirectional interaction ("Both you and the agent can reply, resolve, or reopen items")

Deferred to next run

  • Full Repo Tools deep-dive (tool handler descriptions vs docs summaries) — the tool list itself is accurate; this run focused on the resolution state drift from the recent diff
  • dispatch_review_add_message body max reduced from 50,000 to 1,200 chars — the docs description doesn't mention the limit, and the tool's own schema describes it to agents directly

Test plan

  • pnpm run format:write — clean
  • pnpm run check — types pass
  • No runtime behavior changes — docs-only PR

🤖 Generated with Claude Code

…ions

The review feedback resolution simplified from three states (fixed/ignored/wont_fix)
to two (fixed/dismissed). Updated tools.tsx, agents.tsx, and the diff-review tip.
Also documented that humans can now reply, resolve, and reopen review feedback items
directly from the UI (inline in the diff and from the Reviews sidebar tab).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@selfcontained selfcontained merged commit 54769ca into main Jul 14, 2026
1 check passed
@selfcontained selfcontained deleted the agt_8dc23326528d/job-doc-ter-342206fc branch July 14, 2026 04:13
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