@reddoorla/maintenance 0.95.1 → 0.97.0

package/dist/{chunk-WSJ7RO54.js → chunk-LAG6IP7Y.js}

@@ -1,10 +1,16 @@
+import {
+  RESTORED_NOTE,
+  refusedByGit,
+  undoRefusedWrites
+} from "./chunk-TJLKWQOX.js";
 import {
   PRETTIER_FLAG_NOTE,
-  formatWithPrettier
-} from "./chunk-R53VL3RI.js";
+  formatWithPrettier,
+  resolveTargetPrettier
+} from "./chunk-5CJVUPJU.js";
 import {
   withRecipe
-} from "./chunk-ZQV7UQNE.js";
+} from "./chunk-LUF52EIY.js";
 import {
   defaultSpawn
 } from "./chunk-LBYOLBW7.js";
@@ -91,6 +97,7 @@ export const GET: RequestHandler = async () => {
 `;
 
 // src/recipes/health-endpoint/index.ts
+var PRETTIER_TIMEOUT_MS = 6e4;
 var PRISMICIO_CANDIDATES = [
   "src/lib/prismicio.ts",
   "src/lib/prismicio.js",
@@ -130,10 +137,21 @@ async function healthEndpoint(site, deps = { spawn: defaultSpawn }) {
       if (!planned.hasPrismic) {
         notes.push("no $lib/prismicio \u2014 wrote Prismic-free /health (prismic: skipped)");
       }
-      if (!await formatWithPrettier(deps.spawn, cwd, [HEALTH_ENDPOINT_RELATIVE])) {
+      const bin = await (deps.resolvePrettier ?? resolveTargetPrettier)(cwd);
+      if (bin === null) {
+        notes.push(PRETTIER_FLAG_NOTE);
+      } else if (!await formatWithPrettier(deps.spawn, cwd, [HEALTH_ENDPOINT_RELATIVE], {
+        bin,
+        timeoutMs: PRETTIER_TIMEOUT_MS
+      })) {
         notes.push(PRETTIER_FLAG_NOTE);
       }
       await commit("feat: add /health endpoint (function-health probe)");
+      const refusal = await refusedByGit(cwd, [HEALTH_ENDPOINT_RELATIVE], "the /health endpoint");
+      if (refusal) {
+        await undoRefusedWrites(cwd, /* @__PURE__ */ new Map([[HEALTH_ENDPOINT_RELATIVE, null]]), refusal.missing);
+        return { kind: "failed", notes: refusal.notes + RESTORED_NOTE };
+      }
       return notes.length > 0 ? { kind: "ok", notes: notes.join("; ") } : { kind: "ok" };
     }
   });
@@ -142,4 +160,4 @@ async function healthEndpoint(site, deps = { spawn: defaultSpawn }) {
 export {
   healthEndpoint
 };
-//# sourceMappingURL=chunk-WSJ7RO54.js.map
+//# sourceMappingURL=chunk-LAG6IP7Y.js.map

package/dist/chunk-LAG6IP7Y.js.map

@@ -0,0 +1 @@
+{"version":3,"sources":["../src/recipes/health-endpoint/index.ts","../src/recipes/health-endpoint/template.ts"],"sourcesContent":["import { access, mkdir, writeFile } from \"node:fs/promises\";\nimport { dirname, join } from \"node:path\";\nimport type { RecipeResult, Site } from \"../../types.js\";\nimport { withRecipe } from \"../_with-recipe.js\";\nimport { defaultSpawn, type SpawnFn } from \"../../audits/util/spawn.js\";\nimport { formatWithPrettier, resolveTargetPrettier, PRETTIER_FLAG_NOTE } from \"../_prettier.js\";\nimport { refusedByGit, undoRefusedWrites, RESTORED_NOTE } from \"../_head-guard.js\";\nimport {\n  HEALTH_ENDPOINT_RELATIVE,\n  HEALTH_ENDPOINT_TEMPLATE,\n  HEALTH_ENDPOINT_TEMPLATE_NO_PRISMIC,\n} from \"./template.js\";\n\nexport type HealthEndpointDeps = {\n  spawn: SpawnFn;\n  /** Resolve the TARGET repo's own prettier. Injected so a test can assert the\n   *  absolute-path spawn without a populated `node_modules`. */\n  resolvePrettier?: (repoRoot: string) => Promise<string | null>;\n};\n\n/** Same budget as prismic-ci and match-harness. Without one the default spawn\n *  never detaches and never kills, so a hung formatter runs unbounded. */\nconst PRETTIER_TIMEOUT_MS = 60_000;\n\n/** Where a Prismic client module lives across the fleet — `.ts` on sources,\n * `.js`/`.mjs` on some compiled clones. Existence of ANY picks the Prismic\n * variant; none => the import-free variant (a static import of a missing module\n * breaks the Vite build, which feature-detection can't rescue). */\nconst PRISMICIO_CANDIDATES = [\n  \"src/lib/prismicio.ts\",\n  \"src/lib/prismicio.js\",\n  \"src/lib/prismicio.mjs\",\n];\n\nasync function fileExists(path: string): Promise<boolean> {\n  try {\n    await access(path);\n    return true;\n  } catch {\n    return false;\n  }\n}\n\nasync function siteHasPrismic(sitePath: string): Promise<boolean> {\n  for (const rel of PRISMICIO_CANDIDATES) {\n    if (await fileExists(join(sitePath, rel))) return true;\n  }\n  return false;\n}\n\n/**\n * Writes a resilient `src/routes/health/+server.ts` if the route doesn't already\n * exist. The function-health audit fetches this endpoint; existing sites need it\n * or the Report Health Gate blocks their Maintenance reports on \"unknown\". Operator\n * edits to an existing endpoint are never clobbered (noop on existing file).\n *\n * Two heterogeneity fixes the fleet rollout exposed: (1) it detects whether the\n * site has a `$lib/prismicio` module and writes the Prismic-free variant when it\n * doesn't (a static import of a missing module fails the build); (2) it runs the\n * SITE's own prettier on the written file before committing, so the file matches\n * that site's formatting config (tabs/spaces, quote style vary fleet-wide) and\n * doesn't red the CI format check. Prettier is best-effort — a site without it\n * installed commits unformatted with a flag note rather than failing the recipe.\n */\nexport async function healthEndpoint(\n  site: Site,\n  deps: HealthEndpointDeps = { spawn: defaultSpawn },\n): Promise<RecipeResult> {\n  const target = join(site.path, HEALTH_ENDPOINT_RELATIVE);\n  return withRecipe<{ target: string; template: string; hasPrismic: boolean }>({\n    name: \"health-endpoint\",\n    site,\n    plan: async () => {\n      if (await fileExists(target)) {\n        return { kind: \"noop\", notes: `${HEALTH_ENDPOINT_RELATIVE} already exists` };\n      }\n      const hasPrismic = await siteHasPrismic(site.path);\n      const template = hasPrismic ? HEALTH_ENDPOINT_TEMPLATE : HEALTH_ENDPOINT_TEMPLATE_NO_PRISMIC;\n      return { kind: \"apply\", plan: { target, template, hasPrismic } };\n    },\n    apply: async (planned, { commit, cwd }) => {\n      await mkdir(dirname(planned.target), { recursive: true });\n      await writeFile(planned.target, planned.template, \"utf-8\");\n\n      const notes: string[] = [];\n      if (!planned.hasPrismic) {\n        notes.push(\"no $lib/prismicio — wrote Prismic-free /health (prismic: skipped)\");\n      }\n\n      // Format to the SITE's own prettier config so CI's format check stays green\n      // across the heterogeneous fleet. Best-effort: a site without prettier just\n      // commits unformatted with a flag note (never fails the /health rollout).\n      // — but run the SITE's own prettier, resolved POSITIVELY and invoked by\n      // absolute path. `pnpm exec prettier` here would first run a full, unbounded\n      // `pnpm install` in the client's checkout (the NORMAL path on `--fleet`,\n      // because `prepareFleetSites` clones and never installs) and then, in a\n      // repo whose install left no prettier of its own, fall through to the\n      // CALLING repo's binary and exit 0. A clone with no prettier gets the flag\n      // note and no spawn at all (#737).\n      const bin = await (deps.resolvePrettier ?? resolveTargetPrettier)(cwd);\n      if (bin === null) {\n        notes.push(PRETTIER_FLAG_NOTE);\n      } else if (\n        !(await formatWithPrettier(deps.spawn, cwd, [HEALTH_ENDPOINT_RELATIVE], {\n          bin,\n          timeoutMs: PRETTIER_TIMEOUT_MS,\n        }))\n      ) {\n        notes.push(PRETTIER_FLAG_NOTE);\n      }\n\n      await commit(\"feat: add /health endpoint (function-health probe)\");\n\n      // Did git TAKE it? `git add -A` honours the site's .gitignore and exits 0\n      // either way, so without this the recipe reports the site done while the\n      // endpoint the function-health audit fetches is in no commit — and the\n      // Report Health Gate goes on blocking that site's report on \"unknown\"\n      // (#741).\n      const refusal = await refusedByGit(cwd, [HEALTH_ENDPOINT_RELATIVE], \"the /health endpoint\");\n      if (refusal) {\n        await undoRefusedWrites(cwd, new Map([[HEALTH_ENDPOINT_RELATIVE, null]]), refusal.missing);\n        return { kind: \"failed\", notes: refusal.notes + RESTORED_NOTE };\n      }\n\n      return notes.length > 0 ? { kind: \"ok\", notes: notes.join(\"; \") } : { kind: \"ok\" };\n    },\n  });\n}\n","/** Relative path inside a site where the /health endpoint lives. Deploys as a\n * Netlify function (adapter-netlify v6); the function-health audit fetches it. */\nexport const HEALTH_ENDPOINT_RELATIVE = \"src/routes/health/+server.ts\";\n\n/** Resilient /health for existing sites. Unlike the starter's, it does NOT\n * statically import { createClient, isPlaceholderRepo } from \"$lib/prismicio\" —\n * older clones lack `isPlaceholderRepo`, and a missing named import breaks the\n * Vite build. Instead it namespace-imports and feature-detects: `createClient`\n * is universal to Prismic SvelteKit sites; a missing one => prismic:\"skipped\"\n * (the gate treats CMS as \"never ran\" and keeps blocking — never a false green).\n * Any probe error => \"error\" (reds CMS, fail-safe). The gate keys off ok + prismic. */\nexport const HEALTH_ENDPOINT_TEMPLATE = `import { json } from \"@sveltejs/kit\";\nimport { env as privateEnv } from \"$env/dynamic/private\";\nimport { env as publicEnv } from \"$env/dynamic/public\";\nimport * as prismicio from \"$lib/prismicio\";\nimport type { RequestHandler } from \"./$types\";\n\n// A live probe — must never be prerendered.\nexport const prerender = false;\n\ntype PrismicHealth = \"ok\" | \"error\" | \"skipped\";\n\ntype PrismicClient = { getRepository: () => Promise<unknown> };\ntype PrismicModule = {\n  createClient?: (opts: { fetch: typeof globalThis.fetch }) => PrismicClient;\n  isPlaceholderRepo?: boolean;\n};\n\n// Server-side Prismic reachability probe. Hits the PUBLIC repository-metadata\n// endpoint (getRepository — no token), time-boxed, returning ONLY a status\n// string; the repository body is never included (/health is public).\nasync function probePrismic(fetch: typeof globalThis.fetch): Promise<PrismicHealth> {\n  const mod = prismicio as PrismicModule;\n  const isPlaceholder = mod.isPlaceholderRepo ?? false;\n  if (isPlaceholder || typeof mod.createClient !== \"function\") return \"skipped\";\n  let timer: ReturnType<typeof setTimeout> | undefined;\n  try {\n    const client = mod.createClient({ fetch });\n    const timeout = new Promise<never>((_, reject) => {\n      timer = setTimeout(() => reject(new Error(\"prismic health probe timed out\")), 5000);\n    });\n    await Promise.race([client.getRepository(), timeout]);\n    return \"ok\";\n  } catch {\n    return \"error\";\n  } finally {\n    if (timer) clearTimeout(timer);\n  }\n}\n\nexport const GET: RequestHandler = async ({ fetch }) => {\n  const prismic = await probePrismic(fetch);\n  const forms = {\n    ingestUrl: !!privateEnv.FORMS_INGEST_URL,\n    ingestToken: !!privateEnv.FORMS_INGEST_TOKEN,\n    turnstile: !!publicEnv.PUBLIC_TURNSTILE_SITE_KEY?.trim(),\n  };\n  // We're inside the handler, so the function ran; ok is false only when the\n  // Prismic probe actively errored.\n  const ok = prismic !== \"error\";\n  return json({ ok, prismic, forms });\n};\n`;\n\n/** Prismic-free /health for sites WITHOUT a `$lib/prismicio` module (non-Prismic\n * sites, e.g. a static/bespoke build). Feature-detection can survive a missing\n * *export*, but a static `import * as prismicio from \"$lib/prismicio\"` fails the\n * Vite build when the *module* doesn't exist — so those sites need a variant with\n * no import at all. CMS is reported as a constant \"skipped\" (the gate treats it as\n * \"never ran\" and never false-greens it). Same JSON shape as the Prismic template\n * (ok/prismic/forms) so the function-health audit parses both identically. */\nexport const HEALTH_ENDPOINT_TEMPLATE_NO_PRISMIC = `import { json } from \"@sveltejs/kit\";\nimport { env as privateEnv } from \"$env/dynamic/private\";\nimport { env as publicEnv } from \"$env/dynamic/public\";\nimport type { RequestHandler } from \"./$types\";\n\n// A live probe — must never be prerendered.\nexport const prerender = false;\n\nexport const GET: RequestHandler = async () => {\n  // This site has no $lib/prismicio module, so there is no CMS to probe: report\n  // \"skipped\" (the gate treats CMS as \"never ran\", never a false green).\n  const prismic = \"skipped\" as const;\n  const forms = {\n    ingestUrl: !!privateEnv.FORMS_INGEST_URL,\n    ingestToken: !!privateEnv.FORMS_INGEST_TOKEN,\n    turnstile: !!publicEnv.PUBLIC_TURNSTILE_SITE_KEY?.trim(),\n  };\n  // We're inside the handler, so the function ran.\n  const ok = true;\n  return json({ ok, prismic, forms });\n};\n`;\n"],"mappings":";;;;;;;;;;;;;;;;;;AAAA,SAAS,QAAQ,OAAO,iBAAiB;AACzC,SAAS,SAAS,YAAY;;;ACCvB,IAAM,2BAA2B;AASjC,IAAM,2BAA2B;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AA4DjC,IAAM,sCAAsC;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;AAAA;;;ADjDnD,IAAM,sBAAsB;AAM5B,IAAM,uBAAuB;AAAA,EAC3B;AAAA,EACA;AAAA,EACA;AACF;AAEA,eAAe,WAAW,MAAgC;AACxD,MAAI;AACF,UAAM,OAAO,IAAI;AACjB,WAAO;AAAA,EACT,QAAQ;AACN,WAAO;AAAA,EACT;AACF;AAEA,eAAe,eAAe,UAAoC;AAChE,aAAW,OAAO,sBAAsB;AACtC,QAAI,MAAM,WAAW,KAAK,UAAU,GAAG,CAAC,EAAG,QAAO;AAAA,EACpD;AACA,SAAO;AACT;AAgBA,eAAsB,eACpB,MACA,OAA2B,EAAE,OAAO,aAAa,GAC1B;AACvB,QAAM,SAAS,KAAK,KAAK,MAAM,wBAAwB;AACvD,SAAO,WAAsE;AAAA,IAC3E,MAAM;AAAA,IACN;AAAA,IACA,MAAM,YAAY;AAChB,UAAI,MAAM,WAAW,MAAM,GAAG;AAC5B,eAAO,EAAE,MAAM,QAAQ,OAAO,GAAG,wBAAwB,kBAAkB;AAAA,MAC7E;AACA,YAAM,aAAa,MAAM,eAAe,KAAK,IAAI;AACjD,YAAM,WAAW,aAAa,2BAA2B;AACzD,aAAO,EAAE,MAAM,SAAS,MAAM,EAAE,QAAQ,UAAU,WAAW,EAAE;AAAA,IACjE;AAAA,IACA,OAAO,OAAO,SAAS,EAAE,QAAQ,IAAI,MAAM;AACzC,YAAM,MAAM,QAAQ,QAAQ,MAAM,GAAG,EAAE,WAAW,KAAK,CAAC;AACxD,YAAM,UAAU,QAAQ,QAAQ,QAAQ,UAAU,OAAO;AAEzD,YAAM,QAAkB,CAAC;AACzB,UAAI,CAAC,QAAQ,YAAY;AACvB,cAAM,KAAK,wEAAmE;AAAA,MAChF;AAYA,YAAM,MAAM,OAAO,KAAK,mBAAmB,uBAAuB,GAAG;AACrE,UAAI,QAAQ,MAAM;AAChB,cAAM,KAAK,kBAAkB;AAAA,MAC/B,WACE,CAAE,MAAM,mBAAmB,KAAK,OAAO,KAAK,CAAC,wBAAwB,GAAG;AAAA,QACtE;AAAA,QACA,WAAW;AAAA,MACb,CAAC,GACD;AACA,cAAM,KAAK,kBAAkB;AAAA,MAC/B;AAEA,YAAM,OAAO,oDAAoD;AAOjE,YAAM,UAAU,MAAM,aAAa,KAAK,CAAC,wBAAwB,GAAG,sBAAsB;AAC1F,UAAI,SAAS;AACX,cAAM,kBAAkB,KAAK,oBAAI,IAAI,CAAC,CAAC,0BAA0B,IAAI,CAAC,CAAC,GAAG,QAAQ,OAAO;AACzF,eAAO,EAAE,MAAM,UAAU,OAAO,QAAQ,QAAQ,cAAc;AAAA,MAChE;AAEA,aAAO,MAAM,SAAS,IAAI,EAAE,MAAM,MAAM,OAAO,MAAM,KAAK,IAAI,EAAE,IAAI,EAAE,MAAM,KAAK;AAAA,IACnF;AAAA,EACF,CAAC;AACH;","names":[]}

