set-prompt 0.8.0 → 0.8.2

package/CHANGELOG.md

@@ -4,6 +4,40 @@ All notable changes to this project will be documented in this file.
 
 ---
 
+## [0.8.2] - 2026-05-02
+
+### Fixed
+- **Hermes `register_skill` argument types** — fixed `'str' object has no attribute 'exists'` error that surfaced after enabling the plugin in Hermes. The generated `__init__.py` was passing `str(skill_path)` (the skill directory as a string), but Hermes's `ctx.register_skill(name, path)` expects:
+  1. A `pathlib.Path` object (not a string)
+  2. The path to `SKILL.md` itself (not the parent directory)
+  Verified against the official "Build a Hermes plugin" guide example. Now passes `skill_path / "SKILL.md"` as a `Path`.
+- **Hermes command handler signature** — handler is now defined as `handler(raw_args: str = "") -> Optional[str]` matching the documented Hermes signature (previously `handler(_args=None)`).
+- **Slash commands silent in gateway/TUI mode** — `ctx.inject_message()` is documented as **CLI-mode only** (returns `False` in gateway mode). The handler now falls back to **returning the markdown body** as the command's response when `inject_message` is unavailable, so the content is still visible to the user instead of being silently dropped.
+- **`register_command` now uses keyword arguments** (`handler=`, `description=`) to match the official Hermes plugin examples (e.g. `disk-cleanup`) and stay robust if Hermes reorders positional parameters.
+
+### Notes
+- After upgrading, re-run `set-prompt link hermes` to regenerate `__init__.py`, then restart Hermes. The plugin should now load with `✓ set-prompt v1.0` and skills become loadable via `skill_view("set-prompt:<name>")` (plugin skills are opt-in explicit loads — they do not appear in Hermes's auto-listed `available_skills` index).
+- Hook callbacks remain `def runner(*args, **kwargs)` — Hermes documents that all hook callbacks should accept `**kwargs` for forward compatibility, and the bridge's role is purely to forward the payload to the user-defined hook command via stdin JSON.
+
+---
+
+## [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). **Plugin skills are opt-in explicit loads** — they do not appear in Hermes's auto-listed `available_skills` index; load them via `skill_view("set-prompt:<name>")`. 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

@@ -549,7 +549,7 @@ required_environment_variables:
 
 > **Gemini CLI note**: Only \`name\` and \`description\` are recognized. \`name\` must be lowercase with hyphens and match the directory name.
 
-> **Hermes note (set-prompt integration)**: Hermes does not auto-discover files in standard directories \u2014 plugins must register skills programmatically. set-prompt generates \`~/.hermes/plugins/set-prompt/{plugin.yaml, __init__.py}\` on \`set-prompt link hermes\`. The \`__init__.py\` reads \`<repo>/skills/<skill-name>/SKILL.md\` directly (REPO_DIR is baked in at link time) and calls \`ctx.register_skill()\` at Hermes startup. The skill directory layout is the same as other platforms \u2014 no nested category folder.
+> **Hermes note (set-prompt integration)**: Hermes does not auto-discover files in standard directories \u2014 plugins must register skills programmatically. set-prompt generates \`~/.hermes/plugins/set-prompt/{plugin.yaml, __init__.py}\` on \`set-prompt link hermes\`. The \`__init__.py\` reads \`<repo>/skills/<skill-name>/SKILL.md\` directly (REPO_DIR is baked in at link time) and calls \`ctx.register_skill(name, Path(SKILL.md))\` at Hermes startup. The skill directory layout is the same as other platforms \u2014 no nested category folder. **Plugin skills are opt-in explicit loads** \u2014 they do not appear in Hermes's auto-listed \`available_skills\` index; load via \`skill_view("set-prompt:<skill-name>")\`.
 
 | Field | Required | Platform | Description |
 |-------|----------|----------|-------------|
@@ -659,7 +659,7 @@ Include: 1) Refactored code. 2) Explanation of changes.
 
 #### Hermes commands (set-prompt integration)
 
-Hermes commands are registered programmatically. set-prompt's generated \`__init__.py\` walks \`<repo>/commands/*.md\`, parses each file's YAML frontmatter for \`name\` / \`description\`, and calls \`ctx.register_command(name, handler, description)\`. The handler injects the markdown body as a user message via \`ctx.inject_message(body, role="user")\` when the user invokes the slash command. Hermes does **not** read \`allowed-tools\`, \`model\`, \`agent\`, or other platform-specific frontmatter \u2014 only \`name\` and \`description\` are honored.
+Hermes commands are registered programmatically. set-prompt's generated \`__init__.py\` walks \`<repo>/commands/*.md\`, parses each file's YAML frontmatter for \`name\` / \`description\`, and calls \`ctx.register_command(name, handler=..., description=...)\`. The handler tries to inject the markdown body as a user message via \`ctx.inject_message(body, role="user")\` when the slash command is invoked. **\`inject_message\` is CLI-mode only** \u2014 in gateway/TUI mode it returns \`False\`, and the handler falls back to **returning the body** as the command's response so the content is still visible. Hermes does **not** read \`allowed-tools\`, \`model\`, \`agent\`, or other platform-specific frontmatter \u2014 only \`name\` and \`description\` are honored.
 
 | Field | Required | Platform | Description |
 |-------|----------|----------|-------------|
@@ -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
@@ -2536,8 +2537,18 @@ def _parse_frontmatter(text):
 
 
 def _make_command_handler(ctx, body):
-    def handler(_args=None):
-        ctx.inject_message(body, role="user")
+    # Hermes calls handlers as handler(raw_args: str) -> Optional[str].
+    # Primary path: inject the markdown body as a user message (CLI mode).
+    # Fallback: in gateway mode inject_message returns False \u2014 return the body
+    # as the handler response so the content is at least visible to the user.
+    def handler(raw_args=""):
+        try:
+            ok = ctx.inject_message(body, role="user")
+        except Exception:
+            ok = False
+        if not ok:
+            return body
+        return None
     return handler
 
 
@@ -2606,9 +2617,11 @@ def register(ctx):
         for skill_path in sorted(skills_dir.iterdir()):
             if not skill_path.is_dir():
                 continue
-            if not (skill_path / "SKILL.md").exists():
+            skill_md = skill_path / "SKILL.md"
+            if not skill_md.exists():
                 continue
-            ctx.register_skill(skill_path.name, str(skill_path))
+            # Hermes expects (name, Path-to-SKILL.md), not (name, dir-as-str).
+            ctx.register_skill(skill_path.name, skill_md)
 
     commands_dir = REPO_DIR / "commands"
     if commands_dir.exists():
@@ -2620,7 +2633,11 @@ def register(ctx):
             meta, body = _parse_frontmatter(text)
             name = meta.get("name") or cmd_file.stem
             description = meta.get("description") or ""
-            ctx.register_command(name, _make_command_handler(ctx, body), description)
+            ctx.register_command(
+                name,
+                handler=_make_command_handler(ctx, body),
+                description=description,
+            )
 
     _register_hooks(ctx)
 `;
@@ -2630,13 +2647,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 +2692,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 +2786,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.2",
     "description": "One repo. Every AI coding tool. Always in sync.",
     "main": "dist/index.js",
     "bin": {
@@ -31,7 +31,8 @@
         "cursor",
         "codex",
         "opencode",
-        "gemini-cli"
+        "gemini-cli",
+        "hermes"
     ],
     "author": "juncha9 (https://github.com/juncha9)",
     "license": "MIT",
@@ -52,6 +53,7 @@
         "ora": "^9.3.0",
         "smol-toml": "^1.6.0",
         "typia": "^11.0.3",
+        "yaml": "^2.8.4",
         "zod": "^4.3.6"
     },
     "devDependencies": {