iterate-skill-installer 2.4.1 → 2.4.3

package/README.md

@@ -26,6 +26,9 @@ npx iterate-skill-installer --target ./my-project
 
 # Force overwrite existing skill files
 npx iterate-skill-installer --force
+
+# Skill-only install — skip installing the iterate CLI
+npx iterate-skill-installer --no-cli
 ```
 
 ## What it does
@@ -42,12 +45,11 @@ npx iterate-skill-installer --force
 7. Installs the `iterate` CLI onto your PATH (prefers `pipx`, otherwise
    `pip install --user`) so you can run `iterate onboard` directly.
 
-> **Note:** the installer always puts the `iterate` CLI on your PATH so a single
+> **Note:** the installer normally puts the `iterate` CLI on your PATH so a single
 > command gives you both the skill and the CLI. If you **don't** want the CLI
-> auto-installed, use a manual install instead — copy `SKILL.md` + `config/` +
-> `scripts/` + `templates/` to your assistant dir, or run
-> `python scripts/install.py install` directly. If the CLI install fails, you can
-> still install it later with `pipx install .` or `pip install .` from a checkout.
+> auto-installed, pass `--no-cli` to install the skill only — you can install the
+> CLI later with `pipx install .` or `pip install .` from a checkout. If the CLI
+> install fails, you can still install it later the same way.
 
 ## Supported AI assistants
 

package/README.zh-CN.md

@@ -26,6 +26,9 @@ npx iterate-skill-installer --target ./my-project
 
 # 强制覆盖已有技能文件
 npx iterate-skill-installer --force
+
+# 仅安装技能 —— 跳过安装 iterate CLI
+npx iterate-skill-installer --no-cli
 ```
 
 ## 它做了什么
@@ -42,11 +45,10 @@ npx iterate-skill-installer --force
 7. 把 `iterate` CLI 安装到你的 PATH（优先 `pipx`，否则 `pip install --user`），
    以便你直接运行 `iterate onboard`。
 
-> **注意：** 安装器总是会把你 PATH 上的 `iterate` CLI 装好，所以一条命令就能同时
-> 得到技能和 CLI。如果**不**想自动安装 CLI，请改用手动安装——把
-> `SKILL.md` + `config/` + `scripts/` + `templates/` 复制到你的助手目录，或直接运行
-> `python scripts/install.py install`。如果 CLI 安装失败，你之后仍可从检出目录用
-> `pipx install .` 或 `pip install .` 补装。
+> **注意：** 安装器通常会在你 PATH 上装好 `iterate` CLI，这样一条命令就能同时
+> 得到技能和 CLI。如果**不**想自动安装 CLI，请加 `--no-cli` 只装技能——之后可从
+> 检出目录用 `pipx install .` 或 `pip install .` 补装。如果 CLI 安装失败，你之后
+> 也可用同样的方式补装。
 
 ## 支持的 AI 助手
 

package/bin/cli.js

@@ -10,87 +10,14 @@
  *   npx iterate-skill-installer --ai trae
  *   npx iterate-skill-installer --target /path/to/project
  *   npx iterate-skill-installer --global
+ *   npx iterate-skill-installer --no-cli
  *
  * The installer downloads the latest GitHub release, verifies its SHA256
  * checksum, then delegates file copying and assistant selection to the
  * Python install script bundled in the release.
  */
 