package/dist/{chunk-WJMPCVOR.js → chunk-LAJ3FF54.js}

@@ -419,6 +419,8 @@ import { parse, NodeType } from "node-html-parser";
 var MAX_ANCHORS = 300;
 var MAX_SCRIPTS = 120;
 var MAX_INLINE_URLS = 40;
+var MAX_DATA_TEXT = 2e5;
+var DATA_SCRIPT_TYPE = /^(?:application|text)\/(?:[\w.+-]+\+)?json$/;
 var NAMESPACE_URLS = /^https?:\/\/(www\.)?w3\.org\//i;
 function urlsIn(code) {
   const found = /* @__PURE__ */ new Set();
@@ -528,8 +530,10 @@ function collect(el, out, depth = 0) {
         if (out.title === null) out.title = collapse(e.text) || null;
         break;
       case "SCRIPT": {
-        const isJsonLd = (e.getAttribute("type") ?? "").toLowerCase().trim() === "application/ld+json";
+        const type = (e.getAttribute("type") ?? "").toLowerCase().split(";")[0]?.trim() ?? "";
+        const isJsonLd = type === "application/ld+json";
         if (isJsonLd) out.jsonLd.push(e.text);
+        if (DATA_SCRIPT_TYPE.test(type)) out.dataText.push(e.text);
         const src = (e.getAttribute("src") ?? "").trim();
         if (src) {
           out.scriptSrcs.push(src);
@@ -660,7 +664,8 @@ function extractPage(html) {
     anchors: [],
     forms: [],
     scriptSrcs: [],
-    inlineScriptUrls: []
+    inlineScriptUrls: [],
+    dataText: []
   };
   collect(documentEl, out);
   const social = {};
@@ -737,7 +742,10 @@ function extractPage(html) {
     inlineScriptUrls: [...new Set(out.inlineScriptUrls)].slice(0, MAX_INLINE_URLS),
     // The TRUE total, for the same reason `anchorCount` exists: a capped list
     // must never be mistaken for a complete one.
-    scriptCount: out.scriptSrcs.length
+    scriptCount: out.scriptSrcs.length,
+    // Capped for storage, never sliced mid-measurement: see `MAX_DATA_TEXT`
+    // for why truncating here can only err toward a penalty.
+    dataText: collapse(out.dataText.join(" ")).slice(0, MAX_DATA_TEXT)
   };
 }
 
@@ -4219,4 +4227,4 @@ export {
   tally,
   runSiteChecks
 };
-//# sourceMappingURL=chunk-WJMPCVOR.js.map
+//# sourceMappingURL=chunk-LAJ3FF54.js.map