Repository navigation
Revert "Add Cheaper Inference as a built-in cloud provider" - #6778
Conversation
|
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 configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (4)
💤 Files with no reviewable changes (3)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe 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. ChangesProvider preset removal
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~8 minutes Change: Bug fix Suggested reviewers: Merge Risk: ⚪ Minimal · up to 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 SummaryArchitecture risk: 🔵 Low · up to The change affects 2 systems. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
A rabbit checked the provider row, Comment |
Reverts the changes from #6620, removing Cheaper Inference as a built-in cloud provider and restoring the previous model routing documentation.
Summary by CodeRabbit