harness-everything 0.2.1 → 0.2.2

package/hooks/scripts/context-compact.js

@@ -1,114 +1,114 @@
-#!/usr/bin/env node
-const { execSync } = require('child_process');
-const fs = require('fs');
-const path = require('path');
-const { getWorkspaceRoot, getSessionDir } = require('./lib/harness-state');
-
-function main(payload) {
-  try {
-    // Let's run a quick Git check to estimate context size from modified files
-    const statusRaw = execSync('git status --porcelain', { encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'] }).trim();
-    if (!statusRaw) {
-      process.exit(0);
-    }
-
-    const lines = statusRaw.split('\n').filter(Boolean);
-    const modifiedCount = lines.length;
-
-    let totalDiffLines = 0;
-    try {
-      const diffStat = execSync('git diff --numstat', { encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'] }).trim();
-      if (diffStat) {
-        diffStat.split('\n').forEach(line => {
-          const [added, deleted] = line.split('\t');
-          if (added && added !== '-' && deleted && deleted !== '-') {
-            totalDiffLines += parseInt(added) + parseInt(deleted);
-          }
-        });
-      }
-    } catch (err) {
-      // Ignore git errors
-    }
-
-    // Thresholds for warning
-    const HIGH_FILE_LIMIT = 6;
-    const HIGH_LINE_LIMIT = 400;
-
-    const overThreshold = modifiedCount >= HIGH_FILE_LIMIT || totalDiffLines >= HIGH_LINE_LIMIT;
-
-    // Throttle: this hook fires on EVERY Bash/Edit/Write call, and a warning
-    // about context bloat that repeats on every call becomes context bloat
-    // itself (and trains the model to ignore hook output). Warn on the first
-    // crossing, then only every 10th call while still over - or when the diff
-    // has grown another 50% since the last warning.
-    const stateFile = path.join(getSessionDir(getWorkspaceRoot(), payload && payload.session_id), 'context-compact-state.json');
-    let state = { overCount: 0, lastWarnedLines: 0 };
-    try {
-      if (fs.existsSync(stateFile)) state = JSON.parse(fs.readFileSync(stateFile, 'utf8'));
-    } catch (err) { /* use default */ }
-
-    if (!overThreshold) {
-      if (state.overCount > 0) {
-        try {
-          fs.writeFileSync(stateFile, JSON.stringify({ overCount: 0, lastWarnedLines: 0 }), 'utf8');
-        } catch (err) { /* ignore */ }
-      }
-      process.exit(0);
-    }
-
-    state.overCount += 1;
-    const onBeat = state.overCount === 1 || state.overCount % 10 === 0;
-    const grewALot = totalDiffLines >= (state.lastWarnedLines || 0) * 1.5 && totalDiffLines > state.lastWarnedLines;
-    const shouldWarn = onBeat || grewALot;
-    if (shouldWarn) state.lastWarnedLines = totalDiffLines;
-    try {
-      fs.writeFileSync(stateFile, JSON.stringify(state), 'utf8');
-    } catch (err) { /* ignore */ }
-
-    if (shouldWarn) {
-      // Plain stdout on exit 0 is only surfaced to Claude for UserPromptSubmit/
-      // SessionStart - for PreToolUse it's written to a debug log Claude never
-      // sees. To actually get an advisory (non-blocking) message in front of
-      // Claude here, it has to go through hookSpecificOutput.additionalContext.
-      const message = [
-        `Harness OS - Context Bloat Warning`,
-        `- Modified Files: ${modifiedCount}`,
-        `- Accumulated Diff: ${totalDiffLines} lines`,
-        ``,
-        `Caution: Large number of modified files or big diffs will bloat the Agent context window, leading to:`,
-        `  1. "Lost in the Middle" reasoning degradation`,
-        `  2. Rapidly inflating token costs`,
-        ``,
-        `Recommended Action:`,
-        `  - Run atomic commits or stage completed files.`,
-        `  - Use targeted sub-folders rather than broad searches.`,
-        `  - Ask user to run '/compact' or '/clear' if supported.`
-      ].join('\n');
-
-      console.log(JSON.stringify({
-        hookSpecificOutput: {
-          hookEventName: "PreToolUse",
-          additionalContext: message
-        }
-      }));
-    }
-  } catch (err) {
-    // Fail open - never block user execution on compaction checker
-  }
-
-  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
+const { execSync } = require('child_process');
+const fs = require('fs');
+const path = require('path');
+const { getWorkspaceRoot, getSessionDir } = require('./lib/harness-state');
+
+function main(payload) {
+  try {
+    // Let's run a quick Git check to estimate context size from modified files
+    const statusRaw = execSync('git status --porcelain', { encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'] }).trim();
+    if (!statusRaw) {
+      process.exit(0);
+    }
+
+    const lines = statusRaw.split('\n').filter(Boolean);
+    const modifiedCount = lines.length;
+
+    let totalDiffLines = 0;
+    try {
+      const diffStat = execSync('git diff --numstat', { encoding: 'utf8', stdio: ['ignore', 'pipe', 'ignore'] }).trim();
+      if (diffStat) {
+        diffStat.split('\n').forEach(line => {
+          const [added, deleted] = line.split('\t');
+          if (added && added !== '-' && deleted && deleted !== '-') {
+            totalDiffLines += parseInt(added) + parseInt(deleted);
+          }
+        });
+      }
+    } catch (err) {
+      // Ignore git errors
+    }
+
+    // Thresholds for warning
+    const HIGH_FILE_LIMIT = 6;
+    const HIGH_LINE_LIMIT = 400;
+
+    const overThreshold = modifiedCount >= HIGH_FILE_LIMIT || totalDiffLines >= HIGH_LINE_LIMIT;
+
+    // Throttle: this hook fires on EVERY Bash/Edit/Write call, and a warning
+    // about context bloat that repeats on every call becomes context bloat
+    // itself (and trains the model to ignore hook output). Warn on the first
+    // crossing, then only every 10th call while still over - or when the diff
+    // has grown another 50% since the last warning.
+    const stateFile = path.join(getSessionDir(getWorkspaceRoot(), payload && payload.session_id), 'context-compact-state.json');
+    let state = { overCount: 0, lastWarnedLines: 0 };
+    try {
+      if (fs.existsSync(stateFile)) state = JSON.parse(fs.readFileSync(stateFile, 'utf8'));
+    } catch (err) { /* use default */ }
+
+    if (!overThreshold) {
+      if (state.overCount > 0) {
+        try {
+          fs.writeFileSync(stateFile, JSON.stringify({ overCount: 0, lastWarnedLines: 0 }), 'utf8');
+        } catch (err) { /* ignore */ }
+      }
+      process.exit(0);
+    }
+
+    state.overCount += 1;
+    const onBeat = state.overCount === 1 || state.overCount % 10 === 0;
+    const grewALot = totalDiffLines >= (state.lastWarnedLines || 0) * 1.5 && totalDiffLines > state.lastWarnedLines;
+    const shouldWarn = onBeat || grewALot;
+    if (shouldWarn) state.lastWarnedLines = totalDiffLines;
+    try {
+      fs.writeFileSync(stateFile, JSON.stringify(state), 'utf8');
+    } catch (err) { /* ignore */ }
+
+    if (shouldWarn) {
+      // Plain stdout on exit 0 is only surfaced to Claude for UserPromptSubmit/
+      // SessionStart - for PreToolUse it's written to a debug log Claude never
+      // sees. To actually get an advisory (non-blocking) message in front of
+      // Claude here, it has to go through hookSpecificOutput.additionalContext.
+      const message = [
+        `Harness OS - Context Bloat Warning`,
+        `- Modified Files: ${modifiedCount}`,
+        `- Accumulated Diff: ${totalDiffLines} lines`,
+        ``,
+        `Caution: Large number of modified files or big diffs will bloat the Agent context window, leading to:`,
+        `  1. "Lost in the Middle" reasoning degradation`,
+        `  2. Rapidly inflating token costs`,
+        ``,
+        `Recommended Action:`,
+        `  - Run atomic commits or stage completed files.`,
+        `  - Use targeted sub-folders rather than broad searches.`,
+        `  - Ask user to run '/compact' or '/clear' if supported.`
+      ].join('\n');
+
+      console.log(JSON.stringify({
+        hookSpecificOutput: {
+          hookEventName: "PreToolUse",
+          additionalContext: message
+        }
+      }));
+    }
+  } catch (err) {
+    // Fail open - never block user execution on compaction checker
+  }
+
+  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);
+});

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

@@ -1,42 +1,132 @@
-const path = require('path');
-const fs = require('fs');
-
-// Claude Code has no native project-skill directory this package can piggy
-// back on, so everything Harness adds locally - runtime state AND the
-// installed skill copies - lives under one self-contained subfolder of the
-// platform's own directory: .claude/harness-everything/. That keeps it out
-// of a self-invented top-level folder (the old .harness/) while still being
-// exclusively ours - nothing else creates a directory literally named
-// "harness-everything" inside .claude/, so it's trivial to add/remove
-// without touching anything else Claude Code or the user put in .claude/.
-module.exports = {
-  name: 'claude',
-  label: 'Claude Code',
-  getHarnessDir(workspaceRoot) {
-    return path.join(workspaceRoot, '.claude', 'harness-everything');
-  },
-  getStateDir(workspaceRoot) {
-    return path.join(this.getHarnessDir(workspaceRoot), 'state');
-  },
-  getSkillsDir(workspaceRoot) {
-    return path.join(this.getHarnessDir(workspaceRoot), 'skills');
-  },
-  getIgnorePatterns(workspaceRoot) {
-    return ['.claude/harness-everything/'];
-  },
-  isMatch(pattern, trimmedLine) {
-    if (pattern === '.claude/harness-everything/') {
-      return trimmedLine === '.claude/' ||
-             trimmedLine === '.claude' ||
-             trimmedLine === '.claude/harness-everything' ||
-             trimmedLine === '.claude/harness-everything/';
-    }
-    return trimmedLine === pattern || trimmedLine === pattern.slice(0, -1);
-  },
-  isInstalled(workspaceRoot, userHome, isGlobal) {
-    if (isGlobal) {
-      return fs.existsSync(path.join(userHome, '.claude', 'settings.json'));
-    }
-    return fs.existsSync(path.join(workspaceRoot, '.claude', 'settings.json'));
-  }
-};
+const path = require('path');
+const fs = require('fs');
+
+// Claude Code has native project-skill directory: .claude/skills/.
+// Runtime state converges into .claude/harness-everything/.
+module.exports = {
+  name: 'claude',
+  label: 'Claude Code',
+  getHarnessDir(workspaceRoot) {
+    return path.join(workspaceRoot, '.claude', 'harness-everything');
+  },
+  getStateDir(workspaceRoot) {
+    return path.join(this.getHarnessDir(workspaceRoot), 'state');
+  },
+  getSkillsDir(workspaceRoot) {
+    return path.join(workspaceRoot, '.claude', 'skills');
+  },
+  getIgnorePatterns(workspaceRoot) {
+    const patterns = ['.claude/harness-everything/'];
+    if (fs.existsSync(path.join(workspaceRoot, '.claude', 'skills'))) {
+      patterns.push('.claude/skills/');
+    }
+    return patterns;
+  },
+  isMatch(pattern, trimmedLine) {
+    if (pattern === '.claude/harness-everything/') {
+      return trimmedLine === '.claude/' ||
+             trimmedLine === '.claude' ||
+             trimmedLine === '.claude/harness-everything' ||
+             trimmedLine === '.claude/harness-everything/';
+    }
+    return trimmedLine === pattern || trimmedLine === pattern.slice(0, -1);
+  },
+  isInstalled(workspaceRoot, userHome, isGlobal) {
+    if (isGlobal) {
+      return fs.existsSync(path.join(userHome, '.claude', 'settings.json'));
+    }
+    return fs.existsSync(path.join(workspaceRoot, '.claude', 'settings.json'));
+  },
+  getSkillsTarget({ workspaceRoot, userHome, isGlobal, manifest }) {
+    if (isGlobal) {
+      const claudeDir = path.join(userHome, '.claude');
+      return {
+        path: path.join(claudeDir, 'skills'),
+        label: '~/.claude/skills/',
+        manifestPath: manifest.getManifestPath(claudeDir)
+      };
+    } else {
+      const claudeDir = path.join(workspaceRoot, '.claude');
+      return {
+        path: path.join(claudeDir, 'skills'),
+        label: '.claude/skills/',
+        manifestPath: manifest.getManifestPath(claudeDir)
+      };
+    }
+  },
+  install({ isGlobal, targetWorkspaceRoot, harnessSourceDir, packageVersion, getUserPromptsDir, advisory, claudeHooks, manifest }) {
+    const userHome = require('os').homedir();
+    const claudeDir = isGlobal ? path.join(userHome, '.claude') : path.join(targetWorkspaceRoot, '.claude');
+    if (!fs.existsSync(claudeDir)) {
+      fs.mkdirSync(claudeDir, { recursive: true });
+      console.log(`  Created ${isGlobal ? '~/.claude' : '.claude'} directory`);
+    }
+    const claudeSettingsFile = path.join(claudeDir, 'settings.json');
+    let claudeConfig = {};
+    if (fs.existsSync(claudeSettingsFile)) {
+      try {
+        claudeConfig = JSON.parse(fs.readFileSync(claudeSettingsFile, 'utf8'));
+      } catch (e) {
+        console.warn(`  ⚠️ Existing ${isGlobal ? '~' : ''}/.claude/settings.json is malformed, creating fresh one.`);
+      }
+    }
+
+    const sourceHooksFile = path.join(harnessSourceDir, 'hooks', 'hooks.json');
+    if (fs.existsSync(sourceHooksFile)) {
+      const sourceHooksObj = JSON.parse(fs.readFileSync(sourceHooksFile, 'utf8'));
+      const resolvedHooks = {};
+      for (const [hookType, hookList] of Object.entries(sourceHooksObj.hooks || {})) {
+        resolvedHooks[hookType] = hookList.map(hookItem => {
+          const cloned = JSON.parse(JSON.stringify(hookItem));
+          if (cloned.hooks) {
+            cloned.hooks = cloned.hooks.map(h => {
+              if (h.type === 'command' && h.command) {
+                h.command = h.command.replace(/^node\s+"?([^"\s]+)"?/, (m, scriptPath) => {
+                  const abs = path.isAbsolute(scriptPath) ? scriptPath : path.join(harnessSourceDir, scriptPath);
+                  return `node "${abs}"`;
+                });
+              }
+              return h;
+            });
+          }
+          cloned.harness = { package: manifest.PACKAGE_NAME, version: packageVersion, author: manifest.HARNESS_AUTHOR };
+          return cloned;
+        });
+      }
+      claudeHooks.mergeHarnessHooks(claudeConfig, resolvedHooks);
+    }
+
+    fs.writeFileSync(claudeSettingsFile, JSON.stringify(claudeConfig, null, 2), 'utf8');
+    console.log(`  ✅ Configured Claude Code hooks safely in ${isGlobal ? '~' : ''}/.claude/settings.json`);
+
+    // DOWNWARD COMPATIBLE CLEANUP: Remove old wrong skill folder .claude/harness-everything/skills if it exists
+    const oldSkillsDir = path.join(claudeDir, 'harness-everything', 'skills');
+    if (fs.existsSync(oldSkillsDir)) {
+      try {
+        fs.rmSync(oldSkillsDir, { recursive: true, force: true });
+        console.log(`  🧹 Cleaned up legacy incorrect local skills folder at: .claude/harness-everything/skills/`);
+      } catch (e) {
+        console.warn(`  ⚠️ Failed to remove legacy skills folder: ${e.message}`);
+      }
+    }
+  },
+  uninstall({ removeLocal, removeGlobal, workspaceRoot, userHome, getUserPromptsDir, advisory, claudeHooks, manifest, cleanEmptyDirs }) {
+    if (removeLocal) {
+      const localSettingsFile = path.join(workspaceRoot, '.claude', 'settings.json');
+      claudeHooks.removeHarnessHooks(localSettingsFile);
+      
+      // Clear old wrong skill folder .claude/harness-everything/skills if any remain
+      const oldSkillsDir = path.join(workspaceRoot, '.claude', 'harness-everything', 'skills');
+      if (fs.existsSync(oldSkillsDir)) {
+        fs.rmSync(oldSkillsDir, { recursive: true, force: true });
+        console.log(`  🧹 Cleaned up legacy incorrect local skills folder at: .claude/harness-everything/skills/`);
+      }
+    }
+    if (removeGlobal) {
+      const globalSettingsFile = path.join(userHome, '.claude', 'settings.json');
+      claudeHooks.removeHarnessHooks(globalSettingsFile);
+      const globalClaudeDir = path.join(userHome, '.claude');
+      cleanEmptyDirs(globalClaudeDir, [userHome]);
+    }
+  }
+};

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

@@ -1,46 +1,96 @@
-const path = require('path');
-const fs = require('fs');
-
-// Codex CLI's real project-scoped home is `.codex/` (it reads `.codex/skills/`
-// for project skills and `.codex/config.toml` for CLI/sandbox config, walking
-// up to the `.git` root exactly like AGENTS.md - see
-// https://developers.openai.com/codex/config-advanced and
-// https://www.agensi.io/learn/codex-cli-agents-md-complete-guide). Skills
-// stay at that native `.codex/skills/` location; only bookkeeping that never
-// had a home (runtime state, install manifest) converges into
-// `.codex/harness-everything/`.
-module.exports = {
-  name: 'codex',
-  label: 'Codex / Gemini',
-  getHarnessDir(workspaceRoot) {
-    return path.join(workspaceRoot, '.codex', 'harness-everything');
-  },
-  getStateDir(workspaceRoot) {
-    return path.join(this.getHarnessDir(workspaceRoot), 'state');
-  },
-  getSkillsDir(workspaceRoot) {
-    return path.join(workspaceRoot, '.codex', 'skills');
-  },
-  getIgnorePatterns(workspaceRoot) {
-    const patterns = ['.codex/harness-everything/'];
-    if (fs.existsSync(path.join(workspaceRoot, '.codex', 'skills'))) {
-      patterns.push('.codex/skills/');
-    }
-    return patterns;
-  },
-  isMatch(pattern, trimmedLine) {
-    if (pattern === '.codex/harness-everything/') {
-      return trimmedLine === '.codex/' ||
-             trimmedLine === '.codex' ||
-             trimmedLine === '.codex/harness-everything' ||
-             trimmedLine === '.codex/harness-everything/';
-    }
-    return trimmedLine === pattern || trimmedLine === pattern.slice(0, -1);
-  },
-  isInstalled(workspaceRoot, userHome, isGlobal) {
-    if (isGlobal) {
-      return false;
-    }
-    return fs.existsSync(path.join(workspaceRoot, 'AGENTS.md'));
-  }
-};
+const path = require('path');
+const fs = require('fs');
+
+// Codex CLI's real project-scoped home is `.codex/` (it reads `.codex/skills/`
+// for project skills and `.codex/config.toml` for CLI/sandbox config, walking
+// up to the `.git` root exactly like AGENTS.md - see
+// https://developers.openai.com/codex/config-advanced and
+// https://www.agensi.io/learn/codex-cli-agents-md-complete-guide). Skills
+// stay at that native `.codex/skills/` location; only bookkeeping that never
+// had a home (runtime state, install manifest) converges into
+// `.codex/harness-everything/`.
+module.exports = {
+  name: 'codex',
+  label: 'Codex / Gemini',
+  getHarnessDir(workspaceRoot) {
+    return path.join(workspaceRoot, '.codex', 'harness-everything');
+  },
+  getStateDir(workspaceRoot) {
+    return path.join(this.getHarnessDir(workspaceRoot), 'state');
+  },
+  getSkillsDir(workspaceRoot) {
+    return path.join(workspaceRoot, '.codex', 'skills');
+  },
+  getIgnorePatterns(workspaceRoot) {
+    const patterns = ['.codex/harness-everything/'];
+    if (fs.existsSync(path.join(workspaceRoot, '.codex', 'skills'))) {
+      patterns.push('.codex/skills/');
+    }
+    return patterns;
+  },
+  isMatch(pattern, trimmedLine) {
+    if (pattern === '.codex/harness-everything/') {
+      return trimmedLine === '.codex/' ||
+             trimmedLine === '.codex' ||
+             trimmedLine === '.codex/harness-everything' ||
+             trimmedLine === '.codex/harness-everything/';
+    }
+    return trimmedLine === pattern || trimmedLine === pattern.slice(0, -1);
+  },
+  isInstalled(workspaceRoot, userHome, isGlobal) {
+    if (isGlobal) {
+      return false;
+    }
+    return fs.existsSync(path.join(workspaceRoot, 'AGENTS.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),
+      };
+    } else {
+      const codexDir = path.join(workspaceRoot, '.codex');
+      return {
+        path: path.join(codexDir, 'skills'),
+        label: '.codex/skills/',
+        manifestPath: manifest.getManifestPath(codexDir),
+      };
+    }
+  },
+  install({ isGlobal, targetWorkspaceRoot, getUserPromptsDir, advisory }) {
+    if (!isGlobal) {
+      const targetFile = path.join(targetWorkspaceRoot, 'AGENTS.md');
+      advisory.injectAdvisoryText(targetFile, '# AGENTS.md', 'AGENTS.md');
+    } else {
+      try {
+        const promptsDir = getUserPromptsDir();
+        if (!fs.existsSync(promptsDir)) {
+          fs.mkdirSync(promptsDir, { recursive: true });
+        }
+        const vscodeAgentFile = path.join(promptsDir, 'harness.agent.md');
+        fs.writeFileSync(vscodeAgentFile, advisory.buildCodexGlobalContent(), 'utf8');
+        console.log(`  ✅ Installed global Codex agent to VS Code: ${vscodeAgentFile}`);
+      } catch (err) {
+        console.warn(`  ⚠️ Failed to write VS Code user prompts folder: ${err.message}`);
+      }
+    }
+  },
+  uninstall({ removeLocal, removeGlobal, workspaceRoot, userHome, getUserPromptsDir, cleanEmptyDirs }) {
+    const advisory = require('../../../../scripts/lib/advisory-text');
+    if (removeLocal) {
+      advisory.removeAdvisoryText(path.join(workspaceRoot, 'AGENTS.md'));
+    }
+    if (removeGlobal) {
+      const promptsDir = getUserPromptsDir();
+      const vscodeAgentFile = path.join(promptsDir, 'harness.agent.md');
+      if (fs.existsSync(vscodeAgentFile)) {
+        fs.unlinkSync(vscodeAgentFile);
+        console.log(`  ✅ Removed global Codex agent: ${vscodeAgentFile}`);
+      }
+      cleanEmptyDirs(promptsDir, [userHome]);
+    }
+  }
+};