@chatpanel/bridge 0.10.35 → 0.10.37

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chatpanel/bridge",
-  "version": "0.10.35",
+  "version": "0.10.37",
   "type": "module",
   "description": "Local bridge that exposes the AI coding agents installed on your machine \u2014 Claude Code (CLI), Codex (CLI), and Antigravity CLI (formerly Gemini CLI, which remains available for business/enterprise) \u2014 to the ChatPanel Chrome extension over a localhost SSE endpoint. Bring your own agent.",
   "keywords": [

package/src/engines/cli-agents.js

@@ -8,8 +8,8 @@
 //   kiro     — kiro-cli chat --no-interactive "<prompt>" · --model · --list-models
 //   hermes   — hermes -z "<prompt>"      · -m model · config get model (no list command)
 
-import { runSpec, listSpecModels, runForStdout } from './custom.js';
-import { findAgentBin } from '../env.js';
+import { runSpec, listSpecModels, runForStdout, CHATPANEL_STABLE_MCP_URL } from './custom.js';
+import { findAgentBin, selfMcpStdio } from '../env.js';
 
 // `listModels` override hook: a CLI whose model listing isn't "one id per line"
 // (Copilot prints a quoted list inside `help config`) passes its own parser here
@@ -264,14 +264,31 @@ export async function listDshModels(command = 'dsh', workingDir) {
 export const hermes = makeCliAgent(
   'hermes',
   {
-    args: '-z',
+    // `-z {prompt}`, NOT a bare `-z`: Hermes takes the prompt as the VALUE of -z (Python
+    // argparse), and the runner appends a trailing prompt only when no {prompt} placeholder
+    // exists — after the model flag. That produced `hermes -z -m <model> "<prompt>"`, where
+    // argparse saw -z followed by another flag: "expected one argument". The placeholder
+    // binds the prompt to the flag, so the model flag can sit anywhere after it.
+    args: '-z {prompt}',
     promptVia: 'arg',
     modelArg: '-m {model}',
     requiresStableMcp: true,
     autoSetupStableMcp: true,
     stableMcpConfigCheck: 'hermes',
-    stableMcpSetupArgs: ['mcp', 'add', 'chatpanel_browser', '--url', 'http://127.0.0.1:4319/mcp'],
-    stableMcpSetupCommand: 'hermes mcp add chatpanel_browser --url http://127.0.0.1:4319/mcp',
+    // STDIO, not --url. Hermes's bundled MCP client has no HTTP transport ("requires HTTP
+    // transport but mcp.client.streamable_http is not available"), so a URL registration
+    // saves a config that can never connect. The bridge also speaks MCP over stdio — the
+    // same mode Codex uses — so we register that instead. A function because the executable
+    // is only known at runtime (a compiled binary runs itself; under node it re-runs its
+    // entry script). `--args` must come last, per `hermes mcp add`.
+    stableMcpSetupArgs: () => {
+      const { command, args } = selfMcpStdio(CHATPANEL_STABLE_MCP_URL);
+      return ['mcp', 'add', 'chatpanel_browser', '--command', command, '--args', ...args];
+    },
+    stableMcpSetupCommand: (() => {
+      const { command, args } = selfMcpStdio(CHATPANEL_STABLE_MCP_URL);
+      return `hermes mcp add chatpanel_browser --command ${command} --args ${args.join(' ')}`;
+    })(),
     label: 'Hermes',
   },
   'hermes not found on PATH. Install Hermes Agent, then run `hermes setup` to sign in.',

package/src/engines/custom.js

@@ -64,7 +64,7 @@ const IDLE_MS = Number(process.env.CHATPANEL_CUSTOM_TIMEOUT_MS) || 180_000;
 // ANSI stripping moved to stream-formats.js (imported above), which is where
 // text output is actually rendered - one implementation for every format.
 const OPENCODE_STABLE_MCP_URL = 'http://127.0.0.1:4319/mcp';
-const CHATPANEL_STABLE_MCP_URL = 'http://127.0.0.1:4319/mcp';
+export const CHATPANEL_STABLE_MCP_URL = 'http://127.0.0.1:4319/mcp';
 
 export async function available() {
   // The engine ships in every bridge; individual custom agents are user-defined
@@ -167,7 +167,10 @@ export function stableMcpSetupCommand(spec = {}) {
 }
 
 export function stableMcpSetupPlan(spec = {}) {
-  const args = Array.isArray(spec.stableMcpSetupArgs) ? spec.stableMcpSetupArgs.filter((a) => a != null).map(String) : null;
+  // A FUNCTION is allowed as well as an array: a stdio registration has to name the bridge's
+  // own executable, which is only known at runtime (compiled binary vs. node + entry script).
+  const raw = typeof spec.stableMcpSetupArgs === 'function' ? spec.stableMcpSetupArgs() : spec.stableMcpSetupArgs;
+  const args = Array.isArray(raw) ? raw.filter((a) => a != null).map(String) : null;
   if (!args?.length) return null;
   return { command: spec.stableMcpSetupCommandName || spec.command, args };
 }

package/src/server.js

@@ -67,7 +67,7 @@ import {
 // Hardcoded (not read from package.json) so it survives Bun's single-file
 // --compile, where package.json isn't on a readable FS. CI fails the publish if
 // this drifts from package.json, so the two can't silently diverge.
-const VERSION = '0.10.35';
+const VERSION = '0.10.37';
 const HOST = process.env.CHATPANEL_BRIDGE_HOST || '127.0.0.1';
 const PORT = Number(process.env.CHATPANEL_BRIDGE_PORT) || 4319;