tdd-enforcer 0.1.2 → 0.1.4

package/adapters/pi/hooks.ts

@@ -2,7 +2,7 @@ import { join, relative } from "node:path";
 import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { isToolCallEventType, isBashToolResult } from "@earendil-works/pi-coding-agent";
 import { isAllowed } from "../../engine/enforce.js";
-import { changesSinceSnapshot, restoreFiles } from "../../engine/git.js";
+import { changesSinceSnapshot } from "../../engine/git.js";
 import { loadTddState } from "./helpers.js";
 import { tddLog } from "./log.js";
 
@@ -98,23 +98,22 @@ export function registerHooks(pi: ExtensionAPI): void {
       return;
     }
 
-    tddLog(tddDir, "INFO", "tool_result: reverting violations", {
+    tddLog(tddDir, "WARN", "tool_result: locked files modified by bash", {
       phase,
       violations,
-      allChanged: changed,
     });
 
-    restoreFiles(root, violations);
-
     const existingText = event.content.map((c) => ("text" in c ? c.text : "")).join("");
     return {
+      isError: true,
       content: [
         {
           type: "text",
           text:
             existingText +
-            `\n\n⚠️ TDD: Bash modified files locked in ${phase.toUpperCase()} phase. ` +
-            `Reverted: ${violations.join(", ")}`,
+            `\n\n⛔ ${phase.toUpperCase()}: bash modified locked files: ${violations.join(", ")}\n` +
+            `next_tdd_phase blocked until reverted. ` +
+            `Inspect with: cd .pi/tdd && git diff HEAD -- ${violations[0]}`,
         },
       ],
     };

package/adapters/pi/index.ts

@@ -1,7 +1,7 @@
 import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { existsSync } from "node:fs";
 import { join } from "node:path";
-import { loadPhaseState, loadConfig, savePhaseState, initGit } from "../../engine/index.js";
+import { loadPhaseState, loadConfig, savePhaseState, initGit, snapshot } from "../../engine/index.js";
 import { registerTools } from "./tools.js";
 import { registerHooks } from "./hooks.js";
 import { loadTddState } from "./helpers.js";
@@ -80,6 +80,12 @@ export default function (pi: ExtensionAPI) {
         tddLog(tddDir, "DEBUG", "tdd:on: git repo already exists");
       }
 
+      // Snapshot working tree so stale baseline doesn't nuke user changes
+      snapshot(root, state.current);
+      tddLog(tddDir, "INFO", "tdd:on: snapshot taken", {
+        phase: state.current,
+      });
+
       state.enabled = true;
       savePhaseState(root, state);
       tddLog(tddDir, "INFO", "tdd:on: enabled", {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tdd-enforcer",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "type": "module",
   "keywords": [
     "pi-package"