Skip to content

ci: add Claude Code PR reviewer (comment-only)#14

Closed
Mlaz-code wants to merge 1 commit into
mainfrom
ci/claude-pr-reviewer
Closed

ci: add Claude Code PR reviewer (comment-only)#14
Mlaz-code wants to merge 1 commit into
mainfrom
ci/claude-pr-reviewer

Conversation

@Mlaz-code

Copy link
Copy Markdown
Contributor

Adds an automated Claude Code PR reviewer.

What: .github/workflows/claude-review.yml + REVIEW.md. On every PR it runs anthropics/claude-code-action@v1 and posts inline comments (severity-tagged) plus a summary comment. Comment-only / non-blocking.

Guards: skips drafts, fork PRs (no secret access), dependabot, and the skip-claude-review label. One run per PR (concurrency, cancel-in-progress). Coexists with existing CI + the Paperclip Reviewer gate.

Prereq: a CLAUDE_CODE_OAUTH_TOKEN secret (from claude setup-token) must be added to the org/repo — until then this job fails at the auth step (harmless, non-blocking).

Runner: ubuntu-latest. Model: claude-sonnet-4-6. May run tsc/eslint to verify.

Type: ci

🤖 Generated with Claude Code

Adds .github/workflows/claude-review.yml + REVIEW.md. Auto-reviews PRs via
anthropics/claude-code-action@v1 using the CLAUDE_CODE_OAUTH_TOKEN secret.
Comment-only (inline + summary), non-blocking; skips drafts/forks/dependabot
and the skip-claude-review label.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Mlaz-code

Copy link
Copy Markdown
Contributor Author

Closing — we're not running the Claude Code reviewer on the Sharp-API org repos. They build on GitHub-hosted runners that CCM can't push credentials to, and we decided against standing up self-hosted runners for this org. The reviewer stays on the 3 backend repos (api-adapters, sharp-api-go, sharpapi-site).

@Mlaz-code Mlaz-code closed this Jun 5, 2026
@Mlaz-code Mlaz-code deleted the ci/claude-pr-reviewer branch June 5, 2026 20:57
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