set-prompt 0.8.0 → 0.8.1

package/CHANGELOG.md

@@ -4,6 +4,23 @@ All notable changes to this project will be documented in this file.
 
 ---
 
+## [0.8.1] - 2026-05-02
+
+### Changed
+- **Hermes auto-enable** — `link hermes` now writes `set-prompt` into the existing `~/.hermes/config.yaml` automatically, matching Claude Code / Codex behavior. Previously the file was left untouched and only a manual-merge snippet was printed, which left the plugin detected-but-disabled (`× set-prompt — not enabled in config`).
+  - Existing `plugins.enabled` list → `set-prompt` is appended (other entries preserved).
+  - `plugins` map exists but no `enabled` key → `enabled: [set-prompt]` is added.
+  - `plugins` key missing entirely → `plugins.enabled: [set-prompt]` is added.
+  - YAML comments and formatting are preserved (AST-based modification via `yaml` package).
+  - Atomic write with timestamped backup + rollback on failure (same pattern as `link claudecode` / `link codex`).
+- **`unlinkHermes` surgical removal** — for user-authored `config.yaml` (no `# Generated by set-prompt` header), only the `- set-prompt` entry is removed; the rest of the file is preserved. Auto-generated configs are still deleted whole.
+
+### Added
+- `yaml` (eemeli/yaml) dependency for AST-based YAML editing.
+- Tests for the new auto-enable branches: `plugins` key absent, `plugins.enabled` absent, comment preservation, and `unlinkHermes` surgical removal (17 cases total in `link-hermes.test.ts`).
+
+---
+
 ## [0.8.0] - 2026-05-02
 
 ### Added

package/README.md

@@ -123,7 +123,7 @@ The interactive mode shows all agents with their current state. **Check to link,
 
 > **Note on Gemini CLI**: Skills follow the standard `skills/<name>/SKILL.md` pattern. Commands use `.toml` format (not `.md`) and agents use `.md` with YAML frontmatter. Files in your repo's `commands/` must be TOML for Gemini CLI to recognize them. **Agents have strict frontmatter validation** — unknown keys (e.g. `allowed-tools`, `color`, `mode` from other platforms) cause Gemini CLI to reject the agent. See `SET_PROMPT_GUIDE.md` for the allowed keys.
 
-> **Note on Hermes**: Hermes plugins must register skills/commands/hooks programmatically — directory drop-ins are not auto-discovered. set-prompt generates a small Python adapter (`~/.hermes/plugins/set-prompt/__init__.py`) with the repo's absolute path baked in, so no symlinks are needed. **Restart Hermes after `link` (or after adding/removing a skill in your repo)** — `register()` runs only once at Hermes startup. Activation requires `set-prompt` listed under `plugins.enabled` in `~/.hermes/config.yaml` — set-prompt creates this file if absent but never modifies an existing one (it prints the snippet to add manually). Hooks are observation-only on the Hermes side: the same `hooks/hooks.json` is reused, but Hermes events (`pre_tool_call`, `on_session_start`, etc.) are picked up while Claude/Cursor keys are ignored. Hook scripts can reference `${SET_PROMPT_REPO}`.
+> **Note on Hermes**: Hermes plugins must register skills/commands/hooks programmatically — directory drop-ins are not auto-discovered. set-prompt generates a small Python adapter (`~/.hermes/plugins/set-prompt/__init__.py`) with the repo's absolute path baked in, so no symlinks are needed. **Restart Hermes after `link` (or after adding/removing a skill in your repo)** — `register()` runs only once at Hermes startup. Activation requires `set-prompt` listed under `plugins.enabled` in `~/.hermes/config.yaml` — set-prompt writes this entry automatically (creating the file when absent, or appending to the existing list while preserving other entries and comments via AST-based YAML editing with backup/rollback). Hooks are observation-only on the Hermes side: the same `hooks/hooks.json` is reused, but Hermes events (`pre_tool_call`, `on_session_start`, etc.) are picked up while Claude/Cursor keys are ignored. Hook scripts can reference `${SET_PROMPT_REPO}`.
 
 ---
 
@@ -255,7 +255,7 @@ set-prompt uninstall
 - **Cursor** — replaces directories and `mcp.json` in `~/.cursor/`
 - **OpenCode** — replaces directories in `~/.config/opencode/`
 - **Gemini CLI** — replaces directories in `~/.gemini/` (`antigravity/` subtree untouched)
