Skip to content

fix: replace retired claude model id in workflows - #362

Merged
dkottke-groq merged 1 commit into
mainfrom
fix/claude-action-v1-model-refresh
Aug 7, 2026
Merged

dkottke-groq merged 1 commit into
mainfrom
fix/claude-action-v1-model-refresh

Conversation

@dkottke-groq

Copy link
Copy Markdown
Contributor

Why this is urgent

  • claude-opus-4-1-20250805 was retired 2026-08-05. Anthropic returns a hard error for retired model IDs — no fallback, no silent upgrade — so this workflow fails as soon as Claude is invoked.

Changes

.github/workflows/claude-code-review.yml

  • model claude-opus-4-1-20250805 (retired) -> claude-opus-5

.github/workflows/claude.yml

  • model claude-opus-4-1-20250805 (retired) -> claude-opus-5

Input mapping follows the official v1 migration guide: direct_promptprompt, and model/allowed_tools/disallowed_tools/max_turns/custom_instructionsclaude_args; timeout_minutes moves to the job's timeout-minutes; claude_env becomes a settings env object.

Verification

  • actionlint clean on every changed file.
  • Each file parses as YAML and every remaining with: key is a real input declared by action.yml at v1.
  • Every generated claude_args was run through the action's own tokenizer (shell-quote + stripShellComments, same code path as parse-sdk-options.ts) and asserted to yield byte-identical values to the inputs it replaced.
  • No retired model ID remains in live YAML.

@dkottke-groq

Copy link
Copy Markdown
Contributor Author

The 8 failing checks here are pre-existing on this repo and unrelated to this change, which only touches .github/workflows/.

Every job dies in the Install dependencies step, before any code or workflow logic runs:

× No solution found when resolving dependencies for split (markers: python_full_version >= '3.13'):
╰─▶ Because there is no version of inspect-ai==0.3.141 and your project
    depends on inspect-ai==0.3.141, we can conclude that your project's
    requirements are unsatisfiable.

inspect-ai==0.3.141 is not on PyPI (the current release series is 0.3.252), so uv sync --group dev cannot resolve. PR #350 fails the same way, so this needs a separate dependency-pin fix and is not a blocker for this migration.

@dkottke-groq
dkottke-groq enabled auto-merge (squash) August 7, 2026 19:56
@dkottke-groq
dkottke-groq merged commit 45c63d6 into main Aug 7, 2026
9 of 17 checks passed
@dkottke-groq
dkottke-groq deleted the fix/claude-action-v1-model-refresh branch August 7, 2026 19:59
@github-actions github-actions Bot mentioned this pull request Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants