jig-ui 0.14.0 → 0.14.1

package/CHANGELOG.md

@@ -1,5 +1,30 @@
 # Changelog
 
+## 0.14.1
+
+`jig seo` cites only what a rule says.
+
+### Fixed
+
+- **No finding for a missing sitemap or robots file.** Both were filed under
+  rules that do not ask for them: `J-123` is about a page that must not be
+  indexed and says `robots.txt` is not it, and `J-124` is about a sitemap that
+  contradicts the pages. A project with no domain yet cannot write an honest
+  sitemap, so the only way to clear the old finding was to invent one. Both are
+  now counts on the `JIG_SEO` line.
+- **A sitemap of paths is a `J-124` error.** A `<loc>` that is `/about` rather
+  than a full URL was read as a route, although a crawler drops it. `J-124` now
+  names that case and says no sitemap is the honest one until there is an
+  origin.
+- **`pages=` counts pages.** It counted any file that mentioned `<title>` or
+  `<meta>`, so a build script, a test, a data module and a layout were pages and
+  a home page inheriting its layout was not. A page is now a route a framework
+  serves, or a whole-document template outside the layout and partial folders.
+  A dynamic route counts once.
+- **Framework routes match their sitemap entries.** `src/pages/admin` was read
+  as `/pages/admin`, so a `noindex` Astro or Next page listed in the sitemap
+  went unreported.
+
 ## 0.14.0
 
 What a stranger meets before the page, what the page must not hand them, and a

package/README.md

@@ -357,7 +357,7 @@ overwrites a config or brand file you have edited.
 | `init [--yes]` | Sets the project up: CSS system, brand colour, token files, `jig.config.json`, wired imports, baseline check. The only command that writes into your repo. |
 | `check [--all] [--ci] [--json]` | Runs the rules a machine can decide. Reports findings by rule id. |
 | `update` | Refreshes an install to a newer version, leaving alone any file you have edited. |
-| `seo [--json]` | Audits what a search engine and a link preview read, across the whole project: a route whose metadata says `noindex` sitting in the sitemap, two pages claiming one title, indexable pages with no sitemap, a sitemap that lists nothing. Needs no config, no decisions and no spec. |
+| `seo [--json]` | Audits what a search engine and a link preview read, across the whole project: a route whose metadata says `noindex` sitting in the sitemap, two pages claiming one title, a sitemap that lists nothing or lists paths a crawler drops. Whether a sitemap and a robots file exist is counted, not reported: no rule asks for either, and a site with no domain yet cannot write an honest sitemap. Needs no config, no decisions and no spec. |
 | `verdicts <surface>` | Verifies a critique's verdict files and computes its counts: every rule in each pass judged once, no id that does not exist, no rule in the wrong arm, and no verdict the render probe contradicts. |
 | `probe` | Prints the render probe — one expression the critique runs in a browser at each width. It operates the menu, measures sideways scroll, and reads whether the styles and tokens applied. |
 | `gate` | Run by the Stop hook `install` adds for Claude Code, not by hand. Blocks an agent from finishing while `check` fails on the files it changed, or the step it just ran left its work unfinished. |
@@ -448,8 +448,8 @@ in it tells strangers where to look; a path is safe to name only when something
 else protects it.
 
 `jig seo` covers what one file cannot: a route that says `noindex` and sits in
-the sitemap anyway, two pages claiming one title, a site whose pages nothing
-points at. It writes nothing and needs nothing, so it is safe to run on the first
+the sitemap anyway, two pages claiming one title, a sitemap of paths a crawler
+drops. It writes nothing and needs nothing, so it is safe to run on the first
 day, or on somebody else's codebase.
 
 ## Using it with a coding agent

package/dist/index.js

