fix(ci): disable project MCP auto-load for the Claude Code Action - #1359
Conversation
enableAllProjectMcpServers (the action's default) was auto-loading this repo's .mcp.json, spinning up the "railway" MCP server via npx. With no Railway session in a fresh Actions runner, Claude Code failed fast (~2s, zero turns, no reply) on every @claude mention. --strict-mcp-config skips project/user MCP config entirely for this action run.
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 17 minutes Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Comment |
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
Summary
.github/workflows/claude.yml) defaults toenableAllProjectMcpServers: true, which auto-loads this repo's.mcp.json— currently just therailwayserver, launched vianpx @railway/cli mcp.@claudemention has been failing fast (~2s,is_error: true, zero turns,$0cost) before Claude could do any actual work or post a real reply — confirmed by a live throwaway-PR test.claude_args: --strict-mcp-configto the action step so it skips project/user MCP config entirely and runs with only its own built-in tools.Verification
npm run check:github-actions— pin/policy check passes on the edited workflow.npm run verify:pr-local/ unit suite — this change only touches workflow YAML, no application code.issue_comment/pull_request_review*triggers from the default branch, not the PR head, so this fix can't be exercised by an@claudemention on its own PR before merge. I ran a live throwaway-PR test against the current (unfixed) workflow onmainto diagnose the failure (Claude Code initialized→ OAuth token valid, then immediateis_errorcrash matching the MCP-load failure mode). I'd like to re-run that same live test once this merges tomain, to confirm the fix actually resolves it.Risk and rollout
@claudeActions workflow itself.Notes
@claudetest comment on a disposable PR to confirm the run now completes successfully, then report back and clean up the test PR/branch.Generated by Claude Code