Add Claude Fable 5 model aliases#3010
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
ApprovabilityVerdict: Approved Simple additive change adding 3 model alias entries to an existing alias map, following the established pattern. Includes corresponding unit tests. The review comment references a file not modified in this PR. You can customize Macroscope's approvability policy. Learn more. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8d2ec1e756
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Dismissing prior approval to re-evaluate 05128df
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 05128df. Configure here.
| const CUSTOM_MODEL_PLACEHOLDER_BY_KIND: Partial<Record<ProviderDriverKind, string>> = { | ||
| [ProviderDriverKind.make("codex")]: "gpt-6.7-codex-ultra-preview", | ||
| [ProviderDriverKind.make("claudeAgent")]: "claude-sonnet-5-0", | ||
| [ProviderDriverKind.make("claudeAgent")]: "claude-sonnet-4-6", |
There was a problem hiding this comment.
Wrong Claude placeholder model ID
Low Severity
This PR updates the Claude custom-model placeholder alongside new claude-fable-5 aliases, but the value is claude-sonnet-4-6 (same as the Cursor entry) instead of the public Fable 5 slug claude-fable-5 described in the PR summary.
Reviewed by Cursor Bugbot for commit 05128df. Configure here.


Summary
claude-fable-5.Validation
bun fmtbun lint(passes with existing unrelated unused-disable warnings)bun typecheckbun run test src/model.test.tsfrompackages/sharedNote
Low Risk
Small alias-map and test-only changes with no auth, persistence, or routing logic touched.
Overview
Adds Claude Fable 5 shorthand to the shared model alias table so user-facing inputs like
fable,fable-5, andclaude-fableresolve to the canonical slugclaude-fable-5for theclaudeAgentprovider.Shared model tests now assert that normalization and selectable-model resolution pick
claude-fable-5when those aliases are used alongside the existing selectable options list.Reviewed by Cursor Bugbot for commit 0ef692e. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Add 'fable', 'fable-5', and 'claude-fable' as aliases for 'claude-fable-5'
Adds three new entries to
MODEL_SLUG_ALIASES_BY_PROVIDERin model.ts under the Claude driver, mappingfable,fable-5, andclaude-fableto the canonical slugclaude-fable-5.Macroscope summarized 0ef692e.