@reddoorla/maintenance 0.6.6 → 0.6.7

package/dist/index.js

@@ -31,6 +31,11 @@ var defaultSpawn = (cmd, args, opts = {}) => new Promise((resolve, reject) => {
 import { readFile } from "fs/promises";
 import { join } from "path";
 
+// src/util/site.ts
+function siteLabel(site) {
+  return site.name ?? site.path;
+}
+
 // src/configs/baseline-versions.ts
 var baselineVersions = {
   // SvelteKit core
@@ -71,9 +76,6 @@ var baselineVersions = {
 };
 
 // src/audits/deps.ts
-function siteLabel(site) {
-  return site.name ?? site.path;
-}
 function stripCaret(range) {
   return range.replace(/^[\^~]/, "");
 }
@@ -146,9 +148,6 @@ import { check as prettierCheck, resolveConfig as prettierResolveConfig } from "
 import { glob } from "tinyglobby";
 var TARGET_GLOBS = ["**/*.{ts,js,svelte}"];
 var IGNORE = ["node_modules/**", "dist/**", ".svelte-kit/**", "build/**", ".netlify/**"];
-function siteLabel2(site) {
-  return site.name ?? site.path;
-}
 async function listFiles(cwd) {
   return glob(TARGET_GLOBS, { cwd, ignore: IGNORE, absolute: false });
 }
@@ -158,7 +157,7 @@ async function lintAudit(ctx) {
   if (!existsSync(configPath)) {
     return {
       audit: "lint",
-      site: siteLabel2(site),
+      site: siteLabel(site),
       status: "skip",
       summary: "no eslint config at site root"
     };
@@ -184,7 +183,7 @@ async function lintAudit(ctx) {
   const summary = status === "pass" ? `lint clean across ${relFiles.length} files` : `${eslintErrors} eslint errors, ${eslintWarnings} warnings, ${prettierUnformatted.length} unformatted`;
   return {
     audit: "lint",
-    site: siteLabel2(site),
+    site: siteLabel(site),
     status,
     summary,
     details: {
@@ -197,9 +196,6 @@ async function lintAudit(ctx) {
 }
 
 // src/audits/security.ts
-function siteLabel3(site) {
-  return site.name ?? site.path;
-}
 function classify(v) {
   if (v.critical > 0 || v.high > 0) return "fail";
   if (v.moderate > 0 || v.low > 0) return "warn";
@@ -285,7 +281,8 @@ async function runAuditTool(spawn2, cmd, args, cwd) {
   if (errEnvelope && typeof errEnvelope === "object") {
     return { kind: "error", reason: errEnvelope.code ?? "error envelope returned" };
   }
-  if (!parsed.metadata?.vulnerabilities) {
+  const vulnsMeta = parsed.metadata?.vulnerabilities;
+  if (!vulnsMeta || Object.keys(vulnsMeta).length === 0) {
     return { kind: "error", reason: "no metadata.vulnerabilities in output" };
   }
   return { kind: "ok", parsed };
@@ -293,7 +290,7 @@ async function runAuditTool(spawn2, cmd, args, cwd) {
 async function securityAudit(ctx) {
   const spawn2 = ctx.spawn ?? defaultSpawn;
   const site = ctx.site;
-  const label = siteLabel3(site);
+  const label = siteLabel(site);
   let used = "pnpm audit";
   let result = await runAuditTool(spawn2, "pnpm", ["audit", "--json", "--prod"], site.path);
   if (result.kind !== "ok") {
@@ -374,9 +371,6 @@ var lighthouseConfig = {
 };
 
 // src/audits/lighthouse.ts
-function siteLabel4(site) {
-  return site.name ?? site.path;
-}
 async function readJsonMaybe(path) {
   try {
     const raw = await readFile3(path, "utf-8");
@@ -414,7 +408,7 @@ function messageForAssertion(a) {
 async function lighthouseAudit(ctx) {
   const spawn2 = ctx.spawn ?? defaultSpawn;
   const site = ctx.site;
-  const label = siteLabel4(site);
+  const label = siteLabel(site);
   const configDir = await mkdtemp(join3(tmpdir(), "reddoor-lhci-"));
   const configPath = join3(configDir, "lighthouserc.json");
   await writeFile(configPath, JSON.stringify(lighthouseConfig), "utf-8");
@@ -512,9 +506,6 @@ var playwrightA11yConfig = defineConfig({
 
 // src/audits/a11y.ts
 var RESULTS_REL = ".reddoor-a11y/results.json";
-function siteLabel5(site) {
-  return site.name ?? site.path;
-}
 async function readJsonMaybe2(path) {
   try {
     const raw = await readFile4(path, "utf-8");
@@ -572,7 +563,7 @@ test("a11y across configured routes", async ({ page }) => {
 async function a11yAudit(ctx) {
   const spawn2 = ctx.spawn ?? defaultSpawn;
   const site = ctx.site;
-  const label = siteLabel5(site);
+  const label = siteLabel(site);
   const specDir = await mkdtemp2(join4(tmpdir2(), "reddoor-a11y-spec-"));
   const specPath = join4(specDir, "a11y.spec.ts");
   await writeFile2(specPath, buildSpec(), "utf-8");
@@ -820,9 +811,8 @@ async function git(cwd, args) {
   return exec("git", args, { cwd, env: process.env });
 }
 function branchName(recipe, when = /* @__PURE__ */ new Date()) {
-  const iso = when.toISOString().replace(/[-:.]/g, "").replace(/Z$/, "Z");
-  const trimmed = iso.replace(/(\d{8}T\d{6})\d+(Z)$/, "$1$2");
-  return `maint/${recipe}-${trimmed}`;
+  const compact = when.toISOString().replace(/[-:.]/g, "");
+  return `maint/${recipe}-${compact}`;
 }
 async function isWorkingTreeClean(cwd) {
   const { stdout } = await git(cwd, ["status", "--porcelain"]);
@@ -853,9 +843,6 @@ async function commit(cwd, message) {
 
 // src/recipes/sync-configs.ts
 var GITIGNORE_CONFIG = "gitignore";
-function siteLabel6(site) {
-  return site.name ?? site.path;
-}
 async function readMaybe(path) {
   try {
     return await readFile5(path, "utf-8");
@@ -886,7 +873,7 @@ async function applyGitignore(cwd, plan) {
   }
 }
 async function syncConfigs(site, opts = {}) {
-  const label = siteLabel6(site);
+  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);
@@ -933,9 +920,6 @@ 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);
@@ -954,7 +938,7 @@ function upFlagsForGroup(group) {
   return [];
 }
 async function bumpDeps(site, opts = {}) {
-  const label = siteLabel7(site);
+  const label = siteLabel(site);
   const group = opts.group ?? "minor";
   const spawn2 = opts.spawn ?? defaultSpawn;
   const hasPnpmLock = await exists(join6(site.path, "pnpm-lock.yaml"));
@@ -1196,6 +1180,34 @@ import { glob as glob2 } from "tinyglobby";
 // src/recipes/svelte-5/codemods/on-event-to-handler.ts
 var SCRIPT_BLOCK = /<script\b[^>]*>[\s\S]*?<\/script>/g;
 var SIMPLE_ON_EVENT = /\bon:([a-z]+)(?=\s*=)/g;
+var MODIFIER_EVENT = /\bon:[a-z]+\|[a-zA-Z]+(?=\s*=)/g;
+function flagEventModifiers(source) {
+  const insertions = [];
+  let m;
+  MODIFIER_EVENT.lastIndex = 0;
+  while ((m = MODIFIER_EVENT.exec(source)) !== null) {
+    const tagStart = source.lastIndexOf("<", m.index);
+    if (tagStart === -1) continue;
+    const prevLineEnd = tagStart - 1;
+    if (prevLineEnd >= 0) {
+      const prevLineStart = source.lastIndexOf("\n", prevLineEnd - 1) + 1;
+      const prevLine = source.slice(prevLineStart, prevLineEnd + 1);
+      if (/<!--\s*@migration-task/.test(prevLine)) continue;
+    }
+    const lineStart = source.lastIndexOf("\n", tagStart - 1) + 1;
+    const indent = source.slice(lineStart, tagStart);
+    const safeIndent = /^[ \t]*$/.test(indent) ? indent : "";
+    insertions.push({ tagStart, indent: safeIndent, modifier: m[0] });
+  }
+  let out = source;
+  for (let i = insertions.length - 1; i >= 0; i--) {
+    const { tagStart, indent, modifier } = insertions[i];
+    const comment = `<!-- @migration-task: Svelte 5 removed event modifier syntax (\`${modifier}\`). Rewrite inline, e.g. onclick={(e) => { e.preventDefault(); ... }}. -->
+${indent}`;
+    out = out.slice(0, tagStart) + comment + out.slice(tagStart);
+  }
+  return out;
+}
 function onEventToHandler(source) {
   const masked = [];
   const placeholder = (i) => ` SCRIPT_${i} `;
@@ -1203,8 +1215,9 @@ function onEventToHandler(source) {
     masked.push(match);
     return placeholder(masked.length - 1);
   });
-  const rewritten = intermediate.replace(SIMPLE_ON_EVENT, (_full, name) => `on${name}`);
-  let out = rewritten;
+  let processed = intermediate.replace(SIMPLE_ON_EVENT, (_full, name) => `on${name}`);
+  processed = flagEventModifiers(processed);
+  let out = processed;
   masked.forEach((blk, i) => {
     out = out.replace(placeholder(i), blk);
   });
@@ -1254,6 +1267,22 @@ function exportLetToProps(source) {
   return source.replace(SCRIPT_BLOCK2, (full) => full.replace(inner, body));
 }
 
+// src/util/svelte-source.ts
+function findStringEnd(source, openIdx) {
+  const quote = source[openIdx];
+  let i = openIdx + 1;
+  while (i < source.length) {
+    const ch = source[i];
+    if (ch === "\\") {
+      i += 2;
+      continue;
+    }
+    if (ch === quote) return i;
+    i++;
+  }
+  return -1;
+}
+
 // src/recipes/svelte-5/codemods/dollar-restprops.ts
 function removeInterfaceBlock(source) {
   const re = /^\s*interface\s+\$\$Props\s*\{/m;
@@ -1277,20 +1306,6 @@ function removeInterfaceBlock(source) {
     out = out.slice(0, match.index) + out.slice(endIdx);
   }
 }
-function findStringEnd(source, openIdx) {
-  const quote = source[openIdx];
-  let i = openIdx + 1;
-  while (i < source.length) {
-    const ch = source[i];
-    if (ch === "\\") {
-      i += 2;
-      continue;
-    }
-    if (ch === quote) return i;
-    i++;
-  }
-  return -1;
-}
 function maskStringLiterals(source) {
   const strings = [];
   let out = "";
@@ -1367,7 +1382,8 @@ function stateEffectSyncToDerived(source) {
 
 // src/recipes/svelte-5/codemods/dollar-props-class.ts
 var PROPS_DESTRUCTURE = /let\s*\{([\s\S]*?)\}(\s*:\s*\{([\s\S]*?)\})?\s*=\s*\$props\(\)/;
-var DOLLAR_PROPS_CLASS = /\$\$props\.class\b/g;
+var HAS_DOLLAR_PROPS_CLASS = /\$\$props\.class\b/;
+var DOLLAR_PROPS_CLASS_GLOBAL = /\$\$props\.class\b/g;
 var DOLLAR_PROPS_ANY = /\$\$props\b/;
 var SCRIPT_BLOCK4 = /<script\b[^>]*>[\s\S]*?<\/script>/g;
 var MIGRATION_TASK = /^<!--\s*@migration-task[\s\S]*?-->\s*\n?/gm;
@@ -1389,8 +1405,7 @@ function restoreScripts(masked, blocks) {
 }
 function dollarPropsClass(source) {
   const { masked } = maskScripts(source);
-  if (!DOLLAR_PROPS_CLASS.test(masked)) return source;
-  DOLLAR_PROPS_CLASS.lastIndex = 0;
+  if (!HAS_DOLLAR_PROPS_CLASS.test(masked)) return source;
   if (!PROPS_DESTRUCTURE.test(source)) return source;
   let updated = source.replace(PROPS_DESTRUCTURE, (full, body, typeAnno, typeBody) => {
     if (/\bclass\s*:/.test(body)) return full;
@@ -1404,7 +1419,7 @@ function dollarPropsClass(source) {
     return `let { ${newBody} } = $props()`;
   });
   const reMasked = maskScripts(updated);
-  const templateRewritten = reMasked.masked.replace(DOLLAR_PROPS_CLASS, IDENT);
+  const templateRewritten = reMasked.masked.replace(DOLLAR_PROPS_CLASS_GLOBAL, IDENT);
   updated = restoreScripts(templateRewritten, reMasked.blocks);
   const stripped = updated.replace(MIGRATION_TASK, "");
   if (!DOLLAR_PROPS_ANY.test(stripped)) {
@@ -1423,7 +1438,7 @@ function findMatchingClose(source, openIdx) {
   while (i < source.length) {
     const ch = source[i];
     if (ch === '"' || ch === "'" || ch === "`") {
-      const closeStr = findStringClose(source, i);
+      const closeStr = findStringEnd(source, i);
       if (closeStr === -1) return -1;
       i = closeStr + 1;
       continue;
@@ -1437,20 +1452,6 @@ function findMatchingClose(source, openIdx) {
   }
   return -1;
 }
-function findStringClose(source, openIdx) {
-  const quote = source[openIdx];
-  let i = openIdx + 1;
-  while (i < source.length) {
-    const ch = source[i];
-    if (ch === "\\") {
-      i += 2;
-      continue;
-    }
-    if (ch === quote) return i;
-    i++;
-  }
-  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 = [];
@@ -1562,9 +1563,6 @@ async function writeMigrationSummary(input) {
 }
 
 // src/recipes/svelte-5/index.ts
-function siteLabel8(site) {
-  return site.name ?? site.path;
-}
 async function alreadyOnSvelte5(cwd) {
   try {
     const pkg = await readPackageJson(join12(cwd, "package.json"));
@@ -1575,7 +1573,7 @@ async function alreadyOnSvelte5(cwd) {
   }
 }
 async function upgradeSvelte4to5(site, opts = {}) {
-  const label = siteLabel8(site);
+  const label = siteLabel(site);
   const spawn2 = opts.spawn ?? defaultSpawn;
   if (await alreadyOnSvelte5(site.path)) {
     return {
@@ -1646,11 +1644,8 @@ async function upgradeSvelte4to5(site, opts = {}) {
 // src/recipes/svelte-codemods.ts
 import { writeFile as writeFile8 } from "fs/promises";
 import { join as join13 } from "path";
-function siteLabel9(site) {
-  return site.name ?? site.path;
-}
 async function svelteCodemods(site) {
-  const label = siteLabel9(site);
+  const label = siteLabel(site);
   const changes = await planGotchaCodemods(site.path);
   if (changes.length === 0) {
     return {
@@ -1714,11 +1709,8 @@ async function exists2(path) {
     return false;
   }
 }
-function siteLabel10(site) {
-  return site.name ?? site.path;
-}
 async function convertToPnpm(site, opts = {}) {
-  const label = siteLabel10(site);
+  const label = siteLabel(site);
   const spawn2 = opts.spawn ?? defaultSpawn;
   const pnpmVersion = opts.pnpmVersion ?? DEFAULT_PNPM_VERSION;
   const pnpmLockPath = join14(site.path, "pnpm-lock.yaml");
@@ -1844,14 +1836,11 @@ async function exists3(path) {
     return false;
   }
 }
-function siteLabel11(site) {
-  return site.name ?? site.path;
-}
 function isDeclared(pkg, name) {
   return Boolean(pkg.dependencies?.[name] ?? pkg.devDependencies?.[name]);
 }
 async function onboard(site, opts = {}) {
-  const label = siteLabel11(site);
+  const label = siteLabel(site);
   const spawn2 = opts.spawn ?? defaultSpawn;
   const audits = opts.audits ?? ["lighthouse", "a11y"];
   const packageVersion = opts.packageVersion ?? selfCaretRange(import.meta.url);