Skip to content

ci: replace Claude PR review with CodeRabbit - #12

Merged
musingfox merged 1 commit into
mainfrom
ci/coderabbit-review
Oct 1, 2026
Merged

musingfox merged 1 commit into
mainfrom
ci/coderabbit-review

Conversation

@musingfox

@musingfox musingfox commented Oct 1, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Add .coderabbit.yaml (en-US, chill profile). The Rust review focus from the Claude workflow carries over as path_instructions for src/**/*.rs.
  • Remove .github/workflows/claude-code-review.yml, so each PR gets one automated review instead of two.

Verification

  • .coderabbit.yaml validates against https://www.coderabbit.ai/integrations/schema.v2.json.
  • This PR is the first one CodeRabbit should review after the app install.

Follow-up

  • The CLAUDE_CODE_OAUTH_TOKEN repo secret is no longer used by any workflow.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated project review settings to use English (US) and Rust-focused review guidance.
    • Removed the previous automated pull-request review workflow. These changes affect project maintenance and do not alter the product’s user-facing features or behavior.

Add .coderabbit.yaml (en-US, chill profile) carrying over the Rust review
focus from the Claude workflow as path instructions, and drop
claude-code-review.yml so PRs get one automated review instead of two.

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

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: musingfox/SumVox/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 18a9ca87-667d-45fe-a922-910da5413e1a

📥 Commits

Reviewing files that changed from the base of the PR and between 13463a2 and d9865e1.

📒 Files selected for processing (2)
  • .coderabbit.yaml
  • .github/workflows/claude-code-review.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/claude-code-review.yml

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 8 remain after this review.


📝 Walkthrough

Walkthrough

The repository adds CodeRabbit review configuration for Rust code and removes the Claude Code review workflow that ran on pull requests.

Changes

Review automation

Layer / File(s) Summary
Update review automation
.coderabbit.yaml, .github/workflows/claude-code-review.yml
The new CodeRabbit configuration selects English (US), the chill profile, and Rust-focused review instructions. The Claude Code pull request review workflow is deleted.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~6 minutes

Change: Other

Merge Risk: 🔵 Low · up to d9865

Confirm CodeRabbit is installed and configured to review the intended PRs; otherwise, removing the Claude workflow leaves automated review coverage uncertain. The repository does not make that review a required check.

Architecture Summary

Architecture risk: 🔵 Low · up to d9865

The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.

Changed systems: None identified.

Architecture concerns
No architecture-level concerns identified.

Review details

Before / after behavior

  • observed — Modified behavior in .coderabbit.yaml: Added CodeRabbit review configuration for English (US) and the chill profile, with review instructions scoped to src/**/*.rs covering correctness, Rust idioms, security, subprocess usage, and performance.
  • observed — Modified behavior in .github/workflows/claude-code-review.yml: The Claude auto-review workflow was removed, including its pull-request triggers, permissions, checkout step, and Claude Code review and commenting configuration.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: replacing the Claude pull request review workflow with CodeRabbit.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


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

@musingfox
musingfox merged commit cee95b9 into main Oct 1, 2026
8 checks passed
@musingfox
musingfox deleted the ci/coderabbit-review branch October 1, 2026 03:04
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