codebase-snapshot 0.9.4 → 0.9.5

package/CHANGELOG.md

@@ -21,6 +21,44 @@ Two versions exist on the registry that are not releases, and both are deprecate
 Deleting them requires account-level rights that the release pipeline does not have, so they are
 deprecated with an explanatory message instead. `latest` has pointed at 0.9.0 since it existed.
 
+## 0.9.5
+
+### Every question in the wizard is reversible
+
+The first run asked five questions and accepted one answer each. Reaching the part-size question and
+changing your mind meant killing the process and starting again — and at the review screen,
+answering **No** to "Write this and continue?" threw the same cancellation Ctrl+C does, so "that is
+not what I meant" ended the run instead of correcting it.
+
+- **`←` goes back** on any list or text field, `b` on any list or confirmation. Each footer names
+  the keys that question actually accepts: `text` takes `←` alone (`b` there is a character you are
+  typing) and `confirm` takes `b` alone (`←`/`→` already mean yes/no).
+- **Back is a step, not a restart.** The five questions are now steps — scope → part size →
+  artifacts → credentials → review. Going back re-asks the previous one with your earlier answer
+  still selected, and back from the first step returns to *"How should I set this up?"* rather than
+  out of the wizard.
+- **`No` at the review steps back.** It used to abort. Ctrl+C is still how you say stop.
+- **Answers are rebuilt, not mutated.** Re-entering the scope step clears the globs it added, so
+  choosing "skip tests", going back, and choosing "everything" no longer leaves `**/*.test.*` in
+  the written config.
+
+Four new tests drive real keypress events through a fake TTY, alongside the four that already
+covered the forward path.
+
+### A clean first run
+
+Setup printed a Node `DEP0190` deprecation warning after installing dependencies: `spawnSync` was
+given an args array *and* `shell: true`, which Node deprecated because the arguments are
+concatenated rather than escaped. The Windows path now hands the shell one pre-formed command
+string — both halves are fixed literals from `detectPackageManager` — and every other platform
+spawns with no shell at all.
+
+### Still untested here
+
+The MCP server has not been run against a real client. Its protocol is exercised against the built
+binary, but a client's own configuration, timeouts and approval prompts are theirs. That is the next
+thing to check, not something this release proves.
+
 ## 0.9.4
 
 ### The snapshot is now something an agent can ask for

package/README.md

@@ -197,7 +197,7 @@ Everything an agent needs is available as data, and the tool can install its own
 ```bash
 npx codebase-snapshot describe --format json   # contract: workflow, options, exit codes
 npx codebase-snapshot skill --install          # into .claude/skills + .cursor/rules
-npx -y codebase-snapshot@0.9.4 mcp             # or serve it over MCP
+npx -y codebase-snapshot@0.9.5 mcp             # or serve it over MCP
 ```
 
 An agent told only "use `npx codebase-snapshot`" can read the contract, follow its ordered

package/dist/bin.js

