fix(openai-shim): strip store when baseUrl points at Mistral

[0xfandom]

Summary

Mistral's chat-completions endpoint rejects requests with a store field — 422 body.store: Extra inputs are not permitted (see #739). The shim already strips store for Gemini and Cerebras hosts via hasGeminiApiHost / hasCerebrasApiHost (PRs #959, #1040). Add the symmetric host check for Mistral so users hitting api.mistral.ai directly — without CLAUDE_CODE_USE_MISTRAL=1 to engage the gateway profile — don't hit the same wall.

Impact

• Users: OPENAI_BASE_URL=https://api.mistral.ai/v1 (or Custom-provider preset pointing there) no longer 422s on first request.
• Devs: identical pattern to merged fix(openai-shim): strip store when baseUrl points at Cerebras (#1023) #1040; new hasMistralApiHost(baseUrl) predicate, single OR added to shouldStripResponsesStore.

Testing

• Unit test: Mistral: strips unsupported store on chat_completions (#739) mirrors the Cerebras/Gemini cases, verifies request body has no store key.
• bun test src/services/api/openaiShim.test.ts — 92 pass / 0 fail.
• bun run build — bundle clean.
• bun run smoke — 0.9.2 OK.

Notes

• Scope: store-only, mirroring the merged Cerebras fix. Mistral also rejects max_completion_tokens for some models; users wanting full Mistral profile (including max_tokens mapping, devstral defaults) should set CLAUDE_CODE_USE_MISTRAL=1 to engage the gateway descriptor at src/integrations/gateways/mistral.ts.
• Closes Mistral model issue #739.

[techbrewboss]

Reviewed the Mistral host compatibility change and targeted regression test. The new predicate is narrowly used to strip only store, matching the existing Gemini/Cerebras handling for strict OpenAI-compatible hosts.

Verification: bun test src/services/api/openaiShim.test.ts passes, 92 tests / 0 failures.

[kevincodex1]

hello @0xfandom this is good to merge, kindly rebase to main and fix conflicts

[Vasanthdev2004]

Targeted review of current head 23d5a7e.

Verdict: Needs changes

The actual Mistral store stripping change is small and looks reasonable from the diff: it mirrors the existing Gemini/Cerebras host checks and adds a focused regression test for api.mistral.ai.

Current blocker:

1. GitHub reports this branch as DIRTY / conflicting with main, so it cannot be merged or reliably final-approved as-is. Please rebase/merge latest main and resolve the conflicts, then the focused test should be rerun against the rebased branch.

No additional code-level blocker from me beyond the merge conflict state.

[gnanam1990]

The Mistral store strip mirrors the existing Gemini/Cerebras pattern cleanly and is already approved by kevincodex1 and techbrewboss. Vasanthdev2004's CONFLICTING flag is still accurate — confirming a rebase is needed before merge.

Findings:

1. git merge origin/main from PR head produces real conflicts in src/services/api/openaiShim.ts and src/services/api/openaiShim.test.ts. PR #1048 (strip store for vLLM/custom local providers) landed on main and edits the same shouldStripResponsesStore helper and adjacent test cases. Please rebase, resolve so hasMistralApiHost participates in the same OR chain as the new local-provider checks, and rerun the focused test.

Non-blocking:

• Implementation itself is unchanged in value — only conflict resolution should be needed.
• After rebase, bun test src/services/api/openaiShim.test.ts should still report all cases passing.

Verified locally: gh pr checkout 1047, git fetch origin main, git merge --no-commit --no-ff origin/main → CONFLICT in both files; git merge --abort clean.

[kevincodex1]

hi bro @0xfandom this is good to merge kindly please rebase and fix conflicts

[0xfandom]

Rebased onto latest main (now at 7166400). Conflicts in src/services/api/openaiShim.ts and openaiShim.test.ts resolved against PR #1048 — hasMistralApiHost(...) joins the same OR chain as the local-provider check; Mistral test sits alongside the new local-provider test.

bun test src/services/api/openaiShim.test.ts → 93 pass / 0 fail.

[Vasanthdev2004]

Targeted re-review of current head 4255ad1b, focused on the post-rebase Mistral store stripping fix.

Verdict: Approve-ready

What I checked:

• The Mistral detection uses parsed URL host matching (api.mistral.ai / *.mistral.ai), so it avoids substring-based URL mistakes.
• The change only joins the existing OpenAI-compatible provider compatibility gate that already strips unsupported store for Gemini/Cerebras/local providers.
• The regression test verifies store is omitted for https://api.mistral.ai/v1.
• CI is green on the current head.

Local validation:

• bun test src/services/api/openaiShim.test.ts -> 93/93 passing
• git diff --check main..HEAD -> clean

I do not see a remaining blocker on the current head.

[0xfandom]

Rebased onto latest main (now at f9621ab). No conflicts this round — upstream PR #1109 (Venice provider) touched src/services/api/openaiShim.{ts,test.ts} but in disjoint regions from the Mistral host detection.

bun test src/services/api/openaiShim.test.ts → 93 pass / 0 fail.

Ready for merge. @kevincodex1