pagetrace 0.5.0 → 0.6.0

package/CHANGELOG.md

@@ -6,6 +6,21 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 While the version is below 1.0.0, breaking changes ship in a minor release.
 
+## [0.6.0] - 2026-09-06
+
+### Changed
+
+- **Breaking.** `duplicate.title`, `duplicate.description` and
+  `duplicate.canonical` now emit one finding per affected route instead of a
+  single finding with `route: null`. Integrations reading `route` on these three
+  codes will see a path where they saw null; the codes themselves are unchanged.
+
+  Found by auditing a real site. The report said "2 pages share the same meta
+  description" and could not say which two, because the routes lived in `after`,
+  which the rollup drops — leaving the only actionable part of the finding
+  invisible. Per-route findings also make the counts truthful and point
+  `--format github` annotations at the pages rather than at "site".
+
 ## [0.5.0] - 2026-09-06
 
 ### Added
@@ -20,8 +35,15 @@ While the version is below 1.0.0, breaking changes ship in a minor release.
 - A composite GitHub Action (`action.yml`). Three lines in a workflow run the
   check and post the findings as a pull request comment, editing the previous
   comment on each push rather than stacking new ones. It fetches the baseline ref
-  first, since a shallow CI checkout usually has only the PR head.
+  first, since a shallow CI checkout usually has only the PR head. The comment is
+  skipped for pull requests from a fork, which run with a read-only token and
+  would otherwise fail with a 403 through no fault of the contributor; the
+  findings still reach the step summary and still set the exit code.
 - `snapshotFromGitRef(ref, path)` is exported.
+- `examples/site`, a small deliberately correct site with a committed baseline.
+  The action runs against it on every pull request to this repo, so a change
+  that breaks the action's own wiring fails here rather than in someone else's
+  CI. It doubles as a worked example of what a clean surface looks like.
 
 ### Fixed
 
@@ -194,6 +216,7 @@ Initial release. `snapshot`, `check` and `audit` commands; filesystem and HTTP
 crawling; diff classified by transition; absolute, cross-page and hreflang audit
 rules; pretty, JSON, markdown, GitHub and HTML reporters.
 
+[0.6.0]: https://github.com/shyamexe/pagetrace/compare/v0.5.0...v0.6.0
 [0.5.0]: https://github.com/shyamexe/pagetrace/compare/v0.4.0...v0.5.0
 [0.4.0]: https://github.com/shyamexe/pagetrace/compare/v0.3.0...v0.4.0
 [0.3.0]: https://github.com/shyamexe/pagetrace/compare/v0.2.0...v0.3.0

package/dist/cli.cjs

@@ -266,39 +266,38 @@ function auditCrossPage(snapshot) {
     }
     return map;
   };
-  for (const [title, routes] of group((p) => p.title)) {
-    if (routes.length > 1) {
-      findings.push({
-        code: "duplicate.title",
-        severity: "warn",
-        route: null,
-        message: `${routes.length} pages share the title "${title}".`,
-        after: routes
-      });
-    }
-  }
-  for (const [, routes] of group((p) => p.description)) {
-    if (routes.length > 1) {
-      findings.push({
-        code: "duplicate.description",
-        severity: "warn",
-        route: null,
-        message: `${routes.length} pages share the same meta description.`,
-        after: routes
-      });
-    }
-  }
-  for (const [canonical, routes] of group((p) => p.canonical)) {
-    if (routes.length > 1) {
-      findings.push({
-        code: "duplicate.canonical",
-        severity: "error",
-        route: null,
-        message: `${routes.length} pages canonicalise to ${canonical}.`,
-        after: routes
-      });
+  const duplicates = (code, severity, keyed, message) => {
+    for (const [value, routes] of keyed) {
+      if (routes.length < 2) continue;
+      for (const route of routes) {
+        findings.push({
+          code,
+          severity,
+          route,
+          message: message(value, routes.length),
+          after: routes
+        });
+      }
     }
-  }
+  };
+  duplicates(
+    "duplicate.title",
+    "warn",
+    group((p) => p.title),
+    (title, n) => `${n} pages share the title "${title}".`
+  );
+  duplicates(
+    "duplicate.description",
+    "warn",
+    group((p) => p.description),
+    (_, n) => `${n} pages share the same meta description.`
+  );
+  duplicates(
+    "duplicate.canonical",
+    "error",
+    group((p) => p.canonical),
+    (canonical, n) => `${n} pages canonicalise to ${canonical}.`
+  );
   const expectedOrigin = snapshot.site.origin ?? null;
   for (const page of pages) {
     if (!page.canonical) continue;
@@ -1573,7 +1572,7 @@ cli.command("audit", "Audit a site as it stands, with explanations and fixes").o
   }
 });
 cli.help();
