Skip to content

fix(acp): match toggleable models case-insensitively#938

Draft
tt-a1i wants to merge 1 commit into
MoonshotAI:mainfrom
tt-a1i:fix/acp-model-toggle-case
Draft

fix(acp): match toggleable models case-insensitively#938
tt-a1i wants to merge 1 commit into
MoonshotAI:mainfrom
tt-a1i:fix/acp-model-toggle-case

Conversation

@tt-a1i

@tt-a1i tt-a1i commented Jun 20, 2026

Copy link
Copy Markdown

Summary

  • normalize model names before checking the ACP toggleable-thinking allow-list
  • add a regression test for mixed-case Kimi-Code / Kimi-For-Coding

Related Issue

Addresses #837 finding 7.

Tests

  • pnpm --filter @moonshot-ai/acp-adapter exec vitest run test/model-catalog.test.ts
  • pnpm --filter @moonshot-ai/acp-adapter run test
  • pnpm --filter @moonshot-ai/acp-adapter run typecheck
  • pnpm --filter @moonshot-ai/acp-adapter run build
  • pnpm --filter @moonshot-ai/kimi-code run typecheck
  • pnpm exec oxlint --type-aware packages/acp-adapter/src/model-catalog.ts packages/acp-adapter/test/model-catalog.test.ts --quiet
  • git diff --check

AI Assistance Disclosure

I used Codex to review the changes, sanity-check the implementation against existing patterns, and help spot potential edge cases.

@changeset-bot

changeset-bot Bot commented Jun 20, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 682e0f5

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@moonshot-ai/kimi-code Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

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.

1 participant