@benjaminthomas/spec-driven-dev 1.5.0 → 1.5.1

package/README.md

@@ -375,6 +375,13 @@ run one after another, stopping at the first failure. Two steps always wait for
 spec, and pushing or opening the PR. You don't need slash commands either. Saying "investigate
 STP-123" or "ship it" picks the right skill.
 
+**Setup didn't ask me anything.**
+Update to 1.5.1 or later (`npx @benjaminthomas/spec-driven-dev@latest --version`). Earlier versions
+crashed on Node 14 or 16, which nvm often switches to inside older projects, and wrote a config
+silently when there was no terminal. Now it runs on Node 14+. With no terminal to answer in (a `!`
+command in Claude Code, an editor task, piped input), it stops, changes nothing, and tells you to run
+it in a terminal window or pass flags such as `--layout=multi-repo --repos=…`.
+
 **Setup didn't ask me about repos.**
 It only asks about repos for a multi-repo or monorepo layout. If it detected `single`, type
 `multi-repo` at the layout question, then add your other repos by path. If it asked nothing at all,
@@ -455,7 +462,8 @@ can otherwise reuse an older copy it cached. What changed in each version is in
 `npx github:benjaminthomas/spec-driven-dev` installs straight from GitHub instead, for example to
 try an unreleased commit on `main`.
 
-**Requirements:** Node.js 18 or newer and git. The installer has no dependencies.
+**Requirements:** Node.js 14 or newer and git. The installer has no dependencies, and CI tests it on
+Node 14, 16, 18, 20, 22, and 24.
 
 ### Claude Code (plugin, with automatic updates)
 

package/bin/cli.mjs

@@ -13,7 +13,7 @@ import fs from "node:fs";
 import os from "node:os";
 import path from "node:path";
 import { execFileSync } from "node:child_process";
-import { createInterface } from "node:readline/promises";
+import { createInterface } from "node:readline";  // callback API: readline/promises needs Node 17+
 import { fileURLToPath } from "node:url";
 
 const ROOT = path.dirname(path.dirname(fileURLToPath(import.meta.url)));
@@ -40,8 +40,21 @@ if (flag("version") || flag("v")) {
   console.log(JSON.parse(fs.readFileSync(path.join(ROOT, "package.json"), "utf8")).version);
   process.exit(0);
 }
