feat: Provider/Mistral full support for Mistral on OpenClaw 🇫🇷 by vincentkoc · Pull Request #23845 · openclaw/openclaw

vincentkoc · 2026-02-22T19:12:17Z

Summary

This PR reworks earlier attempts at a Mistral provider with a full provider rewrite and finalizes Mistral integration consistency across memory embeddings, onboarding defaults, and docs.

What changed

Add full runtime support for memorySearch.provider = "mistral":
- New Mistral embeddings client/provider using POST /v1/embeddings
- Default embeddings model: mistral-embed
- Wire into provider auto-selection and fallback resolution
Align memory config/runtime typing + doctor checks with schema support for mistral
Refresh onboarding metadata for mistral/mistral-large-latest:
- context window and max tokens updated to current provider catalog values used by runtime model support
Update docs for Mistral embeddings + endpoints and memory provider selection/fallback
Add changelog entry in Unreleased

Commit breakdown

Memory: add Mistral embeddings provider support
Onboarding: refresh Mistral model metadata
Docs: document Mistral embeddings and endpoints

Validation

pnpm test -- src/memory/embeddings.test.ts src/commands/doctor-memory-search.test.ts src/config/config.schema-regressions.test.ts src/commands/onboard-auth.test.ts

All above targeted tests pass.

Notes

pnpm check currently fails due to a pre-existing unrelated TypeScript error in src/infra/npm-pack-install.ts.

Greptile Summary

This PR adds comprehensive Mistral provider support across OpenClaw's memory embeddings, onboarding, and media understanding systems.

Key changes:

Full Mistral embeddings provider implementation using /v1/embeddings endpoint with mistral-embed default model
Integrated into memory search provider auto-selection, fallback resolution, and doctor checks
Updated onboarding flow with Mistral API key auth and model metadata for mistral-large-latest
Added Voxtral audio transcription support for media understanding
Comprehensive test coverage and documentation updates

Previous review issues resolved:

mistral client field now properly declared and assigned in MemoryManagerSyncOps base class (lines 97, 164 in manager.ts, line 986 in manager-sync-ops.ts)
MISTRAL_DEFAULT_MODEL_REF constant correctly re-exported from onboard-auth.models.ts following established patterns

Implementation quality:

Clean separation of concerns with dedicated embeddings-mistral.ts module
Consistent type safety across all integration points
Proper error handling for string error bodies in batch utilities
Follows existing patterns for provider integration (OpenAI, Gemini, Voyage)

Confidence Score: 5/5

This PR is safe to merge with no identified issues
The implementation is thorough, well-tested, and addresses all previous review concerns. All integration points are properly wired, types are consistent, tests cover the new functionality, and documentation is comprehensive. The code follows established patterns and includes proper error handling.
No files require special attention

_{Last reviewed commit: ca8151d}

greptile-apps

_{48 files reviewed, 4 comments}

_{Edit Code Review Agent Settings | Greptile}

src/memory/manager-sync-ops.ts

src/commands/onboard-auth.credentials.ts

greptile-apps · 2026-02-22T19:17:40Z

Additional Comments (2)

src/memory/manager.ts
Missing this.mistral assignment from provider result

providerResult.mistral is populated by createEmbeddingProvider when the Mistral provider is selected, but it's never stored on the manager instance. Other providers (openAi, gemini, voyage) are all assigned here. Add:

    this.openAi = params.providerResult.openAi;
    this.gemini = params.providerResult.gemini;
    this.voyage = params.providerResult.voyage;
    this.mistral = params.providerResult.mistral;

Prompt To Fix With AI

This is a comment left during a code review.
Path: src/memory/manager.ts
Line: 159-161

Comment:
**Missing `this.mistral` assignment from provider result**

`providerResult.mistral` is populated by `createEmbeddingProvider` when the Mistral provider is selected, but it's never stored on the manager instance. Other providers (`openAi`, `gemini`, `voyage`) are all assigned here. Add:

```suggestion
    this.openAi = params.providerResult.openAi;
    this.gemini = params.providerResult.gemini;
    this.voyage = params.providerResult.voyage;
    this.mistral = params.providerResult.mistral;
```

How can I resolve this? If you propose a fix, please make it concise.

src/memory/manager-sync-ops.ts
Missing this.mistral assignment in fallback activation

