outfitter 0.3.3 → 0.4.0

package/dist/shared/outfitter-x0r7mfvy.js

@@ -0,0 +1,142 @@
+// @bun
+import {
+  DEFAULT_SCAFFOLD_E2E_PRESETS,
+  resolveScaffoldE2EProfile
+} from "./outfitter-4bs5a2n4.js";
+import {
+  isValidInitPreset
+} from "./outfitter-xg5yryp2.js";
+
+// apps/outfitter/src/scaffold-e2e/runner.ts
+import { existsSync } from "fs";
+import { join } from "path";
+function resolveScaffoldCliEntry(baseDir, fileExists = existsSync) {
+  const sourceEntry = join(baseDir, "..", "cli.ts");
+  if (fileExists(sourceEntry)) {
+    return sourceEntry;
+  }
+  const builtEntry = join(baseDir, "..", "cli.js");
+  if (fileExists(builtEntry)) {
+    return builtEntry;
+  }
+  throw new Error(`Unable to resolve outfitter CLI entry from ${baseDir}`);
+}
+var cliEntry = resolveScaffoldCliEntry(import.meta.dir);
+var repoRoot = join(import.meta.dir, "..", "..", "..", "..");
+function createProjectName(preset) {
+  return `scaffold-e2e-${preset}`;
+}
+function resolveScaffoldE2EPresets(values) {
+  if (!values || values.length === 0) {
+    return DEFAULT_SCAFFOLD_E2E_PRESETS;
+  }
+  const presets = values.flatMap((value) => value.split(",").map((part) => part.trim()).filter((part) => part.length > 0));
+  const invalid = presets.filter((preset) => !isValidInitPreset(preset));
+  if (invalid.length > 0) {
+    throw new Error(`Unknown scaffold E2E preset(s): ${invalid.join(", ")}. Available presets: ${DEFAULT_SCAFFOLD_E2E_PRESETS.join(", ")}`);
+  }
+  return [...new Set(presets)];
+}
+async function runCommand(cwd, command, timeoutMs) {
+  const proc = Bun.spawn([...command], {
+    cwd,
+    stdout: "pipe",
+    stderr: "pipe"
+  });
+  const timeout = new Promise((resolveTimeout) => {
+    const timer = setTimeout(() => resolveTimeout("timeout"), timeoutMs);
+    proc.exited.finally(() => clearTimeout(timer));
+  });
+  const race = await Promise.race([proc.exited.then(() => "exit"), timeout]);
+  if (race === "timeout") {
+    proc.kill("SIGKILL");
+  }
+  const [exitCode, stdout, stderr] = await Promise.all([
+    proc.exited,
+    new Response(proc.stdout).text(),
+    new Response(proc.stderr).text()
+  ]);
+  return {
+    exitCode,
+    stdout,
+    stderr,
+    timedOut: race === "timeout"
+  };
+}
+function assertCommandSuccess(preset, step, result) {
+  if (result.timedOut) {
+    throw new Error(`[${preset}] ${step} timed out.
+stdout:
+${result.stdout}
+stderr:
+${result.stderr}`);
+  }
+  if (result.exitCode !== 0) {
+    throw new Error(`[${preset}] ${step} failed with exit code ${result.exitCode}.
+stdout:
+${result.stdout}
+stderr:
+${result.stderr}`);
+  }
+}
+async function runInitViaCli(targetDir, name, preset, timeoutMs) {
+  return runCommand(repoRoot, [
+    "bun",
+    cliEntry,
+    "init",
+    targetDir,
+    "--name",
+    name,
+    "--preset",
+    preset,
+    "--yes",
+    "--skip-install",
+    "--skip-git",
+    "--skip-commit"
+  ], timeoutMs);
+}
+async function runScaffoldE2ESuite(options) {
+  const profile = resolveScaffoldE2EProfile(options.profile);
+  const timeoutMs = options.timeoutMs ?? profile.commandTimeoutMs;
+  const presets = options.presets ?? profile.presets;
+  const results = [];
+  for (const preset of presets) {
+    const presetStartedAt = Date.now();
+    process.stderr.write(`[scaffold-e2e] starting preset: ${preset}
+`);
+    const targetDir = join(options.runDir, preset);
+    const steps = [];
+    const initStartedAt = Date.now();
+    const init = await runInitViaCli(targetDir, createProjectName(preset), preset, timeoutMs);
+    assertCommandSuccess(preset, "outfitter init", init);
+    steps.push({
+      command: "outfitter init",
+      durationMs: Date.now() - initStartedAt
+    });
+    const installStartedAt = Date.now();
+    const install = await runCommand(targetDir, ["bun", "install"], timeoutMs);
+    assertCommandSuccess(preset, "bun install", install);
+    steps.push({
+      command: "bun install",
+      durationMs: Date.now() - installStartedAt
+    });
+    const verifyStartedAt = Date.now();
+    const verify = await runCommand(targetDir, ["bun", "run", "verify:ci"], timeoutMs);
+    assertCommandSuccess(preset, "bun run verify:ci", verify);
+    steps.push({
+      command: "bun run verify:ci",
+      durationMs: Date.now() - verifyStartedAt
+    });
+    results.push({
+      preset,
+      targetDir,
+      steps
+    });
+    const presetDurationMs = Date.now() - presetStartedAt;
+    process.stderr.write(`[scaffold-e2e] completed preset: ${preset} (${presetDurationMs}ms)
+`);
+  }
+  return results;
+}
+
+export { resolveScaffoldCliEntry, resolveScaffoldE2EPresets, runScaffoldE2ESuite };

