Skip to content

CI: add missing --comment flag to claude-code-review.yml - #27

Merged
jnasbyupgrade merged 1 commit into
masterfrom
fix-claude-review-comment-flag
Jul 31, 2026
Merged

CI: add missing --comment flag to claude-code-review.yml#27
jnasbyupgrade merged 1 commit into
masterfrom
fix-claude-review-comment-flag

Conversation

@jnasbyupgrade

@jnasbyupgradejnasbyupgrade commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

The /code-review plugin only prints its findings to the job log by default; it needs --comment to actually post a PR comment. claude-code-review.yml's prompt was missing that flag, so every automated review ran (real API cost, correct analysis) but never posted anything visible on the PR.

Confirmed by capturing the hidden SDK transcript on a canary PR with a deliberately injected bug: the review agent found the bug correctly, but its own final line read "No --comment argument was provided, so no GitHub comments were posted."

Test plan

  • This change only affects a GitHub Actions workflow file.
  • Verified --comment fixes the missing-comment behavior against a live canary PR.

Same issue as pgxntool-test#61/pgxntool#84: the /code-review plugin
only prints its findings to the job log by default; it needs --comment
to post a PR comment. Every automated review here has run correctly
but never posted anything visible on the PR.
@coderabbitai

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 266bbc96-f0d5-489e-8f27-ce4195d82cf0

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jnasbyupgrade
jnasbyupgrade merged commit fc358ab into masterJul 31, 2026
18 checks passed
@jnasbyupgrade
jnasbyupgrade deleted the fix-claude-review-comment-flag branch July 31, 2026 19:37
@jnasbyupgradejnasbyupgrade changed the title claude-code-review.yml: add missing --comment flagCI: add missing --comment flag to claude-code-review.ymlAug 8, 2026
Sign up for freeto 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

@jnasbyupgrade