@reddoorla/maintenance 0.96.0 → 0.98.0

package/dist/{chunk-XY7YV2YQ.js → chunk-DXZRWW2F.js}

@@ -3,14 +3,15 @@ import {
 } from "./chunk-LKGVSM2O.js";
 import {
   readGitHubConfig
-} from "./chunk-CGQZFQAY.js";
+} from "./chunk-MVKNU4QD.js";
 import {
   assertUrlSegment
 } from "./chunk-43SW3FGW.js";
 import {
   lighthouseAudit,
-  readSiteConfig
-} from "./chunk-BIYQN7FH.js";
+  readSiteConfig,
+  readsPlaceholderPrismicRepo
+} from "./chunk-VCPTOZ56.js";
 import {
   smokeAudit
 } from "./chunk-UDRHAQT7.js";
@@ -653,6 +654,12 @@ async function securityAudit(ctx) {
 // src/audits/a11y.ts
 import { readFile as readFile3, writeFile, mkdtemp, rm } from "fs/promises";
 import { join as join4 } from "path";
+var PLACEHOLDER_SKIP_REASON = "placeholder Prismic repo";
+function classifyRouteResponse(input) {
+  if (input.status === 200) return "scan";
+  if (input.placeholder404Ok && input.status === 404) return "skip";
+  return "missing";
+}
 var RESULTS_REL = ".reddoor-a11y/results.json";
 async function readJsonMaybe(path) {
   try {
@@ -722,6 +729,10 @@ import AxeBuilder from "@axe-core/playwright";
 import { mkdir, writeFile } from "node:fs/promises";
 import { dirname } from "node:path";
 
+// Injected, not transcribed \u2014 see classifyRouteResponse in src/audits/a11y.ts.
+const classifyRouteResponse = ${classifyRouteResponse.toString()};
+const SKIP_REASON = ${JSON.stringify(PLACEHOLDER_SKIP_REASON)};
+
 const pages = ${JSON.stringify(axePages)};
 const smokePages = ${JSON.stringify(smokeRoutes)};
 // Absolute origin for the hydration smoke when the site has opted its gates
@@ -739,6 +750,10 @@ test.setTimeout(5 * 60_000);
 
 test("a11y + hydration across configured routes", async ({ page }) => {
   const violations = [];
+  // Routes navigated but deliberately not scanned. Separate from violations so
+  // they cannot fail the run, and written to the artifact so they cannot vanish
+  // from the summary either.
+  const skipped = [];
 
   // Capture uncaught client-side exceptions across every route we visit. A page
   // that builds + SSRs cleanly can still throw on hydrate and blank itself
@@ -754,9 +769,10 @@ test("a11y + hydration across configured routes", async ({ page }) => {
     });
   });
 
-  for (const { path, name } of pages) {
+  for (const { path, name, placeholder404Ok } of pages) {
     currentRoute = name;
     const response = await page.goto(path);
+    const status = response ? response.status() : null;
     // A route that does not exist is a config problem, not a markup one. The
     // audit used to navigate, get a 404, run axe over whatever the error page
     // was and report the count -- for months that page was a bare fallback with
@@ -764,12 +780,24 @@ test("a11y + hydration across configured routes", async ({ page }) => {
     // gave its 404 page a designed watermark the count went to 1 with no route
     // and no rule in the summary, and the "violation" was bisected as markup
     // (#680). Name it as a missing route and do not scan the error page.
-    if (!response || response.status() !== 200) {
+    //
+    // The one exception (#863): a site still on the starter's Prismic sentinel
+    // has no content to serve, so a 404 on ITS OWN routes is the designed
+    // answer. Those carry placeholder404Ok; the /dev fixtures never do.
+    const verdict = classifyRouteResponse({
+      status,
+      placeholder404Ok: placeholder404Ok === true,
+    });
+    if (verdict === "skip") {
+      skipped.push({ route: name, path, status, reason: SKIP_REASON });
+      continue;
+    }
+    if (verdict === "missing") {
       violations.push({
         id: "route-missing",
         impact: "serious",
         route: name,
-        help: \`\${path} returned \${response ? response.status() : "no response"}\`,
+        help: \`\${path} returned \${status === null ? "no response" : status}\`,
       });
       continue;
     }
@@ -818,7 +846,11 @@ test("a11y + hydration across configured routes", async ({ page }) => {
     await mkdir(dirname(OUTPUT), { recursive: true });
     await writeFile(
       OUTPUT,
-      JSON.stringify({ totalViolations: violations.length, byImpact, violations }, null, 2),
+      JSON.stringify(
+        { totalViolations: violations.length, byImpact, violations, skipped },
+        null,
+        2,
+      ),
     );
   }
   expect(violations).toEqual([]);
@@ -843,6 +875,19 @@ function describeViolations(violations) {
   const rest = entries.length - shown.length;
   return shown.join(", ") + (rest > 0 ? `, +${rest} more` : "");
 }
+var NAMED_SKIPS_MAX = 4;
+function describeSkipped(skipped) {
+  if (skipped.length === 0) return "";
+  const shown = skipped.slice(0, NAMED_SKIPS_MAX).map((s) => s.route);
+  const rest = skipped.length - shown.length;
+  const names = shown.join(", ") + (rest > 0 ? `, +${rest} more` : "");
+  const reasons = [
+    ...new Set(
+      skipped.map((s) => s.reason).filter((r) => typeof r === "string" && r.length > 0)
+    )
+  ];
+  return `${skipped.length} skipped: ${names}${reasons.length > 0 ? ` \u2014 ${reasons.join(", ")}` : ""}`;
+}
 async function a11yAudit(ctx) {
   const spawn = ctx.spawn ?? defaultSpawn;
   const site = ctx.site;
@@ -850,9 +895,14 @@ async function a11yAudit(ctx) {
   const specDir = await mkdtemp(join4(site.path, ".reddoor-a11y-spec-"));
   try {
     const { a11yRoutes: siteRoutes, gateServer } = await readSiteConfig(site.path);
+    const placeholderRepo = await readsPlaceholderPrismicRepo(site.path);
     const axePages = [
       ...a11yRoutes,
-      ...(siteRoutes ?? []).map((path) => ({ path, name: path }))
+      ...(siteRoutes ?? []).map((path) => ({
+        path,
+        name: path,
+        ...placeholderRepo ? { placeholder404Ok: true } : {}
+      }))
     ];
     const port = await findFreePort();
     const previewPort = gateServer === "preview" ? await allocateDistinctPort(port) : void 0;
@@ -909,7 +959,12 @@ async function a11yAudit(ctx) {
     const hasAny = artifact.totalViolations > 0;
     const status = hasSerious ? "fail" : hasAny ? "warn" : "pass";
     const siteRouteCount = axePages.length - a11yRoutes.length;
-    const scanned = siteRouteCount > 0 ? `${axePages.length} routes (${a11yRoutes.length} fixtures + ${siteRouteCount} from package.json)` : `${axePages.length} routes`;
+    const splitNote = siteRouteCount > 0 ? `${a11yRoutes.length} fixtures + ${siteRouteCount} from package.json` : "";
+    const skippedRoutes = Array.isArray(artifact.skipped) ? artifact.skipped : [];
+    const skipNote = describeSkipped(skippedRoutes);
+    const countPhrase = skippedRoutes.length > 0 ? `${axePages.length - skippedRoutes.length} of ${axePages.length} routes` : `${axePages.length} routes`;
+    const notes = [splitNote, skipNote].filter((n) => n.length > 0).join("; ");
+    const scanned = notes.length > 0 ? `${countPhrase} (${notes})` : countPhrase;
     const smokeNote = previewPort === void 0 ? `+${smokeRoutes.length} hydration smoke` : `+${smokeRoutes.length} hydration smoke on a production preview`;
     const named = describeViolations(artifact.violations ?? []);
     const summary = status === "pass" ? `a11y: 0 violations across ${scanned} (${smokeNote})` : `a11y: ${artifact.totalViolations} violations across ${scanned}${named ? ` \u2014 ${named}` : ""}`;
@@ -2195,4 +2250,4 @@ export {
   runAudits,
   runAuditsAcross
 };
-//# sourceMappingURL=chunk-XY7YV2YQ.js.map
+//# sourceMappingURL=chunk-DXZRWW2F.js.map