Skip to content

fix: set OpenClaw default model to kimi-k2.5#2419

Merged
louisgv merged 2 commits intoOpenRouterTeam:mainfrom
AhmedTMM:fix/cloud-defaults-mismatch
Mar 10, 2026
Merged

fix: set OpenClaw default model to kimi-k2.5#2419
louisgv merged 2 commits intoOpenRouterTeam:mainfrom
AhmedTMM:fix/cloud-defaults-mismatch

Conversation

@AhmedTMM
Copy link
Collaborator

Summary

  • modelDefault and configure fallback in agent-setup.ts were hardcoded to openrouter/auto, overriding the manifest's moonshotai/kimi-k2.5
  • Now both match the manifest

Test plan

  • bun test — 1447 pass, 0 fail
  • biome check — clean

🤖 Generated with Claude Code

The manifest was updated to moonshotai/kimi-k2.5 but the code still
hardcoded openrouter/auto in both modelDefault and the configure
fallback.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Member

@louisgv louisgv left a comment

Choose a reason for hiding this comment

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

Security Review

Verdict: APPROVED
Commit: c654e7c

Findings

No security issues identified. This is a clean bug fix that aligns OpenClaw's default model with the manifest.json configuration.

Changes reviewed:

  • Line 619: modelDefault: "moonshotai/kimi-k2.5" - matches manifest.json line 59
  • Line 632: Fallback value in configure function - consistent with modelDefault
  • Version bump: 0.15.33 → 0.15.34 (appropriate for bug fix)

Security checks:

  • ✓ No command injection vectors
  • ✓ No credential handling changes
  • ✓ No path traversal risks
  • ✓ No XSS/injection opportunities
  • ✓ No unsafe eval or dynamic code execution
  • ✓ String literals only (no interpolation or dynamic construction)
  • ✓ Aligns with manifest.json configuration

Tests

  • bun test: PASS (1464 tests, 0 failures)
  • biome lint: PASS (0 errors)
  • bash -n: N/A (no shell scripts modified)
  • Configuration consistency: VERIFIED (matches manifest.json line 59)

-- security/pr-reviewer

@louisgv louisgv merged commit 769aa69 into OpenRouterTeam:main Mar 10, 2026
5 checks passed
AhmedTMM added a commit to AhmedTMM/spawn that referenced this pull request Mar 11, 2026
OpenClaw requires the openrouter/ prefix for model IDs routed through
OpenRouter. Without it, openclaw logs "Unknown model: moonshotai/kimi-k2.5"
before auto-correcting. Fixes regression from OpenRouterTeam#2419.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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