Skip to content

Revert "Add Cheaper Inference as a built-in cloud provider" - #6778

Merged
senamakel merged 1 commit into
tinyhumansai:mainfrom
senamakel:revert-pr-6620
Sep 29, 2026
Merged

senamakel merged 1 commit into
tinyhumansai:mainfrom
senamakel:revert-pr-6620

Conversation

@senamakel

@senamakel senamakel commented Sep 29, 2026 •

Copy link
Copy Markdown
Member

Reverts the changes from #6620, removing Cheaper Inference as a built-in cloud provider and restoring the previous model routing documentation.

Summary by CodeRabbit

  • Provider Availability
    • Cheaper Inference is no longer listed among built-in cloud providers or BYOK provider presets.
    • Its bundled provider icon is no longer displayed in settings.

…t/cheaperinference"

This reverts commit cba63eb, reversing
changes made to e2e7f2d.

Co-authored-by: Medulla <medulla@tinyhumans.ai>
@coderabbitai

coderabbitai Bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: da42a126-2f62-40c9-9bfc-1635b3f79a27

📥 Commits

Reviewing files that changed from the base of the PR and between cba63eb and 49e1680.

⛔ Files ignored due to path filters (1)
  • app/src/assets/provider-icons/cheaperinference.svg is excluded by !**/*.svg
📒 Files selected for processing (4)
  • app/src/components/settings/panels/__tests__/builtinCloudProviders.test.ts
  • app/src/components/settings/panels/ai/providerIcons.tsx
  • app/src/components/settings/panels/builtinCloudProviders.ts
  • gitbooks/features/model-routing/local-and-byok-models.md
💤 Files with no reviewable changes (3)
  • app/src/components/settings/panels/builtinCloudProviders.ts
  • app/src/components/settings/panels/ai/providerIcons.tsx
  • app/src/components/settings/panels/tests/builtinCloudProviders.test.ts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 7 remain after this review.


📝 Walkthrough

Walkthrough

The Cheaper Inference provider was removed from the built-in provider list, its bundled icon mapping, and the BYOK provider preset documentation. Related provider tests no longer include it.

Changes

Provider preset removal

Layer / File(s) Summary
Remove Cheaper Inference provider
app/src/components/settings/panels/builtinCloudProviders.ts, app/src/components/settings/panels/ai/providerIcons.tsx, app/src/components/settings/panels/__tests__/builtinCloudProviders.test.ts, gitbooks/features/model-routing/local-and-byok-models.md
Removed the Cheaper Inference provider entry and icon mapping. Removed its related test cases and BYOK preset listing.

Priority: ⬇️ Low

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

Change: Bug fix

Suggested reviewers: yellowsnnowmann

Merge Risk: ⚪ Minimal · up to 49e16

Cheaper Inference is no longer offered as a new built-in choice, while existing saved configurations retain their endpoint and credentials. No concrete merge-blocking regression is established.

Architecture Summary

Architecture risk: 🔵 Low · up to 49e16

The change affects 2 systems.

Changed systems: app, gitbooks

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — app (ui) was modified; 3 changed files map to changed impact.
  • observed — gitbooks (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in app/src/components/settings/panels/tests/builtinCloudProviders.test.ts: The endpoint and auth-style parameterized test no longer includes the cheaperinference case. The separate test asserting empty endpoint and undefined auth style for unknown slugs was also removed.
  • observed — Modified behavior in app/src/components/settings/panels/tests/builtinCloudProviders.test.ts: The expected built-in provider slug set no longer includes cheaperinference.
  • observed — Modified behavior in app/src/components/settings/panels/ai/providerIcons.tsx: The import of the bundled cheaperinference.svg asset was removed.
  • observed — Modified behavior in app/src/components/settings/panels/ai/providerIcons.tsx: The cheaperinference asset mapping was removed, so the slug no longer resolves to a bundled mark or appears in the asset-derived PROVIDER_ICON_SLUGS.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: reverting the addition of Cheaper Inference as a built-in cloud provider.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.

A rabbit checked the provider row,
And saw one entry go.
The icon left its place,
The docs now match the case.
The tests no longer show it so.

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

@senamakel
senamakel merged commit fe1d810 into tinyhumansai:main Sep 29, 2026
31 of 33 checks passed
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