Skip to content

Commit 2c9ad2e

Browse files
authored
fix(pr-sous-chef): use sonnet alias for pr-processor sub-agent to resolve HTTP 400 (#42634)
1 parent a146daa commit 2c9ad2e

3 files changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/pr-sous-chef.lock.yml

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/pr-sous-chef.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -369,7 +369,7 @@ safeoutputs noop --message "processed=4; skipped_checks_running=0; skipped_last_
369369
## agent: `pr-processor`
370370
---
371371
description: Processes one PR with minimal API calls and returns skip/nudge decisions
372-
model: claude-sonnet-4.5
372+
model: sonnet
373373
---
374374
Given one PR number and compact metadata:
375375

pkg/cli/pr_sous_chef_workflow_contract_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ func TestPRSousChefWorkflowAddCommentTargetContract(t *testing.T) {
2929
assert.Contains(t, text, "include an explicit unresolved-reviews list", "Workflow should require explicit unresolved review listing in nudge comments")
3030
assert.Contains(t, text, "Process at most **5 PRs** per run.", "Workflow should cap per-run PR processing to 5")
3131
assert.Contains(t, text, "Make at most 8 tool calls total.", "Sub-agent should have a hard tool-call budget")
32-
assert.Contains(t, text, "model: claude-sonnet-4.5", "Sub-agent should use a Sonnet model")
32+
assert.Contains(t, text, "model: sonnet", "Sub-agent should use a Sonnet model alias")
3333
assert.Contains(t, text, "skip_reason: \"sub_agent_error\"", "Workflow should skip failed sub-agent responses without retry")
3434
assert.Contains(t, text, "eligible_count=", "fetch-prs step must export eligible_count output")
3535
assert.Contains(t, text, ".prs | length", "eligible_count should reflect the number of eligible PRs")

0 commit comments

Comments
 (0)