harness-everything 0.2.1 → 0.3.6

package/hooks/scripts/lib/platforms/hermes.js

@@ -1,32 +1,57 @@
-const path = require('path');
-const fs = require('fs');
-
-module.exports = {
-  name: 'hermes',
-  label: 'Hermes Agent',
-  getStateDir(workspaceRoot) {
-    return path.join(workspaceRoot, '.hermes', 'harness-state');
-  },
-  getIgnorePatterns(workspaceRoot) {
-    const patterns = [];
-    if (fs.existsSync(path.join(workspaceRoot, '.hermes', 'harness-state'))) {
-      patterns.push('.hermes/harness-state/');
-    }
-    return patterns;
-  },
-  isMatch(pattern, trimmedLine) {
-    if (pattern === '.hermes/harness-state/') {
-      return trimmedLine === '.hermes/' || 
-             trimmedLine === '.hermes' || 
-             trimmedLine === '.hermes/harness-state' ||
-             trimmedLine === '.hermes/harness-state/';
-    }
-    return trimmedLine === pattern || trimmedLine === pattern.slice(0, -1);
-  },
-  isInstalled(workspaceRoot, userHome, isGlobal) {
-    if (isGlobal) {
-      return false;
-    }
-    return fs.existsSync(path.join(workspaceRoot, '.hermes.md'));
-  }
-};
+const path = require('path');
+const fs = require('fs');
+
+module.exports = {
+  name: 'hermes',
+  label: 'Hermes Agent',
+  getStateDir(workspaceRoot) {
+    return path.join(workspaceRoot, '.hermes', 'harness-state');
+  },
+  getIgnorePatterns(workspaceRoot) {
+    const patterns = [];
+    if (fs.existsSync(path.join(workspaceRoot, '.hermes', 'harness-state'))) {
+      patterns.push('.hermes/harness-state/');
+    }
+    return patterns;
+  },
+  isMatch(pattern, trimmedLine) {
+    if (pattern === '.hermes/harness-state/') {
+      return trimmedLine === '.hermes/' || 
+             trimmedLine === '.hermes' || 
+             trimmedLine === '.hermes/harness-state' ||
+             trimmedLine === '.hermes/harness-state/';
+    }
+    return trimmedLine === pattern || trimmedLine === pattern.slice(0, -1);
+  },
+  isInstalled(workspaceRoot, userHome, isGlobal) {
+    if (isGlobal) {
+      return false;
+    }
+    return fs.existsSync(path.join(workspaceRoot, '.hermes.md'));
+  },
+  getSkillsTarget({ workspaceRoot, userHome, isGlobal, manifest }) {
+    if (isGlobal) {
+      const globalAgentsDir = path.join(userHome, '.agents');
+      return {
+        path: path.join(globalAgentsDir, 'skills'),
+        label: '~/.agents/skills/',
+        manifestPath: manifest.getManifestPath(globalAgentsDir),
+      };
+    }
+    return null;
+  },
+  install({ isGlobal, targetWorkspaceRoot, advisory }) {
+    if (!isGlobal) {
+      const targetFile = path.join(targetWorkspaceRoot, '.hermes.md');
+      advisory.injectAdvisoryText(targetFile, '# .hermes.md', '.hermes.md');
+    } else {
+      console.log(`  ℹ️  Hermes Agent has no documented global instructions file (it reads .hermes.md from the current project directory only) - skipping global install for --hermes.`);
+    }
+  },
+  uninstall({ removeLocal, removeGlobal, workspaceRoot, userHome }) {
+    const advisory = require('../../../../scripts/lib/advisory-text');
+    if (removeLocal) {
+      advisory.removeAdvisoryText(path.join(workspaceRoot, '.hermes.md'));
+    }
+  }
+};

package/hooks/scripts/lib/platforms/index.js