-cli.version("0.5.0");
+cli.version("0.6.0");
 async function main() {
   try {
     cli.parse(process.argv, { run: false });

package/dist/cli.js

@@ -243,39 +243,38 @@ function auditCrossPage(snapshot) {
     }
     return map;
   };
-  for (const [title, routes] of group((p) => p.title)) {
-    if (routes.length > 1) {
-      findings.push({
-        code: "duplicate.title",
-        severity: "warn",
-        route: null,
-        message: `${routes.length} pages share the title "${title}".`,
-        after: routes
-      });
-    }
-  }
-  for (const [, routes] of group((p) => p.description)) {
-    if (routes.length > 1) {
-      findings.push({
-        code: "duplicate.description",
-        severity: "warn",
-        route: null,
-        message: `${routes.length} pages share the same meta description.`,
-        after: routes
-      });
-    }
-  }
-  for (const [canonical, routes] of group((p) => p.canonical)) {
-    if (routes.length > 1) {
-      findings.push({
-        code: "duplicate.canonical",
-        severity: "error",
-        route: null,
-        message: `${routes.length} pages canonicalise to ${canonical}.`,
-        after: routes
-      });
+  const duplicates = (code, severity, keyed, message) => {
+    for (const [value, routes] of keyed) {
+      if (routes.length < 2) continue;
+      for (const route of routes) {
+        findings.push({
+          code,
+          severity,
+          route,
+          message: message(value, routes.length),
+          after: routes
+        });
+      }
     }
-  }
+  };
+  duplicates(
+    "duplicate.title",
+    "warn",
+    group((p) => p.title),
+    (title, n) => `${n} pages share the title "${title}".`
+  );
+  duplicates(
+    "duplicate.description",
+    "warn",
+    group((p) => p.description),
+    (_, n) => `${n} pages share the same meta description.`
+  );
+  duplicates(
+    "duplicate.canonical",
+    "error",
+    group((p) => p.canonical),
+    (canonical, n) => `${n} pages canonicalise to ${canonical}.`
+  );
   const expectedOrigin = snapshot.site.origin ?? null;
   for (const page of pages) {
     if (!page.canonical) continue;
@@ -1550,7 +1549,7 @@ cli.command("audit", "Audit a site as it stands, with explanations and fixes").o
   }
 });
 cli.help();
-cli.version("0.5.0");
+cli.version("0.6.0");
 async function main() {
   try {
     cli.parse(process.argv, { run: false });

package/dist/index.cjs

@@ -307,39 +307,38 @@ function auditCrossPage(snapshot) {
     }
     return map;
   };
-  for (const [title, routes] of group((p) => p.title)) {
-    if (routes.length > 1) {
-      findings.push({
-        code: "duplicate.title",
-        severity: "warn",
-        route: null,
-        message: `${routes.length} pages share the title "${title}".`,
-        after: routes
-      });
-    }
-  }
-  for (const [, routes] of group((p) => p.description)) {
-    if (routes.length > 1) {
-      findings.push({
-        code: "duplicate.description",
-        severity: "warn",
-        route: null,
-        message: `${routes.length} pages share the same meta description.`,
-        after: routes
-      });
-    }
-  }
-  for (const [canonical, routes] of group((p) => p.canonical)) {
-    if (routes.length > 1) {
-      findings.push({
-        code: "duplicate.canonical",
-        severity: "error",
-        route: null,
-        message: `${routes.length} pages canonicalise to ${canonical}.`,
-        after: routes
-      });
+  const duplicates = (code, severity, keyed, message) => {
+    for (const [value, routes] of keyed) {
+      if (routes.length < 2) continue;
+      for (const route of routes) {
+        findings.push({
+          code,
+          severity,
+          route,
+          message: message(value, routes.length),
+          after: routes
+        });
+      }
     }
-  }
+  };
+  duplicates(
+    "duplicate.title",
+    "warn",
+    group((p) => p.title),
+    (title, n) => `${n} pages share the title "${title}".`
+  );
+  duplicates(
+    "duplicate.description",
+    "warn",
+    group((p) => p.description),
+    (_, n) => `${n} pages share the same meta description.`
+  );
+  duplicates(
+    "duplicate.canonical",
+    "error",
+    group((p) => p.canonical),
+    (canonical, n) => `${n} pages canonicalise to ${canonical}.`
+  );
   const expectedOrigin = snapshot.site.origin ?? null;
   for (const page of pages) {
     if (!page.canonical) continue;

package/dist/index.js

@@ -236,39 +236,38 @@ function auditCrossPage(snapshot) {
     }
     return map;
   };
-  for (const [title, routes] of group((p) => p.title)) {
-    if (routes.length > 1) {
-      findings.push({
-        code: "duplicate.title",
-        severity: "warn",
-        route: null,
-        message: `${routes.length} pages share the title "${title}".`,
-        after: routes
-      });
-    }
-  }
-  for (const [, routes] of group((p) => p.description)) {
-    if (routes.length > 1) {
-      findings.push({
-        code: "duplicate.description",
-        severity: "warn",
-        route: null,
-        message: `${routes.length} pages share the same meta description.`,
-        after: routes
-      });
-    }
-  }
-  for (const [canonical, routes] of group((p) => p.canonical)) {
-    if (routes.length > 1) {
-      findings.push({
-        code: "duplicate.canonical",
-        severity: "error",
-        route: null,
-        message: `${routes.length} pages canonicalise to ${canonical}.`,
-        after: routes
-      });
+  const duplicates = (code, severity, keyed, message) => {
+    for (const [value, routes] of keyed) {
+      if (routes.length < 2) continue;
+      for (const route of routes) {
+        findings.push({
+          code,
+          severity,
+          route,
+          message: message(value, routes.length),
+          after: routes
+        });
+      }
     }
-  }
+  };
+  duplicates(
+    "duplicate.title",
+    "warn",
+    group((p) => p.title),
+    (title, n) => `${n} pages share the title "${title}".`
+  );
+  duplicates(
+    "duplicate.description",
+    "warn",
+    group((p) => p.description),
+    (_, n) => `${n} pages share the same meta description.`
+  );
+  duplicates(
+    "duplicate.canonical",
+    "error",
+    group((p) => p.canonical),
+    (canonical, n) => `${n} pages canonicalise to ${canonical}.`
+  );
   const expectedOrigin = snapshot.site.origin ?? null;
   for (const page of pages) {
     if (!page.canonical) continue;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pagetrace",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "description": "Baseline your site's SEO and AEO surface, diff every build against it, and fail CI on regressions.",
   "main": "./dist/index.cjs",
   "scripts": {