You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
fix(ci): preserve active Claude reviews on unrelated labels - #3
Unrelated label events currently cancel an active Claude review even though their replacement job is skipped. Give those events a separate concurrency group and remove the unused issues: write grant. Verified both group paths and preservation of PR comment and OIDC permissions; existing review triggers remain unchanged.
Summary by CodeRabbit
Chores
Updated automated review workflow handling so label-related events are grouped separately from other review runs.
No actionable comments were generated in the recent review.
Walkthrough
The Claude review workflow now uses separate concurrency groups for claude-review label events and other label events. The review job no longer requests issues: write.
Changes: Workflow adjustments
Layer / File(s)
Summary
Concurrency and permissions .github/workflows/claude-review.yml
The concurrency group uses review or ignored alongside the PR number. The job no longer requests issues: write.
This workflow-only change keeps an active Claude review from being cancelled by unrelated label events and trims token permissions. No merge-blocking risk was found.
Architecture risk: Low · up to fdd98
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: The concurrency group now separates non-claude-review label events from review events using ignored and review suffixes, respectively; both groups remain keyed by PR number.
observed — Modified behavior in .github/workflows/claude-review.yml: Removed the job’s issues: write permission.
Pre-merge checks: 4
Check name
Status
Explanation
Description Check
Check skipped - CodeRabbit’s high-level summary is enabled.
Title check
The title names the main change: the workflow's concurrency handling is changed so unrelated label events no longer cancel an active Claude review. The raw summary confirms the concurrency group split…
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Full details: Title check
Explanation
The title names the main change: the workflow's concurrency handling is changed so unrelated label events no longer cancel an active Claude review. The raw summary confirms the concurrency group split for non-claude-review label events and the removal of issues: write. The title uses the conventional fix(ci) prefix and is specific enough for a teammate scanning history.
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
Reviewing files that changed from the base of the PR and between 93dc2fc and fdd9872.
Files selected for processing: 1
.github/workflows/claude-review.yml
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 7 remain after this review.
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
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.
Unrelated label events currently cancel an active Claude review even though their replacement job is skipped. Give those events a separate concurrency group and remove the unused issues: write grant. Verified both group paths and preservation of PR comment and OIDC permissions; existing review triggers remain unchanged.
Summary by CodeRabbit