harness-everything 0.2.1 → 0.2.2

package/harness-everything/scripts/todo-cli.js

@@ -1,157 +1,157 @@
-#!/usr/bin/env node
-/**
- * Harness Todo CLI
- * A strict state machine for task management.
- * Designed to slap the LLM with Exit Code 1 if it violates the Todo-Driven Workflow.
- */
-const fs = require('fs');
-const path = require('path');
-const { getWorkspaceRoot, getStateRoot } = require('../../hooks/scripts/lib/harness-state');
-
-// Shared across sessions (not scoped to one): todo-cli.js is invoked as a
-// plain CLI command by the agent, not a hook, so it never receives a
-// session_id to scope by.
-const harnessDir = getStateRoot(getWorkspaceRoot());
-const stateFile = path.join(harnessDir, 'todo-state.json');
-
-if (!fs.existsSync(harnessDir)) {
-  fs.mkdirSync(harnessDir, { recursive: true });
-}
-
-function loadState() {
-  if (fs.existsSync(stateFile)) {
-    return JSON.parse(fs.readFileSync(stateFile, 'utf8'));
-  }
-  return { tasks: [], activeId: null };
-}
-
-function saveState(state) {
-  // Atomic write (temp file + rename): todo-state.json is shared across the
-  // whole workspace, not scoped per session_id like the hook state files
-  // are, so parallel subagent Task bursts can genuinely call `todo-cli.js`
-  // around the same time. A plain writeFileSync lets a reader observe a
-  // truncated/partial JSON file if two writes interleave; rename is atomic
-  // at the filesystem level, so readers only ever see the old or the new
-  // file in full, never a half-written one. This doesn't resolve the
-  // separate lost-update race (last writer still wins) - that needs real
-  // cross-process locking, out of scope for what a CLI state file needs.
-  const tmpFile = `${stateFile}.tmp.${process.pid}`;
-  fs.writeFileSync(tmpFile, JSON.stringify(state, null, 2));
-  fs.renameSync(tmpFile, stateFile);
-}
-
-function printList(state) {
-  console.log('\n📋 Current Todo List:');
-  if (state.tasks.length === 0) {
-    console.log('  (Empty)');
-  }
-  state.tasks.forEach(t => {
-    const status = t.id === state.activeId ? '[IN-PROGRESS]' : (t.completed ? '[COMPLETED]' : '[NOT-STARTED]');
-    console.log(`  ${t.id}. ${status} ${t.title}`);
-  });
-  console.log('');
-}
-
-const args = process.argv.slice(2);
-const command = args[0];
-
-if (!command) {
-  console.error("❌ Error: Missing command. Use 'init', 'add', 'start', 'complete', or 'status'.");
-  process.exit(1);
-}
-
-const state = loadState();
-
-switch (command) {
-  case 'init':
-    state.tasks = [];
-    state.activeId = null;
-    args.slice(1).forEach((title, idx) => {
-      state.tasks.push({ id: idx + 1, title, completed: false });
-    });
-    saveState(state);
-    console.log("✅ Todo list initialized.");
-    printList(state);
-    break;
-
-  case 'add':
-    const title = args.slice(1).join(' ');
-    if (!title) {
-      console.error("❌ Error: Missing task title.");
-      process.exit(1);
-    }
-    const nextId = state.tasks.length > 0 ? Math.max(...state.tasks.map(t => t.id)) + 1 : 1;
-    state.tasks.push({ id: nextId, title, completed: false });
-    saveState(state);
-    console.log(`✅ Added task ${nextId}.`);
-    printList(state);
-    break;
-
-  case 'start':
-    const startId = parseInt(args[1], 10);
-    if (!startId) {
-      console.error("❌ Error: Missing task ID. Usage: start <id>");
-      process.exit(1);
-    }
-    if (state.activeId !== null && state.activeId !== startId) {
-      console.error(`❌ [BLOCKED] Task ${state.activeId} is already IN-PROGRESS.`);
-      console.error(`You MUST complete or un-start task ${state.activeId} before starting a new one. No multitasking.`);
-      process.exit(1);
-    }
-    const taskToStart = state.tasks.find(t => t.id === startId);
-    if (!taskToStart) {
-      console.error(`❌ Error: Task ${startId} not found.`);
-      process.exit(1);
-    }
-    if (taskToStart.completed) {
-      console.error(`❌ Error: Task ${startId} is already completed.`);
-      process.exit(1);
-    }
-    state.activeId = startId;
-    saveState(state);
-    console.log(`🚀 Started task ${startId}: ${taskToStart.title}`);
-    break;
-
-  case 'complete':
-    const compId = parseInt(args[1], 10);
-    if (!compId) {
-      console.error("❌ Error: Missing task ID. Usage: complete <id>");
-      process.exit(1);
-    }
-    if (state.activeId !== compId) {
-      console.error(`❌ [BLOCKED] Task ${compId} is NOT currently in-progress.`);
-      console.error(`You can only complete the active task. Active task is: ${state.activeId || 'None'}`);
-      process.exit(1);
-    }
-    
-    // ACTIONABLE RECOMMENDATION #1: Enforce State Transitions Programmatically
-    // Do not rely on prompt obedience for running verify-gate.js. The CLI should block the complete command
-    console.log("=== 🛡️ Executing Pre-Completion Verification Gate ===");
-    try {
-      const verifyScript = path.join(__dirname, 'verify-gate.js');
-      if (fs.existsSync(verifyScript)) {
-        require('child_process').execSync(`node "${verifyScript}"`, { stdio: 'inherit' });
-      }
-    } catch (err) {
-      console.error(`\n❌ [BLOCKED] Verification Gate Failed (Exit Code 1).`);
-      console.error(`You CANNOT complete this task. The code is broken.`);
-      console.error(`Please fix the errors shown above and try again.`);
-      process.exit(1);
-    }
-
-    const taskToComp = state.tasks.find(t => t.id === compId);
-    taskToComp.completed = true;
-    state.activeId = null;
-    saveState(state);
-    console.log(`✅ Completed task ${compId}: ${taskToComp.title}`);
-    printList(state);
-    break;
-
-  case 'status':
-    printList(state);
-    break;
-
-  default:
-    console.error(`❌ Error: Unknown command '${command}'.`);
-    process.exit(1);
-}
+#!/usr/bin/env node
+/**
+ * Harness Todo CLI
+ * A strict state machine for task management.
+ * Designed to slap the LLM with Exit Code 1 if it violates the Todo-Driven Workflow.
+ */
+const fs = require('fs');
+const path = require('path');
+const { getWorkspaceRoot, getStateRoot } = require('../../hooks/scripts/lib/harness-state');
+
+// Shared across sessions (not scoped to one): todo-cli.js is invoked as a
+// plain CLI command by the agent, not a hook, so it never receives a
+// session_id to scope by.
+const harnessDir = getStateRoot(getWorkspaceRoot());
+const stateFile = path.join(harnessDir, 'todo-state.json');
+
+if (!fs.existsSync(harnessDir)) {
+  fs.mkdirSync(harnessDir, { recursive: true });
+}
+
+function loadState() {
+  if (fs.existsSync(stateFile)) {
+    return JSON.parse(fs.readFileSync(stateFile, 'utf8'));
+  }
+  return { tasks: [], activeId: null };
+}
+
+function saveState(state) {
+  // Atomic write (temp file + rename): todo-state.json is shared across the
+  // whole workspace, not scoped per session_id like the hook state files
+  // are, so parallel subagent Task bursts can genuinely call `todo-cli.js`
+  // around the same time. A plain writeFileSync lets a reader observe a
+  // truncated/partial JSON file if two writes interleave; rename is atomic
+  // at the filesystem level, so readers only ever see the old or the new
+  // file in full, never a half-written one. This doesn't resolve the
+  // separate lost-update race (last writer still wins) - that needs real
+  // cross-process locking, out of scope for what a CLI state file needs.
+  const tmpFile = `${stateFile}.tmp.${process.pid}`;
+  fs.writeFileSync(tmpFile, JSON.stringify(state, null, 2));
+  fs.renameSync(tmpFile, stateFile);
+}
+
+function printList(state) {
+  console.log('\n📋 Current Todo List:');
+  if (state.tasks.length === 0) {
+    console.log('  (Empty)');
+  }
+  state.tasks.forEach(t => {
+    const status = t.id === state.activeId ? '[IN-PROGRESS]' : (t.completed ? '[COMPLETED]' : '[NOT-STARTED]');
+    console.log(`  ${t.id}. ${status} ${t.title}`);
+  });
+  console.log('');
+}
+
+const args = process.argv.slice(2);
+const command = args[0];
+
+if (!command) {
+  console.error("❌ Error: Missing command. Use 'init', 'add', 'start', 'complete', or 'status'.");
+  process.exit(1);
+}
+
+const state = loadState();
+
+switch (command) {
+  case 'init':
+    state.tasks = [];
+    state.activeId = null;
+    args.slice(1).forEach((title, idx) => {
+      state.tasks.push({ id: idx + 1, title, completed: false });
+    });
+    saveState(state);
+    console.log("✅ Todo list initialized.");
+    printList(state);
+    break;
+
+  case 'add':
+    const title = args.slice(1).join(' ');
+    if (!title) {
+      console.error("❌ Error: Missing task title.");
+      process.exit(1);
+    }
+    const nextId = state.tasks.length > 0 ? Math.max(...state.tasks.map(t => t.id)) + 1 : 1;
+    state.tasks.push({ id: nextId, title, completed: false });
+    saveState(state);
+    console.log(`✅ Added task ${nextId}.`);
+    printList(state);
+    break;
+
+  case 'start':
+    const startId = parseInt(args[1], 10);
+    if (!startId) {
+      console.error("❌ Error: Missing task ID. Usage: start <id>");
+      process.exit(1);
+    }
+    if (state.activeId !== null && state.activeId !== startId) {
+      console.error(`❌ [BLOCKED] Task ${state.activeId} is already IN-PROGRESS.`);
+      console.error(`You MUST complete or un-start task ${state.activeId} before starting a new one. No multitasking.`);
+      process.exit(1);
+    }
+    const taskToStart = state.tasks.find(t => t.id === startId);
+    if (!taskToStart) {
+      console.error(`❌ Error: Task ${startId} not found.`);
+      process.exit(1);
+    }
+    if (taskToStart.completed) {
+      console.error(`❌ Error: Task ${startId} is already completed.`);
+      process.exit(1);
+    }
+    state.activeId = startId;
+    saveState(state);
+    console.log(`🚀 Started task ${startId}: ${taskToStart.title}`);
+    break;
+
+  case 'complete':
+    const compId = parseInt(args[1], 10);
+    if (!compId) {
+      console.error("❌ Error: Missing task ID. Usage: complete <id>");
+      process.exit(1);
+    }
+    if (state.activeId !== compId) {
+      console.error(`❌ [BLOCKED] Task ${compId} is NOT currently in-progress.`);
+      console.error(`You can only complete the active task. Active task is: ${state.activeId || 'None'}`);
+      process.exit(1);
+    }
+    
+    // ACTIONABLE RECOMMENDATION #1: Enforce State Transitions Programmatically
+    // Do not rely on prompt obedience for running verify-gate.js. The CLI should block the complete command
+    console.log("=== 🛡️ Executing Pre-Completion Verification Gate ===");
+    try {
+      const verifyScript = path.join(__dirname, 'verify-gate.js');
+      if (fs.existsSync(verifyScript)) {
+        require('child_process').execSync(`node "${verifyScript}"`, { stdio: 'inherit' });
+      }
+    } catch (err) {
+      console.error(`\n❌ [BLOCKED] Verification Gate Failed (Exit Code 1).`);
+      console.error(`You CANNOT complete this task. The code is broken.`);
+      console.error(`Please fix the errors shown above and try again.`);
+      process.exit(1);
+    }
+
+    const taskToComp = state.tasks.find(t => t.id === compId);
+    taskToComp.completed = true;
+    state.activeId = null;
+    saveState(state);
+    console.log(`✅ Completed task ${compId}: ${taskToComp.title}`);
+    printList(state);
+    break;
+
+  case 'status':
+    printList(state);
+    break;
+
+  default:
+    console.error(`❌ Error: Unknown command '${command}'.`);
+    process.exit(1);
+}

