harness-everything 0.2.1 → 0.2.2

package/scripts/lib/claude-hooks.js

@@ -1,71 +1,71 @@
-// Merge/remove Harness's hook entries in a Claude Code settings.json without
-// disturbing anything else a user (or another tool) put there.
-const fs = require('fs');
-
-// Every hook this package ships is namespaced "harness:<phase>:<name>" (see
-// hooks/hooks.json) specifically so removal can match on that prefix alone -
-// never on loose text like "does the command mention harness", which could
-// false-positive on someone else's unrelated hook (e.g. a script whose path
-// or name happens to contain "harness").
-function isHarnessHook(hook) {
-  return !!(hook && hook.id && hook.id.startsWith('harness:'));
-}
-
-function mergeHarnessHooks(claudeConfig, resolvedHooks) {
-  claudeConfig.hooks = claudeConfig.hooks || {};
-  for (const [hookType, hookList] of Object.entries(resolvedHooks)) {
-    const existingList = claudeConfig.hooks[hookType] || [];
-    const mergedList = [...existingList];
-    hookList.forEach(newHook => {
-      const existsIdx = mergedList.findIndex(h => h.id === newHook.id);
-      if (existsIdx !== -1) {
-        mergedList[existsIdx] = newHook;
-      } else {
-        mergedList.push(newHook);
-      }
-    });
-    claudeConfig.hooks[hookType] = mergedList;
-  }
-  return claudeConfig;
-}
-
-function removeHarnessHooks(claudeSettingsFile) {
-  if (!fs.existsSync(claudeSettingsFile)) return false;
-  try {
-    const content = fs.readFileSync(claudeSettingsFile, 'utf8');
-    let config = JSON.parse(content);
-    if (!config.hooks) return false;
-
-    let modified = false;
-    for (const [hookType, hookList] of Object.entries(config.hooks)) {
-      if (!Array.isArray(hookList)) continue;
-      const originalLength = hookList.length;
-      config.hooks[hookType] = hookList.filter(hook => !isHarnessHook(hook));
-      if (config.hooks[hookType].length !== originalLength) {
-        modified = true;
-      }
-      if (config.hooks[hookType].length === 0) {
-        delete config.hooks[hookType];
-      }
-    }
-    if (Object.keys(config.hooks || {}).length === 0) {
-      delete config.hooks;
-    }
-
-    if (modified) {
-      if (Object.keys(config).length === 0) {
-        fs.unlinkSync(claudeSettingsFile);
-        console.log(`  ✅ Cleaned up and removed empty Claude settings file: ${claudeSettingsFile}`);
-      } else {
-        fs.writeFileSync(claudeSettingsFile, JSON.stringify(config, null, 2), 'utf8');
-        console.log(`  ✅ Safely removed Harness hooks from Claude settings: ${claudeSettingsFile}`);
-      }
-      return true;
-    }
-  } catch (e) {
-    console.warn(`  ⚠️ Error cleaning Claude settings file ${claudeSettingsFile}: ${e.message}`);
-  }
-  return false;
-}
-
-module.exports = { isHarnessHook, mergeHarnessHooks, removeHarnessHooks };
+// Merge/remove Harness's hook entries in a Claude Code settings.json without
+// disturbing anything else a user (or another tool) put there.
+const fs = require('fs');
+
+// Every hook this package ships is namespaced "harness:<phase>:<name>" (see
+// hooks/hooks.json) specifically so removal can match on that prefix alone -
+// never on loose text like "does the command mention harness", which could
+// false-positive on someone else's unrelated hook (e.g. a script whose path
+// or name happens to contain "harness").
+function isHarnessHook(hook) {
+  return !!(hook && hook.id && hook.id.startsWith('harness:'));
+}
+
+function mergeHarnessHooks(claudeConfig, resolvedHooks) {
+  claudeConfig.hooks = claudeConfig.hooks || {};
+  for (const [hookType, hookList] of Object.entries(resolvedHooks)) {
+    const existingList = claudeConfig.hooks[hookType] || [];
+    const mergedList = [...existingList];
+    hookList.forEach(newHook => {
+      const existsIdx = mergedList.findIndex(h => h.id === newHook.id);
+      if (existsIdx !== -1) {
+        mergedList[existsIdx] = newHook;
+      } else {
+        mergedList.push(newHook);
+      }
+    });
+    claudeConfig.hooks[hookType] = mergedList;
+  }
+  return claudeConfig;
+}
+
+function removeHarnessHooks(claudeSettingsFile) {
+  if (!fs.existsSync(claudeSettingsFile)) return false;
+  try {
+    const content = fs.readFileSync(claudeSettingsFile, 'utf8');
+    let config = JSON.parse(content);
+    if (!config.hooks) return false;
+
+    let modified = false;
+    for (const [hookType, hookList] of Object.entries(config.hooks)) {
+      if (!Array.isArray(hookList)) continue;
+      const originalLength = hookList.length;
+      config.hooks[hookType] = hookList.filter(hook => !isHarnessHook(hook));
+      if (config.hooks[hookType].length !== originalLength) {
+        modified = true;
+      }
+      if (config.hooks[hookType].length === 0) {
+        delete config.hooks[hookType];
+      }
+    }
+    if (Object.keys(config.hooks || {}).length === 0) {
+      delete config.hooks;
+    }
+
+    if (modified) {
+      if (Object.keys(config).length === 0) {
+        fs.unlinkSync(claudeSettingsFile);
+        console.log(`  ✅ Cleaned up and removed empty Claude settings file: ${claudeSettingsFile}`);
+      } else {
+        fs.writeFileSync(claudeSettingsFile, JSON.stringify(config, null, 2), 'utf8');
+        console.log(`  ✅ Safely removed Harness hooks from Claude settings: ${claudeSettingsFile}`);
+      }
+      return true;
+    }
+  } catch (e) {
+    console.warn(`  ⚠️ Error cleaning Claude settings file ${claudeSettingsFile}: ${e.message}`);
+  }
+  return false;
+}
+
+module.exports = { isHarnessHook, mergeHarnessHooks, removeHarnessHooks };

