From cb70c3c41f74aa0518e22cd82f762a3a9c7f58ca Mon Sep 17 00:00:00 2001 From: Brennan Benson <79079362+brennanb2025@users.noreply.github.com> Date: Tue, 28 Jul 2026 19:23:45 -0700 Subject: [PATCH] docs(skills): attribute the CLI's install prompts to the right cause --- src/shared/agent-feature-install-commands.ts | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/shared/agent-feature-install-commands.ts b/src/shared/agent-feature-install-commands.ts index 121783223d6..2b411d82d92 100644 --- a/src/shared/agent-feature-install-commands.ts +++ b/src/shared/agent-feature-install-commands.ts @@ -14,9 +14,11 @@ export function buildAgentFeatureSkillInstallCommand(skillNames: readonly string } // Why: pasted into a live terminal, so any prompt strands the user and closing the panel // kills the session mid-prompt. Both flags are load-bearing, as in the headless updater: - // `npx --yes` skips npm's install-this-package prompt (npx re-resolves `latest` every - // run, so a cold or stale cache prompts); `skills … -y` skips the CLI's own selects, which - // fire even for a single agent because `--global` targets the canonical dir plus the provider's. + // `npx --yes` skips npm's install-this-package prompt (npx re-resolves `latest` every run, + // so a cold or stale cache prompts); `skills … -y` skips the CLI's own, whose `Proceed with + // installation?` confirm is unconditional and whose install-method select fires even for a + // single agent, since the CLI always adds its universal `.agents/skills` target alongside + // the agent's own dir. Neither prompt is a consequence of `--global`. return `npx --yes skills add ${ORCA_SKILLS_REPOSITORY_URL} --skill ${skillNames.join(' ')} --global -y` }