ci: add advisory Claude PR review workflow#3456
Merged
Merged
Conversation
a92cc7b to
6899368
Compare
Wire Claude code review into every non-draft PR as a GitHub Action, since GitHub CODEOWNERS only accepts users/teams (not bots). @Hmbown stays the human owner; the review is advisory and posts alongside it. - claude-review.yml: anthropics/claude-code-action@v1 in review mode with track_progress; auth via CLAUDE_CODE_OAUTH_TOKEN (from `claude setup-token`). No-ops with a notice until the secret is set, so it's safe to merge first. Codex/ChatGPT review is handled out-of-repo via the ChatGPT Codex cloud integration (chatgpt.com/codex), authed by the ChatGPT subscription, so no OpenAI workflow or repo secret is needed. CODEOWNERS documents the split. The review prompt also guards the v0.8.65 EPIC #2608 routing invariant (provider-prefixed model strings are wire ids/hints, not provider-selection proof). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
6899368 to
ddc33d4
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds an advisory Claude code review GitHub Action that runs on every non-draft PR, per Hunter's request to add AI reviewers.
.github/workflows/claude-review.yml—anthropics/claude-code-action@v1(review mode,track_progress), authed by theCLAUDE_CODE_OAUTH_TOKENrepo secret (already added)..github/CODEOWNERS— comment documenting that AI review is advisory and not wired through CODEOWNERS (which only accepts users/teams, not bots);@Hmbownstays the human owner.Codex / ChatGPT review — handled out-of-repo
Per Hunter's choice, Codex review runs via the ChatGPT Codex cloud integration (chatgpt.com/codex → connect GitHub → enable Code review), authed by the ChatGPT subscription — so there's no OpenAI workflow file or repo secret here. (An earlier revision of this PR included a
codex-review.ymlusingOPENAI_API_KEY; it was dropped in favor of the cloud integration to avoid duplicate Codex comments.)Why a workflow, not CODEOWNERS
CODEOWNERS can only list real GitHub users/teams — a bot like
claude[bot]can't be a code owner. So Claude review is apull_requestAction that posts review comments. It's advisory; it doesn't gate merge or replace@Hmbown.Notes
opened, synchronize, reopened, ready_for_reviewon[master, main]; drafts skipped;concurrencycancels superseded runs.deepseek-ai/,deepseek/,anthropic/are wire ids/hints, never provider-selection proof).maincurrently fails to compile (error[E0004]indoctor_api_key_source_label, fixed by fix(tui): cover all ApiKeySource variants in doctor label match (broken main) #3453). This PR only adds workflow YAML; its own CI goes green once fix(tui): cover all ApiKeySource variants in doctor label match (broken main) #3453 lands.🤖 Generated with Claude Code