@reddoorla/maintenance 0.6.7 → 0.6.8

package/dist/index.js

@@ -841,6 +841,65 @@ async function commit(cwd, message) {
   return sha.trim();
 }
 
+// src/recipes/_with-recipe.ts
+async function withRecipe(body) {
+  const label = siteLabel(body.site);
+  if (body.checkTreeFirst && !await isWorkingTreeClean(body.site.path)) {
+    throw new Error(`refusing to run: working tree is not clean at ${body.site.path}`);
+  }
+  const planned = await body.plan();
+  if (planned.kind === "noop") {
+    return {
+      recipe: body.name,
+      site: label,
+      status: "noop",
+      commits: [],
+      ...planned.notes ? { notes: planned.notes } : {}
+    };
+  }
+  if (planned.kind === "failed") {
+    return {
+      recipe: body.name,
+      site: label,
+      status: "failed",
+      commits: [],
+      notes: planned.notes
+    };
+  }
+  if (!body.checkTreeFirst && !await isWorkingTreeClean(body.site.path)) {
+    throw new Error(`refusing to run: working tree is not clean at ${body.site.path}`);
+  }
+  const branch = branchName(body.name);
+  await createBranch(body.site.path, branch);
+  const shas = [];
+  const result = await body.apply(planned.plan, {
+    cwd: body.site.path,
+    branch,
+    commit: async (msg) => {
+      const sha = await commit(body.site.path, msg);
+      if (sha) shas.push(sha);
+      return sha;
+    }
+  });
+  if (result.kind === "failed") {
+    return {
+      recipe: body.name,
+      site: label,
+      status: "failed",
+      commits: shas,
+      notes: result.notes
+    };
+  }
+  const notes = result.notes ? `${result.notes}; branch: ${branch}` : `branch: ${branch}`;
+  return {
+    recipe: body.name,
+    site: label,
+    status: shas.length > 0 ? "applied" : "noop",
+    commits: shas,
+    notes
+  };
+}
+
 // src/recipes/sync-configs.ts
 var GITIGNORE_CONFIG = "gitignore";
 async function readMaybe(path) {
@@ -873,48 +932,33 @@ async function applyGitignore(cwd, plan) {
   }
 }
 async function syncConfigs(site, opts = {}) {
-  const label = siteLabel(site);
   const requested = opts.which ?? ALL_TEMPLATES.map((t) => t.config).concat(GITIGNORE_CONFIG);
   const templateNames = requested.filter((c) => c !== GITIGNORE_CONFIG);
   const templates = templatesByName(templateNames);
   const includeGitignore = requested.includes(GITIGNORE_CONFIG);
-  const templateDiffs = await planTemplateDiffs(site.path, templates);
-  const gitignorePlan = includeGitignore ? await planGitignore(site.path) : { kind: "noop" };
-  if (templateDiffs.length === 0 && gitignorePlan.kind === "noop") {
-    return {
-      recipe: "sync-configs",
-      site: label,
-      status: "noop",
-      commits: [],
-      notes: "all targeted configs already match"
-    };
-  }
-  if (!await isWorkingTreeClean(site.path)) {
-    throw new Error(`refusing to run: working tree is not clean at ${site.path}`);
-  }
-  const branch = branchName("sync-configs");
-  await createBranch(site.path, branch);
-  const shas = [];
-  for (const t of templateDiffs) {
-    await writeFile3(join5(site.path, t.path), t.contents, "utf-8");
-    const sha = await commit(
-      site.path,
-      `chore: sync ${t.config} config from @reddoorla/maintenance`
-    );
-    if (sha) shas.push(sha);
-  }
-  if (gitignorePlan.kind === "apply") {
-    await applyGitignore(site.path, gitignorePlan);
-    const sha = await commit(site.path, `chore: sync gitignore from @reddoorla/maintenance`);
-    if (sha) shas.push(sha);
-  }
-  return {
-    recipe: "sync-configs",
-    site: label,
-    status: "applied",
-    commits: shas,
-    notes: `branch: ${branch}`
-  };
+  return withRecipe({
+    name: "sync-configs",
+    site,
+    plan: async () => {
+      const templateDiffs = await planTemplateDiffs(site.path, templates);
+      const gitignorePlan = includeGitignore ? await planGitignore(site.path) : { kind: "noop" };
+      if (templateDiffs.length === 0 && gitignorePlan.kind === "noop") {
+        return { kind: "noop", notes: "all targeted configs already match" };
+      }
+      return { kind: "apply", plan: { templateDiffs, gitignorePlan } };
+    },
+    apply: async ({ templateDiffs, gitignorePlan }, { commit: commit2 }) => {
+      for (const t of templateDiffs) {
+        await writeFile3(join5(site.path, t.path), t.contents, "utf-8");
+        await commit2(`chore: sync ${t.config} config from @reddoorla/maintenance`);
+      }
+      if (gitignorePlan.kind === "apply") {
+        await applyGitignore(site.path, gitignorePlan);
+        await commit2(`chore: sync gitignore from @reddoorla/maintenance`);
+      }
+      return { kind: "ok" };
+    }
+  });
 }
 
 // src/recipes/bump-deps.ts
