Skip to content

fix(ci): pin E2E coverage bot to claude-sonnet-4-6 (retired model 404)#135

Merged
bnsoni merged 1 commit into
mainfrom
fix/coverage-bot-model-pin
Jun 16, 2026
Merged

fix(ci): pin E2E coverage bot to claude-sonnet-4-6 (retired model 404)#135
bnsoni merged 1 commit into
mainfrom
fix/coverage-bot-model-pin

Conversation

@bnsoni

@bnsoni bnsoni commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Problem

The E2E Coverage Bot (claude-code-action@beta) had no model: input, so it used the action’\s default — the retired snapshot claude-sonnet-4-20250514. The API now 404s it:

API Error: 404 not_found_error: "model: claude-sonnet-4-20250514"
→ exit code 1

So every coverage-bot run fails. Both os and lms had the same gap.

Fix

Pin a current model explicitly:

uses: anthropics/claude-code-action@beta
with:
  model: claude-sonnet-4-6   # current Sonnet; exact string, no date suffix
  anthropic_api_key: ...

claude-sonnet-4-6 is the drop-in for the retired Sonnet 4 and is cost-appropriate for a per-PR review bot. Pinning also prevents the default from floating to another retired model later.

🤖 Generated with Claude Code

The E2E Coverage Bot used claude-code-action without a model: input, so it
defaulted to the retired snapshot claude-sonnet-4-20250514, which the API now
rejects (404 not_found_error) -- every run failed. Pin a current model.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@bnsoni bnsoni requested a review from mamigot as a code owner June 16, 2026 23:24
@mamigot

mamigot commented Jun 16, 2026

Copy link
Copy Markdown
Member

Claude finished @bnsoni's task —— View job


E2E Coverage Bot

  • Dismiss stale REQUEST_CHANGES reviews (none found)
  • Identify changed files
  • Run mechanical coverage check (skipped — no app routes changed)
  • Apply decision logic

✅ E2E Coverage — No Issues

Changed files:

  • .github/workflows/e2e-coverage-bot.yml only (+3 lines: adds model: claude-sonnet-4-6)

No new app/**/page.tsx routes were introduced. No e2e specs were added or modified. All existing coverage is intact.

This PR is a CI-only fix that pins the coverage bot to a non-retired model. No user-facing behavior changed, so no e2e coverage updates are required.

Decision: CASE E — Approve

Note: The gh pr review --approve command required additional tool permissions that were not available in this run. A human reviewer should approve this PR.

@bnsoni bnsoni merged commit a85f51a into main Jun 16, 2026
3 of 4 checks passed
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