@riebeckite/plugin-diagnostics 0.0.15 → 0.0.17

package/README.md

@@ -32,7 +32,7 @@ export default defineConfig({
 
 - `addDiagnostics` — runs the analysis at build time; results go to
   `manifest.diagnostics`
-- `onBuildEnd` — with `failOnError: true`, throws `DiagnosticsFailure` when
+- `buildEnd` — with `failOnError: true`, throws `DiagnosticsFailure` when
   error-level diagnostics exist
 
 ## Checks
@@ -48,8 +48,8 @@ export default defineConfig({
 | `publish-conflict` | `warning` | `publish: true` combined with `draft: true` / `private: true` |
 | `duplicate-title` | `warning` | Multiple published notes share a title within the same language |
 | `slug-collision` | `error` | Slugs collide case-insensitively |
-| `duplicate-content-id` | `error` | Published notes share a stable content ID (`id`/`uid`) |
-| `invalid-content-id` | `error` | `id`/`uid` frontmatter violates the stable content ID contract |
+| `duplicate-content-id` | `error` | Published notes share a stable content ID (`id`) |
+| `invalid-content-id` | `error` | `id` frontmatter violates the stable content ID contract |
 | `excluded-public` | `warning` | Excluded note is marked `publish: true` |
 | `publish-boundary` | `warning` | Published content links to or embeds non-published content |
 | `analytics-untracked` | `info` | Published note has no stable `id` and the analytics plugin will not track it (enabled with `reportAnalyticsCoverage`, or automatically when the config enables the analytics plugin) |
@@ -160,4 +160,3 @@ assertNoErrors(report);
 ## See also
 
 - [Plugin guide](../../../docs/en/docs/reference/plugin-api.md)
-

package/README_ja.md

@@ -24,6 +24,8 @@ export default defineConfig({
 });
 ```
 
+`failOnError: true` を指定すると、`buildEnd` で error レベルの Diagnostics が残っている場合に `DiagnosticsFailure` を送出します。
+
 ## 検出する問題
 
 | コード | 既定の重要度 | 内容 |
@@ -37,8 +39,8 @@ export default defineConfig({
 | `publish-conflict` | warning | `publish: true` と `draft: true` または `private: true` が同居している |
 | `duplicate-title` | warning | 同じ言語の公開ノート同士でタイトルが重複している |
 | `slug-collision` | error | 大文字・小文字を区別しない slug が衝突している |
-| `duplicate-content-id` | error | 公開ノート同士で安定コンテンツ ID（`id`/`uid`）が重複している |
-| `invalid-content-id` | error | `id`/`uid` の frontmatter が安定コンテンツ ID の要件を満たしていない |
+| `duplicate-content-id` | error | 公開ノート同士で安定コンテンツ ID（`id`）が重複している |
+| `invalid-content-id` | error | `id` の frontmatter が安定コンテンツ ID の要件を満たしていない |
 | `excluded-public` | warning | 除外されたノートに `publish: true` が指定されている |
 | `publish-boundary` | warning | 公開コンテンツから非公開コンテンツへリンク・埋め込みしている |
 | `analytics-untracked` | info | 安定 `id` のない公開ノートは analytics プラグインで計測されない（`reportAnalyticsCoverage` で有効化。設定で analytics プラグインが有効な場合は自動で有効化） |
@@ -116,4 +118,3 @@ assertNoErrors(report);
 ## 関連資料
 
 - [プラグインシステム](../../../docs/ja/docs/reference/plugin-api.md)
-

package/dist/bin.js

@@ -143,7 +143,7 @@ function checkAnalyticsCoverage(source, options, diagnostics) {
   }
 }
 function hasAnalyticsId(note) {
-  return hasField(note.fm.values, "id") || hasField(note.fm.values, "uid");
+  return hasField(note.fm.values, "id");
 }
 
 // src/checks/assets.ts
@@ -180,7 +180,7 @@ function checkContentIdIntegrity(source, options, diagnostics) {
         "invalid-content-id",
         `invalid content id frontmatter: ${toErrorMessage(error)}`,
         void 0,
-        'set a unique, trimmed "id" value and remove a conflicting "uid" value'
+        'set a unique, trimmed "id" value'
       );
       continue;
     }
@@ -200,7 +200,7 @@ function checkContentIdIntegrity(source, options, diagnostics) {
         "duplicate-content-id",
         `content id "${id}" is used by multiple published notes: ${slugs}`,
         id,
-        'give each note a unique "id" (or "uid") frontmatter value'
+        'give each note a unique "id" frontmatter value'
       );
     }
   }

package/dist/index.js

@@ -139,7 +139,7 @@ function checkAnalyticsCoverage(source, options, diagnostics2) {
   }
 }
 function hasAnalyticsId(note) {
-  return hasField(note.fm.values, "id") || hasField(note.fm.values, "uid");
+  return hasField(note.fm.values, "id");
 }
 
 // src/checks/assets.ts
@@ -176,7 +176,7 @@ function checkContentIdIntegrity(source, options, diagnostics2) {
         "invalid-content-id",
         `invalid content id frontmatter: ${toErrorMessage(error)}`,
         void 0,
-        'set a unique, trimmed "id" value and remove a conflicting "uid" value'
+        'set a unique, trimmed "id" value'
       );
       continue;
     }
@@ -196,7 +196,7 @@ function checkContentIdIntegrity(source, options, diagnostics2) {
         "duplicate-content-id",
         `content id "${id}" is used by multiple published notes: ${slugs}`,
         id,
-        'give each note a unique "id" (or "uid") frontmatter value'
+        'give each note a unique "id" frontmatter value'
       );
     }
   }
@@ -1010,7 +1010,7 @@ function checkSiteIntegrity(manifest, options = {}) {
     checkResolvedLinks(entry, publicSlugs, assetPaths, diagnostics2, options);
     checkHtmlReferences(entry, publicRoutes, assetPaths, diagnostics2, options);
   }
-  checkRedirectTargets(manifest, publicRoutes, diagnostics2, options);
+  checkRedirectTargets(manifest, diagnostics2, options);
   checkRedirectCycles(manifest, diagnostics2, options);
   checkAmbiguousFolderPageOwners(manifest, diagnostics2, options);
   return diagnostics2;
@@ -1049,13 +1049,6 @@ function buildRouteOwners(manifest, diagnostics2, options) {
       });
     }
   }
-  for (const output of manifest.generatedOutputs ?? []) {
-    reserveRoute(owners, diagnostics2, options, {
-      path: normalizeRoutePath(output.path),
-      slug: "(generated-output)",
-      kind: "output"
-    });
-  }
   return owners;
 }
 function reserveRoute(owners, diagnostics2, options, next) {
@@ -1072,7 +1065,7 @@ function reserveRoute(owners, diagnostics2, options, next) {
       slug: next.slug.startsWith("(") ? void 0 : next.slug,
       target: next.path,
       message: `Public path "${next.path}" is claimed by both ${describeOwner(previous)} and ${describeOwner(next)}.`,
-      suggestion: "Change one permalink, alias, redirect, page path, or generated output path.",
+      suggestion: "Change one permalink, alias, redirect, or page path.",
       meta: { previous, next }
     })
   );
@@ -1161,11 +1154,12 @@ function checkHtmlReferences(entry, publicRoutes, assetPaths, diagnostics2, opti
     );
   }
 }
-function checkRedirectTargets(manifest, publicRoutes, diagnostics2, options) {
-  for (const [path, redirect] of manifest.publicRedirects) {
+function checkRedirectTargets(manifest, diagnostics2, options) {
+  for (const [path, redirect] of manifest.redirects) {
     const owner = manifest.bySlug.get(redirect.slug);
     const target = owner ? normalizeRoutePath(owner.publicLocation.permalink) : null;
-    if (target && publicRoutes.has(target)) continue;
+    if (target && manifest.byRoutablePermalink.get(target)?.slug === redirect.slug)
+      continue;
     diagnostics2.push(
       diagnostic(options, {
         code: "content-integrity:redirect-target-missing",
@@ -1224,8 +1218,6 @@ function buildAssetPaths(manifest) {
   }
   for (const asset of manifest.assets ?? [])
     paths.add(asset.path.replace(/^\/+/, ""));
-  for (const output of manifest.generatedOutputs ?? [])
-    paths.add(output.path.replace(/^\/+/, ""));
   return paths;
 }
 function extractAttributes(html, attribute) {
@@ -1358,7 +1350,7 @@ function diagnostics(options = {}) {
     onManifestCreated: ({ manifest, diagnostics: diagnostics2 }) => {
       diagnostics2.push(...checkSiteIntegrity(manifest, options));
     },
-    onBuildEnd: ({ manifest }) => {
+    buildEnd: ({ manifest }) => {
       if (!options.failOnError) return void 0;
       const errors = manifest.diagnostics.filter(
         (diagnostic2) => diagnostic2.pluginName === PLUGIN_NAME3 && diagnostic2.severity === "error"

package/dist/src/checks/analytics.d.ts

@@ -5,8 +5,8 @@ import { type NormalizedOptions } from "./shared.js";
  * Reports published content that the analytics plugin will not track.
  *
  * The analytics browser initializer emits a page view only for entries that
- * carry Core's source-authored stable content ID (`id`, or the compatible
- * `uid`). Content without that ID builds and publishes normally but is silently
+ * carry Core's source-authored stable content ID (`id`). Content without that
+ * ID builds and publishes normally but is silently
  * absent from analytics. This check surfaces that gap as factual, read-only
  * `info` findings; the analytics plugin itself never autofixes content.
  */

package/dist/src/checks/content_identity.d.ts

@@ -5,9 +5,8 @@ import { type NormalizedOptions } from "./shared.js";
  * Checks stable content ID integrity.
  *
  * Stable content IDs come from the optional source-authored `id` frontmatter
- * field (`uid` is accepted for legacy content). They are the identity key for
- * per-content consumers such as analytics page views, so duplicates silently
- * merge per-content metrics and conflicting declarations break the build.
+ * field. They are the identity key for per-content consumers such as analytics
+ * page views, so duplicates silently merge per-content metrics.
  * This check is intentionally generic: it validates the content-model invariant
  * without knowing which plugin consumes the ID.
  */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@riebeckite/plugin-diagnostics",
-  "version": "0.0.15",
+  "version": "0.0.17",
   "license": "Apache-2.0",
   "type": "module",
   "main": "./dist/index.js",
@@ -20,7 +20,7 @@
     "remark-parse": "^11.0.0",
     "unified": "^11.0.5",
     "unist-util-visit": "^5.1.0",
-    "@riebeckite/core": "0.0.15"
+    "@riebeckite/core": "0.0.17"
   },
   "devDependencies": {
     "@types/mdast": "^4.0.4",