package/dist/shared/{outfitter-b9cpnr7e.js → outfitter-x4f8v5vf.js}

@@ -1,7 +1,7 @@
 // @bun
 import {
   resolveStructuredOutputMode
-} from "./outfitter-7r12fj7f.js";
+} from "./outfitter-sxf8jjjn.js";
 
 // apps/outfitter/src/commands/check-surface-map.ts
 import { resolve } from "path";

package/dist/shared/{outfitter-x8w5sjnd.d.ts → outfitter-x6322tjp.d.ts}

@@ -1,4 +1,4 @@
-import { CliOutputMode } from "./outfitter-a79xrm12.js";
+import { CliOutputMode } from "./outfitter-fhahf9f3.js";
 import { DocsExportTarget, loadDocsModule } from "./outfitter-1tfa9hke.js";
 import { InternalError, Result } from "@outfitter/contracts";
 /** Validated input for the docs.action handler. */

package/dist/shared/{outfitter-6rtcemk7.d.ts → outfitter-xa4915yp.d.ts}

@@ -1,4 +1,4 @@
-import { EngineOptions, PlaceholderValues, ScaffoldError } from "./outfitter-cyvr4r8d.js";
+import { EngineOptions, PlaceholderValues, ScaffoldError } from "./outfitter-crxe5gth.js";
 import { Result } from "@outfitter/contracts";
 /**
 * Get the directory containing scaffold preset files.
@@ -8,6 +8,7 @@ declare function getPresetsBaseDir(): string;
 declare function getOutputFilename(templateFilename: string): string;
 declare function isBinaryFile(filename: string): boolean;
 declare function replacePlaceholders(content: string, values: PlaceholderValues): string;
+declare function sortLeadingImports(filePath: string, content: string): string;
 declare function copyPresetFiles(presetDir: string, targetDir: string, values: PlaceholderValues, options: EngineOptions, copyOptions?: {
 	readonly allowOverwrite?: boolean;
 	readonly overwritablePaths?: ReadonlySet<string>;
@@ -15,4 +16,4 @@ declare function copyPresetFiles(presetDir: string, targetDir: string, values: P
 	readonly skipFilter?: (relativePath: string) => boolean;
 	readonly relativePrefix?: string;
 }): Result<void, ScaffoldError>;
-export { getPresetsBaseDir, getOutputFilename, isBinaryFile, replacePlaceholders, copyPresetFiles };
+export { getPresetsBaseDir, getOutputFilename, isBinaryFile, replacePlaceholders, sortLeadingImports, copyPresetFiles };

package/dist/shared/{outfitter-2z61gp5w.js → outfitter-xg5yryp2.js}

@@ -12,7 +12,7 @@ function isBinaryPreset(preset) {
   return preset === "cli" || preset === "daemon";
 }
 function isValidInitPreset(value) {
-  return value === "minimal" || value === "cli" || value === "mcp" || value === "daemon" || value === "library" || value === "full-stack";
+  return value === "minimal" || value === "basic" || value === "cli" || value === "mcp" || value === "daemon" || value === "library" || value === "full-stack";
 }
 function resolvePresetFromFlags(presetFromFlag, availablePresetIds) {
   if (!presetFromFlag) {

package/dist/shared/{outfitter-rdpw2sbp.d.ts → outfitter-xn5km042.d.ts}

@@ -2,6 +2,7 @@ import { OutputMode } from "@outfitter/cli/types";
 import { Result } from "@outfitter/contracts";
 /** Check bundle mode controlling which steps are included and whether a clean tree is enforced. */
 type CheckOrchestratorMode = "all" | "ci" | "pre-commit" | "pre-push";
