GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||
GeezerVison is showing github.com — chore(providers): remove claude-fable-5 model by waleedlatif1 · Pull Request #5020 · simstudioai/sim Conversation
Collaborator
Summary
Type of Change
TestingTested manually Checklist
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
|
There was an error while loading. Please reload this page.
PR SummaryLow Risk Overview In Reviewed by Cursor Bugbot for commit f8ae9db. Configure here. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Greptile SummaryThis chore removes the
Confidence Score: 5/5Straightforward 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
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
Reviews (1): Last reviewed commit: "chore(providers): remove claude-fable-5 ..." | Re-trigger Greptile |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
* feat(broadcast): add july-1 newsletter (template, assets, font)
* fix(broadcast): rasterize email logo + flow icons to PNG for client compatibility
Gmail and Outlook strip inline <svg> and don't reliably render <img src=*.svg>,
so the header wordmark and the four flow-mark icons could render blank/broken.
Swap to hosted PNGs (logo.png + readme-*-mark.png, transparent, hi-res) with
alt text, matching the v0.5/v0.6 broadcast pattern. Remove the unused SVGs.
* improvement(broadcast): rename july-1 assets to semantic names
Drop the vestigial readme-/real- prefixes for the self-contained newsletter
folder: flow marks -> mark-{integrate,context,build,monitor}.png (context now
matches its label), screenshots -> feature-{chat,tables,knowledge,files,scheduled-tasks}.png.
* fix(broadcast): correct stale model name in newsletter copy
Claude Fable 5 was added in #4921 but removed in #5020, so it is no longer in
the model catalog. Replace it with Claude Opus 4.8 (current flagship in
providers/models.ts) so the newsletter doesn't advertise an unavailable model.
f8ae9db4
Deployed
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.