harness-everything 0.2.2 → 0.4.0

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

@@ -1,60 +1,113 @@
 const path = require('path');
 const fs = require('fs');
 
+const CLAUDE_PLUGIN_ROOT_TOKEN = '${CLAUDE_PLUGIN_ROOT}';
+
+// Removes legacy static-skill subfolders left behind by the pre-0.2.2 bug
+// that installed skills to .claude/harness-everything/skills/ instead of the
+// native .claude/skills/, WITHOUT touching skills/generated/ - self-evolve
+// stores the user's own dynamically-generated skills there (see
+// skill-creator/SKILL.md §4), and a blind recursive rmSync on the whole
+// skills/ folder would silently delete that accumulated learning.
+function cleanupLegacySkillsDir(oldSkillsDir) {
+  if (!fs.existsSync(oldSkillsDir)) return;
+  try {
+    const entries = fs.readdirSync(oldSkillsDir, { withFileTypes: true });
+    let removedAny = false;
+    for (const entry of entries) {
+      if (entry.name === 'generated') continue;
+      fs.rmSync(path.join(oldSkillsDir, entry.name), { recursive: true, force: true });
+      removedAny = true;
+    }
+    if (fs.readdirSync(oldSkillsDir).length === 0) fs.rmdirSync(oldSkillsDir);
+    if (removedAny) console.log(`  🧹 Cleaned up legacy incorrect local skills folder at: .claude/harness-everything/skills/ (preserved skills/generated/ - self-evolve's own dynamic skills)`);
+  } catch (e) {
+    console.warn(`  ⚠️ Failed to clean up legacy skills folder: ${e.message}`);
+  }
+}
+
+function resolveClaudeHookCommand(command, harnessSourceDir) {
+  return command.replace(/^node\s+"?([^"\s]+)"?/, (match, scriptPath) => {
+    let abs;
+    if (scriptPath === CLAUDE_PLUGIN_ROOT_TOKEN || scriptPath.startsWith(`${CLAUDE_PLUGIN_ROOT_TOKEN}/`) || scriptPath.startsWith(`${CLAUDE_PLUGIN_ROOT_TOKEN}\\`)) {
+      const relative = scriptPath
+        .slice(CLAUDE_PLUGIN_ROOT_TOKEN.length)
+        .replace(/^[/\\]+/, '');
+      abs = path.join(harnessSourceDir, relative);
+    } else {
+      abs = path.isAbsolute(scriptPath) ? scriptPath : path.join(harnessSourceDir, scriptPath);
+    }
+    return `node "${abs}"`;
+  });
+}
+
 // 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');
-  },
+  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/');
+    const skillsDir = path.join(workspaceRoot, '.claude', 'skills');
+    if (fs.existsSync(skillsDir)) {
+      try {
+        const entries = fs.readdirSync(skillsDir, { withFileTypes: true });
+        for (const entry of entries) {
+          if (!entry.isDirectory()) continue;
+          const skillMdPath = path.join(skillsDir, entry.name, 'SKILL.md');
+          if (!fs.existsSync(skillMdPath)) continue;
+          const content = fs.readFileSync(skillMdPath, 'utf8');
+          const authorLine = content.split('\n').find(line => line.trim().startsWith('author:'));
+          if (authorLine && authorLine.includes('Miya Daniel')) patterns.push(`.claude/skills/${entry.name}/`);
+        }
+      } catch (e) {
+        // Fallback or ignore to prevent breaking execution
+      }
+    }
+    const agentsDir = path.join(workspaceRoot, '.claude', 'agents');
+    if (fs.existsSync(agentsDir)) {
+      try {
+        for (const entry of fs.readdirSync(agentsDir, { withFileTypes: true })) {
+          if (!entry.isFile() || !/^fable-(?:orchestrator|verifier|worker-haiku|worker-sonnet)\.md$/i.test(entry.name)) continue;
+          const content = fs.readFileSync(path.join(agentsDir, entry.name), 'utf8');
+          if (/^name:\s*fable-/mi.test(content)) patterns.push(`.claude/agents/${entry.name}`);
+        }
+      } catch (e) {
+        // Fallback or ignore to prevent breaking installation.
+      }
     }
     return patterns;
   },
   isMatch(pattern, trimmedLine) {
+    if (trimmedLine === '.claude/' || trimmedLine === '.claude') return true;
     if (pattern === '.claude/harness-everything/') {
-      return trimmedLine === '.claude/' ||
-             trimmedLine === '.claude' ||
-             trimmedLine === '.claude/harness-everything' ||
-             trimmedLine === '.claude/harness-everything/';
+      return 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'));
+    return fs.existsSync(path.join(isGlobal ? userHome : 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)
-      };
-    }
+    const claudeDir = isGlobal ? path.join(userHome, '.claude') : path.join(workspaceRoot, '.claude');
+    return {
+      path: path.join(claudeDir, 'skills'),
+      label: isGlobal ? '~/.claude/skills/' : '.claude/skills/',
+      manifestPath: manifest.getManifestPath(claudeDir)
+    };
+  },
+  getAgentsTarget({ workspaceRoot, userHome, isGlobal, manifest }) {
+    const claudeDir = isGlobal ? path.join(userHome, '.claude') : path.join(workspaceRoot, '.claude');
+    return {
+      path: path.join(claudeDir, 'agents'),
+      label: isGlobal ? '~/.claude/agents/' : '.claude/agents/',
+      manifestPath: manifest.getManifestPath(claudeDir)
+    };
   },
