Skip to content

feat(providers): refresh Qwen, Cohere, and Mistral catalogs#102489

Merged
steipete merged 3 commits into
mainfrom
codex/provider-model-refresh
Jul 9, 2026
Merged

feat(providers): refresh Qwen, Cohere, and Mistral catalogs#102489
steipete merged 3 commits into
mainfrom
codex/provider-model-refresh

Conversation

@steipete

@steipete steipete commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Closes #102460

What Problem This Solves

Qwen and Cohere users cannot select several current models from OpenClaw's direct-provider catalogs, current Qwen Coding Plan models are hidden by stale endpoint filtering, and Mistral Small 4 is advertised with outdated context and pricing metadata.

Why This Change Was Made

Refresh the provider-owned static catalogs from current official contracts: add Qwen 3.7 Max/Plus with endpoint-aware availability, add Cohere North Mini Code 1.0 with its binary reasoning mapping, and align both Mistral Small 4 IDs with current context, pricing, and reasoning behavior. Existing provider defaults, auth flows, protocols, and config surfaces stay unchanged.

User Impact

Users can select Qwen 3.7 Plus on Coding Plan or Standard endpoints, Qwen 3.7 Max on Standard endpoints, and Cohere North Mini Code directly. Mistral users get accurate Small 4 limits/pricing plus the pinned mistral-small-2603 model with the same default-off reasoning controls as mistral-small-latest.

Evidence

@openclaw-barnacle openclaw-barnacle Bot added docs Improvements or additions to documentation extensions: mistral size: M maintainer Maintainer-authored PR labels Jul 9, 2026
@clawsweeper

clawsweeper Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Codex review: needs maintainer review before merge. Reviewed July 9, 2026, 3:15 AM ET / 07:15 UTC.

Summary
The branch refreshes Qwen, Cohere, and Mistral provider catalogs, docs, and focused tests for new model IDs, endpoint availability, reasoning, context, and pricing metadata.

PR surface: Source +93, Tests +143, Docs +8. Total +244 across 20 files.

Reproducibility: not applicable. as a bug reproduction, but the catalog gap is source-reproducible: current main lacks the new Qwen/Cohere IDs and still carries stale Mistral Small metadata.

Review metrics: 1 noteworthy metric.

  • Provider catalog metadata: 3 entries added, 2 entries updated. The changed model rows control user-visible model selection plus context, reasoning, and cost estimates before merge.

Root-cause cluster
Relationship: fixed_by_candidate
Canonical: #102460
Summary: The linked issue is the canonical request for this provider catalog refresh, this PR is the active fix candidate, and the earlier similar PR was closed unmerged by its author.

Members:

Proposal only: this assessment does not dispatch repair, suppress jobs, mutate sibling items, close, or merge anything.

Merge readiness
Overall: 🐚 platinum hermit
Proof: 🐚 platinum hermit
Patch quality: 🐚 platinum hermit
Result: ready for maintainer review.

Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch.

Rank-up moves:

  • [P2] Confirm the remaining exact-head CI shards finish green before merge.
  • [P2] If a safe Cohere credential becomes available, add or link a North Mini Code direct smoke transcript, but the current contract/payload proof is enough for review.

Risk before merge

  • [P1] Cohere direct inference was not run with a credential; the PR relies on official Cohere Compatibility API docs plus emitted-payload tests for that provider.
  • [P1] At review time, the PR was mergeable but behind and two compact CI shards were still pending; exact-head green checks remain required before merge.

Maintainer options:

  1. Decide the mitigation before merge
    Land the provider-owned catalog, docs, and focused test refresh after exact-head CI completes, while preserving existing defaults and letting the linked issue close through the merged PR.
  2. Pause or close
    Do not merge this PR until maintainers decide whether the risk is worth taking.

Next step before merge

  • No automated repair is needed because no blocking patch defect was found; the remaining action is normal exact-head CI and maintainer merge handling.

Security
Cleared: No concrete security or supply-chain regression was found; the diff touches provider manifests/runtime metadata, docs, and tests, not workflows, dependencies, lockfiles, scripts, permissions, or secrets.

Review details

Best possible solution:

Land the provider-owned catalog, docs, and focused test refresh after exact-head CI completes, while preserving existing defaults and letting the linked issue close through the merged PR.

Do we have a high-confidence way to reproduce the issue?

