Skip to content

fix(fullsend): switch back to opus - #192

Merged
kadel merged 1 commit into
redhat-developer:mainfrom
gashcrumb:switch-fullsend-model
Sep 9, 2026
Merged

kadel merged 1 commit into
redhat-developer:mainfrom
gashcrumb:switch-fullsend-model

Conversation

@gashcrumb

Copy link
Copy Markdown
Member

This change reverts the model switch to Sonnet but keeps the longer timeout in place for reviews

rh-pre-commit.version: 2.4.0
rh-pre-commit.check-secrets: ENABLED

This change reverts the model switch to Sonnet but keeps the longer
timeout in place for reviews

rh-pre-commit.version: 2.4.0
rh-pre-commit.check-secrets: ENABLED
@sonarqubecloud

sonarqubecloud Bot commented Sep 9, 2026

Copy link
Copy Markdown

@fullsend-ai-review

fullsend-ai-review Bot commented Sep 9, 2026

Copy link
Copy Markdown

🤖 Finished Review · ❌ Failure (validation failed after 1 iteration(s)) · Started 11:00 AM UTC · Completed 11:02 AM UTC

Commit: cd08061 · View workflow run →

Runtime: claude · Model: claude-sonnet-4-5@20250929 · Effort: high

@kadel
kadel merged commit 9350f3a into redhat-developer:main Sep 9, 2026
25 of 26 checks passed
@fullsend-ai-retro

fullsend-ai-retro Bot commented Sep 9, 2026

Copy link
Copy Markdown

🤖 Finished Retro · ✅ Success · Started 12:03 PM UTC · Completed 12:14 PM UTC

Commit: cd08061 · View workflow run →

Runtime: claude · Model: opus → claude-opus-4-6 · Effort: high · Cost: $3.74

@fullsend-ai-retro

Copy link
Copy Markdown

Retro: PR #192 — Cascading agent failure resolved by human intervention

PR #192 resolved a cascading agent failure that spanned ~4 hours and cost ~$14.53 in agent compute with zero successful agent-generated fixes.

What happened

  1. PR chore: upgrade fullsend review harness #181 (merged Sep 9 08:11 UTC): Added model: sonnet to the review harness. The alias resolved to claude-sonnet-4-6, which is not provisioned on Vertex AI — all reviews broke.
  2. Retro → Issue Review harness model sonnet resolves to unavailable claude-sonnet-4-6 on Vertex AI #187 (08:25 UTC): Detected the breakage but recommended claude-sonnet-4-5@20250929, which is also unavailable on Vertex AI.
  3. Code agent → PR fix: pin review harness model to available Vertex AI version #188 (08:35 UTC): Implemented the bad recommendation with a non-conforming PR title (# in scope). Human fixed the title and merged at 10:14 UTC. Reviews still broken.
  4. Human → PR fix(fullsend): use the correct model identifier #191 (10:47 UTC): Tried the correct model (claude-sonnet-4@20250514). Closed at 10:54 UTC — chicken-and-egg deadlock prevented review.
  5. Retro → Issue Review harness model claude-sonnet-4-5@20250929 is unavailable — all reviews blocked #193 (11:06 UTC): Filed correct recommendation, but PR fix(fullsend): switch back to opus #192 was already open with a different approach.
  6. Code agent → PR fix(#193): update review harness model to available Vertex AI version #194 (11:16 UTC): Correct model but repeated the bad PR title pattern. Review also failed.
  7. Human → PR fix(fullsend): switch back to opus #192 (10:59 UTC, merged 12:02 UTC): Removed the model override entirely, reverting to Opus. Problem resolved.

Evidence for existing issues

Immediate concern

Issue #193 and PR #194 remain open with ready-to-code labels despite being superseded by PR #192. The issue proposes switching to a different Sonnet model, which conflicts with the human's deliberate decision to revert to Opus. See proposal below.

Proposals filed

Proposals skipped (target repo not allowed)

File manually or update create_issues.allow_targets in config.yaml:

  • Retro agent should detect and flag agent-filed issues superseded by merged PRs (fullsend-ai/agents)

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