-- **Hermes** — generates `~/.hermes/plugins/set-prompt/{plugin.yaml, __init__.py}` and writes `~/.hermes/config.yaml` only when absent (existing files are never auto-modified)
+- **Hermes** — generates `~/.hermes/plugins/set-prompt/{plugin.yaml, __init__.py}` and adds `set-prompt` to `~/.hermes/config.yaml` under `plugins.enabled` (creates the file when absent; appends to the existing list while preserving other entries and comments)
 
 Before making any changes, `set-prompt` creates a backup and rolls back automatically on failure. However, you should be aware that:
 

package/dist/index.js

@@ -2490,6 +2490,7 @@ import path12 from "path";
 import fs13 from "fs";
 import chalk14 from "chalk";
 import { confirm as confirm11 } from "@inquirer/prompts";
+import YAML from "yaml";
 var PLUGIN_YAML = `name: ${MARKET_NAME}
 version: "1.0"
 description: Managed by set-prompt
@@ -2630,13 +2631,43 @@ plugins:
   enabled:
     - ${MARKET_NAME}
 `;
-var ENABLE_SNIPPET = `plugins:
-  enabled:
-    - ${MARKET_NAME}`;
 var writePluginManifest = (repoPath) => {
   fs13.writeFileSync(path12.join(HERMES_PLUGIN_DIR, "plugin.yaml"), PLUGIN_YAML, "utf-8");
   fs13.writeFileSync(path12.join(HERMES_PLUGIN_DIR, "__init__.py"), buildInitPy(repoPath), "utf-8");
 };