-const { main } = require('../lib/installer');
-
-function parseArgs(argv) {
-  const options = {
-    global: true,
-    ai: null,
-    target: null,
-    force: false,
-    token: process.env.GITHUB_TOKEN || null,
-    // Track whether the user explicitly chose global/project mode so the
-    // installer can later ask about the current directory's project intent.
-    globalExplicit: false,
-    targetExplicit: false,
-  };
-
-  for (let i = 0; i < argv.length; i++) {
-    const arg = argv[i];
-    const next = argv[i + 1];
-
-    switch (arg) {
-      case '--ai':
-        if (!next) {
-          console.error('Error: --ai requires a value');
-          process.exit(1);
-        }
-        options.ai = next;
-        i++;
-        break;
-      case '--target':
-        if (!next) {
-          console.error('Error: --target requires a value');
-          process.exit(1);
-        }
-        options.target = next;
-        options.global = false;
-        options.targetExplicit = true;
-        i++;
-        break;
-      case '--global':
-        options.global = true;
-        options.target = null;
-        options.globalExplicit = true;
-        break;
-      case '--force':
-        options.force = true;
-        break;
-      case '--token':
-        if (!next) {
-          console.error('Error: --token requires a value');
-          process.exit(1);
-        }
-        options.token = next;
-        i++;
-        break;
-      case '-h':
-      case '--help':
-        printHelp();
-        process.exit(0);
-        break;
-      case '-v':
-      case '--version':
-        console.log(require('../package.json').version);
-        process.exit(0);
-        break;
-      default:
-        if (arg.startsWith('-')) {
-          console.error(`Error: unknown option ${arg}`);
-          process.exit(1);
-        }
-        break;
-    }
-  }
-
-  return options;
-}
+const { main, parseArgs } = require('../lib/installer');
 
 function printHelp() {
   console.log(`
@@ -101,6 +28,7 @@ Options:
   --target <path>     Install into a project directory instead of the global home dir.
   --global            Install into the user's home directory (default).
   --force             Overwrite existing skill files.
+  --no-cli            Skip installing the iterate CLI (skill-only install).
   --token <token>     GitHub token for higher API rate limits.
   -h, --help          Show this help message.
   -v, --version       Show version.
@@ -109,6 +37,7 @@ Examples:
   npx iterate-skill-installer
   npx iterate-skill-installer --ai trae
   npx iterate-skill-installer --target ./my-project
+  npx iterate-skill-installer --no-cli
 `);
 }
 

package/lib/installer.js

@@ -364,6 +364,84 @@ async function installRequirements(pythonBin, requirementsPath) {
   await runCommand(pythonBin, ['-m', 'pip', 'install', '--quiet', '--requirement', requirementsPath]);
 }
 