Not applicable as a bug reproduction, but the catalog gap is source-reproducible: current main lacks the new Qwen/Cohere IDs and still carries stale Mistral Small metadata.

Is this the best way to solve the issue?

Yes. Updating the provider-owned static catalogs, docs, and focused tests is the maintainable fix; live-only discovery would add provider lifecycle complexity for a curated catalog refresh.

AGENTS.md: found and applied where relevant.

Codex review notes: model internal, reasoning high; reviewed against 363cf0461f25.

Label changes

Label changes:

  • add P2: This is a normal provider catalog improvement with limited blast radius and no evidence of a core runtime outage.
  • add proof: sufficient: Contributor real behavior proof is sufficient. The PR body reports live Qwen and Mistral HTTP 200 direct API checks plus focused Testbox validation; Cohere has contract and payload-test proof because no approved credential was available.
  • add rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🐚 platinum hermit and patch quality is 🐚 platinum hermit.
  • add status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Sufficient (live_output): The PR body reports live Qwen and Mistral HTTP 200 direct API checks plus focused Testbox validation; Cohere has contract and payload-test proof because no approved credential was available.

Label justifications:

  • P2: This is a normal provider catalog improvement with limited blast radius and no evidence of a core runtime outage.
  • rating: 🐚 platinum hermit: Overall readiness is 🐚 platinum hermit; proof is 🐚 platinum hermit and patch quality is 🐚 platinum hermit.
  • status: 👀 ready for maintainer look: ClawSweeper has no concrete contributor-facing blocker left for this PR. Sufficient (live_output): The PR body reports live Qwen and Mistral HTTP 200 direct API checks plus focused Testbox validation; Cohere has contract and payload-test proof because no approved credential was available.
  • proof: sufficient: Contributor real behavior proof is sufficient. The PR body reports live Qwen and Mistral HTTP 200 direct API checks plus focused Testbox validation; Cohere has contract and payload-test proof because no approved credential was available.
Evidence reviewed

PR surface:

Source +93, Tests +143, Docs +8. Total +244 across 20 files.

View PR surface stats
Area Files Added Removed Net
Source 9 117 24 +93
Tests 6 176 33 +143
Docs 5 41 33 +8
Config 0 0 0 0
Generated 0 0 0 0
Other 0 0 0 0
Total 20 334 90 +244

What I checked:

  • Repository policy read: Read the full root AGENTS.md plus scoped extensions/AGENTS.md and docs/AGENTS.md; the provider-boundary, docs generation, changelog, and PR review rules shaped the review. (AGENTS.md:1, 363cf0461f25)
  • Current main still has the catalog gap: Current main has no direct Qwen 3.7 entries, no Cohere North Mini Code row, and Mistral Small latest still has 128000 context and older pricing, so the PR is not obsolete. (extensions/mistral/openclaw.plugin.json:109, 363cf0461f25)
  • Qwen PR head updates the owner plugin catalog: PR head adds qwen3.7-max and qwen3.7-plus, marks Qwen 3.6/3.7 reasoning support, and filters only qwen3.7-max out of Coding Plan catalogs. (extensions/qwen/models.ts:22, c3532d2071d9)
  • Cohere PR head updates manifest metadata and tests: PR head adds north-mini-code-1-0 with 256000 context, 64000 max output, OpenAI-compatible max_tokens, and none/high reasoning-effort mapping; focused tests assert the catalog row and payload mapping. (extensions/cohere/openclaw.plugin.json:35, c3532d2071d9)
  • Mistral PR head updates Small 4 metadata and compat: PR head updates mistral-small-latest to 262144 context and 0.15/0.6 pricing, adds mistral-small-2603, and includes the pinned ID in Mistral reasoning-effort compatibility. (extensions/mistral/openclaw.plugin.json:109, c3532d2071d9)
  • Upstream provider contracts checked: Official docs fetched during review list qwen3.7-plus on Coding Plan, qwen3.7-max/qwen3.7-plus as 1M/64k Qwen models, Cohere North Mini Code as north-mini-code-1-0 with 256K/64K, Cohere Compatibility API reasoning_effort as none/high, and Mistral Small 4 as mistral-small-2603 with 256k and $0.15/$0.6 pricing.

