Skip to content

Use centralized Claude Code Review workflow#695

Merged
justin808 merged 1 commit intomasterfrom
jg/use-centralized-review-workflow
Feb 9, 2026
Merged

Use centralized Claude Code Review workflow#695
justin808 merged 1 commit intomasterfrom
jg/use-centralized-review-workflow

Conversation

@justin808
Copy link
Member

@justin808 justin808 commented Feb 9, 2026

Summary

  • Replaces inline Claude Code Review workflow with a call to the shared reusable workflow in shakacode/.github
  • Removes hardcoded model — uses Claude Code's default (currently Opus 4.6)
  • Future changes only need one update in shakacode/.github

Dependencies

Test plan

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated internal code review workflow configuration to use a reusable external workflow, improving maintainability and consistency across the organization.

Replace inline review config with a call to the shared reusable
workflow. Removes hardcoded model so reviews use Claude Code's
default (currently Opus 4.6).

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@coderabbitai
Copy link

coderabbitai bot commented Feb 9, 2026

Walkthrough

The GitHub Actions workflow for Claude Code Review is refactored to use an external reusable workflow instead of inline steps. The internal implementation including checkout, action invocation, and configuration is removed and replaced with a reference to an external workflow from shakacode/.github, maintaining secret token passage.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Refactoring
.github/workflows/claude-code-review.yml
Replaced inline Claude Code Review implementation (checkout, action invocation, prompts, configuration) with a reusable external workflow reference delegated to shakacode/.github. Token secret properly passed through configuration.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A helper from afar now tends our review,
No need to keep it close, our workflow's through!
One call instead of many—delegation's art,
The code review magic, outsourced from the start! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and clearly describes the main change: replacing an inline workflow with a centralized Claude Code Review workflow reference.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch jg/use-centralized-review-workflow

No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@greptile-apps
Copy link

greptile-apps bot commented Feb 9, 2026

Greptile Overview

Greptile Summary

This PR simplifies the local Claude Code Review workflow by delegating the entire claude-review job to the centralized reusable workflow hosted in shakacode/.github, and passes through the repository’s CLAUDE_CODE_OAUTH_TOKEN secret.

Net effect is that future changes to the review logic are managed centrally, but the review behavior in this repo becomes dependent on the referenced reusable workflow ref and its declared inputs/secrets/permissions.

Confidence Score: 3/5

  • This PR is likely safe to merge, but the workflow may break or change behavior unexpectedly due to how the reusable workflow is referenced and integrated.
  • The change is small and limited to GitHub Actions, but it introduces a hard dependency on an external reusable workflow. Referencing @main makes behavior non-deterministic over time, and the explicit secret mapping must match the upstream workflow_call contract or the workflow will fail validation at runtime.
  • .github/workflows/claude-code-review.yml (verify reusable workflow ref pinning and upstream workflow_call inputs/secrets)

Important Files Changed

Filename Overview
.github/workflows/claude-code-review.yml Replaces the local Claude Code Review job with a call to the reusable workflow in shakacode/.github on the main ref; verify ref pinning and required permissions/inputs align with the reusable workflow.

Sequence Diagram

sequenceDiagram
  participant GH as GitHub PR Event
  participant Repo as This repo workflow
  participant Shared as shakacode/.github reusable workflow
  participant Action as anthropics/claude-code-action
  participant API as GitHub API

  GH->>Repo: pull_request (opened/synchronize)
  Repo->>Shared: uses reusable workflow (@ref)
  Shared->>Action: run Claude Code Review
  Action->>API: fetch PR diff / metadata
  Action->>API: post review comment/status
Loading

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

# - "src/**/*.jsx"

jobs:
claude-review:
Copy link

Choose a reason for hiding this comment

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

Unpinned reusable workflow ref

This calls the reusable workflow at shakacode/.github/...@main, which means behavior can change without a PR in this repo and can break review runs unexpectedly. Pin to an immutable ref (tag or commit SHA) for stability, and bump it intentionally when you want updates.

Comment on lines +9 to +10
uses: shakacode/.github/.github/workflows/claude-code-review.yml@main
secrets:
Copy link

Choose a reason for hiding this comment

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

Secret mapping may not match

secrets: CLAUDE_CODE_OAUTH_TOKEN: ... only works if the called reusable workflow declares on: workflow_call with a secrets: CLAUDE_CODE_OAUTH_TOKEN entry. If the upstream expects a differently named secret or uses secrets: inherit, this workflow will fail at runtime with a validation error; confirm alignment with shakacode/.github and update accordingly.

@github-actions
Copy link

github-actions bot commented Feb 9, 2026

🚀 Quick Review App Commands

Welcome! Here are the commands you can use in this PR:

/deploy-review-app

Deploy your PR branch for testing

/delete-review-app

Remove the review app when done

/help

Show detailed instructions, environment setup, and configuration options.


@justin808 justin808 merged commit d03b9f5 into master Feb 9, 2026
10 of 11 checks passed
@justin808 justin808 deleted the jg/use-centralized-review-workflow branch February 9, 2026 02:40
@github-actions
Copy link

github-actions bot commented Feb 9, 2026

✅ Review app for PR #695 was successfully deleted

View Completed Delete Logs

Control Plane Organization

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