@ecoma-io/archkeep 0.23.0 → 0.24.1

package/src/intent/intent-manifest.json

@@ -20,7 +20,7 @@
           "type": "architecture-test",
           "path": "src/conformance/boundary.test.mjs",
           "assertion": "SHIPPED_PACKAGES allow-list prevents unapproved dependencies; specifiersIn() walk catches all imports including createRequire; no provider import found in core layers",
-          "sha256": "12d4e179f88de2622a41e56f42b2bab1c88a677ec2232430752138c57ba5fb18"
+          "sha256": "e4e24ed9f36e31126249c7d079b3656d17ac78ee4f12033321467ffa6666f033"
         },
         {
           "type": "source-evidence",
@@ -104,13 +104,13 @@
           "type": "source-evidence",
           "path": "src/commands/graph.mjs",
           "assertion": "Plain string comparison, never localeCompare; INTERNAL_DATA_FIELDS stripped; SCHEMA_VERSION = 2",
-          "sha256": "25ece527ff5ea847f5863314a12c50c95a80f995d36d3aeb756ca022dd72f283"
+          "sha256": "436902ab3ca994c233318437a896630f969a2c8a22b0f93e598d309faed39d49"
         },
         {
           "type": "source-evidence",
           "path": "src/commands/graph.mjs",
           "assertion": "computePolicyFingerprint produces SHA-256 of canonicalized policy",
-          "sha256": "25ece527ff5ea847f5863314a12c50c95a80f995d36d3aeb756ca022dd72f283"
+          "sha256": "436902ab3ca994c233318437a896630f969a2c8a22b0f93e598d309faed39d49"
         }
       ],
       "status": "proven"
@@ -248,7 +248,7 @@
           "type": "source-evidence",
           "path": "src/commands/context-command.mjs",
           "assertion": "coverage.notes warns that per-edge violations cover only depConstraints (3 of 15 violation types)",
-          "sha256": "1f054400e864c381e65904bc20095f4eb6df3198b19ffcb985895af7ec31d80d"
+          "sha256": "f906671ff4a5720d80ea5a3dab2779f1cfd3648fbc1e6a1f3a6d0e6efef21feb"
         },
         {
           "type": "source-evidence",
@@ -286,7 +286,7 @@
           "type": "source-evidence",
           "path": "src/commands/graph.mjs",
           "assertion": "Plain string comparison throughout; never localeCompare",
-          "sha256": "25ece527ff5ea847f5863314a12c50c95a80f995d36d3aeb756ca022dd72f283"
+          "sha256": "436902ab3ca994c233318437a896630f969a2c8a22b0f93e598d309faed39d49"
         }
       ],
       "status": "proven"
@@ -312,7 +312,7 @@
           "type": "source-evidence",
           "path": "src/providers/moon.mjs",
           "assertion": "inferWorkspaceLayout returns null for partial layouts — same all-or-nothing contract as Nx and Native",
