@reddoorla/maintenance 0.1.0 → 0.1.1

package/dist/index.js

@@ -202,6 +202,48 @@ function classify(v) {
   if (v.moderate > 0 || v.low > 0) return "warn";
   return "pass";
 }
+function normalizeSeverity(s) {
+  if (s === "low" || s === "moderate" || s === "high" || s === "critical") return s;
+  return "moderate";
+}
+function extractAdvisoriesFromPnpm(parsed) {
+  const out = [];
+  for (const a of Object.values(parsed.advisories ?? {})) {
+    if (!a) continue;
+    out.push({
+      module: a.module_name ?? "unknown",
+      severity: normalizeSeverity(a.severity),
+      title: a.title ?? "(no title)",
+      ...a.cves ? { cves: a.cves } : {},
+      ...a.url ? { url: a.url } : {}
+    });
+  }
+  return out;
+}
+function extractAdvisoriesFromNpm(parsed) {
+  const out = [];
+  for (const [name, v] of Object.entries(parsed.vulnerabilities ?? {})) {
+    if (!v) continue;
+    let title = name;
+    let url;
+    if (Array.isArray(v.via)) {
+      const detailed = v.via.find(
+        (entry) => typeof entry === "object" && entry !== null
+      );
+      if (detailed) {
+        title = detailed.title ?? name;
+        url = detailed.url;
+      }
+    }
+    out.push({
+      module: v.name ?? name,
+      severity: normalizeSeverity(v.severity),
+      title,
+      ...url ? { url } : {}
+    });
+  }
+  return out;
+}
 async function tryRun(spawn2, cmd, args, cwd) {
   try {
     return await spawn2(cmd, args, { cwd });
@@ -255,26 +297,27 @@ async function securityAudit(ctx) {
       details: { error: String(err), stdout: raw.stdout.slice(0, 500) }
     };
   }
-  const vuln = {
+  const counts = {
     low: parsed.metadata?.vulnerabilities?.low ?? 0,
     moderate: parsed.metadata?.vulnerabilities?.moderate ?? 0,
     high: parsed.metadata?.vulnerabilities?.high ?? 0,
     critical: parsed.metadata?.vulnerabilities?.critical ?? 0
   };
-  const status = classify(vuln);
-  const total = vuln.low + vuln.moderate + vuln.high + vuln.critical;
-  const summary = status === "pass" ? `${used}: 0 vulnerabilities` : `${used}: ${total} vulnerabilities (${vuln.critical}C/${vuln.high}H/${vuln.moderate}M/${vuln.low}L)`;
+  const advisories = used === "pnpm audit" ? extractAdvisoriesFromPnpm(parsed) : extractAdvisoriesFromNpm(parsed);
+  const status = classify(counts);
+  const total = counts.low + counts.moderate + counts.high + counts.critical;
+  const summary = status === "pass" ? `${used}: 0 vulnerabilities` : `${used}: ${total} vulnerabilities (${counts.critical}C/${counts.high}H/${counts.moderate}M/${counts.low}L)`;
   return {
     audit: "security",
     site: label,
     status,
     summary,
-    details: vuln
+    details: { counts, advisories }
   };
 }
 
 // src/audits/lighthouse.ts
-import { writeFile, mkdtemp, rm } from "fs/promises";
+import { readFile as readFile3, writeFile, mkdtemp, rm } from "fs/promises";
 import { tmpdir } from "os";
 import { join as join3 } from "path";
 
@@ -310,29 +353,56 @@ var lighthouseConfig = {
 function siteLabel4(site) {
   return site.name ?? site.path;
 }
-function isFakeShape(stdout) {
+async function readJsonMaybe(path) {
   try {
-    const parsed = JSON.parse(stdout);
-    if (typeof parsed.assertionsFailed === "number" && parsed.summary) return parsed;
+    const raw = await readFile3(path, "utf-8");
+    return JSON.parse(raw);
   } catch {
     return null;
   }
-  return null;
+}
+function averageSummaries(entries) {
+  if (entries.length === 0) return {};
+  const sums = {};
+  const counts = {};
+  for (const e of entries) {
+    for (const [k, v] of Object.entries(e.summary ?? {})) {
+      if (typeof v !== "number") continue;
+      sums[k] = (sums[k] ?? 0) + v;
+      counts[k] = (counts[k] ?? 0) + 1;
+    }
+  }
+  const out = {};
+  for (const k of Object.keys(sums)) {
+    const total = sums[k] ?? 0;
+    const count = counts[k] ?? 1;
+    out[k] = total / count;
+  }
+  return out;
+}
+function categoryFromAssertion(a) {
+  const colonIdx = a.name.indexOf(":");
+  return colonIdx >= 0 ? a.name.slice(colonIdx + 1) : a.name;
+}
+function messageForAssertion(a) {
+  return `${a.name} ${a.operator} ${a.expected} (actual: ${a.actual.toFixed(2)})`;
 }
 async function lighthouseAudit(ctx) {
   const spawn2 = ctx.spawn ?? defaultSpawn;
   const site = ctx.site;
   const label = siteLabel4(site);
-  const dir = await mkdtemp(join3(tmpdir(), "reddoor-lhci-"));
-  const configPath = join3(dir, "lighthouserc.json");
+  const configDir = await mkdtemp(join3(tmpdir(), "reddoor-lhci-"));
+  const configPath = join3(configDir, "lighthouserc.json");
   await writeFile(configPath, JSON.stringify(lighthouseConfig), "utf-8");
+  const resultsDir = join3(site.path, ".lighthouseci");
+  await rm(resultsDir, { recursive: true, force: true });
   let raw;
   try {
     raw = await spawn2("npx", ["--yes", "@lhci/cli", "autorun", `--config=${configPath}`], {
       cwd: site.path
     });
   } catch (err) {
-    await rm(dir, { recursive: true, force: true });
+    await rm(configDir, { recursive: true, force: true });
     const e = err;
     if (e.code === "ENOENT" || /ENOENT/.test(String(err))) {
       return {
@@ -344,17 +414,32 @@ async function lighthouseAudit(ctx) {
     }
     throw err;
   }
-  await rm(dir, { recursive: true, force: true });
-  const fake = isFakeShape(raw.stdout);
-  const normalized = fake ?? {
-    summary: {},
-    assertionsFailed: raw.code === 0 ? 0 : 1,
-    assertions: raw.code === 0 ? [] : [{ category: "unknown", level: "error", message: raw.stderr.slice(0, 200) }]
-  };
-  const anyError = (normalized.assertions ?? []).some((a) => a.level === "error");
-  const anyWarn = (normalized.assertions ?? []).some((a) => a.level === "warn");
+  await rm(configDir, { recursive: true, force: true });
+  const manifest = await readJsonMaybe(join3(resultsDir, "manifest.json"));
+  if (!manifest || manifest.length === 0) {
+    return {
+      audit: "lighthouse",
+      site: label,
+      status: "fail",
+      summary: `lighthouse: no manifest written (exit ${raw.code})${raw.stderr ? ` \u2014 ${raw.stderr.slice(0, 200)}` : ""}`
+    };
+  }
+  const assertionResults = await readJsonMaybe(join3(resultsDir, "assertion-results.json")) ?? [];
+  const failed = assertionResults.filter((a) => !a.passed);
+  const assertions = failed.map((a) => ({
+    category: categoryFromAssertion(a),
+    level: a.level,
+    message: messageForAssertion(a)
+  }));
+  const anyError = assertions.some((a) => a.level === "error");
+  const anyWarn = assertions.some((a) => a.level === "warn");
   const status = anyError ? "fail" : anyWarn ? "warn" : "pass";
-  const summary = status === "pass" ? "lighthouse: all categories passing" : `lighthouse: ${normalized.assertionsFailed} assertion(s) failed`;
+  const normalized = {
+    summary: averageSummaries(manifest),
+    assertionsFailed: failed.length,
+    assertions
+  };
+  const summary = status === "pass" ? "lighthouse: all categories passing" : `lighthouse: ${failed.length} assertion(s) failed`;
   return {
     audit: "lighthouse",
     site: label,
@@ -365,7 +450,7 @@ async function lighthouseAudit(ctx) {
 }
 
 // src/audits/a11y.ts
-import { writeFile as writeFile2, mkdtemp as mkdtemp2, rm as rm2 } from "fs/promises";
+import { readFile as readFile4, writeFile as writeFile2, mkdtemp as mkdtemp2, rm as rm2 } from "fs/promises";
 import { tmpdir as tmpdir2 } from "os";
 import { join as join4 } from "path";
 
@@ -401,48 +486,77 @@ var playwrightA11yConfig = defineConfig({
 });
 
 // src/audits/a11y.ts
+var RESULTS_REL = ".reddoor-a11y/results.json";
 function siteLabel5(site) {
   return site.name ?? site.path;
 }
-function isFakeShape2(stdout) {
+async function readJsonMaybe2(path) {
   try {
-    const parsed = JSON.parse(stdout);
-    if (typeof parsed.totalViolations === "number" && parsed.byImpact) return parsed;
+    const raw = await readFile4(path, "utf-8");
+    return JSON.parse(raw);
   } catch {
     return null;
   }
-  return null;
 }
 function buildSpec() {
-  return `
-import { test, expect } from "@playwright/test";
+  return `import { test, expect } from "@playwright/test";
 import AxeBuilder from "@axe-core/playwright";
+import { mkdir, writeFile } from "node:fs/promises";
+import { dirname } from "node:path";
+
 const pages = ${JSON.stringify(a11yRoutes)};
-for (const { path, name } of pages) {
-  test(\`\${name} has no axe violations\`, async ({ page }) => {
+const OUTPUT = process.env.REDDOOR_A11Y_OUTPUT;
+
+test("a11y across configured routes", async ({ page }) => {
+  const violations = [];
+  for (const { path, name } of pages) {
     await page.goto(path);
     const results = await new AxeBuilder({ page })
       .withTags(["wcag2a","wcag2aa","wcag21a","wcag21aa","wcag22aa"])
       .analyze();
-    expect(results.violations).toEqual([]);
-  });
-}
+    for (const v of results.violations) {
+      violations.push({
+        id: v.id,
+        impact: v.impact ?? "moderate",
+        route: name,
+        help: v.help,
+        helpUrl: v.helpUrl,
+        nodes: v.nodes.map((n) => ({ html: n.html, target: n.target })),
+      });
+    }
+  }
+  const byImpact = {};
+  for (const v of violations) {
+    byImpact[v.impact] = (byImpact[v.impact] ?? 0) + 1;
+  }
+  if (OUTPUT) {
+    await mkdir(dirname(OUTPUT), { recursive: true });
+    await writeFile(
+      OUTPUT,
+      JSON.stringify({ totalViolations: violations.length, byImpact, violations }, null, 2),
+    );
+  }
+  expect(violations).toEqual([]);
+});
 `;
 }
 async function a11yAudit(ctx) {
   const spawn2 = ctx.spawn ?? defaultSpawn;
   const site = ctx.site;
   const label = siteLabel5(site);
-  const dir = await mkdtemp2(join4(tmpdir2(), "reddoor-a11y-"));
-  const specPath = join4(dir, "a11y.spec.ts");
+  const specDir = await mkdtemp2(join4(tmpdir2(), "reddoor-a11y-spec-"));
+  const specPath = join4(specDir, "a11y.spec.ts");
   await writeFile2(specPath, buildSpec(), "utf-8");
+  const resultsPath = join4(site.path, RESULTS_REL);
+  await rm2(join4(site.path, ".reddoor-a11y"), { recursive: true, force: true });
   let raw;
   try {
-    raw = await spawn2("npx", ["--yes", "playwright", "test", "--reporter=json", specPath], {
-      cwd: site.path
+    raw = await spawn2("npx", ["--yes", "playwright", "test", "--reporter=line", specPath], {
+      cwd: site.path,
+      env: { ...process.env, REDDOOR_A11Y_OUTPUT: resultsPath }
     });
   } catch (err) {
-    await rm2(dir, { recursive: true, force: true });
+    await rm2(specDir, { recursive: true, force: true });
     const e = err;
     if (e.code === "ENOENT" || /ENOENT/.test(String(err))) {
       return {
@@ -454,22 +568,26 @@ async function a11yAudit(ctx) {
     }
     throw err;
   }
-  await rm2(dir, { recursive: true, force: true });
-  const fake = isFakeShape2(raw.stdout);
-  const normalized = fake ?? {
-    totalViolations: raw.code === 0 ? 0 : 1,
-    byImpact: raw.code === 0 ? {} : { moderate: 1 }
-  };
-  const hasSerious = (normalized.byImpact.serious ?? 0) > 0 || (normalized.byImpact.critical ?? 0) > 0;
-  const hasAny = normalized.totalViolations > 0;
+  await rm2(specDir, { recursive: true, force: true });
+  const artifact = await readJsonMaybe2(resultsPath);
+  if (!artifact) {
+    return {
+      audit: "a11y",
+      site: label,
+      status: "fail",
+      summary: `a11y: no results written (exit ${raw.code})${raw.stderr ? ` \u2014 ${raw.stderr.slice(0, 200)}` : ""}`
+    };
+  }
+  const hasSerious = (artifact.byImpact.serious ?? 0) > 0 || (artifact.byImpact.critical ?? 0) > 0;
+  const hasAny = artifact.totalViolations > 0;
   const status = hasSerious ? "fail" : hasAny ? "warn" : "pass";
-  const summary = status === "pass" ? `a11y: 0 violations across ${a11yRoutes.length} routes` : `a11y: ${normalized.totalViolations} violations`;
+  const summary = status === "pass" ? `a11y: 0 violations across ${a11yRoutes.length} routes` : `a11y: ${artifact.totalViolations} violations`;
   return {
     audit: "a11y",
     site: label,
     status,
     summary,
-    details: normalized
+    details: artifact
   };
 }
 
@@ -512,7 +630,7 @@ async function runAuditsAcross(sites, which) {
 }
 
 // src/recipes/sync-configs.ts
-import { readFile as readFile3, writeFile as writeFile3 } from "fs/promises";
+import { readFile as readFile5, writeFile as writeFile3 } from "fs/promises";
 import { join as join5 } from "path";
 
 // src/recipes/sync-configs/templates.ts
@@ -597,7 +715,7 @@ function siteLabel6(site) {
 }
 async function readMaybe(path) {
   try {
-    return await readFile3(path, "utf-8");
+    return await readFile5(path, "utf-8");
   } catch {
     return null;
   }
@@ -703,9 +821,9 @@ async function bumpDeps(site, opts = {}) {
 import { join as join11 } from "path";
 
 // src/util/pkg.ts
-import { readFile as readFile4, writeFile as writeFile4 } from "fs/promises";
+import { readFile as readFile6, writeFile as writeFile4 } from "fs/promises";
 async function readPackageJson(path) {
-  const raw = await readFile4(path, "utf-8");
+  const raw = await readFile6(path, "utf-8");
   return JSON.parse(raw);
 }
 async function writePackageJson(path, pkg) {
@@ -762,13 +880,13 @@ async function bumpToSvelte5Versions(cwd) {
 }
 
 // src/recipes/svelte-5/step-svelte-config.ts
-import { readFile as readFile5, writeFile as writeFile5 } from "fs/promises";
+import { readFile as readFile7, writeFile as writeFile5 } from "fs/promises";
 import { join as join7 } from "path";
 async function migrateSvelteConfig(cwd) {
   const path = join7(cwd, "svelte.config.js");
   let src;
   try {
-    src = await readFile5(path, "utf-8");
+    src = await readFile7(path, "utf-8");
   } catch {
     return false;
   }
@@ -828,7 +946,7 @@ async function upgradeTailwind(cwd, spawn2 = defaultSpawn) {
 }
 
 // src/recipes/svelte-5/step-gotchas.ts
-import { readFile as readFile6, writeFile as writeFile6 } from "fs/promises";
+import { readFile as readFile8, writeFile as writeFile6 } from "fs/promises";
 import { join as join9 } from "path";
 import { glob as glob2 } from "tinyglobby";
 
@@ -903,7 +1021,7 @@ async function applyGotchaCodemods(cwd) {
   const relPaths = await glob2(SVELTE_GLOBS, { cwd, ignore: IGNORE2, absolute: false });
   for (const rel of relPaths) {
     const path = join9(cwd, rel);
-    const before = await readFile6(path, "utf-8");
+    const before = await readFile8(path, "utf-8");
     const after = CODEMODS.reduce((s, fn) => fn(s), before);
     if (after !== before) {
       await writeFile6(path, after, "utf-8");
@@ -1050,7 +1168,7 @@ function localPath(path, opts = {}) {
 }
 
 // src/inventory/json.ts
-import { readFile as readFile7 } from "fs/promises";
+import { readFile as readFile9 } from "fs/promises";
 function validate(raw) {
   if (!Array.isArray(raw)) {
     throw new Error("inventory JSON must be an array of sites");
@@ -1074,7 +1192,7 @@ function validate(raw) {
 }
 function fromJsonFile(path) {
   return async () => {
-    const raw = JSON.parse(await readFile7(path, "utf-8"));
+    const raw = JSON.parse(await readFile9(path, "utf-8"));
     return validate(raw);
   };
 }