@@ -4346,7 +4346,7 @@ function gitignorePath(cwd, filePath) {
 
 // src/version.ts
 var GENERATOR_NAME = "codebase-snapshot";
-var GENERATOR_VERSION = "0.9.4";
+var GENERATOR_VERSION = "0.9.5";
 var LEGACY_GENERATOR_NAMES = ["@ankurhalder/codebase-snapshot"];
 function isOurGeneratorName(name) {
   if (typeof name !== "string") return false;
@@ -4487,6 +4487,12 @@ var PromptCancelled = class extends Error {
     this.name = "PromptCancelled";
   }
 };
+var PromptBack = class extends Error {
+  constructor() {
+    super("Prompt back");
+    this.name = "PromptBack";
+  }
+};
 function defaultIO() {
   return { input: process.stdin, output: process.stdout };
 }
@@ -4510,35 +4516,40 @@ function choiceLine(choice, active) {
   const hint = choice.hint ? `  ${choice.hint}` : "";
   return `  ${marker(active)} ${choice.label}${hint}`;
 }
-function renderSelect(message, choices, index) {
+function listFooter(base, allowBack) {
+  return allowBack ? `${base} \xB7 \u2190 back` : base;
+}
+function renderSelect(message, choices, index, allowBack = false) {
   const lines = [`\u25C6 ${message}`, ""];
   choices.forEach((choice, position) => lines.push(choiceLine(choice, position === index)));
-  lines.push("", "  \u2191\u2193 move \xB7 enter select");
+  lines.push("", `  ${listFooter("\u2191\u2193 move \xB7 enter select", allowBack)}`);
   return lines.join("\n");
 }
-function renderMultiSelect(message, choices, index, selected) {
+function renderMultiSelect(message, choices, index, selected, allowBack = false) {
   const lines = [`\u25C6 ${message}`, ""];
   choices.forEach((choice, position) => {
     const box = selected.has(position) ? "\u25FC" : "\u25FB";
     const hint = choice.hint ? `  ${choice.hint}` : "";
     lines.push(`  ${marker(position === index)} ${box} ${choice.label}${hint}`);
   });
-  lines.push("", "  \u2191\u2193 move \xB7 space toggle \xB7 enter confirm");
+  lines.push("", `  ${listFooter("\u2191\u2193 move \xB7 space toggle \xB7 enter confirm", allowBack)}`);
   return lines.join("\n");
 }
-function renderConfirm(message, value) {
+function renderConfirm(message, value, allowBack = false) {
+  const footer = allowBack ? "\u2190\u2192 or y/n \xB7 enter confirm \xB7 b back" : "\u2190\u2192 or y/n \xB7 enter confirm";
   return [
     `\u25C6 ${message}`,
     "",
     `  ${marker(value)} Yes`,
     `  ${marker(!value)} No`,
     "",
-    "  \u2190\u2192 or y/n \xB7 enter confirm"
+    `  ${footer}`
   ].join("\n");
 }
-function renderText(message, value, error) {
+function renderText(message, value, error, allowBack = false) {
   const lines = [`\u25C6 ${message}`, "", `  ${value || "\u258F"}`];
   if (error) lines.push("", `  \u2716 ${error}`);
+  if (allowBack) lines.push("", "  enter confirm \xB7 \u2190 back");
   return lines.join("\n");
 }
 function keyReader(input) {
@@ -4587,14 +4598,18 @@ var Frame = class {
 function isCancel(key) {
   return key.ctrl === true && key.name === "c" || key.ctrl === true && key.name === "d";
 }
+function isListBack(key, allowBack) {
+  return allowBack && (key.name === "left" || key.name === "b");
+}
 async function select(io, message, choices, options = {}) {
   if (choices.length === 0) throw new Error("select() requires at least one choice.");
+  const allowBack = options.allowBack === true;
   const fallbackIndex = Math.max(
     0,
     choices.findIndex((choice) => choice.value === options.defaultValue)
   );
   if (!isInteractive(io)) {
-    io.output.write(`${renderSelect(message, choices, fallbackIndex)}
+    io.output.write(`${renderSelect(message, choices, fallbackIndex, allowBack)}
 `);
     return choices[fallbackIndex].value;
   }
@@ -4603,9 +4618,10 @@ async function select(io, message, choices, options = {}) {
   let index = fallbackIndex;
   try {
     for (; ; ) {
-      frame.draw(renderSelect(message, choices, index));
+      frame.draw(renderSelect(message, choices, index, allowBack));
       const key = await reader.next();
       if (isCancel(key)) throw new PromptCancelled();
+      if (isListBack(key, allowBack)) throw new PromptBack();
       if (key.name === "up") index = moveSelection(index, -1, choices.length);
       else if (key.name === "down") index = moveSelection(index, 1, choices.length);
       else if (key.name === "return" || key.name === "enter") {
@@ -4614,18 +4630,22 @@ async function select(io, message, choices, options = {}) {
       } else if (key.name === "j") index = moveSelection(index, 1, choices.length);
       else if (key.name === "k") index = moveSelection(index, -1, choices.length);
     }
+  } catch (error) {
+    frame.erase();
+    throw error;
   } finally {
     reader.close();
   }
 }
 async function multiSelect(io, message, choices, options = {}) {
+  const allowBack = options.allowBack === true;
   const selected = /* @__PURE__ */ new Set();
   choices.forEach((choice, position) => {
     if (options.defaults?.includes(choice.value)) selected.add(position);
   });
   const result = () => choices.filter((_, position) => selected.has(position)).map((choice) => choice.value);
   if (!isInteractive(io)) {
-    io.output.write(`${renderMultiSelect(message, choices, 0, selected)}
+    io.output.write(`${renderMultiSelect(message, choices, 0, selected, allowBack)}
 `);
     return result();
   }
@@ -4634,9 +4654,10 @@ async function multiSelect(io, message, choices, options = {}) {
   let index = 0;
   try {
     for (; ; ) {
-      frame.draw(renderMultiSelect(message, choices, index, selected));
+      frame.draw(renderMultiSelect(message, choices, index, selected, allowBack));
       const key = await reader.next();
       if (isCancel(key)) throw new PromptCancelled();
+      if (isListBack(key, allowBack)) throw new PromptBack();
       if (key.name === "up") index = moveSelection(index, -1, choices.length);
       else if (key.name === "down") index = moveSelection(index, 1, choices.length);
       else if (key.name === "space") {
@@ -4651,14 +4672,18 @@ async function multiSelect(io, message, choices, options = {}) {
         return result();
       }
     }
+  } catch (error) {
+    frame.erase();
+    throw error;
   } finally {
     reader.close();
   }
 }
 async function confirm(io, message, options = {}) {
+  const allowBack = options.allowBack === true;
   let value = options.defaultValue ?? true;
   if (!isInteractive(io)) {
-    io.output.write(`${renderConfirm(message, value)}
+    io.output.write(`${renderConfirm(message, value, allowBack)}
 `);
     return value;
   }
@@ -4666,9 +4691,10 @@ async function confirm(io, message, options = {}) {
   const reader = keyReader(io.input);
   try {
     for (; ; ) {
-      frame.draw(renderConfirm(message, value));
+      frame.draw(renderConfirm(message, value, allowBack));
       const key = await reader.next();
       if (isCancel(key)) throw new PromptCancelled();
+      if (allowBack && key.name === "b") throw new PromptBack();
       if (key.name === "up" || key.name === "left") value = true;
       else if (key.name === "down" || key.name === "right") value = false;
       else if (key.name === "y") value = true;
@@ -4678,14 +4704,18 @@ async function confirm(io, message, options = {}) {
         return value;
       }
     }
+  } catch (error) {
+    frame.erase();
+    throw error;
   } finally {
     reader.close();
   }
 }
 async function text(io, message, options = {}) {
+  const allowBack = options.allowBack === true;
   const fallback = options.defaultValue ?? "";
   if (!isInteractive(io)) {
-    io.output.write(`${renderText(message, fallback)}
+    io.output.write(`${renderText(message, fallback, void 0, allowBack)}
 `);
     return fallback;
   }
@@ -4695,9 +4725,10 @@ async function text(io, message, options = {}) {
   let error;
   try {
     for (; ; ) {
-      frame.draw(renderText(message, value, error));
+      frame.draw(renderText(message, value, error, allowBack));
       const key = await reader.next();
       if (isCancel(key)) throw new PromptCancelled();
+      if (allowBack && key.name === "left") throw new PromptBack();
       error = void 0;
       if (key.name === "return" || key.name === "enter") {
         const problem = options.validate ? options.validate(value) : null;
@@ -4718,6 +4749,9 @@ async function text(io, message, options = {}) {
       }
       if (_isPrintable(key)) value += _keyText(key);
     }
+  } catch (error_) {
+    frame.erase();
+    throw error_;
   } finally {
     reader.close();
   }
@@ -4974,98 +5008,108 @@ async function runInit(options) {
       { value: "walk", label: "Go through each setting", hint: "every question starts on its default" },
       { value: "choose", label: "Pick a different preset", hint: "everything else stays at the defaults" }
     ];
-    const pace = await select(io, "How should I set this up?", paceChoices, {
-      defaultValue: fallback ? "choose" : "recommended"
-    });
-    if (pace === "choose") {
-      answers.preset = await select(
-        io,
-        "What kind of project is this?",
-        listPresetNames().map(
-          (name) => resolvePresetChoice(fallback ? null : detected.presetName, name)
-        ),
-        { defaultValue: answers.preset }
-      );
-      const chosen = detectPreset(options.cwd, answers.preset);
-      answers.preset = chosen.presetName;
-      reason = null;
-      fallback = false;
-    }
-    if (pace === "walk") {
-      const scope = await select(
-        io,
-        "What should go into the snapshot?",
-        [
-          { value: "all", label: "Everything the scanner finds", hint: "recommended \xB7 respects .gitignore" },
-          { value: "folders", label: "Only specific folders\u2026" },
-          { value: "no-tests", label: "Everything, but skip tests" },
-          { value: "custom", label: "Let me write glob patterns" }
-        ],
-        { defaultValue: "all" }
-      );
-      if (scope === "folders") {
-        const dirs = topLevelDirs(options.cwd).slice(0, 40);
-        if (dirs.length === 0) {
-          notes.push("No top-level directories found; the snapshot will include everything.");
-        } else {
-          const chosen = await multiSelect(
-            io,
-            "Select folders",
-            dirs.map((name) => ({ value: name, label: `${name}/` })),
-            { defaults: dirs }
-          );
-          answers.include = chosen.map((name) => `${name}/**`);
+    const baseInclude = [...answers.include];
+    const baseExclude = [...answers.excludePatterns];
+    let lastScope = "all";
+    const artifactChoices = [
+      { value: "readme", label: "README.md", hint: "orientation page" },
+      { value: "manifest", label: "manifest.json", hint: "required by validate and report" },
+      { value: "parts", label: "parts/", hint: "the code payload" },
+      { value: "navigation", label: "reports/navigation.md", hint: "file \u2192 part index" },
+      { value: "projectTree", label: "reports/project-tree.md", hint: "ASCII tree" },
+      { value: "audit", label: "reports/audit.md", hint: "security and health findings" },
+      { value: "monolith", label: "full-codebase.json + .md", hint: "roughly doubles output" }
+    ];
+    const scopeStep = async () => {
+      for (; ; ) {
+        const scope = await select(
+          io,
+          "What should go into the snapshot?",
+          [
+            { value: "all", label: "Everything the scanner finds", hint: "recommended \xB7 respects .gitignore" },
+            { value: "folders", label: "Only specific folders\u2026" },
+            { value: "no-tests", label: "Everything, but skip tests" },
+            { value: "custom", label: "Let me write glob patterns" }
+          ],
+          { defaultValue: lastScope, allowBack: true }
+        );
+        lastScope = scope;
+        answers.include = [...baseInclude];
+        answers.excludePatterns = [...baseExclude];
+        try {
+          if (scope === "folders") {
+            const dirs = topLevelDirs(options.cwd).slice(0, 40);
+            if (dirs.length === 0) {
+              notes.push("No top-level directories found; the snapshot will include everything.");
+            } else {
+              const chosen = await multiSelect(
+                io,
+                "Select folders",
+                dirs.map((name) => ({ value: name, label: `${name}/` })),
+                { defaults: dirs, allowBack: true }
+              );
+              answers.include = chosen.map((name) => `${name}/**`);
+            }
+          } else if (scope === "no-tests") {
+            answers.excludePatterns = [...baseExclude, "**/*.test.*", "**/*.spec.*", "tests", "test"];
+          } else if (scope === "custom") {
+            const raw = await text(io, "Glob patterns to include (comma separated)", {
+              defaultValue: baseInclude.join(", "),
+              allowBack: true
+            });
+            answers.include = raw.split(",").map((value) => value.trim()).filter(Boolean);
+          }
+          return;
+        } catch (error) {
+          if (error instanceof PromptBack) continue;
+          throw error;
         }
-      } else if (scope === "no-tests") {
-        answers.excludePatterns = [
-          ...answers.excludePatterns,
-          "**/*.test.*",
-          "**/*.spec.*",
-          "tests",
-          "test"
-        ];
-      } else if (scope === "custom") {
-        const raw = await text(io, "Glob patterns to include (comma separated)", {
-          defaultValue: ""
-        });
-        answers.include = raw.split(",").map((value) => value.trim()).filter(Boolean);
       }
-      const currentBudget = String(answers.maxPartBytes);
-      const sizeChoice = await select(io, "How large should each part be?", SIZE_CHOICES, {
-        defaultValue: SIZE_CHOICES.some((choice) => choice.value === currentBudget) ? currentBudget : "120k"
-      });
-      if (sizeChoice === "custom") {
-        const raw = await text(io, "Part size (e.g. 150k, 1.5mb, 900000)", {
-          defaultValue: String(answers.maxPartBytes),
-          validate: (value) => parseByteSize(value) === void 0 ? "Not a valid size." : null
+    };
+    const sizeStep = async () => {
+      for (; ; ) {
+        const currentBudget = String(answers.maxPartBytes);
+        const sizeChoice = await select(io, "How large should each part be?", SIZE_CHOICES, {
+          defaultValue: SIZE_CHOICES.some((choice) => choice.value === currentBudget) ? currentBudget : "custom",
+          allowBack: true
         });
-        answers.maxPartBytes = parseByteSize(raw) ?? answers.maxPartBytes;
-      } else {
-        answers.maxPartBytes = sizeChoice;
-      }
-      const artifactChoices = [
-        { value: "readme", label: "README.md", hint: "orientation page" },
-        { value: "manifest", label: "manifest.json", hint: "required by validate and report" },
-        { value: "parts", label: "parts/", hint: "the code payload" },
-        { value: "navigation", label: "reports/navigation.md", hint: "file \u2192 part index" },
-        { value: "projectTree", label: "reports/project-tree.md", hint: "ASCII tree" },
-        { value: "audit", label: "reports/audit.md", hint: "security and health findings" },
-        { value: "monolith", label: "full-codebase.json + .md", hint: "roughly doubles output" }
-      ];
-      const presetEnabled = ARTIFACT_NAMES.filter((name) => answers.artifacts[name]);
+        if (sizeChoice !== "custom") {
+          answers.maxPartBytes = sizeChoice;
+          return;
+        }
+        try {
+          const raw = await text(io, "Part size (e.g. 150k, 1.5mb, 900000)", {
+            defaultValue: currentBudget,
+            validate: (value) => parseByteSize(value) === void 0 ? "Not a valid size." : null,
+            allowBack: true
+          });
+          answers.maxPartBytes = parseByteSize(raw) ?? answers.maxPartBytes;
+          return;
+        } catch (error) {
+          if (error instanceof PromptBack) continue;
+          throw error;
+        }
+      }
+    };
+    const artifactsStep = async () => {
+      const enabled = ARTIFACT_NAMES.filter((name) => answers.artifacts[name]);
       const selectedArtifacts = await multiSelect(
         io,
         "Which artifacts should be generated?",
         artifactChoices,
-        { defaults: presetEnabled }
+        { defaults: enabled, allowBack: true }
       );
       for (const name of ARTIFACT_NAMES) answers.artifacts[name] = selectedArtifacts.includes(name);
-      const enabledCount = ARTIFACT_NAMES.filter((name) => answers.artifacts[name]).length;
+    };
+    const credentialsStep = async () => {
       answers.includeSensitiveFiles = await confirm(
         io,
-        "Include credential files (.env, *.pem, .npmrc, .ssh/)?",
-        { defaultValue: false }
+        "Include credential files (.env, *.pem, *.npmrc, *.ssh/)?",
+        { defaultValue: answers.includeSensitiveFiles, allowBack: true }
       );
+    };
+    const reviewStep = async () => {
+      const enabledCount = ARTIFACT_NAMES.filter((name) => answers.artifacts[name]).length;
       io.output.write("\n  Review\n\n");
       io.output.write(`  project          ${answers.preset}
 `);
@@ -5080,8 +5124,53 @@ async function runInit(options) {
 `);
       io.output.write("  Will write snapshot.config with your answers, plus every other setting\n");
       io.output.write("  at its default \u2014 so nothing is hidden behind the docs.\n\n");
-      const proceed = await confirm(io, "Write this and continue?", { defaultValue: true });
-      if (!proceed) throw new PromptCancelled();
+      const proceed = await confirm(io, "Write this and continue?", {
+        defaultValue: true,
+        allowBack: true
+      });
+      if (!proceed) throw new PromptBack();
+    };
+    const walk = async () => {
+      const steps = [scopeStep, sizeStep, artifactsStep, credentialsStep, reviewStep];
+      let index = 0;
+      while (index < steps.length) {
+        try {
+          await steps[index]();
+          index += 1;
+        } catch (error) {
+          if (!(error instanceof PromptBack)) throw error;
+          if (index === 0) return false;
+          index -= 1;
+          io.output.write("\n");
+        }
+      }
+      return true;
+    };
+    for (; ; ) {
+      const pace = await select(io, "How should I set this up?", paceChoices, {
+        defaultValue: fallback ? "choose" : "recommended"
+      });
+      if (pace === "choose") {
+        try {
+          answers.preset = await select(
+            io,
+            "What kind of project is this?",
+            listPresetNames().map(
+              (name) => resolvePresetChoice(fallback ? null : detected.presetName, name)
+            ),
+            { defaultValue: answers.preset, allowBack: true }
+          );
+        } catch (error) {
+          if (error instanceof PromptBack) continue;
+          throw error;
+        }
+        const chosen = detectPreset(options.cwd, answers.preset);
+        answers.preset = chosen.presetName;
+        reason = null;
+        fallback = false;
+      }
+      if (pace === "walk" && !await walk()) continue;
+      break;
     }
   }
   const configText = renderConfig(answers);
@@ -11206,14 +11295,13 @@ function installDependencies(cwd) {
   const manager = detectPackageManager(cwd);
   const command = `${manager.command} ${manager.args.join(" ")}`;
   const started = performance.now();
-  const result = spawnSync(manager.command, manager.args, {
+  const spawnOptions = {
     cwd,
     encoding: "utf8",
-    // npm, pnpm and yarn ship as `.cmd` shims on Windows, which spawnSync cannot exec directly.
-    shell: process.platform === "win32",
     timeout: INSTALL_TIMEOUT_MS,
     env: cleanInstallEnv()
-  });
+  };
+  const result = process.platform === "win32" ? spawnSync(command, { ...spawnOptions, shell: true }) : spawnSync(manager.command, manager.args, spawnOptions);
   const seconds = `${((performance.now() - started) / 1e3).toFixed(1)}s`;
   if (result.error) {
     return { manager: manager.name, command, ok: false, detail: `${result.error.message} (${seconds})` };

package/dist/{chunk-EWZYK3VX.js → chunk-4AHHSNGG.js}

@@ -1,4 +1,4 @@
-import { GENERATOR_VERSION, MIN_MAX_PART_BYTES, SCHEMA_VERSION, GENERATOR_NAME, ARTIFACT_NAMES, PRESETS, generateSnapshot, validateSnapshot, explainPath, renderExplainText, describeManifestShapeProblem, loadConfigFileDetailed, parseByteSize, resolveConfig } from './chunk-FCWD4LY7.js';
+import { GENERATOR_VERSION, MIN_MAX_PART_BYTES, SCHEMA_VERSION, GENERATOR_NAME, ARTIFACT_NAMES, PRESETS, generateSnapshot, validateSnapshot, explainPath, renderExplainText, describeManifestShapeProblem, loadConfigFileDetailed, parseByteSize, resolveConfig } from './chunk-B336MMNC.js';
 import * as fs from 'fs';
 import * as path2 from 'path';
 
@@ -2573,5 +2573,5 @@ async function runStdioServer(options) {
 }
 
 export { COMMAND_OPTION_SCOPE, ERROR_CODES, LEGACY_PROTOCOL_VERSIONS, MAX_LINE_BYTES, METHOD, MODERN_PROTOCOL_VERSIONS, SERVER_INSTRUCTIONS, SKILL_NAME, SUPPORTED_PROTOCOL_VERSIONS, buildAgentContract, buildPrompts, buildResourceProvider, buildTools, classifyRequest, commandsForOption, createLineReader, createMcpServer, createServer, createSession, createWriter, inspectModernMeta, negotiateLegacyVersion, renderContractText, renderSkill, runStdioServer, validateArguments };
-//# sourceMappingURL=chunk-EWZYK3VX.js.map
-//# sourceMappingURL=chunk-EWZYK3VX.js.map
+//# sourceMappingURL=chunk-4AHHSNGG.js.map
+//# sourceMappingURL=chunk-4AHHSNGG.js.map