Skip to content

fix: preserve system prompts for Mistral Chat Completions - #210

Merged
hetaoBackend merged 1 commit into
mainfrom
fix/mistral-system-role
Sep 20, 2026
Merged

hetaoBackend merged 1 commit into
mainfrom
fix/mistral-system-role

Conversation

@SaladDay

@SaladDay SaladDay commented Sep 19, 2026 •

Copy link
Copy Markdown
Collaborator

Change

Default to the system instruction role for thinking-enabled custom OpenAI Chat Completions connections to the exact api.mistral.ai host. Preserve reasoning, explicit compatibility overrides, all other request fields, other hosts, and the native Mistral transport.

Scope excludes authentication, thinking switches, token-limit fields, provider redesign, and a global default change. Kimi (#206), DashScope (#207), and SiliconFlow (#209) are inherited unchanged. The preceding provider PRs are merged; this PR now targets main.

Stack

main -> #210

#206, #207, and #209 are already merged. This final layer is rebased onto main as one commit, preserving the previous PR's file contents. There are no remaining downstream branches.

Evidence

Validation

  • Focused resolver suite: 101 tests passed; the Mistral endpoint case failed before the source fix.
  • npx --yes pnpm@9.12.0 verify: all 14 full-profile gates passed on macOS arm64 / Node 26.7.0 at abc17a4.
  • Fresh independent blind review of the rebased changes found no actionable issues; git diff --check passed.

Publication and maintainer handoff

  • Existing licenses and notices are preserved; publication scope is unchanged.
  • No credentials, account data, real user content, private review material, or internal history is included.
  • No files were added or removed; regressions extend a registered test file. Documentation impact is limited to the vendor patch ledger.
  • Shared-source port: pending.

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

hetaoBackend
hetaoBackend previously approved these changes Sep 20, 2026
@SaladDay
SaladDay changed the base branch from main to fix/siliconflow-system-role September 20, 2026 03:39
@SaladDay
SaladDay added this pull request to stack #229 September 20, 2026 03:42
hetaoBackend
hetaoBackend previously approved these changes Sep 20, 2026
Base automatically changed from fix/siliconflow-system-role to main September 20, 2026 04:57
@SaladDay
SaladDay force-pushed the fix/mistral-system-role branch from cec1030 to abc17a4 Compare September 20, 2026 05:08
@hetaoBackend
hetaoBackend merged commit 8a0b439 into main Sep 20, 2026
11 of 13 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.

2 participants