fix(ci): target enableAllProjectMcpServers via settings, add diagnostics - #1366
Conversation
#1359's claude_args: --strict-mcp-config had zero observable effect — a live re-test still failed in ~2s with $0 cost and zero turns (no model call ever made), so the crash isn't resolved and may not even be MCP-related. This action runs the Agent SDK query() in-process rather than spawning the claude CLI, so claude_args likely doesn't map to --strict-mcp-config at all. Sets `enableAllProjectMcpServers: false` directly via the settings input (the JSON key the action's own logs show it writing), and temporarily enables show_full_output to capture the real error if this still fails. Both are diagnostic and will be trimmed once root-caused.
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 35 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
claude_args: --strict-mcp-configdid not resolve the@claudecrash — a live re-test after merge still failed in ~2s withis_error: true,total_cost_usd: 0,num_turns: 1. Zero cost means no model call was ever made, so the failure happens during setup, before Claude Code even starts working.query()in-process rather than spawning theclaudeCLI binary, soclaude_argsmay not translate to--strict-mcp-config(or any CLI flag) at all in this execution path.enableAllProjectMcpServers: falsedirectly via the action'ssettingsinput — the actual JSON key the action's own logs show it writing ("Updated settings with enableAllProjectMcpServers: true").show_full_output: trueso that if this still fails, we get the real error instead of a silently swallowed one. Both diagnostics get trimmed to a minimal permanent fix once the real cause is confirmed.Verification
npm run check:github-actions— pin/policy check passes.issue_commentworkflow definitions from the default branch, not the PR head). Will re-run the throwaway-PR@claudetest once this merges.Risk and rollout
show_full_output: truewill print full agent tool-call output to (private-repo) Actions logs for this workflow only, temporarily.@claudeActions workflow itself.Notes
show_full_outputback off).Generated by Claude Code