-          "sha256": "f461cfaa2c8267a78fa9afa4b3267e92be4a85ff5f370b9e4b613a1fb85fc211"
+          "sha256": "2b874da56bda5bc6cfcfb4a984556ace0eafe3a6a98d964c78204bd96cc30e48"
         },
         {
           "type": "behavioral-test",

package/src/intent/mask-non-code.mjs

@@ -536,6 +536,7 @@ function scanTemplate(src, i) {
  *   same line structure.
  */
 export function maskNonCode(src) {
+  // used by its own test
   /**
    * Chunks of the result, joined once at the end. Code is copied through in
    * RUNS rather than a token at a time — `plainFrom` is where the current

package/src/lsp/diagnostics.mjs

@@ -38,7 +38,7 @@ import { DIAGNOSTIC_SEVERITY, SERVER_INFO } from "./protocol.mjs";
  * prints beside the message to say which tool spoke. Taken from the server's
  * own identity so the two can never disagree.
  */
-export const DIAGNOSTIC_SOURCE = SERVER_INFO.name;
+export const DIAGNOSTIC_SOURCE = SERVER_INFO.name; // used by its own test
 
 /**
  * The `code` on a diagnostic that reports the ABSENCE of a verdict rather than
@@ -46,7 +46,7 @@ export const DIAGNOSTIC_SOURCE = SERVER_INFO.name;
  * against `MESSAGE_IDS` at load so a future upstream id cannot silently collide
  * with it.
  */
-export const ANALYSIS_FAILURE_CODE = "analysisFailure";
+export const ANALYSIS_FAILURE_CODE = "analysisFailure"; // used by its own test
 
 if (MESSAGE_IDS.includes(ANALYSIS_FAILURE_CODE)) {
   throw new Error(
@@ -100,6 +100,7 @@ const QUOTES = new Set(['"', "'", "`"]);
  * @returns {{start: {line: number, character: number}, end: {line: number, character: number}}}
  */
 export function rangeAt(at, lines) {
+  // used by its own test
   // A failure about the file as a whole carries no position (`contract.md`
   // fixes it as an explicit `null`). It gets the first line, whole: a
   // zero-width range at the origin renders as an invisible caret in most

package/src/lsp/protocol.mjs

@@ -95,7 +95,7 @@ export const DIAGNOSTIC_SEVERITY = Object.freeze({
  * more is not a supported conversation, and saying so beats waiting for bytes
  * that will never arrive.
  */
-export const MAX_CONTENT_LENGTH = 64 * 1024 * 1024;
+export const MAX_CONTENT_LENGTH = 64 * 1024 * 1024; // used by its own test
 
 /**
  * LSP `MessageType`, as `window/showMessage` reports it.
@@ -211,5 +211,6 @@ export function uriToPath(uri) {
  * @returns {string}
  */
 export function pathToUri(path) {
+  // used by its own test
   return pathToFileURL(path).href;
 }

package/src/lsp/server.mjs

@@ -292,6 +292,7 @@ const POLYGLOT_GRAPH_MANIFESTS = Object.freeze([
  * @returns {readonly string[]}
  */
 export function watchedFilesFor(options, { unresolved = false } = {}) {
+  // used by its own test
   return Object.freeze([
     ...(typeof options.boundaryConfig === "string" ? [options.boundaryConfig] : []),
     ...(unresolved
@@ -376,6 +377,7 @@ function markersAt(root) {
  *   convention chain can find while carrying files that need one.
  */
 export function readWorkspaceOptions(root) {
+  // used by its own test
   const { hasNx, hasNative } = markersAt(root);
   if (hasNx && hasNative) {
     throw new Error(
@@ -450,6 +452,7 @@ const WATCHER_REGISTRATION_ID = "archkeep/watched-files";
  * text than the one on screen.
  */
 export const SERVER_CAPABILITIES = Object.freeze({
+  // used by its own test
   textDocumentSync: Object.freeze({
     openClose: true,
     change: TEXT_DOCUMENT_SYNC_KIND.full,

package/src/lsp/workspace-index.mjs

@@ -156,7 +156,7 @@ export { PROJECT_CONFIG_FILE, nodeTypeOf, buildDependencies };
  * @returns {object} Whatever the JSON describes.
  * @throws {Error} when neither parser can read it.
  */
-export const parseProjectJson = parseNxJson;
+const parseProjectJson = parseNxJson;
 
 /**
  * Every file git considers part of the working tree, workspace-relative and
@@ -205,6 +205,7 @@ const directoryOf = (file) => {
  * @returns {{projects: {name: string, root: string, config: object}[], skipped: {file: string, reason: string}[]}}
  */
 export function discoverProjects({ files, readFile }) {
+  // used by its own test
   const projects = [];
   const skipped = [];
   for (const file of files) {
@@ -263,6 +264,7 @@ export function discoverProjects({ files, readFile }) {
  *   publishes the collision through `indexGaps`.
  */
 export function buildNodes(projects) {
+  // used by its own test
   // Null-prototype for the same reason `../providers/native/graph.mjs` and
   // `../providers/moon.mjs` use them: every key here is a project NAME, and
   // project names come from a `project.json`'s own `name` field —

package/src/providers/moon.mjs

@@ -31,6 +31,7 @@ import { existsSync } from "node:fs";
 import { delimiter, join, posix } from "node:path";
 
 import { environmentForTree, runProcess } from "../process.mjs";
+import { isEnoent } from "../errors.mjs";
 import { buildDependencies } from "./native/graph.mjs";
 
 /**
@@ -905,16 +906,15 @@ function resolveMoonCli(workspaceRoot, { resolveMoon = () => "moon" } = {}) {
  * `nxCli` guards against from the other direction, where only a
  * `MODULE_NOT_FOUND` earns the "not installed" story.
  *
- * `../process.mjs`'s `runProcess` wraps the child's failure and carries the
- * original on `cause`, so the code is read from there; the direct `code` is
- * read too, for a `run` seam that surfaces a spawn error unwrapped.
+ * The shape test itself is `../errors.mjs`'s `isEnoent` — the name stays
+ * because what this file asks is not "is this ENOENT" but "does this failure
+ * carry an install action".
  *
  * @param {unknown} error
  * @returns {boolean}
  */
 function isMoonBinaryMissing(error) {
-  const thrown = /** @type {{code?: unknown, cause?: {code?: unknown}}|null|undefined} */ (error);
-  return thrown?.code === "ENOENT" || thrown?.cause?.code === "ENOENT";
+  return isEnoent(error);
 }
 
 /**

package/src/providers/native/differential.fixtures.mjs

@@ -41,11 +41,13 @@ import { nativeProvider } from "./index.mjs";
  * @param {Record<string, string>} files `{relativePath: text}`.
  */
 export function writeAll(write, files) {
+  // used by its own test
   for (const [path, text] of Object.entries(files)) write(path, text);
 }
 
 /** @param {string} root */
 export function writeIn(root) {
+  // used by its own test
   return (relativePath, text) => {
     mkdirSync(join(root, relativePath, ".."), { recursive: true });
     writeFileSync(join(root, relativePath), text);
@@ -53,7 +55,7 @@ export function writeIn(root) {
 }
 
 /** @param {string} root */
-export function readFileFrom(root) {
+function readFileFrom(root) {
   return (path) => {
     try {
       return readFileSync(join(root, path), "utf8");
@@ -76,6 +78,7 @@ function analyze(root, graph, files, tsConfig) {
 
 /** `{name → {root, type, tags}}`, comparable whichever provider produced it. */
 export const nodeShape = (nodes) =>
+  // used by its own test
   Object.fromEntries(
     Object.entries(nodes).map(([name, node]) => [
       name,
@@ -105,13 +108,14 @@ export const nodeShape = (nodes) =>
  * canonicalisation consistent with the code under test rather than a weaker
  * copy of it. */
 export const dependencyShape = (dependencies) =>
+  // used by its own test
   Object.values(dependencies)
     .flat()
     .map(({ source, target, type }) => JSON.stringify([source, target, type]))
     .sort();
 
 /** `["messageId sourceFile:line:column", ...]`, sorted the same way. */
-export const verdictShape = (violations) =>
+const verdictShape = (violations) =>
   violations.map((v) => `${v.messageId} ${v.sourceFile}:${v.line}:${v.column}`).sort();
 
 /**
@@ -128,6 +132,7 @@ export const verdictShape = (violations) =>
  * @type {typeof runProcess}
  */
 export function runNxGraphSpawn(file, args, cwd) {
+  // used by its own test
   const previous = process.env.NX_DAEMON;
   process.env.NX_DAEMON = "false";
   try {
@@ -157,6 +162,7 @@ export function runNxGraphSpawn(file, args, cwd) {
  * @throws {Error} naming every failing file, when either list is non-empty.
  */
 export function assertNoAnalysisFailures(nxFailures, nativeFailures) {
+  // used by its own test
   const named = [
     ...nxFailures.map((f) => `nx:${f.sourceFile} (${f.reason})`),
     ...nativeFailures.map((f) => `native:${f.sourceFile} (${f.reason})`),
@@ -188,6 +194,7 @@ export function assertNoAnalysisFailures(nxFailures, nativeFailures) {
  *   inherits the `NX_DAEMON` handling rather than choosing between the two.
  */
 export async function runBothProviders({
+  // used by its own test
   nxRoot,
   nxFiles,
   nativeRoot,
@@ -280,6 +287,7 @@ export async function runBothProviders({
  * @returns {{kind: string, subject: string, field: string, nx: unknown, native: unknown}[]}
  */
 export function diffGraphs(nxSide, nativeSide) {
+  // used by its own test
   const rows = [];
 
   const nxNames = new Set(Object.keys(nxSide.nodes));
@@ -417,6 +425,7 @@ export function diffGraphs(nxSide, nativeSide) {
  * detection) are scoped to one fixture pair and cannot silently absorb a
  * disagreement in another. */
 export function namespaced(rows, label) {
+  // used by its own test
   return rows.map((row) => ({ ...row, subject: `${label}:${row.subject}` }));
 }
 
@@ -458,7 +467,7 @@ export function namespaced(rows, label) {
  * `classifyDifferences`, not a search for every place `"native-only"` was
  * spelled as a bare string.
  */
-export const LEDGER_DIRECTIONS = Object.freeze(/** @type {const} */ (["native-only"]));
+const LEDGER_DIRECTIONS = Object.freeze(/** @type {const} */ (["native-only"]));
 
 /**
  * Every entry here is a **difference native reports and Nx does not** — the
@@ -479,7 +488,7 @@ export const LEDGER_DIRECTIONS = Object.freeze(/** @type {const} */ (["native-on
  *
  * @type {readonly LedgerRow[]}
  */
-export const LEDGER = Object.freeze([]);
+export const LEDGER = Object.freeze([]); // used by its own test
 
 /**
  * Every fixture-pair label this file actually runs a comparison over — the
@@ -487,7 +496,7 @@ export const LEDGER = Object.freeze([]);
  * list every `assertPairAgrees`/`pairProblems` call site draws its `label`
  * argument from, so the two can never drift into naming different pairs.
  */
-export const PAIR_LABELS = Object.freeze(["simple", "composite", "layout"]);
+export const PAIR_LABELS = Object.freeze(["simple", "composite", "layout"]); // used by its own test
 
 /**
  * The direction a `diffGraphs` row itself claims, in `LedgerRow.direction`'s
@@ -560,6 +569,7 @@ function differenceDirection(difference) {
  *   difference no ledger row may ever cover.
  */
 export function classifyDifferences(differences, ledger) {
+  // used by its own test
   for (const row of ledger) {
     if (!row.reason?.trim()) {
       throw new Error(`ledger row for "${row.subject}"/"${row.field}" has an empty reason`);
@@ -632,6 +642,7 @@ export function classifyDifferences(differences, ledger) {
  * @returns {string[]}
  */
 export function emptyVerdictBreaches(label, counts) {
+  // used by its own test
   const breaches = [];
   for (const [engine, count] of Object.entries(counts)) {
     if (count === 0) {
@@ -659,6 +670,7 @@ export function emptyVerdictBreaches(label, counts) {
  * @returns {string[]}
  */
 export function perMessageBreaches(label, nxViolations, nativeViolations) {
+  // used by its own test
   /** @param {{messageId: string}[]} violations @returns {Map<string, number>} */
   const countBy = (violations) => {
     const counts = new Map();
@@ -696,6 +708,7 @@ export function perMessageBreaches(label, nxViolations, nativeViolations) {
  * @returns {string[]}
  */
 export function pairProblems(label, result, ledger = LEDGER) {
+  // used by its own test
   const problems = [
     ...emptyVerdictBreaches(label, {
       nx: result.nx.violations.length,
@@ -734,6 +747,7 @@ export function pairProblems(label, result, ledger = LEDGER) {
  * @param {readonly LedgerRow[]} [ledger]
  */
 export function assertPairAgrees(label, result, ledger = LEDGER) {
+  // used by its own test
   const problems = pairProblems(label, result, ledger);
   if (problems.length > 0) {
     throw new Error(`${label}: provider differential disagreement:\n  ${problems.join("\n  ")}`);
@@ -758,6 +772,7 @@ export function assertPairAgrees(label, result, ledger = LEDGER) {
  * @returns {LedgerRow[]}
  */
 export function unknownLabelRows(ledger, knownLabels) {
+  // used by its own test
   return ledger.filter((row) => !knownLabels.includes(row.subject.split(":")[0]));
 }
 
@@ -773,7 +788,7 @@ export function unknownLabelRows(ledger, knownLabels) {
 // staying importable and spawn-free at import time for exactly that reuse,
 // and a second copy here is the drift `../../../../../AGENTS.md`'s "never
 // state a rule twice" rule exists to catch.
-export const SIMPLE_BOUNDARY_CONFIG = `export const depConstraints = [
+export const SIMPLE_BOUNDARY_CONFIG = `export const depConstraints = [ // used by its own test
   { sourceTag: "layer:domain", onlyDependOnLibsWithTags: ["layer:domain"] },
   { sourceTag: "layer:adapter", onlyDependOnLibsWithTags: ["layer:domain", "layer:adapter"] },
 ];
@@ -794,6 +809,7 @@ export const moduleBoundaryOptions = {
 // the wrong way (`domain` reaching into `adapter`). Exported for the same
 // reuse reason as `SIMPLE_BOUNDARY_CONFIG` above.
 export const SIMPLE_GO_FILES = {
+  // used by its own test
   "libs/domain/go.mod": "module example.com/domain\n\ngo 1.24\n",
   "libs/adapter/go.mod": "module example.com/adapter\n\ngo 1.24\n",
   "libs/adapter/adapter.go": 'package adapter\n\nvar Name = "adapter"\n',
@@ -819,6 +835,7 @@ var _ = adapter.Name
  * @returns {string[]}
  */
 export function buildSimpleNxTree(root, { boundaryConfig = "module-boundaries.config.mjs" } = {}) {
+  // used by its own test
   const write = writeIn(root);
   write(
     "nx.json",
@@ -851,7 +868,7 @@ export function buildSimpleNxTree(root, { boundaryConfig = "module-boundaries.co
  * @param {{boundaryConfig?: string}} [options]
  * @returns {string[]}
  */
-export function buildSimpleNativeTree(
+export function buildSimpleNativeTree( // used by its own test
   root,
   { boundaryConfig = "module-boundaries.config.mjs" } = {},
 ) {
@@ -950,7 +967,7 @@ var _ = parent.Name
  * @param {{boundaryConfig?: string}} [options]
  * @returns {string[]}
  */
-export function buildCompositeNxTree(
+export function buildCompositeNxTree( // used by its own test
   root,
   { boundaryConfig = "module-boundaries.config.mjs" } = {},
 ) {
@@ -1029,7 +1046,7 @@ export function buildCompositeNxTree(
  * @param {{boundaryConfig?: string}} [options]
  * @returns {string[]}
  */
-export function buildCompositeNativeTree(
+export function buildCompositeNativeTree( // used by its own test
   root,
   { boundaryConfig = "module-boundaries.config.mjs" } = {},
 ) {
@@ -1175,7 +1192,7 @@ export function buildCompositeNativeTree(
 const LAYOUT_BOUNDARY_CONFIG = `export const depConstraints = [
   { sourceTag: "layer:thing", onlyDependOnLibsWithTags: ["layer:thing"] },
 ];
-export const moduleBoundaryOptions = {
+export const moduleBoundaryOptions = { // used by its own test
   allow: [],
   buildTargets: ["build"],
   enforceBuildableLibDependency: false,
@@ -1227,6 +1244,7 @@ var _ = blocked.Name
  * @returns {string[]}
  */
 export function buildLayoutNxTree(root, { boundaryConfig = "module-boundaries.config.mjs" } = {}) {
+  // used by its own test
   const write = writeIn(root);
   write(
     "nx.json",
@@ -1262,7 +1280,7 @@ export function buildLayoutNxTree(root, { boundaryConfig = "module-boundaries.co
  * @param {{boundaryConfig?: string}} [options]
  * @returns {string[]}
  */
-export function buildLayoutNativeTree(
+export function buildLayoutNativeTree( // used by its own test
   root,
   { boundaryConfig = "module-boundaries.config.mjs" } = {},
 ) {

package/src/providers/native/index.mjs

@@ -61,6 +61,7 @@ function projectOfFactory(projects) {
  *   stale `coverage.exempt` row.
  */
 export function discover({ root, files, readFile }) {
+  // used by its own test
   const model = loadNativeModel(root, { readFile });
   const { projects, failures: discoveryFailures } = discoverNativeProjects({
     root,
@@ -128,7 +129,7 @@ export function discover({ root, files, readFile }) {
  * @param {{discovered: ReturnType<typeof discover>, importSites: object[]}} args
  * @returns {{nodes: Record<string, object>, dependencies: Record<string, object[]>, workspaceLayout?: {appsDir: string, libsDir: string}, exemptedFiles?: string[]}}
  */
-export function buildGraph({ discovered, importSites }) {
+function buildGraph({ discovered, importSites }) {
   return buildNativeGraph({
     projects: discovered.projects,
     importSites,

package/src/providers/native/model.mjs

@@ -191,6 +191,7 @@ function parseArchkeepJson(text) {
  * for a tree that kept Nx-shaped project boundaries without keeping Nx.
  */
 export const DEFAULT_MANIFEST_NAMES = Object.freeze([
+  // used by its own test
   "project.json",
   "package.json",
   "go.mod",
@@ -255,6 +256,7 @@ const PROJECT_TYPES = ["app", "lib", "e2e"];
  * @see DEFAULT_MANIFEST_NAMES
  */
 export const DEFAULT_INFER_EXCLUDE = Object.freeze([
+  // used by its own test
   "**/docs/**",
   "**/fixtures/**",
   "**/__fixtures__/**",
@@ -549,6 +551,7 @@ const TOP_LEVEL_KEYS = [
  * @returns {string[]}
  */
 export function findNativeModelViolations(raw) {
+  // used by its own test
   if (!isPlainObject(raw)) return [`archkeep.json: expected an object, got ${describe(raw)}`];
 
   const violations = [];
@@ -699,6 +702,7 @@ export function findNativeModelViolations(raw) {
  * @returns {object} `NativeModel` — see `./index.mjs`.
  */
 export function normalizeNativeModel(raw) {
+  // used by its own test
   // `raw.projects` may be absent — `findNativeModelViolations` above validates
   // that shape the same way it validates `projects: {}`, so this has to read
   // it back the same way rather than assume the key is always present.

package/src/report/envelope-shape.mjs

@@ -89,6 +89,7 @@ function typeName(value) {
  *   JSON document can carry.
  */
 export function envelopeFieldPaths(envelope) {
+  // used by its own test
   if (envelope === null || typeof envelope !== "object" || Array.isArray(envelope)) {
     throw new Error(
       `archkeep: refusing to build an envelope roster over ${envelope === null ? "null" : `a ${Array.isArray(envelope) ? "array" : typeof envelope}`} ` +
@@ -152,6 +153,7 @@ function walk(value, path, byPath) {
  * @returns {{added: string[], removed: string[]}}
  */
 export function compareFieldPaths(recorded, observed) {
+  // used by its own test
   const recordedSet = new Set(recorded);
   const observedSet = new Set(observed);
   return {

package/src/report/sarif.mjs

@@ -62,15 +62,15 @@ import { TSCONFIG_PATHS_MESSAGE_IDS, TSCONFIG_PATHS_MESSAGES } from "../tsconfig
 import { formatConstraint } from "./text.mjs";
 
 /** The schema every consumer of this file validates against. */
-export const SARIF_SCHEMA = "https://json.schemastore.org/sarif-2.1.0.json";
-export const SARIF_VERSION = "2.1.0";
+const SARIF_SCHEMA = "https://json.schemastore.org/sarif-2.1.0.json";
+const SARIF_VERSION = "2.1.0";
 
 /**
  * The single rule id every failing fitness function's result shares — see
  * `sarifRules()`'s own comment for why one id stands in for the whole
  * open-ended, workspace-declared set.
  */
-export const FITNESS_FAILED_RULE_ID = "fitnessFunctionFailed";
+export const FITNESS_FAILED_RULE_ID = "fitnessFunctionFailed"; // used by its own test
 
 /**
  * A workspace-relative path as a URI reference: each segment percent-encoded,
@@ -106,6 +106,7 @@ export const FITNESS_FAILED_RULE_ID = "fitnessFunctionFailed";
  * @returns {string}
  */
 export function toUriReference(path) {
+  // used by its own test
   return path.split("/").map(encodeURIComponent).join("/");
 }
 
@@ -139,6 +140,7 @@ export function toUriReference(path) {
  * @returns {object[]}
  */
 export function sarifRules(customCatalogue = []) {
+  // used by its own test
   return [
     ...MESSAGE_IDS.map((id) => ({
       id,
@@ -231,6 +233,7 @@ const CUSTOM_RULE_INDEX_BASE =
  * @returns {object}
  */
 export function sarifCustomRuleResult(finding, ruleIndex) {
+  // used by its own test
   const physicalLocation =
     finding.sourceFile === undefined
       ? null
@@ -283,6 +286,7 @@ export function sarifCustomRuleResult(finding, ruleIndex) {
  * @returns {object}
  */
 export function sarifCustomRuleNotification(decision) {
+  // used by its own test
   const posture =
     decision.verdict === "not_applicable" ? "did not apply to this run" : "could not be judged";
   return {
@@ -305,6 +309,7 @@ export function sarifCustomRuleNotification(decision) {
  * @returns {object}
  */
 export function sarifResult(violation) {
+  // used by its own test
   const detail =
     `Import ${JSON.stringify(violation.specifier)} (${violation.kind}) ` +
     `from ${violation.sourceProject ?? "(no project)"} ` +
@@ -364,7 +369,7 @@ export function sarifResult(violation) {
  * @param {object} finding A finding from `../go-work.mjs` `compareGoWork`.
  * @returns {object}
  */
-export function sarifGoWorkResult(finding) {
+function sarifGoWorkResult(finding) {
   const physicalLocation = { artifactLocation: { uri: toUriReference(finding.file) } };
   if (finding.line !== null) {
     physicalLocation.region = { startLine: finding.line, startColumn: finding.column };
@@ -395,7 +400,7 @@ export function sarifGoWorkResult(finding) {
  * @param {object} finding A finding from `../tsconfig-paths.mjs`.
  * @returns {object}
  */
-export function sarifTsconfigPathsResult(finding) {
+function sarifTsconfigPathsResult(finding) {
   return {
     ruleId: finding.messageId,
     ruleIndex:
@@ -444,7 +449,7 @@ export function sarifTsconfigPathsResult(finding) {
  *   `declaredEdgeViolationsForCheck`, extended with `file` — workspace-relative.
  * @returns {object}
  */
-export function sarifDeclaredEdgeResult(finding) {
+function sarifDeclaredEdgeResult(finding) {
   return {
     ruleId: finding.messageId,
     ruleIndex: MESSAGE_IDS.indexOf(finding.messageId),
@@ -472,6 +477,7 @@ export function sarifDeclaredEdgeResult(finding) {
  * @returns {object}
  */
 export function sarifIntentResult(finding) {
+  // used by its own test
   return {
     ruleId: finding.rule,
     ruleIndex:
@@ -516,6 +522,7 @@ export function sarifIntentResult(finding) {
  * @returns {object}
  */
 export function sarifFitnessResult(decision, policySource = null) {
+  // used by its own test
   const result = {
     ruleId: FITNESS_FAILED_RULE_ID,
     ruleIndex:
@@ -548,6 +555,7 @@ export function sarifFitnessResult(decision, policySource = null) {
  * @returns {object}
  */
 export function sarifFitnessNotification(decision) {
+  // used by its own test
   return {
     level: "warning",
     message: {
@@ -567,7 +575,7 @@ export function sarifFitnessNotification(decision) {
  * @param {object} failure An `AnalysisFailure`.
  * @returns {object}
  */
-export function sarifNotification(failure) {
+function sarifNotification(failure) {
   const physicalLocation = { artifactLocation: { uri: toUriReference(failure.sourceFile) } };
   if (failure.line !== null) {
     physicalLocation.region = { startLine: failure.line, startColumn: failure.column };
@@ -601,6 +609,7 @@ export function sarifNotification(failure) {
  * @returns {object}
  */
 export function sarifIntentNotification(entry) {
+  // used by its own test
   return {
     level: "warning",
     message: {
@@ -641,6 +650,7 @@ const UNOWNED_SAMPLE_LIMIT = 10;
  * @returns {object}
  */
 export function sarifCoverageGapNotification(gap) {
+  // used by its own test
   const manifests = gap.manifests ?? [];
   const found = manifests.length > 0 ? `: ${manifests.join(", ")}` : "";
   if (gap.kind === "unregistered-plugin") {
@@ -760,7 +770,7 @@ export function sarifCoverageGapNotification(gap) {
  * @param {string} decisionRef The cited value, exactly as the row spelled it.
  * @returns {object}
  */
-export function sarifDecisionRefNotification(decisionRef) {
+function sarifDecisionRefNotification(decisionRef) {
   return {
     level: "warning",
     message: {
@@ -853,6 +863,7 @@ export function sarifDecisionRefNotification(decisionRef) {
  * @returns {object} A SARIF 2.1.0 log, ready to `JSON.stringify`.
  */
 export function buildSarifLog({
+  // used by its own test
   violations,
   failures,
   goWork,
@@ -1016,6 +1027,7 @@ export function buildSarifLog({
  * @returns {object}
  */
 export function sarifDeltaResult(entry, site) {
+  // used by its own test
   const target = entry.targetIsSpecifier
     ? `specifier ${JSON.stringify(entry.target)}`
     : entry.target;
@@ -1102,6 +1114,7 @@ export function sarifDeltaResult(entry, site) {
  * @returns {object} A SARIF 2.1.0 log, ready to `JSON.stringify`.
  */
 export function buildDeltaSarifLog({ delta, coverage, customCatalogue = [] }) {
+  // used by its own test
   const customRuleIndex = new Map(
     customCatalogue.map((entry, index) => [entry.ruleId, CUSTOM_RULE_INDEX_BASE + index]),
   );

package/src/report/snapshot-text.mjs

@@ -35,7 +35,7 @@ export function sanitize(text) {
  * @param {{name: string, root: string, tags: string[]}} project
  * @returns {string}
  */
-export function formatProject(project) {
+function formatProject(project) {
   const tags =
     project.tags.length > 0 ? ` [${project.tags.map((t) => sanitize(t)).join(", ")}]` : "";
   return `  ${sanitize(project.name)}  ${sanitize(project.root)}${tags}`;
@@ -47,7 +47,7 @@ export function formatProject(project) {
  * @param {{source: string, target: string, type: string}} edge
  * @returns {string}
  */
-export function formatEdge(edge) {
+function formatEdge(edge) {
   return `  ${sanitize(edge.source)} → ${sanitize(edge.target)} (${sanitize(edge.type)})`;
 }
 
@@ -57,7 +57,7 @@ export function formatEdge(edge) {
  * @param {{field: string, baseline: *, head: *}} change
  * @returns {string}
  */
-export function formatChange(change) {
+function formatChange(change) {
   const formatValue = (v) => {
     if (Array.isArray(v)) return v.length > 0 ? v.map((x) => sanitize(x)).join(", ") : "(none)";
     if (v === null || v === undefined) return "(none)";