@piercebarney/whs-eleventy 2026.9.23 → 2026.9.24

package/lib/check-links.js

@@ -135,7 +135,7 @@ function outputHas(urlPath) {
   return fs.existsSync(asFile + "/index.html") || fs.existsSync(asFile + ".html");
 }
 
-function checkRef(file, pageUrl, raw, { external = true, isHyperlink = false } = {}) {
+function checkRef(file, pageUrl, resolveBase, raw, { external = true, isHyperlink = false } = {}) {
   if (!raw) return;
   const v = raw.trim();
   if (v === "" || v.startsWith("data:") || v.startsWith("mailto:") || v.startsWith("tel:")) return;
@@ -185,13 +185,15 @@ function checkRef(file, pageUrl, raw, { external = true, isHyperlink = false } =
     return;
   }
   // relative path — resolve against the page dir
-  const resolved = path.posix.join(pageUrl, v);
+  const resolved = path.posix.join(resolveBase, v);
   if (!outputHas(resolved)) err(file, `relative reference does not resolve: ${v}`);
 }
 
 for (const f of htmlFiles) {
   const rel = "/" + path.relative(SITE, f).replace(/\\/g, "/");
   const pageUrl = rel.replace(/index\.html$/, "");
+  const dir = path.posix.dirname(rel);
+  const resolveBase = dir === "/" ? "/" : dir + "/";
   const root = parse(fs.readFileSync(f, "utf8"));
 
   // CSP-safety: the strict CSP (script-src 'self'; style-src 'self') rejects
@@ -216,7 +218,7 @@ for (const f of htmlFiles) {
 
   root.querySelectorAll("a[href]").forEach((a) => {
     const href = a.getAttribute("href");
-    checkRef(f, pageUrl, href, { isHyperlink: true });
+    checkRef(f, pageUrl, resolveBase, href, { isHyperlink: true });
     if (/^https?:\/\//i.test(href || "")) {
       let origin;
       try {
@@ -233,12 +235,18 @@ for (const f of htmlFiles) {
     }
   });
 
-  root.querySelectorAll("link[href]").forEach((l) => checkRef(f, pageUrl, l.getAttribute("href")));
-  root.querySelectorAll("script[src]").forEach((s) => checkRef(f, pageUrl, s.getAttribute("src")));
-  root.querySelectorAll("img[src]").forEach((i) => checkRef(f, pageUrl, i.getAttribute("src")));
+  root
+    .querySelectorAll("link[href]")
+    .forEach((l) => checkRef(f, pageUrl, resolveBase, l.getAttribute("href")));
+  root
+    .querySelectorAll("script[src]")
+    .forEach((s) => checkRef(f, pageUrl, resolveBase, s.getAttribute("src")));
+  root
+    .querySelectorAll("img[src]")
+    .forEach((i) => checkRef(f, pageUrl, resolveBase, i.getAttribute("src")));
   root
     .querySelectorAll('meta[property="og:image"], meta[name="twitter:image"]')
-    .forEach((m) => checkRef(f, pageUrl, m.getAttribute("content")));
+    .forEach((m) => checkRef(f, pageUrl, resolveBase, m.getAttribute("content")));
 
   root.querySelectorAll('script[type="application/ld+json"]').forEach((s, i) => {
     let data;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@piercebarney/whs-eleventy",
-  "version": "2026.9.23",
+  "version": "2026.9.24",
   "description": "The web house style's Eleventy + Netlify tooling — the compliance sweep, the infra doctor, the link/CSP integrity check, and the a11y scan, shared by every project on the stack.",
   "bin": {
     "whs": "cli.js"

package/standard/CHANGELOG.md

@@ -21,6 +21,34 @@ enforced, new *opt-in* chapters/capabilities, tooling-only changes — is
 
 ---
 
+## 2026-09-24 [non-breaking]
+
+### packages/whs-eleventy
+
+`check-links.js` resolved a relative reference (`favicon.svg`, `styles.css`,
+etc.) against `pageUrl`, a literal `index.html`-stripped path — correct for
+directory-style permalinks (`/foo/index.html` → `/foo/`) but wrong for a
+flat, non-directory permalink like `privacy.html`, where `pageUrl` stayed
+`/privacy.html` and `path.posix.join` then treated it as if it were a
+directory, producing false `relative reference does not resolve` findings
+for files that genuinely exist and resolve correctly in a real browser.
+`pageUrl` is also used for same-page `#anchor` lookups, where the literal
+value is exactly what's needed — so the fix doesn't touch `pageUrl` itself.
+Instead `checkRef()` now takes a separate `resolveBase` (the directory
+containing the page, via `path.posix.dirname`) used only for relative-
+reference resolution, leaving anchor lookups untouched. Found while
+covenant-in-color's adoption session ran the newly-fixed `npm run links` for
+the first time and filed as
+`feedback/2026-09-23-check-links-relative-resolution-non-directory-permalink.md`;
+fixed in `7edbc68`.
+
+This is the fourth `packages/whs-eleventy`-only fix shipped this week (the
+third, 2026.9.23, already published and live) — `core.md`, `stacks/*.md`,
+and `index.json` content are unchanged; only the doc pins and
+`packages/whs-eleventy/package.json` (now `2026.9.24`) moved.
+
+---
+
 ## 2026-09-23 — Republish 2026-09-22-2 as a clean bare-date version [non-breaking]
 
 ### packages/whs-eleventy

package/standard/core.md

@@ -1,6 +1,6 @@
 # Web project house style — CORE (stack-agnostic)
 
-**Version:** 2026-09-23 · **Status:** active
+**Version:** 2026-09-24 · **Status:** active
 
 This is the stack-agnostic contract every web project follows, regardless of
 framework, host, or CSS system. It says **what** must be true, with concrete
@@ -1467,7 +1467,7 @@ governs it and follow that chapter — the slugs are the index (a `<script>` →
 - production-url: https://example.com
 - content-type: tool           # tool | article  (article => feeds)
 - publishing-rate: ~5 pages/week
-- standard-version: 2026-09-23  # recommended — enables standard-version drift tracking (#compliance)
+- standard-version: 2026-09-24  # recommended — enables standard-version drift tracking (#compliance)
 ```
 
 If the section is absent, the assistant's first action is to run the flow above