@nielspeter/eess 0.2.2 → 0.4.0

package/dist/errors.js

@@ -18,4 +18,36 @@ export class ArchRuleError extends Error {
         return this.#violations;
     }
 }
+/**
+ * Is this an `ArchRuleError` — including one thrown by a DIFFERENT copy of this
+ * module?
+ *
+ * `instanceof` is an identity check, and a rule file does not necessarily load
+ * through the same module registry as the CLI that loads it. `load-rules.ts`
+ * uses jiti (bug 0074: a `.ts` rule file must load inside a `"type": "commonjs"`
+ * consumer project), and jiti keeps its own registry — so the `ArchRuleError`
+ * a self-executing rule file throws is a different class object from this one
+ * and `instanceof` is FALSE for an error that is, in every way that matters,
+ * an ArchRuleError.
+ *
+ * Measured: that made `check.ts` skip `ruleFileTruncated()`, so a rule file
+ * whose first rule threw reported the finding and said nothing about the rules
+ * after it that never ran — a silent gap in a red run, which is bug 0029
+ * reopened by the loader. The same hazard exists for any consumer with two
+ * copies of eess-ts on disk (a nested `node_modules`), where the loader is
+ * irrelevant.
+ *
+ * Structural, therefore: the `name` this class stamps in its constructor plus
+ * the `violations` array that is its whole reason to exist. Both are part of the
+ * documented public shape (`ArchViolation[]` on a thrown error), so matching on
+ * them is matching on the contract rather than on a coincidence.
+ */
+export function isArchRuleError(error) {
+    if (error instanceof ArchRuleError)
+        return true;
+    return (error instanceof Error &&
+        error.name === 'ArchRuleError' &&
+        'violations' in error &&
+        Array.isArray(error.violations));
+}
 //# sourceMappingURL=errors.js.map

package/dist/errors.js.map

@@ -1 +1 @@
-{"version":3,"file":"errors.js","sourceRoot":"","sources":["../src/errors.ts"],"names":[],"mappings":"AAEA;;;;;GAKG;AACH,MAAM,OAAO,aAAc,SAAQ,KAAK;IAC7B,WAAW,CAAiB;IAErC,YAAY,UAA2B,EAAE,MAAe;QACtD,MAAM,OAAO,GAAG,yBAAyB,UAAU,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,KAAK,MAAM,CAAC,UAAU,CAAC,MAAM,CAAC,SAAS,CAAA;QAClH,MAAM,UAAU,GAAG,MAAM,CAAC,CAAC,CAAC,MAAM,MAAM,EAAE,CAAC,CAAC,CAAC,EAAE,CAAA;QAC/C,KAAK,CAAC,GAAG,OAAO,GAAG,UAAU,EAAE,CAAC,CAAA;QAChC,IAAI,CAAC,IAAI,GAAG,eAAe,CAAA;QAC3B,IAAI,CAAC,WAAW,GAAG,UAAU,CAAA;IAC/B,CAAC;IAED,4DAA4D;IAC5D,IAAI,UAAU;QACZ,OAAO,IAAI,CAAC,WAAW,CAAA;IACzB,CAAC;CACF"}
+{"version":3,"file":"errors.js","sourceRoot":"","sources":["../src/errors.ts"],"names":[],"mappings":"AAEA;;;;;GAKG;AACH,MAAM,OAAO,aAAc,SAAQ,KAAK;IAC7B,WAAW,CAAiB;IAErC,YAAY,UAA2B,EAAE,MAAe;QACtD,MAAM,OAAO,GAAG,yBAAyB,UAAU,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,KAAK,MAAM,CAAC,UAAU,CAAC,MAAM,CAAC,SAAS,CAAA;QAClH,MAAM,UAAU,GAAG,MAAM,CAAC,CAAC,CAAC,MAAM,MAAM,EAAE,CAAC,CAAC,CAAC,EAAE,CAAA;QAC/C,KAAK,CAAC,GAAG,OAAO,GAAG,UAAU,EAAE,CAAC,CAAA;QAChC,IAAI,CAAC,IAAI,GAAG,eAAe,CAAA;QAC3B,IAAI,CAAC,WAAW,GAAG,UAAU,CAAA;IAC/B,CAAC;IAED,4DAA4D;IAC5D,IAAI,UAAU;QACZ,OAAO,IAAI,CAAC,WAAW,CAAA;IACzB,CAAC;CACF;AAED;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AACH,MAAM,UAAU,eAAe,CAAC,KAAc;IAC5C,IAAI,KAAK,YAAY,aAAa;QAAE,OAAO,IAAI,CAAA;IAC/C,OAAO,CACL,KAAK,YAAY,KAAK;QACtB,KAAK,CAAC,IAAI,KAAK,eAAe;QAC9B,YAAY,IAAI,KAAK;QACrB,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,UAAU,CAAC,CAChC,CAAA;AACH,CAAC"}

package/dist/execute-rule.d.ts

@@ -6,7 +6,7 @@ import type { RuleMetadata } from './rule-metadata.js';
  * Shared across all builder types (RuleBuilder, SliceRuleBuilder,
  * SchemaRuleBuilder, ResolverRuleBuilder, PairFinalBuilder, SmellBuilder).
  */
