@esneiderbravo/speclaw 2.0.21 → 2.0.22

package/dist/modules/foundation/lock.js

@@ -8,6 +8,7 @@ import crypto from "node:crypto";
 import fs from "node:fs";
 import path from "node:path";
 import { pkgName, pkgVersion } from "../../shared/version.js";
+import { COMPASS_MAP_END, COMPASS_MAP_START, stripCompassMapBlock, } from "../../shared/compass-map.js";
 export const LOCKFILE_NAME = "speclaw.lock";
 export const LOCKFILE_VERSION = 1;
 /** Delimited provenance block excluded from digests (self-reference). */
@@ -270,23 +271,9 @@ function walkFiles(dir, projectPath, out, pred) {
         }
     }
 }
-/** Markers for the regenerable map block inside `docs/compass.md` (see compass/map.ts). */
-export const COMPASS_MAP_START = "<!-- speclaw:map:start -->";
-export const COMPASS_MAP_END = "<!-- speclaw:map:end -->";
-/**
- * Strip the regenerable map body between markers so integrity digests stay stable
- * across `speclaw index` (which rewrites the map in CI before verify).
- *
- * @param text - Full docs/compass.md contents.
- * @returns The same text with an empty map body, or `text` if markers are missing.
- */
-export function stripCompassMapBlock(text) {
-    const start = text.indexOf(COMPASS_MAP_START);
-    const end = text.indexOf(COMPASS_MAP_END);
-    if (start < 0 || end < 0 || end < start)
-        return text;
-    return text.slice(0, start + COMPASS_MAP_START.length) + "\n" + text.slice(end);
-}
+// The map block helpers moved to shared/ so the lawbook can ignore the
+// regenerated map too; re-exported here for existing importers.
+export { COMPASS_MAP_END, COMPASS_MAP_START, stripCompassMapBlock };
 /**
  * Path-specific bytes that feed {@link digestText}: speclaw-owned coderabbit
  * region, regenerable Compass map body stripped, otherwise the file as-is.

package/dist/modules/lawbook/ship.js

@@ -4,6 +4,7 @@ import { spawn, spawnSync } from "node:child_process";
 import { fileURLToPath } from "node:url";
 import { createHash } from "node:crypto";
 import { changedFiles, isGitRepo, mergeBase, worktreeChangedFiles } from "../../shared/git.js";
+import { COMPASS_DOC, stripCompassMapBlock } from "../../shared/compass-map.js";
 import { handleHarness, readHarness } from "../cortex/harness.js";
 import { deltaSpecFiles, specArchive, specArchivePreconditions } from "./engine.js";
 import { artifactNeeds, confirmedLevel, gatherSignals, loadCeremonyConfig, promoteCeremonyLevel, proposeLevel, readCeremonyRecord, readChangeType, } from "./levels.js";
@@ -89,8 +90,28 @@ function branchFiles(projectPath) {
         ]),
     ]
         .filter((f) => !f.startsWith("lawbook/changes/"))
+        .filter((f) => f !== COMPASS_DOC || !onlyMapChanged(projectPath, base))
         .sort();
 }
+/**
+ * Whether `docs/compass.md` differs from `base` only inside its generated map
+ * block: every full index rewrites that block, so on a fresh branch it would
+ * otherwise read as the agent's work and ship an empty change.
+ */
+function onlyMapChanged(projectPath, base) {
+    if (!base)
+        return false;
+    try {
+        const now = fs.readFileSync(path.join(projectPath, COMPASS_DOC), "utf8");
+        const then = git(projectPath, ["show", `${base}:${COMPASS_DOC}`]);
+        // Line endings are the platform's (core.autocrlf), not the agent's work.
+        const lf = (t) => stripCompassMapBlock(t.replace(/\r\n/g, "\n"));
+        return then !== "" && lf(now) === lf(then);
+    }
+    catch {
+        return false;
+    }
+}
 /** New files not yet added, outside ship's own output (`lawbook/changes/`, `.speclaw/`). */
 function untrackedFiles(projectPath) {
     return git(projectPath, [
@@ -748,12 +769,18 @@ function workFingerprint(projectPath) {
         ":(exclude).speclaw",
     ]).split("\n");
     const hash = createHash("sha256");
-    for (const f of [...new Set([...tracked, ...untrackedFiles(projectPath)])].sort()) {
+    const files = [...new Set([...tracked, ...untrackedFiles(projectPath)])]
+        // A map-only docs/compass.md is no work: neither its bytes nor its name count.
+        .filter((f) => f !== COMPASS_DOC || !onlyMapChanged(projectPath, base))
+        .sort();
+    for (const f of files) {
         if (!f)
             continue;
         hash.update(`${f}\n`);
         try {
-            hash.update(fs.readFileSync(path.join(projectPath, f)));
+            const bytes = fs.readFileSync(path.join(projectPath, f));
+            // A re-index rewrites the map block; only the rest of the file is work.
+            hash.update(f === COMPASS_DOC ? stripCompassMapBlock(bytes.toString("utf8")) : bytes);
         }
         catch {
             // Deleted, vanished or unreadable: its name stands in.

package/dist/shared/compass-map.js

@@ -0,0 +1,20 @@
+/** Markers for the regenerable map block inside `docs/compass.md` (see compass/map.ts). */
+export const COMPASS_MAP_START = "<!-- speclaw:map:start -->";
+export const COMPASS_MAP_END = "<!-- speclaw:map:end -->";
+/** The project-relative path of the Compass guide that carries the generated map. */
+export const COMPASS_DOC = "docs/compass.md";
+/**
+ * Strip the regenerable map body between markers. `speclaw index` rewrites it on
+ * every full run, so integrity digests and "did the agent change this file?"
+ * checks must both ignore it.
+ *
+ * @param text - Full docs/compass.md contents.
+ * @returns The same text with an empty map body, or `text` if markers are missing.
+ */
+export function stripCompassMapBlock(text) {
+    const start = text.indexOf(COMPASS_MAP_START);
+    const end = text.indexOf(COMPASS_MAP_END);
+    if (start < 0 || end < 0 || end < start)
+        return text;
+    return text.slice(0, start + COMPASS_MAP_START.length) + "\n" + text.slice(end);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@esneiderbravo/speclaw",
-  "version": "2.0.21",
+  "version": "2.0.22",
   "publishConfig": {
     "access": "public"
   },