Skip to content

feat(checks): add claude-opus-5 to known model list - #51

Merged
protosphinx merged 1 commit into
mainfrom
bot/add-claude-opus-5-model
Aug 3, 2026
Merged

feat(checks): add claude-opus-5 to known model list#51
protosphinx merged 1 commit into
mainfrom
bot/add-claude-opus-5-model

Conversation

@protosphinx

Copy link
Copy Markdown
Member

Why

claude-opus-5 shipped as part of the Claude 5 family alongside claude-sonnet-5 and claude-fable-5. It was missing from KNOWN_CLAUDE_MODELS in checks.ts, so any skill that sets model: claude-opus-5 receives a spurious model-unknown warning. This closes that gap.

What

  • Add "claude-opus-5" to KNOWN_CLAUDE_MODELS in src/checks.ts.
  • Add test "does not warn on claude-opus-5" in test/checks.test.ts, following the same pattern as the existing per-model tests.

Tests

  • One new test in test/checks.test.ts: verifies that runChecks emits no model-unknown diagnostic when model: claude-opus-5 is set.
  • All 89 existing tests continue to pass unmodified.

Self-merge gate

  • all CI checks pass
  • LOC delta < 250 (9 lines added, 0 removed)
  • no public-API surface change (src/index.ts not modified)
  • no runtime-dependency additions
  • no workflow file changes
  • tests added or extended

Generated by Claude Code

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@protosphinx protosphinx added the automated Opened by the daily bot label Aug 3, 2026 — with Claude
@protosphinx
protosphinx merged commit f49b849 into main Aug 3, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated Opened by the daily bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant