@dualkit/convert 0.3.0-beta.0 → 0.4.0-beta.0

package/dist/index.cjs

@@ -151,6 +151,11 @@ function detectPackageManager(root) {
     if (top || (0, import_node_path.dirname)(dir) === dir) return "npm";
   }
 }
+var I18N_LIBRARIES = ["next-intl", "react-i18next", "i18next", "next-i18next", "react-intl", "@lingui/core", "next-international", "@inlang/paraglide-js"];
+function existingI18n(root) {
+  const deps = readDeps(root);
+  return I18N_LIBRARIES.find((name) => name in deps) ?? null;
+}
 function detectI18n(deps) {
   if ("next-intl" in deps) return "next-intl";
   if ("react-i18next" in deps) return "react-i18next";
@@ -327,7 +332,7 @@ async function stagedFiles(root) {
   return out.split("\0").filter(Boolean);
 }
 async function commit(root, message) {
-  await must(root, ["commit", "-q", "-m", message]);
+  await must(root, ["commit", "-q", "--no-verify", "-m", message]);
 }
 async function trackedFiles(root) {
   const out = await must(root, ["ls-files", "-z"]);
@@ -562,6 +567,26 @@ function writeConfigAnswers(disk, answers2, file = CONFIG_FILE) {
   return writeText(disk, file, mergedConfigText(readText(disk.root, file), answers2, file));
 }
 
+// src/closed-routes.ts
+var CLOSED = /* @__PURE__ */ new Set([401, 403, 404]);
+var STATUS_TIMEOUT_MS = 6e4;
+var closedStatus = async (url) => {
+  try {
+    const res = await fetch(url, { signal: AbortSignal.timeout(STATUS_TIMEOUT_MS), redirect: "manual" });
+    return CLOSED.has(res.status) ? res.status : null;
+  } catch {
+    return null;
+  }
+};
+async function closedRoutes(base, routes) {
+  const found = [];
+  for (const route of routes) {
+    const status = await closedStatus(`${base}${route}`);
+    if (status) found.push({ route, reason: `the English page returns ${status}, so its Arabic page was not checked` });
+  }
+  return found;
+}
+
 // src/baseline.ts
 var import_node_fs8 = require("fs");
 var import_node_path9 = require("path");
@@ -706,6 +731,7 @@ ${tail2()}`);
 
 // src/baseline.ts
 var BASELINE_DIR = `${STATE_DIR}/baseline`;
+var CLOSED_FILE = `${BASELINE_DIR}/closed.json`;
 var WIDTHS = [390, 1280];
 var DEFAULT_PORTS = { "next-app": 3e3, "vite-react": 5173 };
 var routeSlug = (route) => route.replace(/^\/+|\/+$/g, "").replace(/[^a-zA-Z0-9]+/g, "-").replace(/^-|-$/g, "") || "home";
@@ -717,6 +743,7 @@ async function shootBaseline(project, config, shoot, log) {
   const server = await startServer(config.start, port, project.root);
   const shots = [];
   try {
+    (0, import_node_fs8.writeFileSync)((0, import_node_path9.join)(project.root, CLOSED_FILE), JSON.stringify((await closedRoutes(server.url, routes)).map((c) => c.route)));
     for (const route of routes) {
       for (const width of WIDTHS) {
         const path = baselinePath(route, width);
@@ -1093,6 +1120,7 @@ function pieceOf(child, ctx) {
   const source = ctx.file.text.slice(ctx.base + expression.start, ctx.base + expression.end);
   return { kind: "value", name, source: mayBeEmpty(expression) ? `${source} ?? ''` : source, start: start2, end };
 }
+var OWN_DIRECTION = /setAttribute\(\s*['"]dir['"]|documentElement\.dir\s*=[^=]/;
 var NODE_NAMES = /^(children|icon|avatar|logo|image|element|node)$/i;
 var mayBeEmpty = (expression) => /^Optional/.test(expression.type) || expression.type === "CallExpression" && !textMethodTarget(expression);
 var jsxText = (value) => value.replace(/^\s*\n\s*/, "").replace(/\s*\n\s*$/, "");
@@ -1342,6 +1370,15 @@ function extractStrings(file, lib, taken) {
   for (const block of file.js) {
     const program = block.ast.program;
     if (isServerFile(program)) continue;
+    if (OWN_DIRECTION.test(file.text.slice(block.offset))) {
+      flag(
+        { file, flags },
+        block.offset,
+        "unsupported",
+        "This file sets the page direction itself, which can clash with the Arabic layout.",
+        "Check the Arabic pages, and drive this direction from the locale so components read it too."
+      );
+    }
     const ctx = { file, base: block.offset, lib, stem: stemOf(file.path), used, byText: /* @__PURE__ */ new Map(), entries, edits: edits2, flags, components: /* @__PURE__ */ new Set(), sawJsx: false, consumed: /* @__PURE__ */ new Set(), usedTrans: false };
     walk(program, ctx, []);
     if (ctx.sawJsx) flagModuleStrings(program, ctx);
@@ -2096,7 +2133,7 @@ export function DualkitLanguageSwitch() {
   )
 }
 `;
-var INLINE_SCRIPT = `<script>(function(){var m=document.cookie.match(${COOKIE_PATTERN});var l=m&&m[1]==='ar'?'ar':'en';var h=document.documentElement;h.lang=l;h.dir=l==='ar'?'rtl':'ltr'})()</script>`;
+var INLINE_SCRIPT = `<script>(function(){var m=document.cookie.match(${COOKIE_PATTERN});var l=m&&m[1]==='ar'?'ar':'en';var h=document.documentElement;h.lang=l;h.dir=l==='ar'?'rtl':'ltr';if(l==='ar')new MutationObserver(function(){if(h.dir!=='rtl')h.dir='rtl'}).observe(h,{attributes:true,attributeFilter:['dir']})})()</script>`;
 function htmlEdits2(html, mode) {
   if (mode === "bilingual") {
     if (html.includes(LOCALE_COOKIE)) return [];
@@ -2692,12 +2729,27 @@ function visitExpression(container, ancestors, ctx) {
   ctx.edits.push({ start: ctx.base + container.start, end: ctx.base + container.start, text: "<bdi>" });
   ctx.edits.push({ start: ctx.base + container.end, end: ctx.base + container.end, text: "</bdi>" });
 }
+var SIGNED_NUMBER = /^(?:[+\-\u2212]\d[\d,.]*\s?%?|\d[\d,.]*\s?%)$/;
+var NO_BDI = /* @__PURE__ */ new Set(["svg", "text", "option", "title", "textarea", "style", "script"]);
+function visitText(text, ancestors, ctx) {
+  const parent = ancestors[ancestors.length - 1];
+  if (parent.type !== "JSXElement" && parent.type !== "JSXFragment") return;
+  const value = String(text.value);
+  const trimmed = value.trim();
+  if (!SIGNED_NUMBER.test(trimmed)) return;
+  const elements = ancestors.filter((a) => a.type === "JSXElement");
+  if (elements.some((a) => isolates(a) || NO_BDI.has(tagName2(a.openingElement) ?? ""))) return;
+  const start2 = ctx.base + text.start + value.indexOf(trimmed);
+  ctx.edits.push({ start: start2, end: start2, text: "<bdi>" });
+  ctx.edits.push({ start: start2 + trimmed.length, end: start2 + trimmed.length, text: "</bdi>" });
+}
 function walk3(node, ctx, ancestors) {
   if (Array.isArray(node)) return node.forEach((child) => walk3(child, ctx, ancestors));
   if (!node || typeof node !== "object") return;
   const n = node;
   if (n.type === "JSXOpeningElement") visitOpening(n, ctx, ancestors);
   if (n.type === "JSXExpressionContainer" && ancestors.length) visitExpression(n, ancestors, ctx);
+  if (n.type === "JSXText" && ancestors.length) visitText(n, ancestors, ctx);
   const next = typeof n.type === "string" ? [...ancestors, n] : ancestors;
   for (const [key, child] of Object.entries(n)) if (!SKIP6.has(key)) walk3(child, ctx, next);
 }
@@ -2839,6 +2891,15 @@ function groups(fragments) {
   for (const f of fragments) byGroup.set(f.group, [...byGroup.get(f.group) ?? [], f]);
   return [...byGroup.values()];
 }
+var TRACKED = /^tracking-(?!normal$)/;
+function trackingEdits(fragments) {
+  const tokens = tokensOf(fragments);
+  if (tokens.some((t) => t.parsed.core.startsWith("tracking-") && t.parsed.variants.includes("rtl"))) return [];
+  const first = tokens.find((t) => TRACKED.test(t.parsed.core) && KEPT(t));
+  if (!first) return [];
+  const end = first.at + first.text.length;
+  return [{ start: end, end, text: " rtl:tracking-normal" }];
+}
 function tailwindEdits(file, version) {
   const parsed = parseVersion(version);
   const fragments = (0, import_lint10.classStrings)(file);
@@ -2851,7 +2912,7 @@ function tailwindEdits(file, version) {
     return { edits: [], flags: [{ file: file.path, line: lineOf(file.text, first.offset), kind: "unsupported", message, suggestion }] };
   }
   const sources = groups(fragments);
-  const edits2 = sources.flatMap((g) => [...physicalEdits(g, parsed), ...parsed[0] < 4 ? reverseEdits(g) : []]);
+  const edits2 = sources.flatMap((g) => [...physicalEdits(g, parsed), ...parsed[0] < 4 ? reverseEdits(g) : [], ...trackingEdits(g)]);
   const flags = [...sources.flatMap((g) => formFlags(file, g)), ...computedFlags(file)];
   return { edits: edits2, flags: flags.sort((a, b) => a.line - b.line) };
 }
@@ -3096,6 +3157,8 @@ async function convert(options, ioIn) {
   if (options.configAnswers) checkConfigAnswers(root, options.configFile);
   const io = options.configAnswers ? { ...ioIn, config: { ...ioIn.config, ...options.configAnswers } } : ioIn;
   const project = detectProject(root);
+  const library = rerun ? null : existingI18n(root);
+  if (library) throw new UnsupportedError(`This app already uses ${library}. Convert adds its own translation setup and the two collide, so nothing was changed. Convert supports apps with no translation library yet.`);
   const branch = branchFor(mode);
   const generated = new Set(state?.generated ?? []);
   const existing = readLedger(root);
@@ -3269,7 +3332,8 @@ function english(data) {
 }
 function arabic(data) {
   const lines = data.arabic.map(
-    (p) => !Number.isFinite(p.errors) ? `- ${esc(p.route)}: the check returned no result` : `- ${esc(p.route)}: score ${p.score}, ${plural(p.errors, "error", "errors")}, ${plural(p.warnings, "warning", "warnings")}`
+    (p) => !Number.isFinite(p.errors) ? `- ${esc(p.route)}: ${esc(p.problems?.[0] ?? "the check returned no result")}` : `- ${esc(p.route)}: score ${p.score}, ${plural(p.errors, "error", "errors")}, ${plural(p.warnings, "warning", "warnings")}${(p.problems ?? []).map((m) => `
+  - ${esc(m)}`).join("")}`
   );
   return section("Arabic pages", lines, "No pages were checked.");
 }
@@ -3385,11 +3449,21 @@ async function arabicChecks(project, state, base, routes, audit) {
       const result = await audit(`${base}${path}`, width, options);
       const usable = result && Number.isFinite(result.errors);
       const failed = { score: Number.NaN, errors: Number.NaN, warnings: Number.NaN };
-      arabic2.push({ route: `${path} at ${width}px`, ...usable ? result : failed });
+      arabic2.push({ route: `${path} at ${width}px`, ...usable ? result : { ...failed, ...result?.problems ? { problems: result.problems } : {} } });
     }
   }
   return arabic2;
 }
+function closedAtBaseline(root) {
+  const path = (0, import_node_path19.join)(root, CLOSED_FILE);
+  if (!(0, import_node_fs11.existsSync)(path)) return /* @__PURE__ */ new Set();
+  try {
+    const routes = JSON.parse((0, import_node_fs11.readFileSync)(path, "utf8"));
+    return new Set(Array.isArray(routes) ? routes.filter((r) => typeof r === "string") : []);
+  } catch {
+    return /* @__PURE__ */ new Set();
+  }
+}
 async function runChecks(project, state, routes, { config, shoot, audit }, step) {
   const { start: cmd } = config;
   if (!cmd || !shoot) throw new RefusedError(NEEDS_BROWSER);
@@ -3400,8 +3474,13 @@ async function runChecks(project, state, routes, { config, shoot, audit }, step)
   });
   try {
     step.name = "checking the pages";
-    const english2 = state.mode === "bilingual" ? await englishChecks(project.root, server.url, routes, shoot) : { english: [], noBaseline: [] };
-    return { ...english2, arabic: await arabicChecks(project, state, server.url, routes, audit) };
+    const closed = await closedRoutes(server.url, routes);
+    const shut = new Set(closed.map((c) => c.route));
+    const open = routes.filter((route) => !shut.has(route));
+    const before = closedAtBaseline(project.root);
+    const compared = routes.filter((route) => !(shut.has(route) && before.has(route)));
+    const english2 = state.mode === "bilingual" ? await englishChecks(project.root, server.url, compared, shoot) : { english: [], noBaseline: [] };
+    return { ...english2, closed, arabic: await arabicChecks(project, state, server.url, open, audit) };
   } finally {
     await server.stop();
   }
@@ -3442,7 +3521,7 @@ async function run(ready, step) {
     ledger,
     ...checks,
     ...earlier ? {} : { summaryMissing: true },
-    unreachable,
+    unreachable: [...unreachable, ...checks.closed],
     remainingLint,
     ...tests2 ? { tests: { code: tests2.code, tail: tail(tests2.out) } } : {}
   };

package/dist/index.d.cts

@@ -157,6 +157,7 @@ type FinishIo = ConvertIo & {
         score: number;
         errors: number;
         warnings: number;
+        problems?: string[];
     }>;
     lint: (root: string) => Promise<number>;
 };

package/dist/index.d.ts

@@ -157,6 +157,7 @@ type FinishIo = ConvertIo & {
         score: number;
         errors: number;
         warnings: number;
+        problems?: string[];
     }>;
     lint: (root: string) => Promise<number>;
 };

