docs: enhance modelProviders documentation with comprehensive examples and behavior clarifications - #1927
Merged
Merged
Conversation
- Add comprehensive configuration examples for all auth types (openai, anthropic, gemini, vertex-ai) - Add local self-hosted model examples (vLLM, Ollama, LM Studio) - Clarify generation config layering with impermeable provider layer concept - Add Provider Model vs Runtime Model explanation - Document duplicate model ID limitation - Deprecate security.auth.apiKey and security.auth.baseUrl settings - Add notes about extra_body parameter support limitations Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
Mingholy
requested review from
DennisYu07,
LaZzyMan,
gwinthis,
pomelo-nwu and
tanzhenxin
as code owners
February 24, 2026 14:25
Contributor
📋 Review SummaryThis PR significantly enhances the 🔍 General Feedback
🎯 Specific Feedback🟡 High
🟢 Medium
🔵 Low
✅ Highlights
|
Contributor
Code Coverage Summary
CLI Package - Full Text ReportCore Package - Full Text ReportFor detailed HTML reports, please see the 'coverage-reports-22.x-ubuntu-latest' artifact from the main CI run. |
pomelo-nwu
approved these changes
Feb 24, 2026
pomelo-nwu
left a comment
Collaborator
There was a problem hiding this comment.
LGTM!
To be honest, do we need to add a qwen code settings skills so that qwen code can help users configure these contents and achieve true self-service?
… document - Create new model-providers.md with complete model provider configuration guide - Add Bailian Coding Plan documentation with setup and auto-update details - Remove modelProviders content from settings.md to avoid duplication - Document reserved envKey BAILIAN_CODING_PLAN_API_KEY and security recommendations Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
Fix broken link from settings.md#modelproviders to new model-providers.md Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
Change X-Title to X-OpenRouter-Title for proper OpenRouter API compatibility Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
This was referenced Feb 27, 2026
xaelistic
pushed a commit
to xaelistic/qwen-code
that referenced
this pull request
Jun 7, 2026
docs: enhance modelProviders documentation with comprehensive examples and behavior clarifications
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TLDR
Enhanced the
modelProvidersconfiguration documentation with comprehensive examples, clarified the impermeable provider layer behavior, and deprecated legacy credential settings in favor of themodelProvidersapproach.Dive Deeper
This PR significantly improves the
modelProvidersdocumentation indocs/users/configuration/settings.md:Comprehensive Configuration Examples: Replaced the single basic example with detailed, auth-type-specific configurations covering:
Clarified Provider Layer Behavior:
modelProviderslayer is "impermeable" — when a provider model is selected, itsgenerationConfigcompletely replaces all fields, and lower layers (CLI, env, settings) do not participateundefinedrather than inherited from settingsProvider Model vs Runtime Model: Added a new section explaining:
Deprecation Notices:
security.auth.apiKeyandsecurity.auth.baseUrlas deprecated (UI credential input was removed in v0.10.1)extra_bodyis only supported for OpenAI-compatible providersImportant Considerations:
Reviewer Test Plan
security.auth.apiKeysettings see clear migration guidancemodelsConfig.tsTesting Matrix
Note: This is a documentation-only change. No code changes were made.
Linked issues / bugs
Related to #1898
Fixes #1932