@reddoorla/maintenance 0.7.0 → 0.8.0

package/dist/index.d.ts

@@ -92,6 +92,35 @@ declare function localPath(path: string, opts?: LocalPathOptions): InventoryProv
 
 declare function fromJsonFile(path: string): InventoryProvider;
 
+type AirtableConfig = {
+    apiKey: string;
+    baseId: string;
+};
+type AirtableBase = ReturnType<typeof openBase>;
+declare function openBase(cfg: AirtableConfig): airtable_lib_airtable_base_js.AirtableBase;
+
+type AirtableInventoryOptions = {
+    /**
+     * Local workdir to compute each site's path as `{workdir}/{slug}`.
+     * Defaults to REDDOOR_FLEET_WORKDIR env var if not provided.
+     * Airtable doesn't store local checkout paths, so this is required.
+     */
+    workdir?: string;
+};
+/**
+ * Read sites from the Airtable Websites table as an InventoryProvider.
+ * Each row becomes one Site; `path` is computed as `{workdir}/{slug}`.
+ * Sites where BOTH maintenance freq AND testing freq are "None" are excluded
+ * (they're inactive — no scheduled audits or reports).
+ *
+ * Note: `repoUrl` is set to the production URL (Websites.url). For sites
+ * cloned via `--workdir` semantics this is wrong — the convention should be
+ * tightened (e.g. add a `repo` field to Websites) when fleet-clone-from-
+ * airtable becomes a real flow. For local audits where the site is already
+ * checked out at `path`, the `repoUrl` is unused.
+ */
+declare function fromAirtableBase(base: AirtableBase, opts?: AirtableInventoryOptions): InventoryProvider;
+
 type ReportType = "Maintenance" | "Testing";
 type LighthouseScores = {
     performance: number;
@@ -124,18 +153,13 @@ type ReportData = {
     headerImageCid: string;
 };
 
-type AirtableConfig = {
-    apiKey: string;
-    baseId: string;
-};
-type AirtableBase = ReturnType<typeof openBase>;
-declare function openBase(cfg: AirtableConfig): airtable_lib_airtable_base_js.AirtableBase;
-
 type Frequency = "None" | "Monthly" | "Quarterly" | "Yearly";
+type Status = "in development" | "launch period" | "maintenance" | "hosting" | "probably not our problem" | "deprecated";
 type WebsiteRow = {
     id: string;
     name: string;
     url: string;
+    status: Status | null;
     pointOfContact: string | null;
     maintenanceFreq: Frequency;
     testingFreq: Frequency;
@@ -151,11 +175,13 @@ type WebsiteRow = {
         filename: string;
         type: string;
     } | null;
-    /** Lighthouse "current state" snapshot, manually kept fresh by the operator. */
+    /** Lighthouse "current state" snapshot, kept fresh by `audit lighthouse --write-airtable`. */
     pScore: number | null;
     rScore: number | null;
     bpScore: number | null;
     seoScore: number | null;
+    /** ISO timestamp set by `audit lighthouse --write-airtable` when scores were last refreshed. */
+    lastLighthouseAuditAt: string | null;
 };
 
 type DeliveryStatus = "pending" | "delivered" | "bounced" | "complained";
@@ -268,4 +294,4 @@ type DueItem = {
  */
 declare function findDueReports(websites: WebsiteRow[], reports: ReportRow[], today: Date): DueItem[];
 
-export { ALL_AUDIT_NAMES, ALL_RECIPE_NAMES, AuditName, AuditResult, type BumpDepsOptions, type ConvertToPnpmOptions, type DraftOptions, type DraftResult, type DueItem, type HeaderImage, InventoryProvider, type LighthouseScores, type LocalPathOptions, type OnboardAudit, type OnboardOptions, type OrchestrateOptions, RecipeName, RecipeResult, type RenderResult, type ReportData, type ReportType, Site, type UpgradeSvelte4to5Options, a11yAudit, bumpDeps, convertToPnpm, depsAudit, draftReportForSite, findDueReports, fromJsonFile, isRecipeName, lighthouseAudit, lintAudit, localPath, onboard, renderReportHtml, runAudits, runAuditsAcross, securityAudit, sendApprovedReports, svelteCodemods, upgradeSvelte4to5 };
+export { ALL_AUDIT_NAMES, ALL_RECIPE_NAMES, type AirtableInventoryOptions, AuditName, AuditResult, type BumpDepsOptions, type ConvertToPnpmOptions, type DraftOptions, type DraftResult, type DueItem, type HeaderImage, InventoryProvider, type LighthouseScores, type LocalPathOptions, type OnboardAudit, type OnboardOptions, type OrchestrateOptions, RecipeName, RecipeResult, type RenderResult, type ReportData, type ReportType, Site, type UpgradeSvelte4to5Options, a11yAudit, bumpDeps, convertToPnpm, depsAudit, draftReportForSite, findDueReports, fromAirtableBase, fromJsonFile, isRecipeName, lighthouseAudit, lintAudit, localPath, onboard, renderReportHtml, runAudits, runAuditsAcross, securityAudit, sendApprovedReports, svelteCodemods, upgradeSvelte4to5 };

package/dist/index.js

@@ -1769,8 +1769,8 @@ import { fileURLToPath } from "url";
 import { dirname, join as join15 } from "path";
 function selfPackageVersion(callerImportMetaUrl) {
   try {
-    const here = dirname(fileURLToPath(callerImportMetaUrl));
-    const raw = readFileSync(join15(here, "..", "..", "package.json"), "utf-8");
+    const here2 = dirname(fileURLToPath(callerImportMetaUrl));
+    const raw = readFileSync(join15(here2, "..", "..", "package.json"), "utf-8");
     const pkg = JSON.parse(raw);
     return pkg.version ?? "0.0.0";
   } catch {
@@ -1924,16 +1924,106 @@ function fromJsonFile(path) {
   };
 }
 
+// src/reports/airtable/websites.ts
+var WEBSITES_TABLE = "Websites";
+function siteSlug(name) {
+  return name.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-|-$/g, "");
+}
+function mapRow(rec) {
+  const f = rec.fields;
+  const attachments = f["Header image"] ?? [];
+  const header = attachments[0] ?? null;
+  return {
+    id: rec.id,
+    name: String(f["Name"] ?? ""),
+    url: String(f["url"] ?? ""),
+    status: f["Status"] ?? null,
+    pointOfContact: f["point of contact"] ?? null,
+    maintenanceFreq: f["maintenence freq"] ?? "None",
+    testingFreq: f["testing freq"] ?? "None",
+    maintenanceDay: f["maintenance day"] ?? null,
+    testingDay: f["testing day"] ?? null,
+    ga4PropertyId: f["GA4 property ID"] ?? null,
+    reportRecipientsTo: f["Report recipients (To)"] ?? null,
+    reportRecipientsCc: f["Report recipients (CC)"] ?? null,
+    headerImage: header,
+    pScore: f["pScore"] ?? null,
+    rScore: f["rScore"] ?? null,
+    bpScore: f["bpScore"] ?? null,
+    seoScore: f["seoScore"] ?? null,
+    lastLighthouseAuditAt: f["Last lighthouse audit at"] ?? null
+  };
+}
+async function listWebsites(base) {
+  const out = [];
+  await base(WEBSITES_TABLE).select({ pageSize: 100 }).eachPage((records, fetchNextPage) => {
+    for (const rec of records) out.push(mapRow({ id: rec.id, fields: rec.fields }));
+    fetchNextPage();
+  });
+  return out;
+}
+
+// src/inventory/airtable.ts
+function fromAirtableBase(base, opts = {}) {
+  return async () => {
+    const workdir = opts.workdir ?? process.env.REDDOOR_FLEET_WORKDIR;
+    if (!workdir) {
+      throw new Error(
+        "fromAirtableBase requires `workdir` option or REDDOOR_FLEET_WORKDIR env (sites need a local path)"
+      );
+    }
+    const websites = await listWebsites(base);
+    return websites.filter((w) => w.maintenanceFreq !== "None" || w.testingFreq !== "None").map((w) => {
+      const slug = siteSlug(w.name);
+      const site = {
+        path: `${workdir}/${slug}`,
+        name: slug,
+        meta: { airtableRowId: w.id, displayName: w.name }
+      };
+      if (w.url) site.repoUrl = w.url;
+      return site;
+    });
+  };
+}
+
 // src/reports/draft.ts
 import { mkdir, writeFile as writeFile9 } from "fs/promises";
-import { dirname as dirname2 } from "path";
+import { dirname as dirname3 } from "path";
 
 // src/reports/render.ts
 import mjml2html from "mjml";
 
+// src/reports/maintenance-email/assets/index.ts
+import { readFile as readFile10 } from "fs/promises";
+import { dirname as dirname2, join as join17 } from "path";
+import { fileURLToPath as fileURLToPath2 } from "url";
+var here = dirname2(fileURLToPath2(import.meta.url));
+var CHECK_CID = "rd-check-png";
+var BLURRED_CID = "rd-blurred-tests-jpg";
+async function loadBundledImages() {
+  const [check, blurred] = await Promise.all([
+    readFile10(join17(here, "check.png")),
+    readFile10(join17(here, "blurredTests.jpg"))
+  ]);
+  return {
+    check: {
+      bytes: new Uint8Array(check),
+      contentType: "image/png",
+      cid: CHECK_CID,
+      filename: "check.png"
+    },
+    blurred: {
+      bytes: new Uint8Array(blurred),
+      contentType: "image/jpeg",
+      cid: BLURRED_CID,
+      filename: "blurredTests.jpg"
+    }
+  };
+}
+
 // src/reports/maintenance-email/template.ts
-var CHECK_PNG = "https://d3eq0h5l8sxf6t.cloudfront.net/maintenance-email/check.png";
-var BLURRED_TESTS = "https://d3eq0h5l8sxf6t.cloudfront.net/maintenance-email/blurredTests.jpg";
+var CHECK_PNG = `cid:${CHECK_CID}`;
+var BLURRED_TESTS = `cid:${BLURRED_CID}`;
 function fmtDate(d) {
   if (!d) return "";
   const mm = String(d.getUTCMonth() + 1).padStart(2, "0");
@@ -2104,43 +2194,6 @@ async function renderReportHtml(data) {
   return { html: out.html, warnings: out.errors ?? [] };
 }
 
-// src/reports/airtable/websites.ts
-var WEBSITES_TABLE = "Websites";
-function siteSlug(name) {
-  return name.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-|-$/g, "");
-}
-function mapRow(rec) {
-  const f = rec.fields;
-  const attachments = f["Header image"] ?? [];
-  const header = attachments[0] ?? null;
-  return {
-    id: rec.id,
-    name: String(f["Name"] ?? ""),
-    url: String(f["url"] ?? ""),
-    pointOfContact: f["point of contact"] ?? null,
-    maintenanceFreq: f["maintenence freq"] ?? "None",
-    testingFreq: f["testing freq"] ?? "None",
-    maintenanceDay: f["maintenance day"] ?? null,
-    testingDay: f["testing day"] ?? null,
-    ga4PropertyId: f["GA4 property ID"] ?? null,
-    reportRecipientsTo: f["Report recipients (To)"] ?? null,
-    reportRecipientsCc: f["Report recipients (CC)"] ?? null,
-    headerImage: header,
-    pScore: f["pScore"] ?? null,
-    rScore: f["rScore"] ?? null,
-    bpScore: f["bpScore"] ?? null,
-    seoScore: f["seoScore"] ?? null
-  };
-}
-async function listWebsites(base) {
-  const out = [];
-  await base(WEBSITES_TABLE).select({ pageSize: 100 }).eachPage((records, fetchNextPage) => {
-    for (const rec of records) out.push(mapRow({ id: rec.id, fields: rec.fields }));
-    fetchNextPage();
-  });
-  return out;
-}
-
 // src/reports/airtable/reports.ts
 var REPORTS_TABLE = "Reports";
 function mapRow2(rec) {
@@ -2242,6 +2295,40 @@ async function stampSent(base, recordId, sentAt, messageId) {
   ]);
 }
 
+// src/reports/airtable/attachments.ts
+async function fetchAttachmentBytes(url) {
+  const res = await fetch(url);
+  if (!res.ok) {
+    throw new Error(
+      `Failed to fetch Airtable attachment ${res.status} ${res.statusText} (url=${url})`
+    );
+  }
+  const contentType = res.headers.get("content-type") ?? "application/octet-stream";
+  const ab = await res.arrayBuffer();
+  return { bytes: new Uint8Array(ab), contentType };
+}
+async function uploadAttachment(recordId, fieldName, body, filename, contentType) {
+  const apiKey = process.env.AIRTABLE_PAT;
+  const baseId = process.env.AIRTABLE_BASE_ID;
+  if (!apiKey || !baseId) {
+    throw new Error("AIRTABLE_PAT and AIRTABLE_BASE_ID must be set");
+  }
+  const base64 = typeof body === "string" ? Buffer.from(body, "utf-8").toString("base64") : Buffer.from(body).toString("base64");
+  const payload = { contentType, file: base64, filename };
+  const url = `https://content.airtable.com/v0/${baseId}/${recordId}/${encodeURIComponent(fieldName)}/uploadAttachment`;
+  const res = await fetch(url, {
+    method: "POST",
+    headers: {
+      Authorization: `Bearer ${apiKey}`,
+      "Content-Type": "application/json"
+    },
+    body: JSON.stringify(payload)
+  });
+  if (!res.ok) {
+    throw new Error(`Airtable upload failed: ${res.status} ${res.statusText} ${await res.text()}`);
+  }
+}
+
 // src/reports/draft.ts
 function scoresFromWebsite(siteRow) {
   const { pScore, rScore, bpScore, seoScore } = siteRow;
@@ -2280,7 +2367,7 @@ async function draftReportForSite(base, siteRow, reportType, options = {}) {
   });
   if (options.previewOnly) {
     const path = options.previewPath ?? `reports/${slug}/draft.html`;
-    await mkdir(dirname2(path), { recursive: true });
+    await mkdir(dirname3(path), { recursive: true });
     await writeFile9(path, html, "utf-8");
     return { reportRow: null, htmlPath: path, html };
   }
@@ -2296,7 +2383,8 @@ async function draftReportForSite(base, siteRow, reportType, options = {}) {
     lighthouse: scores,
     lastTestedDate
   });