package/dist/index.js

@@ -116,6 +116,11 @@ function detectPackageManager(root) {
     if (top || dirname(dir) === dir) return "npm";
   }
 }
+var I18N_LIBRARIES = ["next-intl", "react-i18next", "i18next", "next-i18next", "react-intl", "@lingui/core", "next-international", "@inlang/paraglide-js"];
+function existingI18n(root) {
+  const deps = readDeps(root);
+  return I18N_LIBRARIES.find((name) => name in deps) ?? null;
+}
 function detectI18n(deps) {
   if ("next-intl" in deps) return "next-intl";
   if ("react-i18next" in deps) return "react-i18next";
@@ -292,7 +297,7 @@ async function stagedFiles(root) {
   return out.split("\0").filter(Boolean);
 }
 async function commit(root, message) {
-  await must(root, ["commit", "-q", "-m", message]);
+  await must(root, ["commit", "-q", "--no-verify", "-m", message]);
 }
 async function trackedFiles(root) {
   const out = await must(root, ["ls-files", "-z"]);
@@ -527,6 +532,26 @@ function writeConfigAnswers(disk, answers2, file = CONFIG_FILE) {
   return writeText(disk, file, mergedConfigText(readText(disk.root, file), answers2, file));
 }
 
+// src/closed-routes.ts
+var CLOSED = /* @__PURE__ */ new Set([401, 403, 404]);
+var STATUS_TIMEOUT_MS = 6e4;
+var closedStatus = async (url) => {
+  try {
+    const res = await fetch(url, { signal: AbortSignal.timeout(STATUS_TIMEOUT_MS), redirect: "manual" });
+    return CLOSED.has(res.status) ? res.status : null;
+  } catch {
+    return null;
+  }
+};
+async function closedRoutes(base, routes) {
+  const found = [];
+  for (const route of routes) {
+    const status = await closedStatus(`${base}${route}`);
+    if (status) found.push({ route, reason: `the English page returns ${status}, so its Arabic page was not checked` });
+  }
+  return found;
+}
+
 // src/baseline.ts
 import { mkdirSync as mkdirSync4, writeFileSync as writeFileSync5 } from "fs";
 import { join as join7 } from "path";
@@ -671,6 +696,7 @@ ${tail2()}`);
 
 // src/baseline.ts
 var BASELINE_DIR = `${STATE_DIR}/baseline`;
+var CLOSED_FILE = `${BASELINE_DIR}/closed.json`;
 var WIDTHS = [390, 1280];
 var DEFAULT_PORTS = { "next-app": 3e3, "vite-react": 5173 };
 var routeSlug = (route) => route.replace(/^\/+|\/+$/g, "").replace(/[^a-zA-Z0-9]+/g, "-").replace(/^-|-$/g, "") || "home";
@@ -682,6 +708,7 @@ async function shootBaseline(project, config, shoot, log) {
   const server = await startServer(config.start, port, project.root);
   const shots = [];
   try {
+    writeFileSync5(join7(project.root, CLOSED_FILE), JSON.stringify((await closedRoutes(server.url, routes)).map((c) => c.route)));
     for (const route of routes) {
       for (const width of WIDTHS) {
         const path = baselinePath(route, width);
@@ -1058,6 +1085,7 @@ function pieceOf(child, ctx) {
   const source = ctx.file.text.slice(ctx.base + expression.start, ctx.base + expression.end);
   return { kind: "value", name, source: mayBeEmpty(expression) ? `${source} ?? ''` : source, start: start2, end };
 }
+var OWN_DIRECTION = /setAttribute\(\s*['"]dir['"]|documentElement\.dir\s*=[^=]/;
 var NODE_NAMES = /^(children|icon|avatar|logo|image|element|node)$/i;
 var mayBeEmpty = (expression) => /^Optional/.test(expression.type) || expression.type === "CallExpression" && !textMethodTarget(expression);
 var jsxText = (value) => value.replace(/^\s*\n\s*/, "").replace(/\s*\n\s*$/, "");
@@ -1307,6 +1335,15 @@ function extractStrings(file, lib, taken) {
   for (const block of file.js) {
     const program = block.ast.program;
     if (isServerFile(program)) continue;
+    if (OWN_DIRECTION.test(file.text.slice(block.offset))) {
+      flag(
+        { file, flags },
+        block.offset,
+        "unsupported",
+        "This file sets the page direction itself, which can clash with the Arabic layout.",
+        "Check the Arabic pages, and drive this direction from the locale so components read it too."
+      );
+    }
     const ctx = { file, base: block.offset, lib, stem: stemOf(file.path), used, byText: /* @__PURE__ */ new Map(), entries, edits: edits2, flags, components: /* @__PURE__ */ new Set(), sawJsx: false, consumed: /* @__PURE__ */ new Set(), usedTrans: false };
     walk(program, ctx, []);
     if (ctx.sawJsx) flagModuleStrings(program, ctx);
@@ -2061,7 +2098,7 @@ export function DualkitLanguageSwitch() {
   )
 }
 `;
-var INLINE_SCRIPT = `<script>(function(){var m=document.cookie.match(${COOKIE_PATTERN});var l=m&&m[1]==='ar'?'ar':'en';var h=document.documentElement;h.lang=l;h.dir=l==='ar'?'rtl':'ltr'})()</script>`;
+var INLINE_SCRIPT = `<script>(function(){var m=document.cookie.match(${COOKIE_PATTERN});var l=m&&m[1]==='ar'?'ar':'en';var h=document.documentElement;h.lang=l;h.dir=l==='ar'?'rtl':'ltr';if(l==='ar')new MutationObserver(function(){if(h.dir!=='rtl')h.dir='rtl'}).observe(h,{attributes:true,attributeFilter:['dir']})})()</script>`;
 function htmlEdits2(html, mode) {
   if (mode === "bilingual") {
     if (html.includes(LOCALE_COOKIE)) return [];
@@ -2657,12 +2694,27 @@ function visitExpression(container, ancestors, ctx) {
   ctx.edits.push({ start: ctx.base + container.start, end: ctx.base + container.start, text: "<bdi>" });
   ctx.edits.push({ start: ctx.base + container.end, end: ctx.base + container.end, text: "</bdi>" });
 }
+var SIGNED_NUMBER = /^(?:[+\-\u2212]\d[\d,.]*\s?%?|\d[\d,.]*\s?%)$/;
+var NO_BDI = /* @__PURE__ */ new Set(["svg", "text", "option", "title", "textarea", "style", "script"]);
+function visitText(text, ancestors, ctx) {
+  const parent = ancestors[ancestors.length - 1];
+  if (parent.type !== "JSXElement" && parent.type !== "JSXFragment") return;
+  const value = String(text.value);
+  const trimmed = value.trim();
+  if (!SIGNED_NUMBER.test(trimmed)) return;
+  const elements = ancestors.filter((a) => a.type === "JSXElement");
+  if (elements.some((a) => isolates(a) || NO_BDI.has(tagName2(a.openingElement) ?? ""))) return;
+  const start2 = ctx.base + text.start + value.indexOf(trimmed);
+  ctx.edits.push({ start: start2, end: start2, text: "<bdi>" });
+  ctx.edits.push({ start: start2 + trimmed.length, end: start2 + trimmed.length, text: "</bdi>" });
+}
 function walk3(node, ctx, ancestors) {
   if (Array.isArray(node)) return node.forEach((child) => walk3(child, ctx, ancestors));
   if (!node || typeof node !== "object") return;
   const n = node;
   if (n.type === "JSXOpeningElement") visitOpening(n, ctx, ancestors);
   if (n.type === "JSXExpressionContainer" && ancestors.length) visitExpression(n, ancestors, ctx);
+  if (n.type === "JSXText" && ancestors.length) visitText(n, ancestors, ctx);
   const next = typeof n.type === "string" ? [...ancestors, n] : ancestors;
   for (const [key, child] of Object.entries(n)) if (!SKIP6.has(key)) walk3(child, ctx, next);
 }
@@ -2804,6 +2856,15 @@ function groups(fragments) {
   for (const f of fragments) byGroup.set(f.group, [...byGroup.get(f.group) ?? [], f]);
   return [...byGroup.values()];
 }
+var TRACKED = /^tracking-(?!normal$)/;
+function trackingEdits(fragments) {
+  const tokens = tokensOf(fragments);
+  if (tokens.some((t) => t.parsed.core.startsWith("tracking-") && t.parsed.variants.includes("rtl"))) return [];
+  const first = tokens.find((t) => TRACKED.test(t.parsed.core) && KEPT(t));
+  if (!first) return [];
+  const end = first.at + first.text.length;
+  return [{ start: end, end, text: " rtl:tracking-normal" }];
+}
 function tailwindEdits(file, version) {
   const parsed = parseVersion(version);
   const fragments = classStrings(file);
@@ -2816,7 +2877,7 @@ function tailwindEdits(file, version) {
     return { edits: [], flags: [{ file: file.path, line: lineOf(file.text, first.offset), kind: "unsupported", message, suggestion }] };
   }
   const sources = groups(fragments);
-  const edits2 = sources.flatMap((g) => [...physicalEdits(g, parsed), ...parsed[0] < 4 ? reverseEdits(g) : []]);
+  const edits2 = sources.flatMap((g) => [...physicalEdits(g, parsed), ...parsed[0] < 4 ? reverseEdits(g) : [], ...trackingEdits(g)]);
   const flags = [...sources.flatMap((g) => formFlags(file, g)), ...computedFlags(file)];
   return { edits: edits2, flags: flags.sort((a, b) => a.line - b.line) };
 }
@@ -3061,6 +3122,8 @@ async function convert(options, ioIn) {
   if (options.configAnswers) checkConfigAnswers(root, options.configFile);
   const io = options.configAnswers ? { ...ioIn, config: { ...ioIn.config, ...options.configAnswers } } : ioIn;
   const project = detectProject(root);
+  const library = rerun ? null : existingI18n(root);
+  if (library) throw new UnsupportedError(`This app already uses ${library}. Convert adds its own translation setup and the two collide, so nothing was changed. Convert supports apps with no translation library yet.`);
   const branch = branchFor(mode);
   const generated = new Set(state?.generated ?? []);
   const existing = readLedger(root);
@@ -3234,7 +3297,8 @@ function english(data) {
 }
 function arabic(data) {
   const lines = data.arabic.map(
-    (p) => !Number.isFinite(p.errors) ? `- ${esc(p.route)}: the check returned no result` : `- ${esc(p.route)}: score ${p.score}, ${plural(p.errors, "error", "errors")}, ${plural(p.warnings, "warning", "warnings")}`
+    (p) => !Number.isFinite(p.errors) ? `- ${esc(p.route)}: ${esc(p.problems?.[0] ?? "the check returned no result")}` : `- ${esc(p.route)}: score ${p.score}, ${plural(p.errors, "error", "errors")}, ${plural(p.warnings, "warning", "warnings")}${(p.problems ?? []).map((m) => `
+  - ${esc(m)}`).join("")}`
   );
   return section("Arabic pages", lines, "No pages were checked.");
 }
@@ -3350,11 +3414,21 @@ async function arabicChecks(project, state, base, routes, audit) {
       const result = await audit(`${base}${path}`, width, options);
       const usable = result && Number.isFinite(result.errors);
       const failed = { score: Number.NaN, errors: Number.NaN, warnings: Number.NaN };
-      arabic2.push({ route: `${path} at ${width}px`, ...usable ? result : failed });
+      arabic2.push({ route: `${path} at ${width}px`, ...usable ? result : { ...failed, ...result?.problems ? { problems: result.problems } : {} } });
     }
   }
   return arabic2;
 }
+function closedAtBaseline(root) {
+  const path = join11(root, CLOSED_FILE);
+  if (!existsSync8(path)) return /* @__PURE__ */ new Set();
+  try {
+    const routes = JSON.parse(readFileSync7(path, "utf8"));
+    return new Set(Array.isArray(routes) ? routes.filter((r) => typeof r === "string") : []);
+  } catch {
+    return /* @__PURE__ */ new Set();
+  }
+}
 async function runChecks(project, state, routes, { config, shoot, audit }, step) {
   const { start: cmd } = config;
   if (!cmd || !shoot) throw new RefusedError(NEEDS_BROWSER);
@@ -3365,8 +3439,13 @@ async function runChecks(project, state, routes, { config, shoot, audit }, step)
   });
   try {
     step.name = "checking the pages";
-    const english2 = state.mode === "bilingual" ? await englishChecks(project.root, server.url, routes, shoot) : { english: [], noBaseline: [] };
-    return { ...english2, arabic: await arabicChecks(project, state, server.url, routes, audit) };
+    const closed = await closedRoutes(server.url, routes);
+    const shut = new Set(closed.map((c) => c.route));
+    const open = routes.filter((route) => !shut.has(route));
+    const before = closedAtBaseline(project.root);
+    const compared = routes.filter((route) => !(shut.has(route) && before.has(route)));
+    const english2 = state.mode === "bilingual" ? await englishChecks(project.root, server.url, compared, shoot) : { english: [], noBaseline: [] };
+    return { ...english2, closed, arabic: await arabicChecks(project, state, server.url, open, audit) };
   } finally {
     await server.stop();
   }
@@ -3407,7 +3486,7 @@ async function run(ready, step) {
     ledger,
     ...checks,
     ...earlier ? {} : { summaryMissing: true },
-    unreachable,
+    unreachable: [...unreachable, ...checks.closed],
     remainingLint,
     ...tests2 ? { tests: { code: tests2.code, tail: tail(tests2.out) } } : {}
   };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dualkit/convert",
-  "version": "0.3.0-beta.0",
+  "version": "0.4.0-beta.0",
   "description": "Converts an English React or Next.js app to Arabic on a new git branch.",
   "license": "MIT",
   "type": "module",