Skip to content

fix: add openrouter/ prefix to OpenClaw default model ID#2508

Closed
AhmedTMM wants to merge 2 commits intoOpenRouterTeam:mainfrom
AhmedTMM:fix/openclaw-model-prefix
Closed

fix: add openrouter/ prefix to OpenClaw default model ID#2508
AhmedTMM wants to merge 2 commits intoOpenRouterTeam:mainfrom
AhmedTMM:fix/openclaw-model-prefix

Conversation

@AhmedTMM
Copy link
Collaborator

Summary

  • OpenClaw requires openrouter/ prefix for model IDs routed through OpenRouter
  • Without it: Unknown model: moonshotai/kimi-k2.5 warning before auto-correcting
  • Changed to openrouter/moonshotai/kimi-k2.5 in code + manifest
  • Fixes regression from fix: set OpenClaw default model to kimi-k2.5 #2419

Test plan

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

🤖 Generated with Claude Code

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>
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: 9cc1449

Findings

No security issues found. This PR adds the openrouter/ prefix to OpenClaw's default model ID configuration.

Changes:

  • manifest.json: Updated default model prompt from moonshotai/kimi-k2.5 to openrouter/moonshotai/kimi-k2.5
  • agent-setup.ts: Updated modelDefault and configure fallback to use prefixed model ID
  • package.json: Version bump to 0.16.13

Security assessment:

  • No command injection risks
  • No credential leaks or exposure
  • No path traversal vulnerabilities
  • No XSS/injection issues
  • Changes are purely configuration strings (no code execution paths)

Tests

  • bash -n: N/A (no shell scripts modified)
  • bun test: Cannot run in worktree (workspace dependency issue)
  • Biome lint: PASS (0 errors)
  • curl|bash: N/A (no installation scripts modified)
  • macOS compat: N/A (no shell scripts modified)

-- security/pr-reviewer

@AhmedTMM
Copy link
Collaborator Author

Superseded — changing to openrouter/openrouter/auto instead

@AhmedTMM AhmedTMM closed this Mar 12, 2026
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