Skip to content

chore(providers): remove claude-fable-5 model#5020

Merged
waleedlatif1 merged 1 commit into
stagingfrom
remove-fable-5
Jun 13, 2026
Merged

chore(providers): remove claude-fable-5 model#5020
waleedlatif1 merged 1 commit into
stagingfrom
remove-fable-5

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Remove the claude-fable-5 model entry from the Anthropic provider model list
  • Drop the fable-5 branch from adaptive-thinking detection and clean up the related TSDoc in anthropic/core.ts

Type of Change

  • Chore (maintenance)

Testing

Tested manually

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@vercel

vercel Bot commented Jun 13, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
docs Skipped Skipped Jun 13, 2026 2:02am

Request Review

@cursor

cursor Bot commented Jun 13, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Catalog and comment-only removal with no changes to request paths for remaining models; existing workflows on other Claude IDs are unaffected.

Overview
Removes claude-fable-5 from the Anthropic provider catalog in models.ts, so it no longer appears in pickers or pricing/capability lookups.

In anthropic/core.ts, drops fable-5 from supportsAdaptiveThinking and updates related comments so adaptive-thinking docs no longer mention Fable 5.

Reviewed by Cursor Bugbot for commit f8ae9db. Configure here.

@waleedlatif1 waleedlatif1 merged commit 26224b8 into staging Jun 13, 2026
10 checks passed
@waleedlatif1 waleedlatif1 deleted the remove-fable-5 branch June 13, 2026 02:02
@greptile-apps

greptile-apps Bot commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This chore removes the claude-fable-5 model that was added but is no longer needed — its entry is deleted from the Anthropic provider model list in models.ts, and the related fable-5 branch and TSDoc are cleaned up in anthropic/core.ts.

  • Removes the full claude-fable-5 model definition (pricing, capabilities, 1 M token context window, release date 2026-06-09) from the Anthropic provider in models.ts.
  • Drops the normalizedModel.includes('fable-5') guard from supportsAdaptiveThinking() and updates the function-level comment in core.ts to match.

Confidence Score: 5/5

Straightforward model-entry removal with no logic changes — safe to merge.

Both touched files are limited to deleting a single model definition and its single detection branch. No logic is restructured, no other models are affected, and the remaining supportsAdaptiveThinking branches are unchanged and correct.

No files require special attention.

Important Files Changed

Filename Overview
apps/sim/providers/models.ts Removes the claude-fable-5 model entry (pricing, capabilities, context window, release date) from the Anthropic provider definition.
apps/sim/providers/anthropic/core.ts Removes fable-5 branch from supportsAdaptiveThinking() and updates the TSDoc on buildThinkingConfig() to drop the Fable 5 reference.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[modelId input] --> B[supportsAdaptiveThinking]
    B --> C{model includes\nopus-4-8 / opus-4.8?}
    C -- yes --> E[return true\nadaptive thinking]
    C -- no --> D{model includes\nopus-4-7 / opus-4.7?}
    D -- yes --> E
    D -- no --> F{model includes\nopus-4-6 / opus-4.6?}
    F -- yes --> E
    F -- no --> G{model includes\nsonnet-4-6 / sonnet-4.6?}
    G -- yes --> E
    G -- no --> H[return false\nextended thinking / budget_tokens]

    style E fill:#22c55e,color:#fff
    style H fill:#64748b,color:#fff
Loading

Reviews (1): Last reviewed commit: "chore(providers): remove claude-fable-5 ..." | Re-trigger Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant