feat(cohere): add current Command models#102563
Conversation
|
Codex review: needs maintainer review before merge. Reviewed July 9, 2026, 6:06 AM ET / 10:06 UTC. Summary PR surface: Source +107, Tests +81, Docs +7. Total +195 across 13 files. Reproducibility: not applicable. as a feature PR rather than a broken existing-behavior report. Source inspection shows current main lacks the requested Cohere Command entries, and the PR head adds them. Review metrics: 3 noteworthy metrics.
Root-cause cluster Members:
Proposal only: this assessment does not dispatch repair, suppress jobs, mutate sibling items, close, or merge anything. Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Risk before merge
Maintainer options:
Next step before merge
Maintainer decision needed
Security Review detailsBest possible solution: Land the provider-owned catalog, docs, and tests only after maintainers explicitly approve Command A+ as the fresh Cohere default and modern sweep representative. Do we have a high-confidence way to reproduce the issue? Not applicable as a feature PR rather than a broken existing-behavior report. Source inspection shows current main lacks the requested Cohere Command entries, and the PR head adds them. Is this the best way to solve the issue? Mostly yes: the Cohere plugin manifest, onboarding helper, provider entry, docs, and focused tests are the right owner boundary for this catalog refresh. The open question is the maintainable product/default decision, not a concrete patch defect. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against d27d7ba6d358. Label changesLabel changes:
Label justifications:
Evidence reviewedPR surface: Source +107, Tests +81, Docs +7. Total +195 across 13 files. View PR surface stats
What I checked:
Likely related people:
What the crustacean ranks mean
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
Review history (1 earlier review cycle)
|
|
Land-ready proof for fd244c2:
Known proof gaps: none for the changed Cohere catalog/default/tool-capability path. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fd244c29f7
ℹ️ 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".
|
|
||
| ### Changes | ||
|
|
||
| - **Cohere models:** add Command A+, Command A Reasoning, and Command A Vision to the direct provider, make Command A+ the onboarding default, and include it in modern live-model sweeps. |
There was a problem hiding this comment.
Remove the normal feature entry from CHANGELOG
For this non-release Cohere model change, adding an Unreleased changelog bullet bypasses the repo's release-generation contract (AGENTS.md says CHANGELOG.md is release-only and owned by release generation). Leaving this here can make the next generated changelog duplicate or preserve an entry that was not derived from the merged PR/direct-commit release sweep; put the release-note context in the PR/commit metadata instead.
Useful? React with 👍 / 👎.
|
Merged via squash.
|
* feat(cohere): add current Command models * docs: keep release notes out of PR
Closes #102545
What Problem This Solves
OpenClaw's direct Cohere provider exposes Command A from March 2025 and North Mini Code, but omits Cohere's current Command A+, Command A Reasoning, and Command A Vision models. New Cohere users therefore get a stale onboarding default and must hand-author incomplete model metadata to use current models.
Why This Change Was Made
Add the current Command catalog with Cohere's documented modality, context, output, reasoning, and tool-use contracts; make Command A+ the onboarding default; and advertise only tool-capable current Cohere models to modern agent sweeps. The same sweep refresh selects stable Gemini 3.5 Flash, retains both Grok 4.5 and the dated Grok 4.20 reasoning release, and rejects mutable
-latest-...aliases.Command A Vision remains selectable but is explicitly non-tool-capable and excluded from tool-oriented modern sweeps.
User Impact
Users can select Command A+, Command A Reasoning, and Command A Vision without custom provider definitions. Fresh Cohere onboarding selects Command A+, thinking levels map to the Compatibility API's supported
none/highvalues, and Vision requests do not receive unsupported agent tools.Evidence
tbx_01kx2xyv9heryvxpk1re9t6hds(blacksmith-testbox, Actions run):corepack pnpm test extensions/cohere/index.test.ts extensions/cohere/onboard.test.ts src/agents/model-compat.test.ts— 66 tests passed.corepack pnpm test:extension cohere— 9 tests passed.env OPENCLAW_CHECK_CHANGED_REMOTE_CHILD=1 OPENCLAW_CHANGED_LANES_RAW_SYNC=1 corepack pnpm check:changed— all selected typecheck, lint, docs, and guard lanes passed.corepack pnpm build— passed.cbx_5502273e1db8(gcp,n2-standard-8, run):OPENCLAW_LIVE_GATEWAY=1 OPENCLAW_LIVE_GATEWAY_PROVIDERS=cohere OPENCLAW_LIVE_GATEWAY_MODELS=cohere/command-a-plus-05-2026 OPENCLAW_LIVE_GATEWAY_MAX_MODELS=1 corepack pnpm test:live src/gateway/gateway-models.profiles.live.test.ts— 110 tests passed.read,exec+read, and image probes all passed.COHERE_API_KEYwas injected as one redacted, process-scoped environment variable and was not written to the checkout or logs.AI-assisted: yes. No agent transcript attached.