+type CheckOrchestratorCommandProfile = "default" | "hook";
 interface CheckOrchestratorStep {
 	readonly command: readonly string[];
 	readonly id: string;
@@ -9,11 +10,15 @@ interface CheckOrchestratorStep {
 }
 /** Options controlling which checks the orchestrator runs. */
 interface CheckOrchestratorOptions {
+	/** Command profile for nested tool invocations. */
+	readonly commandProfile?: CheckOrchestratorCommandProfile;
 	/** Workspace root for resolving commands and tree-clean detection. */
 	readonly cwd: string;
 	readonly mode: CheckOrchestratorMode;
 	/** Staged file paths for pre-commit scoping. Ignored by other modes. */
 	readonly stagedFiles?: readonly string[];
+	/** Stream child process output to the terminal as each step runs. */
+	readonly streamOutput?: boolean;
 }
 interface CheckOrchestratorStepResult {
 	readonly command: readonly string[];
@@ -64,6 +69,7 @@ declare function parseTreePaths(statusOutput: string): string[];
 declare function runCheckOrchestrator(options: CheckOrchestratorOptions): Promise<Result<CheckOrchestratorResult, CheckOrchestratorError>>;
 interface PrintCheckOrchestratorResultsOptions {
 	readonly compact?: boolean;
+	readonly liveOutput?: boolean;
 	readonly mode?: OutputMode;
 }
 /**

package/dist/shared/{outfitter-cyhzstz0.js → outfitter-xqr4fp1n.js}

@@ -7,7 +7,7 @@ import {
 } from "./outfitter-5vx1bp7h.js";
 import {
   resolveStructuredOutputMode
-} from "./outfitter-7r12fj7f.js";
+} from "./outfitter-sxf8jjjn.js";
 
 // apps/outfitter/src/commands/docs-api.ts
 import { relative } from "path";
@@ -50,7 +50,7 @@ async function printDocsApiResults(result, options) {
       });
       process.stdout.write(filtered);
     } else {
-      await output(result, { mode: structuredMode });
+      await output(result, structuredMode);
     }
     return;
   }
@@ -59,7 +59,7 @@ async function printDocsApiResults(result, options) {
   const totalDeclarations = result.summary.total;
   if (totalDeclarations === 0) {
     lines.push(theme.muted("No API declarations found."));
-    await output(lines, { mode: "human" });
+    await output(lines, "human");
     return;
   }
   lines.push("");
@@ -87,7 +87,7 @@ async function printDocsApiResults(result, options) {
     }
   }
   lines.push("");
-  await output(lines, { mode: "human" });
+  await output(lines, "human");
 }
 
 export { runDocsApi, printDocsApiResults };

package/dist/shared/outfitter-z6tg0swx.js

@@ -0,0 +1,87 @@
+// @bun
+// apps/outfitter/src/scaffold-e2e/workspace.ts
+import { existsSync, mkdirSync, readdirSync, rmSync } from "fs";
+import { tmpdir } from "os";
+import { join, resolve } from "path";
+var SCAFFOLD_E2E_ROOT_NAME = "outfitter-scaffold-e2e";
+var DEFAULT_SCAFFOLD_E2E_RETENTION_MS = 24 * 60 * 60 * 1000;
+var RUN_DIR_NAME_PATTERN = /^(?<year>\d{4})(?<month>\d{2})(?<day>\d{2})T(?<hour>\d{2})(?<minute>\d{2})(?<second>\d{2})-(?<label>[a-z0-9-]+?)(?:-[0-9a-f-]+)?$/u;
+function pad(value) {
+  return value.toString().padStart(2, "0");
+}
+function formatTimestamp(now) {
+  const date = new Date(now);
+  return [
+    date.getUTCFullYear().toString(),
+    pad(date.getUTCMonth() + 1),
+    pad(date.getUTCDate()),
+    "T",
+    pad(date.getUTCHours()),
+    pad(date.getUTCMinutes()),
+    pad(date.getUTCSeconds())
+  ].join("");
+}
+function slugifyLabel(label) {
+  const normalized = (label ?? "run").toLowerCase().replace(/[^a-z0-9]+/gu, "-").replace(/^-+|-+$/gu, "");
+  return normalized.length > 0 ? normalized : "run";
+}
+function parseRunStart(name) {
+  const match = RUN_DIR_NAME_PATTERN.exec(name);
+  if (!match?.groups) {
+    return;
+  }
+  const { year, month, day, hour, minute, second } = match.groups;
+  return Date.UTC(Number(year), Number(month) - 1, Number(day), Number(hour), Number(minute), Number(second));
+}
+function resolveScaffoldE2ERoot(rootDir) {
+  return resolve(rootDir ?? join(tmpdir(), SCAFFOLD_E2E_ROOT_NAME));
+}
+function createScaffoldE2ERunDir(options = {}) {
+  const rootDir = resolveScaffoldE2ERoot(options.rootDir);
+  const timestamp = formatTimestamp(options.now ?? Date.now());
+  const label = slugifyLabel(options.runLabel);
+  const runId = Bun.randomUUIDv7().toLowerCase();
+  const runDir = join(rootDir, `${timestamp}-${label}-${runId}`);
+  mkdirSync(runDir, { recursive: true });
+  return runDir;
+}
+function cleanupScaffoldE2ERunDir(runDir) {
+  if (existsSync(runDir)) {
+    rmSync(runDir, { recursive: true, force: true });
+  }
+}
+function pruneScaffoldE2ERuns(options = {}) {
+  const rootDir = resolveScaffoldE2ERoot(options.rootDir);
+  const now = options.now ?? Date.now();
+  const maxAgeMs = options.maxAgeMs ?? DEFAULT_SCAFFOLD_E2E_RETENTION_MS;
+  const removed = [];
+  const kept = [];
+  if (!existsSync(rootDir)) {
+    return { rootDir, removed, kept };
+  }
+  for (const entry of readdirSync(rootDir, { withFileTypes: true })) {
+    if (!entry.isDirectory()) {
+      continue;
+    }
+    const startedAt = parseRunStart(entry.name);
+    if (startedAt === undefined) {
+      continue;
+    }
+    const path = join(rootDir, entry.name);
+    const ageMs = Math.max(0, now - startedAt);
+    const shouldRemove = options.removeAll === true || ageMs > maxAgeMs;
+    if (shouldRemove) {
+      cleanupScaffoldE2ERunDir(path);
+      removed.push({ path, ageMs });
+      continue;
+    }
+    kept.push({ path, ageMs });
+  }
+  return {
+    rootDir,
+    removed: removed.toSorted((left, right) => left.path.localeCompare(right.path)),
+    kept: kept.toSorted((left, right) => left.path.localeCompare(right.path))
+  };
+}
+
+export { SCAFFOLD_E2E_ROOT_NAME, DEFAULT_SCAFFOLD_E2E_RETENTION_MS, resolveScaffoldE2ERoot, createScaffoldE2ERunDir, cleanupScaffoldE2ERunDir, pruneScaffoldE2ERuns };

package/dist/shared/{outfitter-dx4hn4ta.js → outfitter-zmzrsvcn.js}

@@ -1,39 +1,40 @@
 // @bun
 import {
   checkTsdocOutputSchema
-} from "./outfitter-k6zyvg2n.js";
-import {
-  resolveDocsOutputMode
-} from "./outfitter-ec83h4v2.js";
+} from "./outfitter-dna8exj2.js";
 import {
   outputModeSchema,
   resolveCwdFromPreset,
   resolveStringFlag
-} from "./outfitter-7n7vsz95.js";
+} from "./outfitter-7t7ces1x.js";
 import {
   printDocsListResults,
   runDocsList
-} from "./outfitter-yhb23pjc.js";
+} from "./outfitter-8jd1sak0.js";
 import {
   printDocsApiResults,
   runDocsApi
-} from "./outfitter-cyhzstz0.js";
+} from "./outfitter-xqr4fp1n.js";
 import {
   printDocsSearchResults,
   runDocsSearch
-} from "./outfitter-hf5bj2gq.js";
+} from "./outfitter-sh015v8k.js";
 import {
   printDocsExportResults,
   runDocsExport
-} from "./outfitter-940h0x7b.js";
+} from "./outfitter-e44qcs0w.js";
 import {
   printDocsShowResults,
   runDocsShow
-} from "./outfitter-c7sbs7es.js";
+} from "./outfitter-g6r9avgz.js";
 
 // apps/outfitter/src/actions/docs.ts
 import { cwdPreset } from "@outfitter/cli/flags";
-import { jqPreset, outputModePreset } from "@outfitter/cli/query";
+import {
+  jqPreset,
+  outputModePreset,
+  resolveOutputMode
+} from "@outfitter/cli/query";
 import { defineAction, Result } from "@outfitter/contracts";
 import { z } from "zod";
 var docsListInputSchema = z.object({
@@ -69,9 +70,8 @@ var docsListAction = defineAction({
       ...docsListCwd.options
     ],
     mapInput: (context) => {
-      const { outputMode: presetOutputMode } = docsListOutputMode.resolve(context.flags);
       const { jq } = docsListJq.resolve(context.flags);
-      const outputMode = resolveDocsOutputMode(context.flags, presetOutputMode);
+      const { mode: outputMode } = resolveOutputMode(context.flags);
       const kind = resolveStringFlag(context.flags["kind"]);
       const pkg = resolveStringFlag(context.flags["package"]);
       return {
@@ -117,9 +117,8 @@ var docsShowAction = defineAction({
       ...docsShowCwd.options
     ],
     mapInput: (context) => {
-      const { outputMode: presetOutputMode } = docsShowOutputMode.resolve(context.flags);
       const { jq } = docsShowJq.resolve(context.flags);
-      const outputMode = resolveDocsOutputMode(context.flags, presetOutputMode);
+      const { mode: outputMode } = resolveOutputMode(context.flags);
       return {
         id: context.args[0],
         cwd: resolveCwdFromPreset(context.flags, docsShowCwd),
@@ -172,9 +171,8 @@ var docsSearchAction = defineAction({
       ...docsSearchCwd.options
     ],
     mapInput: (context) => {
-      const { outputMode: presetOutputMode } = docsSearchOutputMode.resolve(context.flags);
       const { jq } = docsSearchJq.resolve(context.flags);
-      const outputMode = resolveDocsOutputMode(context.flags, presetOutputMode);
+      const { mode: outputMode } = resolveOutputMode(context.flags);
       const kind = resolveStringFlag(context.flags["kind"]);
       const pkg = resolveStringFlag(context.flags["package"]);
       return {
@@ -231,9 +229,8 @@ var docsApiAction = defineAction({
       ...docsApiCwd.options
     ],
     mapInput: (context) => {
-      const { outputMode: presetOutputMode } = docsApiOutputMode.resolve(context.flags);
       const { jq } = docsApiJq.resolve(context.flags);
-      const outputMode = resolveDocsOutputMode(context.flags, presetOutputMode);
+      const { mode: outputMode } = resolveOutputMode(context.flags);
       const levelRaw = context.flags["level"];
       const validLevels = new Set(["documented", "partial", "undocumented"]);
       const level = typeof levelRaw === "string" && validLevels.has(levelRaw) ? levelRaw : undefined;
@@ -300,8 +297,7 @@ var docsExportAction = defineAction({
       ...docsExportCwd.options
     ],
     mapInput: (context) => {
-      const { outputMode: presetOutputMode } = docsExportOutputMode.resolve(context.flags);
-      const outputMode = resolveDocsOutputMode(context.flags, presetOutputMode);
+      const { mode: outputMode } = resolveOutputMode(context.flags);
       const targetRaw = resolveStringFlag(context.flags["target"]);
       const target = targetRaw ?? "all";
       return {

package/dist/shared/{outfitter-8ggmja91.js → outfitter-zqj1nte1.js}

@@ -1,20 +1,20 @@
 // @bun
 import {
   resolveStructuredOutputMode
-} from "./outfitter-7r12fj7f.js";
+} from "./outfitter-sxf8jjjn.js";
+import {
+  DEPENDENCY_SECTIONS,
+  normalizeVersionRange
+} from "./outfitter-hr4cvmjy.js";
 
 // apps/outfitter/src/commands/check-preset-versions.ts
 import { existsSync, readFileSync } from "fs";
 import { join, resolve } from "path";
-import { Result } from "@outfitter/contracts";
+import { extractMessage, Result } from "@outfitter/contracts";
 import { getResolvedVersions } from "@outfitter/presets";
 import { isTypesBunVersionCompatible } from "@outfitter/tooling";
-var DEPENDENCY_SECTIONS = [
-  "dependencies",
-  "devDependencies",
-  "peerDependencies",
-  "optionalDependencies"
-];
+import { isPlainObject } from "@outfitter/types";
+var EXTERNAL_TEMPLATE_VERSION = "catalog:";
 
 class CheckPresetVersionsError extends Error {
   _tag = "CheckPresetVersionsError";
@@ -23,61 +23,49 @@ class CheckPresetVersionsError extends Error {
     this.name = "CheckPresetVersionsError";
   }
 }
-function isRecord(value) {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-function normalizeVersionRange(version) {
-  const trimmed = version.trim();
-  const semverMatch = trimmed.match(/\d+\.\d+\.\d+(?:-[0-9A-Za-z.-]+)?/);
-  if (semverMatch) {
-    return semverMatch[0];
-  }
-  return trimmed.replace(/^[\^~>=<]+/, "");
-}
 function validatePresetDeps(workspaceRoot, resolvedVersions, problems) {
-  const templateRoots = ["templates", "packages/presets/presets"];
+  const presetRoot = "packages/presets/presets";
   const glob = new Bun.Glob("**/package.json.template");
-  for (const rootPath of templateRoots) {
-    const absoluteRoot = join(workspaceRoot, rootPath);
-    if (!existsSync(absoluteRoot)) {
+  const absoluteRoot = join(workspaceRoot, presetRoot);
+  if (!existsSync(absoluteRoot)) {
+    problems.push(`Canonical presets root not found: ${presetRoot}`);
+    return;
+  }
+  for (const relativePath of glob.scanSync({
+    cwd: absoluteRoot,
+    absolute: false
+  })) {
+    const templatePath = join(presetRoot, relativePath);
+    const absoluteTemplatePath = join(absoluteRoot, relativePath);
+    const parsed = JSON.parse(readFileSync(absoluteTemplatePath, "utf-8"));
+    if (!isPlainObject(parsed)) {
       continue;
     }
-    for (const relativePath of glob.scanSync({
-      cwd: absoluteRoot,
-      absolute: false
-    })) {
-      const templatePath = join(rootPath, relativePath);
-      const absoluteTemplatePath = join(absoluteRoot, relativePath);
-      const parsed = JSON.parse(readFileSync(absoluteTemplatePath, "utf-8"));
-      if (!isRecord(parsed)) {
+    for (const section of DEPENDENCY_SECTIONS) {
+      const deps = parsed[section];
+      if (!isPlainObject(deps)) {
         continue;
       }
-      for (const section of DEPENDENCY_SECTIONS) {
-        const deps = parsed[section];
-        if (!isRecord(deps)) {
+      for (const [name, value] of Object.entries(deps)) {
+        if (typeof value !== "string") {
           continue;
         }
-        for (const [name, value] of Object.entries(deps)) {
-          if (typeof value !== "string") {
-            continue;
-          }
-          if (name.startsWith("@outfitter/")) {
-            if (value !== "workspace:*") {
-              problems.push(`${templatePath}: ${name} must use workspace:* (found ${value})`);
-            }
-            continue;
-          }
-          if (name.includes("{{") || value.startsWith("workspace:")) {
-            continue;
-          }
-          const expected = resolvedVersions[name];
-          if (!expected) {
-            problems.push(`${templatePath}: external dependency "${name}" is not declared in @outfitter/presets`);
-            continue;
-          }
-          if (normalizeVersionRange(value) !== normalizeVersionRange(expected)) {
-            problems.push(`${templatePath}: ${name} expected ${expected} (found ${value})`);
+        if (name.startsWith("@outfitter/")) {
+          if (value !== "workspace:*") {
+            problems.push(`${templatePath}: ${name} must use workspace:* (found ${value})`);
           }
+          continue;
+        }
+        if (name.includes("{{") || value.startsWith("workspace:")) {
+          continue;
+        }
+        const expected = resolvedVersions[name];
+        if (!expected) {
+          problems.push(`${templatePath}: external dependency "${name}" is not declared in @outfitter/presets`);
+          continue;
+        }
+        if (value !== EXTERNAL_TEMPLATE_VERSION) {
+          problems.push(`${templatePath}: ${name} must use ${EXTERNAL_TEMPLATE_VERSION} (found ${value})`);
         }
       }
     }
@@ -93,20 +81,20 @@ function validateRegistryVersions(workspaceRoot, resolvedVersions, problems) {
     problems.push(`Registry not found or unreadable: ${registryPath}`);
     return;
   }
-  if (!isRecord(registry)) {
+  if (!isPlainObject(registry)) {
     problems.push(`Registry has invalid shape (expected object): ${registryPath}`);
     return;
   }
-  if (!isRecord(registry["blocks"])) {
+  if (!isPlainObject(registry["blocks"])) {
     problems.push(`Registry has invalid shape (missing object "blocks" field): ${registryPath}`);
     return;
   }
   for (const [blockName, block] of Object.entries(registry["blocks"])) {
-    if (!isRecord(block)) {
+    if (!isPlainObject(block)) {
       continue;
     }
     const devDeps = block["devDependencies"];
-    if (!isRecord(devDeps)) {
+    if (!isPlainObject(devDeps)) {
       continue;
     }
     for (const [name, value] of Object.entries(devDeps)) {
@@ -125,7 +113,7 @@ function validateRegistryVersions(workspaceRoot, resolvedVersions, problems) {
 }
 function readDependencyVersion(packageJson, section, name) {
   const container = packageJson[section];
-  if (!isRecord(container)) {
+  if (!isPlainObject(container)) {
     return;
   }
   const value = container[name];
@@ -133,14 +121,26 @@ function readDependencyVersion(packageJson, section, name) {
 }
 function validateBunVersionConsistency(workspaceRoot, problems) {
   const bunVersionPath = join(workspaceRoot, ".bun-version");
-  const bunVersionFile = readFileSync(bunVersionPath, "utf-8").trim();
+  let bunVersionFile;
+  try {
+    bunVersionFile = readFileSync(bunVersionPath, "utf-8").trim();
+  } catch (error) {
+    problems.push(`.bun-version not found or unreadable: ${extractMessage(error)}`);
+    return;
+  }
   const rootPackagePath = join(workspaceRoot, "package.json");
-  const parsedRootPackage = JSON.parse(readFileSync(rootPackagePath, "utf-8"));
-  if (!isRecord(parsedRootPackage)) {
+  let parsedRootPackage;
+  try {
+    parsedRootPackage = JSON.parse(readFileSync(rootPackagePath, "utf-8"));
+  } catch (error) {
+    problems.push(`Root package.json could not be read or parsed: ${extractMessage(error)}`);
+    return;
+  }
+  if (!isPlainObject(parsedRootPackage)) {
     return;
   }
   const engines = parsedRootPackage["engines"];
-  if (isRecord(engines) && typeof engines["bun"] === "string") {
+  if (isPlainObject(engines) && typeof engines["bun"] === "string") {
     const engineBun = normalizeVersionRange(engines["bun"]);
     if (engineBun !== bunVersionFile) {
       problems.push(`Bun version drift: .bun-version is ${bunVersionFile} but engines.bun is ${engines["bun"]}`);
@@ -155,7 +155,7 @@ function validateBunVersionConsistency(workspaceRoot, problems) {
   }
   let bunTypesVersion;
   const catalog = parsedRootPackage["catalog"];
-  if (isRecord(catalog) && typeof catalog["@types/bun"] === "string") {
+  if (isPlainObject(catalog) && typeof catalog["@types/bun"] === "string") {
     bunTypesVersion = catalog["@types/bun"];
   }
   if (!bunTypesVersion) {
@@ -219,8 +219,7 @@ async function runCheckPresetVersions(options) {
       ok: problems.length === 0
     });
   } catch (error) {
-    const message = error instanceof Error ? error.message : "Failed to check preset versions";
-    return Result.err(new CheckPresetVersionsError(message));
+    return Result.err(new CheckPresetVersionsError(extractMessage(error)));
   }
 }
 async function printCheckPresetVersionsResult(result, options) {
@@ -298,4 +297,4 @@ if (import.meta.main) {
   });
 }
 
-export { CheckPresetVersionsError, runCheckPresetVersions, printCheckPresetVersionsResult, runCheckPresetVersionsFromArgv };
+export { EXTERNAL_TEMPLATE_VERSION, CheckPresetVersionsError, validatePresetDeps, runCheckPresetVersions, printCheckPresetVersionsResult, runCheckPresetVersionsFromArgv };

package/dist/targets/index.d.ts

@@ -1,4 +1,4 @@
 import "../shared/outfitter-y784nh31.js";
-import { INIT_TARGET_IDS, READY_TARGET_IDS, SCAFFOLD_TARGET_IDS, TARGET_IDS, TARGET_REGISTRY, getInitTarget, getReadyTarget, getScaffoldTarget, getTarget, listTargets, resolvePlacement } from "../shared/outfitter-5r6q2749.js";
-import { TargetCategory, TargetDefinition, TargetId, TargetScope, TargetStatus } from "../shared/outfitter-58rn1sj1.js";
+import { INIT_TARGET_IDS, READY_TARGET_IDS, SCAFFOLD_TARGET_IDS, TARGET_IDS, TARGET_REGISTRY, getInitTarget, getReadyTarget, getScaffoldTarget, getTarget, listTargets, resolvePlacement } from "../shared/outfitter-pw74st5t.js";
+import { TargetCategory, TargetDefinition, TargetId, TargetScope, TargetStatus } from "../shared/outfitter-gsjbcta2.js";
 export { resolvePlacement, listTargets, getTarget, getScaffoldTarget, getReadyTarget, getInitTarget, TargetStatus, TargetScope, TargetId, TargetDefinition, TargetCategory, TARGET_REGISTRY, TARGET_IDS, SCAFFOLD_TARGET_IDS, READY_TARGET_IDS, INIT_TARGET_IDS };