Skip to content

fix(ci): use built-in Claude /code-review for PR reviews - #13

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

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

Conversation

@AnnatarHe

@AnnatarHe AnnatarHe commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

The Claude review workflow was finishing in a few seconds and posting “No issues found.” This switches .github/workflows/claude-code-review.yml to the built-in high-effort /code-review flow. Only that file changed. The runner stays ubuntu-latest.

This PR’s own review run may be skipped, because the workflow file differs from the default branch. That is expected.

What changed

  • Prompt. Replaced the custom review prompt with /code-review high <PR#> --comment (the built-in command, not the code-review@claude-code-plugins plugin).
  • Checkout. actions/checkout is now 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: actions: read.
  • Tools. claude_args allows Bash(gh:*), Bash(git:*), and mcp__github_inline_comment__create_inline_comment.
  • Logs. show_full_output: true, so a run that stops early still leaves a reason in the job log. The comment notes that this repository is public and those logs are visible with the run.
  • Concurrency. Group claude-review-<PR#> with cancel-in-progress: true, so a new push replaces a review of a stale diff.
  • Triggers. Also runs on ready_for_review and reopened.
  • Skip. Drafts and bot PRs are skipped. Drafts are reviewed once marked ready.
  • Assertion. New “Assert the review actually ran” step fails a successful action that spent 0 turns and $0, and warns when a review launches no subagents.
  • Not set. CLAUDE_CODE_DISABLE_BACKGROUND_TASKS was not in this workflow, so nothing was removed.
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 custom prompt was finishing in a few seconds with "No issues found".
Switch to the built-in high-effort review, full git history, write
permissions, and an assertion that the review actually did work.

Co-authored-by: Le He <Annatar.He+github@gmail.com>
@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 451e0d9 into main Oct 14, 2026
2 of 7 checks passed
@AnnatarHe
AnnatarHe deleted the cursor/claude-code-review-workflow-a296 branch October 14, 2026 09:55
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