+/**
+ * Parse the installer's command-line arguments.
+ *
+ * Kept as a pure function (exported) so the flag surface — including the
+ * skill-only ``--no-cli`` switch — can be unit-tested without spawning the
+ * full download-and-install pipeline. ``process.exit`` is only reached for
+ * hard argument errors (e.g. a value-consuming flag missing its value).
+ */
+function parseArgs(argv) {
+  const options = {
+    global: true,
+    ai: null,
+    target: null,
+    force: false,
+    noCli: false,
+    token: process.env.GITHUB_TOKEN || null,
+    // Track whether the user explicitly chose global/project mode so the
+    // installer can later ask about the current directory's project intent.
+    globalExplicit: false,
+    targetExplicit: false,
+  };
+
+  for (let i = 0; i < argv.length; i++) {
+    const arg = argv[i];
+    const next = argv[i + 1];
+
+    switch (arg) {
+      case '--ai':
+        if (!next) {
+          console.error('Error: --ai requires a value');
+          process.exit(1);
+        }
+        options.ai = next;
+        i++;
+        break;
+      case '--target':
+        if (!next) {
+          console.error('Error: --target requires a value');
+          process.exit(1);
+        }
+        options.target = next;
+        options.global = false;
+        options.targetExplicit = true;
+        i++;
+        break;
+      case '--global':
+        options.global = true;
+        options.target = null;
+        options.globalExplicit = true;
+        break;
+      case '--force':
+        options.force = true;
+        break;
+      case '--no-cli':
+        // Skill-only install: skip the automated `iterate` CLI install so a
+        // user who only wants the skill is not surprised by a global install.
+        options.noCli = true;
+        break;
+      case '--token':
+        if (!next) {
+          console.error('Error: --token requires a value');
+          process.exit(1);
+        }
+        options.token = next;
+        i++;
+        break;
+      default:
+        if (arg.startsWith('-')) {
+          console.error(`Error: unknown option ${arg}`);
+          process.exit(1);
+        }
+        break;
+    }
+  }
+
+  return options;
+}
+
 async function main(options = {}) {
   printBanner();
 
@@ -372,7 +450,14 @@ async function main(options = {}) {
   // whether that directory is the target project.
   options = await resolveInstallMode(options);
 
-  const { global: globalInstall = true, ai = null, token = null, target = null, force = false } = options;
+  const {
+    global: globalInstall = true,
+    ai = null,
+    token = null,
+    target = null,
+    force = false,
+    noCli = false,
+  } = options;
 
   step('Checking environment');
   const pythonBin = await findPython();
@@ -495,8 +580,18 @@ async function main(options = {}) {
     }
     success('iterate-skill installation finished.');
 
-    // Install the iterate CLI so `iterate onboard` works right after install.
-    await installCli(pythonBin, sourceDir);
+    if (noCli) {
+      // Skill-only install: explain how to add the CLI later instead of
+      // silently installing it globally.
+      hint('Skipped installing the iterate CLI (--no-cli).');
+      hint('Install it later for `iterate onboard` with:  pipx install <repo>  (or)  pip install .');
+    } else {
+      // Install the iterate CLI so `iterate onboard` works right after
+      // install. Made explicit up front so skill-only users are not surprised
+      // by a global CLI install (skip with --no-cli).
+      step('Installing the iterate CLI (skip with --no-cli)');
+      await installCli(pythonBin, sourceDir);
+    }
 
     frameSection('Done', [
       `\x1b[32m✓\x1b[0m iterate-skill ${tag} installed`,
@@ -510,6 +605,7 @@ async function main(options = {}) {
 
 module.exports = {
   main,
+  parseArgs,
   ITERATE_BANNER,
   InstallerError,
   resolveInstallMode,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "iterate-skill-installer",
-  "version": "2.4.1",
+  "version": "2.4.3",
   "description": "One-command installer for iterate-skill across AI coding assistants",
   "main": "lib/installer.js",
   "bin": {

package/test/mode.test.js

@@ -9,7 +9,7 @@ const assert = require('node:assert');
 const fs = require('node:fs');
 const path = require('node:path');
 const os = require('node:os');
-const { resolveInstallMode, parseChecksums } = require('../lib/installer');
+const { resolveInstallMode, parseChecksums, parseArgs } = require('../lib/installer');
 
 const originalCwd = process.cwd();
 const originalHome = os.homedir;
@@ -97,6 +97,21 @@ async function run() {
   assert.strictEqual(multi.get('b.txt'), 'bb');
 
   console.log('mode.test.js: all parseChecksums tests passed');
+
+  // parseArgs: --no-cli must toggle skill-only mode, and default to off.
+  assert.strictEqual(parseArgs([]).noCli, false, 'noCli should default to false');
+  assert.strictEqual(parseArgs(['--no-cli']).noCli, true, '--no-cli should set noCli');
+  assert.strictEqual(parseArgs(['--no-cli', '--force', '--global']).noCli, true, 'noCli persists with other flags');
+  assert.strictEqual(parseArgs(['--global']).noCli, false, 'unrelated flags do not set noCli');
+
+  // parseArgs: unrelated flag surface is preserved (target/global/ai/force).
+  const combo = parseArgs(['--ai', 'trae', '--target', '/proj', '--force']);
+  assert.strictEqual(combo.ai, 'trae', 'ai should parse');
+  assert.strictEqual(combo.target, '/proj', 'target should parse');
+  assert.strictEqual(combo.force, true, 'force should parse');
+  assert.strictEqual(combo.global, false, 'target forces project mode');
+
+  console.log('mode.test.js: all parseArgs tests passed');
 }
 
 run().catch((err) => {