tdd-enforcer 0.1.8 → 0.2.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, snapshot } from "../../engine/index.js";
+import { loadPhaseState, loadConfig, savePhaseState, initGit, resetGit, snapshot } from "../../engine/index.js";
 import { registerTools } from "./tools.js";
 import { registerHooks } from "./hooks.js";
 import { loadTddState } from "./helpers.js";
@@ -163,6 +163,49 @@ export default function (pi: ExtensionAPI) {
     },
   });
 
+  pi.registerCommand("tdd:reset", {
+    description:
+      "WARNING: Destroys ALL TDD snapshot history and resets to RED phase. " +
+      "Working tree is preserved. Run /tdd:on to re-enable after reset.",
+    handler: async (_args: string, ctx: ExtensionContext) => {
+      const root = ctx.cwd;
+      const tddDir = join(root, ".pi", "tdd");
+
+      tddLog(tddDir, "INFO", "tdd:reset: starting");
+
+      if (!existsSync(tddDir)) {
+        tddLog(tddDir, "WARN", "tdd:reset: missing .pi/tdd/ directory");
+        ctx.ui.notify("No .pi/tdd/ directory found — nothing to reset.", "error");
+        return;
+      }
+
+      // Nuke git history and re-init
+      try {
+        resetGit(root);
+        tddLog(tddDir, "INFO", "tdd:reset: git reset and re-initialised");
+      } catch (e) {
+        tddLog(tddDir, "ERROR", "tdd:reset: git reset failed", {
+          error: (e as Error).message,
+        });
+        ctx.ui.notify("Failed to reset private git repo.", "error");
+        return;
+      }
+
+      // Snapshot current working tree
+      snapshot(root, "red");
+      tddLog(tddDir, "INFO", "tdd:reset: snapshot taken");
+
+      // Reset state to RED (disabled, user must run /tdd:on)
+      savePhaseState(root, { enabled: false, current: "red" });
+      tddLog(tddDir, "INFO", "tdd:reset: complete");
+
+      ctx.ui.notify(
+        "TDD snapshot history reset. Run /tdd:on to re-enable enforcement.",
+        "warning",
+      );
+    },
+  });
+
   registerTools(pi);
   registerHooks(pi);
 }

package/engine/git.ts

@@ -1,5 +1,5 @@
 import { execSync, type ExecSyncOptions } from "node:child_process";
-import { existsSync, mkdirSync, writeFileSync, unlinkSync } from "node:fs";
+import { existsSync, mkdirSync, writeFileSync, unlinkSync, rmSync } from "node:fs";
 import { join } from "node:path";
 
 const TDD_DIR = ".pi/tdd";
@@ -40,6 +40,16 @@ export function initGit(projectRoot: string): void {
   gitExec('commit --allow-empty -m "tdd: init"', projectRoot, { stdio: "pipe" as const });
 }
 