-  install({ isGlobal, targetWorkspaceRoot, harnessSourceDir, packageVersion, getUserPromptsDir, advisory, claudeHooks, manifest }) {
+  install({ isGlobal, targetWorkspaceRoot, harnessSourceDir, packageVersion, claudeHooks, manifest }) {
     const userHome = require('os').homedir();
     const claudeDir = isGlobal ? path.join(userHome, '.claude') : path.join(targetWorkspaceRoot, '.claude');
     if (!fs.existsSync(claudeDir)) {
@@ -67,7 +120,7 @@ module.exports = {
       try {
         claudeConfig = JSON.parse(fs.readFileSync(claudeSettingsFile, 'utf8'));
       } catch (e) {
-        console.warn(`  ⚠️ Existing ${isGlobal ? '~' : ''}/.claude/settings.json is malformed, creating fresh one.`);
+        throw new Error(`Refusing to modify malformed ${isGlobal ? '~/' : ''}.claude/settings.json: ${e.message}`);
       }
     }
 
@@ -81,10 +134,7 @@ module.exports = {
           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}"`;
-                });
+                h.command = resolveClaudeHookCommand(h.command, harnessSourceDir);
               }
               return h;
             });
@@ -98,35 +148,21 @@ module.exports = {
 
     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}`);
-      }
-    }
+    cleanupLegacySkillsDir(path.join(claudeDir, 'harness-everything', 'skills'));
   },
-  uninstall({ removeLocal, removeGlobal, workspaceRoot, userHome, getUserPromptsDir, advisory, claudeHooks, manifest, cleanEmptyDirs }) {
+  uninstall({ removeLocal, removeGlobal, workspaceRoot, userHome, claudeHooks, 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/`);
-      }
+      cleanupLegacySkillsDir(path.join(workspaceRoot, '.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]);
+      cleanEmptyDirs(path.join(userHome, '.claude'), [userHome]);
     }
+  },
+  resolveHookCommand(command, harnessSourceDir) {
+    return resolveClaudeHookCommand(command, harnessSourceDir);
   }
 };

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 discovers project Agent Skills from repo-scoped `.agents/skills/`
+// (walking from cwd toward the repository root). Keep project skills at that
+// native shared location so every installer mode, including `--copy`, remains
+// discoverable. Codex-specific runtime/install bookkeeping still lives under
 // `.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');
