Skip to content

ci: add Claude PR review, drop @claude mention handler - #96

Merged
ak-bugsee merged 1 commit into
mainfrom
ci/claude-review
Oct 16, 2026
Merged

ak-bugsee merged 1 commit into
mainfrom
ci/claude-review

Conversation

@ak-bugsee

Copy link
Copy Markdown
Contributor

Adds a caller for the shared review workflow in bugsee/.github (.github/workflows/claude-review.yml), and removes the per-repo claude.yml @claude mention handler where there was one.

Notes:

  • The review is skipped on this PR itself: the action only runs a workflow file that is identical on the default branch. It starts working on PRs opened after this one is merged.
  • Private repos need a repo-level CLAUDE_CODE_OAUTH_TOKEN secret, because the org secret is not shared with private repos on the Free plan.
  • Add the deep-review label to a PR to get the review from Opus instead of Sonnet.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Lduv96TKHugriihj4RYUe6


Generated by Claude Code

Adds a caller for the shared review workflow in bugsee/.github and removes
the per-repo claude.yml mention handler (if there was one).

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Lduv96TKHugriihj4RYUe6
@coderabbitai

coderabbitai Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 2bacfcd8-a668-4f77-9fea-155a11013672
📥 Commits

Reviewing files that changed from the base of the PR and between ebf1775 and 97443ed.

📒 Files selected for processing: 1
  • .github/workflows/claude-review.yml

Limit details: You’ve used all 10 included reviews currently available.

📝 Summary

Summary by CodeRabbit

  • Chores
    • Added automated reviews for pull requests when they are opened, updated, marked ready, reopened, or labeled. This helps provide review feedback across common stages of the pull request process.

The new GitHub Actions workflow runs for selected pull request events. It calls the reusable Claude review workflow with specified permissions and the Claude OAuth token secret.

Priority: ⬇️ Low

Change: Feature

Merge Risk: ⚪ Minimal · up to 97443

The new review workflow is ready to merge after normal checks.

### Architecture Summary

Architecture risk: 🔵 Low · up to 97443

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 .github/workflows/claude-review.yml: Adds pull request triggers for opened, synchronized, ready-for-review, reopened, and labeled events.
  • observed — Modified behavior in .github/workflows/claude-review.yml: Adds a job that calls the reusable Claude review workflow at bugsee/.github/.github/workflows/claude-review.yml@main, grants the listed repository, pull request, issue, checks, and ID-token permissions, and passes the Claude OAuth token secret.
Pre-merge checks: Passed 5
Check name Status Explanation
Title check Passed The title clearly summarizes the two main changes: adding Claude PR review and removing the @claude mention handler.
Description check Passed The description directly explains the workflow addition, handler removal, review behavior, secret requirement, and deep-review label.
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.
Full details: Docstring Coverage

Explanation

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 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Usage-based review receipt

  • Mode: Continue automatically
  • Reviewed files: 1
  • Waived: $0.25 (charged $0.00)
  • View usage details

Note

This review exceeded your plan’s limits and used usage-based reviews—free during trial, billed after paid activation unless disabled. Manage usage-based reviews.

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

@ak-bugsee
ak-bugsee merged commit 72878ea into main Oct 16, 2026
6 checks passed
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