feat(cli): port portable fixes and publish KKM - #1
Merged
Conversation
Curate and adapt 30 generally useful fixes from MoonshotAI/kimi-code, then add atomic inline multi-skill prompts across the CLI, SDK, and both agent engines.
Pidbid
marked this pull request as ready for review
July 28, 2026 09:18
|
❌ Nix build failed |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related Issue
No linked issue. This PR curates portable upstream fixes for the
Pidbid/kkmfork and closes a fork-specific gap around using multiple skills in one prompt.Problem
The fork was still at the same base as upstream while a number of broadly useful correctness, compatibility, and reliability fixes had accumulated. Importing everything would also pull in changes coupled to Moonshot/Kimi services, so this needs a deliberately portable subset.
The TUI and SDK also could not activate multiple skills atomically in one user message. Inputs such as
Use /skill:review and then /skill:commitwere not delivered as one turn with both skill instructions and the original prompt.What changed
Selection approach
Selected exactly 30 upstream PRs using these criteria:
PR MoonshotAI#1720 is the only provider-named exception: its behavior is generic MCP/JSON-schema compatibility rather than a hosted Kimi dependency.
requiredfieldsmax_tokenshandling for custom Anthropic-compatible endpointsrandomUUIDfor VS Code webview IDs]and invalid glob character classes safelyMulti-skill prompts
/skill:<name>tokens anywhere in a prompt, deduplicate them in first-occurrence order, and preserve the original prompt and media.Session.promptWithSkills(...)to the Node SDK.KKM release and branding
kkm.kimias an npm compatibility alias so existing automation continues to work.0.29.2-kkm.1.main.manifest.json, and support idempotent asset re-upload on workflow reruns.kkm upgradedirects users to this repository's Releases.Integration adaptations
Validation
pnpm run typecheck— passed across the monorepo.pnpm run lint— passed with 0 errors (2,677 existing warnings).pnpm exec changeset status— valid patch/minor release set.Usage: kkmand version0.29.2-kkm.1.pi-tuifull suite — 720 tests passed./tmp/.gitdiscovery, unwritable/root/.kimi-code, and restricted network/IPC), not failures in the targeted paths.Checklist
gen-changesetsskill; this PR includes the required changesets.