@reddoorla/maintenance 0.10.4 → 0.10.6

package/dist/index.js

@@ -335,7 +335,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";
 
@@ -397,6 +397,32 @@ async function readSiteConfig(sitePath) {
   return out;
 }
 
+// src/util/free-port.ts
+import { createServer } from "net";
+async function findFreePort() {
+  return new Promise((resolve, reject) => {
+    const server = createServer();
+    server.unref();
+    server.on("error", reject);
+    server.listen(0, "127.0.0.1", () => {
+      const addr = server.address();
+      if (typeof addr === "object" && addr) {
+        const port = addr.port;
+        server.close(() => resolve(port));
+      } else {
+        server.close();
+        reject(new Error("findFreePort: could not determine assigned port from socket"));
+      }
+    });
+  });
+}
+function withFreePort(url, port) {
+  const u = new URL(url);
+  u.hostname = "localhost";
+  u.port = String(port);
+  return u.toString();
+}
+
 // src/audits/lighthouse.ts
 async function readJsonMaybe(path) {
   try {
@@ -406,6 +432,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 = {};
@@ -437,13 +478,19 @@ async function lighthouseAudit(ctx) {
   const site = ctx.site;
   const label = siteLabel(site);
   const siteCfg = await readSiteConfig(site.path);
-  const resolvedConfig = siteCfg.lighthouseUrl ? {
+  const port = await findFreePort();
+  const baseUrl = siteCfg.lighthouseUrl ?? lighthouseConfig.ci.collect.url[0];
+  const resolvedConfig = {
     ...lighthouseConfig,
     ci: {
       ...lighthouseConfig.ci,
-      collect: { ...lighthouseConfig.ci.collect, url: [siteCfg.lighthouseUrl] }
+      collect: {
+        ...lighthouseConfig.ci.collect,
+        url: [withFreePort(baseUrl, port)],
+        startServerCommand: `npm run vite:dev -- --port ${port} --strictPort`
+      }
     }
-  } : lighthouseConfig;
+  };
   const configDir = await mkdtemp(join4(tmpdir(), "reddoor-lhci-"));
   const configPath = join4(configDir, "lighthouserc.json");
   await writeFile(configPath, JSON.stringify(resolvedConfig), "utf-8");
@@ -473,13 +520,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")) ?? [];
@@ -554,6 +601,37 @@ async function readJsonMaybe2(path) {
     return null;
   }
 }
+function buildPlaywrightConfig(port, sitePath) {
+  return `import { defineConfig } from "@playwright/test";
+
+export default defineConfig({
+  testDir: ".",
+  testMatch: /.*\\.spec\\.ts$/,
+  fullyParallel: true,
+  forbidOnly: !!process.env.CI,
+  retries: process.env.CI ? 2 : 0,
+  reporter: process.env.CI ? "github" : "list",
+  use: {
+    baseURL: "http://localhost:${port}",
+    trace: "on-first-retry",
+  },
+  webServer: {
+    // --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,
+  },
+});
+`;
+}
 function buildSpec() {
   return `import { test, expect } from "@playwright/test";
 import AxeBuilder from "@axe-core/playwright";
@@ -607,19 +685,26 @@ async function a11yAudit(ctx) {
   const specDir = await mkdtemp2(join5(tmpdir2(), "reddoor-a11y-spec-"));
   const specPath = join5(specDir, "a11y.spec.ts");
   await writeFile2(specPath, buildSpec(), "utf-8");
+  const port = await findFreePort();
+  const configPath = join5(specDir, "playwright.config.ts");
+  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;
   try {
-    raw = await spawn2("npx", ["--yes", "playwright", "test", "--reporter=line", specPath], {
-      cwd: site.path,
-      env: { ...process.env, REDDOOR_A11Y_OUTPUT: resultsPath },
-      // playwright on a cold tree downloads Chrome, boots the site's dev
-      // server, and runs axe over every configured route. The shared 30 s
-      // default in runAudits is fine for deps/lint/security but starves
-      // playwright (mirrors the lighthouse fix shipped earlier).
-      timeoutMs: 5 * 6e4
-    });
+    raw = await spawn2(
+      "npx",
+      ["--yes", "playwright", "test", `--config=${configPath}`, "--reporter=line", specPath],
+      {
+        cwd: site.path,
+        env: { ...process.env, REDDOOR_A11Y_OUTPUT: resultsPath },
+        // playwright on a cold tree downloads Chrome, boots the site's dev
+        // server, and runs axe over every configured route. The shared 30 s
+        // default in runAudits is fine for deps/lint/security but starves
+        // playwright (mirrors the lighthouse fix shipped earlier).
+        timeoutMs: 5 * 6e4
+      }
+    );
   } catch (err) {
     await rm2(specDir, { recursive: true, force: true });
     const e = err;