Skip to content

fix: don't encourage AI to resolve comments#210

Merged
tomasz-tomczyk merged 7 commits intomainfrom
fix/204
Mar 27, 2026
Merged

fix: don't encourage AI to resolve comments#210
tomasz-tomczyk merged 7 commits intomainfrom
fix/204

Conversation

@tomasz-tomczyk
Copy link
Copy Markdown
Owner

Summary

  • Remove --resolve from all AI-facing prompts in server.go — AI should reply to comments but not mark them resolved
  • Drop the orange "Unresolved" border and label on carried-forward comments; replaced with a neutral "Carried forward" label in muted text
  • Remove --resolve from all integration docs (claude-code, cursor, codex, cline, windsurf, aider, opencode, github-copilot) — --resolve flag still exists but is no longer advertised as the expected workflow

Resolving comments is now a consensual, user-driven action. AI addresses the code and replies; the human decides what counts as resolved.

Closes #204

Reproduction

N/A — prompt/UI change

Review

  • Expert code review: passed
  • Go tests: passing

Test plan

  • Review the prompt returned by POST /api/finish no longer contains --resolve
  • Carried-forward comments show "Carried forward" in muted text (no orange border)
  • All integration docs no longer include --resolve in workflow examples

🤖 Generated with Claude Code

AI should address comments and reply, but resolving is a user action.
Remove --resolve from all AI-facing prompts. Drop the orange 'Unresolved'
border and label on carried-forward comments — replace with a neutral
'Carried forward' label in muted text.

Closes #204
AI should reply to comments but not mark them resolved — resolving is a
manual human action. Updated all integration docs to remove --resolve from
workflow examples and "resolve": true from --json bulk examples. Kept the
--resolve flag in CLI reference sections (it exists as a feature) but
updated the `resolve` field description in schema tables to clarify it's
user-controlled.
@tomasz-tomczyk tomasz-tomczyk merged commit 0e5fee4 into main Mar 27, 2026
2 checks passed
@tomasz-tomczyk tomasz-tomczyk deleted the fix/204 branch March 27, 2026 22:38
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.

All Comments in Current Round of Reviews Should Be Expanded

1 participant