Skip to content

fix(sdk): use Claude harness model identifiers#5365

Merged
mmabrouk merged 1 commit into
release/v0.105.6from
fix/claude-fable-model-id
Jul 19, 2026
Merged

fix(sdk): use Claude harness model identifiers#5365
mmabrouk merged 1 commit into
release/v0.105.6from
fix/claude-fable-model-id

Conversation

@mmabrouk

Copy link
Copy Markdown
Member

Problem

The Claude harness picks a model by sending a bare request value (an alias like sonnet, or a canonical id like claude-fable-5), not a provider/id string. The catalog had drifted from what the harness actually accepts:

  • It listed the friendly forms fable and fable[1m]. The harness does not recognize the Fable model family under those ids, so selecting Fable could not resolve.
  • It carried a full grid of [1m] long-context variants (default[1m], sonnet[1m], haiku[1m], fable[1m]) as if they were stable request values. They are context-hinted variants that a session exposes or drops as account entitlements and promotions change, so pinning them in the catalog is unreliable.

Fix

Publish the stable request values the picker can always send, and let the runner widen a bare id to a hinted variant when that is the only one offered.

  • CLAUDE_MODEL_ALIASES now: default, sonnet, haiku, opus[1m], claude-fable-5.
  • For Fable, publish the bare canonical claude-fable-5: it matches a live bare option exactly, and the runner resolves it to claude-fable-5[1m] when that is the only variant on offer. The friendly fable / fable[1m] forms are gone.
  • claude_models.curated.json trimmed to the stable set; the curation note and the sync-model-catalog skill now explain the stable-bare-id-plus-runner-widening rule so the catalog does not get re-pinned to one session's hinted set.

Tests

test_capabilities.py and test_model_catalog.py updated to assert the exact harness config values — claude-fable-5 present, friendly fable absent, no provider/-prefixed ids.

@vercel

vercel Bot commented Jul 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agenta-documentation Ready Ready Preview, Comment Jul 17, 2026 7:37pm

Request Review

@dosubot dosubot Bot added size:M This PR changes 30-99 lines, ignoring generated files. python Pull requests that update Python code labels Jul 17, 2026
@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Updates Claude’s stable harness aliases and curated catalog, adds claude-fable-5, removes deprecated variants, revises synchronization guidance, and aligns tests with the new exact alias set.

Changes

Claude alias catalog

Layer / File(s) Summary
Stable alias contract and catalog
.agents/skills/sync-model-catalog/SKILL.md, sdks/python/agenta/sdk/agents/capabilities.py, sdks/python/agenta/sdk/agents/data/claude_models.curated.json
Defines the stable Claude aliases, adds claude-fable-5, removes deprecated model variants, updates catalog metadata, and documents canonical Fable identifiers.
Alias contract validation
sdks/python/oss/tests/pytest/unit/agents/connections/test_capabilities.py, sdks/python/oss/tests/pytest/unit/agents/connections/test_model_catalog.py
Updates Claude catalog assertions and verifies the exact ordered CLAUDE_MODEL_ALIASES list.

Estimated code review effort: 2 (Simple) | ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly summarizes the main change: updating Claude model identifiers for the SDK.
Description check ✅ Passed The description matches the changes and explains the identifier and catalog updates.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/claude-fable-model-id

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

Railway Preview Environment

Preview URL https://gateway-production-ad27.up.railway.app/w
Project agenta-oss-pr-5365
Image tag pr-5365-5a18d27
Status Deployed
Railway logs Open logs
Workflow logs View workflow run
Updated at 2026-07-17T19:45:30.236Z

@mmabrouk
mmabrouk changed the base branch from main to release/v0.105.6 July 19, 2026 14:01
@mmabrouk
mmabrouk merged commit 2f229b7 into release/v0.105.6 Jul 19, 2026
58 of 64 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python Pull requests that update Python code size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant