fix(#193): update review harness model to available Vertex AI version - #194
Closed
fullsend-ai-coder[bot] wants to merge 1 commit into
Closed
fullsend-ai-coder[bot] wants to merge 1 commit into
fullsend-ai-coder[bot] wants to merge 1 commit into
Conversation
The review harness was configured with claude-sonnet-4-5@20250929, which is not provisioned on the repo's Vertex AI endpoint. This caused all automated review agent runs to fail with zero tokens and a model not available error. PR #188 introduced this model (replacing the original unavailable sonnet alias), and PR #191 attempted to correct it but could not be reviewed due to the chicken-and-egg problem. Update the model to claude-sonnet-4@20250514, which is the version the Vertex AI error message explicitly recommends and is confirmed available on the endpoint. Closes #193 Assisted-by: claude-opus-4-6
Contributor
|
Hey there and thank you for opening this pull request! 👋🏼 We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted. Details: |
|
|
🤖 Finished Review · ❌ Failure (validation failed after 1 iteration(s)) · Started 11:18 AM UTC · Completed 11:19 AM UTC Commit: Runtime: claude · Model: claude-sonnet-4-5@20250929 · Effort: high |
Member
|
sonnet isn't available on this vertex AI account, this is probably a better fix for this |
This was referenced Sep 9, 2026
|
🤖 Finished Retro · ✅ Success · Started 12:39 PM UTC · Completed 12:48 PM UTC Commit: Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $2.34 |
Retro: PR #194 — Review harness model fix (closed without merge)TimelinePR #194 was the endpoint of a cascading agent failure involving the review harness model configuration. The full incident chain:
Agents repo: Key findings
Existing issue evidence
Proposals filed
Proposals skipped (target repo not allowed)File manually or update
|
This was referenced Sep 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.fullsend/harness/review.yamlfromclaude-sonnet-4-5@20250929(unavailable) toclaude-sonnet-4@20250514(available on Vertex AI)Context
PR #188 pinned the review model to
claude-sonnet-4-5@20250929, but that model is also not provisioned on the Vertex AI endpoint. PR #191 attempted to fix this but could not be reviewed (chicken-and-egg: the review agent reads model config from the base branch, which has the broken model). This PR applies the same fix PR #191 attempted.Testing
After merge, the next review agent run on any PR should complete successfully (non-zero tokens, review output produced) instead of failing with 'model not available'.
Note
The review agent cannot review this PR because it reads model config from the base branch where the model is broken. A maintainer should merge directly or bypass the review check.
Closes #193
Post-script verification
agent/193-fix-review-harness-model)cb1efdbaf9d6188d148c463ecd47939cde4a02b9..HEAD)