codebase-snapshot 0.9.12 → 0.9.14

package/dist/cli.d.cts

@@ -1,4 +1,4 @@
-import { O as OutputFormat, D as DiscoveryMode, R as ResolvedSnapshotConfig } from './types-CTewen3J.cjs';
+import { O as OutputFormat, D as DiscoveryMode, R as ResolvedSnapshotConfig } from './types-DIX6zWSt.cjs';
 
 /**
  * The machine-readable description of this tool.

package/dist/cli.d.ts

@@ -1,4 +1,4 @@
-import { O as OutputFormat, D as DiscoveryMode, R as ResolvedSnapshotConfig } from './types-CTewen3J.js';
+import { O as OutputFormat, D as DiscoveryMode, R as ResolvedSnapshotConfig } from './types-DIX6zWSt.js';
 
 /**
  * The machine-readable description of this tool.

package/dist/cli.js

@@ -1,6 +1,6 @@
-import { buildAgentContract, runStdioServer, COMMAND_OPTION_SCOPE, commandsForOption, SKILL_NAME, renderSkill, renderContractText } from './chunk-5OR3WVPW.js';
-export { buildAgentContract } from './chunk-5OR3WVPW.js';
-import { shouldCheck, checkForUpdate, GENERATOR_VERSION, cacheFilePath, formatUpdateNotice, installKind, loadConfigFileDetailed, resolveConfig, validateSnapshot, describeManifestShapeProblem, explainPath, compareManifests, loadManifest, renderDiffText, generateSnapshot, formatNumber, isDiscoveryMode, ARTIFACT_NAMES, getPreset, unknownPresetMessage, GENERATOR_NAME, runInit, installPreCommitHook, PromptCancelled, planFirstRun, isEffectivelyEmpty, isInteractive, defaultIO, hasPriorSnapshot, DEFAULT_ANSWERS, findConfigFile, firstRunMessage, parseByteSize, gitignorePath, appendGitignoreEntries, escapeControls, MIN_MAX_PART_TOKENS, MIN_MAX_PART_BYTES, renderExplainText } from './chunk-3H45YMBG.js';
+import { buildAgentContract, runStdioServer, COMMAND_OPTION_SCOPE, commandsForOption, SKILL_NAME, renderSkill, renderContractText } from './chunk-J4W74N4Y.js';
+export { buildAgentContract } from './chunk-J4W74N4Y.js';
+import { shouldCheck, checkForUpdate, GENERATOR_VERSION, cacheFilePath, formatUpdateNotice, installKind, loadConfigFileDetailed, resolveConfig, validateSnapshot, describeManifestShapeProblem, explainPath, compareManifests, loadManifest, renderDiffText, generateSnapshot, formatNumber, isDiscoveryMode, ARTIFACT_NAMES, getPreset, unknownPresetMessage, GENERATOR_NAME, runInit, installPreCommitHook, PromptCancelled, planFirstRun, isEffectivelyEmpty, isInteractive, defaultIO, hasPriorSnapshot, DEFAULT_ANSWERS, findConfigFile, firstRunMessage, parseByteSize, stripJsonComments, gitignorePath, appendGitignoreEntries, runWithStdoutCapture, escapeControls, MIN_MAX_PART_TOKENS, MIN_MAX_PART_BYTES, renderExplainText } from './chunk-DUZK5VL2.js';
 import { spawnSync } from 'child_process';
 import * as fs from 'fs';
 import * as path2 from 'path';
@@ -576,7 +576,9 @@ function nextStepHint(cwd, scriptsAddedNow) {
   let hasScript = scriptsAddedNow;
   if (!hasScript) {
     try {
-      const parsed = JSON.parse(fs.readFileSync(path2.join(cwd, "package.json"), "utf8"));
+      const parsed = JSON.parse(
+        stripJsonComments(fs.readFileSync(path2.join(cwd, "package.json"), "utf8"))
+      );
       hasScript = typeof parsed.scripts?.snapshot === "string";
     } catch {
       hasScript = false;
@@ -678,18 +680,8 @@ function installDependencies(cwd) {
   return { manager: manager.name, command, ok: true, detail: seconds };
 }
 async function captureStdout(fn) {
-  const original = process.stdout.write.bind(process.stdout);
-  let text = "";
-  process.stdout.write = ((chunk) => {
-    text += typeof chunk === "string" ? chunk : Buffer.from(chunk).toString("utf8");
-    return true;
-  });
-  try {
-    const code = await fn();
-    return { code, text };
-  } finally {
-    process.stdout.write = original;
-  }
+  const { value, text } = await runWithStdoutCapture(fn);
+  return { code: value, text };
 }
 async function runSetupCommand(invocation, options = {}) {
   try {
@@ -804,31 +796,33 @@ async function runSetupCommand(invocation, options = {}) {
     if (invocation.watch) generateArgs.push("--watch");
     if (invocation.format === "json") {
       const generated = await captureStdout(() => main(generateArgs));
-      let manifest = null;
+      let parsed = null;
       try {
-        manifest = JSON.parse(generated.text);
+        parsed = JSON.parse(generated.text);
       } catch {
-        manifest = null;
+        parsed = null;
+      }
+      const setup = {
+        configPath: init.configPath,
+        configReused: init.reused,
+        preset: init.preset,
+        scripts: init.scripts,
+        dependency: init.dependency,
+        gitignoreEntries: init.gitignoreEntries,
+        install,
+        notes: init.notes,
+        hook
+      };
+      if (generated.code !== 0) {
+        const inner = parsed && typeof parsed.error === "object" && parsed.error !== null ? parsed.error : { message: "Snapshot generation failed during setup.", exitCode: generated.code };
+        process.stdout.write(
+          `${JSON.stringify({ error: inner, setup, manifest: null }, null, 2)}
+`
+        );
+        return generated.code;
       }
       process.stdout.write(
-        `${JSON.stringify(
-          {
-            setup: {
-              configPath: init.configPath,
-              configReused: init.reused,
-              preset: init.preset,
-              scripts: init.scripts,
-              dependency: init.dependency,
-              gitignoreEntries: init.gitignoreEntries,
-              install,
-              notes: init.notes,
-              hook
-            },
-            manifest
-          },
-          null,
-          2
-        )}
+        `${JSON.stringify({ setup, manifest: parsed }, null, 2)}
 `
       );
       return generated.code;
@@ -1021,6 +1015,10 @@ ${USAGE}
       largeFileBytes: invocation.largeFileBytes,
       maxMonolithBytes: invocation.maxMonolithBytes,
       skipArtifacts: invocation.skipArtifacts,
+      // `--only-artifacts` is forwarded as itself, not only as the derived suppression list:
+      // suppression cannot switch an artifact *on*, so a config that had turned one of the
+      // requested artifacts off left the run with nothing to write.
+      onlyArtifacts: invocation.onlyArtifacts,
       discovery: invocation.discovery,
       changedSince: invocation.changedSince,
       staged: invocation.staged,
@@ -1032,19 +1030,6 @@ ${USAGE}
   );
   if (invocation.command === "validate") {
     const result = validateSnapshot(resolved);
-    if (result.valid) {
-      const manifestPath = path2.resolve(resolved.cwd, resolved.outDir, "manifest.json");
-      let manifest;
-      try {
-        manifest = JSON.parse(fs.readFileSync(manifestPath, "utf8"));
-      } catch {
-        manifest = void 0;
-      }
-      const shapeError = manifest === void 0 ? null : describeManifestShapeProblem(manifest);
-      if (shapeError) {
-        return fail(invocation.format, `${manifestPath} is not a snapshot manifest: ${shapeError}`);
-      }
-    }
     emit(invocation.format, result, () => renderValidateText(result));
     return result.valid ? 0 : 1;
   }
@@ -1239,10 +1224,13 @@ function isGeneratedWatchPath(candidate, paths) {
   const outDir = comparablePath(path2.resolve(paths.cwd, paths.outDir));
   const tmpDir = comparablePath(path2.resolve(paths.cwd, paths.tmpDir));
   if (isSameOrInside(outDir, absolute) || isSameOrInside(tmpDir, absolute)) return true;
-  const parent = path2.dirname(absolute);
-  if (parent === path2.dirname(outDir) && path2.basename(absolute).startsWith(`${path2.basename(outDir)}.backup-`)) {
-    return true;
-  }
+  const offsets = [path2.basename(outDir), path2.basename(tmpDir)].filter((name) => name.length > 0);
+  const leftover = absolute.split(/[\\/]+/).some(
+    (segment) => offsets.some(
+      (name) => segment.startsWith(`${name}.backup-`) || segment.startsWith(`${name}.partial-`)
+    )
+  );
+  if (leftover) return true;
   return absolute.split(/[\\/]+/).some((segment) => VCS_SEGMENTS.has(segment));
 }
 function decideWatchEvent(state, event, paths) {
@@ -1317,6 +1305,7 @@ async function runWatchLoop(invocation, resolved, options) {
           largeFileBytes: invocation.largeFileBytes,
           maxMonolithBytes: invocation.maxMonolithBytes,
           skipArtifacts: invocation.skipArtifacts,
+          onlyArtifacts: invocation.onlyArtifacts,
           discovery: invocation.discovery,
           changedSince: invocation.changedSince,
           staged: invocation.staged,

package/dist/{generate-worker-D5Xgh27c.d.cts → generate-worker-D_mUjgXw.d.cts}

@@ -1,3 +1,23 @@
+/** The options `resolveConfig` was called with, as the CLI would have passed them. */
+interface WorkspaceCliOptions {
+    cwd: string;
+    maxPartBytes?: number;
+    maxPartTokens?: number;
+    outDir?: string;
+    /**
+     * `true` or absent, never `false`.
+     *
+     * `resolveConfig` reads these as `cliOptions.x ?? config.x`, so a `false` here is an *explicit
+     * override* that beats `skeleton: true` in the project's configuration — the same trap the CLI
+     * documents. Absent means "whatever the configuration says".
+     */
+    skeleton?: true;
+    sarif?: true;
+    staged?: true;
+    withDependents?: true;
+    changedSince?: string;
+}
+
 /** What the caller needs from a finished run; the whole manifest can be megabytes. */
 interface GenerateSummary {
     fingerprint: string;
@@ -48,10 +68,6 @@ declare function workerEntry(directory: string, isCjs: boolean): {
  * as a tool failure naming the reason — never silently retried on the calling thread, because a
  * quiet fallback would hide exactly the condition this exists to avoid.
  */
-declare function generateInWorker(cwd: string, cliOptions: {
-    cwd: string;
-    maxPartBytes?: number;
-    outDir?: string;
-}): Promise<WorkerResults>;
+declare function generateInWorker(cwd: string, cliOptions: WorkspaceCliOptions): Promise<WorkerResults>;
 
 export { type GenerateSummary as G, type WorkerResults as W, generateInWorker as g, workerEntry as w };

package/dist/{generate-worker-D5Xgh27c.d.ts → generate-worker-D_mUjgXw.d.ts}

@@ -1,3 +1,23 @@
+/** The options `resolveConfig` was called with, as the CLI would have passed them. */
+interface WorkspaceCliOptions {
+    cwd: string;
+    maxPartBytes?: number;
+    maxPartTokens?: number;
+    outDir?: string;
+    /**
+     * `true` or absent, never `false`.
+     *
+     * `resolveConfig` reads these as `cliOptions.x ?? config.x`, so a `false` here is an *explicit
+     * override* that beats `skeleton: true` in the project's configuration — the same trap the CLI
+     * documents. Absent means "whatever the configuration says".
+     */
+    skeleton?: true;
+    sarif?: true;
+    staged?: true;
+    withDependents?: true;
+    changedSince?: string;
+}
+
 /** What the caller needs from a finished run; the whole manifest can be megabytes. */
 interface GenerateSummary {
     fingerprint: string;
@@ -48,10 +68,6 @@ declare function workerEntry(directory: string, isCjs: boolean): {
  * as a tool failure naming the reason — never silently retried on the calling thread, because a
  * quiet fallback would hide exactly the condition this exists to avoid.
  */
-declare function generateInWorker(cwd: string, cliOptions: {
-    cwd: string;
-    maxPartBytes?: number;
-    outDir?: string;
-}): Promise<WorkerResults>;
+declare function generateInWorker(cwd: string, cliOptions: WorkspaceCliOptions): Promise<WorkerResults>;
 
 export { type GenerateSummary as G, type WorkerResults as W, generateInWorker as g, workerEntry as w };