@reddoorla/maintenance 0.10.5 → 0.10.6

package/dist/cli/bin.js

@@ -1104,7 +1104,7 @@ async function securityAudit(ctx) {
 }
 
 // src/audits/lighthouse.ts
-import { readFile as readFile4, writeFile, mkdtemp, rm } from "fs/promises";
+import { readFile as readFile4, writeFile, mkdtemp, rm, readdir } from "fs/promises";
 import { tmpdir } from "os";
 import { join as join4 } from "path";
 
@@ -1201,6 +1201,21 @@ async function readJsonMaybe(path) {
     return null;
   }
 }
+async function readLhrEntries(resultsDir) {
+  const files = await readdir(resultsDir).catch(() => []);
+  const entries = [];
+  for (const f of files) {
+    if (!f.startsWith("lhr-") || !f.endsWith(".json")) continue;
+    const lhr = await readJsonMaybe(join4(resultsDir, f));
+    if (!lhr || !lhr.categories) continue;
+    const summary = {};
+    for (const [k, v] of Object.entries(lhr.categories)) {
+      if (typeof v?.score === "number") summary[k] = v.score;
+    }
+    entries.push({ url: lhr.requestedUrl, summary });
+  }
+  return entries;
+}
 function averageSummaries(entries) {
   if (entries.length === 0) return {};
   const sums = {};
@@ -1274,13 +1289,13 @@ async function lighthouseAudit(ctx) {
     throw err;
   }
   await rm(configDir, { recursive: true, force: true });
-  const manifest = await readJsonMaybe(join4(resultsDir, "manifest.json"));
-  if (!manifest || manifest.length === 0) {
+  const manifest = await readLhrEntries(resultsDir);
+  if (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)}` : ""}`
+      summary: `lighthouse: no lhr-*.json written (exit ${raw.code})${raw.stderr ? ` \u2014 ${raw.stderr.slice(0, 200)}` : ""}`
     };
   }
   const assertionResults = await readJsonMaybe(join4(resultsDir, "assertion-results.json")) ?? [];
@@ -1355,7 +1370,7 @@ async function readJsonMaybe2(path) {
     return null;
   }
 }
-function buildPlaywrightConfig(port) {
+function buildPlaywrightConfig(port, sitePath) {
   return `import { defineConfig } from "@playwright/test";
 
 export default defineConfig({
@@ -1373,8 +1388,13 @@ export default defineConfig({
     // --strictPort: refuse to bump to a different port if ours is taken,
     //   so the audit fails loudly instead of probing a zombie.
     // reuseExistingServer:false: never reuse \u2014 we control the lifecycle.
+    // cwd: playwright's default webServer.cwd is the config file's
+    //   directory. Our config lives in /tmp so without this override,
+    //   "npm run vite:dev" tries to read /tmp/.../package.json and
+    //   ENOENTs before vite ever starts. Caltex 2026-05-28 (0.10.5).
     command: "npm run vite:dev -- --port ${port} --strictPort",
     url: "http://localhost:${port}/dev/a11y-fixtures",
+    cwd: ${JSON.stringify(sitePath)},
     reuseExistingServer: false,
     timeout: 120_000,
   },
@@ -1436,7 +1456,7 @@ async function a11yAudit(ctx) {
   await writeFile2(specPath, buildSpec(), "utf-8");
   const port = await findFreePort();
   const configPath = join5(specDir, "playwright.config.ts");
-  await writeFile2(configPath, buildPlaywrightConfig(port), "utf-8");
+  await writeFile2(configPath, buildPlaywrightConfig(port, site.path), "utf-8");
   const resultsPath = join5(site.path, RESULTS_REL);
   await rm2(join5(site.path, ".reddoor-a11y"), { recursive: true, force: true });
   let raw;
@@ -1615,7 +1635,7 @@ async function resolveSites(input) {
 }
 
 // src/cli/fleet/clone-if-needed.ts
-import { stat, readdir, mkdir } from "fs/promises";
+import { stat, readdir as readdir2, mkdir } from "fs/promises";
 import { isAbsolute as isAbsolute2, join as join6 } from "path";
 function deriveNameFromRepoUrl(repoUrl) {
   const slash = repoUrl.split("/").pop() ?? repoUrl;
@@ -1643,7 +1663,7 @@ async function isNonEmptyDir(path) {
   try {
     const s = await stat(path);
     if (!s.isDirectory()) return false;
-    const entries = await readdir(path);
+    const entries = await readdir2(path);
     return entries.length > 0;
   } catch {
     return false;