Skip to content

fix(providers): strip provider prefix from model names for openai_com…#3232

Merged
sigridjineth merged 1 commit into
ultraworkers:mainfrom
Ajinkya-Ghuge:fix/deepseek-model-routing
Jun 7, 2026
Merged

fix(providers): strip provider prefix from model names for openai_com…#3232
sigridjineth merged 1 commit into
ultraworkers:mainfrom
Ajinkya-Ghuge:fix/deepseek-model-routing

Conversation

@Ajinkya-Ghuge
Copy link
Copy Markdown
Contributor

@Ajinkya-Ghuge Ajinkya-Ghuge commented Jun 6, 2026

…pat endpoints

Summary

Fixes an issue where OpenAI-compatible clients (like DeepSeek) fail due to strict provider/model naming conventions by dynamically stripping the provider prefix before sending the payload downstream.

Anti-slop triage

  • Classification:
  • Evidence:
  • Non-destructive review result:

Verification

  • Targeted tests/docs checks ran, or the gap is explicitly recorded.
  • git diff --check passes.
  • No live secrets, tokens, private logs, or unrelated generated churn are included.

Resolution gate

Closes #3229

  • If this PR resolves an issue, the issue number and fix evidence are linked.
  • If this PR should not merge, the rejection/defer rationale is evidence-backed and does not rely on vibes.
  • I did not merge/close remote PRs or issues from an automation lane without owner approval.

@sigridjineth sigridjineth merged commit ae2f203 into ultraworkers:main Jun 7, 2026
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.

DeepSeek native API appears incompatible with provider/model model routing

2 participants