create-zudo-doc 0.2.0 → 0.2.1

package/templates/features/claudeResources/files/src/integrations/claude-resources/generate.ts

@@ -64,7 +64,9 @@ function parseFrontmatter(content: string) {
 }
 
 function escapeTitle(s: string): string {
-  return s.replace(/"/g, '\\"');
+  // Backslashes must be escaped first — the value is embedded in
+  // double-quoted YAML frontmatter where `\d` or `C:\path` is invalid.
+  return s.replace(/\\/g, "\\\\").replace(/"/g, '\\"');
 }
 
 function listFiles(dir: string): string[] {
@@ -93,6 +95,35 @@ generated: true
   fs.writeFileSync(path.join(outputDir, "index.mdx"), mdx);
 }
 
+/**
+ * Writes an unlisted sub-page MDX file (flat file with a custom nested slug).
+ * Used for skill references, scripts, and assets.
+ */
+function writeUnlistedSubPage(
+  outputPath: string,
+  title: string,
+  slug: string,
+  body: string,
+) {
+  fs.writeFileSync(
+    outputPath,
+    `---\ntitle: "${escapeTitle(title)}"\nslug: "${slug}"\nunlisted: true\ngenerated: true\n---\n\n${body}\n`,
+  );
+}
+
+/**
+ * Guards that the given name/slug is not the reserved "index" value.
+ * Throws with a contextual message if it is.
+ */
+function assertNotIndexReserved(
+  nameOrSlug: string,
+  errorMessage: string,
+) {
+  if (nameOrSlug === "index") {
+    throw new Error(errorMessage);
+  }
+}
+
 // ---------------------------------------------------------------------------
 // CLAUDE.md discovery
 // ---------------------------------------------------------------------------
@@ -176,11 +207,10 @@ function generateClaudemdDocs(
 
   const emittedSlugs = new Map<string, string>();
   items.forEach((item, index) => {
-    if (item.slug === "index") {
-      throw new Error(
-        `claude-resources: "${item.relPath}" maps to the reserved slug "index", which is used for the category metadata file. Rename the directory to resolve the conflict.`,
-      );
-    }
+    assertNotIndexReserved(
+      item.slug,
+      `claude-resources: "${item.relPath}" maps to the reserved slug "index", which is used for the category metadata file. Rename the directory to resolve the conflict.`,
+    );
     const previous = emittedSlugs.get(item.slug);
     if (previous !== undefined) {
       throw new Error(
@@ -232,11 +262,10 @@ function generateCommandsDocs(config: ClaudeResourcesConfig): CommandItem[] {
     if (!parsed) continue;
 
     const name = file.replace(/\.md$/, "");
-    if (name === "index") {
-      throw new Error(
-        `claude-resources: ".claude/commands/index.md" uses the reserved name "index", which is used for the category metadata file. Rename the command file to resolve the conflict.`,
-      );
-    }
+    assertNotIndexReserved(
+      name,
+      `claude-resources: ".claude/commands/index.md" uses the reserved name "index", which is used for the category metadata file. Rename the command file to resolve the conflict.`,
+    );
     const description = (parsed.data.description as string) || "";
 
     items.push({ name, description });
@@ -357,11 +386,10 @@ function generateSkillsDocs(config: ClaudeResourcesConfig): SkillItem[] {
   const items: SkillItem[] = [];
 
   for (const dir of dirs) {
-    if (dir === "index") {
-      throw new Error(
-        `claude-resources: skill directory ".claude/skills/index/" uses the reserved name "index", which is used for the category metadata file. Rename the skill directory to resolve the conflict.`,
-      );
-    }
+    assertNotIndexReserved(
+      dir,
+      `claude-resources: skill directory ".claude/skills/index/" uses the reserved name "index", which is used for the category metadata file. Rename the skill directory to resolve the conflict.`,
+    );
     const content = fs.readFileSync(
       path.join(skillsDir, dir, "SKILL.md"),
       "utf8",
@@ -445,46 +473,43 @@ ${body}`;
     const skillSlugBase = `claude-skills/${dir}`;
 
     for (const ref of references) {
-      const subSlug = `${skillSlugBase}/ref-${ref.name}`;
-      const refMdx = `---
-title: "${escapeTitle(ref.title)}"
-slug: "${subSlug}"
-unlisted: true
-generated: true
----
-
-${escapeForMdx(ref.content.trim())}
-`;
-      fs.writeFileSync(path.join(outputDir, `${dir}--ref-${ref.name}.mdx`), refMdx);
+      writeUnlistedSubPage(
+        path.join(outputDir, `${dir}--ref-${ref.name}.mdx`),
+        ref.title,
+        `${skillSlugBase}/ref-${ref.name}`,
+        escapeForMdx(ref.content.trim()),
+      );
     }
 
     for (const f of scriptFiles.filter((s) => s.endsWith(".md"))) {
       const slug = f.replace(/\.md$/, "");
-      const subSlug = `${skillSlugBase}/script-${slug}`;
       const raw = fs.readFileSync(
         path.join(skillsDir, dir, "scripts", f),
         "utf8",
       );
       const h1Match = raw.match(/^#\s+(.+)$/m);
       const title = h1Match ? h1Match[1] : slug;
-      fs.writeFileSync(
+      writeUnlistedSubPage(
         path.join(outputDir, `${dir}--script-${slug}.mdx`),
-        `---\ntitle: "${escapeTitle(title)}"\nslug: "${subSlug}"\nunlisted: true\ngenerated: true\n---\n\n${escapeForMdx(raw.trim())}\n`,
+        title,
+        `${skillSlugBase}/script-${slug}`,
+        escapeForMdx(raw.trim()),
       );
     }
 
     for (const f of assetFiles.filter((a) => a.endsWith(".md"))) {
       const slug = f.replace(/\.md$/, "");
-      const subSlug = `${skillSlugBase}/asset-${slug}`;
       const raw = fs.readFileSync(
         path.join(skillsDir, dir, "assets", f),
         "utf8",
       );
       const h1Match = raw.match(/^#\s+(.+)$/m);
       const title = h1Match ? h1Match[1] : slug;
-      fs.writeFileSync(
+      writeUnlistedSubPage(
         path.join(outputDir, `${dir}--asset-${slug}.mdx`),
-        `---\ntitle: "${escapeTitle(title)}"\nslug: "${subSlug}"\nunlisted: true\ngenerated: true\n---\n\n${escapeForMdx(raw.trim())}\n`,
+        title,
+        `${skillSlugBase}/asset-${slug}`,
+        escapeForMdx(raw.trim()),
       );
     }
   }
@@ -522,11 +547,10 @@ function generateAgentsDocs(config: ClaudeResourcesConfig): AgentItem[] {
     const description = (parsed.data.description as string) || "";
     const model = (parsed.data.model as string) || "";
     const fileSlug = file.replace(/\.md$/, "");
-    if (fileSlug === "index") {
-      throw new Error(
-        `claude-resources: ".claude/agents/index.md" uses the reserved name "index", which is used for the category metadata file. Rename the agent file to resolve the conflict.`,
-      );
-    }
+    assertNotIndexReserved(
+      fileSlug,
+      `claude-resources: ".claude/agents/index.md" uses the reserved name "index", which is used for the category metadata file. Rename the agent file to resolve the conflict.`,
+    );
 
     items.push({ name, file: fileSlug, description, model });
 

package/templates/features/designTokenPanel/files/src/config/design-token-panel-config.ts

@@ -6,11 +6,12 @@
  *
  * Type notes:
  * - zdtp's `ColorScheme` requires a `shikiTheme: string` field that is not
- *   present in zudo-doc's local `ColorScheme` type or data. The cast below
- *   (`as unknown as Record<string, ZdtpColorScheme>`) is intentional: zdtp
- *   uses `shikiTheme` only for the code-block preview inside the panel; when
- *   absent at runtime it falls back to `colorExtras.defaultShikiTheme`. No
- *   user-visible regression results from the missing field.
+ *   present in this project's local `ColorScheme` type or data (zdtp uses it
+ *   only for the panel's client-side code-block preview). Rather than an unsafe
+ *   `as unknown as Record<string, ZdtpColorScheme>` double-cast, every local
+ *   scheme map is run through `toZdtpColorSchemes()` below, which supplies
+ *   `DEFAULT_SHIKI_THEME` as the fallback so the result satisfies zdtp's
+ *   required-field shape with an ordinary type-checked assignment.
  */
 
 import type {
@@ -28,6 +29,7 @@ import {
   SIZE_TOKENS,
 } from "./design-tokens-manifest";
 import { colorSchemes } from "./color-schemes";
+import type { ColorScheme as LocalColorScheme } from "./color-schemes";
 import { SEMANTIC_DEFAULTS, SEMANTIC_CSS_NAMES } from "./color-scheme-utils";
 import { settings } from "./settings";
 import { DESIGN_TOKEN_SCHEMA } from "@takazudo/zudo-doc/theme";
@@ -49,6 +51,30 @@ const BASE_DEFAULTS = {
  */
 const DEFAULT_SHIKI_THEME = "github-dark";
 
+/**
+ * Normalize this project's local `ColorScheme` records into zdtp's
+ * `ColorScheme` shape. zdtp's type requires `shikiTheme: string`; the local
+ * scheme type makes it optional. This helper fills `DEFAULT_SHIKI_THEME` only
+ * when a scheme doesn't declare its own, so the result is assignable to
+ * `Record<string, ZdtpColorScheme>` via an ordinary type-checked assignment —
+ * replacing the previous `as unknown as` double-cast that bypassed every field
+ * check. Tracked upstream at Takazudo/zudo-design-token-panel#342 (shikiTheme
+ * should be optional in zdtp's `ColorScheme` type); drop this helper once that
+ * lands.
+ */
+function toZdtpColorSchemes(
+  schemes: Record<string, LocalColorScheme>,
+): Record<string, ZdtpColorScheme> {
+  const normalized: Record<string, ZdtpColorScheme> = {};
+  for (const [name, scheme] of Object.entries(schemes)) {
+    normalized[name] = {
+      ...scheme,
+      shikiTheme: scheme.shikiTheme ?? DEFAULT_SHIKI_THEME,
+    };
+  }
+  return normalized;
+}
+
 /**
  * Initial palette taken from the configured active scheme.
  */
@@ -178,9 +204,9 @@ const COLOR_EXTRAS: ColorClusterExtras = {
   },
   baseDefaults: BASE_DEFAULTS,
   defaultShikiTheme: DEFAULT_SHIKI_THEME,
-  // Local ColorScheme lacks shikiTheme; cast is safe — zdtp falls back to
-  // defaultShikiTheme when shikiTheme is absent at runtime.
-  colorSchemes: colorSchemes as unknown as Record<string, ZdtpColorScheme>,
+  // toZdtpColorSchemes fills the fallback only for schemes without their own
+  // shikiTheme, so this is a type-checked assignment rather than an unsafe cast.
+  colorSchemes: toZdtpColorSchemes(colorSchemes),
   panelSettings: {
     colorScheme: settings.colorScheme,
     // colorMode: strip off respectPrefersColorScheme (not in zdtp's shape).

package/templates/features/docHistory/files/plugins/doc-history-plugin.mjs

@@ -1,12 +1,12 @@
+// @ts-check
 // zfb plugin module: doc-history.
 //
 // Wires three lifecycle hooks for the doc-history integration:
 //
 //   preBuild  — emits `.zfb/doc-history-meta.json` (per-page git metadata
-//               consumed at bundle time). Uses the tsx-e shim because the
-//               runner imports Node-only modules (`fs`, `child_process`) and
-//               TypeScript source that the plain-Node plugin host cannot load
-//               directly. Honours `SKIP_DOC_HISTORY=1` via `env: process.env`.
+//               consumed at bundle time). Calls `runDocHistoryMetaStep`
+//               directly; honours `SKIP_DOC_HISTORY=1` via the runner's
+//               own env check.
 //
 //   postBuild — invokes `runDocHistoryPostBuild` to write
 //               `<outDir>/doc-history/<slug>.json` files. Skipped by default
@@ -20,69 +20,45 @@
 // Inline functions are not supported by zfb's plugin runtime — see
 // `@takazudo/zfb/plugins` source comment. Plugins must be authored as
 // standalone modules and referenced from `zfb.config.ts` by `name`.
-//
-// The legacy `scripts/zfb-{pre,post}build.mjs` npm-script glue and
-// `scripts/dev-sidecar.mjs` stay in place during the merge window;
-// T6 retires them once all lifecycle epics land.
 
-import { spawnSync } from "node:child_process";
-import { fileURLToPath } from "node:url";
-import { dirname, resolve } from "node:path";
+/** @import { ZfbBuildHookContext, ZfbDevMiddlewareContext, ZfbPlugin } from "@takazudo/zfb/plugins" */
 
+import { runDocHistoryMetaStep } from "@takazudo/zudo-doc/integrations/doc-history";
 import { runDocHistoryPostBuild } from "@takazudo/zudo-doc/integrations/doc-history";
 import { createDocHistoryDevMiddleware } from "@takazudo/zudo-doc/integrations/doc-history";
 import { connectToZfbHandler } from "./connect-adapter.mjs";
 
-const HERE = dirname(fileURLToPath(import.meta.url));
-// tsx is a workspace dep; resolving the binary explicitly avoids PATH
-// dependency — the plugin host is spawned by zfb without the user's shell profile.
-const TSX_BIN = resolve(HERE, "..", "node_modules", ".bin", "tsx");
-
+/** @type {ZfbPlugin} */
 export default {
   name: "doc-history",
 
+  /** @param {ZfbBuildHookContext} ctx */
   async preBuild(ctx) {
     const { docsDir, locales } = ctx.options;
-    // Serialize options as JSON for the tsx-e inline script. The runner
-    // (`runDocHistoryMetaStep`) honours SKIP_DOC_HISTORY=1 internally;
-    // passing `env: process.env` propagates the flag to the child process.
-    const optsJson = JSON.stringify({
+    await runDocHistoryMetaStep({
       projectRoot: ctx.projectRoot,
       docsDir: typeof docsDir === "string" ? docsDir : "src/content/docs",
-      locales: locales ?? {},
-    });
-    const script = `
-      (async () => {
-        const { runDocHistoryMetaStep } = await import("@takazudo/zudo-doc/integrations/doc-history");
-        const opts = ${optsJson};
-        await runDocHistoryMetaStep(opts);
-      })().catch((err) => {
-        process.stderr.write(err && err.stack ? err.stack : String(err));
-        process.exit(1);
-      });
-    `;
-    const result = spawnSync(TSX_BIN, ["-e", script], {
-      cwd: ctx.projectRoot,
-      stdio: "inherit",
-      env: process.env,
+      locales:
+        locales != null
+          ? /** @type {Record<string,{dir:string}>} */ (locales)
+          : undefined,
     });
-    if (result.status !== 0) {
-      throw new Error(
-        `doc-history-meta preBuild failed (exit ${result.status})`,
-      );
-    }
   },
 
+  /** @param {ZfbBuildHookContext} ctx */
   async postBuild(ctx) {
-    const { docsDir, locales } = ctx.options;
     await runDocHistoryPostBuild(
-      { docsDir, locales },
+      /** @type {import("@takazudo/zudo-doc/integrations/doc-history").DocHistoryOptions} */ (/** @type {unknown} */ (ctx.options)),
       { outDir: ctx.outDir, logger: ctx.logger },
     );
   },
 
+  /** @param {ZfbDevMiddlewareContext} ctx */
   devMiddleware(ctx) {
-    const middleware = createDocHistoryDevMiddleware(ctx.options, ctx.logger);
+    const middleware = createDocHistoryDevMiddleware(
+      /** @type {import("@takazudo/zudo-doc/integrations/doc-history").DocHistoryOptions} */ (/** @type {unknown} */ (ctx.options)),
+      ctx.logger,
+    );
     // zfb's `register(path, handler)` matches against the FULL request
     // URL (no base-stripping). For a non-root base (e.g. "/my-docs/"),
     // requests arrive as `/my-docs/doc-history/foo.json`, so we register
@@ -90,11 +66,17 @@ export default {
     // and the route is `/doc-history` as expected. The v2 middleware
     // itself is base-tolerant (matches via `url.includes("/doc-history/")`)
     // and slices from `/doc-history/` onward when proxying upstream.
-    const basePrefix = stripTrailingSlash(ctx.options.base ?? "");
+    const basePrefix = stripTrailingSlash(
+      typeof ctx.options["base"] === "string" ? ctx.options["base"] : "",
+    );
     ctx.register(`${basePrefix}/doc-history`, connectToZfbHandler(middleware));
   },
 };
 
+/**
+ * @param {string} s
+ * @returns {string}
+ */
 function stripTrailingSlash(s) {
   if (typeof s !== "string" || s.length === 0) return "";
   return s.endsWith("/") ? s.slice(0, -1) : s;

package/templates/features/docHistory/files/src/components/doc-history.tsx

@@ -1,5 +1,5 @@
 import { useState, useEffect, useCallback, useMemo, useRef } from "preact/compat";
-import { diffLines } from "diff";
+import type { Change } from "diff";
 import type { DocHistoryData, DocHistoryEntry } from "@/types/doc-history";
 import { SmartBreak } from "@/utils/smart-break";
 
@@ -104,8 +104,10 @@ interface DiffRow {
   type: "context" | "removed" | "added" | "changed";
 }
 
+type DiffChanges = Change[];
+
 function buildSideBySideRows(
-  changes: ReturnType<typeof diffLines>,
+  changes: DiffChanges,
 ): DiffRow[] {
   const rows: DiffRow[] = [];
   let leftNum = 0;
@@ -177,11 +179,24 @@ function DiffViewer({
   onBack: () => void;
   showBackButton: boolean;
 }) {
-  const changes = useMemo(
-    () => diffLines(selection.older.content, selection.newer.content),
-    [selection.older.content, selection.newer.content],
+  const [changes, setChanges] = useState<DiffChanges | null>(null);
+
+  useEffect(() => {
+    let cancelled = false;
+    // Lazy-load diff — only needed after History → Compare. This keeps the
+    // module out of the eager islands bundle.
+    import("diff").then(({ diffLines }) => {
+      if (!cancelled) {
+        setChanges(diffLines(selection.older.content, selection.newer.content));
+      }
+    });
+    return () => { cancelled = true; };
+  }, [selection.older.content, selection.newer.content]);
+
+  const rows = useMemo(
+    () => (changes ? buildSideBySideRows(changes) : []),
+    [changes],
   );
-  const rows = useMemo(() => buildSideBySideRows(changes), [changes]);
 
   return (
     <div className="flex flex-col h-full">
@@ -207,8 +222,13 @@ function DiffViewer({
         </div>
       </div>
 
-      {/* Side-by-side diff */}
-      <div className="flex-1 overflow-auto">
+      {/* Side-by-side diff — shows a loading state while the diff module lazy-loads */}
+      {!changes && (
+        <div className="flex-1 flex items-center justify-center py-vsp-xl">
+          <p className="text-small text-muted">Loading diff…</p>
+        </div>
+      )}
+      <div className={`flex-1 overflow-auto${!changes ? " hidden" : ""}`}>
         <table className="w-full border-collapse" style={{ tableLayout: "fixed" }}>
           <colgroup>
             <col style={{ width: "2.5rem" }} />

package/templates/features/docTags/files/pages/[locale]/docs/tags/[tag].tsx

@@ -11,29 +11,14 @@
 //   params: { locale: string; tag: string }
 //   props:  { tagInfo: TagInfo }
 //
+// Tag collection + rendering are shared with the default-locale route via
+// pages/lib/_tag-pages.tsx (#2010) — this file owns only the param shape.
 // Fallback strategy: see pages/lib/locale-merge.ts for full details.
-//   Locale docs take priority; base docs fill in missing slugs.
-//   Build tag map; emit one route per (locale, tag) pair.
 
-import { mergeLocaleDocs } from "../../../lib/locale-merge";
-import { loadDocs } from "../../../_data";
-import { collectTags } from "@/utils/tags";
-import type { TagInfo } from "@/utils/tags";
-import { toRouteSlug } from "@/utils/slug";
-import { t } from "@/config/i18n";
-import { withBase, docsUrl } from "@/utils/base";
 import { settings } from "@/config/settings";
-import { DocLayoutWithDefaults } from "@takazudo/zudo-doc/doclayout";
-import { Breadcrumb } from "@takazudo/zudo-doc/breadcrumb";
-import type { BreadcrumbItem } from "@takazudo/zudo-doc/breadcrumb";
-import { DocCardGrid } from "@takazudo/zudo-doc/nav-indexing";
+import type { TagInfo } from "@/utils/tags";
 import type { JSX } from "preact";
-import { FooterWithDefaults } from "../../../lib/_footer-with-defaults";
-import { HeaderWithDefaults } from "../../../lib/_header-with-defaults";
-import { HeadWithDefaults } from "../../../lib/_head-with-defaults";
-import { composeMetaTitle } from "../../../lib/_compose-meta-title";
-import { DocHistoryArea } from "../../../lib/_doc-history-area";
-import { BodyEndIslands } from "../../../lib/_body-end-islands";
+import { collectTagMapForLocale, TagDetailPageView } from "../../../lib/_tag-pages";
 
 export const frontmatter = { title: "Tag" };
 
@@ -48,18 +33,7 @@ export function paths(): Array<{
   }> = [];
 
   for (const locale of Object.keys(settings.locales)) {
-    const { docs: mergedDocs } = mergeLocaleDocs({
-      baseDocs: loadDocs("docs").filter((d) => !d.data.draft),
-      localeDocs: loadDocs(`docs-${locale}`).filter((d) => !d.data.draft),
-      applyDefaultLocaleOnlyFilter: true,
-    });
-    // category_no_page index files build no route — drop them AFTER the merge
-    // so a locale override carrying the flag first wins the merge (suppressing
-    // the base doc); pre-merge filtering would drop it from localeSlugSet and
-    // the unflagged base doc would resurface as a card linking to a locale
-    // route the docs route never builds.
-    const docs = mergedDocs.filter((d) => !d.data.category_no_page);
-    const tagMap = collectTags(docs, (id, data) => data.slug ?? toRouteSlug(id));
+    const tagMap = collectTagMapForLocale(locale);
 
     for (const [tag, tagInfo] of tagMap.entries()) {
       result.push({
@@ -81,47 +55,5 @@ export default function LocaleDocTagPage({
   params,
   tagInfo,
 }: PageProps): JSX.Element {
-  const { locale, tag } = params;
-
-  const countText =
-    tagInfo.count === 1
-      ? t("doc.pageCountSingle", locale).replace("{count}", String(tagInfo.count))
-      : t("doc.pageCount", locale).replace("{count}", String(tagInfo.count));
-
-  const pageTitle = `${t("doc.taggedWith", locale)}: ${tag}`;
-
-  const breadcrumbItems: BreadcrumbItem[] = [
-    { label: "Docs" },
-    {
-      label: t("doc.allTags", locale),
-      href: withBase(`/${locale}/docs/tags`),
-    },
-    { label: tag },
-  ];
-
-  const cardItems = tagInfo.docs.map((doc) => ({
-    href: docsUrl(doc.slug, locale),
-    title: doc.title,
-    description: doc.description,
-  }));
-
-  return (
-    <DocLayoutWithDefaults
-      title={composeMetaTitle(pageTitle)}
-      head={<HeadWithDefaults title={pageTitle} />}
-      lang={locale}
-      noindex={settings.noindex}
-      hideSidebar={true}
-      hideToc={true}
-      headerOverride={<HeaderWithDefaults lang={locale} currentPath={withBase(`/${locale}/docs/tags/${tag}`)} />}
-      breadcrumbOverride={<Breadcrumb items={breadcrumbItems} />}
-      footerOverride={<FooterWithDefaults lang={locale} />}
-      bodyEndComponents={<BodyEndIslands basePath={settings.base ?? "/"} />}
-    >
-      <h1 class="text-heading font-bold mb-vsp-xs">{pageTitle}</h1>
-      <p class="text-muted mb-vsp-lg">{countText}</p>
-      <DocCardGrid ariaLabel={pageTitle} items={cardItems} />
-      <DocHistoryArea slug={`tags/${tag}`} locale={locale} />
-    </DocLayoutWithDefaults>
-  );
+  return <TagDetailPageView locale={params.locale} tag={params.tag} tagInfo={tagInfo} />;
 }

package/templates/features/docTags/files/pages/[locale]/docs/tags/index.tsx

@@ -3,36 +3,21 @@
 // Page module for the locale-prefixed "All Tags" index route.
 //
 // Non-default-locale "All Tags" index page. paths() emits one route per
-// locale defined in settings.locales (English has no /en prefix — it is
-// handled by pages/docs/tags/index.tsx). The component recomputes the tag
+// locale defined in settings.locales (the default locale has no prefix — it
+// is handled by pages/docs/tags/index.tsx). The component recomputes the tag
 // map at render time using a locale-doc + base-doc fallback strategy
 // (see pages/lib/locale-merge.ts for the merge logic).
 //
-// Fallback strategy (locale first, base as fill): see pages/lib/locale-merge.ts
+// Tag collection + rendering are shared with the default-locale route via
+// pages/lib/_tag-pages.tsx (#2010).
 //
 // paths() contract (zfb ADR-004 — synchronous):
 //   params: { locale: string }
 //   props:  (none — tag map computed at render time)
 
-import { mergeLocaleDocs } from "../../../lib/locale-merge";
-import { loadDocs } from "../../../_data";
-import { collectTags } from "@/utils/tags";
-import { toRouteSlug } from "@/utils/slug";
-import { t } from "@/config/i18n";
-import { withBase } from "@/utils/base";
 import { settings } from "@/config/settings";
-import { DocLayoutWithDefaults } from "@takazudo/zudo-doc/doclayout";
-import { Breadcrumb } from "@takazudo/zudo-doc/breadcrumb";
-import type { BreadcrumbItem } from "@takazudo/zudo-doc/breadcrumb";
-import { TagNav } from "@takazudo/zudo-doc/nav-indexing";
-import type { TagItem, TagNavLabels } from "@takazudo/zudo-doc/nav-indexing";
 import type { JSX } from "preact";
-import { FooterWithDefaults } from "../../../lib/_footer-with-defaults";
-import { HeaderWithDefaults } from "../../../lib/_header-with-defaults";
-import { HeadWithDefaults } from "../../../lib/_head-with-defaults";
-import { composeMetaTitle } from "../../../lib/_compose-meta-title";
-import { DocHistoryArea } from "../../../lib/_doc-history-area";
-import { BodyEndIslands } from "../../../lib/_body-end-islands";
+import { TagsIndexPageView } from "../../../lib/_tag-pages";
 
 export const frontmatter = { title: "All Tags" };
 
@@ -50,61 +35,5 @@ interface PageProps {
 export default function LocaleTagsIndexPage({
   params,
 }: PageProps): JSX.Element {
-  const { locale } = params;
-  const pageTitle = t("doc.allTags", locale);
-
-  const { docs: mergedDocs } = mergeLocaleDocs({
-    baseDocs: loadDocs("docs").filter((d) => !d.data.draft),
-    localeDocs: loadDocs(`docs-${locale}`).filter((d) => !d.data.draft),
-    applyDefaultLocaleOnlyFilter: true,
-  });
-  // category_no_page index files build no route — drop them AFTER the merge
-  // so a locale override carrying the flag first wins the merge (suppressing
-  // the base doc); pre-merge filtering would drop it from localeSlugSet and
-  // the unflagged base doc would resurface as a card linking to a locale
-  // route the docs route never builds.
-  const docs = mergedDocs.filter((d) => !d.data.category_no_page);
-  const tagMap = collectTags(docs, (id, data) => data.slug ?? toRouteSlug(id));
-
-  const labels: TagNavLabels = {
-    tags: t("doc.tags", locale),
-    taggedWith: t("doc.taggedWith", locale),
-  };
-
-  // Sort alphabetically using the page locale — matches documented tag-nav sort order.
-  const tags: TagItem[] = [...tagMap.values()]
-    .sort((a, b) => a.tag.localeCompare(b.tag, locale))
-    .map((info) => ({
-      tag: info.tag,
-      count: info.count,
-      href: withBase(`/${locale}/docs/tags/${info.tag}`),
-    }));
-
-  const breadcrumbItems: BreadcrumbItem[] = [
-    { label: "Docs" },
-    { label: pageTitle },
-  ];
-
-  return (
-    <DocLayoutWithDefaults
-      title={composeMetaTitle(pageTitle)}
-      head={<HeadWithDefaults title={pageTitle} />}
-      lang={locale}
-      noindex={settings.noindex}
-      hideSidebar={true}
-      hideToc={true}
-      headerOverride={<HeaderWithDefaults lang={locale} currentPath={withBase(`/${locale}/docs/tags`)} />}
-      breadcrumbOverride={<Breadcrumb items={breadcrumbItems} />}
-      footerOverride={<FooterWithDefaults lang={locale} />}
-      bodyEndComponents={<BodyEndIslands basePath={settings.base ?? "/"} />}
-    >
-      <h1 class="text-heading font-bold mb-vsp-lg">{pageTitle}</h1>
-      {!settings.docTags || tags.length === 0 ? (
-        <p class="text-muted">{t("doc.noTags", locale)}</p>
-      ) : (
-        <TagNav variant="all" tags={tags} labels={labels} />
-      )}
-      <DocHistoryArea slug="tags" locale={locale} />
-    </DocLayoutWithDefaults>
-  );
+  return <TagsIndexPageView locale={params.locale} />;
 }