Skip to content

Conversation

@chrarnoldus
Copy link
Collaborator

No fallbacks are a terrible UX, especially when we start routing more to Vercel, which has fewer providers currently

No fallbacks are a terrible UX, especially when we start routing more to Vercel, which has fewer providers currently
@changeset-bot
Copy link

changeset-bot bot commented Jan 20, 2026

🦋 Changeset detected

Latest commit: b834a25

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
kilo-code Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@kiloconnect
Copy link
Contributor

kiloconnect bot commented Jan 20, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Overview

This PR simplifies the OpenRouter provider configuration by:

  1. Removing the only parameter from OpenRouterProviderParams type
  2. Removing allow_fallbacks: false when a specific provider is selected, allowing fallbacks
  3. Removing duplicate provider configuration logic from completionParams
  4. Updating documentation to reflect that specific provider selection now allows fallbacks

The changes are clean and consistent across the codebase. The getProviderParams() method now handles all provider configuration in one place, which is a good refactoring.

Files Reviewed (4 files)
  • .changeset/upset-glasses-begin.md - Changeset for the patch
  • apps/kilocode-docs/docs/providers/kilocode.md - Documentation update
  • apps/kilocode-docs/docs/providers/openrouter.md - Documentation update
  • src/api/providers/openrouter.ts - Core logic changes

stream: true,
stream_options: { include_usage: true },
...this.getProviderParams(), // kilocode_change: original expression was moved into function
parallel_tool_calls: false, // Ensure only one tool call at a time
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this line is not there in the latest upstream commit

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.

3 participants