fix: update deepseek-v4-pro reasoning efforts and mappings - #1236
fix: update deepseek-v4-pro reasoning efforts and mappings#1236WHMHammer wants to merge 2 commits into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughDeepSeek V4 metadata now reflects the August 2026 revision. V4 Pro supports ChangesDeepSeek reasoning effort updates
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
src/api/providers/__tests__/deepseek.spec.tsESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox. src/api/providers/deepseek.tsESLint skipped: the ESLint configuration for this file references a package that is not available in the sandbox. 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 |
|
Review and test-failure triage complete.
The remaining action for the contributor is to revert the date-only pricing annotation changes, or grant maintainers write access to the PR branch so the prepared cleanup can be applied. |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Related GitHub Issue
Closes: #1235
Description
Updated the supported reasoning efforts for
deepseek-v4-proand thenormalizeDeepSeekReasoningEffortfunction. Also updated the dates for prices (I checked https://api-docs.deepseek.com/quick_start/pricing again and the prices haven't changed).Test Procedure
I built and installed the extension locally and confirmed that it's working fine.
Pre-Submission Checklist
Visual Snapshot (UI changes only): If a user would notice this change at a glance (layout, theme tokens, brand elements, empty/error states), I've added or updated a*.visual.tsxsnapshot inwebview-ui/. Seewebview-ui/AGENTS.md→ "When a UI change needs a snapshot".Visual Snapshots
N/A
Videos (interaction / animation only)
N/A
Documentation Updates
N/A
Additional Notes
N/A
Get in Touch
I don't use Discord. Please comment in this PR or the linked issue. I signed up for email notifications and will respond promptly.
Summary by CodeRabbit
New Features
Updates