-if (["install", "setup"].includes(command) && !process.stdin.isTTY && !flag("yes") && !flag("y"))
-  console.log("• No interactive terminal detected, so no questions will be asked: using the detected values.\n  Run it directly in a terminal to answer them, or pass flags (--layout, --repos, --tools …). Add --yes to hide this note.");
+// No terminal to answer in (e.g. `!` in Claude Code, an IDE task, a pipe) and nothing passed explicitly:
+// stop rather than write a config from guesses.
+const ANSWER_FLAGS = ["yes", "y", "layout", "repos", "packages", "tools", "workspace", "after-approval", "skills", "global", "no-setup"];
+if (["install", "setup"].includes(command) && !process.stdin.isTTY && !args.some((a) => ANSWER_FLAGS.some((f) => a === `--${f}` || a.startsWith(`--${f}=`)))) {
+  console.log(`✗ No interactive terminal, so setup can't ask its questions. Nothing was changed.
+
+  This happens when it runs through another program: a "!" command in Claude Code, an editor task,
+  or piped input. Either:
+    • run it yourself in a terminal window (Terminal, iTerm, or your editor's terminal panel):
+        npx @benjaminthomas/spec-driven-dev@latest ${command === "setup" ? "setup" : ""}
+    • or give the answers as flags, for example:
+        npx @benjaminthomas/spec-driven-dev@latest setup --layout=multi-repo --repos=~/Projects/frontend,~/Projects/backend
+      (add --yes to accept everything it detects)`);
+  process.exit(1);
+}
 
 if (flag("help") || flag("h")) {
   console.log(fs.readFileSync(fileURLToPath(import.meta.url), "utf8").split("\n").slice(1, 11).join("\n").replace(/^\/\/ ?/gm, ""));
@@ -51,7 +64,7 @@ if (flag("help") || flag("h")) {
 const rl = YES ? null : createInterface({ input: process.stdin, output: process.stdout });
 async function ask(q, def = "") {
   if (YES) return def;
-  const a = (await rl.question(`${q}${def ? ` [${def}]` : ""}: `)).trim();
+  const a = (await new Promise((resolve) => rl.question(`${q}${def ? ` [${def}]` : ""}: `, resolve))).trim();
   return a || def;
 }
 async function pick(q, items, defaults) {
@@ -68,6 +81,17 @@ const run = (cmd, argv, cwd) => {
 };
 const onPath = (bin) => !!run(process.platform === "win32" ? "where" : "which", [bin]);
 
+// fs.cpSync needs Node 16.7+; this keeps Node 14 working. Follows symlinks, skips .DS_Store.
+function copyDir(from, to) {
+  fs.mkdirSync(to, { recursive: true });
+  for (const e of fs.readdirSync(from)) {
+    if (e === ".DS_Store") continue;
+    const src = path.join(from, e), dst = path.join(to, e);
+    if (fs.statSync(src).isDirectory()) copyDir(src, dst);
+    else fs.copyFileSync(src, dst);
+  }
+}
+
 // ---------- install ----------
 async function install() {
   const detected = Object.keys(TOOLS).filter((t) => fs.existsSync(path.join(HOME, TOOLS[t].home)) || onPath(TOOLS[t].bin));
@@ -96,7 +120,7 @@ async function install() {
     for (const s of skills) {
       const to = path.join(dest, s);
       fs.rmSync(to, { recursive: true, force: true });
-      fs.cpSync(path.join(SKILLS_DIR, s), to, { recursive: true, dereference: true, filter: (f) => !f.endsWith(".DS_Store") });
+      copyDir(path.join(SKILLS_DIR, s), to);
     }
     console.log(`✓ ${skills.length} skills → ${dest}`);
   }
@@ -247,7 +271,7 @@ async function chooseRepos(d) {
     const found = findRepos(scanRoot);
     if (found.length) {
       const owners = {};
-      for (const r of found) (owners[gitFacts(path.join(scanRoot, r))?.owner || "?"] ||= []).push(r);
+      for (const r of found) { const o = gitFacts(path.join(scanRoot, r))?.owner || "?"; (owners[o] = owners[o] || []).push(r); }
       const biggest = Object.values(owners).sort((a, b) => b.length - a.length)[0] || [];
       const here = insideRepo ? [path.relative(scanRoot, d.root)] : [];
       console.log(`Found ${found.length} git repos under ${scanRoot}. Which ones belong to this workspace?`);

package/bin/spec-driven-dev.cjs

@@ -0,0 +1,10 @@
+#!/usr/bin/env node
+// Launcher: plain CommonJS so it runs on any Node and can explain a too-old version,
+// instead of the ESM entry failing with a syntax error.
+const major = Number(process.versions.node.split(".")[0]);
+if (major < 14) {
+  console.error(`✗ spec-driven-dev needs Node.js 14 or newer; this is ${process.version}.`);
+  console.error("  If you use nvm, run it from a folder without an old .nvmrc, or: nvm exec 20 npx @benjaminthomas/spec-driven-dev@latest");
+  process.exit(1);
+}
+import("./cli.mjs").catch((e) => { console.error(e); process.exit(1); });

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "@benjaminthomas/spec-driven-dev",
-  "version": "1.5.0",
+  "version": "1.5.1",
   "description": "Spec-driven AI development workflow (investigate → spec → implement → verify → ship) as Agent Skills for Claude Code, Cursor, Codex CLI and Antigravity.",
   "type": "module",
-  "bin": { "spec-driven-dev": "bin/cli.mjs" },
+  "bin": { "spec-driven-dev": "bin/spec-driven-dev.cjs" },
   "files": ["bin", "skills"],
   "scripts": { "test": "python3 scripts/check.py && node scripts/test-cli.mjs" },
-  "engines": { "node": ">=18" },
+  "engines": { "node": ">=14" },
   "keywords": ["agent-skills", "claude-code", "cursor", "codex", "antigravity", "spec-driven-development", "ai-agents"],
   "repository": { "type": "git", "url": "git+https://github.com/benjaminthomas/spec-driven-dev.git" },
   "license": "MIT",