Skip to content

Make the review bot post inline comments - #177

Merged
wimaha merged 1 commit into
mainfrom
fix/claude-review-inline-comments
Oct 19, 2026
Merged

wimaha merged 1 commit into
mainfrom
fix/claude-review-inline-comments

Conversation

@wimaha

@wimaha wimaha commented Oct 19, 2026

Copy link
Copy Markdown
Owner

Problem

The Claude review bot never opened inline threads. The run logs show the github_inline_comment MCP server is set up correctly and the tool is allowed, but the bot never called it ("No buffered inline comments", 0 review comments on #174–#176). The prompt only mentioned inline comments at the very end, as an optional extra after "post ONE top-level comment".

Change

The output section now has two ordered steps:

  1. Inline comments first (confirmed: true) for the at most 3 most important to-dos that point to a changed line.
  2. Then the single top-level summary comment.

Note

The workflow uses pull_request_target, so it runs from main. The change takes effect only after this PR is merged; this PR's own review still uses the old prompt.

🤖 Generated with Claude Code

The inline comment server was set up, but the prompt only mentioned the
tool at the very end as optional, so the bot never called it. It is now
a required first step for to-dos that point to a changed line.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

✅ Ready to merge

This only changes the text of the review bot prompt in the CI workflow. The proxy code, API, config and Docker image are not touched.

Impact on existing users: None – the change only affects the review bot, not the released image.

To-do for the author

  • Nothing left to do

Test before merging: Open a small test PR after merging and check that the bot now posts inline comments.

Technical details
  • .github/workflows/claude-review.yml:90-98: output order is now inline comments first, then one top-level comment. The old inline paragraph is removed, so there are no conflicting instructions.
  • Because of pull_request_target, the new prompt is only used after merge. I could not verify that the bot will really call the inline tool now.

@wimaha
wimaha merged commit c00bdc0 into main Oct 19, 2026
9 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.

1 participant