@@ -1,17 +1,17 @@
-const claude = require('./claude');
-const cursor = require('./cursor');
-const copilot = require('./copilot');
-const codex = require('./codex');
-const _continue = require('./continue');
-const hermes = require('./hermes');
-const worktrees = require('./worktrees');
-
-module.exports = [
-  claude,
-  cursor,
-  copilot,
-  codex,
-  _continue,
-  hermes,
-  worktrees
-];
+const claude = require('./claude');
+const cursor = require('./cursor');
+const copilot = require('./copilot');
+const codex = require('./codex');
+const _continue = require('./continue');
+const hermes = require('./hermes');
+const worktrees = require('./worktrees');
+
+module.exports = [
+  claude,
+  cursor,
+  copilot,
+  codex,
+  _continue,
+  hermes,
+  worktrees
+];

package/hooks/scripts/lib/platforms/worktrees.js

@@ -1,20 +1,20 @@
-const path = require('path');
-const fs = require('fs');
-
-module.exports = {
-  name: 'worktrees',
-  label: 'Git Worktrees',
-  getIgnorePatterns(workspaceRoot) {
-    const patterns = [];
-    if (fs.existsSync(path.join(workspaceRoot, '.worktrees'))) {
-      patterns.push('.worktrees/');
-    }
-    if (fs.existsSync(path.join(workspaceRoot, 'worktrees'))) {
-      patterns.push('worktrees/');
-    }
-    return patterns;
-  },
-  isInstalled() {
-    return false; // Not a standalone platform hook/settings installer
-  }
-};
+const path = require('path');
+const fs = require('fs');
+
+module.exports = {
+  name: 'worktrees',
+  label: 'Git Worktrees',
+  getIgnorePatterns(workspaceRoot) {
+    const patterns = [];
+    if (fs.existsSync(path.join(workspaceRoot, '.worktrees'))) {
+      patterns.push('.worktrees/');
+    }
+    if (fs.existsSync(path.join(workspaceRoot, 'worktrees'))) {
+      patterns.push('worktrees/');
+    }
+    return patterns;
+  },
+  isInstalled() {
+    return false; // Not a standalone platform hook/settings installer
+  }
+};

package/hooks/scripts/reset-circuit-breaker.js

@@ -1,30 +1,30 @@
-#!/usr/bin/env node
-// Manual escape hatch for the Rule of 3 circuit breaker. Run this directly
-// in your own terminal (not through the agent — Bash is blocked while the
-// breaker is tripped) to clear it without waiting for a new session.
-const fs = require('fs');
-const path = require('path');
-const { getWorkspaceRoot, getSessionDir, readCurrentSession } = require('./lib/harness-state');
-
-// This is run directly by the human, not through the agent, so there's no
-// hook payload to read a session_id from - resolve "the session that's
-// currently blocked" via the pointer bootstrap.js wrote at its last
-// SessionStart.
-const root = getWorkspaceRoot();
-const harnessDir = getSessionDir(root, readCurrentSession(root));
-const stateFile = path.join(harnessDir, 'rule-of-3-state.json');
-const reportFile = path.join(harnessDir, 'zoom-out-report.md');
-
-if (fs.existsSync(stateFile)) {
-  fs.unlinkSync(stateFile);
-  console.log('Rule of 3 circuit breaker cleared.');
-} else {
-  console.log('Rule of 3 circuit breaker is not currently tripped.');
-}
-
-// A human reset is a full clear: the stale reflection report belongs to the
-// loop that was just abandoned.
-if (fs.existsSync(reportFile)) {
-  fs.unlinkSync(reportFile);
-  console.log('Stale zoom-out report removed.');
-}
+#!/usr/bin/env node
+// Manual escape hatch for the Rule of 3 circuit breaker. Run this directly
+// in your own terminal (not through the agent — Bash is blocked while the
+// breaker is tripped) to clear it without waiting for a new session.
+const fs = require('fs');
+const path = require('path');
+const { getWorkspaceRoot, getSessionDir, readCurrentSession } = require('./lib/harness-state');
+
+// This is run directly by the human, not through the agent, so there's no
+// hook payload to read a session_id from - resolve "the session that's
+// currently blocked" via the pointer bootstrap.js wrote at its last
+// SessionStart.
+const root = getWorkspaceRoot();
+const harnessDir = getSessionDir(root, readCurrentSession(root));
+const stateFile = path.join(harnessDir, 'rule-of-3-state.json');
+const reportFile = path.join(harnessDir, 'zoom-out-report.md');
+
+if (fs.existsSync(stateFile)) {
+  fs.unlinkSync(stateFile);
+  console.log('Rule of 3 circuit breaker cleared.');
+} else {
+  console.log('Rule of 3 circuit breaker is not currently tripped.');
+}
+
+// A human reset is a full clear: the stale reflection report belongs to the
+// loop that was just abandoned.
+if (fs.existsSync(reportFile)) {
+  fs.unlinkSync(reportFile);
+  console.log('Stale zoom-out report removed.');
+}

