Skip to content

ci: use built-in Claude /code-review instead of the plugin - #14

Merged
AnnatarHe merged 1 commit into
mainfrom
cursor/claude-code-review-workflow-407c
Oct 14, 2026
Merged

AnnatarHe merged 1 commit into
mainfrom
cursor/claude-code-review-workflow-407c

Conversation

@AnnatarHe

@AnnatarHe AnnatarHe commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Updates .github/workflows/claude-code-review.yml so Claude review matches the working rawback-app/web workflow. The plugin command was posting "No issues found" after a few seconds. runs-on stays ubuntu-latest. No other files changed. CLAUDE_CODE_DISABLE_BACKGROUND_TASKS was not set, so nothing was removed for it.

This PR changes the workflow file, so the action may skip its own review until the file matches the default branch. That skip is expected.

What changed

  • Prompt is the built-in /code-review high <PR#> --comment. Removed plugin_marketplaces and plugins: code-review@claude-code-plugins, and the old /code-review:code-review prompt.
  • Checkout uses actions/checkout@v7 with fetch-depth: 0 so the review can diff, log, and blame against the base branch.
  • Permissions: pull-requests and issues are write. Added checks: read, statuses: read, and actions: read, plus additional_permissions for actions: read.
  • show_full_output: true so the job log shows why a run stopped or skimmed. The comment notes this repository is public, so those logs are visible.
  • claude_args allows Bash(gh:*), Bash(git:*), and mcp__github_inline_comment__create_inline_comment.
  • Concurrency group claude-review-<PR number> with cancel-in-progress: true, so a new push replaces a review that is still running.
  • Job if skips draft PRs and bot PRs (including release-please).
  • New step Assert the review actually ran fails a successful action that spent 0 turns and $0, and warns when a review launched no subagents. A missing execution file only warns, which is the expected case when this workflow differs from the default branch.
  • Comment that cited another repository's pull request was rewritten to describe the same failure mode without a foreign PR number.
Open in Web Open in Cursor 

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

The plugin review was posting "No issues found" after a few seconds.
Switch to /code-review high, full history, write permissions, and an
assertion that the review actually ran.

Co-authored-by: Le He <Annatar.He+github@gmail.com>
@AnnatarHe
AnnatarHe marked this pull request as ready for review October 14, 2026 09:56
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@AnnatarHe
AnnatarHe merged commit 73adb5a into main Oct 14, 2026
4 checks passed
@AnnatarHe
AnnatarHe deleted the cursor/claude-code-review-workflow-407c branch October 14, 2026 09:56
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.

2 participants