chore: upgrade fullsend review harness - #181
Conversation
Assisted-By: OpenCode rh-pre-commit.version: 2.4.0 rh-pre-commit.check-secrets: ENABLED
|
|
🤖 Finished Review · ✅ Success · Started 3:40 PM UTC · Completed 3:54 PM UTC Commit: Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $3.06 |
|
This change should fix this kind of behavior and probably also we can get away with sonnet instead of opus for code reviews. |
|
Risk Assessment: moderate (2/5) DetailsSmall CI/config change (4 files, 11 lines) with two protected-path changes (.github/ workflows) and CI workflow modifications, offset by minimal churn, single-author history, and no dependency or security-sensitive file changes. |
ReviewFindingsHigh
Next steps:
|
|
🤖 Finished Retro · ✅ Success · Started 8:12 AM UTC · Completed 8:25 AM UTC Commit: Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $3.16 |
Retro: PR #181 — Upgrade fullsend review harnessPR #181 by gashcrumb upgraded fullsend from v0.41.0 to v0.42.0 and added a derived review harness ( Workflow timeline
Key finding: Sonnet model not available on Vertex AIAfter PR #181 merged, the first review using the new config (on PR #176, run 34327800934) failed immediately with: "The model claude-sonnet-4-6 is not available on your vertex deployment. Try --model to switch to claude-sonnet-4-5@20250929." The What went well
No duplicates foundNo existing issues in Proposals filed
|
) The model: sonnet alias in the review harness config resolved to claude-sonnet-4-6, which is not provisioned on the repo's GCP Vertex AI endpoint. This caused all review runs to fail immediately with zero tokens used since PR #181 merged. Pin the model to claude-sonnet-4-5@20250929, which the error message in run 34327800934 confirmed is available on the deployment. Using an explicit version avoids future breakage if the sonnet alias advances again before the deployment is updated. Note: yarn lint:check and yarn prettier:check could not run (yarn not available in sandbox). The change is a single-line YAML config update that is not covered by ESLint or Prettier. Closes #187 Assisted-by: claude-opus-4-6 Co-authored-by: fullsend-code <278716306+fullsend-ai-coder[bot]@users.noreply.github.com>
Upgrades Fullsend from v0.41.0 to v0.42.0. Uses a derived review harness with Sonnet and the upstream 45-minute review budget, resolving reviews that timed out before producing agent-result.json.