@esneiderbravo/speclaw 2.0.18 → 2.0.19

package/dist/cli/index.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 import { parseFlags, REPEATABLE_FLAGS } from "./lib/args.js";
-import { ui, header } from "./lib/ui.js";
+import { ui, header, detectTheme } from "./lib/ui.js";
 import { maybeNotifyUpdate } from "./lib/update-check.js";
 import { GLOBAL_HELP as HELP, helpFor, knownCommands, wantsHelp } from "./lib/help.js";
 // Commands that open with the one-line branded header. These are the
@@ -38,13 +38,15 @@ const HEADER_COMMANDS = new Set([
  * child process. `budget --json`, `doctor --json`, and `coverage` when emitting
  * TAP/JSON (or when stdout is not a TTY) are machine-consumed and suppress the
  * header. `session-start` and `reindex-file` are not header-eligible: they must print
- * nothing.
+ * nothing. The palette is matched to the terminal background first, so every
+ * styled line after it — not only `init`'s — stays legible on a light theme.
  */
-function maybeHeader(cmd, flags) {
+async function maybeHeader(cmd, flags) {
     if (!process.stdout.isTTY && process.env.FORCE_COLOR !== "1")
         return;
     if (!HEADER_COMMANDS.has(cmd))
         return;
+    await detectTheme();
     // A self-update child re-runs `update`; the parent already printed the header.
     if (cmd === "update" && process.env.SPECLAW_SELF_UPDATED)
         return;
@@ -172,7 +174,7 @@ async function main() {
         return;
     }
     const flags = parseFlags(rest, REPEATABLE_FLAGS);
-    maybeHeader(cmd, flags);
+    await maybeHeader(cmd, flags);
     await dispatch(cmd, flags);
     // The SessionStart and edit hooks must stay silent: no update notice either.
     if (cmd === "session-start" || cmd === "reindex-file" || cmd === "ship-on-stop")

package/dist/cli/lib/ui.js

@@ -118,6 +118,9 @@ export async function detectTheme() {
         theme = themeFromOsc11(await queryBackground()) ?? (await macAppearance()) ?? "dark";
     }
     setTheme(theme);
+    // Child speclaw processes (a self-update re-run) inherit the answer instead
+    // of querying the terminal again.
+    process.env.SPECLAW_THEME ??= theme;
     return theme;
 }
 const colorOn = (Boolean(process.stdout.isTTY) || process.env.FORCE_COLOR === "1") && !process.env.NO_COLOR;

package/dist/modules/lawbook/ship.js

@@ -79,10 +79,28 @@ function branchFiles(projectPath) {
         return [];
     const base = mergeBase(projectPath, "main") ?? mergeBase(projectPath, "master");
     const committed = base ? changedFiles(projectPath, base) : [];
-    return [...new Set([...committed, ...worktreeChangedFiles(projectPath)])]
+    return [
+        ...new Set([
+            ...committed,
+            ...worktreeChangedFiles(projectPath),
+            ...untrackedFiles(projectPath),
+        ]),
+    ]
         .filter((f) => !f.startsWith("lawbook/changes/"))
         .sort();
 }
+/** New files not yet added, outside ship's own output (`lawbook/changes/`, `.speclaw/`). */
+function untrackedFiles(projectPath) {
+    return git(projectPath, [
+        "-c",
+        "core.quotePath=false",
+        "ls-files",
+        "--others",
+        "--exclude-standard",
+    ])
+        .split("\n")
+        .filter((f) => f && !f.startsWith("lawbook/changes/") && !f.startsWith(".speclaw/"));
+}
 function secs(ms) {
     return `${(ms / 1000).toFixed(1)} s`;
 }
@@ -311,30 +329,41 @@ function readMarker(projectPath) {
     return { branch: "", change: "", fingerprint: raw };
 }
 /**
- * Fingerprint of the branch's work outside `lawbook/changes/`: the working tree
- * diffed against the merge base with `main`/`master`, plus untracked files
- * (names and contents). Committing that work leaves it unchanged — a commit is not new work —
- * and ship writes under `lawbook/changes/`, so its own output never changes it.
- * Without a merge base it falls back to the diff against HEAD plus HEAD itself.
+ * Fingerprint of the branch's work outside `lawbook/changes/` and `.speclaw/`: every path that
+ * differs from the merge base with `main`/`master` or is untracked, with its
+ * working-tree contents. A file hashes the same whether it is untracked, staged
+ * or committed, so committing work — new files included — is not new work; ship
+ * writes under `lawbook/changes/`, so its own output never changes it either.
+ * Without a merge base it falls back to the paths changed against HEAD plus HEAD itself.
  */
 function workFingerprint(projectPath) {
     const base = mergeBase(projectPath, "main") ?? mergeBase(projectPath, "master");
-    const diff = git(projectPath, ["diff", base ?? "HEAD", "--", ".", ":(exclude)lawbook/changes"]);
+    const tracked = git(projectPath, [
+        "-c",
+        "core.quotePath=false",
+        "diff",
+        "--name-only",
+        base ?? "HEAD",
+        "--",
+        ".",
+        ":(exclude)lawbook/changes",
+        ":(exclude).speclaw",
+    ]).split("\n");
     const hash = createHash("sha256");
-    for (const f of git(projectPath, ["ls-files", "--others", "--exclude-standard"]).split("\n")) {
-        if (!f || f.startsWith("lawbook/changes/") || f.startsWith(".speclaw/"))
+    for (const f of [...new Set([...tracked, ...untrackedFiles(projectPath)])].sort()) {
+        if (!f)
             continue;
-        // Contents, not just names: editing a new, not-yet-added file is new work.
         hash.update(`${f}\n`);
         try {
             hash.update(fs.readFileSync(path.join(projectPath, f)));
         }
         catch {
-            // Vanished or unreadable since listing: its name stands in.
+            // Deleted, vanished or unreadable: its name stands in.
+            hash.update("\0");
         }
     }
     const anchor = base ?? git(projectPath, ["rev-parse", "HEAD"]).trim();
-    return hash.update(`\n${anchor}\n${diff}`).digest("hex");
+    return hash.update(`\n${anchor}\n`).digest("hex");
 }
 /**
  * The `Stop` hook body: when the agent ends a turn on a feature branch with

package/package.json

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