@danielsimonjr/repo-tools 2.2.6 → 2.2.7

package/CHANGELOG.md

@@ -6,6 +6,26 @@ 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

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.6.
+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.6):
+[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.6",
+  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,
@@ -6378,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";
@@ -12435,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))
@@ -13631,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("");
@@ -14227,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
   });
@@ -14287,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}
 
 `;
@@ -14313,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)
@@ -14332,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/`."],
@@ -19975,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) {
@@ -20060,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)) {
@@ -20093,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.6",
+  "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",