Skip to content

fix(llms): remove tool_choice for deepseek models#482

Merged
gaomeng1900 merged 1 commit intomainfrom
fix/llms-deepseek-tool-choice
Apr 27, 2026
Merged

fix(llms): remove tool_choice for deepseek models#482
gaomeng1900 merged 1 commit intomainfrom
fix/llms-deepseek-tool-choice

Conversation

@gaomeng1900
Copy link
Copy Markdown
Collaborator

@gaomeng1900 gaomeng1900 commented Apr 27, 2026

What

Remove tool_choice from DeepSeek request bodies because DeepSeek models do not accept that parameter.

Closes #475

Type

  • Breaking change
  • Bug fix
  • Feature / Improvement
  • Refactor / Chores
  • Documentation / Website / Demo / Testing

Testing

  • Tested in modern browsers
  • No console errors
  • Types/doc added

Automated validation run:

  • npx eslint packages/llms/src/utils.ts passed
  • npm run build --workspace=@page-agent/llms passed
  • npm run typecheck passed

Requirements / 要求

  • I have read and follow the Code of Conduct and Contributing Guide . / 我已阅读并遵守行为准则。
  • This PR is NOT generated by a bot or AI agent acting autonomously. I have authored or meaningfully reviewed every change. / 此 PR 不是由 bot 或 AI 自主生成的,我已亲自编写或充分审查了每一处变更。

@gaomeng1900 gaomeng1900 linked an issue Apr 27, 2026 that may be closed by this pull request
3 tasks
@gaomeng1900 gaomeng1900 merged commit dcad67d into main Apr 27, 2026
4 checks passed
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.

[Feature] deepseek-v4支持

1 participant