harness-everything 0.2.0 → 0.2.1

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/harness-state.js

@@ -112,9 +112,35 @@ function ensureHarnessStateIgnored(rootPath) {
       }
     }
     
-    if (patternsToAdd.length > 0) {
-      const separator = content.length === 0 || content.endsWith('\n') ? '' : '\n';
-      fs.appendFileSync(gitignorePath, separator + patternsToAdd.join('\n') + '\n', 'utf8');
+    // This runs once per hook invocation - i.e. once per Claude Code
+    // subprocess - with no cross-process lock around the read-then-append
+    // below. Two hook invocations firing close together (e.g. two tool
+    // calls in the same turn) can each read the file before either has
+    // written, so both decide the same pattern is missing and both append
+    // it, leaving an exact duplicate line behind. Rather than add real
+    // cross-process locking for a housekeeping file that already fails
+    // silently by design, every call also collapses any exact-duplicate
+    // non-comment/non-blank line it finds - so a duplicate from a lost
+    // race self-heals on the very next invocation instead of accumulating.
+    const seen = new Set();
+    let sawDuplicate = false;
+    const dedupedLines = lines.filter(line => {
+      const trimmed = line.trim();
+      if (trimmed === '' || trimmed.startsWith('#')) return true;
+      if (seen.has(trimmed)) {
+        sawDuplicate = true;
+        return false;
+      }
+      seen.add(trimmed);
+      return true;
+    });
+
+    if (patternsToAdd.length > 0 || sawDuplicate) {
+      while (dedupedLines.length > 0 && dedupedLines[dedupedLines.length - 1] === '') {
+        dedupedLines.pop();
+      }
+      const finalLines = dedupedLines.concat(patternsToAdd);
+      fs.writeFileSync(gitignorePath, finalLines.join('\n') + '\n', 'utf8');
     }
   } catch (err) {
     // Fail silently to avoid breaking execution if .gitignore is write-locked

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

@@ -1,42 +1,42 @@
-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 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'));
+  }
+};

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

@@ -1,46 +1,46 @@
-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'));
+  }
+};

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

@@ -1,41 +1,41 @@
-const path = require('path');
-const fs = require('fs');
-
-// Continue's own skills location (.continue/skills/) is unchanged - it
-// already has an established home. Only bookkeeping that never had a home
-// (runtime state, install manifest) converges into .continue/harness-everything/.
-module.exports = {
-  name: 'continue',
-  label: 'Continue.dev',
-  getHarnessDir(workspaceRoot) {
-    return path.join(workspaceRoot, '.continue', 'harness-everything');
-  },
-  getStateDir(workspaceRoot) {
-    return path.join(this.getHarnessDir(workspaceRoot), 'state');
-  },
-  getSkillsDir(workspaceRoot) {
-    return path.join(workspaceRoot, '.continue', 'skills');
-  },
-  getIgnorePatterns(workspaceRoot) {
-    const patterns = ['.continue/harness-everything/'];
-    if (fs.existsSync(path.join(workspaceRoot, '.continue', 'skills'))) {
-      patterns.push('.continue/skills/');
-    }
-    return patterns;
-  },
-  isMatch(pattern, trimmedLine) {
-    if (pattern === '.continue/harness-everything/') {
-      return trimmedLine === '.continue/' ||
-             trimmedLine === '.continue' ||
-             trimmedLine === '.continue/harness-everything' ||
-             trimmedLine === '.continue/harness-everything/';
-    }
-    return trimmedLine === pattern || trimmedLine === pattern.slice(0, -1);
-  },
-  isInstalled(workspaceRoot, userHome, isGlobal) {
-    if (isGlobal) {
-      return fs.existsSync(path.join(userHome, '.continue', 'rules', 'harness.md'));
-    }
-    return fs.existsSync(path.join(workspaceRoot, '.continue', 'rules', 'harness.md'));
-  }
-};
+const path = require('path');
+const fs = require('fs');
+
+// Continue's own skills location (.continue/skills/) is unchanged - it
+// already has an established home. Only bookkeeping that never had a home
+// (runtime state, install manifest) converges into .continue/harness-everything/.
+module.exports = {
+  name: 'continue',
+  label: 'Continue.dev',
+  getHarnessDir(workspaceRoot) {
+    return path.join(workspaceRoot, '.continue', 'harness-everything');
+  },
+  getStateDir(workspaceRoot) {
+    return path.join(this.getHarnessDir(workspaceRoot), 'state');
+  },
+  getSkillsDir(workspaceRoot) {
+    return path.join(workspaceRoot, '.continue', 'skills');
+  },
+  getIgnorePatterns(workspaceRoot) {
+    const patterns = ['.continue/harness-everything/'];
+    if (fs.existsSync(path.join(workspaceRoot, '.continue', 'skills'))) {
+      patterns.push('.continue/skills/');
+    }
+    return patterns;
+  },
+  isMatch(pattern, trimmedLine) {
+    if (pattern === '.continue/harness-everything/') {
+      return trimmedLine === '.continue/' ||
+             trimmedLine === '.continue' ||
+             trimmedLine === '.continue/harness-everything' ||
+             trimmedLine === '.continue/harness-everything/';
+    }
+    return trimmedLine === pattern || trimmedLine === pattern.slice(0, -1);
+  },
+  isInstalled(workspaceRoot, userHome, isGlobal) {
+    if (isGlobal) {
+      return fs.existsSync(path.join(userHome, '.continue', 'rules', 'harness.md'));
+    }
+    return fs.existsSync(path.join(workspaceRoot, '.continue', 'rules', 'harness.md'));
+  }
+};

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

