@reddoorla/maintenance 0.13.0 → 0.15.0

package/dist/cli/commands/audit.js

@@ -39,7 +39,10 @@ __export(websites_exports, {
   listWebsites: () => listWebsites,
   mapRow: () => mapRow,
   siteSlug: () => siteSlug,
-  updateScores: () => updateScores
+  updateA11yCounts: () => updateA11yCounts,
+  updateDepsCounts: () => updateDepsCounts,
+  updateScores: () => updateScores,
+  updateSecurityCounts: () => updateSecurityCounts
 });
 function siteSlug(name) {
   return name.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-|-$/g, "");
@@ -67,6 +70,13 @@ function mapRow(rec) {
     bpScore: f["bpScore"] ?? null,
     seoScore: f["seoScore"] ?? null,
     lastLighthouseAuditAt: f["Last lighthouse audit at"] ?? null,
+    a11yViolations: f["A11y Violations"] ?? null,
+    depsDrifted: f["Deps Drifted"] ?? null,
+    depsMajorBehind: f["Deps Major Behind"] ?? null,
+    securityVulnsCritical: f["Security Vulns Critical"] ?? null,
+    securityVulnsHigh: f["Security Vulns High"] ?? null,
+    securityVulnsModerate: f["Security Vulns Moderate"] ?? null,
+    securityVulnsLow: f["Security Vulns Low"] ?? null,
     dashboardToken: (() => {
       const raw = f["Dashboard Token"];
       if (typeof raw !== "string") return null;
@@ -97,6 +107,28 @@ async function updateScores(base, recordId, scores) {
   };
   await base(WEBSITES_TABLE).update([{ id: recordId, fields }]);
 }
+async function updateA11yCounts(base, recordId, counts) {
+  const fields = {
+    "A11y Violations": counts.violations
+  };
+  await base(WEBSITES_TABLE).update([{ id: recordId, fields }]);
+}
+async function updateDepsCounts(base, recordId, counts) {
+  const fields = {
+    "Deps Drifted": counts.drifted,
+    "Deps Major Behind": counts.majorBehind
+  };
+  await base(WEBSITES_TABLE).update([{ id: recordId, fields }]);
+}
+async function updateSecurityCounts(base, recordId, counts) {
+  const fields = {
+    "Security Vulns Critical": counts.critical,
+    "Security Vulns High": counts.high,
+    "Security Vulns Moderate": counts.moderate,
+    "Security Vulns Low": counts.low
+  };
+  await base(WEBSITES_TABLE).update([{ id: recordId, fields }]);
+}
 var WEBSITES_TABLE;
 var init_websites = __esm({
   "src/reports/airtable/websites.ts"() {
@@ -192,8 +224,131 @@ var init_lighthouse_airtable = __esm({
   }
 });
 
+// src/audits/a11y-airtable.ts
+function hasA11yCounts(result) {
+  if (result.audit !== "a11y") return false;
+  const details = result.details;
+  return typeof details?.totalViolations === "number";
+}
+function a11yCountsFromResult(result) {
+  if (result.audit !== "a11y") {
+    throw new Error(`Expected an 'a11y' AuditResult, got '${result.audit}'`);
+  }
+  const details = result.details;
+  return { violations: details?.totalViolations ?? 0 };
+}
+var init_a11y_airtable = __esm({
+  "src/audits/a11y-airtable.ts"() {
+    "use strict";
+  }
+});
+
+// src/audits/deps-airtable.ts
+function hasDepsCounts(result) {
+  if (result.audit !== "deps") return false;
+  return Array.isArray(result.details);
+}
+function depsCountsFromResult(result) {
+  if (result.audit !== "deps") {
+    throw new Error(`Expected a 'deps' AuditResult, got '${result.audit}'`);
+  }
+  const entries = result.details ?? [];
+  const drifted = entries.filter((e) => e.drift !== "same").length;
+  const majorBehind = entries.filter((e) => e.drift === "major").length;
+  return { drifted, majorBehind };
+}
+var init_deps_airtable = __esm({
+  "src/audits/deps-airtable.ts"() {
+    "use strict";
+  }
+});
+
+// src/audits/security-airtable.ts
+function hasSecurityCounts(result) {
+  if (result.audit !== "security") return false;
+  const details = result.details;
+  return !!details && typeof details.counts === "object";
+}
+function securityCountsFromResult(result) {
+  if (result.audit !== "security") {
+    throw new Error(`Expected a 'security' AuditResult, got '${result.audit}'`);
+  }
+  const details = result.details;
+  const c = details?.counts ?? { low: 0, moderate: 0, high: 0, critical: 0 };
+  return { critical: c.critical, high: c.high, moderate: c.moderate, low: c.low };
+}
+var init_security_airtable = __esm({
+  "src/audits/security-airtable.ts"() {
+    "use strict";
+  }
+});
+
+// src/audits/write-audits-to-airtable.ts
+var write_audits_to_airtable_exports = {};
+__export(write_audits_to_airtable_exports, {
+  writeAuditsToAirtable: () => writeAuditsToAirtable
+});
+async function writeAuditsToAirtable(args) {
+  const { base, websites, slug, results } = args;
+  const lhResult = results.find((r) => r.audit === "lighthouse");
+  if (!lhResult) {
+    throw Object.assign(
+      new Error(
+        "--write-airtable requires a lighthouse result; did you pass --only without lighthouse?"
+      ),
+      { exitCode: 2 }
+    );
+  }
+  if (!hasRealScores(lhResult)) {
+    throw Object.assign(
+      new Error(
+        `Lighthouse audit produced no scores; refusing to write to Airtable. Summary: ${lhResult.summary}`
+      ),
+      { exitCode: 1 }
+    );
+  }
+  const target = websites.find((w) => siteSlug(w.name) === slug);
+  if (!target) {
+    throw Object.assign(new Error(`No Websites row matched slug "${slug}"`), { exitCode: 2 });
+  }
+  const writes = [];
+  const scores = lighthouseScoresFromResult(lhResult);
+  await updateScores(base, target.id, scores);
+  writes.push({ audit: "lighthouse", counts: scores });
+  const a11y = results.find((r) => r.audit === "a11y");
+  if (a11y && hasA11yCounts(a11y)) {
+    const counts = a11yCountsFromResult(a11y);
+    await updateA11yCounts(base, target.id, counts);
+    writes.push({ audit: "a11y", counts });
+  }
+  const deps = results.find((r) => r.audit === "deps");
+  if (deps && hasDepsCounts(deps)) {
+    const counts = depsCountsFromResult(deps);
+    await updateDepsCounts(base, target.id, counts);
+    writes.push({ audit: "deps", counts });
+  }
+  const sec = results.find((r) => r.audit === "security");
+  if (sec && hasSecurityCounts(sec)) {
+    const counts = securityCountsFromResult(sec);
+    await updateSecurityCounts(base, target.id, counts);
+    writes.push({ audit: "security", counts });
+  }
+  return { siteName: target.name, writes };
+}
+var init_write_audits_to_airtable = __esm({
+  "src/audits/write-audits-to-airtable.ts"() {
+    "use strict";
+    init_websites();
+    init_lighthouse_airtable();
+    init_a11y_airtable();
+    init_deps_airtable();
+    init_security_airtable();
+  }
+});
+
 // src/cli/commands/audit.ts
 import { resolve as resolve2 } from "path";
+import { Listr } from "listr2";
 
 // src/audits/util/spawn.ts
 import { spawn } from "child_process";
@@ -950,29 +1105,20 @@ var DEFAULT_AUDIT_TIMEOUT_MS = 3e4;
 function timedSpawn(timeoutMs) {
   return (cmd, args, opts = {}) => defaultSpawn(cmd, args, { ...opts, timeoutMs: opts.timeoutMs ?? timeoutMs });
 }
-async function runAudits(site, which) {
-  const names = which ?? ALL_AUDIT_NAMES;
-  for (const n of names) {
-    if (!(n in REGISTRY)) throw new Error(`unknown audit: ${n}`);
-  }
+async function runOneAudit(site, name) {
+  if (!(name in REGISTRY)) throw new Error(`unknown audit: ${name}`);
   const spawn2 = timedSpawn(DEFAULT_AUDIT_TIMEOUT_MS);
   const label = site.name ?? site.path;
-  return Promise.all(
-    names.map(
-      (n) => REGISTRY[n]({ site, spawn: spawn2 }).catch(
-        (err) => ({
-          audit: n,
-          site: label,
-          status: "fail",
-          summary: `${n}: unexpected error \u2014 ${String(err)}`
-        })
-      )
-    )
-  );
-}
-async function runAuditsAcross(sites, which) {
-  const all = await Promise.all(sites.map((s) => runAudits(s, which)));
-  return all.flat();
+  try {
+    return await REGISTRY[name]({ site, spawn: spawn2 });
+  } catch (err) {
+    return {
+      audit: name,
+      site: label,
+      status: "fail",
+      summary: `${name}: unexpected error \u2014 ${String(err)}`
+    };
+  }
 }
 
 // src/cli/fleet/resolve-sites.ts
@@ -1138,8 +1284,87 @@ function formatTable(results) {
 function exitCode(results) {
   return results.some((r) => r.status === "fail") ? 1 : 0;
 }
+function formatDuration(ms) {
+  if (ms < 1e3) return `${ms}ms`;
+  const totalSeconds = Math.round(ms / 1e3);
+  if (totalSeconds < 60) return `${totalSeconds}s`;
+  const m = Math.floor(totalSeconds / 60);
+  const s = totalSeconds % 60;
+  return `${m}m${s.toString().padStart(2, "0")}s`;
+}
+function buildAuditTasks(sites, which, results, renderer) {
+  const singleSite = sites.length === 1;
+  if (singleSite) {
+    const site = sites[0];
+    return new Listr(
+      which.map((name) => ({
+        title: name,
+        task: async (_ctx, task) => {
+          const start = Date.now();
+          const result = await runOneAudit(site, name);
+          results.push(result);
+          const elapsed = formatDuration(Date.now() - start);
+          task.title = `${name}: ${result.summary} (${elapsed})`;
+          if (result.status === "fail") throw new Error(result.summary);
+        }
+      })),
+      { concurrent: true, exitOnError: false, renderer }
+    );
+  }
+  return new Listr(
+    sites.map((site) => {
+      const label = site.name ?? site.path;
+      return {
+        title: label,
+        task: async (_ctx, task) => {
+          const start = Date.now();
+          let done = 0;
+          task.output = `0/${which.length} audits`;
+          const settled = await Promise.all(
+            which.map(async (name) => {
+              const r = await runOneAudit(site, name);
+              results.push(r);
+              done += 1;
+              task.output = `${done}/${which.length} audits`;
+              return r;
+            })
+          );
+          const elapsed = formatDuration(Date.now() - start);
+          const failed = settled.filter((r) => r.status === "fail").length;
+          const warned = settled.filter((r) => r.status === "warn").length;
+          const note = failed > 0 ? `${failed} failed` : warned > 0 ? `${warned} warning${warned === 1 ? "" : "s"}` : "all green";
+          task.title = `${label}: ${note} (${elapsed})`;
+          if (failed > 0) throw new Error(`${label}: ${failed} audit(s) failed`);
+        }
+      };
+    }),
+    { concurrent: true, exitOnError: false, renderer }
+  );
+}
+function formatWriteSummary(summary) {
+  const lines = summary.writes.map((w) => {
+    if (w.audit === "lighthouse") {
+      const s = w.counts;
+      return `  lighthouse: P=${s.performance} A=${s.accessibility} BP=${s.bestPractices} SEO=${s.seo}`;
+    }
+    if (w.audit === "a11y") {
+      return `  a11y: ${w.counts.violations} violations`;
+    }
+    if (w.audit === "deps") {
+      const c2 = w.counts;
+      return `  deps: ${c2.drifted} drifted (${c2.majorBehind} major)`;
+    }
+    const c = w.counts;
+    return `  security: ${c.critical}C/${c.high}H/${c.moderate}M/${c.low}L`;
+  });
+  return `\u2192 wrote to Websites[${summary.siteName}]:
+${lines.join("\n")}`;
+}
+function rendererFor(json) {
+  return json ? "silent" : "default";
+}
 async function runAuditCommand(site, opts) {
-  const which = parseOnly(opts.only);
+  const which = parseOnly(opts.only) ?? ALL_AUDIT_NAMES;
   const cwd = opts.cwd ? resolve2(opts.cwd) : process.cwd();
   let sites = await resolveSites({
     ...site !== void 0 ? { site } : {},
@@ -1151,41 +1376,36 @@ async function runAuditCommand(site, opts) {
     const workdir = opts.workdir ?? `${process.env.HOME ?? ""}/.reddoor-maint/sites`;
     sites = await Promise.all(sites.map((s) => cloneIfNeeded(s, { workdir })));
   }
-  const results = await runAuditsAcross(sites, which);
+  const results = [];
+  const renderer = rendererFor(opts.json);
+  await buildAuditTasks(sites, which, results, renderer).run();
   let output = opts.json ? JSON.stringify(results, null, 2) : formatTable(results);
   if (opts.writeAirtable !== void 0) {
     const { openBase: openBase2, readAirtableConfig: readAirtableConfig2 } = await Promise.resolve().then(() => (init_client(), client_exports));
-    const { listWebsites: listWebsites2, updateScores: updateScores2, siteSlug: siteSlug2 } = await Promise.resolve().then(() => (init_websites(), websites_exports));
-    const { lighthouseScoresFromResult: lighthouseScoresFromResult2, hasRealScores: hasRealScores2, resolveSlugFromCwd: resolveSlugFromCwd2 } = await Promise.resolve().then(() => (init_lighthouse_airtable(), lighthouse_airtable_exports));
+    const { listWebsites: listWebsites2 } = await Promise.resolve().then(() => (init_websites(), websites_exports));
+    const { resolveSlugFromCwd: resolveSlugFromCwd2 } = await Promise.resolve().then(() => (init_lighthouse_airtable(), lighthouse_airtable_exports));
+    const { writeAuditsToAirtable: writeAuditsToAirtable2 } = await Promise.resolve().then(() => (init_write_audits_to_airtable(), write_audits_to_airtable_exports));
     const slug = typeof opts.writeAirtable === "string" && opts.writeAirtable.length > 0 ? opts.writeAirtable : await resolveSlugFromCwd2(cwd);
-    const lhResult = results.find((r) => r.audit === "lighthouse");
-    if (!lhResult) {
-      throw Object.assign(
-        new Error(
-          "--write-airtable requires a lighthouse result; did you pass --only without lighthouse?"
-        ),
-        { exitCode: 2 }
-      );
-    }
-    if (!hasRealScores2(lhResult)) {
-      throw Object.assign(
-        new Error(
-          `Lighthouse audit produced no scores; refusing to write to Airtable. Summary: ${lhResult.summary}`
-        ),
-        { exitCode: 1 }
-      );
-    }
-    const base = openBase2(readAirtableConfig2());
-    const websites = await listWebsites2(base);
-    const target = websites.find((w) => siteSlug2(w.name) === slug);
-    if (!target) {
-      throw Object.assign(new Error(`No Websites row matched slug "${slug}"`), { exitCode: 2 });
-    }
-    const scores = lighthouseScoresFromResult2(lhResult);
-    await updateScores2(base, target.id, scores);
-    output += `
+    let writeSummary = null;
+    await new Listr(
+      [
+        {
+          title: `Write to Airtable[${slug}]`,
+          task: async (_ctx, task) => {
+            const base = openBase2(readAirtableConfig2());
+            task.output = "loading Websites\u2026";
+            const websites = await listWebsites2(base);
+            task.output = "writing scores\u2026";
+            writeSummary = await writeAuditsToAirtable2({ base, websites, slug, results });
+            task.title = `Wrote to Websites[${writeSummary.siteName}] (${writeSummary.writes.length} audit type${writeSummary.writes.length === 1 ? "" : "s"})`;
+          }
+        }
+      ],
+      { renderer }
+    ).run();
+    if (writeSummary) output += `
 
-\u2192 wrote scores to Websites[${target.name}]: P=${scores.performance} A=${scores.accessibility} BP=${scores.bestPractices} SEO=${scores.seo}`;
+${formatWriteSummary(writeSummary)}`;
   }
   return { output, code: exitCode(results) };
 }