Skip to content

fix(web): preserve embedding dimensions in gateway#3375

Open
marius-kilocode wants to merge 1 commit into
mainfrom
fix/codestral-embedding-dimensions
Open

fix(web): preserve embedding dimensions in gateway#3375
marius-kilocode wants to merge 1 commit into
mainfrom
fix/codestral-embedding-dimensions

Conversation

@marius-kilocode
Copy link
Copy Markdown
Contributor

Preserve embedding dimensions in the embeddings gateway body so Codestral requests keep the hosted catalog's 256-dimensional contract.
Continue removing native Mistral-only fields while covering the Codestral proxy shape in the sanitizer tests.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented May 20, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The change correctly un-strips dimensions from the upstream embedding body so the Codestral (and any other provider honouring the standard parameter) receives the vector-size contract, while continuing to strip the native Mistral-only fields (output_dtype, output_dimension) and the deprecated user field. Tests are well-targeted and cover the new behaviour.

Files Reviewed (2 files)
  • apps/web/src/lib/ai-gateway/embeddings/embedding-request.ts
  • apps/web/src/lib/ai-gateway/embeddings/embedding-request.test.ts

Reviewed by claude-sonnet-4.6 · 265,867 tokens

Review guidance: REVIEW.md from base branch main

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