@@ -1,41 +1,41 @@
-const path = require('path');
-const fs = require('fs');
-
-// Copilot's own skills location (.github/skills/) is unchanged - it already
-// has an established home. Only bookkeeping that never had a home (runtime
-// state, install manifest) converges into .github/harness-everything/.
-module.exports = {
-  name: 'copilot',
-  label: 'GitHub Copilot',
-  getHarnessDir(workspaceRoot) {
-    return path.join(workspaceRoot, '.github', 'harness-everything');
-  },
-  getStateDir(workspaceRoot) {
-    return path.join(this.getHarnessDir(workspaceRoot), 'state');
-  },
-  getSkillsDir(workspaceRoot) {
-    return path.join(workspaceRoot, '.github', 'skills');
-  },
-  getIgnorePatterns(workspaceRoot) {
-    const patterns = ['.github/harness-everything/'];
-    if (fs.existsSync(path.join(workspaceRoot, '.github', 'skills'))) {
-      patterns.push('.github/skills/');
-    }
-    return patterns;
-  },
-  isMatch(pattern, trimmedLine) {
-    if (pattern === '.github/harness-everything/') {
-      return trimmedLine === '.github/' ||
-             trimmedLine === '.github' ||
-             trimmedLine === '.github/harness-everything' ||
-             trimmedLine === '.github/harness-everything/';
-    }
-    return trimmedLine === pattern || trimmedLine === pattern.slice(0, -1);
-  },
-  isInstalled(workspaceRoot, userHome, isGlobal) {
-    if (isGlobal) {
-      return false; // Handled via global userPromptsDir check
-    }
-    return fs.existsSync(path.join(workspaceRoot, '.github', 'copilot-instructions.md'));
-  }
-};
+const path = require('path');
+const fs = require('fs');
+
+// Copilot's own skills location (.github/skills/) is unchanged - it already
+// has an established home. Only bookkeeping that never had a home (runtime
+// state, install manifest) converges into .github/harness-everything/.
+module.exports = {
+  name: 'copilot',
+  label: 'GitHub Copilot',
+  getHarnessDir(workspaceRoot) {
+    return path.join(workspaceRoot, '.github', 'harness-everything');
+  },
+  getStateDir(workspaceRoot) {
+    return path.join(this.getHarnessDir(workspaceRoot), 'state');
+  },
+  getSkillsDir(workspaceRoot) {
+    return path.join(workspaceRoot, '.github', 'skills');
+  },
+  getIgnorePatterns(workspaceRoot) {
+    const patterns = ['.github/harness-everything/'];
+    if (fs.existsSync(path.join(workspaceRoot, '.github', 'skills'))) {
+      patterns.push('.github/skills/');
+    }
+    return patterns;
+  },
+  isMatch(pattern, trimmedLine) {
+    if (pattern === '.github/harness-everything/') {
+      return trimmedLine === '.github/' ||
+             trimmedLine === '.github' ||
+             trimmedLine === '.github/harness-everything' ||
+             trimmedLine === '.github/harness-everything/';
+    }
+    return trimmedLine === pattern || trimmedLine === pattern.slice(0, -1);
+  },
+  isInstalled(workspaceRoot, userHome, isGlobal) {
+    if (isGlobal) {
+      return false; // Handled via global userPromptsDir check
+    }
+    return fs.existsSync(path.join(workspaceRoot, '.github', 'copilot-instructions.md'));
+  }
+};