+/** Destroy the private git repo and re-init from scratch. */
+export function resetGit(projectRoot: string): void {
+  const tddPath = join(projectRoot, TDD_DIR);
+  const gitDir = join(tddPath, ".git");
+  if (existsSync(gitDir)) {
+    rmSync(gitDir, { recursive: true, force: true });
+  }
+  initGit(projectRoot);
+}
+
 /** Stage all + commit with --allow-empty so every phase transition has a labeled commit. */
 export function snapshot(projectRoot: string, phase: string): string {
   gitExec("add -A", projectRoot, { stdio: "pipe" as const });

package/engine/index.ts

@@ -1,5 +1,5 @@
 export { isAllowed, disallowedFiles } from "./enforce.js";
-export { initGit, snapshot, changesSinceSnapshot, changesSince, modifiedFiles, untrackedFiles, restoreFilesTo, gitStashCreate, headHash, headMessage, hasParent, resetHard, undoLastCommit } from "./git.js";
+export { initGit, resetGit, snapshot, changesSinceSnapshot, changesSince, modifiedFiles, untrackedFiles, restoreFilesTo, gitStashCreate, headHash, headMessage, hasParent, resetHard, undoLastCommit } from "./git.js";
 export { loadConfig } from "./config.js";
 export { loadPhaseState, savePhaseState } from "./state.js";
 export { nextPhase, checkGate, getDisallowedChanges } from "./transition.js";

package/engine/transition.ts

@@ -36,7 +36,7 @@ export async function checkGate(
       if (result.passed) {
         return {
           passed: false,
-          message: "Tests pass. Add a failing test before transitioning to GREEN.",
+          message: "Tests passed. Add a failing test before transitioning to GREEN.",
         };
       }
       return { passed: true, message: "Tests fail — proceed to GREEN." };
@@ -45,7 +45,7 @@ export async function checkGate(
       if (!result.passed) {
         return {
           passed: false,
-          message: "Tests fail. Fix them before transitioning to REFACTOR.",
+          message: "Tests failed. Fix them before transitioning to REFACTOR.",
         };
       }
       return { passed: true, message: "All tests pass — proceeding." };
@@ -54,7 +54,7 @@ export async function checkGate(
       if (!result.passed) {
         return {
           passed: false,
-          message: "Tests fail. Fix them before transitioning to RED.",
+          message: "Tests failed. Fix them before transitioning to RED.",
         };
       }
       return { passed: true, message: "All tests pass — proceeding." };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tdd-enforcer",
-  "version": "0.1.8",
+  "version": "0.2.0",
   "type": "module",
   "keywords": [
     "pi-package"
@@ -14,6 +14,9 @@
   "pi": {
     "extensions": [
       "./adapters/pi/index.ts"
+    ],
+    "skills": [
+      "./skills"
     ]
   }
 }

package/skills/tdd-init/SKILL.md

@@ -0,0 +1,90 @@
+---
+name: tdd-init
+description: Use when the TDD enforcer extension reports missing configuration — .pi/tdd/ directory, rules.json, or state.json not found. Also use when starting TDD on a new project.
+---
+
+# TDD Init
+
+## Overview
+
+Set up the TDD enforcer. The extension enforces the Red-Green-Refactor cycle by restricting which files the agent can modify per phase.
+
+## What to Create
+
+```
+project-root/
+  .pi/
+    tdd/
+      rules.json        # File patterns and test commands
+      state.json        # Phase state (create, then /tdd:on sets enabled)
+```
+
+## rules.json
+
+```json
+{
+  "allowedRedPhaseFiles": ["tests/**/*.test.ts"],
+  "allowedGreenPhaseFiles": ["src/**/*.ts"],
+  "testCommands": ["npm run test"],
+  "timeoutSeconds": 120
+}
+```
+
+| Field | What | Why |
+|-------|------|-----|
+| `allowedRedPhaseFiles` | Glob patterns for files writable in RED phase | Typically test files |
+| `allowedGreenPhaseFiles` | Glob patterns for files writable in GREEN phase | Typically implementation files |
+| `testCommands` | Non-interactive commands (string or array) | Run on phase transitions to check gate |
+| `timeoutSeconds` | Per-command timeout (default 120) | Prevents hung suites |
+
+- Globs are relative to project root
+- Files matching neither set are free in all phases
+- All 3 array fields **must** be non-empty
+
+## state.json
+
+```json
+{
+  "enabled": false,
+  "current": "red"
+}
+```
+
+Start with `enabled: false`. Run `/tdd:on` — it validates config, initialises the private git repo, snapshots the working tree, and sets `enabled: true`.
+
+## Setup Steps
+
+1. Create `.pi/tdd/` directory
+2. Create `.pi/tdd/rules.json` with file patterns and test commands
+3. Create `.pi/tdd/state.json` with `enabled: false, current: "red"`
+4. Run `/tdd:on` to enable enforcement
+
+## TDD Cycle
+
+| Phase | Allowed | Gate to advance |
+|-------|---------|-----------------|
+| RED | Test files only | Tests must fail |
+| GREEN | Implementation files only | Tests must pass |
+| REFACTOR | All files | Tests must pass |
+
+Use `next_tdd_phase` to advance, `previous_tdd_phase` to revert.
+
+## Common Patterns
+
+**Standard split:**
+```json
+"allowedRedPhaseFiles": ["tests/**/*.test.ts"],
+"allowedGreenPhaseFiles": ["src/**/*.ts"]
+```
+
+**Monorepo:**
+```json
+"testCommands": ["npm run test:unit", "npm run test:e2e"]
+```
+
+## Recovery
+
+```bash
+/tdd:reset    # Destroys snapshot history, resets to RED (disabled)
+/tdd:on       # Re-enable with fresh snapshot
+```