Skip to content

Add Kilo AI - #11129

Open
WebReflection wants to merge 1 commit into
omacom:quattrofrom
WebReflection:omarchy-kilo-ai
Open

Add Kilo AI#11129
WebReflection wants to merge 1 commit into
omacom:quattrofrom
WebReflection:omarchy-kilo-ai

Conversation

@WebReflection

@WebReflection WebReflection commented Sep 10, 2026

Copy link
Copy Markdown

Adds Kilo (@kilocode/cli) as a supported default coding agent, following the existing lazy-installed mise tool pattern.

Generated changes

  • bin/omarchy-default-agent: a kilo case selecting agent="kilo" with the npm:@kilocode/cli backend; usage and omarchy:args metadata list kilo. No aliases.
  • bin/omarchy-agent: kilo shares the OpenCode case, launching as kilo --auto and kilo --auto --prompt <prompt> inline or in the agent terminal. The OpenCode case keeps its exact prior argv.
  • default/omarchy/omarchy-menu.jsonc: setup.default.agent.kilo between Hermes and Muse Code (alphabetical by label), labeled kilo, acting on omarchy-default-agent kilo, using the existing generic Agent glyph U+F06A9 — not a new icon-font glyph. No when/aliases, and no duplicate Install > AI entry.
  • install/user/mise.sh: omarchy-cmd-missing kilo && omarchy-mise-install npm:@kilocode/cli kilo, keeping a user-managed launcher (e.g. kilo-bin from the AUR) safe across re-provisions, like Cursor's guard.
  • migrations/1789043987.sh (0644, no shebang): a Muse-shaped migration — creates the same stub only when kilo is missing and the preinstalls opt-out marker is absent, then leaves both alone.
  • bin/omarchy-remove-preinstalls: deletes ~/.local/bin/kilo only when a nonsymlink wrapper's generated mise use -g line names "npm:@kilocode/cli". Symlinks, mention-only wrappers, and other executables survive, as does kilo's own configuration and state.
  • manual/17-ai.md: table row plus a Defaults section paragraph covering omarchy default agent kilo, the npm backend, /connect login, kilo-bin on the AUR, and the shadowing caveat recommending one installation method.

Agent usage collectors, the agents panel, quick console, first-run picker, and crash diagnosis carry over through the shared agent path with no list edits on my part.

Verification

Validated on this machine (macOS, only Bash 3.2 available):

  • test/shell.d/menu-test.sh — full pass, including the kilo entry assertions, ordering, alias-free and no-Install-entry checks, and the icon-font glyph check.
  • test/shell.d/mise-install-test.sh — full pass, including the new scoped-npm kilo case. The local sandbox blocks mktemp, so it ran with a workspace-backed mktemp shim on PATH — the only environmental deviation in that run.
  • A 41-assertion behavior probe mirroring the suites' assertions passed under Bash 3.2 with a temp HOME and mocked command binaries: wrapper generation and mise-invocation argv, cold selection, failed install (nonzero exit, message names mise, selection preserved), inline open on success, installed-selection and user-owned-launcher flows, bypass/prompted/literal leading-dash argv, unchanged OpenCode argv through the shared case, migration create/preserve/opt-out, user-setup provisioning create/preserve, remove-preinstalls delete-vs-preserve for all four wrapper shapes, and the usage line.

Not runnable here — this machine has no Bash 5, no container/VM tooling, and no sibling ISO checkout:

  • test/shell.d/default-agent-test.sh and test/shell.d/preinstalls-test.sh — require Bash ≥ 4.4 (declare -A, mapfile -d ''); with the extensions in this PR, ./test/shell as a whole needs that environment too.
  • ./test/cli — its runner uses local -n namerefs (Bash ≥ 4.3).
  • VM-based graphical acceptance and real-package runtime checks: the actual mise use download, second-run --version latency, third-party kilo --help, /connect, and skill-file discovery by the CLI at runtime.

@WebReflection
WebReflection marked this pull request as ready for review September 10, 2026 12:53
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