package/scripts/lib/manifest.js

@@ -1,82 +1,82 @@
-// Tracks exactly what this package installed, so uninstall can remove
-// precisely those paths instead of sweeping shared directories that may also
-// hold content from other tools, the user's own manual additions, or
-// self-evolve's locally-generated skills.
-//
-// The manifest always lives at <platform-home>/harness-everything/manifest.json
-// - e.g. .claude/harness-everything/manifest.json, .cursor/harness-everything/
-// manifest.json, or (global scope) ~/.agents/harness-everything/manifest.json.
-// harness-everything/ is a subfolder this package exclusively owns inside a
-// directory that already belongs to that platform (or, for global scope, the
-// shared ~/.agents convention) - nothing else creates a directory literally
-// named "harness-everything" there, so nesting under it is what makes "the
-// manifest can only describe our own installs" actually true.
-const fs = require('fs');
-const path = require('path');
-
-const PACKAGE_NAME = 'harness-everything';
-const HARNESS_AUTHOR = 'Miya Daniel | Harness Core Team';
-const HARNESS_DIR_NAME = 'harness-everything';
-
-function getHarnessDir(homeDir) {
-  return path.join(homeDir, HARNESS_DIR_NAME);
-}
-
-function getManifestPath(homeDir) {
-  return path.join(getHarnessDir(homeDir), 'manifest.json');
-}
-
-function readManifest(manifestPath) {
-  if (!fs.existsSync(manifestPath)) {
-    return { package: PACKAGE_NAME, skills: [] };
-  }
-  try {
-    const data = JSON.parse(fs.readFileSync(manifestPath, 'utf8'));
-    if (!Array.isArray(data.skills)) data.skills = [];
-    return data;
-  } catch (e) {
-    return { package: PACKAGE_NAME, skills: [] };
-  }
-}
-
-function writeManifest(manifestPath, data) {
-  fs.mkdirSync(path.dirname(manifestPath), { recursive: true });
-  fs.writeFileSync(manifestPath, JSON.stringify(data, null, 2), 'utf8');
-}
-
-function recordSkillInstall(manifestPath, packageVersion, skillId, dirPath) {
-  const data = readManifest(manifestPath);
-  data.package = PACKAGE_NAME;
-  data.version = packageVersion;
-  data.updatedAt = new Date().toISOString();
-  const idx = data.skills.findIndex(s => s.dirPath === dirPath);
-  const entry = { id: skillId, dirPath, installedAt: new Date().toISOString() };
-  if (idx !== -1) data.skills[idx] = entry;
-  else data.skills.push(entry);
-  writeManifest(manifestPath, data);
-}
-
-// Removes one skill entry, and - once no skills remain in it - deletes the
-// manifest file itself rather than leaving an empty bookkeeping file behind.
-function removeSkillFromManifest(manifestPath, dirPath) {
-  if (!fs.existsSync(manifestPath)) return;
-  const data = readManifest(manifestPath);
-  data.skills = data.skills.filter(s => s.dirPath !== dirPath);
-  if (data.skills.length === 0) {
-    fs.unlinkSync(manifestPath);
-  } else {
-    writeManifest(manifestPath, data);
-  }
-}
-
-module.exports = {
-  PACKAGE_NAME,
-  HARNESS_AUTHOR,
-  HARNESS_DIR_NAME,
-  getHarnessDir,
-  getManifestPath,
-  readManifest,
-  writeManifest,
-  recordSkillInstall,
-  removeSkillFromManifest,
-};
+// Tracks exactly what this package installed, so uninstall can remove
+// precisely those paths instead of sweeping shared directories that may also
+// hold content from other tools, the user's own manual additions, or
+// self-evolve's locally-generated skills.
+//
+// The manifest always lives at <platform-home>/harness-everything/manifest.json
+// - e.g. .claude/harness-everything/manifest.json, .cursor/harness-everything/
+// manifest.json, or (global scope) ~/.agents/harness-everything/manifest.json.
+// harness-everything/ is a subfolder this package exclusively owns inside a
+// directory that already belongs to that platform (or, for global scope, the
+// shared ~/.agents convention) - nothing else creates a directory literally
+// named "harness-everything" there, so nesting under it is what makes "the
+// manifest can only describe our own installs" actually true.
+const fs = require('fs');
+const path = require('path');
+
+const PACKAGE_NAME = 'harness-everything';
+const HARNESS_AUTHOR = 'Miya Daniel | Harness Core Team';
+const HARNESS_DIR_NAME = 'harness-everything';
+
+function getHarnessDir(homeDir) {
+  return path.join(homeDir, HARNESS_DIR_NAME);
+}
+
+function getManifestPath(homeDir) {
+  return path.join(getHarnessDir(homeDir), 'manifest.json');
+}
+
+function readManifest(manifestPath) {
+  if (!fs.existsSync(manifestPath)) {
+    return { package: PACKAGE_NAME, skills: [] };
+  }
+  try {
+    const data = JSON.parse(fs.readFileSync(manifestPath, 'utf8'));
+    if (!Array.isArray(data.skills)) data.skills = [];
+    return data;
+  } catch (e) {
+    return { package: PACKAGE_NAME, skills: [] };
+  }
+}
+
+function writeManifest(manifestPath, data) {
+  fs.mkdirSync(path.dirname(manifestPath), { recursive: true });
+  fs.writeFileSync(manifestPath, JSON.stringify(data, null, 2), 'utf8');
+}
+
+function recordSkillInstall(manifestPath, packageVersion, skillId, dirPath) {
+  const data = readManifest(manifestPath);
+  data.package = PACKAGE_NAME;
+  data.version = packageVersion;
+  data.updatedAt = new Date().toISOString();
+  const idx = data.skills.findIndex(s => s.dirPath === dirPath);
+  const entry = { id: skillId, dirPath, installedAt: new Date().toISOString() };
+  if (idx !== -1) data.skills[idx] = entry;
+  else data.skills.push(entry);
+  writeManifest(manifestPath, data);
+}
+
+// Removes one skill entry, and - once no skills remain in it - deletes the
+// manifest file itself rather than leaving an empty bookkeeping file behind.
+function removeSkillFromManifest(manifestPath, dirPath) {
+  if (!fs.existsSync(manifestPath)) return;
+  const data = readManifest(manifestPath);
+  data.skills = data.skills.filter(s => s.dirPath !== dirPath);
+  if (data.skills.length === 0) {
+    fs.unlinkSync(manifestPath);
+  } else {
+    writeManifest(manifestPath, data);
+  }
+}
+
+module.exports = {
+  PACKAGE_NAME,
+  HARNESS_AUTHOR,
+  HARNESS_DIR_NAME,
+  getHarnessDir,
+  getManifestPath,
+  readManifest,
+  writeManifest,
+  recordSkillInstall,
+  removeSkillFromManifest,
+};