-export interface ExecuteRuleContext {
+interface ExecuteRuleContext {
     reason?: string;
     metadata?: RuleMetadata;
     exclusions?: (string | RegExp)[];
@@ -33,4 +33,5 @@ export declare function executeCheck(violations: ArchViolation[], ctx: ExecuteRu
  * Execute the terminal "warn" action: apply options, format, log to stderr.
  */
 export declare function executeWarn(violations: ArchViolation[], ctx: ExecuteRuleContext, options?: CheckOptions): void;
+export {};
 //# sourceMappingURL=execute-rule.d.ts.map

package/dist/execute-rule.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"execute-rule.d.ts","sourceRoot":"","sources":["../src/execute-rule.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAA;AACnD,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAA;AACtD,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAA;AAQtD;;;;GAIG;AAEH,MAAM,WAAW,kBAAkB;IACjC,MAAM,CAAC,EAAE,MAAM,CAAA;IACf,QAAQ,CAAC,EAAE,YAAY,CAAA;IACvB,UAAU,CAAC,EAAE,CAAC,MAAM,GAAG,MAAM,CAAC,EAAE,CAAA;IAChC,aAAa,CAAC,EAAE,GAAG,CAAC,MAAM,CAAC,CAAA;CAC5B;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,YAAY,CAC1B,UAAU,EAAE,aAAa,EAAE,EAC3B,GAAG,EAAE,kBAAkB,GACtB,aAAa,EAAE,CAoGjB;AAED;;GAEG;AACH,wBAAgB,YAAY,CAC1B,UAAU,EAAE,aAAa,EAAE,EAC3B,GAAG,EAAE,kBAAkB,EACvB,OAAO,CAAC,EAAE,YAAY,GACrB,IAAI,CAeN;AAED;;GAEG;AACH,wBAAgB,WAAW,CACzB,UAAU,EAAE,aAAa,EAAE,EAC3B,GAAG,EAAE,kBAAkB,EACvB,OAAO,CAAC,EAAE,YAAY,GACrB,IAAI,CAmBN"}
+{"version":3,"file":"execute-rule.d.ts","sourceRoot":"","sources":["../src/execute-rule.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAA;AAEnD,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAA;AACtD,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAA;AAWtD;;;;GAIG;AACH,UAAU,kBAAkB;IAC1B,MAAM,CAAC,EAAE,MAAM,CAAA;IACf,QAAQ,CAAC,EAAE,YAAY,CAAA;IACvB,UAAU,CAAC,EAAE,CAAC,MAAM,GAAG,MAAM,CAAC,EAAE,CAAA;IAChC,aAAa,CAAC,EAAE,GAAG,CAAC,MAAM,CAAC,CAAA;CAC5B;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,YAAY,CAC1B,UAAU,EAAE,aAAa,EAAE,EAC3B,GAAG,EAAE,kBAAkB,GACtB,aAAa,EAAE,CA8GjB;AAED;;GAEG;AACH,wBAAgB,YAAY,CAC1B,UAAU,EAAE,aAAa,EAAE,EAC3B,GAAG,EAAE,kBAAkB,EACvB,OAAO,CAAC,EAAE,YAAY,GACrB,IAAI,CAkBN;AAED;;GAEG;AACH,wBAAgB,WAAW,CACzB,UAAU,EAAE,aAAa,EAAE,EAC3B,GAAG,EAAE,kBAAkB,EACvB,OAAO,CAAC,EAAE,YAAY,GACrB,IAAI,CA8BN"}

package/dist/execute-rule.js

@@ -1,10 +1,14 @@
 import fs from 'node:fs';
+import { severityFor } from './violation.js';
 import { ArchRuleError } from './errors.js';
 import { formatViolations } from './format.js';
 import { formatViolationsJson } from './format-json.js';
 import { formatViolationsGitHub } from './format-github.js';
 import { reportViolations } from './report.js';
 import { parseExclusionComments, isExcludedByComment } from './exclusion-comments.js';
+import { writeStderr } from './stderr.js';
+import { activeNotice } from './diff-disclosure.js';
+import { recordCommentSuppression } from './comment-suppression.js';
 /**
  * Stamp rule metadata onto violations and apply the rule's exclusions.
  *
@@ -19,11 +23,16 @@ import { parseExclusionComments, isExcludedByComment } from './exclusion-comment
  */
 export function applyFilters(violations, ctx) {
     let result = violations;
-    // Apply .excluding() chain exclusions
+    // Apply .excluding() chain exclusions — never against a bypassFilters
+    // (ADR-010 configuration) finding: it reports the rule's own instrument is
+    // broken, not a fault in what was examined, so an exclusion aimed at the
+    // latter cannot correctly suppress the former.
     const exclusions = ctx.exclusions ?? [];
     if (exclusions.length > 0) {
         const matchedPatterns = new Set();
         result = result.filter((v) => {
+            if (v.bypassFilters === true)
+                return true;
             // Match against element, file, or message — so that custom conditions
             // using createViolation() can be excluded by file path or message content,
             // not just by element name (which may be a generic AST node kind).
@@ -41,7 +50,7 @@ export function applyFilters(violations, ctx) {
         const silentIndices = ctx.silentIndices ?? new Set();
         exclusions.forEach((pattern, index) => {
             if (!matchedPatterns.has(index) && !silentIndices.has(index)) {
-                console.warn(`[eess] Unused exclusion '${String(pattern)}' in rule '${ruleId}'. ` +
+                writeStderr(`[eess] Unused exclusion '${String(pattern)}' in rule '${ruleId}'. ` +
                     `It matched zero violations — it may be stale after a rename.`);
             }
         });
@@ -94,7 +103,7 @@ export function applyFilters(violations, ctx) {
                 const sourceText = fs.readFileSync(filePath, 'utf-8');
                 const parseResult = parseExclusionComments(sourceText, filePath);
                 for (const warning of parseResult.warnings) {
-                    console.warn(`[eess] ${warning.message}`);
+                    writeStderr(`[eess] ${warning.message}`);
                 }
                 return parseResult.exclusions;
             }
@@ -107,7 +116,15 @@ export function applyFilters(violations, ctx) {
             }
         });
         if (allComments.length > 0) {
-            result = result.filter((v) => !isExcludedByComment(v, allComments));
+            const ruleId = ctx.metadata.id;
+            result = result.filter((v) => {
+                if (v.bypassFilters === true)
+                    return true;
+                const excluded = isExcludedByComment(v, allComments);
+                if (excluded)
+                    recordCommentSuppression(ruleId, v.file);
+                return !excluded;
+            });
         }
     }
     return result;
@@ -121,7 +138,11 @@ export function executeCheck(violations, ctx, options) {
         filtered = options.baseline.filterNew(filtered);
     }
     if (options?.diff) {
+        const before = filtered.length;
         filtered = options.diff.filterToChanged(filtered);
+        const notice = activeNotice(before - filtered.length);
+        if (notice !== undefined)
+            writeStderr(`${notice}\n`);
     }
     if (filtered.length > 0) {
         // One emitter for both paths (plan 0070) — text/json/github, then throw.
@@ -138,18 +159,29 @@ export function executeWarn(violations, ctx, options) {
         filtered = options.baseline.filterNew(filtered);
     }
     if (options?.diff) {
+        const before = filtered.length;
         filtered = options.diff.filterToChanged(filtered);
+        const notice = activeNotice(before - filtered.length);
+        if (notice !== undefined)
+            writeStderr(`${notice}\n`);
     }
     if (filtered.length > 0) {
         if (options?.format === 'json') {
-            console.warn(formatViolationsJson(filtered, ctx.reason));
+            writeStderr(formatViolationsJson(filtered, ctx.reason));
         }
         else if (options?.format === 'github') {
             process.stdout.write(formatViolationsGitHub(filtered, 'warning') + '\n');
         }
         else {
-            console.warn(formatViolations(filtered, ctx.reason));
+            writeStderr(formatViolations(filtered, ctx.reason));
         }
     }
+    // `bypassFilters` outranks `.warn()` (see `severityFor`) — a config finding
+    // whose own text promises "not by .warn()" must still fail the build even
+    // though the rest of this rule's findings are legitimately warn-only.
+    const escalated = filtered.filter((v) => severityFor(v, 'warn') === 'error');
+    if (escalated.length > 0) {
+        throw new ArchRuleError(escalated, ctx.reason);
+    }
 }
 //# sourceMappingURL=execute-rule.js.map

package/dist/execute-rule.js.map

@@ -1 +1 @@
-{"version":3,"file":"execute-rule.js","sourceRoot":"","sources":["../src/execute-rule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,SAAS,CAAA;AAIxB,OAAO,EAAE,aAAa,EAAE,MAAM,aAAa,CAAA;AAC3C,OAAO,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAA;AAC9C,OAAO,EAAE,oBAAoB,EAAE,MAAM,kBAAkB,CAAA;AACvD,OAAO,EAAE,sBAAsB,EAAE,MAAM,oBAAoB,CAAA;AAC3D,OAAO,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAA;AAC9C,OAAO,EAAE,sBAAsB,EAAE,mBAAmB,EAAE,MAAM,yBAAyB,CAAA;AAerF;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,YAAY,CAC1B,UAA2B,EAC3B,GAAuB;IAEvB,IAAI,MAAM,GAAG,UAAU,CAAA;IAEvB,sCAAsC;IACtC,MAAM,UAAU,GAAG,GAAG,CAAC,UAAU,IAAI,EAAE,CAAA;IACvC,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC1B,MAAM,eAAe,GAAG,IAAI,GAAG,EAAU,CAAA;QACzC,MAAM,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE;YAC3B,sEAAsE;YACtE,2EAA2E;YAC3E,mEAAmE;YACnE,MAAM,OAAO,GAAG,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAA;YAC9C,MAAM,UAAU,GAAG,UAAU,CAAC,SAAS,CAAC,CAAC,OAAO,EAAE,EAAE,CAClD,OAAO,OAAO,KAAK,QAAQ;gBACzB,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,OAAO,CAAC;gBACpC,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CACzC,CAAA;YACD,IAAI,UAAU,IAAI,CAAC,EAAE,CAAC;gBACpB,eAAe,CAAC,GAAG,CAAC,UAAU,CAAC,CAAA;gBAC/B,OAAO,KAAK,CAAA;YACd,CAAC;YACD,OAAO,IAAI,CAAA;QACb,CAAC,CAAC,CAAA;QAEF,MAAM,MAAM,GAAG,GAAG,CAAC,QAAQ,EAAE,EAAE,IAAI,SAAS,CAAA;QAC5C,MAAM,aAAa,GAAG,GAAG,CAAC,aAAa,IAAI,IAAI,GAAG,EAAE,CAAA;QACpD,UAAU,CAAC,OAAO,CAAC,CAAC,OAAO,EAAE,KAAK,EAAE,EAAE;YACpC,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,aAAa,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,CAAC;gBAC7D,OAAO,CAAC,IAAI,CACV,4BAA4B,MAAM,CAAC,OAAO,CAAC,cAAc,MAAM,KAAK;oBAClE,8DAA8D,CACjE,CAAA;YACH,CAAC;QACH,CAAC,CAAC,CAAA;IACJ,CAAC;IAED,6EAA6E;IAC7E,EAAE;IACF,+EAA+E;IAC/E,4EAA4E;IAC5E,2EAA2E;IAC3E,+EAA+E;IAC/E,+EAA+E;IAC/E,4EAA4E;IAC5E,8DAA8D;IAC9D,EAAE;IACF,6EAA6E;IAC7E,2EAA2E;IAC3E,4EAA4E;IAC5E,8EAA8E;IAC9E,6EAA6E;IAC7E,6EAA6E;IAC7E,EAAE;IACF,2EAA2E;IAC3E,8EAA8E;IAC9E,6EAA6E;IAC7E,qEAAqE;IACrE,uDAAuD;IACvD,0EAA0E;IAC1E,mDAAmD;IACnD,EAAE;IACF,8EAA8E;IAC9E,yDAAyD;IACzD,IAAI,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACtB,KAAK,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;YACvB,IAAI,CAAC,CAAC,MAAM,KAAK,SAAS,IAAI,GAAG,CAAC,QAAQ,EAAE,EAAE,KAAK,SAAS;gBAAE,CAAC,CAAC,MAAM,GAAG,GAAG,CAAC,QAAQ,CAAC,EAAE,CAAA;YACxF,IAAI,CAAC,CAAC,OAAO,KAAK,SAAS,IAAI,GAAG,CAAC,MAAM,KAAK,SAAS;gBAAE,CAAC,CAAC,OAAO,GAAG,GAAG,CAAC,MAAM,CAAA;YAC/E,IAAI,CAAC,CAAC,UAAU,KAAK,SAAS,IAAI,GAAG,CAAC,QAAQ,EAAE,UAAU,KAAK,SAAS;gBACtE,CAAC,CAAC,UAAU,GAAG,GAAG,CAAC,QAAQ,CAAC,UAAU,CAAA;YACxC,IAAI,CAAC,CAAC,IAAI,KAAK,SAAS,IAAI,GAAG,CAAC,QAAQ,EAAE,IAAI,KAAK,SAAS;gBAAE,CAAC,CAAC,IAAI,GAAG,GAAG,CAAC,QAAQ,CAAC,IAAI,CAAA;QAC1F,CAAC;IACH,CAAC;IAED,yEAAyE;IACzE,+EAA+E;IAC/E,IAAI,GAAG,CAAC,QAAQ,EAAE,EAAE,IAAI,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC1C,MAAM,SAAS,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAA;QACpD,MAAM,WAAW,GAAG,CAAC,GAAG,SAAS,CAAC,CAAC,OAAO,CAAC,CAAC,QAAQ,EAAE,EAAE;YACtD,IAAI,CAAC;gBACH,MAAM,UAAU,GAAG,EAAE,CAAC,YAAY,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;gBACrD,MAAM,WAAW,GAAG,sBAAsB,CAAC,UAAU,EAAE,QAAQ,CAAC,CAAA;gBAChE,KAAK,MAAM,OAAO,IAAI,WAAW,CAAC,QAAQ,EAAE,CAAC;oBAC3C,OAAO,CAAC,IAAI,CAAC,UAAU,OAAO,CAAC,OAAO,EAAE,CAAC,CAAA;gBAC3C,CAAC;gBACD,OAAO,WAAW,CAAC,UAAU,CAAA;YAC/B,CAAC;YAAC,OAAO,GAAG,EAAE,CAAC;gBACb,gEAAgE;gBAChE,mEAAmE;gBACnE,kEAAkE;gBAClE,KAAK,GAAG,CAAA;gBACR,OAAO,EAAE,CAAA;YACX,CAAC;QACH,CAAC,CAAC,CAAA;QAEF,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC3B,MAAM,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,mBAAmB,CAAC,CAAC,EAAE,WAAW,CAAC,CAAC,CAAA;QACrE,CAAC;IACH,CAAC;IAED,OAAO,MAAM,CAAA;AACf,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,YAAY,CAC1B,UAA2B,EAC3B,GAAuB,EACvB,OAAsB;IAEtB,IAAI,QAAQ,GAAG,YAAY,CAAC,UAAU,EAAE,GAAG,CAAC,CAAA;IAE5C,IAAI,OAAO,EAAE,QAAQ,EAAE,CAAC;QACtB,QAAQ,GAAG,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAA;IACjD,CAAC;IACD,IAAI,OAAO,EAAE,IAAI,EAAE,CAAC;QAClB,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,eAAe,CAAC,QAAQ,CAAC,CAAA;IACnD,CAAC;IAED,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACxB,yEAAyE;QACzE,gBAAgB,CAAC,QAAQ,EAAE,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,CAAC,MAAM,EAAE,CAAC,CAAA;QAC3E,MAAM,IAAI,aAAa,CAAC,QAAQ,EAAE,GAAG,CAAC,MAAM,CAAC,CAAA;IAC/C,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,WAAW,CACzB,UAA2B,EAC3B,GAAuB,EACvB,OAAsB;IAEtB,IAAI,QAAQ,GAAG,YAAY,CAAC,UAAU,EAAE,GAAG,CAAC,CAAA;IAE5C,IAAI,OAAO,EAAE,QAAQ,EAAE,CAAC;QACtB,QAAQ,GAAG,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAA;IACjD,CAAC;IACD,IAAI,OAAO,EAAE,IAAI,EAAE,CAAC;QAClB,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,eAAe,CAAC,QAAQ,CAAC,CAAA;IACnD,CAAC;IAED,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACxB,IAAI,OAAO,EAAE,MAAM,KAAK,MAAM,EAAE,CAAC;YAC/B,OAAO,CAAC,IAAI,CAAC,oBAAoB,CAAC,QAAQ,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC,CAAA;QAC1D,CAAC;aAAM,IAAI,OAAO,EAAE,MAAM,KAAK,QAAQ,EAAE,CAAC;YACxC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,sBAAsB,CAAC,QAAQ,EAAE,SAAS,CAAC,GAAG,IAAI,CAAC,CAAA;QAC1E,CAAC;aAAM,CAAC;YACN,OAAO,CAAC,IAAI,CAAC,gBAAgB,CAAC,QAAQ,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC,CAAA;QACtD,CAAC;IACH,CAAC;AACH,CAAC"}
+{"version":3,"file":"execute-rule.js","sourceRoot":"","sources":["../src/execute-rule.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,MAAM,SAAS,CAAA;AAExB,OAAO,EAAE,WAAW,EAAE,MAAM,gBAAgB,CAAA;AAG5C,OAAO,EAAE,aAAa,EAAE,MAAM,aAAa,CAAA;AAC3C,OAAO,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAA;AAC9C,OAAO,EAAE,oBAAoB,EAAE,MAAM,kBAAkB,CAAA;AACvD,OAAO,EAAE,sBAAsB,EAAE,MAAM,oBAAoB,CAAA;AAC3D,OAAO,EAAE,gBAAgB,EAAE,MAAM,aAAa,CAAA;AAC9C,OAAO,EAAE,sBAAsB,EAAE,mBAAmB,EAAE,MAAM,yBAAyB,CAAA;AACrF,OAAO,EAAE,WAAW,EAAE,MAAM,aAAa,CAAA;AACzC,OAAO,EAAE,YAAY,EAAE,MAAM,sBAAsB,CAAA;AACnD,OAAO,EAAE,wBAAwB,EAAE,MAAM,0BAA0B,CAAA;AAcnE;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,YAAY,CAC1B,UAA2B,EAC3B,GAAuB;IAEvB,IAAI,MAAM,GAAG,UAAU,CAAA;IAEvB,sEAAsE;IACtE,2EAA2E;IAC3E,yEAAyE;IACzE,+CAA+C;IAC/C,MAAM,UAAU,GAAG,GAAG,CAAC,UAAU,IAAI,EAAE,CAAA;IACvC,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC1B,MAAM,eAAe,GAAG,IAAI,GAAG,EAAU,CAAA;QACzC,MAAM,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE;YAC3B,IAAI,CAAC,CAAC,aAAa,KAAK,IAAI;gBAAE,OAAO,IAAI,CAAA;YACzC,sEAAsE;YACtE,2EAA2E;YAC3E,mEAAmE;YACnE,MAAM,OAAO,GAAG,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,CAAA;YAC9C,MAAM,UAAU,GAAG,UAAU,CAAC,SAAS,CAAC,CAAC,OAAO,EAAE,EAAE,CAClD,OAAO,OAAO,KAAK,QAAQ;gBACzB,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,OAAO,CAAC;gBACpC,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CACzC,CAAA;YACD,IAAI,UAAU,IAAI,CAAC,EAAE,CAAC;gBACpB,eAAe,CAAC,GAAG,CAAC,UAAU,CAAC,CAAA;gBAC/B,OAAO,KAAK,CAAA;YACd,CAAC;YACD,OAAO,IAAI,CAAA;QACb,CAAC,CAAC,CAAA;QAEF,MAAM,MAAM,GAAG,GAAG,CAAC,QAAQ,EAAE,EAAE,IAAI,SAAS,CAAA;QAC5C,MAAM,aAAa,GAAG,GAAG,CAAC,aAAa,IAAI,IAAI,GAAG,EAAE,CAAA;QACpD,UAAU,CAAC,OAAO,CAAC,CAAC,OAAO,EAAE,KAAK,EAAE,EAAE;YACpC,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,aAAa,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,CAAC;gBAC7D,WAAW,CACT,4BAA4B,MAAM,CAAC,OAAO,CAAC,cAAc,MAAM,KAAK;oBAClE,8DAA8D,CACjE,CAAA;YACH,CAAC;QACH,CAAC,CAAC,CAAA;IACJ,CAAC;IAED,6EAA6E;IAC7E,EAAE;IACF,+EAA+E;IAC/E,4EAA4E;IAC5E,2EAA2E;IAC3E,+EAA+E;IAC/E,+EAA+E;IAC/E,4EAA4E;IAC5E,8DAA8D;IAC9D,EAAE;IACF,6EAA6E;IAC7E,2EAA2E;IAC3E,4EAA4E;IAC5E,8EAA8E;IAC9E,6EAA6E;IAC7E,6EAA6E;IAC7E,EAAE;IACF,2EAA2E;IAC3E,8EAA8E;IAC9E,6EAA6E;IAC7E,qEAAqE;IACrE,uDAAuD;IACvD,0EAA0E;IAC1E,mDAAmD;IACnD,EAAE;IACF,8EAA8E;IAC9E,yDAAyD;IACzD,IAAI,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACtB,KAAK,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;YACvB,IAAI,CAAC,CAAC,MAAM,KAAK,SAAS,IAAI,GAAG,CAAC,QAAQ,EAAE,EAAE,KAAK,SAAS;gBAAE,CAAC,CAAC,MAAM,GAAG,GAAG,CAAC,QAAQ,CAAC,EAAE,CAAA;YACxF,IAAI,CAAC,CAAC,OAAO,KAAK,SAAS,IAAI,GAAG,CAAC,MAAM,KAAK,SAAS;gBAAE,CAAC,CAAC,OAAO,GAAG,GAAG,CAAC,MAAM,CAAA;YAC/E,IAAI,CAAC,CAAC,UAAU,KAAK,SAAS,IAAI,GAAG,CAAC,QAAQ,EAAE,UAAU,KAAK,SAAS;gBACtE,CAAC,CAAC,UAAU,GAAG,GAAG,CAAC,QAAQ,CAAC,UAAU,CAAA;YACxC,IAAI,CAAC,CAAC,IAAI,KAAK,SAAS,IAAI,GAAG,CAAC,QAAQ,EAAE,IAAI,KAAK,SAAS;gBAAE,CAAC,CAAC,IAAI,GAAG,GAAG,CAAC,QAAQ,CAAC,IAAI,CAAA;QAC1F,CAAC;IACH,CAAC;IAED,yEAAyE;IACzE,+EAA+E;IAC/E,IAAI,GAAG,CAAC,QAAQ,EAAE,EAAE,IAAI,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC1C,MAAM,SAAS,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAA;QACpD,MAAM,WAAW,GAAG,CAAC,GAAG,SAAS,CAAC,CAAC,OAAO,CAAC,CAAC,QAAQ,EAAE,EAAE;YACtD,IAAI,CAAC;gBACH,MAAM,UAAU,GAAG,EAAE,CAAC,YAAY,CAAC,QAAQ,EAAE,OAAO,CAAC,CAAA;gBACrD,MAAM,WAAW,GAAG,sBAAsB,CAAC,UAAU,EAAE,QAAQ,CAAC,CAAA;gBAChE,KAAK,MAAM,OAAO,IAAI,WAAW,CAAC,QAAQ,EAAE,CAAC;oBAC3C,WAAW,CAAC,UAAU,OAAO,CAAC,OAAO,EAAE,CAAC,CAAA;gBAC1C,CAAC;gBACD,OAAO,WAAW,CAAC,UAAU,CAAA;YAC/B,CAAC;YAAC,OAAO,GAAG,EAAE,CAAC;gBACb,gEAAgE;gBAChE,mEAAmE;gBACnE,kEAAkE;gBAClE,KAAK,GAAG,CAAA;gBACR,OAAO,EAAE,CAAA;YACX,CAAC;QACH,CAAC,CAAC,CAAA;QAEF,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC3B,MAAM,MAAM,GAAG,GAAG,CAAC,QAAQ,CAAC,EAAE,CAAA;YAC9B,MAAM,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE;gBAC3B,IAAI,CAAC,CAAC,aAAa,KAAK,IAAI;oBAAE,OAAO,IAAI,CAAA;gBACzC,MAAM,QAAQ,GAAG,mBAAmB,CAAC,CAAC,EAAE,WAAW,CAAC,CAAA;gBACpD,IAAI,QAAQ;oBAAE,wBAAwB,CAAC,MAAM,EAAE,CAAC,CAAC,IAAI,CAAC,CAAA;gBACtD,OAAO,CAAC,QAAQ,CAAA;YAClB,CAAC,CAAC,CAAA;QACJ,CAAC;IACH,CAAC;IAED,OAAO,MAAM,CAAA;AACf,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,YAAY,CAC1B,UAA2B,EAC3B,GAAuB,EACvB,OAAsB;IAEtB,IAAI,QAAQ,GAAG,YAAY,CAAC,UAAU,EAAE,GAAG,CAAC,CAAA;IAE5C,IAAI,OAAO,EAAE,QAAQ,EAAE,CAAC;QACtB,QAAQ,GAAG,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAA;IACjD,CAAC;IACD,IAAI,OAAO,EAAE,IAAI,EAAE,CAAC;QAClB,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAA;QAC9B,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,eAAe,CAAC,QAAQ,CAAC,CAAA;QACjD,MAAM,MAAM,GAAG,YAAY,CAAC,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAA;QACrD,IAAI,MAAM,KAAK,SAAS;YAAE,WAAW,CAAC,GAAG,MAAM,IAAI,CAAC,CAAA;IACtD,CAAC;IAED,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACxB,yEAAyE;QACzE,gBAAgB,CAAC,QAAQ,EAAE,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,CAAC,MAAM,EAAE,CAAC,CAAA;QAC3E,MAAM,IAAI,aAAa,CAAC,QAAQ,EAAE,GAAG,CAAC,MAAM,CAAC,CAAA;IAC/C,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,WAAW,CACzB,UAA2B,EAC3B,GAAuB,EACvB,OAAsB;IAEtB,IAAI,QAAQ,GAAG,YAAY,CAAC,UAAU,EAAE,GAAG,CAAC,CAAA;IAE5C,IAAI,OAAO,EAAE,QAAQ,EAAE,CAAC;QACtB,QAAQ,GAAG,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAC,QAAQ,CAAC,CAAA;IACjD,CAAC;IACD,IAAI,OAAO,EAAE,IAAI,EAAE,CAAC;QAClB,MAAM,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAA;QAC9B,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC,eAAe,CAAC,QAAQ,CAAC,CAAA;QACjD,MAAM,MAAM,GAAG,YAAY,CAAC,MAAM,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAA;QACrD,IAAI,MAAM,KAAK,SAAS;YAAE,WAAW,CAAC,GAAG,MAAM,IAAI,CAAC,CAAA;IACtD,CAAC;IAED,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACxB,IAAI,OAAO,EAAE,MAAM,KAAK,MAAM,EAAE,CAAC;YAC/B,WAAW,CAAC,oBAAoB,CAAC,QAAQ,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC,CAAA;QACzD,CAAC;aAAM,IAAI,OAAO,EAAE,MAAM,KAAK,QAAQ,EAAE,CAAC;YACxC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,sBAAsB,CAAC,QAAQ,EAAE,SAAS,CAAC,GAAG,IAAI,CAAC,CAAA;QAC1E,CAAC;aAAM,CAAC;YACN,WAAW,CAAC,gBAAgB,CAAC,QAAQ,EAAE,GAAG,CAAC,MAAM,CAAC,CAAC,CAAA;QACrD,CAAC;IACH,CAAC;IAED,4EAA4E;IAC5E,0EAA0E;IAC1E,sEAAsE;IACtE,MAAM,SAAS,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,WAAW,CAAC,CAAC,EAAE,MAAM,CAAC,KAAK,OAAO,CAAC,CAAA;IAC5E,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACzB,MAAM,IAAI,aAAa,CAAC,SAAS,EAAE,GAAG,CAAC,MAAM,CAAC,CAAA;IAChD,CAAC;AACH,CAAC"}

package/dist/format-github.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"format-github.d.ts","sourceRoot":"","sources":["../src/format-github.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAA;AAEnD;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,sBAAsB,CACpC,UAAU,EAAE,aAAa,EAAE,EAC3B,QAAQ,GAAE,OAAO,GAAG,SAAmB,GACtC,MAAM,CAiBR;AAED;;;GAGG;AACH,wBAAgB,YAAY,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAEjD"}
+{"version":3,"file":"format-github.d.ts","sourceRoot":"","sources":["../src/format-github.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAA;AAGnD;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,sBAAsB,CACpC,UAAU,EAAE,aAAa,EAAE,EAC3B,QAAQ,GAAE,OAAO,GAAG,SAAmB,GACtC,MAAM,CAuCR;AAED;;;GAGG;AACH,wBAAgB,YAAY,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAEjD"}

package/dist/format-github.js

@@ -1,4 +1,5 @@
 import path from 'node:path';
+import { remedyRepeatsMessage } from './violation.js';
 /**
  * Format violations as GitHub Actions annotation commands.
  *
@@ -20,17 +21,38 @@ export function formatViolationsGitHub(violations, severity = 'error') {
     const cwd = process.cwd();
     return violations
         .map((v) => {
-        const relativePath = path.relative(cwd, v.file);
+        // Escaped as a PROPERTY: the runner splits the property list on commas,
+        // and a path containing one truncates the annotation onto a file that
+        // does not exist (measured). Same treatment as `title` two lines down.
+        const relativePath = escapeGitHubProperty(path.relative(cwd, v.file));
         const title = v.ruleId
             ? `Architecture Violation: ${v.ruleId}`
             : `Architecture Violation: ${v.rule}`;
         let message = v.because ? `${v.message} (${v.because})` : v.message;
-        if (v.suggestion)
-            message += `. Fix: ${v.suggestion}`;
+        // The annotation body opens with `message`, so a remedy identical to it is
+        // already there — see `remedyRepeatsMessage`.
+        //
+        // `join` rather than a bare `'. '`: this format is the one that
+        // concatenates the parts into a single line, and a message already ending
+        // in punctuation got a second period — `…(reading 'config').. Fix: …`,
+        // measured on the real CLI once a producer wrote a well-punctuated
+        // message. The rich and plain formats put `Fix:` on its own line and never
+        // had the problem.
+        const join = (text) => (/[.!?]$/.test(text) ? ' ' : '. ');
+        if (v.suggestion && !remedyRepeatsMessage(v))
+            message += `${join(message)}Fix: ${v.suggestion}`;
         if (v.docs)
-            message += `. Docs: ${v.docs}`;
+            message += `${join(message)}Docs: ${v.docs}`;
         // GitHub annotation format: ::level file=path,line=N,title=T::message
-        return `::${severity} file=${relativePath},line=${String(v.line)},title=${escapeGitHub(title)}::${escapeGitHub(message)}`;
+        //
+        // A configuration finding has no source location (file '', line 0), and
+        // `::error file=,line=0` is not a valid annotation — GitHub drops or
+        // misplaces it. Emit a run-level annotation instead, which renders on
+        // the workflow summary (plan 0070).
+        if (v.file === '') {
+            return `::${severity} title=${escapeGitHubProperty(title)}::${escapeGitHub(message)}`;
+        }
+        return `::${severity} file=${relativePath},line=${String(v.line)},title=${escapeGitHubProperty(title)}::${escapeGitHub(message)}`;
     })
         .join('\n');
 }
@@ -41,4 +63,14 @@ export function formatViolationsGitHub(violations, severity = 'error') {
 export function escapeGitHub(text) {
     return text.replace(/%/g, '%25').replace(/\r/g, '%0D').replace(/\n/g, '%0A');
 }
+/**
+ * Property values (e.g. `title=`) additionally need `,` and `:` escaped —
+ * the workflow-command parser splits the property list on commas, and rule
+ * descriptions legitimately contain both (`{a,b}` globs, prose). The
+ * run-level annotation for locationless findings made `title` the sole
+ * identity carrier, which is what surfaced this.
+ */
+function escapeGitHubProperty(text) {
+    return escapeGitHub(text).replace(/,/g, '%2C').replace(/:/g, '%3A');
+}
 //# sourceMappingURL=format-github.js.map

package/dist/format-github.js.map

@@ -1 +1 @@
-{"version":3,"file":"format-github.js","sourceRoot":"","sources":["../src/format-github.ts"],"names":[],"mappings":"AAAA,OAAO,IAAI,MAAM,WAAW,CAAA;AAG5B;;;;;;;;;;;;;;;;GAgBG;AACH,MAAM,UAAU,sBAAsB,CACpC,UAA2B,EAC3B,WAAgC,OAAO;IAEvC,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,EAAE,CAAA;IAEzB,OAAO,UAAU;SACd,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE;QACT,MAAM,YAAY,GAAG,IAAI,CAAC,QAAQ,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,CAAC,CAAA;QAC/C,MAAM,KAAK,GAAG,CAAC,CAAC,MAAM;YACpB,CAAC,CAAC,2BAA2B,CAAC,CAAC,MAAM,EAAE;YACvC,CAAC,CAAC,2BAA2B,CAAC,CAAC,IAAI,EAAE,CAAA;QACvC,IAAI,OAAO,GAAG,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,OAAO,KAAK,CAAC,CAAC,OAAO,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAA;QACnE,IAAI,CAAC,CAAC,UAAU;YAAE,OAAO,IAAI,UAAU,CAAC,CAAC,UAAU,EAAE,CAAA;QACrD,IAAI,CAAC,CAAC,IAAI;YAAE,OAAO,IAAI,WAAW,CAAC,CAAC,IAAI,EAAE,CAAA;QAE1C,sEAAsE;QACtE,OAAO,KAAK,QAAQ,SAAS,YAAY,SAAS,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,UAAU,YAAY,CAAC,KAAK,CAAC,KAAK,YAAY,CAAC,OAAO,CAAC,EAAE,CAAA;IAC3H,CAAC,CAAC;SACD,IAAI,CAAC,IAAI,CAAC,CAAA;AACf,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,YAAY,CAAC,IAAY;IACvC,OAAO,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC,OAAO,CAAC,KAAK,EAAE,KAAK,CAAC,CAAC,OAAO,CAAC,KAAK,EAAE,KAAK,CAAC,CAAA;AAC9E,CAAC"}
+{"version":3,"file":"format-github.js","sourceRoot":"","sources":["../src/format-github.ts"],"names":[],"mappings":"AAAA,OAAO,IAAI,MAAM,WAAW,CAAA;AAE5B,OAAO,EAAE,oBAAoB,EAAE,MAAM,gBAAgB,CAAA;AAErD;;;;;;;;;;;;;;;;GAgBG;AACH,MAAM,UAAU,sBAAsB,CACpC,UAA2B,EAC3B,WAAgC,OAAO;IAEvC,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,EAAE,CAAA;IAEzB,OAAO,UAAU;SACd,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE;QACT,wEAAwE;QACxE,sEAAsE;QACtE,uEAAuE;QACvE,MAAM,YAAY,GAAG,oBAAoB,CAAC,IAAI,CAAC,QAAQ,CAAC,GAAG,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,CAAA;QACrE,MAAM,KAAK,GAAG,CAAC,CAAC,MAAM;YACpB,CAAC,CAAC,2BAA2B,CAAC,CAAC,MAAM,EAAE;YACvC,CAAC,CAAC,2BAA2B,CAAC,CAAC,IAAI,EAAE,CAAA;QACvC,IAAI,OAAO,GAAG,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,OAAO,KAAK,CAAC,CAAC,OAAO,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAA;QACnE,2EAA2E;QAC3E,8CAA8C;QAC9C,EAAE;QACF,gEAAgE;QAChE,0EAA0E;QAC1E,uEAAuE;QACvE,mEAAmE;QACnE,2EAA2E;QAC3E,mBAAmB;QACnB,MAAM,IAAI,GAAG,CAAC,IAAY,EAAU,EAAE,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,CAAA;QACzE,IAAI,CAAC,CAAC,UAAU,IAAI,CAAC,oBAAoB,CAAC,CAAC,CAAC;YAC1C,OAAO,IAAI,GAAG,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC,UAAU,EAAE,CAAA;QACnD,IAAI,CAAC,CAAC,IAAI;YAAE,OAAO,IAAI,GAAG,IAAI,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC,IAAI,EAAE,CAAA;QAExD,sEAAsE;QACtE,EAAE;QACF,wEAAwE;QACxE,qEAAqE;QACrE,sEAAsE;QACtE,oCAAoC;QACpC,IAAI,CAAC,CAAC,IAAI,KAAK,EAAE,EAAE,CAAC;YAClB,OAAO,KAAK,QAAQ,UAAU,oBAAoB,CAAC,KAAK,CAAC,KAAK,YAAY,CAAC,OAAO,CAAC,EAAE,CAAA;QACvF,CAAC;QACD,OAAO,KAAK,QAAQ,SAAS,YAAY,SAAS,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,UAAU,oBAAoB,CAAC,KAAK,CAAC,KAAK,YAAY,CAAC,OAAO,CAAC,EAAE,CAAA;IACnI,CAAC,CAAC;SACD,IAAI,CAAC,IAAI,CAAC,CAAA;AACf,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,YAAY,CAAC,IAAY;IACvC,OAAO,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC,OAAO,CAAC,KAAK,EAAE,KAAK,CAAC,CAAC,OAAO,CAAC,KAAK,EAAE,KAAK,CAAC,CAAA;AAC9E,CAAC;AAED;;;;;;GAMG;AACH,SAAS,oBAAoB,CAAC,IAAY;IACxC,OAAO,YAAY,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC,OAAO,CAAC,IAAI,EAAE,KAAK,CAAC,CAAA;AACrE,CAAC"}

package/dist/format-json.d.ts

@@ -1,4 +1,5 @@
 import type { ArchViolation } from './violation.js';
+import type { EdgeCoverage } from './edge-coverage.js';
 /**
  * Format violations as a JSON string.
  *
@@ -8,5 +9,86 @@ import type { ArchViolation } from './violation.js';
  * const violations = collectViolations(rule1, rule2)
  * console.log(formatViolationsJson(violations))
  */
-export declare function formatViolationsJson(violations: ArchViolation[], reason?: string): string;
+/**
+ * The `check --format json` document, as a type.
+ *
+ * Exported because it is a **contract**, and an unexported contract is one a
+ * consumer can only discover by breaking. Without it a TypeScript consumer gets
+ * a runtime failure where they could have had a red build — and our own tests
+ * had to hand-roll the shape and cast through `unknown`, which is the kind of
+ * ADR-005 exemption that should not be needed against our own output.
+ *
+ * Changing these types is changing the agent contract. Treat a field removal or
+ * a narrowed union as breaking.
+ */
+export interface ArchJsonViolation {
+    readonly rule: string;
+    readonly ruleId: string | null;
+    readonly severity: 'error' | 'warn';
+    readonly element: string;
+    /**
+     * The source file, or `null` when the finding has no location.
+     *
+     * For `kind: 'configuration'` this is the **rule file** that declared the
+     * rule — not the code under test — because the CLI attributes findings to
+     * their origin before rendering. It is `null` only for findings produced
+     * after that step. Never use it to detect a configuration finding; use `kind`.
+     */
+    readonly file: string | null;
+    /** `null` when there is no location; `1` on an attributed file means the file, not a position. */
+    readonly line: number | null;
+    /**
+     * What kind of finding this is, and therefore what to do about it.
+     *
+     * `'violation'` — the code is wrong. `'configuration'` — the **rule**
+     * enforces nothing, so editing the code cannot clear it.
+     *
+     * A consumer should treat an **unrecognised** value as `'violation'`: that
+     * keeps widening this union non-breaking, and configuration findings today
+     * already span four distinct remedies (a rule that matched nothing, a rule
+     * file that could not be evaluated, rules after a failure that never ran, and
+     * a stale baseline) which may earn their own names later.
+     */
+    readonly kind: 'violation' | 'configuration';
+    readonly message: string;
+    readonly because: string | null;
+    readonly suggestion: string | null;
+    readonly docs: string | null;
+    readonly codeFrame: string | null;
+    readonly measured: number | null;
+}
+/** One finding removed by an inline `// eess-exclude` comment. */
+export interface ArchJsonSuppression {
+    readonly ruleId: string;
+    readonly file: string;
+}
+/** An allowlist rule that had subjects but tested no edges. */
+export interface ArchJsonUntestedAllowlist {
+    readonly rule: string;
+    readonly subjects: number;
+    readonly edges: number;
+}
+/** The whole document emitted by `check --format json`. */
+export interface ArchJsonReport {
+    readonly summary: {
+        readonly total: number;
+        readonly errors: number;
+        readonly warnings: number;
+        readonly reason: string | null;
+    };
+    readonly untestedAllowlists: readonly ArchJsonUntestedAllowlist[];
+    readonly commentSuppressed: readonly ArchJsonSuppression[];
+    readonly violations: readonly ArchJsonViolation[];
+}
+export declare function formatViolationsJson(violations: ArchViolation[], reason?: string, 
+/**
+ * Allowlist rules that tested no edges (bug 0015).
+ *
+ * **Passed in, not read from module state.** The first cut called
+ * `untestedRules()` here, which made a documented pure formatter impure: a
+ * per-rule JSON document in a vitest suite named rules from every earlier
+ * rule in the process, because nothing resets between them. Same input,
+ * different output depending on history.
+ */
+untested?: readonly EdgeCoverage[]): string;
 //# sourceMappingURL=format-json.d.ts.map

package/dist/format-json.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"format-json.d.ts","sourceRoot":"","sources":["../src/format-json.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAA;AAEnD;;;;;;;;GAQG;AACH,wBAAgB,oBAAoB,CAAC,UAAU,EAAE,aAAa,EAAE,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,MAAM,CAmBzF"}
+{"version":3,"file":"format-json.d.ts","sourceRoot":"","sources":["../src/format-json.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAA;AACnD,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAA;AAGtD;;;;;;;;GAQG;AACH;;;;;;;;;;;GAWG;AACH,MAAM,WAAW,iBAAiB;IAChC,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAA;IACrB,QAAQ,CAAC,MAAM,EAAE,MAAM,GAAG,IAAI,CAAA;IAC9B,QAAQ,CAAC,QAAQ,EAAE,OAAO,GAAG,MAAM,CAAA;IACnC,QAAQ,CAAC,OAAO,EAAE,MAAM,CAAA;IACxB;;;;;;;OAOG;IACH,QAAQ,CAAC,IAAI,EAAE,MAAM,GAAG,IAAI,CAAA;IAC5B,kGAAkG;IAClG,QAAQ,CAAC,IAAI,EAAE,MAAM,GAAG,IAAI,CAAA;IAC5B;;;;;;;;;;;OAWG;IACH,QAAQ,CAAC,IAAI,EAAE,WAAW,GAAG,eAAe,CAAA;IAC5C,QAAQ,CAAC,OAAO,EAAE,MAAM,CAAA;IACxB,QAAQ,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAAA;IAC/B,QAAQ,CAAC,UAAU,EAAE,MAAM,GAAG,IAAI,CAAA;IAClC,QAAQ,CAAC,IAAI,EAAE,MAAM,GAAG,IAAI,CAAA;IAC5B,QAAQ,CAAC,SAAS,EAAE,MAAM,GAAG,IAAI,CAAA;IACjC,QAAQ,CAAC,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAA;CACjC;AAED,kEAAkE;AAClE,MAAM,WAAW,mBAAmB;IAClC,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAA;IACvB,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAA;CACtB;AAED,+DAA+D;AAC/D,MAAM,WAAW,yBAAyB;IACxC,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAA;IACrB,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAA;IACzB,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAA;CACvB;AAED,2DAA2D;AAC3D,MAAM,WAAW,cAAc;IAC7B,QAAQ,CAAC,OAAO,EAAE;QAChB,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAA;QACtB,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAA;QACvB,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAA;QACzB,QAAQ,CAAC,MAAM,EAAE,MAAM,GAAG,IAAI,CAAA;KAC/B,CAAA;IACD,QAAQ,CAAC,kBAAkB,EAAE,SAAS,yBAAyB,EAAE,CAAA;IACjE,QAAQ,CAAC,iBAAiB,EAAE,SAAS,mBAAmB,EAAE,CAAA;IAC1D,QAAQ,CAAC,UAAU,EAAE,SAAS,iBAAiB,EAAE,CAAA;CAClD;AAED,wBAAgB,oBAAoB,CAClC,UAAU,EAAE,aAAa,EAAE,EAC3B,MAAM,CAAC,EAAE,MAAM;AACf;;;;;;;;GAQG;AACH,QAAQ,GAAE,SAAS,YAAY,EAAO,GACrC,MAAM,CA8ER"}

package/dist/format-json.js

@@ -1,28 +1,89 @@
+import { commentSuppressions } from './comment-suppression.js';
+export function formatViolationsJson(violations, reason, 
 /**
- * Format violations as a JSON string.
+ * Allowlist rules that tested no edges (bug 0015).
  *
- * Useful for CI pipelines, custom dashboards, or piping to other tools.
- *
- * @example
- * const violations = collectViolations(rule1, rule2)
- * console.log(formatViolationsJson(violations))
+ * **Passed in, not read from module state.** The first cut called
+ * `untestedRules()` here, which made a documented pure formatter impure: a
+ * per-rule JSON document in a vitest suite named rules from every earlier
+ * rule in the process, because nothing resets between them. Same input,
+ * different output depending on history.
  */
-export function formatViolationsJson(violations, reason) {
+untested = []) {
+    const errors = violations.filter((v) => (v.severity ?? 'error') === 'error').length;
     const output = {
         summary: {
             total: violations.length,
+            errors,
+            warnings: violations.length - errors,
             reason: reason ?? null,
         },
+        // Bug 0015: an allowlist constrains edges, so a subject with none passes
+        // whatever the allowlist says. Reported rather than failed — for the `only*`
+        // family zero edges is maximal compliance, and only the reader can tell a
+        // dependency-free module from a rule that certified nothing. Always present
+        // (an empty array, not omitted) so a consumer can distinguish "none" from
+        // "this version does not report it".
+        untestedAllowlists: untested.map((c) => ({
+            rule: c.rule,
+            subjects: c.subjects,
+            edges: c.edges,
+        })),
+        // What inline `// eess-exclude` comments removed from this report.
+        // Structural rather than prose, for the same reason `untestedAllowlists` is:
+        // the agent reading this document needs identities it can act on, not a
+        // sentence to grep. Always present — an empty array, not omitted — so a
+        // consumer can tell "nothing was suppressed" from "this version does not
+        // report it".
+        //
+        // Every other filter in the pipeline discloses itself; this one did not,
+        // which after bug 0041 made it the widest silent filter we ship.
+        commentSuppressed: commentSuppressions().map((c) => ({
+            ruleId: c.ruleId,
+            file: c.file,
+        })),
         violations: violations.map((v) => ({
             rule: v.rule,
             ruleId: v.ruleId ?? null,
+            severity: v.severity ?? 'error',
             element: v.element,
-            file: v.file,
-            line: v.line,
+            // `null`, not `''`/`1`, when there is no source location. A configuration
+            // finding reports that a RULE enforces nothing; it has no line, and saying
+            // `"file": "", "line": 1` states one that looks real (bug 0047). A human
+            // skims past it; an agent may open it or anchor an edit to it.
+            //
+            // `null` rather than omission, and consistent with the rest of this
+            // document — `ruleId`, `because`, `suggestion`, `docs` and `measured` all
+            // null when absent, so a consumer already handles the idiom here.
+            file: v.file === '' ? null : v.file,
+            line: v.file === '' ? null : v.line,
+            // What kind of finding this is, which drives what the reader should DO:
+            // `'violation'` — the code is wrong, edit the named file;
+            // `'configuration'` — the RULE enforces nothing, so editing the code
+            // cannot clear it. This distinction was absent from the payload entirely,
+            // so an agent could not tell them apart by any field.
+            //
+            // A string rather than a boolean, decided deliberately. There are twelve
+            // configuration-finding producers with materially different causes — an
+            // empty selector, a dead glob, a rule with no condition, a preset that
+            // discovered nothing — and someone will want to distinguish them. A
+            // boolean leaves nowhere to put that; adding a second overlapping field
+            // later is worse than one field with room in it. This is the agent
+            // contract, so the cost of the wrong shape is permanent.
+            //
+            // Not named after `bypassFilters`: the consumer cares what the finding
+            // IS, not which of our filters it happens to survive.
+            kind: v.bypassFilters === true ? 'configuration' : 'violation',
             message: v.message,
             because: v.because ?? null,
             suggestion: v.suggestion ?? null,
             docs: v.docs ?? null,
+            codeFrame: v.codeFrame ?? null,
+            // Bug 0012's measurement, on the wire. Without it the ratchet is
+            // machine-readable only inside the baseline file, and a dashboard wanting
+            // the number has to regex it back out of the message — which is the
+            // fragility bug 0012 was filed about.
+            measured: v.measured ?? null,
         })),
     };
     return JSON.stringify(output, null, 2);

package/dist/format-json.js.map

@@ -1 +1 @@
-{"version":3,"file":"format-json.js","sourceRoot":"","sources":["../src/format-json.ts"],"names":[],"mappings":"AAEA;;;;;;;;GAQG;AACH,MAAM,UAAU,oBAAoB,CAAC,UAA2B,EAAE,MAAe;IAC/E,MAAM,MAAM,GAAG;QACb,OAAO,EAAE;YACP,KAAK,EAAE,UAAU,CAAC,MAAM;YACxB,MAAM,EAAE,MAAM,IAAI,IAAI;SACvB;QACD,UAAU,EAAE,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YACjC,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,MAAM,EAAE,CAAC,CAAC,MAAM,IAAI,IAAI;YACxB,OAAO,EAAE,CAAC,CAAC,OAAO;YAClB,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,OAAO,EAAE,CAAC,CAAC,OAAO;YAClB,OAAO,EAAE,CAAC,CAAC,OAAO,IAAI,IAAI;YAC1B,UAAU,EAAE,CAAC,CAAC,UAAU,IAAI,IAAI;YAChC,IAAI,EAAE,CAAC,CAAC,IAAI,IAAI,IAAI;SACrB,CAAC,CAAC;KACJ,CAAA;IACD,OAAO,IAAI,CAAC,SAAS,CAAC,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC,CAAA;AACxC,CAAC"}
+{"version":3,"file":"format-json.js","sourceRoot":"","sources":["../src/format-json.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,mBAAmB,EAAE,MAAM,0BAA0B,CAAA;AAsF9D,MAAM,UAAU,oBAAoB,CAClC,UAA2B,EAC3B,MAAe;AACf;;;;;;;;GAQG;AACH,WAAoC,EAAE;IAEtC,MAAM,MAAM,GAAG,UAAU,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,QAAQ,IAAI,OAAO,CAAC,KAAK,OAAO,CAAC,CAAC,MAAM,CAAA;IACnF,MAAM,MAAM,GAAG;QACb,OAAO,EAAE;YACP,KAAK,EAAE,UAAU,CAAC,MAAM;YACxB,MAAM;YACN,QAAQ,EAAE,UAAU,CAAC,MAAM,GAAG,MAAM;YACpC,MAAM,EAAE,MAAM,IAAI,IAAI;SACvB;QACD,yEAAyE;QACzE,6EAA6E;QAC7E,0EAA0E;QAC1E,4EAA4E;QAC5E,0EAA0E;QAC1E,qCAAqC;QACrC,kBAAkB,EAAE,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YACvC,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,QAAQ,EAAE,CAAC,CAAC,QAAQ;YACpB,KAAK,EAAE,CAAC,CAAC,KAAK;SACf,CAAC,CAAC;QACH,mEAAmE;QACnE,6EAA6E;QAC7E,wEAAwE;QACxE,wEAAwE;QACxE,yEAAyE;QACzE,cAAc;QACd,EAAE;QACF,yEAAyE;QACzE,iEAAiE;QACjE,iBAAiB,EAAE,mBAAmB,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YACnD,MAAM,EAAE,CAAC,CAAC,MAAM;YAChB,IAAI,EAAE,CAAC,CAAC,IAAI;SACb,CAAC,CAAC;QACH,UAAU,EAAE,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;YACjC,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,MAAM,EAAE,CAAC,CAAC,MAAM,IAAI,IAAI;YACxB,QAAQ,EAAE,CAAC,CAAC,QAAQ,IAAI,OAAO;YAC/B,OAAO,EAAE,CAAC,CAAC,OAAO;YAClB,0EAA0E;YAC1E,2EAA2E;YAC3E,yEAAyE;YACzE,+DAA+D;YAC/D,EAAE;YACF,oEAAoE;YACpE,0EAA0E;YAC1E,kEAAkE;YAClE,IAAI,EAAE,CAAC,CAAC,IAAI,KAAK,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI;YACnC,IAAI,EAAE,CAAC,CAAC,IAAI,KAAK,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI;YACnC,wEAAwE;YACxE,0DAA0D;YAC1D,qEAAqE;YACrE,0EAA0E;YAC1E,sDAAsD;YACtD,EAAE;YACF,yEAAyE;YACzE,wEAAwE;YACxE,uEAAuE;YACvE,oEAAoE;YACpE,wEAAwE;YACxE,mEAAmE;YACnE,yDAAyD;YACzD,EAAE;YACF,uEAAuE;YACvE,sDAAsD;YACtD,IAAI,EAAE,CAAC,CAAC,aAAa,KAAK,IAAI,CAAC,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,WAAW;YAC9D,OAAO,EAAE,CAAC,CAAC,OAAO;YAClB,OAAO,EAAE,CAAC,CAAC,OAAO,IAAI,IAAI;YAC1B,UAAU,EAAE,CAAC,CAAC,UAAU,IAAI,IAAI;YAChC,IAAI,EAAE,CAAC,CAAC,IAAI,IAAI,IAAI;YACpB,SAAS,EAAE,CAAC,CAAC,SAAS,IAAI,IAAI;YAC9B,iEAAiE;YACjE,0EAA0E;YAC1E,oEAAoE;YACpE,sCAAsC;YACtC,QAAQ,EAAE,CAAC,CAAC,QAAQ,IAAI,IAAI;SAC7B,CAAC,CAAC;KACJ,CAAA;IACD,OAAO,IAAI,CAAC,SAAS,CAAC,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC,CAAA;AACxC,CAAC"}

package/dist/format.d.ts

@@ -18,8 +18,10 @@ export declare function formatViolations(violations: ArchViolation[], reason?: s
 /**
  * Format violations into a plain-text string (no ANSI codes).
  *
- * Used by ArchRuleError.message — error messages should be plain text
- * since they may be captured by test runners, serialized, or logged to files.
+ * Public export, for callers that aggregate violations themselves and need
+ * output free of ANSI codes — serialized, logged to a file, or embedded in
+ * another tool's report. `ArchRuleError.message` is a one-line summary and does
+ * not use this; the run's detail is written by `writeReport`.
  */
 export declare function formatViolationsPlain(violations: ArchViolation[], reason?: string): string;
 //# sourceMappingURL=format.d.ts.map

package/dist/format.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"format.d.ts","sourceRoot":"","sources":["../src/format.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAA;AAInD;;GAEG;AACH,MAAM,WAAW,aAAa;IAC5B,0EAA0E;IAC1E,GAAG,CAAC,EAAE,MAAM,CAAA;IACZ,8DAA8D;IAC9D,UAAU,CAAC,EAAE,OAAO,CAAA;CACrB;AA0CD;;;;;GAKG;AACH,wBAAgB,gBAAgB,CAC9B,UAAU,EAAE,aAAa,EAAE,EAC3B,MAAM,CAAC,EAAE,MAAM,EACf,OAAO,CAAC,EAAE,aAAa,GACtB,MAAM,CAYR;AAED;;;;;GAKG;AACH,wBAAgB,qBAAqB,CAAC,UAAU,EAAE,aAAa,EAAE,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,MAAM,CAkB1F"}
+{"version":3,"file":"format.d.ts","sourceRoot":"","sources":["../src/format.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAA;AAKnD;;GAEG;AACH,MAAM,WAAW,aAAa;IAC5B,0EAA0E;IAC1E,GAAG,CAAC,EAAE,MAAM,CAAA;IACZ,8DAA8D;IAC9D,UAAU,CAAC,EAAE,OAAO,CAAA;CACrB;AAoDD;;;;;GAKG;AACH,wBAAgB,gBAAgB,CAC9B,UAAU,EAAE,aAAa,EAAE,EAC3B,MAAM,CAAC,EAAE,MAAM,EACf,OAAO,CAAC,EAAE,aAAa,GACtB,MAAM,CAYR;AAED;;;;;;;GAOG;AACH,wBAAgB,qBAAqB,CAAC,UAAU,EAAE,aAAa,EAAE,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,MAAM,CA2B1F"}

package/dist/format.js

@@ -1,28 +1,39 @@
+import { remedyRepeatsMessage } from './violation.js';
 import { bold, red, cyan, dim } from './ansi.js';
 import path from 'node:path';
 /** Format a single violation into a rich terminal section. */
 function formatSingleViolation(v, index, total, cwd, showCodeFrames, reason) {
     const counter = bold(red(`Architecture Violation [${String(index + 1)} of ${String(total)}]`));
     const ruleLine = `  ${dim('Rule:')} ${v.rule}`;
-    const relativePath = path.relative(cwd, v.file);
-    const locationRef = cyan(relativePath + ':' + String(v.line));
-    const location = `  ${locationRef} ${dim('—')} ${v.element}`;
+    // `message` is printed for BOTH shapes, and it used to be printed for neither
+    // — a located violation rendered only `file:line — element`, so the one
+    // sentence saying what is actually wrong was dropped from the default surface
+    // for every ordinary violation. Nothing failed when this was fixed, because
+    // nothing pinned it either way; `formatViolationsPlain` had always printed it,
+    // so the two formatters disagreed about what a violation IS.
+    //
+    // A config-level finding has no source location: `path.relative(cwd, '')`
+    // renders as the cwd and ':0' is noise, so it gets the message alone.
+    const location = v.file
+        ? `  ${v.message}\n  ${cyan(path.relative(cwd, v.file) + ':' + String(v.line))} ${dim('—')} ${v.element}`
+        : `  ${v.message}`;
     const codeLine = showCodeFrames && v.codeFrame ? `\n${v.codeFrame}` : '';
     const whyText = v.because ?? reason;
     const whyLine = whyText ? `  ${dim('Why:')} ${whyText}` : '';
-    const fixLine = v.suggestion ? `  ${dim('Fix:')} ${v.suggestion}` : '';
+    // Suppressed whenever the `location` slot above already printed this exact
+    // text, which is now BOTH shapes — see the note there. It used to be
+    // `!v.file && remedyRepeatsMessage(v)`, and that asymmetry was load-bearing
+    // and pinned, because a located violation did not render `message` at all: its
+    // `Fix:` line was the remedy's only appearance. Rendering `message` for both
+    // shapes removed the premise, and the `!v.file` half became a defect —
+    // measured, two occurrences of the remedy for a located finding whose
+    // suggestion is its message.
+    //
+    // So do not "restore" the asymmetry from the old comment: check whether this
+    // formatter still prints `message` for a located violation first.
+    const fixLine = v.suggestion && !remedyRepeatsMessage(v) ? `  ${dim('Fix:')} ${v.suggestion}` : '';
     const docsLine = v.docs ? `  ${dim('Docs:')} ${v.docs}` : '';
-    // The message is the finding itself. It was omitted here for years, which is
-    // survivable for a one-sided rule — `element` plus the rule description plus a
-    // code frame usually carry the meaning — and not survivable for a two-sided
-    // one, where `message` is the ONLY place the correspondence states which side
-    // drifted. Measured before adding: `check:spec` rendered a ghost ADR row as
-    // `Rule: correspondence / CLAUDE.md:24 — 099 / Why: …` and never said what was
-    // wrong. Rendered in full, not just the first line: `correspondence()` folds
-    // its per-side `suggest` remedy onto a continuation line, so truncating here
-    // is what made that remedy invisible.
-    const messageLine = `  ${dim('What:')} ${v.message.split('\n').join('\n  ')}`;
-    const parts = [counter, '', ruleLine, '', location, messageLine];
+    const parts = [counter, '', ruleLine, '', location];
     if (codeLine)
         parts.push(codeLine);
     if (whyLine)
@@ -51,8 +62,10 @@ export function formatViolations(violations, reason, options) {
 /**
  * Format violations into a plain-text string (no ANSI codes).
  *
- * Used by ArchRuleError.message — error messages should be plain text
- * since they may be captured by test runners, serialized, or logged to files.
+ * Public export, for callers that aggregate violations themselves and need
+ * output free of ANSI codes — serialized, logged to a file, or embedded in
+ * another tool's report. `ArchRuleError.message` is a one-line summary and does
+ * not use this; the run's detail is written by `writeReport`.
  */
 export function formatViolationsPlain(violations, reason) {
     if (violations.length === 0)
@@ -61,12 +74,21 @@ export function formatViolationsPlain(violations, reason) {
     const reasonLine = reason ? `\nReason: ${reason}` : '';
     const details = violations
         .map((v, i) => {
+        // A configuration finding has no source location — it reports that a RULE
+        // enforces nothing, not that a line is wrong — so it carries `file: ''`
+        // and a `line` that means nothing. Rendering it produced a bare `(:1)`
+        // (bug 0047). The rich formatter and the GitHub formatter
+        // (`format-github.ts:58`) both already special-case this; plain and JSON
+        // were the two that did not.
+        const where = v.file === '' ? '' : ` (${v.file}:${String(v.line)})`;
         const parts = [
-            `  [${String(i + 1)}/${String(violations.length)}] ${v.element}: ${v.message} (${v.file}:${String(v.line)})`,
+            `  [${String(i + 1)}/${String(violations.length)}] ${v.element}: ${v.message}${where}`,
         ];
         if (v.codeFrame)
             parts.push(v.codeFrame);
-        if (v.suggestion)
+        // This format always renders `message`, so a remedy identical to it is
+        // already shown — see `remedyRepeatsMessage`.
+        if (v.suggestion && !remedyRepeatsMessage(v))
             parts.push(`  Fix: ${v.suggestion}`);
         return parts.join('\n');
     })