Conversation
CCR now handles code review on this repo (since ~2026-03-08), making this workflow redundant. The workflow has been running on every PR push but cannot post reviews due to read-only pull-requests permissions, so it was consuming compute without producing visible output. The claude.yml workflow for @claude mentions is retained. #2337 already added the '@claude review' exclusion guard so it does not overlap with CCR. Matches typescript-sdk#1754.
There was a problem hiding this comment.
LGTM — straightforward removal of a redundant CI workflow.
Extended reasoning...
Overview
This PR deletes a single file: .github/workflows/claude-code-review.yml. The workflow was an automated code review action that triggered on every PR push. It has been superseded by CCR (Claude Code Review) which is now active on the repo, making this workflow redundant. The PR description notes the workflow also had insufficient permissions (pull-requests: read instead of write) so it was running without producing visible output.
Security risks
None. This is a pure deletion of a CI workflow file. No application code, dependencies, or permissions are modified. Removing the workflow actually eliminates unnecessary CI runs and an unused OIDC token grant.
Level of scrutiny
Minimal scrutiny is appropriate. This is a single-file deletion of a CI config with no impact on application code or tests. The motivation is well-documented and the change mirrors an equivalent PR in the TypeScript SDK.
Other factors
No bugs were found. No outstanding reviewer comments. The change is self-contained and clearly correct — removing a workflow that was both redundant and non-functional.
Removes the automated code review workflow that ran on every PR push.
Motivation and Context
CCR now handles code review on this repo (since ~2026-03-08), making this workflow redundant. Worse, the workflow has
pull-requests: readpermissions so it cannot actually post reviews — it's been running on every push (19 times on #2356 alone) and producing no visible output.The
claude.ymlworkflow for@claudementions is retained. #2337 already added the@claude reviewexclusion guard so it doesn't overlap with CCR.Matches modelcontextprotocol/typescript-sdk#1754.
How Has This Been Tested?
Workflow-only change; no code impact. Verified CCR is active by checking recent PRs (#2345, #2356, #2358) — all show the
claudeapp check-suite posting reviews with inline comments and "Extended reasoning" summaries.Breaking Changes
None.
Types of changes
Checklist
Additional context
v1.x never had this workflow, so no backport needed.
AI Disclaimer