@chrisdudek/yg 5.1.1 → 5.2.0

package/dist/structure.d.ts

@@ -243,7 +243,7 @@ interface LlmConfig {
     timeout?: number;
     /**
      * Optional per-tier cap on assembled reviewer-prompt length in characters.
-     * Absent = unlimited. This is a GATE checked deterministically before the LLM
+     * Absent defaults to 50000. This is a GATE checked deterministically before the LLM
      * call — it never participates in verdict identity or hash computation (excluded
      * from canonicalTierJson like api_key and timeout).
      */
@@ -460,4 +460,53 @@ interface RunStructureAspectResult {
 }
 declare function runStructureAspect(params: RunStructureAspectParams): Promise<RunStructureAspectResult>;
 
-export { type CheckFunction, type CompanionDescriptor, type CompanionFunction, type Ctx, type File, type FsEntry, type GraphNode$1 as GraphNode, type Port, type Relation$1 as Relation, type RelationType$1 as RelationType, StructureRunnerError, type Violation, runStructureAspect };
+/**
+ * Pre-resolved suppress line ranges injected into the reviewer prompt so the LLM
+ * honors EXACTLY the same `(file, startLine..endLine)` spans the deterministic
+ * matcher (`ast/suppress.ts`) computes — no model-side re-derivation of marker
+ * scope. `byFile` carries only files that have at least one applicable range;
+ * an empty `byFile` (or an omitted `suppressedRanges`) renders no block and keeps
+ * the prompt byte-identical to the no-suppress case.
+ */
+interface PromptSuppressedRangesInput {
+    byFile: Array<{
+        path: string;
+        ranges: Array<{
+            startLine: number;
+            endLine: number;
+        }>;
+    }>;
+}
+
+declare class SuppressMarkerError extends Error {
+    file: string;
+    line: number;
+    readonly code = "SUPPRESS_MARKER_MISSING_REASON";
+    constructor(message: string, file: string, line: number);
+}
+
+/**
+ * Resolve, per subject file, the suppress line ranges that apply to `aspectId`,
+ * shaped for injection into the reviewer prompt.
+ *
+ * Parse strategy mirrors the deterministic AST runner (`ast/runner.ts`): a file
+ * whose extension has a registered tree-sitter grammar is parsed and its markers
+ * read from comment nodes; a file with no grammar (`.sql`, `.md`, `.sh`, …) gets
+ * a raw-line content scan. Either way `collectSuppressions` produces the spans
+ * and `formatSuppressedRangesForAspect` filters them to this aspect (wildcard
+ * markers included).
+ *
+ * Only files with at least one applicable range appear in `byFile`, so an empty
+ * result renders no `<suppressed-ranges>` block and the prompt stays byte-identical
+ * to the no-suppress case.
+ *
+ * A reasonless marker throws `SuppressMarkerError` (out of `collectSuppressions`)
+ * — the caller treats that as an infrastructure failure and writes nothing, the
+ * same fail-closed disposition the deterministic path takes.
+ */
+declare function resolveSuppressedRangesForPrompt(subjects: Array<{
+    path: string;
+    bytes: Buffer;
+}>, aspectId: string): Promise<PromptSuppressedRangesInput>;
+
+export { type CheckFunction, type CompanionDescriptor, type CompanionFunction, type Ctx, type File, type FsEntry, type GraphNode$1 as GraphNode, type Port, type Relation$1 as Relation, type RelationType$1 as RelationType, StructureRunnerError, SuppressMarkerError, type Violation, resolveSuppressedRangesForPrompt, runStructureAspect };

package/dist/structure.js

@@ -820,6 +820,9 @@ function isLineSuppressed(ranges, aspectId, line) {
     return r.isWildcard || r.aspectIds.has(aspectId);
   });
 }
+function formatSuppressedRangesForAspect(ranges, aspectId) {
+  return ranges.filter((r) => r.isWildcard || r.aspectIds.has(aspectId)).map((r) => ({ startLine: r.startLine, endLine: r.endLine })).sort((a, b) => a.startLine - b.startLine || a.endLine - b.endLine);
+}
 
 // src/utils/validate-check-module.ts
 function validateCheckModuleExport(mod, opts) {
@@ -1571,6 +1574,27 @@ ${err.stack ?? ""}`,
   };
 }
 
+// src/structure/suppress-ranges.ts
+import { extname as extname5 } from "path";
+async function resolveSuppressedRangesForPrompt(subjects, aspectId) {
+  const byFile = [];
+  for (const subject of subjects) {
+    const content = subject.bytes.toString("utf8");
+    const hasGrammar = getLanguageForExtension(extname5(subject.path).toLowerCase()) !== null;
+    let tree;
+    if (hasGrammar) {
+      tree = await parseFile(subject.path, content);
+    }
+    const totalLines = content.split("\n").length;
+    const all = collectSuppressions(tree, subject.path, totalLines, content);
+    const ranges = formatSuppressedRangesForAspect(all, aspectId);
+    if (ranges.length > 0) {
+      byFile.push({ path: subject.path, ranges });
+    }
+  }
+  return { byFile };
+}
+
 // src/ast/walk.ts
 function walk(node, visitor) {
   const result = visitor(node);
@@ -1609,10 +1633,12 @@ function inFile(file, pattern) {
 }
 export {
   StructureRunnerError,
+  SuppressMarkerError,
   closest,
   findComments,
   inFile,
   report,
+  resolveSuppressedRangesForPrompt,
   runStructureAspect,
   walk
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chrisdudek/yg",
-  "version": "5.1.1",
+  "version": "5.2.0",
   "description": "Architecture rules your AI coding agent can't ignore. It gets the rules for a file before it edits, and every change is checked — by a free local script or an LLM reviewer — before it moves on. Works with Claude Code, Cursor, Copilot, Codex, Cline.",
   "type": "module",
   "bin": {