-  },
+  label: 'Codex',
+  getHarnessDir(workspaceRoot) { return path.join(workspaceRoot, '.codex', 'harness-everything'); },
+  getStateDir(workspaceRoot) { return path.join(this.getHarnessDir(workspaceRoot), 'state'); },
+  getSkillsDir(workspaceRoot) { return path.join(workspaceRoot, '.agents', 'skills'); },
   getIgnorePatterns(workspaceRoot) {
     const patterns = ['.codex/harness-everything/'];
-    if (fs.existsSync(path.join(workspaceRoot, '.codex', 'skills'))) {
-      patterns.push('.codex/skills/');
+    const skillsDir = path.join(workspaceRoot, '.agents', 'skills');
+    if (fs.existsSync(skillsDir)) {
+      try {
+        for (const entry of fs.readdirSync(skillsDir, { withFileTypes: true })) {
+          if (!entry.isDirectory()) continue;
+          const skillMdPath = path.join(skillsDir, entry.name, 'SKILL.md');
+          if (!fs.existsSync(skillMdPath)) continue;
+          const content = fs.readFileSync(skillMdPath, 'utf8');
+          const authorLine = content.split('\n').find(line => line.trim().startsWith('author:'));
+          if (authorLine && authorLine.includes('Miya Daniel')) patterns.push(`.agents/skills/${entry.name}/`);
+        }
+      } catch (e) {
+        // Fallback or ignore to prevent breaking execution
+      }
     }
     return patterns;
   },
   isMatch(pattern, trimmedLine) {
+    if (trimmedLine === '.codex/' || trimmedLine === '.codex') return pattern === '.codex/harness-everything/';
+    if (trimmedLine === '.agents/' || trimmedLine === '.agents') return pattern.startsWith('.agents/skills/');
     if (pattern === '.codex/harness-everything/') {
-      return trimmedLine === '.codex/' ||
-             trimmedLine === '.codex' ||
-             trimmedLine === '.codex/harness-everything' ||
-             trimmedLine === '.codex/harness-everything/';
+      return trimmedLine === '.codex/harness-everything' || trimmedLine === '.codex/harness-everything/';
     }
     return trimmedLine === pattern || trimmedLine === pattern.slice(0, -1);
   },
   isInstalled(workspaceRoot, userHome, isGlobal) {
-    if (isGlobal) {
-      return false;
-    }
+    if (isGlobal) return false;
     return fs.existsSync(path.join(workspaceRoot, 'AGENTS.md'));
   },
   getSkillsTarget({ workspaceRoot, userHome, isGlobal, manifest }) {
@@ -51,46 +51,38 @@ module.exports = {
         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),
-      };
     }
+    const codexDir = path.join(workspaceRoot, '.codex');
+    return {
+      path: path.join(workspaceRoot, '.agents', 'skills'),
+      label: '.agents/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}`);
-      }
+      return;
     }
+    const promptsDir = getUserPromptsDir();
+    if (!fs.existsSync(promptsDir)) fs.mkdirSync(promptsDir, { recursive: true });
+    const vscodeAgentFile = path.join(promptsDir, 'harness.agent.md');
+    advisory.assertHarnessOwnedOrAbsent(vscodeAgentFile, 'global Codex agent');
+    fs.writeFileSync(vscodeAgentFile, advisory.buildCodexGlobalContent(), 'utf8');
+    console.log(`  ✅ Installed global Codex agent to VS Code: ${vscodeAgentFile}`);
   },
   uninstall({ removeLocal, removeGlobal, workspaceRoot, userHome, getUserPromptsDir, cleanEmptyDirs }) {
     const advisory = require('../../../../scripts/lib/advisory-text');
-    if (removeLocal) {
-      advisory.removeAdvisoryText(path.join(workspaceRoot, 'AGENTS.md'));
-    }
+    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)) {
+      if (fs.existsSync(vscodeAgentFile) && advisory.hasHarnessMarker(fs.readFileSync(vscodeAgentFile, 'utf8'))) {
         fs.unlinkSync(vscodeAgentFile);
         console.log(`  ✅ Removed global Codex agent: ${vscodeAgentFile}`);
       }
       cleanEmptyDirs(promptsDir, [userHome]);
     }
   }
-};
+};

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

@@ -1,9 +1,9 @@
 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/.
