tdd-enforcer 0.1.3 → 0.1.5

package/adapters/pi/hooks.ts

@@ -105,12 +105,13 @@ export function registerHooks(pi: ExtensionAPI): void {
 
     const existingText = event.content.map((c) => ("text" in c ? c.text : "")).join("");
     return {
+      isError: true,
       content: [
         {
           type: "text",
           text:
             existingText +
-            `\n\n⚠️ ${phase.toUpperCase()}: bash modified locked files: ${violations.join(", ")}\n` +
+            `\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/log.test.ts

@@ -0,0 +1,73 @@
+import { describe, it, expect } from "vitest";
+import { mkdirSync, writeFileSync, rmSync, readFileSync, existsSync } from "node:fs";
+import { join } from "node:path";
+import { tmpdir } from "node:os";
+import { tddLog } from "./log.js";
+
+function setup() {
+  const dir = join(tmpdir(), `tdd-log-test-${Date.now()}`);
+  mkdirSync(dir, { recursive: true });
+  return dir;
+}
+
+function teardown(dir: string) {
+  rmSync(dir, { recursive: true, force: true });
+}
+
+describe("tddLog", () => {
+  it("creates the log file if it doesn't exist", () => {
+    const dir = setup();
+    tddLog(dir, "INFO", "hello");
+    expect(existsSync(join(dir, "tdd.log"))).toBe(true);
+    teardown(dir);
+  });
+
+  it("appends multiple lines", () => {
+    const dir = setup();
+    tddLog(dir, "INFO", "line one");
+    tddLog(dir, "DEBUG", "line two");
+    const content = readFileSync(join(dir, "tdd.log"), "utf-8");
+    const lines = content.trim().split("\n");
+    expect(lines).toHaveLength(2);
+    expect(lines[0]).toContain("[INFO]");
+    expect(lines[0]).toContain("line one");
+    expect(lines[1]).toContain("[DEBUG]");
+    expect(lines[1]).toContain("line two");
+    teardown(dir);
+  });
+
+  it("includes data as JSON when provided", () => {
+    const dir = setup();
+    tddLog(dir, "INFO", "with data", { key: "val", num: 42 });
+    const content = readFileSync(join(dir, "tdd.log"), "utf-8");
+    expect(content).toContain('{"key":"val","num":42}');
+    teardown(dir);
+  });
+
+  it("trims to last 1000 lines when exceeded", () => {
+    const dir = setup();
+    for (let i = 0; i < 1005; i++) {
+      tddLog(dir, "DEBUG", `line ${i}`);
+    }
+    const content = readFileSync(join(dir, "tdd.log"), "utf-8");
+    const lines = content.trim().split("\n");
+    expect(lines).toHaveLength(1000);
+    expect(lines[0]).toContain("line 5");
+    expect(lines[999]).toContain("line 1004");
+    teardown(dir);
+  });
+
+  it("does not throw on invalid tddDir", () => {
+    expect(() => tddLog("/nonexistent/path/tdd", "INFO", "fail")).not.toThrow();
+  });
+
+  it("handles missing data field gracefully", () => {
+    const dir = setup();
+    tddLog(dir, "WARN", "no data");
+    const content = readFileSync(join(dir, "tdd.log"), "utf-8");
+    expect(content).toContain("[WARN]");
+    expect(content).toContain("no data");
+    expect(content).not.toContain("{}");
+    teardown(dir);
+  });
+});

package/adapters/pi/log.ts

@@ -19,8 +19,8 @@ export function tddLog(
 
     // Trim to last MAX_LINES
     const content = readFileSync(logPath, "utf-8");
-    const lines = content.split("\n");
-    if (lines.length > MAX_LINES + 1) {
+    const lines = content.trimEnd().split("\n");
+    if (lines.length > MAX_LINES) {
       const trimmed = lines.slice(-MAX_LINES).join("\n") + "\n";
       writeFileSync(logPath, trimmed, "utf-8");
     }

package/adapters/pi/tools.ts

@@ -1,3 +1,4 @@
+import { join } from "node:path";
 import { Type } from "typebox";
 import { execSync } from "node:child_process";
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
@@ -15,7 +16,7 @@ import {
 import type { TestRunner, Phase } from "../../engine/index.js";
 import { getNudgePrompt } from "./prompts.js";
 import { loadTddState } from "./helpers.js";
-
+import { tddLog } from "./log.js";
 
 export function registerTools(pi: ExtensionAPI): void {
   pi.registerTool({

package/package.json

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