@danielsimonjr/repo-tools 2.2.5 → 2.2.7

package/CHANGELOG.md

@@ -6,6 +6,44 @@ All notable changes to this project are recorded in this file. The format follow
 
 ## [Unreleased]
 
+## [2.2.7] - 2026-10-22
+
+### Fixed
+
+- `repo-tools ste` exits 2 when any path in the argument list does not exist, and prints `ERROR: path does not exist:
+  <path>` on stderr for each such path. Before, a mistyped path among valid paths was ignored and the run exited 0.
+  The report of each valid path is still printed.
+- The warning that `repo-tools map` writes for a `package.json` entry that does not resolve is shorter (15 words).
+  It had 30 words, so `repo-tools ste` flagged the generated `FILE_INVENTORY.md`. A test now checks the word limit
+  of every warning and that the generated report passes `repo-tools ste`.
+- `repo-tools map` reads a folder named `test/` as a tests folder, as version 1.x did. Before, only `tests/` counted,
+  so a file such as `test/setup.ts` got the area `src` in `FILE_INVENTORY.md` and counted as a source file in
+  `TEST_COVERAGE.md`. The disposition legend and the `--tests` message name both folders.
+- The `TEST_COVERAGE.md` note on the coverage policy no longer links `COVERAGE_POLICY.md`, a file that the tool does
+  not write, and no longer names mathjs or AssemblyScript.
+- The `Expected test` hint in `TEST_COVERAGE.md` names `tests/unit/<module>/`. It appears only when the repository
+  keeps tests in `tests/unit/`. Other repositories get the plain list of untested files.
+- The text of `duplicate-symbols.md` and of the `duplicate-symbols.json` note no longer names `Bucket C` or
+  `AssemblyScript`.
+
+## [2.2.6] - 2026-10-22
+
+### Fixed
+
+- `repo-tools map` finds the entry roots of an app whose build output is not `dist/` and whose sources are not
+  only under `src/`. On an Electron app (main process built to `dist-electron/`, CLI built to `dist-cli/`, renderer
+  loaded from `src/index.html`) it reported 74 dormant files, among them 11 orphans, and now reports 5 dormant files,
+  among them 3 orphans. Four sources of roots were missing or broken:
+  - A `tsc -p <tsconfig>` script of the root package in a repository without workspaces seeds the files of that
+    tsconfig. Only a workspace package did this before.
+  - A tsconfig that holds comments or trailing commas is read. `JSON.parse` rejected it and the whole tsconfig was
+    ignored without a message.
+  - A root build or test config names entry files with an `entry`, `input`, `setupFiles`, `globalSetup` or
+    `globalTeardown` key (for example `path.resolve(__dirname, 'electron/main.ts')`). Each such file is a root.
+    Only `new URL(...)` was read before.
+  - The module script of `index.html` or `src/index.html` is a root.
+  The ambient declaration files (`vite-env.d.ts`) stay in the list of files without an importer, as before.
+
 ## [2.2.5] - 2026-10-22
 
 ### Changed

package/README.md

@@ -13,7 +13,7 @@
 | `query` | Answers structural questions from the graph of `map`, and writes two derived reports. |
 | `ste` | Checks Markdown or docstring prose against Simplified Technical English. |
 
-Status: version 2.2.5.
+Status: version 2.2.7.
 
 - `map`, `check` and `query` read their settings from `repo-tools.config.json` in the root. The
   section of `map` is `map` (its old name, `depgraph`, is also read). `--config=<file>` makes any
@@ -41,7 +41,7 @@ repo-tools --version
 
 The executable holds the tool and the Bun runtime. It needs no Bun, Node or `node_modules`.
 Download the file for your platform and `SHA256SUMS` from the
