c3b-lint 0.0.13 → 0.0.15

package/README.md

@@ -17,9 +17,9 @@ markdown that a tool can check, with one command and one config to install. It r
 4. **HTML** on every page under `src/` (partials are skipped): doctype, `lang`, charset, viewport without
    `user-scalable=no`, the required head set (title, description, absolute canonical, `og:*` with an absolute
    `og:image`, `twitter:card`, one SVG favicon; a `noindex` page is excused the canonical and social tags), no external
-   stylesheets (webfonts are self-hosted), stylesheet order (`reset`, `tokens`, `main`, …, page file last),
+   stylesheets (webfonts are self-hosted; a warning, not an error), stylesheet order (`reset`, `tokens`, `main`, …, page file last),
    the skip-link rule (`<a id="skip-link" href="#main">` first in `<body>` when there is a header or nav, absent
-   otherwise, `<main id="main">`), one `<h1>`, `alt` on every image, no inline handlers, no `<style>` or `style=""`
+   otherwise, landing on `<main id="main">` — or on the id `<main>` already has), one `<h1>`, `alt` on every image, no inline handlers, no `<style>` or `style=""`
    (except a JS-driven custom property), relative paths only (a Barba site's links outside its container — header, menu — stay root-relative), kebab-case ids and classes, no duplicate ids,
    `integrity` on CDN scripts.
 5. **Project structure**: singular directory names, lowercase kebab-case asset names (and no camera exports), and,

package/check/html.js

@@ -128,9 +128,11 @@ function checkPage(report, file, source) {
     // --- stylesheet link order: reset, tokens, main, components, page file last
     const allSheets = headEls.filter((el) => el.name === 'link' && attr(el, 'rel') === 'stylesheet')
             .map((el) => ({ el, href: attr(el, 'href') ?? '' }));
-    // frontend.md: webfonts are self-hosted woff2 in src/asset/font/; no stylesheet comes from another origin
+    // frontend.md: webfonts are self-hosted woff2 in src/asset/font/; no stylesheet comes from another origin.
+    // A warning, not an error: a licensed font can come with a stylesheet that has to load from the foundry
+    // (Monotype's tracking CSS), and a font still waiting on its licence shouldn't block every commit.
     for (const sheet of allSheets.filter((s) => /^(https?:)?\/\//.test(s.href))) {
-        report.error(at(sheet.el), `external stylesheet ${ sheet.href } — self-host it (fonts as woff2 in src/asset/font/)`);
+        report.warn(at(sheet.el), `external stylesheet ${ sheet.href } — self-host it (fonts as woff2 in src/asset/font/)`);
     }
     const sheets = allSheets.filter((s) => !/^(https?:)?\/\//.test(s.href));
     const names = sheets.map((s) => basename(s.href.split('?')[0]));
@@ -151,11 +153,14 @@ function checkPage(report, file, source) {
     const skipLink = first && first.tagName === 'a' && attr(first, 'id') === 'skip-link';
     const main = all.find((el) => el.name === 'main');
     if (hasChrome) {
-        if (!skipLink || attr(first, 'href') !== '#main') {
-            report.error(file, 'a page with a header or nav starts <body> with <a id="skip-link" href="#main">');
+        // The skip link lands on <main> by its id: "main" by default, or the id <main> already has
+        // (a Barba site scopes each page's CSS by it)
+        const target = main && attr(main, 'id');
+        if (!target) {
+            report.error(file, '<main> carries an id for the skip link to land on (id="main", or the one it already has)');
         }
-        if (!main || attr(main, 'id') !== 'main') {
-            report.error(file, '<main> carries id="main" for the skip link');
+        if (!skipLink || attr(first, 'href') !== `#${ target || 'main' }`) {
+            report.error(file, `a page with a header or nav starts <body> with <a id="skip-link" href="#${ target || 'main' }">`);
         }
     } else if (skipLink) {
         report.error(at(first), 'a page with no header or nav has no skip link');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "c3b-lint",
-  "version": "0.0.13",
+  "version": "0.0.15",
   "description": "Lints a co:lab project against the rules a tool can check: ESLint, Stylelint, the 500-line limit and the shared .editorconfig; --init writes the config files.",
   "license": "GPL-3.0",
   "type": "module",