Skip to content

Comments

feat(moonshot): add first-class kimi-k2.5 provider support#53

Open
Plaidmustache wants to merge 3 commits intoFast-Editor:mainfrom
Plaidmustache:codex/pr-moonshot-provider-only
Open

feat(moonshot): add first-class kimi-k2.5 provider support#53
Plaidmustache wants to merge 3 commits intoFast-Editor:mainfrom
Plaidmustache:codex/pr-moonshot-provider-only

Conversation

@Plaidmustache
Copy link
Contributor

@Plaidmustache Plaidmustache commented Feb 16, 2026

Description

Adds first-class Moonshot/Kimi (kimi-k2.5) provider support and updates provider handling for stable Anthropic-style tool loop semantics on /v1/messages.

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Performance improvement
  • Refactoring

Changes Made

  • Added moonshot as a supported model provider in config and validation paths.
  • Added Moonshot provider wiring in routing and provider handlers.
  • Added Moonshot model mapping defaults (e.g., kimi-k2.5).
  • Updated Anthropic-tool-protocol handling so tool loop behavior is consistent for Moonshot on /v1/messages.

Testing

  • Existing tests pass
  • Added new tests
  • Manual testing performed

Manual checks:

  • npm run test:quick passed.
  • Verified provider route and response path behavior for Moonshot integration.

Checklist

  • Code follows style guidelines
  • Documentation updated
  • Tests added/updated
  • No new warnings

Notes

  • Scope is provider-focused and intentionally excludes local ToolHive-specific integration glue.
  • No secrets or local environment values are included.

Includes provider config/validation, anthropic-format handling, providers API exposure, and investigation summary doc.
Treat moonshot as anthropic tool-protocol provider in orchestrator, skip smart-tool filtering on tool-followup turns, and avoid standard-tool injection when tool history exists.
@Plaidmustache
Copy link
Contributor Author

Plaidmustache commented Feb 16, 2026

Maintainer note: this PR is intentionally provider-scoped (Moonshot/Kimi) and excludes local ToolHive-specific glue to keep it generally applicable. Validation run: npm run test:quick (pass).

@veerareddyvishal144
Copy link

Screenshot 2026-02-20 at 12 16 48 AM This is failing with the above error

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