@chrisdudek/yg 5.2.2 → 5.2.4

package/README.md

@@ -1,8 +1,8 @@
 # @chrisdudek/yg
 
-**Your agent will ignore CLAUDE.md. Yggdrasil makes sure it doesn't.**
+**Stop babysitting your agent.**
 
-Architecture rules your agent can't ignore. Before it edits a file, your agent gets the few rules that apply — and writes to them. After, every change is checked before it moves on: by a script that runs locally for free, or by an LLM reviewer. A rule written as a script *runs* — your agent can't quietly optimize it away the way it drops a line in CLAUDE.md. Works with Claude Code, Cursor, Copilot, Codex, Cline, and more. Checks run against your code, not your diffs; the feedback is specific; the agent has to fix before it can move on.
+Your architecture rules become checks it can't skip, run on every change before it moves on. A script runs them locally for free, or an LLM reviews the call a script can't make. Checks run against your code, not your diffs. The feedback is specific, and the agent has to fix before it can move on. Works with Claude Code, Cursor, Copilot, Codex, Cline, and more.
 
 See the [main README](https://github.com/krzysztofdudek/Yggdrasil#readme) for documentation, or visit
 [krzysztofdudek.github.io/Yggdrasil](https://krzysztofdudek.github.io/Yggdrasil/).

package/dist/bin.js

@@ -3621,13 +3621,12 @@ async function writeLock(yggRoot, lock, opts = {}) {
   }
   const { det, nondet } = partitionVerdicts(lock.verdicts, detIds);
   if (scope === "deterministic") {
-    const content20 = serializeLock({ version: lock.version, verdicts: det, nodes: {} });
-    await writeFileIfChanged(detLockPath(yggRoot), content20);
+    await writeOrRemoveSplitFile(detLockPath(yggRoot), lock.version, det, {});
     return;
   }
-  await writeFileIfChanged(nondetLockPath(yggRoot), serializeLock({ version: lock.version, verdicts: nondet, nodes: {} }));
+  await writeOrRemoveSplitFile(nondetLockPath(yggRoot), lock.version, nondet, {});
   await writeOrRemoveSplitFile(logsLockPath(yggRoot), lock.version, {}, lock.nodes);
-  await writeFileIfChanged(detLockPath(yggRoot), serializeLock({ version: lock.version, verdicts: det, nodes: {} }));
+  await writeOrRemoveSplitFile(detLockPath(yggRoot), lock.version, det, {});
 }
 
 // src/cli/preamble.ts
@@ -15822,20 +15821,20 @@ function registerCheckCommand(program2) {
       const gitFiles = collectGitFiles(projectRoot);
       const wantsTop = opts.top !== void 0;
       if (wantsTop && opts.summary) {
-        process.stdout.write(buildIssueMessage({
+        process.stderr.write(chalk8.red(buildIssueMessage({
           what: "--top and --summary cannot be combined.",
           why: "Both are read-only triage VIEWS of the same `yg check` result \u2014 --top renders the N highest-priority blocks, --summary renders per-node counts only. Asking for both at once is ambiguous; pick one lens.",
           next: "Run: yg check --top <n> (priority blocks), or yg check --summary (per-node counts)."
-        }) + "\n");
+        }) + "\n"));
         await exitAfterFlush(1);
         return;
       }
       if ((wantsTop || opts.summary) && opts.approve) {
-        process.stdout.write(buildIssueMessage({
+        process.stderr.write(chalk8.red(buildIssueMessage({
           what: `${wantsTop ? "--top" : "--summary"} cannot be combined with --approve.`,
           why: "--top and --summary triage the READ-ONLY check wall (they narrow the output of plain `yg check`, which writes nothing). --approve is the writer path; its own free cost preview is --dry-run. Mixing a read-only triage view with the writer is contradictory.",
           next: `Run: yg check ${wantsTop ? "--top <n>" : "--summary"} (read-only triage), or yg check --approve --dry-run (preview the writer's cost).`
-        }) + "\n");
+        }) + "\n"));
         await exitAfterFlush(1);
         return;
       }
@@ -15845,22 +15844,22 @@ function registerCheckCommand(program2) {
       } else if (wantsTop) {
         const n = resolveTopValue(opts.top);
         if (n === null) {
-          process.stdout.write(buildIssueMessage({
+          process.stderr.write(chalk8.red(buildIssueMessage({
             what: `--top expects a non-negative whole number; got "${String(opts.top)}".`,
             why: "--top N prints the N highest-priority issue blocks. A negative, fractional, or non-numeric value is meaningless, and printing the full wall instead would silently hide that the flag was ignored \u2014 masking the very output you tried to narrow.",
             next: "Run: yg check --top 5 (top 5 blocks), yg check --top (just the suggestedNext block), or yg check (full output)."
-          }) + "\n");
+          }) + "\n"));
           await exitAfterFlush(1);
           return;
         }
         view = { kind: "top", n };
       }
       if (opts.dryRun && !opts.approve) {
-        process.stdout.write(buildIssueMessage({
+        process.stderr.write(chalk8.red(buildIssueMessage({
           what: "--dry-run requires --approve.",
           why: "--dry-run previews what `yg check --approve` would fill (the reviewer-call budget and per-node breakdown) without writing or calling the reviewer; it is a mode of --approve, not a variant of the plain read. Plain `yg check` is already a free, no-write read.",
           next: "Run: yg check --approve --dry-run (cost preview), or yg check (plain read)."
-        }) + "\n");
+        }) + "\n"));
         await exitAfterFlush(1);
         return;
       }
@@ -15880,7 +15879,7 @@ function registerCheckCommand(program2) {
             return;
           }
           const hasErrors2 = fill.checkResult.issues.some((i) => i.severity === "error");
-          if (hasErrors2) await exitAfterFlush(1);
+          await exitAfterFlush(hasErrors2 ? 1 : 0);
           return;
         } catch (err) {
           if (err instanceof FillGatingError) {
@@ -15893,7 +15892,7 @@ function registerCheckCommand(program2) {
       const result = await runCheck(graph, gitFiles);
       process.stdout.write(formatOutput(result, view));
       const hasErrors = result.issues.some((i) => i.severity === "error");
-      if (hasErrors) await exitAfterFlush(1);
+      await exitAfterFlush(hasErrors ? 1 : 0);
     } catch (error) {
       debugWrite(`[check] error: ${error.message}`);
       abortOnUnexpectedError(error, "running check");
@@ -19534,10 +19533,14 @@ pairs on that node \u2014 a legitimate vacuous pass, no verdict, no entry.
   \`yg knowledge read log-management\`) and the append-only log integrity baseline.
   The source fingerprint is the log gate's drift basis, so it is recorded ONLY for
   \`log_required\` nodes \u2014 a non-log_required node gets a \`nodes\` entry only when it
-  owns a \`log.md\` (then holding just the \`log\` baseline, no \`source\`). When the
-  \`nodes\` section is empty (no log_required node, no \`log.md\`), \`yg-lock.logs.json\`
-  is not written at all \u2014 an empty committed husk is removed, and readLock treats
-  an absent file as empty state.
+  owns a \`log.md\` (then holding just the \`log\` baseline, no \`source\`).
+- Empty section \u21D2 no file. Each of the three split files is written ONLY when its
+  section is non-empty; when empty it is not written at all (an existing empty husk
+  is removed). So a repo with no LLM aspects has no \`yg-lock.nondeterministic.json\`,
+  one with no \`log_required\` node and no \`log.md\` has no \`yg-lock.logs.json\`, and
+  one with no deterministic aspects has no \`.yg-lock.deterministic.json\`. readLock
+  treats an absent file as empty state, so this is transparent to every reader \u2014 a
+  repo only carries the lock files it actually needs.
 - The built-in relation-conformance check is NOT stored in the lock \u2014 it is
   recomputed live on every \`yg check\`. The lock holds only aspect \`verdicts\`
   and per-node \`nodes\` facts; there is no relation section. See "Relation

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chrisdudek/yg",
-  "version": "5.2.2",
+  "version": "5.2.4",
   "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": {
@@ -85,7 +85,7 @@
     "@tree-sitter-grammars/tree-sitter-kotlin": "^1.1.0",
     "@tree-sitter-grammars/tree-sitter-toml": "^0.7.0",
     "@tree-sitter-grammars/tree-sitter-yaml": "^0.7.1",
-    "@types/node": "^25.9.1",
+    "@types/node": "^26.0.0",
     "@types/semver": "^7.7.1",
     "@vitest/coverage-v8": "^4.1.8",
     "eslint": "^10.4.1",