docs: explain native vision and vision proxy - #246
Merged
Conversation
There was a problem hiding this comment.
🟡 Changes recommended
Several descriptions incorrectly promise that images go directly to DeepSeek even when a custom API endpoint is configured.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Documents native vision for Flash Vision Exp and Vision Proxy behavior for Flash/Pro across English and Simplified Chinese content.
Changes:
- Adds Flash Vision Exp to model and onboarding documentation.
- Explains native versus proxied image handling.
- Updates localized UI and extension descriptions.
File summaries
| File | Description |
|---|---|
src/i18n.ts |
Updates localized vision UI copy. |
resources/walkthrough/show-models.nls.zh-cn.md |
Adds Vision Exp to Chinese walkthrough. |
resources/walkthrough/show-models.md |
Adds Vision Exp to walkthrough. |
resources/walkthrough/set-api-key.nls.zh-cn.md |
Updates Chinese setup instructions. |
resources/walkthrough/set-api-key.md |
Updates setup instructions. |
README.zh-cn.md |
Documents vision behavior in Chinese. |
README.md |
Documents vision behavior in English. |
package.nls.zh-cn.json |
Updates Chinese setting descriptions. |
package.nls.json |
Updates English setting descriptions. |
package.json |
Revises extension marketplace description. |
Review details
Suppressed comments (1)
src/i18n.ts:221
deepseek-copilot.baseUrlcan point to a proxy or compatible provider, and native image bytes are sent to that configured endpoint rather than necessarily to DeepSeek. Stating “directly to DeepSeek” misrepresents the data destination; describe the configured API endpoint instead.
'Experimental DeepSeek V4 Flash vision model that sends user image attachments directly to DeepSeek for native image understanding.',
- Files reviewed: 10/12 changed files
- Comments generated: 3
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
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.
Summary
Validation