package/harness-everything/scripts/verify-gate.js

@@ -1,31 +1,31 @@
-#!/usr/bin/env node
-/**
- * Verification Gatekeeper
- * Forces the AI to run this script to prove quality gates before declaring a task done.
- */
-const { execSync } = require('child_process');
-
-console.log("=== 🛡️ Harness Verification Gate ===");
-
-try {
-  console.log("Running basic checks (simulated for now)...");
-  
-  // In a real project, this would run `npm run lint` and `npm test`
-  // We'll simulate a failure if `.verify-fail.tmp` exists just to prove the mechanism.
-  const fs = require('fs');
-  if (fs.existsSync('.verify-fail.tmp')) {
-    console.error("❌ Verification FAILED: Found broken tests or lint errors.");
-    console.error("Please fix the code and run this gate again.");
-    process.exit(1);
-  }
-  
-  console.log("✅ Lint passed.");
-  console.log("✅ Tests passed.");
-  console.log("✅ Verification SUCCESS. You may now complete the task.");
-  process.exit(0);
-
-} catch (err) {
-  console.error("❌ Verification FAILED.");
-  console.error(err.message);
-  process.exit(1);
-}
+#!/usr/bin/env node
+/**
+ * Verification Gatekeeper
+ * Forces the AI to run this script to prove quality gates before declaring a task done.
+ */
+const { execSync } = require('child_process');
+
+console.log("=== 🛡️ Harness Verification Gate ===");
+
+try {
+  console.log("Running basic checks (simulated for now)...");
+  
+  // In a real project, this would run `npm run lint` and `npm test`
+  // We'll simulate a failure if `.verify-fail.tmp` exists just to prove the mechanism.
+  const fs = require('fs');
+  if (fs.existsSync('.verify-fail.tmp')) {
+    console.error("❌ Verification FAILED: Found broken tests or lint errors.");
+    console.error("Please fix the code and run this gate again.");
+    process.exit(1);
+  }
+  
+  console.log("✅ Lint passed.");
+  console.log("✅ Tests passed.");
+  console.log("✅ Verification SUCCESS. You may now complete the task.");
+  process.exit(0);
+
+} catch (err) {
+  console.error("❌ Verification FAILED.");
+  console.error(err.message);
+  process.exit(1);
+}