-  await uploadHtmlAttachment(created.id, html, slug, periodEnd);
+  const htmlFilename = `${slug}-${periodEnd.toISOString().slice(0, 10)}.html`;
+  await uploadAttachment(created.id, "Rendered HTML", html, htmlFilename, "text/html");
   await setDraftReady(base, created.id, true);
   return { reportRow: created, htmlPath: null, html };
 }
@@ -2306,28 +2394,6 @@ async function derivePeriodStart(base, siteRow, reportType, today) {
   const latest = sameType[sameType.length - 1];
   return latest ? new Date(latest) : daysAgo(today, 30);
 }
-async function uploadHtmlAttachment(recordId, html, slug, periodEnd) {
-  const apiKey = process.env.AIRTABLE_PAT;
-  const baseId = process.env.AIRTABLE_BASE_ID;
-  const filename = `${slug}-${periodEnd.toISOString().slice(0, 10)}.html`;
-  const body = {
-    contentType: "text/html",
-    file: Buffer.from(html, "utf-8").toString("base64"),
-    filename
-  };
-  const url = `https://content.airtable.com/v0/${baseId}/${recordId}/Rendered%20HTML/uploadAttachment`;
-  const res = await fetch(url, {
-    method: "POST",
-    headers: {
-      Authorization: `Bearer ${apiKey}`,
-      "Content-Type": "application/json"
-    },
-    body: JSON.stringify(body)
-  });
-  if (!res.ok) {
-    throw new Error(`Airtable upload failed: ${res.status} ${res.statusText} ${await res.text()}`);
-  }
-}
 
 // src/reports/airtable/client.ts
 import Airtable from "airtable";
