fix: use custom api type to prevent conflict with built-in Anthropic provider

[ramonvg]

Problem

When KIMI_CODE_PROTOCOL is not set (the default), the provider registers with api: "anthropic-messages" — the same built-in API type used by the Anthropic provider. Combined with a custom streamSimple, this causes pi-coding-agent to route Claude/Anthropic requests through streamSimpleKimi, resulting in:

Error: 401 {"type":"error","error":{"type":"authentication_error","message":"invalid x-api-key"}}

The official custom-provider-anthropic example shows the correct pattern: when using streamSimple, always register with a custom api string (e.g. "custom-anthropic-api"), not a built-in one.

Changes

1. Add KIMI_API_TYPE — resolves to "kimi-anthropic-messages" or "kimi-openai-completions" instead of the built-in names.

2. Change api: PROTOCOL → api: KIMI_API_TYPE in registerProvider — the root fix; the kimi provider no longer claims a built-in protocol handler.

3. Route by PROTOCOL instead of model.api in streamSimpleKimi — since model.api is now a custom string, the OpenAI vs Anthropic branch uses the module-level PROTOCOL constant instead.

4. Guard apiKey forwarding in buildPatchedOptions — never spread an empty apiKey onto patchedOptions, which would silently overwrite a valid OAuth credential passed in by the caller.

---

Summary by cubic

Prevents conflicts with the built-in Anthropic provider by registering kimi-coding with custom API types, so Claude/Anthropic requests no longer route through streamSimpleKimi and return 401s. Also improves credential handling and internal routing.

• Bug Fixes
  • Register with custom api values: kimi-anthropic-messages / kimi-openai-completions (via KIMI_API_TYPE).
  • Route upstream by PROTOCOL instead of model.api to align with the custom api.
  • Forward apiKey only when present to avoid overwriting caller OAuth credentials.

^{Written for commit 489fd3c. Summary will update on new commits.}

[cubic-dev-ai]

No issues found across 1 file

[LeonardoRick]

+1, Thanks for the fix! This is affecting me right now as I'm trying to toggle Kimi <-> Claude after installing this extension.

Could we get it merged?

[LeonardoRick]

Hello @Leechael! Any change to get this merged?