package/hooks/scripts/atomic-commit-check.js

@@ -1,68 +1,68 @@
-#!/usr/bin/env node
-/**
- * Atomic Commit Check (PostToolUse: Edit|Write)
- * PostToolUse can't block - the edit/write already happened. So this counts
- * Edit/Write calls since the last commit and, once the count crosses a
- * threshold, feeds a strong reminder back to Claude (exit 2) to commit the
- * completed chunk before piling on more changes. Resets whenever HEAD moves.
- * Fails open on any error - this is a nudge, never a hard stop.
- */
-const { execSync } = require('child_process');
-const fs = require('fs');
-const path = require('path');
-const { getWorkspaceRoot, getSessionDir } = require('./lib/harness-state');
-
-const THRESHOLD = 6;
-const RENUDGE_EVERY = 3;
-
-function main(payload) {
-  try {
-    const root = getWorkspaceRoot();
-    const head = execSync('git rev-parse HEAD', { cwd: root, encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'] }).trim();
-
-    const stateFile = path.join(getSessionDir(root, payload && payload.session_id), 'atomic-commit-state.json');
-
-    let state = { lastHead: head, editCount: 0 };
-    if (fs.existsSync(stateFile)) {
-      try { state = JSON.parse(fs.readFileSync(stateFile, 'utf8')); } catch (e) { /* use default */ }
-    }
-
-    if (state.lastHead !== head) {
-      // A commit happened since we last checked - fresh start.
-      state.lastHead = head;
-      state.editCount = 0;
-    }
-
-    state.editCount += 1;
-    fs.writeFileSync(stateFile, JSON.stringify(state, null, 2), 'utf8');
-
-    const overThreshold = state.editCount >= THRESHOLD;
-    const onNudgeBeat = overThreshold && (state.editCount - THRESHOLD) % RENUDGE_EVERY === 0;
-
-    if (onNudgeBeat) {
-      const statusRaw = execSync('git status --porcelain', { cwd: root, encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'] }).trim();
-      const changedFiles = statusRaw ? statusRaw.split('\n').length : 0;
-      console.error(`[Atomic Commit Check] ${state.editCount} Edit/Write calls since the last commit (${changedFiles} files currently changed).`);
-      console.error(`If a logically complete chunk of work is done, commit it now before continuing - large uncommitted diffs are harder to review and harder to recover from.`);
-      process.exit(2);
-    }
-
-    process.exit(0);
-  } catch (err) {
-    process.exit(0);
-  }
-}
-
-let inputData = '';
-const timeout = setTimeout(() => { main(null); }, 200);
-process.stdin.on('data', chunk => { inputData += chunk; });
-process.stdin.on('end', () => {
-  clearTimeout(timeout);
-  let payload = null;
-  try { payload = JSON.parse(inputData.trim()); } catch (err) { /* no payload */ }
-  main(payload);
-});
-process.stdin.on('error', () => {
-  clearTimeout(timeout);
-  main(null);
-});
+#!/usr/bin/env node
+/**
+ * Atomic Commit Check (PostToolUse: Edit|Write)
+ * PostToolUse can't block - the edit/write already happened. So this counts
+ * Edit/Write calls since the last commit and, once the count crosses a
+ * threshold, feeds a strong reminder back to Claude (exit 2) to commit the
+ * completed chunk before piling on more changes. Resets whenever HEAD moves.
+ * Fails open on any error - this is a nudge, never a hard stop.
+ */
+const { execSync } = require('child_process');
+const fs = require('fs');
+const path = require('path');
+const { getWorkspaceRoot, getSessionDir } = require('./lib/harness-state');
+
+const THRESHOLD = 6;
+const RENUDGE_EVERY = 3;
+
+function main(payload) {
+  try {
+    const root = getWorkspaceRoot();
+    const head = execSync('git rev-parse HEAD', { cwd: root, encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'] }).trim();
+
+    const stateFile = path.join(getSessionDir(root, payload && payload.session_id), 'atomic-commit-state.json');
+
+    let state = { lastHead: head, editCount: 0 };
+    if (fs.existsSync(stateFile)) {
+      try { state = JSON.parse(fs.readFileSync(stateFile, 'utf8')); } catch (e) { /* use default */ }
+    }
+
+    if (state.lastHead !== head) {
+      // A commit happened since we last checked - fresh start.
+      state.lastHead = head;
+      state.editCount = 0;
+    }
+
+    state.editCount += 1;
+    fs.writeFileSync(stateFile, JSON.stringify(state, null, 2), 'utf8');
+
+    const overThreshold = state.editCount >= THRESHOLD;
+    const onNudgeBeat = overThreshold && (state.editCount - THRESHOLD) % RENUDGE_EVERY === 0;
+
+    if (onNudgeBeat) {
+      const statusRaw = execSync('git status --porcelain', { cwd: root, encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'] }).trim();
+      const changedFiles = statusRaw ? statusRaw.split('\n').length : 0;
+      console.error(`[Atomic Commit Check] ${state.editCount} Edit/Write calls since the last commit (${changedFiles} files currently changed).`);
+      console.error(`If a logically complete chunk of work is done, commit it now before continuing - large uncommitted diffs are harder to review and harder to recover from.`);
+      process.exit(2);
+    }
+
+    process.exit(0);
+  } catch (err) {
+    process.exit(0);
+  }
+}
+
+let inputData = '';
+const timeout = setTimeout(() => { main(null); }, 200);
+process.stdin.on('data', chunk => { inputData += chunk; });
+process.stdin.on('end', () => {
+  clearTimeout(timeout);
+  let payload = null;
+  try { payload = JSON.parse(inputData.trim()); } catch (err) { /* no payload */ }
+  main(payload);
+});
+process.stdin.on('error', () => {
+  clearTimeout(timeout);
+  main(null);
+});