When the fallback provider switches to Mistral, fallbackResult.mistral is populated but never assigned to this.mistral. This matches the same gap in the constructor (manager.ts:159-161).

    this.openAi = fallbackResult.openAi;
    this.gemini = fallbackResult.gemini;
    this.voyage = fallbackResult.voyage;
    this.mistral = fallbackResult.mistral;

Prompt To Fix With AI

This is a comment left during a code review.
Path: src/memory/manager-sync-ops.ts
Line: 981-983

Comment:
**Missing `this.mistral` assignment in fallback activation**

When the fallback provider switches to Mistral, `fallbackResult.mistral` is populated but never assigned to `this.mistral`. This matches the same gap in the constructor (`manager.ts:159-161`).

```suggestion
    this.openAi = fallbackResult.openAi;
    this.gemini = fallbackResult.gemini;
    this.voyage = fallbackResult.voyage;
    this.mistral = fallbackResult.mistral;
```

How can I resolve this? If you propose a fix, please make it concise.

vincentkoc · 2026-02-22T19:52:05Z

Addressed the Greptile findings in follow-up commit af1170fbfa.

Fixes applied:

src/memory/manager.ts: persist providerResult.mistral on manager init.
src/memory/manager-sync-ops.ts: persist fallbackResult.mistral when switching fallback providers.

Validation run:

pnpm format:check
pnpm test -- src/memory/embeddings.test.ts src/commands/doctor-memory-search.test.ts

vincentkoc · 2026-02-22T21:34:20Z

@greptileai review

greptile-apps

_{52 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

src/commands/onboard-auth.credentials.ts

vincentkoc · 2026-02-22T22:24:21Z

@greptileai review

@vincentkoc

…3845, thanks @vincentkoc)

…law#23845) * Onboard: add Mistral auth choice and CLI flags * Onboard/Auth: add Mistral provider config defaults * Auth choice: wire Mistral API-key flow * Onboard non-interactive: support --mistral-api-key * Media understanding: add Mistral Voxtral audio provider * Changelog: note Mistral onboarding and media support * Docs: add Mistral provider and onboarding/media references * Tests: cover Mistral media registry/defaults and auth mapping * Memory: add Mistral embeddings provider support * Onboarding: refresh Mistral model metadata * Docs: document Mistral embeddings and endpoints * Memory: persist Mistral embedding client state in managers * Memory: add regressions for mistral provider wiring * Gateway: add live tool probe retry helper * Gateway: cover live tool probe retry helper * Gateway: retry malformed live tool-read probe responses * Memory: support plain-text batch error bodies * Tests: add Mistral Voxtral live transcription smoke * Docs: add Mistral live audio test command * Revert: remove Mistral live voice test and docs entry * Onboard: re-export Mistral default model ref from models * Changelog: credit joeVenner for Mistral work * fix: include Mistral in auto audio key fallback * Update CHANGELOG.md * Update CHANGELOG.md --------- Co-authored-by: Shakker <shakkerdroid@gmail.com>

vincentkoc marked this pull request as ready for review February 22, 2026 19:12

openclaw-barnacle bot added docs Improvements or additions to documentation cli CLI command changes commands Command implementations agents Agent runtime and tooling size: L maintainer Maintainer-authored PR labels Feb 22, 2026

greptile-apps bot reviewed Feb 22, 2026

View reviewed changes

src/memory/manager-sync-ops.ts Show resolved Hide resolved

src/commands/onboard-auth.credentials.ts Outdated Show resolved Hide resolved

vincentkoc changed the title ~~Memory/Mistral: align embeddings runtime, onboarding metadata, and docs~~ feat: Provider/Mistrral full support for Mistral on OpenClaw Feb 22, 2026

vincentkoc changed the title ~~feat: Provider/Mistrral full support for Mistral on OpenClaw~~ feat: Provider/Mistral full support for Mistral on OpenClaw Feb 22, 2026

openclaw-barnacle bot added the gateway Gateway runtime label Feb 22, 2026

vincentkoc mentioned this pull request Feb 22, 2026

Add Mistral AI as a model provider #14730

Closed

vincentkoc force-pushed the vincentkoc-code/mistral-rework-semantic-commits branch from 9c2c8df to ec5abc0 Compare February 22, 2026 21:29

greptile-apps bot reviewed Feb 22, 2026

View reviewed changes

src/commands/onboard-auth.credentials.ts Outdated Show resolved Hide resolved

vincentkoc force-pushed the vincentkoc-code/mistral-rework-semantic-commits branch from dce8c75 to d62a35c Compare February 22, 2026 22:02

