Skip to content

Conversation

@priyansh4320
Copy link
Collaborator

@priyansh4320 priyansh4320 commented Oct 16, 2025

Why are these changes needed?

The current Claude Code Review GitHub Actions workflow fails when reviewing pull requests from forked repositories with a "file not found" error. This prevents external contributors' code from being automatically reviewed, creating an inconsistent review experience.
Example failing PR: #2097
Root causes:
The workflow uses pull_request trigger, which doesn't provide secrets access to forked PRs
After checking out the base repository, the workflow never switches to the PR branch
Claude attempts to review PR files while still on the main branch, resulting in "file not found" errors

Related issue number

Checks

@priyansh4320 priyansh4320 self-assigned this Oct 16, 2025
@priyansh4320 priyansh4320 linked an issue Oct 16, 2025 that may be closed by this pull request
@joggrbot
Copy link

joggrbot bot commented Oct 16, 2025

📝 Documentation Analysis

All docs are up to date! 🎉


✅ Latest commit analyzed: 9a09cf7 | Powered by Joggr

@priyansh4320 priyansh4320 marked this pull request as draft October 16, 2025 11:34
@priyansh4320 priyansh4320 marked this pull request as ready for review October 16, 2025 11:43
@priyansh4320 priyansh4320 marked this pull request as draft October 16, 2025 11:45
@priyansh4320 priyansh4320 force-pushed the 2148-bug-claude-review-wontt-work-on-forked-branches branch from 2ffcf18 to 93dd30f Compare October 16, 2025 11:55
@priyansh4320 priyansh4320 marked this pull request as ready for review October 16, 2025 12:02
Copy link
Collaborator

@marklysze marklysze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@marklysze marklysze added this pull request to the merge queue Oct 21, 2025
Merged via the queue into main with commit 3a7aa79 Oct 21, 2025
19 of 20 checks passed
@marklysze marklysze deleted the 2148-bug-claude-review-wontt-work-on-forked-branches branch October 21, 2025 19:47
@codecov
Copy link

codecov bot commented Oct 21, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 41 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

[Bug]: Claude review wont't work on forked branches

3 participants