scribe-cms 0.0.9 → 0.0.11

package/dist/cli/index.js

@@ -5,6 +5,9 @@ import fs2 from 'fs';
 import Database from 'better-sqlite3';
 import { z } from 'zod';
 import matter from 'gray-matter';
+import remarkMdx from 'remark-mdx';
+import remarkParse from 'remark-parse';
+import { unified } from 'unified';
 import { createJiti } from 'jiti';
 import { createHash } from 'crypto';
 import { GoogleGenAI } from '@google/genai';
@@ -712,6 +715,140 @@ function bulkLoadTranslations(db) {
   return db.prepare(`SELECT * FROM translations ORDER BY content_type, en_slug, locale`).all();
 }
 
+// src/translate/validate-mdx-body.ts
+init_esm_shims();
+
+// src/translate/normalize-mdx-body.ts
+init_esm_shims();
+function needsMdxEscapeNormalization(body) {
+  return /\\n[ \t/>]|<[\w.-][^>]*\\n/.test(body);
+}
+function normalizeTranslatedMdxBody(body) {
+  if (!needsMdxEscapeNormalization(body)) {
+    return { body, adjusted: false };
+  }
+  const normalized = body.replace(/\\r\\n/g, "\n").replace(/\\n/g, "\n").replace(/\\t/g, "	").replace(/\\"/g, '"').replace(/\\\\/g, "\\");
+  return { body: normalized, adjusted: normalized !== body };
+}
+
+// src/translate/sanitize-mdx-jsx.ts
+init_esm_shims();
+function sanitizeMdxJsxAttributeQuotes(body) {
+  let adjusted = false;
+  let out = "";
+  let i = 0;
+  while (i < body.length) {
+    if (body[i] !== "<" || !/[\w.]/.test(body[i + 1] ?? "")) {
+      out += body[i];
+      i += 1;
+      continue;
+    }
+    const tagStart = i;
+    i += 1;
+    while (i < body.length && /[\w.-]/.test(body[i] ?? "")) i += 1;
+    const tagName = body.slice(tagStart + 1, i);
+    let tagOut = `<${tagName}`;
+    let tagAdjusted = false;
+    while (i < body.length && body[i] !== ">" && body[i] !== "/") {
+      while (i < body.length && /\s/.test(body[i] ?? "")) {
+        tagOut += body[i];
+        i += 1;
+      }
+      if (i >= body.length || body[i] === ">" || body[i] === "/") break;
+      const attrStart = i;
+      while (i < body.length && /[\w:.-]/.test(body[i] ?? "")) i += 1;
+      body.slice(attrStart, i);
+      while (i < body.length && /\s/.test(body[i] ?? "")) i += 1;
+      if (body[i] !== "=") {
+        tagOut += body.slice(attrStart, i);
+        continue;
+      }
+      tagOut += body.slice(attrStart, i);
+      tagOut += "=";
+      i += 1;
+      while (i < body.length && /\s/.test(body[i] ?? "")) i += 1;
+      const quote = body[i];
+      if (quote !== '"' && quote !== "'") {
+        continue;
+      }
+      if (quote === "'") {
+        const valStart2 = i + 1;
+        i += 1;
+        while (i < body.length) {
+          if (body[i] === "\\") {
+            i += 2;
+            continue;
+          }
+          if (body[i] === "'") break;
+          i += 1;
+        }
+        tagOut += body.slice(valStart2 - 1, i + 1);
+        i += 1;
+        continue;
+      }
+      const valStart = i + 1;
+      i += 1;
+      let closeIdx = -1;
+      for (let scan = valStart; scan < body.length; scan += 1) {
+        if (body[scan] !== '"') continue;
+        let j = scan + 1;
+        while (j < body.length && /\s/.test(body[j] ?? "")) j += 1;
+        if (j < body.length && (body[j] === ">" || body[j] === "/" || /[a-zA-Z]/.test(body[j] ?? ""))) {
+          closeIdx = scan;
+          break;
+        }
+      }
+      if (closeIdx === -1) {
+        tagOut += body.slice(valStart - 1, i);
+        break;
+      }
+      const value = body.slice(valStart, closeIdx);
+      const hasInternalQuote = value.includes('"');
+      if (hasInternalQuote) {
+        const escaped = value.replace(/\\/g, "\\\\").replace(/'/g, "\\'");
+        tagOut += `'${escaped}'`;
+        tagAdjusted = true;
+      } else {
+        tagOut += `"${value}"`;
+      }
+      i = closeIdx + 1;
+    }
+    if (tagAdjusted) adjusted = true;
+    tagOut += body[i] ?? "";
+    out += tagOut;
+    i += 1;
+  }
+  return { body: out, adjusted };
+}
+
+// src/translate/validate-mdx-body.ts
+var parser = unified().use(remarkParse).use(remarkMdx);
+function prepareTranslatedMdxBody(body) {
+  const normalized = normalizeTranslatedMdxBody(body);
+  const sanitized = sanitizeMdxJsxAttributeQuotes(normalized.body);
+  return {
+    body: sanitized.body,
+    adjusted: normalized.adjusted || sanitized.adjusted
+  };
+}
+function validateMdxBody(body) {
+  try {
+    parser.parse(body);
+    return { ok: true };
+  } catch (error) {
+    const message = error instanceof Error ? error.message : String(error);
+    return { ok: false, error: message };
+  }
+}
+function assertValidTranslatedMdxBody(body) {
+  const prepared = prepareTranslatedMdxBody(body);
+  const validated = validateMdxBody(prepared.body);
+  if (!validated.ok) {
+    throw new Error(`MDX validation failed: ${validated.error}`);
+  }
+  return prepared;
+}
+
 // src/loader/normalize-en.ts
 init_esm_shims();
 function normalizeEnFrontmatter(data) {
@@ -833,7 +970,7 @@ function buildDocumentFromTranslation(row, enDoc, type, config) {
     noindex: seo.noindex,
     canonicalPathOverride: seo.canonicalPathOverride,
     frontmatter,
-    content: row.body
+    content: prepareTranslatedMdxBody(row.body).body
   };
 }
 var DEV_REVALIDATE_MS = 1500;
@@ -1435,7 +1572,7 @@ function validateTypeRedirects(project) {
     }
     for (const entry of loaded.entries) {
       for (const from of entry.fromSlugs) {
-        if (from === entry.toSlug) {
+        if (entry.kind === "same-type" && from === entry.toSlug) {
           issues.push({
             level: "error",
             contentType: type.id,
@@ -1529,16 +1666,17 @@ function validateTypeRedirects(project) {
       if (!targetType?.path) continue;
       for (const from of entry.fromSlugs) {
         if (globalFrom.get(from)?.typeId !== file.contentTypeId) continue;
-        const chainTarget = globalFrom.get(entry.toSlug);
-        if (chainTarget) {
-          issues.push({
-            level: "error",
-            contentType: file.contentTypeId,
-            enSlug: from,
-            field: TYPE_REDIRECTS_FILENAME,
-            message: `Redirect chain detected: "${from}" \u2192 "${entry.toSlug}" \u2192 existing redirect source`
-          });
-        }
+        const targetAsSource = globalFrom.get(entry.toSlug);
+        if (!targetAsSource) continue;
+        const sameSlugCrossType = entry.kind === "cross-type" && entry.fromSlugs.includes(entry.toSlug) && targetAsSource.typeId === file.contentTypeId;
+        if (sameSlugCrossType) continue;
+        issues.push({
+          level: "error",
+          contentType: file.contentTypeId,
+          enSlug: from,
+          field: TYPE_REDIRECTS_FILENAME,
+          message: `Redirect chain detected: "${from}" \u2192 "${entry.toSlug}" \u2192 existing redirect source`
+        });
       }
     }
   }
@@ -1824,15 +1962,27 @@ function validateProject(config) {
             message: issue.message
           });
         }
+        const preparedBody = prepareTranslatedMdxBody(row.body).body;
         for (const issue of validateDocumentAssets(config, {
           contentType: type.id,
           enSlug,
           locale,
           frontmatter: localeFm,
-          body: row.body
+          body: preparedBody
         })) {
           issues.push(issue);
         }
+        const mdxValidation = validateMdxBody(preparedBody);
+        if (!mdxValidation.ok) {
+          issues.push({
+            level: "error",
+            contentType: type.id,
+            enSlug,
+            locale,
+            field: "body",
+            message: `Invalid MDX: ${mdxValidation.error}`
+          });
+        }
       }
     }
     try {
@@ -2200,24 +2350,21 @@ function buildGeminiResponseSchema(schema, slugStrategy) {
 
 // src/translate/resolve-translate-config.ts
 init_esm_shims();
-function slugStrategyRules(slugStrategy, preserveTerms) {
+var BUILTIN_TRANSLATE_RULES = [
+  "Do not translate brand or product names unless the brand has a well-known localized name in the target market (rare). Keep the original spelling and capitalization.",
+  "Return the MDX body with real line breaks; do not use JSON escape sequences like \\n or \\t in the body string.",
+  'In JSX attributes (e.g. FaqItem question="..."), use single quotes when the value contains double-quote characters (e.g. Hebrew \u05D3\u05D5\u05D0"\u05DC), or escape them as \\".'
+];
+function slugStrategyRules(slugStrategy) {
   if (slugStrategy === "localized") {
-    const rules = [
+    return [
       "Provide a URL slug in JSON field `slug` that is Localized into the target language.",
       "Base the slug on the meaning of the translated title \u2014 do NOT reuse the English slug words.",
       "Slug MUST be ASCII only: a-z, 0-9, hyphens. No uppercase, accents, underscores, or spaces.",
       "For non-Latin languages, write the words in the target language and transliterate them into Latin script (e.g. Russian -> romanized Russian, not English).",
       "Do NOT append locale codes to the slug (e.g. -fr, -he, -zh-cn). Locale routing is handled by the URL prefix, not the slug.",
-      'In JSX attributes (e.g. FaqItem question="..."), use single quotes when the value contains double-quote characters (e.g. Hebrew \u05D3\u05D5\u05D0"\u05DC), or escape them as \\".'
+      "Preserve 4-digit years and proper nouns in slugs."
     ];
-    if (preserveTerms?.length) {
-      rules.push(
-        `Preserve these terms verbatim in slugs (lowercase): ${preserveTerms.join(", ")}.`
-      );
-    } else {
-      rules.push("Preserve 4-digit years and proper nouns in slugs.");
-    }
-    return rules;
   }
   return ["Do NOT output a slug \u2014 slugStrategy is fixed."];
 }
@@ -2230,9 +2377,10 @@ function mergeTranslateConfig(project, type, slugStrategy) {
     promptOverride: type?.prompt ?? project?.prompt,
     context: mergeContext(project, type),
     rules: [
+      ...BUILTIN_TRANSLATE_RULES,
       ...project?.rules ?? [],
       ...type?.rules ?? [],
-      ...slugStrategyRules(slugStrategy, project?.slugPreserveTerms)
+      ...slugStrategyRules(slugStrategy)
     ],
     model: type?.model ?? project?.defaultModel
   };
@@ -2241,96 +2389,6 @@ function resolveTranslateConfig(config, type) {
   return mergeTranslateConfig(config.translate, type.translate, type.slugStrategy);
 }
 
-// src/translate/sanitize-mdx-jsx.ts
-init_esm_shims();
-function sanitizeMdxJsxAttributeQuotes(body) {
-  let adjusted = false;
-  let out = "";
-  let i = 0;
-  while (i < body.length) {
-    if (body[i] !== "<" || !/[\w.]/.test(body[i + 1] ?? "")) {
-      out += body[i];
-      i += 1;
-      continue;
-    }
-    const tagStart = i;
-    i += 1;
-    while (i < body.length && /[\w.-]/.test(body[i] ?? "")) i += 1;
-    const tagName = body.slice(tagStart + 1, i);
-    let tagOut = `<${tagName}`;
-    let tagAdjusted = false;
-    while (i < body.length && body[i] !== ">" && body[i] !== "/") {
-      while (i < body.length && /\s/.test(body[i] ?? "")) {
-        tagOut += body[i];
-        i += 1;
-      }
-      if (i >= body.length || body[i] === ">" || body[i] === "/") break;
-      const attrStart = i;
-      while (i < body.length && /[\w:.-]/.test(body[i] ?? "")) i += 1;
-      body.slice(attrStart, i);
-      while (i < body.length && /\s/.test(body[i] ?? "")) i += 1;
-      if (body[i] !== "=") {
-        tagOut += body.slice(attrStart, i);
-        continue;
-      }
-      tagOut += body.slice(attrStart, i);
-      tagOut += "=";
-      i += 1;
-      while (i < body.length && /\s/.test(body[i] ?? "")) i += 1;
-      const quote = body[i];
-      if (quote !== '"' && quote !== "'") {
-        continue;
-      }
-      if (quote === "'") {
-        const valStart2 = i + 1;
-        i += 1;
-        while (i < body.length) {
-          if (body[i] === "\\") {
-            i += 2;
-            continue;
-          }
-          if (body[i] === "'") break;
-          i += 1;
-        }
-        tagOut += body.slice(valStart2 - 1, i + 1);
-        i += 1;
-        continue;
-      }
-      const valStart = i + 1;
-      i += 1;
-      let closeIdx = -1;
-      for (let scan = valStart; scan < body.length; scan += 1) {
-        if (body[scan] !== '"') continue;
-        let j = scan + 1;
-        while (j < body.length && /\s/.test(body[j] ?? "")) j += 1;
-        if (j < body.length && (body[j] === ">" || body[j] === "/" || /[a-zA-Z]/.test(body[j] ?? ""))) {
-          closeIdx = scan;
-          break;
-        }
-      }
-      if (closeIdx === -1) {
-        tagOut += body.slice(valStart - 1, i);
-        break;
-      }
-      const value = body.slice(valStart, closeIdx);
-      const hasInternalQuote = value.includes('"');
-      if (hasInternalQuote) {
-        const escaped = value.replace(/\\/g, "\\\\").replace(/'/g, "\\'");
-        tagOut += `'${escaped}'`;
-        tagAdjusted = true;
-      } else {
-        tagOut += `"${value}"`;
-      }
-      i = closeIdx + 1;
-    }
-    if (tagAdjusted) adjusted = true;
-    tagOut += body[i] ?? "";
-    out += tagOut;
-    i += 1;
-  }
-  return { body: out, adjusted };
-}
-
 // src/translate/validate-translation.ts
 init_esm_shims();
 function formatZodIssues(error) {
@@ -2465,7 +2523,7 @@ async function translatePage(config, item, options = {}) {
     if (!validated.ok) {
       throw new Error(`Translation validation failed: ${validated.error}`);
     }
-    const { body: translatedBody, adjusted: mdxAdjusted } = sanitizeMdxJsxAttributeQuotes(
+    const { body: translatedBody, adjusted: mdxAdjusted } = assertValidTranslatedMdxBody(
       result.parsed.body
     );
     const writeDb = openStore(config, "readwrite");