This was referenced Feb 22, 2026

feat: add Mistral/Voxtral audio transcription provider #11334

Closed

feat(models): add Mistral AI provider support #16258

Closed

shakkernerd self-assigned this Feb 22, 2026

vincentkoc requested a review from shakkernerd February 22, 2026 22:49

vincentkoc added 7 commits February 22, 2026 23:25

Onboard: add Mistral auth choice and CLI flags

56905e8

Onboard/Auth: add Mistral provider config defaults

b16f453

Auth choice: wire Mistral API-key flow

f968cac

Onboard non-interactive: support --mistral-api-key

4cac864

Media understanding: add Mistral Voxtral audio provider

5f43dac

Changelog: note Mistral onboarding and media support

d237ca2

Docs: add Mistral provider and onboarding/media references

b210e1d

vincentkoc and others added 12 commits February 22, 2026 23:26

Memory: persist Mistral embedding client state in managers

19c461e

Memory: add regressions for mistral provider wiring

bdf9889

Gateway: add live tool probe retry helper

0a0e7b1

Gateway: cover live tool probe retry helper

2628fb3

Gateway: retry malformed live tool-read probe responses

707bead

Memory: support plain-text batch error bodies

84236ed

Tests: add Mistral Voxtral live transcription smoke

764939a

Docs: add Mistral live audio test command

9085878

Revert: remove Mistral live voice test and docs entry

2a92aeb

Onboard: re-export Mistral default model ref from models

b98a2aa

Changelog: credit joeVenner for Mistral work

e31d545

fix: include Mistral in auto audio key fallback

0b6ea2a

shakkernerd force-pushed the vincentkoc-code/mistral-rework-semantic-commits branch from 46aa76b to 0b6ea2a Compare February 22, 2026 23:28

vincentkoc changed the title ~~feat: Provider/Mistral full support for Mistral on OpenClaw~~ feat: Provider/Mistral full support for Mistral on OpenClaw 🇫🇷 Feb 22, 2026

vincentkoc added 2 commits February 22, 2026 18:42

Update CHANGELOG.md

62d1376

Update CHANGELOG.md

df2f6da

shakkernerd added a commit that referenced this pull request Feb 23, 2026

Mistral: full provider integration across auth, media, and memory (#2…

ba37d80

…3845, thanks @vincentkoc)

shakkernerd merged commit d92ba4f into main Feb 23, 2026
25 checks passed

shakkernerd deleted the vincentkoc-code/mistral-rework-semantic-commits branch February 23, 2026 00:03

github-actions bot mentioned this pull request Feb 23, 2026

📡 Upstream Digest — 2026-02-23 01:22 UTC curtismercier/openclaw-mods#105

Open

gemini-code-assist bot mentioned this pull request Feb 23, 2026

Sync/upstream 2026 02 23 QVerisAI/QVerisBot#70

Merged

vincentkoc mentioned this pull request Feb 23, 2026

feat: Provider/Cohere onboarding + auth-choice support #24661

Draft

18 tasks

gemini-code-assist bot mentioned this pull request Feb 23, 2026

Sync with remote: 2/23/2026 ArchitectVS7/OpenClaw#18

Merged

18 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat: Provider/Mistral full support for Mistral on OpenClaw 🇫🇷#23845

feat: Provider/Mistral full support for Mistral on OpenClaw 🇫🇷#23845
shakkernerd merged 25 commits intomainfrom
vincentkoc-code/mistral-rework-semantic-commits

vincentkoc commented Feb 22, 2026 •

edited

Loading

Uh oh!

greptile-apps bot left a comment

Uh oh!

Uh oh!

Uh oh!

greptile-apps bot commented Feb 22, 2026

Uh oh!

vincentkoc commented Feb 22, 2026

Uh oh!

vincentkoc commented Feb 22, 2026

Uh oh!

greptile-apps bot left a comment

Uh oh!

Uh oh!

vincentkoc commented Feb 22, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

vincentkoc commented Feb 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

What changed

Commit breakdown

Validation

Notes

Greptile Summary

Confidence Score: 5/5

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

greptile-apps bot commented Feb 22, 2026

Uh oh!

vincentkoc commented Feb 22, 2026

Uh oh!

vincentkoc commented Feb 22, 2026

Uh oh!

greptile-apps bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

vincentkoc commented Feb 22, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

vincentkoc commented Feb 22, 2026 •

edited

Loading