argsbarg 4.0.3 → 4.1.0

package/src/install/index.ts

@@ -1,22 +1,34 @@
 import { readSync } from "node:fs";
 import { resolveCapabilities } from "../capabilities.ts";
 import { displayAppConfigPath, runInstallConfigure } from "../config/bootstrap.ts";
-import { cliSkillInstall } from "../skill/install.ts";
+import { cliSkillInstall, skillTargetFromActionKind } from "../skill/install.ts";
 import type { CliProgram } from "../types.ts";
-import { checkCodexMcpConflict } from "./mcp-codex.ts";
-import { checkMcpConflict, expectedMcpEntry } from "./mcp-config.ts";
-import { checkOpenCodeMcpConflict, expectedOpenCodeMcpEntry } from "./mcp-opencode.ts";
-import { resolveInstallPaths, userHome } from "./paths.ts";
+import { normalizeInstallRawOpts } from "./normalize.ts";
+import { resolveInstallPaths } from "./paths.ts";
 import { buildInstallPlan, buildUpdatePlan, type InstallAction, type InstallOpts } from "./plan.ts";
-import { installErr, installInfo, installOut, printInstallStatus } from "./status.ts";
-import { buildUninstallPlan, type UninstallAction, uninstallSkillDir } from "./uninstall.ts";
+import {
+  installErr,
+  installInfo,
+  installOut,
+  printInstallStatus,
+  writeInteractiveInstallIntro,
+} from "./status.ts";
+import { resolveEffectiveInstallTargets } from "./target-effective.ts";
+import { runTargetPreflight } from "./target-plan-build.ts";
+import type { InstallActionKind } from "./target-types.ts";
+import {
+  buildUninstallPlan,
+  skillDirFromUninstallSummary,
+  type UninstallAction,
+  uninstallSkillDir,
+} from "./uninstall.ts";
 import { cliUpdate } from "./update.ts";
 
 export function parseInstallOpts(raw: Record<string, string>): InstallOpts {
   const flag = (name: string) => raw[name] === "1";
   return {
     all: flag("all"),
-    bin: flag("bin"),
+    app: flag("app"),
     completions: flag("completions"),
     skill: flag("skill"),
     mcp: flag("mcp"),
@@ -26,44 +38,33 @@ export function parseInstallOpts(raw: Record<string, string>): InstallOpts {
     status: flag("status"),
     uninstall: flag("uninstall"),
     configure: flag("configure"),
-    config: flag("config"),
     yes: flag("yes"),
     dry: flag("dry"),
     json: flag("json"),
-    quiet: flag("quiet"),
-    prefix: raw.prefix,
   };
 }
 
 export function validateInstallOpts(opts: InstallOpts): string | null {
-  if (opts.configure) {
-    if (
-      opts.all ||
-      opts.bin ||
-      opts.completions ||
-      opts.skill ||
-      opts.mcp ||
-      opts.config ||
-      opts.reinstall ||
-      opts.update ||
-      opts.uninstall ||
-      opts.status
-    ) {
-      return "--configure cannot be combined with other install flags.";
-    }
+  const configureOnlyInstall =
+    opts.configure &&
+    !opts.uninstall &&
+    !opts.all &&
+    !opts.app &&
+    !opts.completions &&
+    !opts.skill &&
+    !opts.mcp &&
+    !opts.reinstall &&
+    !opts.update &&
+    !opts.status;
+
+  if (configureOnlyInstall) {
     return null;
   }
-  if (opts.quiet && opts.dry) {
-    return "--quiet cannot be combined with --dry.";
-  }
-  if (opts.quiet && !opts.yes && !opts.json && !opts.reinstall && !opts.update) {
-    return "--quiet requires --yes (or --json / --reinstall / --update).";
-  }
+
   if (opts.json) {
     opts.yes = true;
   }
   if (opts.reinstall) {
-    opts.bin = true;
     opts.yes = true;
   }
   if (opts.update) {
@@ -72,13 +73,14 @@ export function validateInstallOpts(opts: InstallOpts): string | null {
 
   const mutationFlags =
     opts.all ||
-    opts.bin ||
+    opts.app ||
     opts.completions ||
     opts.skill ||
     opts.mcp ||
     opts.reinstall ||
     opts.update ||
-    opts.uninstall;
+    opts.uninstall ||
+    opts.configure;
   if (opts.status && mutationFlags) {
     return "--status is mutually exclusive with install/reinstall/uninstall targets.";
   }
@@ -90,20 +92,22 @@ export function validateInstallOpts(opts: InstallOpts): string | null {
       opts.mcp ||
       opts.uninstall ||
       opts.status ||
-      opts.update)
+      opts.update ||
+      opts.configure)
   ) {
     return "--reinstall cannot be combined with other target flags.";
   }
   if (
     opts.update &&
     (opts.all ||
-      opts.bin ||
+      opts.app ||
       opts.completions ||
       opts.skill ||
       opts.mcp ||
       opts.uninstall ||
       opts.status ||
-      opts.reinstall)
+      opts.reinstall ||
+      opts.configure)
   ) {
     return "--update cannot be combined with other target flags.";
   }
@@ -112,35 +116,81 @@ export function validateInstallOpts(opts: InstallOpts): string | null {
   }
   if (opts.uninstall) {
     const hasUninstallTarget =
-      opts.all || opts.bin || opts.completions || opts.skill || opts.mcp || opts.config;
+      opts.all || opts.app || opts.completions || opts.skill || opts.mcp || opts.configure;
     if (!hasUninstallTarget) {
-      return "Specify at least one target: --all, --bin, --completions, --skill, --mcp, or --config.";
+      return "Specify at least one target: --all, --app, --completions, --skill, --mcp, or --configure.";
     }
   }
   if (!opts.status && !opts.reinstall && !opts.update && !opts.uninstall) {
     const hasTarget =
-      opts.all || opts.bin || opts.completions || opts.skill || opts.mcp || opts.config;
+      opts.all || opts.app || opts.completions || opts.skill || opts.mcp || opts.configure;
     if (!hasTarget) {
-      return "Specify at least one target: --all, --bin, --completions, --skill, --mcp, or --config.";
+      return "Specify at least one target: --all, --app, --completions, --skill, --mcp, or --configure.";
     }
   }
   return null;
 }
 
-function promptConfirm(): boolean {
-  process.stderr.write("Continue? [y/N] ");
+function parseSelectionIndices(input: string, max: number): number[] | null {
+  const trimmed = input.trim();
+  if (!trimmed) return null;
+  const parts = trimmed.split(/[\s,]+/).filter(Boolean);
+  const indices: number[] = [];
+  for (const part of parts) {
+    const n = Number(part);
+    if (!Number.isInteger(n) || n < 1 || n > max) {
+      return null;
+    }
+    if (!indices.includes(n)) {
+      indices.push(n);
+    }
+  }
+  return indices.sort((a, b) => a - b);
+}
+
+/** Interactive install: item 1 is the app when the plan leads with `app`. */
+export function interactiveSelectionAssumesApp(
+  actions: Array<InstallAction | UninstallAction>,
+  uninstall: boolean,
+): boolean {
+  if (uninstall) return false;
+  const first = actions[0];
+  return first !== undefined && "kind" in first && first.kind === "app";
+}
+
+export function mergeInteractiveSelection(
+  indices: number[],
+  count: number,
+  alwaysIncludeFirst: boolean,
+): number[] {
+  if (!alwaysIncludeFirst || count === 0 || indices.includes(1)) {
+    return indices;
+  }
+  return [1, ...indices].sort((a, b) => a - b);
+}
+
+function promptSelectiveConfirm(count: number, alwaysIncludeFirst: boolean): number[] | null {
+  const hint = alwaysIncludeFirst
+    ? "Proceed [y/N], or numbers to include (e.g. 2,3): "
+    : "Proceed [y/N], or numbers to include (e.g. 1,3): ";
+  process.stderr.write(hint);
   const buf = Buffer.alloc(256);
   const n = readSync(0, buf, { length: 256 });
   const ans = buf.toString("utf8", 0, n).trim();
-  return ans === "y" || ans === "Y";
+  if (ans === "y" || ans === "Y") {
+    return Array.from({ length: count }, (_, i) => i + 1);
+  }
+  if (ans === "n" || ans === "N" || ans === "") {
+    return null;
+  }
+  const parsed = parseSelectionIndices(ans, count);
+  if (!parsed) return null;
+  return mergeInteractiveSelection(parsed, count, alwaysIncludeFirst);
 }
 
-function runSkillAction(
-  root: CliProgram,
-  kind: "cursor-skill" | "claude-skill",
-  opts: InstallOpts,
-): string[] {
-  const target = kind === "cursor-skill" ? "cursor" : "claude";
+function runSkillAction(root: CliProgram, kind: InstallActionKind, opts: InstallOpts): string[] {
+  const target = skillTargetFromActionKind(kind);
+  if (!target) return [];
   return cliSkillInstall(root, target, {
     global: true,
     rimraf: true,
@@ -152,29 +202,44 @@ function executePlan(
   root: CliProgram,
   actions: Array<InstallAction | UninstallAction>,
   opts: InstallOpts,
+  showProgress: boolean,
 ): string[] {
   const changed: string[] = [];
+  const paths = resolveInstallPaths(root);
   for (const action of actions) {
-    installInfo(action.message, opts);
-    if ("kind" in action && (action.kind === "cursor-skill" || action.kind === "claude-skill")) {
-      changed.push(...runSkillAction(root, action.kind, opts));
-      continue;
+    if (showProgress) {
+      installInfo(action.message, opts);
     }
-    if (!("kind" in action) && action.summary.startsWith("cursor skill")) {
-      const paths = resolveInstallPaths(root, opts);
-      changed.push(...uninstallSkillDir(paths.cursorSkillDir, !!opts.dry));
-      continue;
+    if ("kind" in action && action.kind) {
+      const skillTarget = skillTargetFromActionKind(action.kind);
+      if (skillTarget) {
+        changed.push(...runSkillAction(root, action.kind as InstallActionKind, opts));
+        continue;
+      }
     }
-    if (!("kind" in action) && action.summary.startsWith("claude skill")) {
-      const paths = resolveInstallPaths(root, opts);
-      changed.push(...uninstallSkillDir(paths.claudeSkillDir, !!opts.dry));
-      continue;
+    if (!("kind" in action) || !action.kind) {
+      const skillDir = skillDirFromUninstallSummary(action.summary, paths);
+      if (skillDir) {
+        changed.push(...uninstallSkillDir(skillDir, !!opts.dry));
+        continue;
+      }
     }
     changed.push(...action.run());
   }
   return changed;
 }
 
+function shouldRunConfigureWizardAfterInstall(root: CliProgram, opts: InstallOpts): boolean {
+  if (opts.uninstall || opts.reinstall || opts.update || opts.status || !root.appConfig) {
+    return false;
+  }
+  if (!opts.all && !opts.app && !opts.completions && !opts.skill && !opts.mcp) {
+    return false;
+  }
+  const effective = resolveEffectiveInstallTargets(root.install, root);
+  return effective.configure.includedInAll && effective.configure.enabled;
+}
+
 /** Runs install/reinstall/uninstall mutations without exiting the process. */
 export async function runInstallMutation(
   root: CliProgram,
@@ -184,49 +249,20 @@ export async function runInstallMutation(
   opts: InstallOpts;
   paths: ReturnType<typeof resolveInstallPaths>;
 }> {
-  const opts = parseInstallOpts(rawOpts);
+  const normalized = normalizeInstallRawOpts(rawOpts);
+  const opts = parseInstallOpts(normalized);
   const err = validateInstallOpts(opts);
   if (err) {
     throw new Error(err);
   }
 
-  const paths = resolveInstallPaths(root, opts);
+  const paths = resolveInstallPaths(root);
 
   if (opts.status) {
     printInstallStatus(root, opts);
     return { changed: [], opts, paths };
   }
 
-  if (!opts.uninstall && resolveCapabilities(root).mcp && (opts.all || opts.mcp)) {
-    const entry = expectedMcpEntry(root);
-    const openCodeEntry = expectedOpenCodeMcpEntry(root);
-    const yes = !!opts.yes;
-    for (const p of [
-      paths.cursorMcpPath,
-      paths.claudeMcpPath,
-      paths.claudeDesktopMcpPath,
-      paths.chatGptMcpPath,
-    ]) {
-      const conflict = checkMcpConflict(p, paths.mcpName, entry, yes);
-      if (conflict) {
-        throw new Error(conflict);
-      }
-    }
-    const openCodeConflict = checkOpenCodeMcpConflict(
-      paths.opencodeMcpPath,
-      paths.mcpName,
-      openCodeEntry,
-      yes,
-    );
-    if (openCodeConflict) {
-      throw new Error(openCodeConflict);
-    }
-    const codexConflict = checkCodexMcpConflict(userHome(), paths.mcpName, entry, yes);
-    if (codexConflict) {
-      throw new Error(codexConflict);
-    }
-  }
-
   let actions: Array<InstallAction | UninstallAction>;
   if (opts.uninstall) {
     actions = buildUninstallPlan(root, paths, opts);
@@ -236,31 +272,48 @@ export async function runInstallMutation(
     actions = buildInstallPlan(root, paths, opts);
   }
 
+  const installActions = actions.filter(
+    (a): a is InstallAction => "kind" in a && typeof a.kind === "string" && a.kind.includes("-mcp"),
+  );
+  if (!opts.uninstall && resolveCapabilities(root).mcp && installActions.length > 0) {
+    runTargetPreflight(root, paths, opts, installActions);
+  }
+
+  let selectedActions = actions;
+  const autoYes = !!(opts.yes || opts.json || opts.reinstall || opts.update || opts.dry);
+
+  if (!autoYes && !opts.json && process.stdin.isTTY) {
+    writeInteractiveInstallIntro(root);
+  }
+
   if (actions.length === 0) {
     return { changed: [], opts, paths };
   }
 
-  if (!opts.quiet && !opts.json) {
+  if (!autoYes) {
     installOut(
       `About to ${opts.uninstall ? "remove" : opts.reinstall ? "reinstall" : "install"}:`,
       opts,
     );
-    for (const a of actions) {
-      installOut(`  - ${a.summary}`, opts);
+    for (let i = 0; i < actions.length; i++) {
+      installOut(`  ${i + 1}. ${actions[i]?.summary}`, opts);
     }
   }
 
-  const autoYes = opts.yes || opts.json || opts.reinstall;
   if (!autoYes) {
     if (!process.stdin.isTTY) {
       throw new Error("Refusing to proceed without --yes (stdin is not a TTY).");
     }
-    if (!promptConfirm()) {
+    const assumeApp = interactiveSelectionAssumesApp(actions, !!opts.uninstall);
+    const selection = promptSelectiveConfirm(actions.length, assumeApp);
+    if (!selection) {
       throw new Error("Aborted.");
     }
+    process.stderr.write("Done.\n");
+    selectedActions = selection.map((i) => actions[i - 1]).filter(Boolean) as typeof actions;
   }
 
-  const changed = executePlan(root, actions, opts);
+  const changed = executePlan(root, selectedActions, opts, autoYes);
   return { changed, opts, paths };
 }
 
@@ -269,14 +322,27 @@ export async function cliInstall(
   root: CliProgram,
   rawOpts: Record<string, string>,
 ): Promise<never> {
-  const opts = parseInstallOpts(rawOpts);
+  const normalized = normalizeInstallRawOpts(rawOpts);
+  const opts = parseInstallOpts(normalized);
 
-  if (opts.configure) {
+  const configureOnly =
+    opts.configure &&
+    !opts.uninstall &&
+    !opts.all &&
+    !opts.app &&
+    !opts.completions &&
+    !opts.skill &&
+    !opts.mcp;
+
+  if (configureOnly) {
     if (!root.appConfig) {
-      installErr("install --configure requires program.appConfig on the program root.");
+      installErr("This app does not support --configure.");
       process.exit(1);
     }
-    const result = runInstallConfigure(root);
+    if (process.stdin.isTTY) {
+      writeInteractiveInstallIntro(root);
+    }
+    const result = runInstallConfigure(root, { context: "standalone" });
     if (result.changed) {
       installOut(`Wrote config: ${displayAppConfigPath(root)}`, opts);
     } else {
@@ -293,7 +359,7 @@ export async function cliInstall(
 
   if (opts.update) {
     if (!resolveCapabilities(root).update) {
-      installErr("install --update requires install.updateGetLatest on the program root.");
+      installErr("Remote updates are not supported by this app.");
       process.exit(1);
     }
     await cliUpdate(root);
@@ -301,7 +367,7 @@ export async function cliInstall(
 
   let result: Awaited<ReturnType<typeof runInstallMutation>>;
   try {
-    result = await runInstallMutation(root, rawOpts);
+    result = await runInstallMutation(root, normalized);
   } catch (err) {
     installErr(err instanceof Error ? err.message : String(err));
     process.exit(1);
@@ -313,12 +379,23 @@ export async function cliInstall(
     process.exit(0);
   }
 
+  if (
+    shouldRunConfigureWizardAfterInstall(root, mutationOpts) &&
+    !mutationOpts.uninstall &&
+    !mutationOpts.dry
+  ) {
+    const configResult = runInstallConfigure(root, { context: "after-install" });
+    if (configResult.changed) {
+      installOut(`Wrote config: ${displayAppConfigPath(root)}`, mutationOpts);
+    }
+  }
+
   if (mutationOpts.json) {
     process.stdout.write(`${JSON.stringify(changed, null, 2)}\n`);
     process.exit(0);
   }
 
-  if (!mutationOpts.quiet && changed.length > 0) {
+  if (changed.length > 0) {
     const verb = mutationOpts.uninstall
       ? "Removed"
       : mutationOpts.reinstall
@@ -327,8 +404,8 @@ export async function cliInstall(
     installOut(`${verb} ${changed.length} file(s).`, mutationOpts);
     if (
       !mutationOpts.uninstall &&
-      (mutationOpts.all || mutationOpts.bin) &&
-      changed.some((p) => p === paths.bashRc || p === paths.zshRc || p === paths.binaryPath)
+      (mutationOpts.all || mutationOpts.app) &&
+      changed.some((p) => p === paths.bashRc || p === paths.zshRc || p === paths.appPath)
     ) {
       installOut("Open a new shell, or run: hash -r (bash) / rehash (zsh)", mutationOpts);
     }

package/src/install/install-validate.test.ts

@@ -0,0 +1,61 @@
+import { describe, expect, test } from "bun:test";
+import type { CliProgram } from "../types.ts";
+import { CliSchemaValidationError } from "../types.ts";
+import { cliValidateProgram } from "../validate.ts";
+
+const base: CliProgram = {
+  key: "app",
+  version: "1.0.0",
+  description: "Test",
+  handler: () => {},
+};
+
+describe("validateInstallConfig", () => {
+  test("accepts empty install config", () => {
+    expect(() => cliValidateProgram(base)).not.toThrow();
+  });
+
+  test("rejects removed allSkills shorthand", () => {
+    const program = {
+      ...base,
+      install: { targets: { allSkills: true } },
+    } as CliProgram;
+    expect(() => cliValidateProgram(program)).toThrow(CliSchemaValidationError);
+    expect(() => cliValidateProgram(program)).toThrow(/allSkills/);
+  });
+
+  test("rejects both sides of pair without both integration", () => {
+    const program: CliProgram = {
+      ...base,
+      mcpServer: { enabled: true },
+      install: {
+        agentIntegration: "mcp",
+        targets: { cursorMcp: true, cursorSkill: true },
+      },
+    };
+    expect(() => cliValidateProgram(program)).toThrow(/both/);
+  });
+
+  test("rejects explicit MCP target in skill integration mode", () => {
+    const program: CliProgram = {
+      ...base,
+      install: {
+        agentIntegration: "skill",
+        targets: { cursorMcp: true },
+      },
+    };
+    expect(() => cliValidateProgram(program)).toThrow(/cursorMcp/);
+  });
+
+  test("allows explicit pair targets with both integration", () => {
+    const program: CliProgram = {
+      ...base,
+      mcpServer: { enabled: true },
+      install: {
+        agentIntegration: "both",
+        targets: { cursorMcp: true, cursorSkill: true },
+      },
+    };
+    expect(() => cliValidateProgram(program)).not.toThrow();
+  });
+});