-[GitHub release](https://github.com/danielsimonjr/repo-tools/releases/tag/v2.2.5):
+[GitHub release](https://github.com/danielsimonjr/repo-tools/releases/tag/v2.2.7):
 
 | Platform | File |
 |---|---|

package/dist/cli.js

@@ -2,7 +2,7 @@
 // package.json
 var package_default = {
   name: "@danielsimonjr/repo-tools",
-  version: "2.2.5",
+  version: "2.2.7",
   description: "One CLI for repository analysis: dependency graphs creation and query with per-export API facts, file chunking, and context compression. TypeScript on Bun, compiled to a single executable.",
   type: "module",
   license: "MIT",
@@ -974,7 +974,7 @@ function mergeDepgraphConfig(cli, file) {
 // src/depgraph/args.ts
 var SCAN_SCOPE_FLAGS = {
   "--src": "the census finds the source files itself; the area of a file comes from its path",
-  "--tests": "a test file is found by its name or its tests/ folder",
+  "--tests": "a test file is found by its name or by its test/ or tests/ folder",
   "--exclude": "the census reads the files that git tracks; list a file in .gitignore instead",
   "--also-exclude": "the census reads the files that git tracks; list a file in .gitignore instead",
   "--all": "the graph always holds every file; the disposition of each file shows reachability",
@@ -1372,7 +1372,7 @@ var TYPE_DUP_CATEGORIES = [
   { cat: "type", key: "types" },
   { cat: "enum", key: "enums" }
 ];
-var DUPLICATE_SYMBOLS_NOTE = "This report groups names by OWN definition, not by call graph, then classifies each " + "flagged name (see DupEntryTag): TRUE_DUPLICATE (the actionable merge targets), " + "DISPATCH_VARIANT (>=2 mathTyped(...) registrations of the same public name — distinct " + "dispatch surfaces, Bucket C delegation candidates, not copy-paste bodies), " + "ALIAS_DELEGATION (a const-alias forward to an imported symbol, not an independent body — " + "excluded once fewer than 2 real bodies remain), and ALLOWLISTED (matches " + "duplicate-allowlist.json: hot-path is* guards, AssemblyScript mirrors, per-package " + "VERSION strings). NOT detected: same-file typed-dispatch overload polymorphism for " + "different argument shapes within one registration — a human still triages TRUE_DUPLICATE " + "entries using the defining files + public flags before merging anything.";
+var DUPLICATE_SYMBOLS_NOTE = "This report groups names by OWN definition, not by call graph, then classifies each " + "flagged name (see DupEntryTag): TRUE_DUPLICATE (the actionable merge targets), " + "DISPATCH_VARIANT (>=2 mathTyped(...) registrations of the same public name — distinct " + "dispatch surfaces, not copy-paste bodies), " + "ALIAS_DELEGATION (a const-alias forward to an imported symbol, not an independent body — " + "excluded once fewer than 2 real bodies remain), and ALLOWLISTED (matches " + "duplicate-allowlist.json: for example hot-path is* guards, mirrors in another language, " + "per-package VERSION strings). NOT detected: same-file typed-dispatch overload polymorphism for " + "different argument shapes within one registration — a human still triages TRUE_DUPLICATE " + "entries using the defining files + public flags before merging anything.";
 var DUP_ENTRY_TAG_SORT_ORDER = {
   TRUE_DUPLICATE: 0,
   DISPATCH_VARIANT: 1,
@@ -2066,13 +2066,55 @@ function tsupConfigEntries(root, pkgDir) {
   }
   return out2;
 }
+function stringEnd(text, start2) {
+  let j = start2 + 1;
+  while (j < text.length && text.charAt(j) !== '"')
+    j += text.charAt(j) === "\\" ? 2 : 1;
+  return Math.min(j + 1, text.length);
+}
+function stripJsonc(text) {
+  let plain = "";
+  for (let i2 = 0;i2 < text.length; ) {
+    const c = text.charAt(i2);
+    if (c === '"') {
+      const end = stringEnd(text, i2);
+      plain += text.slice(i2, end);
+      i2 = end;
+    } else if (c === "/" && text.charAt(i2 + 1) === "/") {
+      while (i2 < text.length && text.charAt(i2) !== `
+`)
+        i2++;
+    } else if (c === "/" && text.charAt(i2 + 1) === "*") {
+      const end = text.indexOf("*/", i2 + 2);
+      i2 = end === -1 ? text.length : end + 2;
+    } else {
+      plain += c;
+      i2++;
+    }
+  }
+  let out2 = "";
+  for (let i2 = 0;i2 < plain.length; ) {
+    const c = plain.charAt(i2);
+    if (c === '"') {
+      const end = stringEnd(plain, i2);
+      out2 += plain.slice(i2, end);
+      i2 = end;
+    } else if (c === "," && /^\s*[}\]]/.test(plain.slice(i2 + 1))) {
+      i2++;
+    } else {
+      out2 += c;
+      i2++;
+    }
+  }
+  return out2;
+}
 function seedTsconfigEntries(root, pkgDir, cfgRel, add) {
   const cfgPath = join7(root, pkgDir, cfgRel);
   if (!existsSync4(cfgPath))
     return;
   let cfg;
   try {
-    cfg = JSON.parse(readFileSync3(cfgPath, "utf-8"));
+    cfg = JSON.parse(stripJsonc(readFileSync3(cfgPath, "utf-8")));
   } catch {
     return;
   }
@@ -2226,6 +2268,8 @@ function selfPackage(root) {
     entryFiles: packageEntryFiles(root, pkg)
   };
 }
+var CONFIG_ENTRY_KEY = /\b(?:entry|input|setupFiles|globalSetup|globalTeardown)["']?\s*:\s*(\[[^\]]*\]|\{[^}]*\}|[\w$.]+\([^)]*\)|['"`][^'"`]*['"`])/g;
+var SOURCE_LITERAL = /['"`]([^'"`]+\.[cm]?[jt]sx?)['"`]/g;
 function configReferencedEntries(root) {
   const out2 = new Set;
   let names;
@@ -2247,9 +2291,36 @@ function configReferencedEntries(root) {
     for (const m of code.matchAll(/new\s+URL\(\s*['"`]([^'"`]*?src\/[\w./-]+\.ts)['"`]/g)) {
       out2.add((m[1] ?? "").replace(/^\.\//, ""));
     }
+    for (const key of code.matchAll(CONFIG_ENTRY_KEY)) {
+      for (const lit of (key[1] ?? "").matchAll(SOURCE_LITERAL)) {
+        out2.add(toPosix(lit[1] ?? "").replace(/^\.\//, ""));
+      }
+    }
   }
   return [...out2];
 }
+function htmlReferencedEntries(root) {
+  const out2 = [];
+  for (const page of ["index.html", "src/index.html"]) {
+    let html;
+    try {
+      html = readFileSync3(join7(root, page), "utf-8");
+    } catch {
+      continue;
+    }
+    for (const m of html.matchAll(/<script\b[^>]*?\bsrc\s*=\s*["']([^"']+)["']/gi)) {
+      const src = m[1] ?? "";
+      if (/^(?:[a-z][a-z0-9+.-]*:|\/\/)/i.test(src) || !/\.[cm]?[jt]sx?$/.test(src))
+        continue;
+      const fromPage = toPosix(join7(dirname4(page), src)).replace(/^\.\//, "");
+      const candidates = src.startsWith("/") ? [src.slice(1), fromPage] : [fromPage];
+      const found = candidates.find((c) => existsSync4(join7(root, c)));
+      if (found !== undefined && !out2.includes(found))
+        out2.push(found);
+    }
+  }
+  return out2;
+}
 function runtimeLaunchedEntries(root, parsed) {
   const known = new Set(parsed.map((f) => f.path));
   const out2 = new Set;
@@ -6307,7 +6378,7 @@ function classifyArea(relPath) {
   }
   if (CONFIG_PATTERN.test(relPath))
     return "config";
-  if (isTestFile || /(^|\/)tests\//.test(relPath))
+  if (isTestFile || /(^|\/)tests?\//.test(relPath))
     return "tests";
   if (/^(bench|benchmarks)\//.test(relPath) || /\.bench\.\w+$/.test(relPath))
     return "benchmarks";
@@ -12364,7 +12435,7 @@ function packageJsonRoots(root, known) {
       continue;
     const mapped = mapDistToSrc(spec, known) ?? launcherEntry(root, spec, known) ?? declaredSource(spec, known);
     if (mapped === null) {
-      warnings.push(`package.json entry ${pyRepr(spec)} did not resolve to a known source file -- ` + "ignored; if no other declared entry resolves either, the entry-point root falls back to " + "a conventional src/index.* file instead");
+      warnings.push(`package.json entry ${pyRepr(spec)} did not resolve to a known source file. The entry is ignored.`);
       continue;
     }
     if (!roots.includes(mapped))
@@ -12513,7 +12584,8 @@ function findRoots(root, known, language) {
     return rustRoots(known);
   const [rootRoots, rootWarnings] = packageJsonRoots(root, known);
   const [wsRoots, wsWarnings] = workspaceRoots(root, known);
-  const roots = [...new Set([...rootRoots, ...wsRoots])];
+  const scriptRoots = detectWorkspaces(root).size === 0 ? rootPackageEntries(root).filter((e) => known.has(e)) : [];
+  const roots = [...new Set([...rootRoots, ...wsRoots, ...scriptRoots])];
   const warnings = [...rootWarnings, ...wsWarnings];
   if (roots.length > 0)
     return [roots, warnings];
@@ -12689,7 +12761,7 @@ async function buildGraph(root) {
   const [declaredRoots, rootWarnings] = findRoots(root, known, language);
   const roots = [...declaredRoots];
   if (language === "typescript") {
-    for (const entry of configReferencedEntries(root)) {
+    for (const entry of [...configReferencedEntries(root), ...htmlReferencedEntries(root)]) {
       if (known.has(entry) && !roots.includes(entry))
         roots.push(entry);
     }
@@ -13559,7 +13631,7 @@ function generateTestCoverageMarkdown(coverage, options = {}) {
     lines.push("");
     lines.push(...basisNote);
     const policyPath = options.policyPath ?? "docs/architecture/coverage-policy.json";
-    lines.push("> The raw figure counts every source file the CDG tool finds, including code that is " + "intentionally not direct-imported by a vitest `*.test.ts` (synced mathjs categories, " + "AssemblyScript sources, type-only barrels, …). The **effective** figure excludes those " + `per \`${policyPath}\` so the number reflects the genuinely-active hand-written code only. ` + "See [`COVERAGE_POLICY.md`](./COVERAGE_POLICY.md) for the policy.");
+    lines.push("> The raw figure counts every source file the map finds, including files that no test " + "imports directly by design (for example generated code, type-only barrels and code in " + `another language). The **effective** figure excludes the files that \`${policyPath}\` ` + "classifies, so it shows the active code only.");
     lines.push("");
     lines.push("### Untested-file breakdown by category");
     lines.push("");
@@ -14155,7 +14227,7 @@ function emitTestCoverage(graph, root, outDir, options = {}) {
   const jsonPath = join16(outDir, "test-coverage.json");
   const namespaceImportBasis = !importsNameSymbols(graph.language);
   const markdown = generateTestCoverageMarkdown(coverage, {
-    expectedTestHint: graph.language === "typescript",
+    expectedTestHint: graph.language === "typescript" && tests.some((t) => t.path.startsWith("tests/unit/")),
     policyPath: options.policyNote ?? "docs/architecture/coverage-policy.json",
     namespaceImportBasis
   });
@@ -14215,17 +14287,14 @@ function generateDuplicateSymbolsMarkdown(report) {
   md += "Each name carries one tag (see `DupEntryTag`), so the actionable subset is clear:\n\n";
   md += "- `TRUE_DUPLICATE`: a real merge target.\n";
   md += "- `DISPATCH_VARIANT`: two or more `mathTyped(...)` registrations of the same public name. ";
-  md += "The registrations are distinct dispatch surfaces and Bucket C delegation candidates, ";
-  md += `not copy-paste bodies.
+  md += `The registrations are distinct dispatch surfaces, not copy-paste bodies.
 `;
   md += "- `ALIAS_DELEGATION`: a `const X = importedY` forward. ";
   md += `The entry is excluded once fewer than two real bodies remain.
 `;
   md += "- `ALLOWLISTED`: the name matches `duplicate-allowlist.json`. ";
-  md += "Examples are hot-path `is*` guards, AssemblyScript mirrors and per-package `VERSION` ";
-  md += `strings.
-
-`;
+  md += "Examples are hot-path `is*` guards, mirrors in another language and per-package ";
+  md += "`VERSION` strings.\n\n";
   md += `> **Note:** ${report.note}
 
 `;
@@ -14241,7 +14310,7 @@ function generateDuplicateSymbolsMarkdown(report) {
 
 `;
   md += renderTaggedSection("TRUE_DUPLICATE — actionable merge targets", report.runtime, "TRUE_DUPLICATE");
-  md += renderTaggedSection("DISPATCH_VARIANT — distinct public typed-dispatch surfaces (Bucket C candidates)", report.runtime, "DISPATCH_VARIANT");
+  md += renderTaggedSection("DISPATCH_VARIANT — distinct public typed-dispatch surfaces", report.runtime, "DISPATCH_VARIANT");
   md += renderTaggedSection("ALIAS_DELEGATION — const-alias forwards (not independent bodies)", report.runtime, "ALIAS_DELEGATION");
   md += renderTaggedSection("ALLOWLISTED — accepted layering (see duplicate-allowlist.json)", report.runtime, "ALLOWLISTED");
   md += `## Type duplicates (lower priority)
@@ -14260,7 +14329,7 @@ var DISPOSITION_LEGEND = [
   ["build-entry", "A `src`-area file that is an entry root."],
   ["test-only", "A `src`-area file that only a test reaches."],
   ["orphan", "A `src`-area file that nothing reaches. Delete it, or wire it to a root."],
-  ["test", "A file in a `tests/` folder, or a `*.test.ts` or `*.spec.ts` file."],
+  ["test", "A file in a `test/` or `tests/` folder, or a `*.test.ts` or `*.spec.ts` file."],
   ["tool", "A file in a `tools/` or `scripts/` folder outside `src/`."],
   ["config", "A `*.config.*` file of TypeScript or JavaScript."],
   ["example", "A file in `examples/` or `docs/`."],
@@ -19903,8 +19972,8 @@ identifiers are not checked. A file whose first five lines hold
   --help, -h      Show this help.
 
 Exit codes: 0 when no finding (Markdown mode) or always (--prose mode). 1 when a
-Markdown finding exists. 2 on a usage error, when no Markdown file was examined,
-or when a file is not UTF-8.
+Markdown finding exists. 2 on a usage error, when a path does not exist, when no
+Markdown file was examined, or when a file is not UTF-8.
 `;
 var HISTORICAL_MARKER = "<!-- ste:historical-record -->";
 function readText(path) {
@@ -19988,7 +20057,10 @@ async function run7(argv, io) {
   let total = 0;
   let checked = 0;
   let skipped = 0;
+  const missing = [];
   for (const arg of argv) {
+    if (!existsSync13(arg))
+      missing.push(arg);
     for (const target of targetsFor(arg)) {
       const name2 = basename10(target);
       if (isHistoricalRecord(target)) {
@@ -20021,12 +20093,16 @@ async function run7(argv, io) {
   const tail = skipped > 0 ? `; ${skipped} skipped as historical record(s)` : "";
   io.stdout(`
 TOTAL: ${total} problem(s) in ${checked} file(s)${tail}
+`);
+  for (const path of missing)
+    io.stderr(`ERROR: path does not exist: ${path}
 `);
   if (checked === 0) {
     io.stderr(`ERROR: no Markdown file was examined. This is a failure, not a clean result -- check the path(s) given.
 `);
-    return 2;
   }
+  if (missing.length > 0 || checked === 0)
+    return 2;
   return total > 0 ? 1 : 0;
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danielsimonjr/repo-tools",
-  "version": "2.2.5",
+  "version": "2.2.7",
   "description": "One CLI for repository analysis: dependency graphs creation and query with per-export API facts, file chunking, and context compression. TypeScript on Bun, compiled to a single executable.",
   "type": "module",
   "license": "MIT",