@@ -938,62 +982,51 @@ function upFlagsForGroup(group) {
   return [];
 }
 async function bumpDeps(site, opts = {}) {
-  const label = siteLabel(site);
   const group = opts.group ?? "minor";
   const spawn2 = opts.spawn ?? defaultSpawn;
-  const hasPnpmLock = await exists(join6(site.path, "pnpm-lock.yaml"));
-  if (!hasPnpmLock) {
-    const hasNpmLock = await exists(join6(site.path, "package-lock.json"));
-    const hasYarnLock = await exists(join6(site.path, "yarn.lock"));
-    if (hasNpmLock || hasYarnLock) {
-      const competing = hasNpmLock ? "package-lock.json" : "yarn.lock";
-      return {
-        recipe: "bump-deps",
-        site: label,
-        status: "failed",
-        commits: [],
-        notes: `site has ${competing} but no pnpm-lock.yaml \u2014 run convert-to-pnpm first`
-      };
+  return withRecipe({
+    name: "bump-deps",
+    site,
+    // pnpm install (in plan) mutates the lockfile, so the clean-tree check
+    // MUST happen first — otherwise a desynced-lockfile resync would silently
+    // land on top of whatever else was in the tree.
+    checkTreeFirst: true,
+    plan: async () => {
+      const hasPnpmLock = await exists(join6(site.path, "pnpm-lock.yaml"));
+      if (!hasPnpmLock) {
+        const hasNpmLock = await exists(join6(site.path, "package-lock.json"));
+        const hasYarnLock = await exists(join6(site.path, "yarn.lock"));
+        if (hasNpmLock || hasYarnLock) {
+          const competing = hasNpmLock ? "package-lock.json" : "yarn.lock";
+          return {
+            kind: "failed",
+            notes: `site has ${competing} but no pnpm-lock.yaml \u2014 run convert-to-pnpm first`
+          };
+        }
+      }
+      await spawn2("pnpm", ["install"], { cwd: site.path, streaming: true });
+      const outdated = await spawn2(
+        "pnpm",
+        ["outdated", "--json", ...outdatedFlagsForGroup(group)],
+        { cwd: site.path }
+      );
+      let parsed;
+      try {
+        parsed = JSON.parse(outdated.stdout || "{}");
+      } catch {
+        parsed = {};
+      }
+      if (Object.keys(parsed).length === 0) {
+        return { kind: "noop", notes: `pnpm outdated reported nothing for group=${group}` };
+      }
+      return { kind: "apply", plan: { group } };
+    },
+    apply: async ({ group: g }, { commit: commit2, cwd }) => {
+      await spawn2("pnpm", ["up", ...upFlagsForGroup(g)], { cwd, streaming: true });
+      await commit2(`chore(deps): bump dependencies (${g})`);
+      return { kind: "ok" };
     }
-  }
-  if (!await isWorkingTreeClean(site.path)) {
-    throw new Error(`refusing to run: working tree is not clean at ${site.path}`);
-  }
-  await spawn2("pnpm", ["install"], { cwd: site.path, streaming: true });
-  const outdated = await spawn2("pnpm", ["outdated", "--json", ...outdatedFlagsForGroup(group)], {
-    cwd: site.path
   });
-  let parsed;
-  try {
-    parsed = JSON.parse(outdated.stdout || "{}");
-  } catch {
-    parsed = {};
-  }
-  const nothingToDo = Object.keys(parsed).length === 0;
-  if (nothingToDo) {
-    return {
-      recipe: "bump-deps",
-      site: label,
-      status: "noop",
-      commits: [],
-      notes: `pnpm outdated reported nothing for group=${group}`
-    };
-  }
-  const branch = branchName("bump-deps");
-  await createBranch(site.path, branch);
-  await spawn2("pnpm", ["up", ...upFlagsForGroup(group)], {
-    cwd: site.path,
-    streaming: true
-  });
-  const sha = await commit(site.path, `chore(deps): bump dependencies (${group})`);
-  const shas = sha ? [sha] : [];
-  return {
-    recipe: "bump-deps",
-    site: label,
-    status: shas.length > 0 ? "applied" : "noop",
-    commits: shas,
-    notes: `branch: ${branch}`
-  };
 }
 
 // src/recipes/svelte-5/index.ts
@@ -1573,108 +1606,73 @@ async function alreadyOnSvelte5(cwd) {
   }
 }
 async function upgradeSvelte4to5(site, opts = {}) {
-  const label = siteLabel(site);
   const spawn2 = opts.spawn ?? defaultSpawn;
-  if (await alreadyOnSvelte5(site.path)) {
-    return {
-      recipe: "svelte-4-to-5",
-      site: label,
-      status: "noop",
-      commits: [],
-      notes: "site already declares svelte ^5.x"
-    };
-  }
-  if (!await isWorkingTreeClean(site.path)) {
-    throw new Error(`refusing to run: working tree is not clean at ${site.path}`);
-  }
-  const branch = branchName("svelte-4-to-5");
-  await createBranch(site.path, branch);
-  const shas = [];
-  const bumped = await bumpToSvelte5Versions(site.path);
-  if (bumped) {
-    const sha = await commit(site.path, "chore(svelte5): bump svelte/kit/vite/vite-plugin-svelte");
-    if (sha) shas.push(sha);
-  }
-  const configChanged = await migrateSvelteConfig(site.path);
-  if (configChanged) {
-    const sha = await commit(
-      site.path,
-      "refactor(svelte5): migrate svelte.config.js (drop vitePreprocess)"
-    );
-    if (sha) shas.push(sha);
-  }
-  const migrate = await runSvelteMigrate(site.path, spawn2);
-  if (migrate.ran) {
-    const sha = await commit(site.path, "refactor(svelte5): run official svelte-migrate codemod");
-    if (sha) shas.push(sha);
-  }
-  const tw = await upgradeTailwind(site.path, spawn2);
-  if (tw.ran) {
-    const sha = await commit(site.path, "chore(svelte5): tailwindcss 3 \u2192 4 upgrade");
-    if (sha) shas.push(sha);
-  }
-  const codemods = await applyGotchaCodemods(site.path);
-  if (codemods.filesChanged > 0) {
-    const sha = await commit(
-      site.path,
-      `refactor(svelte5): apply gotcha codemods (${codemods.filesChanged} files)`
-    );
-    if (sha) shas.push(sha);
-  }
-  await verifyMigration(site.path, spawn2);
-  const verifySha = await commit(site.path, "chore(svelte5): pnpm install + check");
-  if (verifySha) shas.push(verifySha);
-  await writeMigrationSummary({
-    cwd: site.path,
-    filesChangedByCodemods: codemods.filesChanged,
-    svelteMigrateRan: migrate.ran,
-    tailwindUpgraded: tw.ran
+  return withRecipe({
+    name: "svelte-4-to-5",
+    site,
+    plan: async () => {
+      if (await alreadyOnSvelte5(site.path)) {
+        return { kind: "noop", notes: "site already declares svelte ^5.x" };
+      }
+      return { kind: "apply", plan: true };
+    },
+    apply: async (_plan, { commit: commit2, cwd }) => {
+      const bumped = await bumpToSvelte5Versions(cwd);
+      if (bumped) {
+        await commit2("chore(svelte5): bump svelte/kit/vite/vite-plugin-svelte");
+      }
+      const configChanged = await migrateSvelteConfig(cwd);
+      if (configChanged) {
+        await commit2("refactor(svelte5): migrate svelte.config.js (drop vitePreprocess)");
+      }
+      const migrate = await runSvelteMigrate(cwd, spawn2);
+      if (migrate.ran) {
+        await commit2("refactor(svelte5): run official svelte-migrate codemod");
+      }
+      const tw = await upgradeTailwind(cwd, spawn2);
+      if (tw.ran) {
+        await commit2("chore(svelte5): tailwindcss 3 \u2192 4 upgrade");
+      }
+      const codemods = await applyGotchaCodemods(cwd);
+      if (codemods.filesChanged > 0) {
+        await commit2(`refactor(svelte5): apply gotcha codemods (${codemods.filesChanged} files)`);
+      }
+      await verifyMigration(cwd, spawn2);
+      await commit2("chore(svelte5): pnpm install + check");
+      await writeMigrationSummary({
+        cwd,
+        filesChangedByCodemods: codemods.filesChanged,
+        svelteMigrateRan: migrate.ran,
+        tailwindUpgraded: tw.ran
+      });
+      await commit2("docs(svelte5): add MIGRATION_SVELTE_5.md summary");
+      return { kind: "ok" };
+    }
   });
-  const summarySha = await commit(site.path, "docs(svelte5): add MIGRATION_SVELTE_5.md summary");
-  if (summarySha) shas.push(summarySha);
-  return {
-    recipe: "svelte-4-to-5",
-    site: label,
-    status: shas.length > 0 ? "applied" : "noop",
-    commits: shas,
-    notes: `branch: ${branch}`
-  };
 }
 
 // src/recipes/svelte-codemods.ts
 import { writeFile as writeFile8 } from "fs/promises";
 import { join as join13 } from "path";
 async function svelteCodemods(site) {
-  const label = siteLabel(site);
-  const changes = await planGotchaCodemods(site.path);
-  if (changes.length === 0) {
-    return {
-      recipe: "svelte-codemods",
-      site: label,
-      status: "noop",
-      commits: [],
-      notes: "no codemod targets matched"
-    };
-  }
-  if (!await isWorkingTreeClean(site.path)) {
-    throw new Error(`refusing to run: working tree is not clean at ${site.path}`);
-  }
-  const branch = branchName("svelte-codemods");
-  await createBranch(site.path, branch);
-  for (const c of changes) {
-    await writeFile8(join13(site.path, c.rel), c.after, "utf-8");
-  }
-  const sha = await commit(
-    site.path,
-    `refactor(svelte5): apply codemods (${changes.length} files)`
-  );
-  return {
-    recipe: "svelte-codemods",
-    site: label,
-    status: "applied",
-    commits: sha ? [sha] : [],
-    notes: `branch: ${branch}`
-  };
+  return withRecipe({
+    name: "svelte-codemods",
+    site,
+    plan: async () => {
+      const changes = await planGotchaCodemods(site.path);
+      if (changes.length === 0) {
+        return { kind: "noop", notes: "no codemod targets matched" };
+      }
+      return { kind: "apply", plan: changes };
+    },
+    apply: async (changes, { commit: commit2, cwd }) => {
+      for (const c of changes) {
+        await writeFile8(join13(cwd, c.rel), c.after, "utf-8");
+      }
+      await commit2(`refactor(svelte5): apply codemods (${changes.length} files)`);
+      return { kind: "ok" };
+    }
+  });
 }
 
 // src/recipes/convert-to-pnpm.ts
@@ -1710,80 +1708,55 @@ async function exists2(path) {
   }
 }
 async function convertToPnpm(site, opts = {}) {
-  const label = siteLabel(site);
   const spawn2 = opts.spawn ?? defaultSpawn;
   const pnpmVersion = opts.pnpmVersion ?? DEFAULT_PNPM_VERSION;
   const pnpmLockPath = join14(site.path, "pnpm-lock.yaml");
   const npmLockPath = join14(site.path, "package-lock.json");
   const yarnLockPath = join14(site.path, "yarn.lock");
-  if (await exists2(pnpmLockPath)) {
-    return {
-      recipe: "convert-to-pnpm",
-      site: label,
-      status: "noop",
-      commits: [],
-      notes: "site already has pnpm-lock.yaml"
-    };
-  }
-  const hasNpmLock = await exists2(npmLockPath);
-  const hasYarnLock = await exists2(yarnLockPath);
-  if (!hasNpmLock && !hasYarnLock) {
-    return {
-      recipe: "convert-to-pnpm",
-      site: label,
-      status: "noop",
-      commits: [],
-      notes: "no convertible lockfile (package-lock.json or yarn.lock) at site root"
-    };
-  }
-  if (!await isWorkingTreeClean(site.path)) {
-    throw new Error(`refusing to run: working tree is not clean at ${site.path}`);
-  }
-  const branch = branchName("convert-to-pnpm");
-  await createBranch(site.path, branch);
-  const shas = [];
-  if (hasNpmLock) await rm3(npmLockPath, { force: true });
-  if (hasYarnLock) await rm3(yarnLockPath, { force: true });
-  const sourceLock = hasNpmLock ? "package-lock.json" : "yarn.lock";
-  const lockSha = await commit(site.path, `chore(pnpm): remove ${sourceLock}`);
-  if (lockSha) shas.push(lockSha);
-  const pkgPath = join14(site.path, "package.json");
-  const pkg = await readPackageJson(pkgPath);
-  const next = { ...pkg, packageManager: `pnpm@${pnpmVersion}` };
-  if (pkg.scripts && typeof pkg.scripts === "object") {
-    const { scripts: rewritten, changedCount } = rewriteScriptsForPnpm(
-      pkg.scripts
-    );
-    if (changedCount > 0) {
-      next.scripts = rewritten;
+  return withRecipe({
+    name: "convert-to-pnpm",
+    site,
+    plan: async () => {
+      if (await exists2(pnpmLockPath)) {
+        return { kind: "noop", notes: "site already has pnpm-lock.yaml" };
+      }
+      const hasNpmLock = await exists2(npmLockPath);
+      const hasYarnLock = await exists2(yarnLockPath);
+      if (!hasNpmLock && !hasYarnLock) {
+        return {
+          kind: "noop",
+          notes: "no convertible lockfile (package-lock.json or yarn.lock) at site root"
+        };
+      }
+      return { kind: "apply", plan: { hasNpmLock, hasYarnLock } };
+    },
+    apply: async ({ hasNpmLock, hasYarnLock }, { commit: commit2, cwd }) => {
+      if (hasNpmLock) await rm3(npmLockPath, { force: true });
+      if (hasYarnLock) await rm3(yarnLockPath, { force: true });
+      const sourceLock = hasNpmLock ? "package-lock.json" : "yarn.lock";
+      await commit2(`chore(pnpm): remove ${sourceLock}`);
+      const pkgPath = join14(cwd, "package.json");
+      const pkg = await readPackageJson(pkgPath);
+      const next = { ...pkg, packageManager: `pnpm@${pnpmVersion}` };
+      if (pkg.scripts && typeof pkg.scripts === "object") {
+        const { scripts: rewritten, changedCount } = rewriteScriptsForPnpm(
+          pkg.scripts
+        );
+        if (changedCount > 0) {
+          next.scripts = rewritten;
+        }
+      }
+      await writePackageJson(pkgPath, next);
+      await commit2("chore(pnpm): pin packageManager + rewrite npm scripts");
+      await rm3(join14(cwd, "node_modules"), { recursive: true, force: true });
+      const installResult = await spawn2("pnpm", ["install"], { cwd, streaming: true });
+      if (installResult.code !== 0) {
+        return { kind: "failed", notes: `pnpm install failed (exit ${installResult.code})` };
+      }
+      await commit2("chore(pnpm): add pnpm-lock.yaml");
+      return { kind: "ok" };
     }
-  }
-  await writePackageJson(pkgPath, next);
-  const pkgSha = await commit(site.path, "chore(pnpm): pin packageManager + rewrite npm scripts");
-  if (pkgSha) shas.push(pkgSha);
-  await rm3(join14(site.path, "node_modules"), { recursive: true, force: true });
-  const installResult = await spawn2("pnpm", ["install"], {
-    cwd: site.path,
-    streaming: true
   });
-  if (installResult.code !== 0) {
-    return {
-      recipe: "convert-to-pnpm",
-      site: label,
-      status: "failed",
-      commits: shas,
-      notes: `pnpm install failed (exit ${installResult.code}). branch ${branch} left for inspection.`
-    };
-  }
-  const installSha = await commit(site.path, "chore(pnpm): add pnpm-lock.yaml");
-  if (installSha) shas.push(installSha);
-  return {
-    recipe: "convert-to-pnpm",
-    site: label,
-    status: "applied",
-    commits: shas,
-    notes: `branch: ${branch}`
-  };
 }
 
 // src/recipes/onboard.ts
@@ -1840,74 +1813,59 @@ function isDeclared(pkg, name) {
   return Boolean(pkg.dependencies?.[name] ?? pkg.devDependencies?.[name]);
 }
 async function onboard(site, opts = {}) {
-  const label = siteLabel(site);
   const spawn2 = opts.spawn ?? defaultSpawn;
   const audits = opts.audits ?? ["lighthouse", "a11y"];
   const packageVersion = opts.packageVersion ?? selfCaretRange(import.meta.url);
-  if (!await exists3(join16(site.path, "pnpm-lock.yaml"))) {
-    return {
-      recipe: "onboard",
-      site: label,
-      status: "failed",
-      commits: [],
-      notes: "no pnpm-lock.yaml at site root \u2014 run convert-to-pnpm first"
-    };
-  }
-  const pkgPath = join16(site.path, "package.json");
-  const pkg = await readPackageJson(pkgPath);
-  const toAdd = [];
-  if (!isDeclared(pkg, PACKAGE_NAME)) {
-    toAdd.push({ name: PACKAGE_NAME, version: packageVersion });
-  }
-  for (const audit of audits) {
-    for (const dep of AUDIT_DEPS[audit]) {
-      if (!isDeclared(pkg, dep.name)) toAdd.push(dep);
+  return withRecipe({
+    name: "onboard",
+    site,
+    plan: async () => {
+      if (!await exists3(join16(site.path, "pnpm-lock.yaml"))) {
+        return {
+          kind: "failed",
+          notes: "no pnpm-lock.yaml at site root \u2014 run convert-to-pnpm first"
+        };
+      }
+      const pkgPath = join16(site.path, "package.json");
+      const pkg = await readPackageJson(pkgPath);
+      const toAdd = [];
+      if (!isDeclared(pkg, PACKAGE_NAME)) {
+        toAdd.push({ name: PACKAGE_NAME, version: packageVersion });
+      }
+      for (const audit of audits) {
+        for (const dep of AUDIT_DEPS[audit]) {
+          if (!isDeclared(pkg, dep.name)) toAdd.push(dep);
+        }
+      }
+      if (toAdd.length === 0) {
+        return {
+          kind: "noop",
+          notes: `site already has ${PACKAGE_NAME} and audit deps (${audits.join("+")})`
+        };
+      }
+      return { kind: "apply", plan: { pkg, toAdd } };
+    },
+    apply: async ({ pkg, toAdd }, { commit: commit2, cwd }) => {
+      const pkgPath = join16(cwd, "package.json");
+      let next = pkg;
+      for (const dep of toAdd) {
+        next = bumpDep(next, dep.name, dep.version);
+      }
+      await writePackageJson(pkgPath, next);
+      const installResult = await spawn2("pnpm", ["install"], { cwd, streaming: true });
+      if (installResult.code !== 0) {
+        return {
+          kind: "failed",
+          notes: `pnpm install failed (exit ${installResult.code})`
+        };
+      }
+      await commit2(`chore(reddoor): onboard with ${PACKAGE_NAME} ${packageVersion}`);
+      return {
+        kind: "ok",
+        notes: `Added ${toAdd.length} dep(s): ${toAdd.map((d) => d.name).join(", ")}`
+      };
     }
-  }
-  if (toAdd.length === 0) {
-    return {
-      recipe: "onboard",
-      site: label,
-      status: "noop",
-      commits: [],
-      notes: `site already has ${PACKAGE_NAME} and audit deps (${audits.join("+")})`
-    };
-  }
-  if (!await isWorkingTreeClean(site.path)) {
-    throw new Error(`refusing to run: working tree is not clean at ${site.path}`);
-  }
-  const branch = branchName("onboard");
-  await createBranch(site.path, branch);
-  let next = pkg;
-  for (const dep of toAdd) {
-    next = bumpDep(next, dep.name, dep.version);
-  }
-  await writePackageJson(pkgPath, next);
-  const installResult = await spawn2("pnpm", ["install"], {
-    cwd: site.path,
-    streaming: true
   });
-  if (installResult.code !== 0) {
-    return {
-      recipe: "onboard",
-      site: label,
-      status: "failed",
-      commits: [],
-      notes: `pnpm install failed (exit ${installResult.code}). branch ${branch} left for inspection.`
-    };
-  }
-  const sha = await commit(
-    site.path,
-    `chore(reddoor): onboard with ${PACKAGE_NAME} ${packageVersion}`
-  );
-  const shas = sha ? [sha] : [];
-  return {
-    recipe: "onboard",
-    site: label,
-    status: "applied",
-    commits: shas,
-    notes: `branch: ${branch}. Added ${toAdd.length} dep(s): ${toAdd.map((d) => d.name).join(", ")}`
-  };
 }
 
 // src/recipes/index.ts