Skip to content

ci: keep Claude review subagents in the foreground - #44

Merged
Jish2 merged 1 commit into
mainfrom
claude/trusting-bardeen-j8u1yc
Oct 7, 2026
Merged

Jish2 merged 1 commit into
mainfrom
claude/trusting-bardeen-j8u1yc

Conversation

@Jish2

@Jish2 Jish2 commented Oct 7, 2026

Copy link
Copy Markdown
Owner

With full output on (run 37700704913), the review on #40 showed the real failure: the code-review plugin launched its first-step check as a background subagent, Claude ended its turn to wait for it, and in CI the run ends with the turn, so the job exited "successfully" before any review happened.

This tells Claude to run subagents in the foreground and not end its turn until the review is posted.

It can't be tested from a branch: claude-code-action refuses to run a workflow whose contents differ from the default branch.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Pe7k1XG8178GSmRsq1KkAR


Generated by Claude Code

In CI the run ends when Claude's turn ends. The code-review plugin's
first step was launched as a background subagent, so Claude ended its
turn waiting on it and the job exited before any review happened.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Pe7k1XG8178GSmRsq1KkAR
@Jish2
Jish2 merged commit be0318f into main Oct 7, 2026
1 check passed
@Jish2
Jish2 deleted the claude/trusting-bardeen-j8u1yc branch October 7, 2026 23:12
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