+// The reviewed official Continue documentation establishes `.continue/rules/`,
+// not a native `SKILL.md` discovery contract. Keep these paths as explicit
+// installer adapter targets without claiming that Continue will load them.
 module.exports = {
   name: 'continue',
   label: 'Continue.dev',
@@ -18,16 +18,34 @@ module.exports = {
   },
   getIgnorePatterns(workspaceRoot) {
     const patterns = ['.continue/harness-everything/'];
-    if (fs.existsSync(path.join(workspaceRoot, '.continue', 'skills'))) {
-      patterns.push('.continue/skills/');
+    const skillsDir = path.join(workspaceRoot, '.continue', 'skills');
+    if (fs.existsSync(skillsDir)) {
+      try {
+        const entries = fs.readdirSync(skillsDir, { withFileTypes: true });
+        for (const entry of entries) {
+          if (entry.isDirectory()) {
+            const skillMdPath = path.join(skillsDir, entry.name, 'SKILL.md');
+            if (fs.existsSync(skillMdPath)) {
+              const content = fs.readFileSync(skillMdPath, 'utf8');
+              const authorLine = content.split('\n').find(line => line.trim().startsWith('author:'));
+              if (authorLine && (authorLine.includes('Miya Daniel'))) {
+                patterns.push(`.continue/skills/${entry.name}/`);
+              }
+            }
+          }
+        }
+      } catch (e) {
+        // Fallback or ignore to prevent breaking execution
+      }
     }
     return patterns;
   },
   isMatch(pattern, trimmedLine) {
+    if (trimmedLine === '.continue/' || trimmedLine === '.continue') {
+      return true;
+    }
     if (pattern === '.continue/harness-everything/') {
-      return trimmedLine === '.continue/' ||
-             trimmedLine === '.continue' ||
-             trimmedLine === '.continue/harness-everything' ||
+      return trimmedLine === '.continue/harness-everything' ||
              trimmedLine === '.continue/harness-everything/';
     }
     return trimmedLine === pattern || trimmedLine === pattern.slice(0, -1);
@@ -40,11 +58,11 @@ module.exports = {
   },
   getSkillsTarget({ workspaceRoot, userHome, isGlobal, manifest }) {
     if (isGlobal) {
-      const globalAgentsDir = path.join(userHome, '.agents');
+      const continueDir = path.join(userHome, '.continue');
       return {
-        path: path.join(globalAgentsDir, 'skills'),
-        label: '~/.agents/skills/',
-        manifestPath: manifest.getManifestPath(globalAgentsDir),
+        path: path.join(continueDir, 'skills'),
+        label: '~/.continue/skills/',
+        manifestPath: manifest.getManifestPath(continueDir),
       };
     } else {
       const continueDir = path.join(workspaceRoot, '.continue');

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

@@ -7,35 +7,37 @@ const fs = require('fs');
 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');
-  },
+  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/');
+    const skillsDir = path.join(workspaceRoot, '.github', 'skills');
+    if (fs.existsSync(skillsDir)) {
+      try {
+        for (const entry of fs.readdirSync(skillsDir, { withFileTypes: true })) {
+          if (!entry.isDirectory()) continue;
+          const skillMdPath = path.join(skillsDir, entry.name, 'SKILL.md');
+          if (!fs.existsSync(skillMdPath)) continue;
+          const content = fs.readFileSync(skillMdPath, 'utf8');
+          const authorLine = content.split('\n').find(line => line.trim().startsWith('author:'));
+          if (authorLine && authorLine.includes('Miya Daniel')) patterns.push(`.github/skills/${entry.name}/`);
+        }
+      } catch (e) {
+        // Fallback or ignore to prevent breaking execution
+      }
     }
     return patterns;
   },
   isMatch(pattern, trimmedLine) {
+    if (trimmedLine === '.github/' || trimmedLine === '.github') return true;
     if (pattern === '.github/harness-everything/') {
-      return trimmedLine === '.github/' ||
-             trimmedLine === '.github' ||
-             trimmedLine === '.github/harness-everything' ||
-             trimmedLine === '.github/harness-everything/';
+      return 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
-    }
+    if (isGlobal) return false;
     return fs.existsSync(path.join(workspaceRoot, '.github', 'copilot-instructions.md'));
   },
   getSkillsTarget({ workspaceRoot, userHome, isGlobal, manifest }) {
@@ -46,46 +48,38 @@ module.exports = {
         label: '~/.agents/skills/',
         manifestPath: manifest.getManifestPath(globalAgentsDir),
       };
-    } else {
-      const githubDir = path.join(workspaceRoot, '.github');
-      return {
-        path: path.join(githubDir, 'skills'),
-        label: '.github/skills/',
-        manifestPath: manifest.getManifestPath(githubDir),
-      };
     }
+    const githubDir = path.join(workspaceRoot, '.github');
+    return {
+      path: path.join(githubDir, 'skills'),
+      label: '.github/skills/',
+      manifestPath: manifest.getManifestPath(githubDir),
+    };
   },
   install({ isGlobal, targetWorkspaceRoot, getUserPromptsDir, advisory }) {
     if (!isGlobal) {
       const targetFile = path.join(targetWorkspaceRoot, '.github', 'copilot-instructions.md');
       advisory.injectAdvisoryText(targetFile, '# Copilot Instructions', '.github/copilot-instructions.md');
-    } else {
-      try {
-        const promptsDir = getUserPromptsDir();
-        if (!fs.existsSync(promptsDir)) {
-          fs.mkdirSync(promptsDir, { recursive: true });
-        }
-        const vscodeInstFile = path.join(promptsDir, 'harness.instructions.md');
-        fs.writeFileSync(vscodeInstFile, advisory.buildCopilotGlobalContent(), 'utf8');
-        console.log(`  ✅ Installed global Copilot instructions to VS Code: ${vscodeInstFile}`);
-      } catch (err) {
-        console.warn(`  ⚠️ Failed to write VS Code user prompts folder: ${err.message}`);
-      }
+      return;
     }
+    const promptsDir = getUserPromptsDir();
+    if (!fs.existsSync(promptsDir)) fs.mkdirSync(promptsDir, { recursive: true });
+    const vscodeInstFile = path.join(promptsDir, 'harness.instructions.md');
+    advisory.assertHarnessOwnedOrAbsent(vscodeInstFile, 'global Copilot instructions');
+    fs.writeFileSync(vscodeInstFile, advisory.buildCopilotGlobalContent(), 'utf8');
+    console.log(`  ✅ Installed global Copilot instructions to VS Code: ${vscodeInstFile}`);
   },
   uninstall({ removeLocal, removeGlobal, workspaceRoot, userHome, getUserPromptsDir, cleanEmptyDirs }) {
     const advisory = require('../../../../scripts/lib/advisory-text');
-    if (removeLocal) {
-      advisory.removeAdvisoryText(path.join(workspaceRoot, '.github', 'copilot-instructions.md'));
-    }
+    if (removeLocal) advisory.removeAdvisoryText(path.join(workspaceRoot, '.github', 'copilot-instructions.md'));
     if (removeGlobal) {
       const promptsDir = getUserPromptsDir();
       const vscodeInstFile = path.join(promptsDir, 'harness.instructions.md');
-      if (fs.existsSync(vscodeInstFile)) {
+      if (fs.existsSync(vscodeInstFile) && advisory.hasHarnessMarker(fs.readFileSync(vscodeInstFile, 'utf8'))) {
         fs.unlinkSync(vscodeInstFile);
         console.log(`  ✅ Removed global Copilot instructions: ${vscodeInstFile}`);
       }
       cleanEmptyDirs(promptsDir, [userHome]);
     }
   }
-};
+};

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

@@ -18,16 +18,34 @@ module.exports = {
   },
   getIgnorePatterns(workspaceRoot) {
     const patterns = ['.cursor/harness-everything/'];
-    if (fs.existsSync(path.join(workspaceRoot, '.cursor', 'skills'))) {
-      patterns.push('.cursor/skills/');
+    const skillsDir = path.join(workspaceRoot, '.cursor', 'skills');
+    if (fs.existsSync(skillsDir)) {
+      try {
+        const entries = fs.readdirSync(skillsDir, { withFileTypes: true });
+        for (const entry of entries) {
+          if (entry.isDirectory()) {
+            const skillMdPath = path.join(skillsDir, entry.name, 'SKILL.md');
+            if (fs.existsSync(skillMdPath)) {
+              const content = fs.readFileSync(skillMdPath, 'utf8');
+              const authorLine = content.split('\n').find(line => line.trim().startsWith('author:'));
+              if (authorLine && (authorLine.includes('Miya Daniel'))) {
+                patterns.push(`.cursor/skills/${entry.name}/`);
+              }
+            }
+          }
+        }
+      } catch (e) {
+        // Fallback or ignore to prevent breaking execution
+      }
     }
     return patterns;
   },
   isMatch(pattern, trimmedLine) {
+    if (trimmedLine === '.cursor/' || trimmedLine === '.cursor') {
+      return true;
+    }
     if (pattern === '.cursor/harness-everything/') {
-      return trimmedLine === '.cursor/' ||
-             trimmedLine === '.cursor' ||
-             trimmedLine === '.cursor/harness-everything' ||
+      return trimmedLine === '.cursor/harness-everything' ||
              trimmedLine === '.cursor/harness-everything/';
     }
     return trimmedLine === pattern || trimmedLine === pattern.slice(0, -1);