fix(ai-deepseek): require clean api base urls#839
Conversation
|
🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: |
3 similar comments
|
🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: |
|
🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: |
|
🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: |
|
🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: |
Summary
baseUrlvalues before building the chat completions URLWhy
The adapter trimmed trailing slashes, but still accepted malformed or surprising custom API bases. This change makes DeepSeek fail early with adapter-specific configuration errors instead of sending requests to unclear destinations.
Validation
vitest run packages/ai/deepseek/src/index.test.tstsc -p packages/ai/deepseek/tsconfig.json --noEmitgit diff --check