+var writeWithBackup = (filePath, content) => {
+  let backupPath = null;
+  if (fs13.existsSync(filePath)) {
+    const timestamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
+    backupPath = `${filePath}.bak.${timestamp}`;
+    try {
+      fs13.copyFileSync(filePath, backupPath);
+    } catch {
+      backupPath = null;
+    }
+  }
+  try {
+    fs13.mkdirSync(path12.dirname(filePath), { recursive: true });
+    fs13.writeFileSync(filePath, content, "utf-8");
+  } catch (ex) {
+    if (backupPath !== null) {
+      try {
+        fs13.copyFileSync(backupPath, filePath);
+        fs13.unlinkSync(backupPath);
+        console.warn(chalk14.yellow("  \u26A0 Write failed \u2014 rolled back to original."));
+      } catch {
+        console.error(chalk14.red(`  \u274C Rollback failed. Backup preserved at: ${backupPath}`));
+      }
+    }
+    throw ex;
+  }
+  if (backupPath !== null) {
+    try {
+      fs13.unlinkSync(backupPath);
+    } catch {
+    }
+  }
+};
 var ensureHermesConfigEnabled = () => {
   if (fs13.existsSync(HERMES_CONFIG_PATH) === false) {
     fs13.mkdirSync(path12.dirname(HERMES_CONFIG_PATH), { recursive: true });
@@ -2645,17 +2676,54 @@ var ensureHermesConfigEnabled = () => {
     return;
   }
   const content = fs13.readFileSync(HERMES_CONFIG_PATH, "utf-8");
-  const enabledPattern = new RegExp(`^\\s*-\\s+${MARKET_NAME}\\s*$`, "m");
-  if (enabledPattern.test(content)) {
+  let doc;
+  try {
+    doc = YAML.parseDocument(content);
+  } catch (ex) {
+    console.warn(chalk14.yellow(`  \u26A0 Failed to parse Hermes config (${ex.message}) \u2014 leaving file untouched.`));
+    console.log(chalk14.dim(`    File: ${HERMES_CONFIG_PATH}`));
+    return;
+  }
+  if (doc.errors.length > 0) {
+    console.warn(chalk14.yellow(`  \u26A0 Hermes config has YAML errors \u2014 leaving file untouched.`));
+    console.log(chalk14.dim(`    File: ${HERMES_CONFIG_PATH}`));
+    return;
+  }
+  let plugins = doc.get("plugins");
+  if (plugins == null) {
+    doc.set("plugins", doc.createNode({ enabled: [MARKET_NAME] }));
+    writeWithBackup(HERMES_CONFIG_PATH, String(doc));
+    console.log(chalk14.green("  \u2713 ") + chalk14.dim(`added plugins.enabled to ${HERMES_CONFIG_PATH}`));
+    return;
+  }
+  if (!YAML.isMap(plugins)) {
+    console.warn(chalk14.yellow(`  \u26A0 "plugins" is not a map \u2014 leaving file untouched.`));
+    console.log(chalk14.dim(`    File: ${HERMES_CONFIG_PATH}`));
+    return;
+  }
+  let enabled = plugins.get("enabled");
+  if (enabled == null) {
+    plugins.set("enabled", doc.createNode([MARKET_NAME]));
+    writeWithBackup(HERMES_CONFIG_PATH, String(doc));
+    console.log(chalk14.green("  \u2713 ") + chalk14.dim(`added plugins.enabled to ${HERMES_CONFIG_PATH}`));
+    return;
+  }
+  if (!YAML.isSeq(enabled)) {
+    console.warn(chalk14.yellow(`  \u26A0 "plugins.enabled" is not a list \u2014 leaving file untouched.`));
+    console.log(chalk14.dim(`    File: ${HERMES_CONFIG_PATH}`));
+    return;
+  }
+  const alreadyListed = enabled.items.some((item) => {
+    const v = YAML.isScalar(item) ? item.value : item;
+    return v === MARKET_NAME;
+  });
+  if (alreadyListed) {
     console.log(chalk14.dim(`  \u2713 ${MARKET_NAME} already listed in ${HERMES_CONFIG_PATH}`));
     return;
   }
-  console.log(chalk14.yellow(`
-  \u26A0 Hermes config exists but does not list "${MARKET_NAME}" as enabled.`));
-  console.log(chalk14.dim("    Add the following entry under plugins.enabled to activate this plugin:\n"));
-  console.log(chalk14.cyan(ENABLE_SNIPPET.split("\n").map((l) => `      ${l}`).join("\n")));
-  console.log(chalk14.dim(`
-    File: ${HERMES_CONFIG_PATH}`));
+  enabled.add(MARKET_NAME);
+  writeWithBackup(HERMES_CONFIG_PATH, String(doc));
+  console.log(chalk14.green("  \u2713 ") + chalk14.dim(`enabled "${MARKET_NAME}" in ${HERMES_CONFIG_PATH}`));
 };
 var linkHermes = async () => {
   const repoPath = resolveRepoPath();
@@ -2702,14 +2770,44 @@ Removing Hermes plugin...`));
     if (content.trimStart().startsWith("# Generated by set-prompt")) {
       fs13.unlinkSync(HERMES_CONFIG_PATH);
       console.log(chalk14.red("  removed") + chalk14.dim(`: ${HERMES_CONFIG_PATH}`));
-    } else if (new RegExp(`^\\s*-\\s+${MARKET_NAME}\\s*$`, "m").test(content)) {
-      console.log(chalk14.yellow(`  \u26A0 Hermes config still lists "${MARKET_NAME}" \u2014 remove it manually:`));
-      console.log(chalk14.dim(`    ${HERMES_CONFIG_PATH}`));
+    } else {
+      removeMarketEntryFromConfig();
     }
   }
   configManager.hermes = null;
   configManager.save();
 };
+var removeMarketEntryFromConfig = () => {
+  const content = fs13.readFileSync(HERMES_CONFIG_PATH, "utf-8");
+  let doc;
+  try {
+    doc = YAML.parseDocument(content);
+  } catch (ex) {
+    console.warn(chalk14.yellow(`  \u26A0 Failed to parse Hermes config (${ex.message}) \u2014 leaving file untouched.`));
+    return;
+  }
+  if (doc.errors.length > 0) {
+    return;
+  }
+  const plugins = doc.get("plugins");
+  if (!YAML.isMap(plugins)) {
+    return;
+  }
+  const enabled = plugins.get("enabled");
+  if (!YAML.isSeq(enabled)) {
+    return;
+  }
+  const idx = enabled.items.findIndex((item) => {
+    const v = YAML.isScalar(item) ? item.value : item;
+    return v === MARKET_NAME;
+  });
+  if (idx === -1) {
+    return;
+  }
+  enabled.delete(idx);
+  writeWithBackup(HERMES_CONFIG_PATH, String(doc));
+  console.log(chalk14.red("  removed") + chalk14.dim(` "${MARKET_NAME}" from: ${HERMES_CONFIG_PATH}`));
+};
 
 // src/commands/link-command.ts
 var LINK_MAP = {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "set-prompt",
-    "version": "0.8.0",
+    "version": "0.8.1",
     "description": "One repo. Every AI coding tool. Always in sync.",
     "main": "dist/index.js",
     "bin": {
@@ -52,6 +52,7 @@
         "ora": "^9.3.0",
         "smol-toml": "^1.6.0",
         "typia": "^11.0.3",
+        "yaml": "^2.8.4",
         "zod": "^4.3.6"
     },
     "devDependencies": {