@ecoma-io/archkeep 0.24.0 → 0.25.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ecoma-io/archkeep",
-  "version": "0.24.0",
+  "version": "0.25.0",
   "description": "Architecture authority for human and agentic software development — deterministic, evidence-backed enforcement of declared architecture.",
   "keywords": [
     "architecture",

package/src/analysis/csharp.mjs

@@ -188,7 +188,7 @@ function withoutGenericArguments(rhs) {
  * @returns {{ specifier: string, importableName: string|null, specifierStartInBody: number }|null} `null`
  *   when the body is a using STATEMENT's shape, not a directive's.
  */
-export function classifyUsingBody(body) {
+function classifyUsingBody(body) {
   const trimmed = body.trim();
   if (trimmed === "") return null;
   const staticForm = STATIC_FORM.exec(trimmed);
@@ -236,6 +236,7 @@ export function classifyUsingBody(body) {
  * @returns {{ specifier: string, importableName: string|null, offset: number }[]}
  */
 export function parseCSharpDirectiveSites(csharpText) {
+  // used by its own test
   const source = maskCSharpComments(csharpText);
   const sites = [];
   for (const match of source.matchAll(CS_USING_BODY)) {
@@ -305,6 +306,7 @@ export function parseCSharpDirectiveSites(csharpText) {
  *   each naming its line. Empty when the directives read fully.
  */
 export function csharpDirectiveMalformations(csharpText) {
+  // used by its own test
   const source = maskCSharpComments(csharpText);
   /** @type {string[]} */
   const reasons = [];

package/src/analysis/dotnet/csproj.mjs

@@ -122,6 +122,7 @@ function collectElements(node, name) {
  *            { project?: undefined, reason: string }}
  */
 export function parseCsproj(text) {
+  // used by its own test
   const { parser, validate, error } = xmlParser();
   if (parser === null) {
     return { reason: `${XML_PARSER} is unavailable (${error})` };
@@ -153,6 +154,7 @@ export function parseCsproj(text) {
  * @returns {{ paths: string[], problems: string[] }}
  */
 export function projectReferenceFacts(project, csprojDir) {
+  // used by its own test
   const paths = [];
   const problems = [];
   for (const ref of collectElements(project, "ProjectReference")) {
@@ -184,6 +186,7 @@ export function projectReferenceFacts(project, csprojDir) {
  * @returns {string[]}
  */
 export function usingNamespacesOf(project) {
+  // used by its own test
   const namespaces = [];
   for (const item of collectElements(project, "Using")) {
     const include = textOf(item["@_Include"] ?? item.Include);
@@ -215,6 +218,7 @@ export function usingNamespacesOf(project) {
  *            { entry?: undefined, problems?: undefined, reason: string }}
  */
 export function csprojEntryOf(projectName, csprojPath, text) {
+  // used by its own test
   const parsed = parseCsproj(text);
   if (parsed.reason !== undefined) return { reason: parsed.reason };
   // A manifest at the workspace root has no separator: `lastIndexOf` answers
@@ -251,7 +255,7 @@ export function csprojEntryOf(projectName, csprojPath, text) {
  *             usingEdges: { source: string, target: string, sourceFile: string, type: string }[],
  *             failures: { sourceFile: string, line: null, column: null, reason: string }[] }}
  */
-export const csprojModelOf = perWorkspace(({ projects, filesOf, readFile }) => {
+const csprojModelOf = perWorkspace(({ projects, filesOf, readFile }) => {
   const entries = [];
   const failures = [];
   const identity = new Map();

package/src/analysis/dotnet/namespaces.mjs

@@ -62,6 +62,7 @@ import { maskCSharpComments } from "./mask.mjs";
  *   which declares no name the index can carry.
  */
 export function parseCSharpNamespaceDeclarations(maskedText) {
+  // used by its own test
   const CS_NAMESPACE_DECLARATION =
     /(?:^\uFEFF?|\n)[ \t]*namespace[ \t]+([\p{L}_][\p{L}\p{Nd}_]*(?:[ \t]*\.[ \t]*[\p{L}_][\p{L}\p{Nd}_]*)*)[ \t]*(?:;|(?=[{;\r\n]|$))/gu;
   const declarations = [];

package/src/analysis/go.mjs

@@ -132,6 +132,7 @@ import {
  * character shifts nothing any record points at.
  */
 export function parseGoModulePath(goModText) {
+  // used by its own test
   const match = goModText.replace(/^\uFEFF/, "").match(/^module\s+(\S+)/m);
   if (!match) return null;
   const token = match[1];
@@ -170,6 +171,7 @@ const LEXICAL_START = /\/\/|\/\*|["'`]/g;
  * @returns {string} Same length as `goText`.
  */
 export function maskGoComments(goText) {
+  // used by its own test
   const scan = new RegExp(LEXICAL_START.source, "g");
   let masked = "";
   let copied = 0;
@@ -298,6 +300,7 @@ const GO_ALIAS_AT = /[\p{L}_.][\p{L}\p{Nd}_.]*\s+/uy;
  *   seen on. Empty when the imports read fully.
  */
 export function goImportMalformations(goText) {
+  // used by its own test
   const source = blankGoStringContents(maskGoComments(goText));
   /** @type {string[]} */
   const reasons = [];
@@ -417,6 +420,7 @@ const GO_IMPORT_ALIAS = "[\\p{L}_.][\\p{L}\\p{Nd}_.]*";
  * @returns {{ specifier: string, offset: number }[]}
  */
 export function parseGoImportSites(goText) {
+  // used by its own test
   const source = maskGoComments(goText);
   const sites = [];
   // import "p" | import alias "p" | import _ "p" | import . "p" — the path
@@ -457,6 +461,7 @@ export function parseGoImportSites(goText) {
 
 /** Every import path in a .go file (single-form and block-form), deduped. */
 export function parseGoImports(goText) {
+  // used by its own test
   return [...new Set(parseGoImportSites(goText).map((site) => site.specifier))];
 }
 
@@ -604,6 +609,7 @@ const isUnderModule = (importPath, modulePath) =>
  *   no module claims `importPath`.
  */
 export function resolveGoModule(importPath, modules) {
+  // used by its own test
   let matched = "";
   let project = null;
   for (const { modulePath, project: target } of modules) {

package/src/analysis/java.mjs

@@ -69,6 +69,7 @@ import {
  * @returns {{ specifier: string, importableName: string, offset: number }[]}
  */
 export function parseJavaImportSites(javaText) {
+  // used by its own test
   const source = maskJavaComments(javaText);
   // Anchored to a line head — through a leading UTF-8 BOM, matched rather than
   // stripped so offsets keep indexing the bytes on disk, the same anchor
@@ -132,6 +133,7 @@ export function parseJavaImportSites(javaText) {
  *   read fully.
  */
 export function javaImportMalformations(javaText) {
+  // used by its own test
   const source = maskJavaComments(javaText);
   const JAVA_IMPORT_HEAD = /(?:^\uFEFF?|[\n;])[ \t]*(?:import[ \t]+)/gu;
   /** @type {string[]} */

package/src/analysis/jvm/gradle.mjs

@@ -90,6 +90,7 @@ import { fileFailure, perWorkspace, refuseUnreadTree } from "../source-util.mjs"
  *            { rootProjectName?: undefined, includedProjects?: undefined, reason: string }}
  */
 export function parseGradleSettings(text) {
+  // used by its own test
   // First, remove block comments to avoid matching include statements inside them
   let processedText = text.replace(/\/\*[\s\S]*?\*\//g, "");
 
@@ -187,6 +188,7 @@ export function parseGradleSettings(text) {
  *            { projectDependencies?: undefined, reason: string }}
  */
 export function parseGradleBuild(text) {
+  // used by its own test
   const lines = text.split(/\r?\n/);
   const projectDependencies = [];
   let inString = false;
@@ -480,7 +482,7 @@ function buildGradleModel(workspace) {
   return { entries, pathToDirectory, failures };
 }
 
-export const gradleModelOf = perWorkspace(buildGradleModel);
+const gradleModelOf = perWorkspace(buildGradleModel);
 
 /**
  * Manifest-edge resolver: one edge per declared project dependency whose

package/src/analysis/jvm/maven.mjs

@@ -128,6 +128,7 @@ const textOf = (value) => {
  *            { project?: undefined, reason: string }}
  */
 export function parsePomProject(text) {
+  // used by its own test
   const { parser, validate, error } = xmlParser();
   if (parser === null) {
     return { reason: `${XML_PARSER} is unavailable (${error})` };
@@ -179,6 +180,7 @@ export function parsePomProject(text) {
  * @returns {{ entry: PomEntry, reason?: undefined } | { entry?: undefined, reason: string }}
  */
 export function pomEntryOf(projectName, pomPath, text) {
+  // used by its own test
   const parsed = parsePomProject(text);
   if (parsed.reason !== undefined) return { reason: parsed.reason };
   const project = parsed.project;
@@ -252,6 +254,7 @@ export function pomEntryOf(projectName, pomPath, text) {
  * @returns {string|null}
  */
 export function parentPomPath(pomPath, relativePath) {
+  // used by its own test
   const dir = pomPath.includes("/") ? pomPath.slice(0, pomPath.lastIndexOf("/")) : "";
   const direct = resolveWithinWorkspace(dir, relativePath);
   if (direct === null) return null;
@@ -270,6 +273,7 @@ export function parentPomPath(pomPath, relativePath) {
  * @returns {Record<string, string>}
  */
 export function mavenConfigProperties(readFile, candidatePaths) {
+  // used by its own test
   const props = /** @type {Record<string, string>} */ ({});
   for (const path of candidatePaths) {
     const text = readFile(path);
@@ -300,6 +304,7 @@ export function mavenConfigProperties(readFile, candidatePaths) {
  * @returns {{ value: string, resolved: boolean }}
  */
 export function interpolateCoordinate(value, props, builtins) {
+  // used by its own test
   let resolved = true;
   const out = value.replace(/\$\{([^}]+)\}/g, (_, key) => {
     if (Object.hasOwn(builtins, key)) return builtins[key];
@@ -557,7 +562,7 @@ function buildMavenModel(workspace) {
   return { entries, identityHolders, failures };
 }
 
-export const mavenModelOf = perWorkspace(buildMavenModel);
+export const mavenModelOf = perWorkspace(buildMavenModel); // used by its own test
 
 /**
  * Manifest-edge resolver: one edge per declared dependency whose coordinates

package/src/analysis/jvm/packages.mjs

@@ -81,6 +81,7 @@ import { maskJavaComments, maskKotlinComments } from "./mask.mjs";
  *   package file, which declares no name the index can carry.
  */
 export function parseJvmPackageDeclaration(maskedText) {
+  // used by its own test
   // An identifier segment starts with a letter (Unicode, via \p{L}), `_`, or
   // `$`, continues with those plus digits; segments join on optional spaces
   // around the dot, because `com . example` is legal if absurd. The match is

package/src/analysis/jvm/resolve.mjs

@@ -40,7 +40,7 @@ import { resolveJvmPackagePrefix } from "./packages.mjs";
  * their own explicit imports, which is why an explicit `import java.util.List`
  * classifies as ordinary external rather than by this table.
  */
-export const JAVA_DEFAULT_IMPORT_ROOTS = Object.freeze(["java.lang"]);
+export const JAVA_DEFAULT_IMPORT_ROOTS = Object.freeze(["java.lang"]); // used by its own test
 
 /**
  * Kotlin's default imports (kotlinlang.org, "Default imports", page dated
@@ -50,6 +50,7 @@ export const JAVA_DEFAULT_IMPORT_ROOTS = Object.freeze(["java.lang"]);
  * testable classification of names the compiler brings in unasked.
  */
 export const KOTLIN_DEFAULT_IMPORT_ROOTS = Object.freeze([
+  // used by its own test
   "kotlin",
   "kotlin.annotation",
   "kotlin.collections",
@@ -70,7 +71,7 @@ const DEFAULT_IMPORT_ROOTS_BY_LANGUAGE = {
 };
 
 /** The default-import roots a language's files carry, or `[]` when unknown. */
-export const defaultImportRootsFor = (language) => DEFAULT_IMPORT_ROOTS_BY_LANGUAGE[language] ?? [];
+export const defaultImportRootsFor = (language) => DEFAULT_IMPORT_ROOTS_BY_LANGUAGE[language] ?? []; // used by its own test
 
 /**
  * True when `specifier` falls under one of `roots`: equal to a root or a dot-
@@ -83,6 +84,7 @@ export const defaultImportRootsFor = (language) => DEFAULT_IMPORT_ROOTS_BY_LANGU
  * @returns {boolean}
  */
 export const underAnyRoot = (specifier, roots) =>
+  // used by its own test
   roots.some((root) => specifier === root || specifier.startsWith(`${root}.`));
 
 /**

package/src/analysis/kotlin.mjs

@@ -69,6 +69,7 @@ const KOTLIN_IMPORT = new RegExp(
  * @returns {{ specifier: string, importableName: string, offset: number }[]}
  */
 export function parseKotlinImportSites(kotlinText) {
+  // used by its own test
   // A shebang needs no handling of its own: `#!…` cannot anchor an import
   // match (`import` must follow a line head, `;`, or newline), and masking
   // runs before anything reads the text anyway.
@@ -112,6 +113,7 @@ const importableNameOf = (name) => (name.endsWith(".*") ? name.slice(0, -2) : na
  * @returns {string[]} Reasons, at most one per malformation kind.
  */
 export function kotlinImportMalformations(kotlinText) {
+  // used by its own test
   const source = maskKotlinComments(kotlinText);
   const KOTLIN_IMPORT_HEAD = /(?:^\uFEFF?|[\n;])[ \t]*(?:import[ \t]+)/gu;
   /** @type {string[]} */

package/src/analysis/markdown.mjs

@@ -259,6 +259,7 @@ export function foldMarkdownTrack({ tracked, owned, readFile, workspace, markdow
  * @returns {string[]}
  */
 export function markdownIncludedFiles({ include, tracked }) {
+  // used by its own test
   return tracked.filter(
     (file) =>
       file.endsWith(MARKDOWN_EXTENSION) && include.some((pattern) => matchesInclude(file, pattern)),

package/src/analysis/python.mjs

@@ -227,17 +227,20 @@ import {
 
 /** PEP 503 name normalization: case-insensitive, runs of `-_.` collapse to `-`. */
 export function normalizePackageName(name) {
+  // used by its own test
   return name.toLowerCase().replace(/[-_.]+/g, "-");
 }
 
 /** The package name a PEP 508 requirement string refers to, or null. */
 export function parseRequirementName(requirement) {
+  // used by its own test
   const match = requirement.trim().match(/^[A-Za-z0-9](?:[A-Za-z0-9._-]*[A-Za-z0-9])?/);
   return match ? normalizePackageName(match[0]) : null;
 }
 
 /** Every dependency name a pyproject manifest declares, deduped. */
 export function collectDeclaredDependencies(manifest) {
+  // used by its own test
   const names = new Set();
   const groups = [
     manifest.project?.dependencies ?? [],
@@ -630,6 +633,7 @@ const READ_BUILD_BACKENDS = new Set([
  * @returns {{ directories: string[], unmodelled: string[] }}
  */
 export function pythonPackageLayout(manifestText) {
+  // used by its own test
   if (manifestText === null) return { directories: [], unmodelled: [] };
   const manifest = parseManifest(manifestText);
   if (manifest === null) {
@@ -730,6 +734,7 @@ function ownPackageOf(file, projectRoot, directories) {
  * @returns {Map<string, { file: string|null, namespace: boolean }>}
  */
 export function pythonModuleIndex(projectRoot, files, directories = []) {
+  // used by its own test
   const index = new Map();
   for (const file of files) {
     if (!file.endsWith(".py")) continue;
@@ -755,6 +760,7 @@ export function pythonModuleIndex(projectRoot, files, directories = []) {
  * @returns {string[]}
  */
 export function pythonImportRoots(projectRoot, files, directories = []) {
+  // used by its own test
   return [...pythonModuleIndex(projectRoot, files, directories).keys()]
     .filter((name) => !name.includes("."))
     .sort();
@@ -1059,6 +1065,7 @@ function joinContinuedStatement(physicalLines, lineOffsets, index) {
  *   continuation?: boolean }[]}
  */
 export function parsePythonImportSites(pythonText) {
+  // used by its own test
   const sites = [];
   // Byte tolerance (`contract.md`): the lines a CRLF file splits into still
   // carry their `\r`, and a BOM-prefixed file's first line starts with
@@ -1166,6 +1173,7 @@ export function parsePythonImportSites(pythonText) {
  * @returns {string[]} Reasons, at most one per malformation kind.
  */
 export function pythonImportMalformations(pythonText) {
+  // used by its own test
   /** @type {string[]} */
   const reasons = [];
   // A bare `import` or `from` at the end of the file, or on a line with

package/src/analysis/rust.mjs

@@ -238,6 +238,7 @@ export function resolveRustDependencies(projects, filesOf, readFile) {
 
 /** Cargo's identifier spelling of a crate name: `-` and `.` become `_`. */
 export function crateIdentifier(name) {
+  // used by its own test
   return name.replace(/[-.]/g, "_");
 }
 
@@ -250,6 +251,7 @@ export function crateIdentifier(name) {
  * @returns {string|null}
  */
 export function crateImportName(manifest) {
+  // used by its own test
   const declared = manifest?.lib?.name ?? manifest?.package?.name;
   return typeof declared === "string" && declared !== "" ? crateIdentifier(declared) : null;
 }
@@ -471,7 +473,7 @@ function isOwnProjectPath(root, owner, byCrate) {
  * @param {string} path The `use` path, from the first non-space to the `;`.
  * @returns {{text: string, offset: number}[] | null}
  */
-export function braceGroupArms(path) {
+function braceGroupArms(path) {
   const open = path.indexOf("{");
   if (open === -1 || path.slice(0, open).trim() !== "") return null;
   /** @type {{text: string, offset: number}[]} */
@@ -520,6 +522,7 @@ export function braceGroupArms(path) {
  * with a brace group and names none.
  */
 export function useRootSegment(path) {
+  // used by its own test
   const match = /^\s*(?:::\s*)?([A-Za-z_]\w*)/.exec(path);
   return match ? match[1] : null;
 }
@@ -560,6 +563,7 @@ export function useRootSegment(path) {
  *   produces from reading as a clean file.
  */
 export function parseRustUseSites(rustText, knownCrates = new Set(), options = {}) {
+  // used by its own test
   const { returnMetrics = false } = options;
   /** The offset of the first `use` opener no `;` terminates, `null` before one is seen. */
   let unterminatedUseAt = null;

package/src/analysis/source-util.mjs

@@ -160,7 +160,7 @@ function ownershipIndexOf(projects) {
  * one per equality probe.
  */
 let rootComparisons = 0;
-export const ownershipRootComparisons = () => rootComparisons;
+export const ownershipRootComparisons = () => rootComparisons; // used by its own test
 
 /**
  * The first index in `roots` (sorted ascending) whose value is at or after

package/src/analysis/typescript.mjs

@@ -318,6 +318,7 @@ const SCRIPT_KIND_BY_LANG = Object.freeze({
  * @returns {{ path: boolean, relative: boolean, namesOnly: boolean }}
  */
 export function specifierSpelling(specifier) {
+  // used by its own test
   const relative =
     specifier === "." ||
     specifier === ".." ||
@@ -340,6 +341,7 @@ export function specifierSpelling(specifier) {
  * @returns {string|null}
  */
 export function packageNameOf(specifier) {
+  // used by its own test
   if (specifier === "" || specifier.startsWith(".") || specifier.startsWith("/")) return null;
   const segments = specifier.split("/");
   if (specifier.startsWith("@"))

package/src/architecture-intent/model.mjs

@@ -51,7 +51,7 @@ import { GOVERNANCE_ROW_KEYS, rowSchemaViolations } from "../governance/row-sche
 export const INTENT_FILE = "architecture-intent.json";
 
 /** The one supported `version`. A different value is a load error. */
-export const INTENT_VERSION = "1";
+export const INTENT_VERSION = "1"; // used by its own test
 
 /**
  * The only keys a valid intent file may carry at the top level.
@@ -66,6 +66,7 @@ export const INTENT_VERSION = "1";
  * engine judges them.
  */
 export const TOP_LEVEL_KEYS = Object.freeze([
+  // used by its own test
   "version",
   "boundaries",
   "allowed",
@@ -76,17 +77,17 @@ export const TOP_LEVEL_KEYS = Object.freeze([
 ]);
 
 /** The sub-keys a `projects` section may carry. */
-export const PROJECT_SECTION_KEYS = Object.freeze(["required", "forbidden"]);
+const PROJECT_SECTION_KEYS = Object.freeze(["required", "forbidden"]);
 /** The sub-keys a `dependencies` section may carry. */
-export const DEPENDENCY_SECTION_KEYS = Object.freeze(["allowed", "forbidden"]);
+const DEPENDENCY_SECTION_KEYS = Object.freeze(["allowed", "forbidden"]);
 /** The keys a `projects.required[]` row may carry. */
-export const REQUIRED_PROJECT_KEYS = Object.freeze(["name", "tags", "decisionRef"]);
+const REQUIRED_PROJECT_KEYS = Object.freeze(["name", "tags", "decisionRef"]);
 /** The keys a `projects.forbidden[]` row may carry. */
-export const FORBIDDEN_PROJECT_KEYS = Object.freeze(["name", "decisionRef"]);
+const FORBIDDEN_PROJECT_KEYS = Object.freeze(["name", "decisionRef"]);
 /** The keys a `dependencies.allowed[]` / `dependencies.forbidden[]` row may carry. */
-export const DEPENDENCY_ROW_KEYS = Object.freeze(["source", "target", "decisionRef"]);
+const DEPENDENCY_ROW_KEYS = Object.freeze(["source", "target", "decisionRef"]);
 /** The keys a `forbiddenTags[]` row may carry. */
-export const TAG_ROW_KEYS = Object.freeze(["from", "to", "decisionRef"]);
+const TAG_ROW_KEYS = Object.freeze(["from", "to", "decisionRef"]);
 
 /**
  * The shared governance-block check for any intent row (Contract 2): when the
@@ -157,6 +158,7 @@ function isSingleProjectSelector(selector) {
  * resolve against.
  */
 export function boundaryNames(intent) {
+  // used by its own test
   return (intent.boundaries ?? []).map((b) => b.name);
 }
 
@@ -168,6 +170,7 @@ export function boundaryNames(intent) {
  * @returns {string[]}
  */
 export function findIntentViolations(raw) {
+  // used by its own test
   const violations = [];
 
   if (!isPlainObject(raw)) {
@@ -603,6 +606,7 @@ export function findIntentViolations(raw) {
  * @returns {object} The normalized model: `{version, boundaries: {name, match, members?}[], allowed, forbidden, projects?, dependencies?, forbiddenTags?}`.
  */
 export function normalizeIntent(intent) {
+  // used by its own test
   return {
     version: intent.version,
     boundaries: intent.boundaries.map((b) => ({ name: b.name, match: [...b.match] })),

package/src/architecture-intent/selectors.mjs

@@ -30,7 +30,7 @@
  */
 
 /** The three selector labels. `unlabeled` is a bare project name (equals `name:`). */
-export const SELECTOR_LABELS = Object.freeze(["name", "tag", "directory"]);
+const SELECTOR_LABELS = Object.freeze(["name", "tag", "directory"]);
 
 /**
  * Split a selector into `{exclude, label, value}` — `!` prefix removed, `*`
@@ -71,6 +71,7 @@ export function isValidSelector(value) {
  * @returns {string[]}
  */
 export function selectProjects(selector, nodes) {
+  // used by its own test
   const { label, value } = splitSelector(selector);
 
   let found;

package/src/commands/change-intent.mjs

@@ -44,10 +44,10 @@ import { readFile as readFileFromDisk } from "node:fs/promises";
 import { describe, isPlainObject } from "../values.mjs";
 
 /** The only `version` this module accepts. A different value is a load error. */
-export const CHANGE_INTENT_VERSION = "1";
+const CHANGE_INTENT_VERSION = "1";
 
 /** The only keys a valid change-intent file may carry at the top level. */
-export const CHANGE_INTENT_TOP_LEVEL_KEYS = Object.freeze([
+const CHANGE_INTENT_TOP_LEVEL_KEYS = Object.freeze([
   "version",
   "base",
   "summary",
@@ -57,16 +57,16 @@ export const CHANGE_INTENT_TOP_LEVEL_KEYS = Object.freeze([
 ]);
 
 /** The keys the `base` section may carry. */
-export const CHANGE_INTENT_BASE_KEYS = Object.freeze(["commit"]);
+const CHANGE_INTENT_BASE_KEYS = Object.freeze(["commit"]);
 
 /** The sub-keys the `projects` section may carry. */
-export const CHANGE_INTENT_PROJECT_SECTION_KEYS = Object.freeze(["add", "remove"]);
+const CHANGE_INTENT_PROJECT_SECTION_KEYS = Object.freeze(["add", "remove"]);
 /** The sub-keys the `edges` section may carry. */
-export const CHANGE_INTENT_EDGE_SECTION_KEYS = Object.freeze(["add", "remove"]);
+const CHANGE_INTENT_EDGE_SECTION_KEYS = Object.freeze(["add", "remove"]);
 /** The keys an edge row may carry. */
-export const CHANGE_INTENT_EDGE_ROW_KEYS = Object.freeze(["from", "to"]);
+const CHANGE_INTENT_EDGE_ROW_KEYS = Object.freeze(["from", "to"]);
 /** The keys the `constraints` section may carry. */
-export const CHANGE_INTENT_CONSTRAINT_KEYS = Object.freeze(["noNewViolations", "noNewCycles"]);
+const CHANGE_INTENT_CONSTRAINT_KEYS = Object.freeze(["noNewViolations", "noNewCycles"]);
 
 /**
  * The constraints the reconciliation judges, in the fixed order their verdict
@@ -204,7 +204,7 @@ function sectionListViolations(section, spec) {
  * @param {unknown} raw The parsed JSON value.
  * @returns {string[]}
  */
-export function findChangeIntentViolations(raw) {
+function findChangeIntentViolations(raw) {
   const violations = [];
   if (!isPlainObject(raw)) {
     return [`top level: must be an object, got ${describe(raw)}`];
@@ -351,7 +351,7 @@ export function findChangeIntentViolations(raw) {
  *   with absent raw sections already normalized to empty arrays.
  * @returns {string[]} Messages; empty when the intent is not a catch-all.
  */
-export function findChangeIntentBreadthViolations(intent) {
+function findChangeIntentBreadthViolations(intent) {
   const declaredRows =
     intent.projects.add.length +
     intent.projects.remove.length +
@@ -389,6 +389,7 @@ export function findChangeIntentBreadthViolations(intent) {
  *   silently would reconcile against an expectation nobody wrote.
  */
 export function parseChangeIntent(text, path) {
+  // used by its own test
   let parsed;
   try {
     parsed = JSON.parse(text);