Skip to content

Add Claude Code review reusable workflow - #1

Merged
xdanger merged 4 commits into
mainfrom
xdanger-claude/sleepy-carson
Mar 28, 2026
Merged

xdanger merged 4 commits into
mainfrom
xdanger-claude/sleepy-carson

Conversation

@xdanger

@xdanger xdanger commented Mar 28, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Add a reusable Claude Code review workflow (workflows/claude-code-review.yml) that reviews non-draft PRs for code quality, correctness, and security via anthropics/claude-code-action@v1
  • Rewrite README with usage guide, secrets setup, and customization instructions

Test plan

  • Verify the workflow is callable from another repo via uses: taptap/.github/.github/workflows/claude-code-review.yml@main
  • Confirm ANTHROPIC_API_KEY org secret is set
  • Open a test PR in a downstream repo to trigger the review

🤖 Generated with Claude Code

xdanger and others added 2 commits March 28, 2026 15:48
- Add claude-code-review.yml for automated PR code review
- Runs on non-draft PRs via `anthropics/claude-code-action@v1`
- Supports `workflow_call` for cross-repo reuse

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Document how to call the reusable Claude Code review workflow
- Include secrets setup, behavior details, and customization tips

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 28, 2026 19:51
@xdanger xdanger self-assigned this Mar 28, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9625fffd93

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an org-shared GitHub Actions reusable workflow for automated PR review using anthropics/claude-code-action@v1, and updates the repository README with setup/usage instructions for downstream repos.

Changes:

  • Added a “Claude Code Review” workflow intended to run on non-draft PRs and post inline review comments.
  • Documented how to consume the reusable workflow, required secrets, and expected behavior/customization via REVIEW.md.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
workflows/claude-code-review.yml Introduces the reusable PR review workflow configuration and permissions/concurrency settings.
README.md Replaces the template README with documentation for adopting the Claude review workflow across repos.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Resolved
Comment thread workflows/claude-code-review.yml Resolved
Comment thread README.md Resolved
xdanger and others added 2 commits March 28, 2026 15:56
- Move `claude-code-review.yml` back to `.github/workflows/` where
  GitHub Actions can discover it for `workflow_call`
- Remove unnecessary `id-token: write` permission (least privilege)
- Add `permissions` block to README caller snippet so downstream
  repos grant `pull-requests: write` for inline review comments

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- `anthropics/claude-code-action@v1` uses OIDC to mint a GitHub token,
  so `id-token: write` is mandatory — not optional as Copilot suggested
- Add it back to both the workflow and the README caller snippet

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

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Resolved
@xdanger
xdanger merged commit 6670e6b into main Mar 28, 2026
7 checks passed
@xdanger
xdanger deleted the xdanger-claude/sleepy-carson branch March 28, 2026 21: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