@chappibunny/repolens 1.15.5 → 1.16.0

package/src/renderers/renderDiff.js

@@ -1,3 +1,5 @@
+import { replaceBracketParams } from "../utils/route-params.js";
+
 function isRouteFile(file) {
   return (
     file.includes("/pages/api/") ||
@@ -15,10 +17,10 @@ function routePathFromFile(file) {
     const appIndex = file.indexOf("/app/");
     const relative = file.slice(appIndex + 5);
 
-    return "/" + relative
+    const cleaned = relative
       .replace(/\/page\.(ts|tsx|js|jsx)$/, "")
-      .replace(/\/route\.(ts|tsx|js|jsx)$/, "")
-      .replace(/\[(.*?)\]/g, ":$1");
+      .replace(/\/route\.(ts|tsx|js|jsx)$/, "");
+    return `/${replaceBracketParams(cleaned)}`;
   }
 
   if (file.includes("/pages/api/")) {

package/src/renderers/renderMap.js

@@ -1,13 +1,12 @@
+import { replaceLeadingAppGroup, replaceSlashGroups } from "../utils/route-params.js";
+
 function sanitizeNodeId(value) {
   return value.replace(/[^a-zA-Z0-9_]/g, "_");
 }
 
 function normalizeLabel(value) {
-  return value
-    .replace(/^src\//, "")
-    .replace(/^app\/\((.*?)\)/, "app/$1")
-    .replace(/\/\((.*?)\)/g, "/$1")
-    .replace(/\/+/g, "/");
+  const withoutSrc = value.replace(/^src\//, "");
+  return replaceSlashGroups(replaceLeadingAppGroup(withoutSrc)).replace(/\/+/g, "/");
 }
 
 function buildModuleGraph(modules, depGraph) {
@@ -96,9 +95,11 @@ function buildModuleGraph(modules, depGraph) {
  */
 function findModuleForFile(fileKey, modules) {
   const normalized = fileKey.replace(/\\/g, "/");
+  const flat = modules.find((mod) => mod.flat) || modules.find((mod) => mod.key === "root");
   let bestMatch = null;
   for (const mod of modules) {
     const modKey = mod.key;
+    if (flat && modKey === flat.key) continue;
     // Direct match (with or without extension)
     if (normalized === modKey || normalized === modKey.replace(/\.[^/.]+$/, "")) {
       return modKey;
@@ -110,6 +111,7 @@ function findModuleForFile(fileKey, modules) {
       }
     }
   }
+  if (!bestMatch && flat && !normalized.includes("/")) return flat.key;
   return bestMatch;
 }
 

package/src/utils/route-params.js

@@ -0,0 +1,50 @@
+/**
+ * Linear replacements for route and label text.
+ *
+ * A lazy `.*?` between two markers is polynomial: on a long string with no
+ * closer, each start position scans to the end (CodeQL js/polynomial-redos).
+ * These walks stop at the first closer and never re-scan, which matches the
+ * old `String.prototype.replace` result.
+ */
+
+function replaceMarked(input, open, close, mapInner) {
+  const value = String(input ?? "");
+  let out = "";
+  let cursor = 0;
+  while (cursor < value.length) {
+    const start = value.indexOf(open, cursor);
+    if (start === -1) {
+      out += value.slice(cursor);
+      break;
+    }
+    const end = value.indexOf(close, start + open.length);
+    if (end === -1) {
+      out += value.slice(cursor);
+      break;
+    }
+    out += value.slice(cursor, start);
+    out += mapInner(value.slice(start + open.length, end));
+    cursor = end + close.length;
+  }
+  return out;
+}
+
+/** `[param]` → `:param`, including the first `]` only. Global, non-overlapping. */
+export function replaceBracketParams(value) {
+  return replaceMarked(value, "[", "]", (inner) => `:${inner}`);
+}
+
+/** `/(group)` → `/group`. Used for Next.js route groups in diagram labels. */
+export function replaceSlashGroups(value) {
+  return replaceMarked(value, "/(", ")", (inner) => `/${inner}`);
+}
+
+/** A leading `app/(group)` → `app/group`. Later groups use {@link replaceSlashGroups}. */
+export function replaceLeadingAppGroup(value) {
+  const input = String(value ?? "");
+  const prefix = "app/(";
+  if (!input.startsWith(prefix)) return input;
+  const end = input.indexOf(")", prefix.length);
+  if (end === -1) return input;
+  return `app/${input.slice(prefix.length, end)}${input.slice(end + 1)}`;
+}

package/src/utils/yaml.js

@@ -1,14 +1,15 @@
-import yaml from "js-yaml";
+import { CORE_SCHEMA, load, mergeTag } from "js-yaml";
 
 const MAX_YAML_CHARS = 1_000_000;
 const MAX_YAML_ALIASES = 2000;
 
 /**
  * Parse YAML without executing custom tags.
- * js-yaml 4 `load` is already safe (no JS types). This adds a size cap and
- * an alias cap so a config cannot expand into a billion-laughs document.
- * The js-yaml 4 default schema is unchanged (`true`/`false` are booleans;
- * the YAML 1.1 spellings `yes` and `no` stay strings).
+ * js-yaml 5 `load` uses the YAML 1.2 Core schema (`true`/`false` are booleans;
+ * the YAML 1.1 spellings `yes` and `no` stay strings). The merge key (`<<`)
+ * is added back so configs that relied on js-yaml 4's merge still parse.
+ * A size cap and `maxAliases` stop a billion-laughs document.
+ * An empty file returns `undefined`, matching the previous loader.
  */
 export function loadYaml(raw) {
   if (typeof raw !== "string") {
@@ -17,11 +18,15 @@ export function loadYaml(raw) {
   if (raw.length > MAX_YAML_CHARS) {
     throw new Error(`YAML document exceeds ${MAX_YAML_CHARS} characters`);
   }
+  if (raw.trim() === "") return undefined;
 
   const aliasUses = raw.match(/(^|[\s\[\{,])\*[A-Za-z_][\w-]*/g);
   if (aliasUses && aliasUses.length > MAX_YAML_ALIASES) {
     throw new Error(`YAML document has too many aliases (${aliasUses.length})`);
   }
 
-  return yaml.load(raw);
+  return load(raw, {
+    schema: CORE_SCHEMA.withTags(mergeTag),
+    maxAliases: MAX_YAML_ALIASES,
+  });
 }