@@ -2342,19 +2408,6 @@ function openBase(cfg) {
   return new Airtable({ apiKey: cfg.apiKey }).base(cfg.baseId);
 }
 
-// src/reports/airtable/attachments.ts
-async function fetchAttachmentBytes(url) {
-  const res = await fetch(url);
-  if (!res.ok) {
-    throw new Error(
-      `Failed to fetch Airtable attachment ${res.status} ${res.statusText} (url=${url})`
-    );
-  }
-  const contentType = res.headers.get("content-type") ?? "application/octet-stream";
-  const ab = await res.arrayBuffer();
-  return { bytes: new Uint8Array(ab), contentType };
-}
-
 // src/reports/send/resend.ts
 import { Resend } from "resend";
 function defaultResendClient() {
@@ -2419,6 +2472,7 @@ async function sendOne(client, base, site, report) {
     throw new Error(`Report ${report.reportId} has no Lighthouse scores`);
   }
   const { bytes, contentType } = await fetchAttachmentBytes(site.headerImage.url);
+  const bundled = await loadBundledImages();
   const slug = siteSlug(site.name);
   const cidName = `${slug}-header`;
   const { html } = await renderReportHtml({
@@ -2471,6 +2525,21 @@ async function sendOne(client, base, site, report) {
         content: Buffer.from(bytes).toString("base64"),
         contentType,
         inlineContentId: cidName
+      },
+      // Bundled images referenced via cid:rd-check-png / cid:rd-blurred-tests-jpg
+      // in the template. Attached inline so the email is self-contained — no
+      // external CDN dependency, no image-blocked broken icons in webmail.
+      {
+        filename: bundled.check.filename,
+        content: Buffer.from(bundled.check.bytes).toString("base64"),
+        contentType: bundled.check.contentType,
+        inlineContentId: bundled.check.cid
+      },
+      {
+        filename: bundled.blurred.filename,
+        content: Buffer.from(bundled.blurred.bytes).toString("base64"),
+        contentType: bundled.blurred.contentType,
+        inlineContentId: bundled.blurred.cid
       }
     ],
     // Stable across retries of the same row — if Airtable stamping fails after a
@@ -2508,6 +2577,12 @@ function isProbablyEmail(s) {
 }
 
 // src/reports/due.ts
+var ELIGIBLE_STATUSES = /* @__PURE__ */ new Set([
+  "in development",
+  "launch period",
+  "maintenance",
+  "hosting"
+]);
 var MONTHS = {
   Monthly: 1,
   Quarterly: 3,
@@ -2537,6 +2612,7 @@ function findDueReports(websites, reports, today) {
   const out = [];
   const todayStart = startOfDay(today);
   for (const site of websites) {
+    if (site.status !== null && !ELIGIBLE_STATUSES.has(site.status)) continue;
     for (const type of ["Maintenance", "Testing"]) {
       const freq = type === "Maintenance" ? site.maintenanceFreq : site.testingFreq;
       if (freq === "None") continue;
@@ -2564,6 +2640,7 @@ export {
   depsAudit,
   draftReportForSite,
   findDueReports,
+  fromAirtableBase,
   fromJsonFile,
   isRecipeName,
   lighthouseAudit,