@reddoorla/maintenance 0.6.4 → 0.6.6

package/dist/index.d.ts

@@ -1,5 +1,5 @@
-import { S as Site, a as AuditResult, A as AuditName, b as RecipeResult, R as RecipeName, I as InventoryProvider } from './sync-configs-DRmSAnfV.js';
-export { C as ConfigName, c as SyncConfigsOptions, s as syncConfigs } from './sync-configs-DRmSAnfV.js';
+import { S as Site, b as AuditResult, a as AuditName, c as RecipeResult, R as RecipeName, I as InventoryProvider } from './sync-configs-pMPW4wTq.js';
+export { C as ConfigName, d as SyncConfigsOptions, s as syncConfigs } from './sync-configs-pMPW4wTq.js';
 
 type SpawnResult = {
     code: number;

package/dist/index.js

@@ -931,9 +931,19 @@ async function syncConfigs(site, opts = {}) {
 }
 
 // src/recipes/bump-deps.ts
+import { stat } from "fs/promises";
+import { join as join6 } from "path";
 function siteLabel7(site) {
   return site.name ?? site.path;
 }
+async function exists(path) {
+  try {
+    await stat(path);
+    return true;
+  } catch {
+    return false;
+  }
+}
 function outdatedFlagsForGroup(group) {
   if (group === "major") return ["--latest"];
   if (group === "minor") return [];
@@ -947,6 +957,24 @@ async function bumpDeps(site, opts = {}) {
   const label = siteLabel7(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`
+      };
+    }
+  }
+  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
@@ -967,9 +995,6 @@ async function bumpDeps(site, opts = {}) {
       notes: `pnpm outdated reported nothing for group=${group}`
     };
   }
-  if (!await isWorkingTreeClean(site.path)) {
-    throw new Error(`refusing to run: working tree is not clean at ${site.path}`);
-  }
   const branch = branchName("bump-deps");
   await createBranch(site.path, branch);
   await spawn2("pnpm", ["up", ...upFlagsForGroup(group)], {
@@ -988,7 +1013,7 @@ async function bumpDeps(site, opts = {}) {
 }
 
 // src/recipes/svelte-5/index.ts
-import { join as join11 } from "path";
+import { join as join12 } from "path";
 
 // src/util/pkg.ts
 import { readFile as readFile6, writeFile as writeFile4 } from "fs/promises";
@@ -996,8 +1021,18 @@ async function readPackageJson(path) {
   const raw = await readFile6(path, "utf-8");
   return JSON.parse(raw);
 }
+function detectIndentFromContent(raw) {
+  const match = raw.match(/\n([ \t]+)"/);
+  return match ? match[1] ?? "  " : "  ";
+}
 async function writePackageJson(path, pkg) {
-  const content = JSON.stringify(pkg, null, 2) + "\n";
+  let indent = "  ";
+  try {
+    const existing = await readFile6(path, "utf-8");
+    indent = detectIndentFromContent(existing);
+  } catch {
+  }
+  const content = JSON.stringify(pkg, null, indent) + "\n";
   await writeFile4(path, content, "utf-8");
 }
 function bumpDep(pkg, name, version, opts = {}) {
@@ -1027,7 +1062,7 @@ function bumpDep(pkg, name, version, opts = {}) {
 }
 
 // src/recipes/svelte-5/step-bump-versions.ts
-import { join as join6 } from "path";
+import { join as join7 } from "path";
 var SVELTE_5_VERSIONS = {
   svelte: "^5.55.5",
   "@sveltejs/kit": "^2.59.0",
@@ -1040,7 +1075,7 @@ var SVELTE_5_VERSIONS = {
   "typescript-svelte-plugin": "^0.3.52"
 };
 async function bumpToSvelte5Versions(cwd) {
-  const pkgPath = join6(cwd, "package.json");
+  const pkgPath = join7(cwd, "package.json");
   const pkg = await readPackageJson(pkgPath);
   let next = pkg;
   for (const [name, version] of Object.entries(SVELTE_5_VERSIONS)) {
@@ -1053,7 +1088,7 @@ async function bumpToSvelte5Versions(cwd) {
 
 // src/recipes/svelte-5/step-svelte-config.ts
 import { readFile as readFile7, writeFile as writeFile5 } from "fs/promises";
-import { join as join7 } from "path";
+import { join as join8 } from "path";
 var VITE_PLUGIN_PKG = "@sveltejs/vite-plugin-svelte";
 var IMPORT_FROM_VITE_PLUGIN = new RegExp(
   String.raw`^import\s+\{\s*([^}]+?)\s*\}\s+from\s+["']` + VITE_PLUGIN_PKG.replace(/[/]/g, "\\/") + String.raw`["'];?[ \t]*\n`,
@@ -1094,7 +1129,7 @@ function dropPreprocessKey(source) {
   return source.slice(0, m.index) + source.slice(tailIdx).replace(new RegExp(`^${indent}\\n`), "");
 }
 async function migrateSvelteConfig(cwd) {
-  const path = join7(cwd, "svelte.config.js");
+  const path = join8(cwd, "svelte.config.js");
   let src;
   try {
     src = await readFile7(path, "utf-8");
@@ -1131,9 +1166,9 @@ async function runSvelteMigrate(cwd, spawn2 = defaultSpawn) {
 }
 
 // src/recipes/svelte-5/step-tailwind-upgrade.ts
-import { join as join8 } from "path";
+import { join as join9 } from "path";
 async function upgradeTailwind(cwd, spawn2 = defaultSpawn) {
-  const pkg = await readPackageJson(join8(cwd, "package.json"));
+  const pkg = await readPackageJson(join9(cwd, "package.json"));
   const tailwindVersion = pkg.devDependencies?.tailwindcss ?? pkg.dependencies?.tailwindcss;
   if (!tailwindVersion) return { ran: false, reason: "tailwindcss not installed" };
   if (/^\^?4\./.test(tailwindVersion)) return { ran: false, reason: "already on tailwind 4.x" };
@@ -1155,7 +1190,7 @@ async function upgradeTailwind(cwd, spawn2 = defaultSpawn) {
 
 // src/recipes/svelte-5/step-gotchas.ts
 import { readFile as readFile8, writeFile as writeFile6 } from "fs/promises";
-import { join as join9 } from "path";
+import { join as join10 } from "path";
 import { glob as glob2 } from "tinyglobby";
 
 // src/recipes/svelte-5/codemods/on-event-to-handler.ts
@@ -1416,6 +1451,7 @@ function findStringClose(source, openIdx) {
   }
   return -1;
 }
+var MIGRATION_MARKER = "// @migration-task: $effect won't trigger UI updates on plain `let` bindings \u2014 refine mutated locals to $state or split into per-variable $derived.";
 function transformBlocks(body) {
   const out = [];
   let last = 0;
@@ -1431,6 +1467,8 @@ function transformBlocks(body) {
     out.push(body.slice(last, m.index));
     out.push(leadingNewline);
     const blockBody = body.slice(openBraceIdx + 1, closeBraceIdx);
+    out.push(`${indent}${MIGRATION_MARKER}
+`);
     out.push(`${indent}$effect(() => {${blockBody}});`);
     last = closeBraceIdx + 1;
     BLOCK_REACTIVE_HEAD.lastIndex = last;
@@ -1467,7 +1505,7 @@ async function planGotchaCodemods(cwd) {
   const changes = [];
   const relPaths = await glob2(SVELTE_GLOBS, { cwd, ignore: IGNORE2, absolute: false });
   for (const rel of relPaths) {
-    const path = join9(cwd, rel);
+    const path = join10(cwd, rel);
     const before = await readFile8(path, "utf-8");
     const after = CODEMODS.reduce((s, fn) => fn(s), before);
     if (after !== before) changes.push({ rel, after });
@@ -1477,7 +1515,7 @@ async function planGotchaCodemods(cwd) {
 async function applyGotchaCodemods(cwd) {
   const changes = await planGotchaCodemods(cwd);
   for (const c of changes) {
-    await writeFile6(join9(cwd, c.rel), c.after, "utf-8");
+    await writeFile6(join10(cwd, c.rel), c.after, "utf-8");
   }
   return { filesChanged: changes.length };
 }
@@ -1501,7 +1539,7 @@ async function verifyMigration(cwd, spawn2 = defaultSpawn) {
 
 // src/recipes/svelte-5/step-summary.ts
 import { writeFile as writeFile7 } from "fs/promises";
-import { join as join10 } from "path";
+import { join as join11 } from "path";
 async function writeMigrationSummary(input) {
   const lines = [
     `# Svelte 4 \u2192 5 migration summary`,
@@ -1518,7 +1556,7 @@ async function writeMigrationSummary(input) {
     `- Verify Playwright a11y tests still pass.`
   ];
   const content = lines.join("\n") + "\n";
-  const path = join10(input.cwd, "MIGRATION_SVELTE_5.md");
+  const path = join11(input.cwd, "MIGRATION_SVELTE_5.md");
   await writeFile7(path, content, "utf-8");
   return path;
 }
@@ -1529,7 +1567,7 @@ function siteLabel8(site) {
 }
 async function alreadyOnSvelte5(cwd) {
   try {
-    const pkg = await readPackageJson(join11(cwd, "package.json"));
+    const pkg = await readPackageJson(join12(cwd, "package.json"));
     const v = pkg.devDependencies?.svelte ?? pkg.dependencies?.svelte;
     return !!v && /^\^?5\./.test(v);
   } catch {
@@ -1607,7 +1645,7 @@ async function upgradeSvelte4to5(site, opts = {}) {
 
 // src/recipes/svelte-codemods.ts
 import { writeFile as writeFile8 } from "fs/promises";
-import { join as join12 } from "path";
+import { join as join13 } from "path";
 function siteLabel9(site) {
   return site.name ?? site.path;
 }
@@ -1629,7 +1667,7 @@ async function svelteCodemods(site) {
   const branch = branchName("svelte-codemods");
   await createBranch(site.path, branch);
   for (const c of changes) {
-    await writeFile8(join12(site.path, c.rel), c.after, "utf-8");
+    await writeFile8(join13(site.path, c.rel), c.after, "utf-8");
   }
   const sha = await commit(
     site.path,
@@ -1645,8 +1683,8 @@ async function svelteCodemods(site) {
 }
 
 // src/recipes/convert-to-pnpm.ts
-import { rm as rm3, stat } from "fs/promises";
-import { join as join13 } from "path";
+import { rm as rm3, stat as stat2 } from "fs/promises";
+import { join as join14 } from "path";
 
 // src/recipes/convert-to-pnpm/script-rewrites.ts
 function rewriteScriptForPnpm(script) {
@@ -1668,9 +1706,9 @@ function rewriteScriptsForPnpm(scripts) {
 
 // src/recipes/convert-to-pnpm.ts
 var DEFAULT_PNPM_VERSION = "10.33.1";
-async function exists(path) {
+async function exists2(path) {
   try {
-    await stat(path);
+    await stat2(path);
     return true;
   } catch {
     return false;
@@ -1683,10 +1721,10 @@ async function convertToPnpm(site, opts = {}) {
   const label = siteLabel10(site);
   const spawn2 = opts.spawn ?? defaultSpawn;
   const pnpmVersion = opts.pnpmVersion ?? DEFAULT_PNPM_VERSION;
-  const pnpmLockPath = join13(site.path, "pnpm-lock.yaml");
-  const npmLockPath = join13(site.path, "package-lock.json");
-  const yarnLockPath = join13(site.path, "yarn.lock");
-  if (await exists(pnpmLockPath)) {
+  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,
@@ -1695,8 +1733,8 @@ async function convertToPnpm(site, opts = {}) {
       notes: "site already has pnpm-lock.yaml"
     };
   }
-  const hasNpmLock = await exists(npmLockPath);
-  const hasYarnLock = await exists(yarnLockPath);
+  const hasNpmLock = await exists2(npmLockPath);
+  const hasYarnLock = await exists2(yarnLockPath);
   if (!hasNpmLock && !hasYarnLock) {
     return {
       recipe: "convert-to-pnpm",
@@ -1717,7 +1755,7 @@ async function convertToPnpm(site, opts = {}) {
   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 = join13(site.path, "package.json");
+  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") {
@@ -1731,6 +1769,7 @@ async function convertToPnpm(site, opts = {}) {
   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
@@ -1756,17 +1795,17 @@ async function convertToPnpm(site, opts = {}) {
 }
 
 // src/recipes/onboard.ts
-import { stat as stat2 } from "fs/promises";
-import { join as join15 } from "path";
+import { stat as stat3 } from "fs/promises";
+import { join as join16 } from "path";
 
 // src/util/self-version.ts
 import { readFileSync } from "fs";
 import { fileURLToPath } from "url";
-import { dirname, join as join14 } from "path";
+import { dirname, join as join15 } from "path";
 function selfPackageVersion(callerImportMetaUrl) {
   try {
     const here = dirname(fileURLToPath(callerImportMetaUrl));
-    const raw = readFileSync(join14(here, "..", "..", "package.json"), "utf-8");
+    const raw = readFileSync(join15(here, "..", "..", "package.json"), "utf-8");
     const pkg = JSON.parse(raw);
     return pkg.version ?? "0.0.0";
   } catch {
@@ -1779,16 +1818,27 @@ function selfCaretRange(callerImportMetaUrl) {
 
 // src/recipes/onboard.ts
 var PACKAGE_NAME = "@reddoorla/maintenance";
-var AUDIT_DEPS = {
-  lighthouse: [{ name: "@lhci/cli", version: "^0.15.1" }],
-  a11y: [
-    { name: "@playwright/test", version: "^1.59.1" },
-    { name: "@axe-core/playwright", version: "^4.11.3" }
-  ]
+var AUDIT_DEP_NAMES = {
+  lighthouse: ["@lhci/cli"],
+  a11y: ["@playwright/test", "@axe-core/playwright"]
 };
-async function exists2(path) {
+var AUDIT_DEPS = Object.fromEntries(
+  Object.entries(AUDIT_DEP_NAMES).map(([audit, names]) => [
+    audit,
+    names.map((name) => {
+      const version = baselineVersions[name];
+      if (!version) {
+        throw new Error(
+          `baseline-versions is missing audit dep "${name}" \u2014 add it to src/configs/baseline-versions.ts`
+        );
+      }
+      return { name, version };
+    })
+  ])
+);
+async function exists3(path) {
   try {
-    await stat2(path);
+    await stat3(path);
     return true;
   } catch {
     return false;
@@ -1805,7 +1855,7 @@ async function onboard(site, opts = {}) {
   const spawn2 = opts.spawn ?? defaultSpawn;
   const audits = opts.audits ?? ["lighthouse", "a11y"];
   const packageVersion = opts.packageVersion ?? selfCaretRange(import.meta.url);
-  if (!await exists2(join15(site.path, "pnpm-lock.yaml"))) {
+  if (!await exists3(join16(site.path, "pnpm-lock.yaml"))) {
     return {
       recipe: "onboard",
       site: label,
@@ -1814,7 +1864,7 @@ async function onboard(site, opts = {}) {
       notes: "no pnpm-lock.yaml at site root \u2014 run convert-to-pnpm first"
     };
   }
-  const pkgPath = join15(site.path, "package.json");
+  const pkgPath = join16(site.path, "package.json");
   const pkg = await readPackageJson(pkgPath);
   const toAdd = [];
   if (!isDeclared(pkg, PACKAGE_NAME)) {