Skip to content

ci: pass --comment so claude-review posts findings to PRs#535

Merged
danielchalef merged 1 commit into
mainfrom
fix/claude-review-post-comments
Jun 18, 2026
Merged

ci: pass --comment so claude-review posts findings to PRs#535
danielchalef merged 1 commit into
mainfrom
fix/claude-review-post-comments

Conversation

@danielchalef

Copy link
Copy Markdown
Member

Problem

The Claude Code Review workflow invokes the code-review plugin without --comment:

prompt: '/code-review:code-review .../pull/<n>'

Per the plugin spec, "If --comment was NOT provided, stop here. Do not post any GitHub comments." So the review runs (and spends tokens, ~$2/PR) but posts nothing — confirmed on #522#527/#529: runs complete success with write perms (post-#531) yet log "No buffered inline comments" and leave zero comments.

This was the actual blocker; #531's read→write permission bump was necessary (the plugin needs write to post) but not sufficient on its own.

Fix

Append --comment to the prompt. Keeps #531's pull-requests: write / issues: write.

Verify (after merge)

Trigger a fresh synchronize on a same-repo PR (push a commit / rebase — a re-run replays the old prompt), then check for a claude[bot] review comment via gh api repos/getzep/zep/pulls/<n>/comments. (Fork PRs like #511 still won't post — pull_request runs from forks get no CLAUDE_CODE_OAUTH_TOKEN; that needs a separate trigger.)

🤖 Generated with Claude Code

The code-review plugin only posts findings when invoked with `--comment`;
without it the action reviews the PR (spending tokens) but prints to the
hidden terminal and posts nothing. Confirmed across PRs #522#527/#529:
runs complete `success` with write perms (post-#531) but log "No buffered
inline comments" and leave zero comments. Per the plugin spec ("If --comment
was NOT provided, stop here. Do not post any GitHub comments.").

Append --comment to the prompt; #531's pull-requests/issues write perms are
required for it to post and stay in place.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@danielchalef danielchalef merged commit dd565c1 into main Jun 18, 2026
4 of 5 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