nodality 1.3.19 → 1.3.21

package/bin/mcp-server.mjs

@@ -148,7 +148,11 @@ const TOOLS = [
       "width and reports horizontal overflow, elements past the viewport, " +
       "clipped content, images with no description, tap targets under 24px, " +
       "controls with no accessible name, skipped heading levels and text " +
-      "below the WCAG contrast minimum. Same report shape as validate_nodes; " +
+      "below the WCAG contrast minimum, and a shop product whose loading " +
+      "copy sits somewhere other than the loaded element (the shop script is " +
+      "blocked, so this needs no store). For a prerendered site page, pass " +
+      "`base` (the dev server's origin) so its fonts and assets load. Same " +
+      "report shape as validate_nodes; " +
       "`path` in an error is a CSS selector into the rendered page. It " +
       "reports facts, never taste \u2014 it will not tell you whether a " +
       "design is good.",
@@ -157,6 +161,7 @@ const TOOLS = [
       properties: {
         html: { type: "string", description: "Rendered HTML: a document or a fragment." },
         path: { type: "string", description: "Path to an HTML file, e.g. what `preview` returned." },
+        base: { type: "string", description: "Origin the page is served from, e.g. http://localhost:4000/ — resolves its /assets/ fonts and styles." },
       },
     },
   },
@@ -259,7 +264,7 @@ const IMPL = {
   // generator is a build-time script and this server answers in-process. The
   // drift test keeps the committed copy honest, so reading it is not a second
   // source of truth, it is the same one at rest.
-  check_page: async ({ html, path: file } = {}) => {
+  check_page: async ({ html, path: file, base } = {}) => {
     const { checkPage } = await import("../lib/check-page.js");
     let source = html;
     if (!source && file) {
@@ -276,7 +281,7 @@ const IMPL = {
         }],
       });
     }
-    return ok(await checkPage(source));
+    return ok(await checkPage(source, base ? { base, waitUntil: "networkidle" } : {}));
   },
 
   get_schema: async ({ type } = {}) => {

package/bin/nodality.mjs

@@ -126,6 +126,13 @@ function showUsage() {
                                               #   --dir=<path>        copy SKILL.md anywhere
                                               #   --no-mcp            skip MCP registration
                                               # Re-run after upgrading to refresh in place.
+  nodality check <page.html>... [--base=<url>] # Check rendered pages in a real browser
+                                              # (Playwright): overflow, clipping,
+                                              # contrast, labels, tap targets, a shop
+                                              # product's loading shift. --base is the
+                                              # origin they are served from, so fonts
+                                              # and /assets/ load. Exit 1 on findings.
+                                              # The same check as the MCP check_page.
   nodality schema [type] [--check]            # Print the machine-readable element
                                               # schema, derived from the source
                                               # at run time. With a type, print
@@ -916,6 +923,35 @@ async function runFanoutStandalone(rawArgs) {
   }
 }
 
+// ─── check subcommand ──────────────────────────────────────────
+
+async function runCheck(rest) {
+  const fs = await import("node:fs");
+  const files = rest.filter((a) => !a.startsWith("--"));
+  const baseArg = rest.find((a) => a.startsWith("--base="));
+  if (!files.length) {
+    console.error("[nodality] check: name one or more HTML files, e.g. npx nodality check upload/index.html --base=http://localhost:4000/");
+    process.exit(1);
+  }
+  const { checkPage } = await import("../lib/check-page.js");
+  let findings = 0;
+  for (const file of files) {
+    const html = fs.readFileSync(file, "utf8");
+    const opts = baseArg ? { base: baseArg.slice("--base=".length), waitUntil: "networkidle" } : {};
+    const r = await checkPage(html, opts);
+    if (r.errors[0]?.code === "MISSING_PEER_DEPENDENCY") {
+      console.error(`[nodality] check: ${r.errors[0].detail}\n  ${r.errors[0].suggestions.join("\n  ")}`);
+      process.exit(1);
+    }
+    findings += r.errors.length;
+    console.log(r.errors.length ? `✗ ${file} — ${r.errors.length} finding(s)` : `✓ ${file}`);
+    for (const e of r.errors) {
+      console.log(`  ${e.code}  ${e.path}  ${e.got ?? ""}  [${e.viewport}]${e.detail ? `\n    ${e.detail}` : ""}${e.suggestions.length ? `\n    → ${e.suggestions[0]}` : ""}`);
+    }
+  }
+  if (findings) process.exit(1);
+}
+
 // ─── Dispatch ──────────────────────────────────────────────────
 
 async function main() {
@@ -960,6 +996,11 @@ async function main() {
     // should cost nothing, and a broken generator must not take the CLI down.
     const { runSchema } = await import("./schema-cli.mjs");
     await runSchema(rest);
+  } else if (command === "check") {
+    // check_page without the MCP. A session where the server is not loaded
+    // (it needs approving per project) had no way to run it: the module was
+    // not exported and no command reached it.
+    await runCheck(rest);
   } else if (command === "skill") {
     const { runSkillInstall } = await import("./install-skill.mjs");
     await runSkillInstall(rest);