Skip to content

fix: revert claude review trigger to pull_request#2152

Merged
GigaHierz merged 1 commit intomainfrom
fix/claude-review-oidc-revert
Mar 16, 2026
Merged

fix: revert claude review trigger to pull_request#2152
GigaHierz merged 1 commit intomainfrom
fix/claude-review-oidc-revert

Conversation

@viral-sangani
Copy link
Collaborator

Problem

The Claude Code Review workflow has been failing on every single PR since March 9th with:

App token exchange failed: 401 Unauthorized - Invalid OIDC token

Root Cause

PR #2145 changed the trigger from pull_request to pull_request_target to support fork PRs. However, Anthropic's OIDC token exchange endpoint does not accept tokens from pull_request_target events — it only allowlists pull_request in the OIDC event_name claim.

This is a known upstream bug: anthropics/claude-code-action#713

Fix

Revert to pull_request trigger, which restores working automatic code reviews for all same-repo branch PRs.

Tradeoff

Fork PRs will not get automatic Claude reviews until Anthropic fixes the upstream issue. This is acceptable since the majority of PRs to this repo come from same-repo branches.

Verification

  • Before (all failing): Actions history
  • Last successful run was on March 2nd when the trigger was still pull_request

Reverts the pull_request_target trigger back to pull_request.

The pull_request_target trigger was introduced in #2145 to support fork
PRs, but Anthropic's OIDC token exchange endpoint does not accept tokens
from pull_request_target events, causing 100% failure rate on all
reviews since March 9th.

This is a known upstream issue:
anthropics/claude-code-action#713

Until Anthropic adds pull_request_target to their OIDC allowlist, we
revert to pull_request which works for same-repo branch PRs.
@GigaHierz GigaHierz merged commit 7231baf into main Mar 16, 2026
5 of 7 checks passed
@GigaHierz GigaHierz deleted the fix/claude-review-oidc-revert branch March 16, 2026 15:24
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