package/hooks/scripts/rule-of-3-tracker.js

@@ -1,91 +1,124 @@
-#!/usr/bin/env node
-const fs = require('fs');
-const path = require('path');
-const crypto = require('crypto');
-const { getWorkspaceRoot, getSessionDir } = require('./lib/harness-state');
-
-// PostToolUse hooks in Claude Code typically receive the tool output via stdin
-let inputData = '';
-
-process.stdin.on('data', chunk => {
-  inputData += chunk;
-});
-
-process.stdin.on('end', () => {
-  try {
-    const payload = JSON.parse(inputData);
-    // Claude Code's documented PostToolUse payload nests tool output under
-    // tool_response rather than at the top level, and does not guarantee a
-    // numeric exit code field. Treating a missing/unknown exit code as a
-    // failure (the old `exitCode !== 0` check) flags every successful call
-    // as an error. Only count a call as a failure when we have positive
-    // evidence of one.
-    const toolResponse = payload.tool_response || {};
-    const stdout = toolResponse.stdout ?? payload.stdout ?? '';
-    const stderr = toolResponse.stderr ?? payload.stderr ?? '';
-    const rawExitCode = toolResponse.exitCode ?? toolResponse.exit_code ?? payload.exitCode;
-    const exitCode = typeof rawExitCode === 'number' ? rawExitCode : undefined;
-
-    const explicitFailure = exitCode !== undefined && exitCode !== 0;
-    const explicitSuccess = exitCode === 0;
-    const stderrSignal = typeof stderr === 'string' && stderr.trim().length > 0;
-    // Without an exit code, bare stderr is not enough evidence: plenty of
-    // tools chat on stderr while succeeding (git, npm warnings, progress
-    // bars), and three identical warnings must not trip the breaker. Require
-    // the text to actually look like a failure.
-    const looksLikeError = /\b(error|fail|failed|failure|exception|fatal|panic|traceback|denied|refused|cannot|unable)\b/i.test(stderr);
-    const isFailure = explicitFailure || (exitCode === undefined && stderrSignal && looksLikeError);
-    const errorText = (stderrSignal ? stderr : stdout) || '';
-
-    const stateFile = path.join(getSessionDir(getWorkspaceRoot(), payload.session_id), 'rule-of-3-state.json');
-
-    let state = { count: 0, lastHash: null, zoomOutResolved: false, zoomOutCycles: 0, lastFailureAt: 0 };
-    if (fs.existsSync(stateFile)) {
-      state = JSON.parse(fs.readFileSync(stateFile, 'utf8'));
-    }
-
-    if (isFailure && errorText) {
-      // Hash the tail of the error to identify identical loops. Normalize
-      // first: timestamps, line numbers, addresses and counters vary between
-      // otherwise-identical failures, and an ever-changing hash would hide a
-      // genuine loop from the breaker.
-      const normalized = errorText.slice(-400)
-        .toLowerCase()
-        .replace(/0x[0-9a-f]+/g, '#')
-        .replace(/\d+/g, '#')
-        .replace(/\s+/g, ' ')
-        .trim();
-      const hash = crypto.createHash('md5').update(normalized.slice(-200)).digest('hex');
-
-      if (state.lastHash === hash) {
-        state.count += 1;
-      } else {
-        state.count = 1;
-        state.lastHash = hash;
-        // New signature = new problem: the zoom-out cycle budget starts over.
-        state.zoomOutCycles = 0;
-      }
-      // Any fresh failure re-arms the breaker. Without this, a signature that
-      // failed again after a success (or after a zoom-out release) would keep
-      // zoomOutResolved=true and the breaker could never trip on it again.
-      state.zoomOutResolved = false;
-      // rule-of-3.js only honors a zoom-out report written AFTER this moment,
-      // so a stale report can't unlock a later trip.
-      state.lastFailureAt = Date.now();
-
-      fs.writeFileSync(stateFile, JSON.stringify(state, null, 2), 'utf8');
-    } else if (explicitSuccess) {
-      // Only reset on a *confirmed* zero exit code, not merely "not a failure".
-      if (state.count > 0 || state.zoomOutCycles > 0) {
-        state.count = 0;
-        state.zoomOutResolved = true;
-        state.zoomOutCycles = 0;
-        fs.writeFileSync(stateFile, JSON.stringify(state, null, 2), 'utf8');
-      }
-    }
-    
-  } catch (err) {
-    // silently fail
-  }
-  process.exit(0);
-});
+#!/usr/bin/env node
+const fs = require('fs');
+const path = require('path');
+const crypto = require('crypto');
+const { getWorkspaceRoot, getSessionDir } = require('./lib/harness-state');
+
+// PostToolUse hooks in Claude Code typically receive the tool output via stdin
+let inputData = '';
+
+process.stdin.on('data', chunk => {
+  inputData += chunk;
+});
+
+process.stdin.on('end', () => {
+  try {
+    const payload = JSON.parse(inputData);
+    const toolResponse = payload.tool_response || {};
+    const stdout = toolResponse.stdout ?? payload.stdout ?? '';
+    const stderr = toolResponse.stderr ?? payload.stderr ?? '';
+    const rawExitCode = toolResponse.exitCode ?? toolResponse.exit_code ?? payload.exitCode;
+    const exitCode = typeof rawExitCode === 'number' ? rawExitCode : undefined;
+
+    const explicitFailure = exitCode !== undefined && exitCode !== 0;
+    const explicitSuccess = exitCode === 0;
+    const stderrSignal = typeof stderr === 'string' && stderr.trim().length > 0;
+    const looksLikeError = /\b(error|fail|failed|failure|exception|fatal|panic|traceback|denied|refused|cannot|unable)\b/i.test(stderr);
+    const isFailure = explicitFailure || (exitCode === undefined && stderrSignal && looksLikeError);
+    const errorText = (stderrSignal ? stderr : stdout) || '';
+
+    const stateFile = path.join(getSessionDir(getWorkspaceRoot(), payload.session_id), 'rule-of-3-state.json');
+
+    let state = { count: 0, lastHash: null, zoomOutResolved: false, zoomOutCycles: 0, lastFailureAt: 0 };
+    if (fs.existsSync(stateFile)) {
+      state = JSON.parse(fs.readFileSync(stateFile, 'utf8'));
+    }
+
+    if (isFailure && errorText) {
+      // Enhanced failure signature detection with context-aware categorization
+      // Hash includes: error message, file path, command attempted, and failure category
+      const filePath = payload.tool_input?.file_path || payload.tool_input?.filePath || '';
+      const command = payload.tool_input?.command || '';
+      
+      // Categorize failure type for context-aware thresholds
+      let category = 'unknown';
+      if (/SyntaxError|Unexpected token|Syntax Error/i.test(errorText)) {
+        category = 'syntax';
+      } else if (/ENOENT|not found|No such file|找不到|不存在/i.test(errorText)) {
+        category = 'environment';
+      } else if (/timeout|timed out|超時|逾時/i.test(errorText)) {
+        category = 'timeout';
+      } else if (/test.*fail|assertion|AssertionError|測試.*失敗|斷言.*錯誤/i.test(errorText)) {
+        category = 'test';
+      } else if (/permission|denied|EACCES|權限/i.test(errorText)) {
+        category = 'permission';
+      } else if (/dependency|module not found|Cannot find module|模組.*找不到/i.test(errorText)) {
+        category = 'dependency';
+      }
+      
+      // Normalize error text with context for better loop detection
+      const normalized = errorText.slice(-400)
+        .toLowerCase()
+        .replace(/0x[0-9a-f]+/g, '#')
+        .replace(/\d+/g, '#')
+        .replace(/\s+/g, ' ')
+        .trim();
+      
+      // Include file path and command in hash for better context
+      const contextString = `${category}:${filePath}:${command}:${normalized.slice(-200)}`;
+      const hash = crypto.createHash('md5').update(contextString).digest('hex');
+
+      // Context-aware thresholds: different failure types have different breaking points
+      let threshold = 3; // Default: 3 strikes
+      if (category === 'syntax') {
+        threshold = 3; // Same syntax error 3 times = stop
+      } else if (category === 'environment') {
+        threshold = 4; // Environment issues may need more exploration
+      } else if (category === 'test') {
+        threshold = 3; // Same test failure 3 times = stop
+      } else if (category === 'timeout') {
+        threshold = 2; // Timeouts are expensive, fail faster
+      } else if (category === 'permission') {
+        threshold = 2; // Permission issues rarely resolve by retrying
+      } else if (category === 'dependency') {
+        threshold = 4; // Dependency issues may need different approaches
+      }
+
+      if (state.lastHash === hash) {
+        state.count += 1;
+        state.category = category; // Track failure category for smarter decisions
+      } else {
+        state.count = 1;
+        state.lastHash = hash;
+        state.category = category;
+        // New signature = new problem: the zoom-out cycle budget starts over.
+        state.zoomOutCycles = 0;
+      }
+      
+      // Store threshold for this failure type
+      state.threshold = threshold;
+      
+      // Any fresh failure re-arms the breaker. Without this, a signature that
+      // failed again after a success (or after a zoom-out release) would keep
+      // zoomOutResolved=true and the breaker could never trip on it again.
+      state.zoomOutResolved = false;
+      // rule-of-3.js only honors a zoom-out report written AFTER this moment,
+      // so a stale report can't unlock a later trip.
+      state.lastFailureAt = Date.now();
+
+      fs.writeFileSync(stateFile, JSON.stringify(state, null, 2), 'utf8');
+    } else if (explicitSuccess) {
+      // Only reset on a *confirmed* zero exit code, not merely "not a failure".
+      if (state.count > 0 || state.zoomOutCycles > 0) {
+        state.count = 0;
+        state.zoomOutResolved = true;
+        state.zoomOutCycles = 0;
+        fs.writeFileSync(stateFile, JSON.stringify(state, null, 2), 'utf8');
+      }
+    }
+    
+  } catch (err) {
+    // silently fail
+  }
+  process.exit(0);
+});

