fix(hosts): point kimi-cli host at Kimi Code's ~/.agents/skills - #41
Merged
Merged
Conversation
Kimi Code reads ~/.agents/skills (shared) and its data-root skills directory, but not ~/.config/agents/skills. Keep the config path as a secondary entry so earlier kitup installs there remain updatable, and update the auto-host-detection golden case to the new canonical target.
…allback Kimi Code does not read ~/.config/agents/skills, but kitup's choose_scope_path prefers an already-existing scope directory on fresh installs, so keeping the path as a secondary entry silently installed new skills into a location Kimi never scans on hosts where ~/.agents/skills does not exist yet. Remove the fallback entirely and document why.
delete-cloud
changed the title
Collaborator
|
@delete-cloud thanks for the clean fix. |
This was referenced Sep 11, 2026
samzong
pushed a commit
to samzong/confer
that referenced
this pull request
Sep 12, 2026
* feat(adapters): add Kimi Code via native ACP Add `kimi` as a seat agent and MCP host. Seats run `kimi acp` over native ACP; model and reasoning_effort (thinking) are applied through session/set_config_option, unattended seats use mode=auto, and resume uses session/resume. Readiness checks the Kimi Code data root ($KIMI_CODE_HOME, default ~/.kimi-code); confer mcp install writes $KIMI_CODE_HOME/mcp.json; the Skill installs to kitup's kimi-cli host. KIMI_CODE_HOME is pinned on the ACP child. Patch crates.io kitup to lathe-cli/kitup main so kimi-cli user skills land in ~/.agents/skills (lathe-cli/kitup#41). Remove the patch after the next kitup crates.io release. * style: rustfmt Kimi ACP additions CI make check failed on cargo fmt -- --check. No behavior change.
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
2 participants
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.
Fixes #40
Summary
kitup host id
kimi-cli(aliaskimi-code-cli) is the adapter for Kimi Code, not the older kimi-cli product.In 0.1.4 that adapter's
userSkillsDirsprefers~/.config/agents/skills(Amp). Kimi Code 0.41.0 scans~/.agents/skillsand$KIMI_CODE_HOME/skills, not the Amp path.This is a host-data fix only (
spec/hosts.json+ generated constants). The secondary Amp path is not kept:choose_scope_pathprefers an already-existing candidate, so leaving it would still install into a directory Kimi Code does not read when Amp has already created~/.config/agents/skills.Testing
node scripts/check.mjshosts / cases / generated-hosts (run in the contributor environment)node scripts/sync-hosts.mjs