@@ -6177,8 +6177,28 @@ import { join as join36 } from "path";
 var SITEMAP_FILE = /(^|\/)sitemap(-index)?\.(xml|ts|js|mjs|tsx|jsx|rb|php|py)$/i;
 var ROBOTS_FILE = /(^|\/)robots\.(txt|ts|js|mjs|tsx|jsx)$/i;
 var ROUTE_IN_SITEMAP = /(?:<loc>\s*([^<\s]+)\s*<\/loc>)|(?:url\s*:\s*["'`]([^"'`]+)["'`])/gi;
+var TEST_FILE = /(^|\/)(test|tests|__tests__|spec|e2e|fixtures?|stories)\/|\.(test|spec|stories)\.[a-z]+$/i;
+var ROUTE_DIR = /(?:^|\/)(?:src\/|app\/)?(?:pages|routes)\/(.+)$/;
+var TEMPLATE = /\.(html?|php|erb|njk|hbs|handlebars|liquid|ejs|twig|jinja2?|j2|mustache)$/i;
+var PARTS_DIR = /(^|\/)_?(layouts?|components?|partials?|includes?)\//i;
+var ENDPOINT = /export\s+(?:const|async\s+function|function)\s+(?:GET|POST|PUT|PATCH|DELETE|ALL)\b/;
+function isPage(file, raw) {
+  if (TEST_FILE.test(file)) return false;
+  if (/(^|\/)app\/(.+\/)?page\.(tsx|jsx|ts|js|mdx|md)$/.test(file)) return true;
+  const route = ROUTE_DIR.exec(file);
+  if (route) {
+    const rest = route[1];
+    const name = rest.split("/").pop();
+    if (name.startsWith("+")) return name === "+page.svelte";
+    if (/(^|\/)api\//.test(rest) || name.startsWith("_")) return false;
+    if (/\.[a-z]+\.[cm]?[jt]s$/i.test(name)) return false;
+    if (/\.[cm]?[jt]s$/i.test(name)) return /export\s+default\b/.test(raw) && !ENDPOINT.test(raw);
+    return /\.(astro|html?|mdx?|vue|svelte|[jt]sx)$/i.test(name);
+  }
+  return TEMPLATE.test(file) && !PARTS_DIR.test(file) && isWholeDocument(raw);
+}
 function routeOf(file) {
-  const path = file.replace(/^(src|app|pages|public|content|routes)\//, "").replace(/(^|\/)(index|page)\.(html?|astro|vue|svelte|[jt]sx?|md|mdx)$/i, "$1").replace(/\.(html?|astro|vue|svelte|[jt]sx?|md|mdx)$/i, "").replace(/\/$/, "");
+  const path = file.replace(/^(?:src\/pages|app\/pages|app\/routes|src\/routes|pages|routes|app|src|public|content)\//, "").replace(/(^|\/)(index|\+?page)\.(html?|astro|vue|svelte|[jt]sx?|md|mdx)$/i, "$1").replace(/\.(html?|astro|vue|svelte|[jt]sx?|md|mdx)$/i, "").replace(/\/$/, "");
   return `/${path}`.replace(/\/+/g, "/");
 }
 function seo(opts) {
@@ -6190,6 +6210,7 @@ function seo(opts) {
   let sitemapFiles = [];
   let robotsFiles = [];
   const sitemapRoutes = /* @__PURE__ */ new Map();
+  const pathLocs = [];
   for (const file of files) {
     let raw;
     try {
@@ -6202,6 +6223,7 @@ function seo(opts) {
       for (const match of raw.matchAll(ROUTE_IN_SITEMAP)) {
         const value = (match[1] ?? match[2] ?? "").trim();
         if (!value) continue;
+        if (match[1] && !/^https?:\/\//i.test(value)) pathLocs.push({ file, loc: value });
         const path = value.replace(/^https?:\/\/[^/]+/i, "") || "/";
         sitemapRoutes.set(path.replace(/\/$/, "") || "/", file);
       }
@@ -6211,9 +6233,8 @@ function seo(opts) {
       robotsFiles.push(file);
       continue;
     }
-    if (!isReaderText(file)) continue;
+    if (!isPage(file, raw)) continue;
     const found = readMetadata(raw);
-    if (!found.declares && !isWholeDocument(raw)) continue;
     const settled = found.titles.length === 1 && !found.computedTitle ? found.titles[0].value : void 0;
     pages.push({ file, route: routeOf(file), title: settled, noindex: found.noindex });
   }
@@ -6223,6 +6244,10 @@ function seo(opts) {
       add(listed, "J-124", "error", `the sitemap lists ${page.route}, and ${page.file} says noindex. One answer per route: a crawler reading two picks one.`);
     }
   }
+  if (pathLocs.length > 0) {
+    const shown = pathLocs.slice(0, 3).map((p) => p.loc).join(", ");
+    add(pathLocs[0].file, "J-124", "error", `${pathLocs.length} sitemap entr${pathLocs.length === 1 ? "y is a path" : "ies are paths"} (${shown}), not a full URL. A crawler drops them; with no origin yet, no sitemap is the honest one.`);
+  }
   const byTitle = /* @__PURE__ */ new Map();
   for (const page of pages) {
     if (!page.title || page.noindex) continue;
@@ -6235,15 +6260,8 @@ function seo(opts) {
   }
   const indexable = pages.filter((p) => !p.noindex);
   const publicSurface = opts.mode !== "product" && opts.mode !== "operator";
-  if (publicSurface && indexable.length > 0) {
-    if (sitemapFiles.length === 0) {
-      add(".", "J-124", "warning", `${indexable.length} indexable page(s) and no sitemap. A crawler then finds what it happens to link to.`);
-    } else if (sitemapRoutes.size === 0) {
-      add(sitemapFiles[0], "J-127", "error", "the sitemap lists no routes. An empty sitemap is not a missing one: it is a claim that the site has nothing.");
-    }
-    if (robotsFiles.length === 0) {
-      add(".", "J-123", "note", "no robots file. Nothing is blocked by its absence, but nothing points at the sitemap either.");
-    }
+  if (publicSurface && indexable.length > 0 && sitemapFiles.length > 0 && sitemapRoutes.size === 0) {
+    add(sitemapFiles[0], "J-127", "error", "the sitemap lists no routes. An empty sitemap is not a missing one: it is a claim that the site has nothing.");
   }
   const titleBudget = pages.filter((p) => p.title && p.title.length > TITLE_BUDGET).length;
   const line = `JIG_SEO: pages=${pages.length} indexable=${indexable.length} noindex=${pages.length - indexable.length} sitemap=${sitemapFiles.length > 0 ? sitemapRoutes.size : "none"} robots=${robotsFiles.length > 0 ? "yes" : "no"} overlong=${titleBudget} findings=${findings.length}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jig-ui",
-  "version": "0.14.0",
+  "version": "0.14.1",
   "description": "A design system for coding agents. 130 numbered UI rules, brand x mode design tokens, and an installer for Claude Code, Codex, Cursor and opencode.",
   "license": "Apache-2.0",
   "type": "module",

package/rules/00-anti-patterns.md

@@ -608,9 +608,9 @@ The budget is not a style preference; it is the width of the box someone else re
 `robots.txt` is not this. It is public, advisory, and read by strangers as a list of interesting places: naming `/admin` there tells everyone where it is. A path is safe to name only when something else protects it — a session guard, an authenticating API — and never because the file asked politely.
 
 ### J-124 A sitemap that contradicts the page
-❌ A route listed in the sitemap whose own metadata says `noindex`; a sitemap entry for a page that does not exist
-✅ One answer per route. The sitemap lists what is indexable, and nothing else.
-Contradicting yourself in two files tells a crawler you do not know which is true, and it will decide for you.
+❌ A route listed in the sitemap whose own metadata says `noindex`; a sitemap entry for a page that does not exist; a `<loc>` written as a path, `/pricing`, which the format does not accept and a crawler drops
+✅ One answer per route. The sitemap lists what is indexable, as full URLs, and nothing else.
+Contradicting yourself in two files tells a crawler you do not know which is true, and it will decide for you. A sitemap needs the site's origin, so a project that has none yet has nothing to write: no sitemap is honest, and one full of paths is not.
 
 ### J-125 Invented facts in metadata
 ❌ `lastModified: new Date()` in a sitemap; a `datePublished` filled in because the field existed; an author, rating or price nobody supplied

package/templates/COMMAND.md.tmpl

@@ -1160,8 +1160,11 @@ Run `{{scripts_path}} seo` and report what it says. It needs no `DECISIONS.md`,
 no spec and no confirmation: it reads what the project already serves, so it is
 safe to run on the first day and on somebody else's codebase. It audits the whole project,
 which is what one file cannot: a route whose metadata says `noindex` sitting in
-the sitemap, two pages claiming the same title, indexable pages with no sitemap,
-a sitemap that lists nothing.
+the sitemap, two pages claiming the same title, a sitemap that lists nothing or
+lists paths instead of full URLs. It counts the pages it found, a dynamic route
+once, and whether a sitemap and a robots file exist; neither is a finding, since
+no rule asks for them and a project with no domain yet cannot write an honest
+sitemap. Do not invent one to clear a count.
 
 It is not a phase. The per-page half belongs where the copy is decided and
 written — `spec` records whether a page is indexable and what it claims, `make`

package/templates/command-metadata.json

@@ -25,7 +25,7 @@
     "status": "available"
   },
   "seo": {
-    "description": "Audit what a search engine and a link preview read, across the whole project: a noindex route in the sitemap, two pages with one title, indexable pages with no sitemap, a sitemap that lists nothing. The per-page half lives in spec, make and check.",
+    "description": "Audit what a search engine and a link preview read, across the whole project: a noindex route in the sitemap, two pages with one title, a sitemap that lists nothing or lists paths instead of URLs. The per-page half lives in spec, make and check.",
     "argumentHint": "[--json]",
     "status": "available"
   },