Skip to content

feat(models): add Fable 5 and Sonnet 5 to Claude subscription model list#1981

Open
dgokeeffe wants to merge 1 commit into
omnigent-ai:mainfrom
dgokeeffe:feat/claude-code-fable-model-selection
Open

feat(models): add Fable 5 and Sonnet 5 to Claude subscription model list#1981
dgokeeffe wants to merge 1 commit into
omnigent-ai:mainfrom
dgokeeffe:feat/claude-code-fable-model-selection

Conversation

@dgokeeffe

Copy link
Copy Markdown

Related issue

Closes #

Summary

  • Adds claude-fable-5 and claude-sonnet-5 to the curated Claude subscription
    model list (_SUBSCRIPTION_STATIC_MODELS in omnigent/model_catalog.py), so
    the claude-native worker's model picker offers both new tiers.
  • Keeps claude-sonnet-4-6 in the list rather than replacing it — Sonnet 4.6 is
    still the only Sonnet tier available in some regions/workspaces, so removing
    it would regress model selection there.
  • Subscription CLIs (Claude Code) authenticate via OAuth, not an API key, so
    there's no live /v1/models endpoint to enumerate — this list is a hardcoded
    curation that has to be updated by hand whenever Anthropic ships or retires a
    model alias.

Test Plan

  • uv run pytest tests/test_model_catalog.py -q — 40/40 passed.
  • uv run pytest tests/test_claude_native.py tests/test_claude_native_forwarder.py tests/inner/test_claude_sdk_executor.py tests/inner/test_claude_gateway_shim.py tests/policies/builtins/test_cost.py -q — 415/417 passed; the 2 failures (test_resume_workspace_action_tty_down_enter_selects_move, test_resume_workspace_action_tty_escape_leaves) are pre-existing on this branch, reproduced identically with this change reverted, and unrelated to model IDs.

Demo

N/A — no UI change; this is a backend curated-list update consumed by sys_list_models.

Type of change

  • Bug fix
  • Feature
  • UI / frontend change
  • Refactor / chore
  • Docs
  • Test / CI
  • Breaking change

Test coverage

  • Unit tests added / updated
  • Integration tests added / updated
  • E2E tests added / updated
  • Manual verification completed
  • Existing tests cover this change
  • Not applicable

Coverage notes

Changelog

Added Claude Fable 5 and Sonnet 5 to the available Claude model selections

Adds claude-fable-5 and claude-sonnet-5 to the curated subscription
model catalog alongside the existing claude-sonnet-4-6 (kept since
Sonnet 4.6 remains the only option in some regions/workspaces).
@github-actions github-actions Bot added the size/S Pull request size: S label Jul 5, 2026
@github-actions github-actions Bot requested a review from dhruv0811 July 5, 2026 09:31
@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown
Contributor

@dgokeeffe This PR is a Bug fix, Feature, or UI / frontend change but the Demo section is missing or only contains a placeholder.

These change types require a screenshot or screen recording so reviewers can see the new behaviour without checking out the branch. Please update the Demo section with:

  • A screenshot or screen recording of the change, or
  • A link to a hosted video or GIF showing the new behaviour.

Use N/A only when the change has no user-visible effect whatsoever (e.g. a pure refactor or test-only change). If that's the case, uncheck the relevant type box and check Refactor / chore or Test / CI instead.

@github-actions github-actions Bot added the needs-demo PR needs a demo screenshot or recording label Jul 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-demo PR needs a demo screenshot or recording size/S Pull request size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants