fix(adapter-deepseek): set discovered context window#915
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
概览该 PR 在 DeepSeek 适配器的客户端中为模型刷新方法补充了显式的最大令牌上限配置,将 变更内容DeepSeek 模型最大令牌配置
估计代码审查工作量🎯 1 (极简) | ⏱️ ~2 分钟 可能相关的 PR
诗
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Code Review
This pull request bumps the version of the DeepSeek adapter and introduces a hardcoded maxTokens limit of 1,000,000. However, this limit is too high and inaccurate for DeepSeek models, which support 128,000 tokens for LLMs and 8,192 tokens for embeddings. It is recommended to set the maxTokens conditionally based on the model type to avoid API errors and context overflow.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
This pr fixes DeepSeek model metadata for models returned by the provider API.
Bug fixes
Other Changes
Validation