@ecoma-io/archkeep 0.24.0 → 0.25.0

package/src/rules/tags.mjs

@@ -29,7 +29,7 @@ import { tagMatches } from "./match.mjs";
 import { getPath, pathExists } from "./reachability.mjs";
 
 /** `"a", "b"` — how upstream renders a tag list inside a message. */
-export function stringifyTags(tags) {
+function stringifyTags(tags) {
   return tags.map((t) => `"${t}"`).join(", ");
 }
 
@@ -55,7 +55,7 @@ export function hasTag(project, tag) {
 }
 
 /** True when the project carries NONE of these tags — upstream's spelling. */
-export function hasNoneOfTheseTags(project, tags) {
+function hasNoneOfTheseTags(project, tags) {
   return tags.filter((tag) => hasTag(project, tag)).length === 0;
 }
 
@@ -165,6 +165,7 @@ export function orphanedNotDependOnTags(depConstraints, graph) {
  * are what the message prints so a reader can see the hop that did it.
  */
 export function findDependenciesWithTags(targetProject, tags, graph, reach) {
+  // used by its own test
   const reachable = Object.keys(graph.nodes)
     .sort()
     .filter(

package/src/rules/topology.mjs

@@ -57,6 +57,7 @@ export function appIsMFERemote(project) {
  * — the common case, and the one that keeps `noSelfCircularDependencies` armed.
  */
 export function entryPointOf(file, projectRoot, entryPoints) {
+  // used by its own test
   if (!entryPoints || entryPoints.length === 0) return undefined;
   const fileEntryPoint = entryPoints.find((entry) => entry.file === file);
   if (fileEntryPoint) return fileEntryPoint.file;
@@ -88,7 +89,7 @@ export function entryPointOf(file, projectRoot, entryPoints) {
  * standing in for upstream's module resolution — the resolver already ran; this
  * layer never runs a second one.
  */
-export function secondaryEntryPointPath(resolvedFile, targetProject) {
+function secondaryEntryPointPath(resolvedFile, targetProject) {
   if (!resolvedFile) return undefined;
   return entryPointOf(resolvedFile, targetProject.data.root, targetProject.data?.entryPoints);
 }
@@ -128,6 +129,7 @@ export function belongsToDifferentEntryPoint(resolvedFile, sourceFile, sourcePro
  * @returns {string[]|null}
  */
 export function findDynamicImportPath(graph, sourceProjectName, targetProjectName, visited = []) {
+  // used by its own test
   if (visited.indexOf(sourceProjectName) > -1) return null;
   for (const dependency of graph.dependencies?.[sourceProjectName] ?? []) {
     if (dependency.type !== DYNAMIC) continue;
@@ -152,6 +154,7 @@ export function findDynamicImportPath(graph, sourceProjectName, targetProjectNam
  * fired, and then the verdict would name evidence for a route nobody walked.
  */
 export function hasDynamicImport(graph, sourceProjectName, targetProjectName) {
+  // used by its own test
   return findDynamicImportPath(graph, sourceProjectName, targetProjectName) !== null;
 }
 
@@ -209,6 +212,7 @@ export function createFileDependencyIndex(edges) {
 
 /** The files carrying each hop of a path — one entry per consecutive pair. */
 export function findFilesInCircularPath(fileIndex, circularPath) {
+  // used by its own test
   const chain = [];
   for (let i = 0; i < circularPath.length - 1; i++) {
     chain.push(fileIndex.any.get(edgeKey(circularPath[i].name, circularPath[i + 1].name)) ?? []);
@@ -218,6 +222,7 @@ export function findFilesInCircularPath(fileIndex, circularPath) {
 
 /** The files that lazy-load `target` from inside `source`. */
 export function findFilesWithDynamicImports(fileIndex, sourceProjectName, targetProjectName) {
+  // used by its own test
   return fileIndex.dynamic.get(edgeKey(sourceProjectName, targetProjectName)) ?? [];
 }
 

package/src/values.mjs

@@ -36,6 +36,17 @@ export const isPlainObject = (value) =>
 export const isStringArray = (value) =>
   Array.isArray(value) && value.every((item) => typeof item === "string");
 
+/**
+ * Whether a value is a string a reader could actually act on — non-empty
+ * after trimming. The verdict contract's reason fields (I3/I4) refuse
+ * everything else: `""` and `"   "` are byte-present but semantically
+ * absent, and a non-string reason would ship a `typeof` artifact where the
+ * reader was promised a sentence.
+ *
+ * @type {(value: unknown) => value is string}
+ */
+export const isNonEmptyString = (value) => typeof value === "string" && value.trim() !== "";
+
 /**
  * A value's type, for an error message that shows what was actually there.
  *

package/src/workspace.mjs

@@ -721,6 +721,7 @@ const DOTLESS_FURNITURE = Object.freeze(
  * @returns {boolean}
  */
 export function exemptFromUnsupportedLanguage(sourceFile) {
+  // used by its own test
   const base = sourceFile.slice(sourceFile.lastIndexOf("/") + 1);
   // Dotfiles are editor and tool state (`.gitignore`, `.env`, `.npmrc`).
   if (base.startsWith(".")) return true;