Likely related people:

  • steipete: Opened the linked provider catalog issue and this PR, and prior merged history introduced the Qwen provider and Qwen catalog work. (role: feature owner and adjacent provider contributor; confidence: high; commits: e3ac0f43df3e, 7bf0496dd8f3; files: extensions/qwen/models.ts, extensions/qwen/index.ts, docs/providers/qwen.md)
  • Patrick-Erichsen: Recent current-main blame for the touched Qwen, Cohere, and Mistral provider files points to PR history around commit 1252a3d. (role: recent area contributor; confidence: medium; commits: 1252a3da59a8; files: extensions/qwen/models.ts, extensions/cohere/models.ts, extensions/mistral/api.ts)
  • Vincent Koc: History shows Mistral curated catalog work and later provider/docs touches in the affected provider area. (role: adjacent provider catalog contributor; confidence: medium; commits: 45ede8729ec5, e085fa1a3ffd; files: extensions/mistral/openclaw.plugin.json, extensions/mistral/model-definitions.ts, docs/providers/mistral.md)
  • Ayaan Zaidi: Mistral reasoning compatibility history includes refactoring of the compat patch touched by this PR. (role: Mistral compat contributor; confidence: medium; commits: 880def088edc; files: extensions/mistral/api.ts)
What the crustacean ranks mean
  • 🦀 challenger crab: rare, exceptional readiness with strong proof, clean implementation, and convincing validation.
  • 🦞 diamond lobster: very strong readiness with only minor maintainer review expected.
  • 🐚 platinum hermit: good normal PR, likely mergeable with ordinary maintainer review.
  • 🦐 gold shrimp: useful signal, but proof or patch confidence is still limited.
  • 🦪 silver shellfish: thin signal; proof, validation, or implementation needs work.
  • 🧂 unranked krab: not merge-ready because proof is missing/unusable or there are serious correctness or safety concerns.
  • 🌊 off-meta tidepool: rating does not apply to this item.

Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics.

How this review workflow works
  • ClawSweeper keeps one durable marker-backed review comment per issue or PR.
  • Re-runs edit this comment so the latest verdict, findings, and automation markers stay together instead of adding duplicate bot comments.
  • A fresh review can be triggered by eligible @clawsweeper re-review comments, exact-item GitHub events, scheduled/background review runs, or manual workflow dispatch.
  • PR/issue authors and users with repository write access can comment @clawsweeper re-review or @clawsweeper re-run on an open PR or issue to request a fresh review only.
  • Maintainers can also comment @clawsweeper review to request a fresh review only.
  • Fresh-review commands do not start repair, autofix, rebase, CI repair, or automerge.
  • Maintainer-only repair and merge flows require explicit commands such as @clawsweeper autofix, @clawsweeper automerge, @clawsweeper fix ci, or @clawsweeper address review.
  • Maintainers can comment @clawsweeper explain to ask for more context, or @clawsweeper stop to stop active automation.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1a42e91e52

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread extensions/qwen/models.ts
name: QWEN_37_MAX_MODEL_ID,
reasoning: true,
input: ["text"],
cost: QWEN_DEFAULT_COST,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Use nonzero pricing for qwen3.7-max

When users select qwen/qwen3.7-max on the Standard endpoint, this new Standard-only catalog row reuses the zero-cost placeholder, and calculateCost multiplies model.cost.* directly, so usage reporting will show every run as $0 even though Qwen lists nonzero pay-as-you-go pricing for the model ($2.50/M input, $7.50/M output: https://docs.qwencloud.com/developer-guides/getting-started/pricing). Please give this Standard-only model its actual cost (or endpoint-specific pricing) so cost displays are not misleading.

Useful? React with 👍 / 👎.

@clawsweeper clawsweeper Bot added proof: sufficient ClawSweeper judged the real behavior proof convincing. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels Jul 9, 2026
@steipete
steipete merged commit e7492c6 into main Jul 9, 2026
95 of 97 checks passed
@clawsweeper clawsweeper Bot added the P2 Normal backlog priority with limited blast radius. label Jul 9, 2026
@steipete
steipete deleted the codex/provider-model-refresh branch July 9, 2026 07:16
@steipete

steipete commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

Merged via squash.

github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jul 9, 2026
…#102489)

* feat(providers): refresh popular model catalogs

* chore: leave release changelog to release automation

* docs: refresh provider docs map
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation extensions: mistral maintainer Maintainer-authored PR P2 Normal backlog priority with limited blast radius. proof: sufficient ClawSweeper judged the real behavior proof convincing. rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. size: M status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Refresh direct provider catalogs for Qwen 3.7, North Mini Code, and Mistral Small 4

1 participant