Skip to content

chore(provider): limit forced none thinking to official api#153

Merged
Vizards merged 1 commit into
mainfrom
chore/force-none-thinking-official-only
Jun 12, 2026
Merged

chore(provider): limit forced none thinking to official api#153
Vizards merged 1 commit into
mainfrom
chore/force-none-thinking-official-only

Conversation

@Vizards

@Vizards Vizards commented Jun 11, 2026

Copy link
Copy Markdown
Owner

Summary

Verification

  • npm run format:check
  • npm run lint
  • npm run compile

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR centralizes DeepSeek “official endpoint” helpers into src/endpoint.ts and scopes the “force thinking=none for helper/internal requests” behavior so it only applies when using the official DeepSeek API, preserving user-configured thinking effort for custom/third-party endpoints.

Changes:

  • Added src/endpoint.ts to host isOfficialDeepSeekBaseUrl/normalizeBaseUrl and the official host constant.
  • Updated provider request preparation to only force thinking=none for helper requests when targeting the official DeepSeek base URL.
  • Rewired pricing and error-handling code to use the centralized endpoint helpers and removed the duplicated official-host constant.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/provider/request.ts Restricts forced none-thinking behavior to the official API (and refactors baseUrl handling).
src/provider/pricing/currency.ts Switches to centralized endpoint helpers and removes local duplicates.
src/endpoint.ts New shared utilities/constants for normalizing and recognizing official endpoints.
src/client/error/index.ts Uses the centralized official-endpoint detection for provider identification.
src/client/consts.ts Removes the official host constant now owned by src/endpoint.ts.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/provider/request.ts

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

@Vizards Vizards merged commit 3c1077d into main Jun 12, 2026
3 checks passed
@Vizards Vizards deleted the chore/force-none-thinking-official-only branch June 12, 2026 05:17
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.

2 participants