package/hooks/scripts/rule-of-3.js

@@ -33,7 +33,11 @@ function readState(stateFile) {
 }
 
 function isTripped(state) {
-  return !!state && state.count >= 3 && !state.zoomOutResolved;
+  if (!state || state.zoomOutResolved) return false;
+  
+  // Use context-aware thresholds if available, otherwise default to 3
+  const threshold = state.threshold || 3;
+  return state.count >= threshold;
 }
 
 function anySessionTripped(root) {
@@ -41,10 +45,17 @@ function anySessionTripped(root) {
 }
 
 // The one mutation allowed while tripped: writing the reflection report.
+const ALLOWED_WRITE_TOOLS = new Set([
+  'Write', 'Edit', 'create_file', 'replace_string_in_file',
+  'insert_edit_into_file', 'write_file', 'edit_file'
+]);
+
 function isReportWrite(payload, reportFile) {
   if (!payload) return false;
-  if (payload.tool_name !== 'Write' && payload.tool_name !== 'Edit') return false;
-  const target = payload.tool_input && payload.tool_input.file_path;
+  const toolName = payload.tool_name || payload.toolName;
+  if (!ALLOWED_WRITE_TOOLS.has(toolName)) return false;
+  const input = payload.tool_input || payload.toolInput || payload.input || {};
+  const target = input.file_path || input.filePath || input.path || input.file;
   if (!target) return false;
   return path.resolve(target).toLowerCase() === path.resolve(reportFile).toLowerCase();
 }