mio-agent-runtime 0.13.4 → 0.13.5

package/adapters/hermes.js

@@ -1,176 +1,182 @@
-'use strict'
-
-const fs = require('fs')
-const path = require('path')
-const os = require('os')
-const { spawnSync } = require('child_process')
-
-const SERVER_NAME = 'mio-intelligence'
-
-function hermesHome() {
-  return process.env.HERMES_HOME || path.join(os.homedir(), 'AppData', 'Local', 'hermes')
-}
-
-function hermesConfigPath() {
-  return path.join(hermesHome(), 'config.yaml')
-}
-
-function configNeedsRepair() {
-  const cfg = hermesConfigPath()
-  if (!fs.existsSync(cfg)) return false
-  try {
-    const src = fs.readFileSync(cfg, 'utf8')
-    // Legacy installs stored the env pairs as plain args (--env KEY=VALUE)
-    // because --env came after --args; Hermes now stores them under env:.
-    return /- MIO_DATA_DIR=/.test(src) || /- MIO_CONTEXT=/.test(src)
-  } catch (_) {
-    return false
-  }
-}
-
-function hermesBin() {
-  if (process.env.HERMES_BIN && fs.existsSync(process.env.HERMES_BIN)) return process.env.HERMES_BIN
-  const home = hermesHome()
-  const candidates = [
-    // Official Windows native install: venv script registered on User PATH
-    path.join(home, 'hermes-agent', 'venv', 'Scripts', 'hermes.exe'),
-    // Newer installer shim under <LocalAppData>/hermes/bin
-    path.join(home, 'bin', 'hermes.cmd'),
-    // Legacy layout
-    path.join(home, 'hermes-agent', 'bin', 'hermes.exe'),
-    path.join(os.homedir(), '.hermes', 'bin', 'hermes.exe'),
-  ]
-  for (const c of candidates) {
-    if (fs.existsSync(c)) return c
-  }
-  return null
-}
-
-// spawnSync() with a piped stdin fails on Windows with EBUSY before the child
-// even starts: status is null and stderr is empty, so every call below used to
-// return null and the Hermes adapter silently reported "not installed".
-// `stdio[0] = 'ignore'` fixes the calls that read no stdin. When there IS a
-// prompt answer to deliver it has to travel through a real file descriptor:
-// passing `input` alongside a non-pipe stdin makes Node drop the payload with
-// no error at all, which would leave `hermes mcp add` waiting forever for an
-// answer that never arrives -- worse than the crash it replaces.
-function spawnWithOptionalInput(command, args, opts) {
-  const { input, ...rest } = opts
-  if (input === undefined) {
-    return spawnSync(command, args, { ...rest, stdio: ['ignore', 'pipe', 'pipe'] })
-  }
-  const file = path.join(os.tmpdir(), 'mio-hermes-stdin-' + process.pid + '-' + Date.now())
-  fs.writeFileSync(file, input)
-  const fd = fs.openSync(file, 'r')
-  try {
-    return spawnSync(command, args, { ...rest, stdio: [fd, 'pipe', 'pipe'] })
-  } finally {
-    try {
-      fs.closeSync(fd)
-    } catch (_) {}
-    try {
-      fs.unlinkSync(file)
-    } catch (_) {}
-  }
-}
-
-function runHermes(args) {
-  const bin = hermesBin()
-  // `hermes mcp add` asks to confirm tool enablement interactively; pre-answer
-  // it so the install is non-interactive.
-  const input = args[0] === 'mcp' && args[1] === 'add' ? 'Y\n' : undefined
-  let res
-  try {
-    const isCmdShim = Boolean(bin && /\.(cmd|bat)$/i.test(bin))
-    res = bin
-      ? spawnWithOptionalInput(bin, args, { encoding: 'utf8', timeout: 60000, windowsHide: true, input: input, shell: isCmdShim })
-      : spawnWithOptionalInput('hermes', args, { encoding: 'utf8', timeout: 60000, shell: true, windowsHide: true, input: input })
-  } catch (_) {
-    return null
-  }
-  if (res.error || res.status !== 0) return null
-  return (res.stdout || '') + (res.stderr || '')
-}
-
-function isInstalled() {
-  // Read Hermes's own config instead of asking the binary. `hermes mcp list`
-  // costs ~1.8 s here (measured: Hermes is a packaged Python executable with a
-  // slow cold start), and listHostCapabilities() calls this once per host, so
-  // the moment the spawn started working again `mio.host.capabilities` went
-  // from ~3 ms to ~2 s. Hermes records registered MCP servers in config.yaml
-  // under `mcp_servers:`, so the file answers the same question in ~1 ms.
-  //
-  // Anchored on the YAML key rather than a plain substring: the config also
-  // contains the server script path, which includes "mio-intelligence-mcp" and
-  // would make a loose contains() report "installed" for a config that merely
-  // mentions the path.
-  //
-  // Trade-off, deliberately accepted: if a future Hermes stores MCP config
-  // somewhere this does not read, this reports false. That is no worse than
-  // what it did for a long time -- the spawn used to fail silently, so the
-  // answer was always false regardless.
-  const cfg = hermesConfigPath()
-  if (!fs.existsSync(cfg)) return false
-  try {
-    return new RegExp('^\\s*' + SERVER_NAME + '\\s*:', 'm').test(fs.readFileSync(cfg, 'utf8'))
-  } catch (_) {
-    return false
-  }
-}
-
-function install({ node, serverScript, home, workspace, project }) {
-  if (!hermesBin() && !isHermesOnPath()) {
-    return {
-      changed: false,
-      message: 'Hermes binary not found. Install hermes-agent first.',
-      configPath: null,
-    }
-  }
-  if (isInstalled()) {
-    if (!configNeedsRepair()) {
-      return { changed: false, message: 'Hermes Mio MCP already installed.', configPath: null }
-    }
-    runHermes(['mcp', 'remove', SERVER_NAME])
-  }
-
-  const context = JSON.stringify({
-    agentId: 'hermes',
-    project: project || path.basename(workspace || process.cwd()),
-    workspace: workspace || process.cwd(),
-    sessionId: 'hermes-session',
-  })
-
-  // Hermes requires --env before --args: --args swallows everything after it.
-  const args = [
-    'mcp', 'add', SERVER_NAME,
-    '--command', node,
-    '--env', 'MIO_DATA_DIR=' + home, 'MIO_CONTEXT=' + context,
-    '--args', serverScript,
-  ]
-  const out = runHermes(args)
-  const installed = isInstalled()
-
-  let message
-  if (installed) {
-    message = 'Hermes Mio MCP installed (config: <LocalAppData>/hermes/config.yaml). Start a new Hermes session to load it.'
-  } else {
-    message = 'Hermes mcp add returned but Mio was not detected. Run `hermes mcp add ' + SERVER_NAME + ' --command ' + node + ' --args ' + serverScript + '` manually and answer Y to the prompt.'
-  }
-  return { changed: installed, configPath: null, message }
-}
-
-function isHermesOnPath() {
-  try {
-    // stdio[0] must not be a pipe: see spawnWithOptionalInput above.
-    const res = spawnSync('where', ['hermes'], {
-      encoding: 'utf8',
-      shell: true,
-      stdio: ['ignore', 'pipe', 'pipe'],
-    })
-    return res.status === 0 && /hermes/.test(res.stdout || '')
-  } catch (_) {
-    return false
-  }
-}
-
-module.exports = { hermesBin, isInstalled, install }
+'use strict'
+
+const fs = require('fs')
+const path = require('path')
+const os = require('os')
+const { spawnSync } = require('child_process')
+
+const SERVER_NAME = 'mio-intelligence'
+
+function hermesHome() {
+  return process.env.HERMES_HOME || path.join(os.homedir(), 'AppData', 'Local', 'hermes')
+}
+
+function hermesConfigPath() {
+  return path.join(hermesHome(), 'config.yaml')
+}
+
+function configNeedsRepair() {
+  const cfg = hermesConfigPath()
+  if (!fs.existsSync(cfg)) return false
+  try {
+    const src = fs.readFileSync(cfg, 'utf8')
+    // Legacy installs stored the env pairs as plain args (--env KEY=VALUE)
+    // because --env came after --args; Hermes now stores them under env:.
+    return /- MIO_DATA_DIR=/.test(src) || /- MIO_CONTEXT=/.test(src)
+  } catch (_) {
+    return false
+  }
+}
+
+function hermesBin() {
+  if (process.env.HERMES_BIN && fs.existsSync(process.env.HERMES_BIN)) return process.env.HERMES_BIN
+  const home = hermesHome()
+  const candidates = [
+    // Official Windows native install: venv script registered on User PATH
+    path.join(home, 'hermes-agent', 'venv', 'Scripts', 'hermes.exe'),
+    // Newer installer shim under <LocalAppData>/hermes/bin
+    path.join(home, 'bin', 'hermes.cmd'),
+    // Legacy layout
+    path.join(home, 'hermes-agent', 'bin', 'hermes.exe'),
+    path.join(os.homedir(), '.hermes', 'bin', 'hermes.exe'),
+  ]
+  for (const c of candidates) {
+    if (fs.existsSync(c)) return c
+  }
+  return null
+}
+
+// spawnSync() with a piped stdin fails on Windows with EBUSY before the child
+// even starts: status is null and stderr is empty, so every call below used to
+// return null and the Hermes adapter silently reported "not installed".
+// `stdio[0] = 'ignore'` fixes the calls that read no stdin. When there IS a
+// prompt answer to deliver it has to travel through a real file descriptor:
+// passing `input` alongside a non-pipe stdin makes Node drop the payload with
+// no error at all, which would leave `hermes mcp add` waiting forever for an
+// answer that never arrives -- worse than the crash it replaces.
+function spawnWithOptionalInput(command, args, opts) {
+  const { input, ...rest } = opts
+  if (input === undefined) {
+    return spawnSync(command, args, { ...rest, stdio: ['ignore', 'pipe', 'pipe'] })
+  }
+  const file = path.join(os.tmpdir(), 'mio-hermes-stdin-' + process.pid + '-' + Date.now())
+  fs.writeFileSync(file, input)
+  const fd = fs.openSync(file, 'r')
+  try {
+    return spawnSync(command, args, { ...rest, stdio: [fd, 'pipe', 'pipe'] })
+  } finally {
+    try {
+      fs.closeSync(fd)
+    } catch (_) {}
+    try {
+      fs.unlinkSync(file)
+    } catch (_) {}
+  }
+}
+
+function runHermes(args) {
+  const bin = hermesBin()
+  // `hermes mcp add` asks to confirm tool enablement interactively; pre-answer
+  // it so the install is non-interactive.
+  const input = args[0] === 'mcp' && args[1] === 'add' ? 'Y\n' : undefined
+  let res
+  try {
+    const isCmdShim = Boolean(bin && /\.(cmd|bat)$/i.test(bin))
+    res = bin
+      ? spawnWithOptionalInput(bin, args, { encoding: 'utf8', timeout: 60000, windowsHide: true, input: input, shell: isCmdShim })
+      : spawnWithOptionalInput('hermes', args, { encoding: 'utf8', timeout: 60000, shell: true, windowsHide: true, input: input })
+  } catch (err) {
+    const msg = `hermes ${args.join(' ')} failed: ${err.message || err}`
+    console.error(msg)
+    throw new Error(msg)
+  }
+  if (res.error || res.status !== 0) {
+    const msg = `hermes ${args.join(' ')} exited ${res.status}: ${res.stderr || res.stdout || 'no output'}`
+    console.error(msg)
+    throw new Error(msg)
+  }
+  return (res.stdout || '') + (res.stderr || '')
+}
+
+function isInstalled() {
+  // Read Hermes's own config instead of asking the binary. `hermes mcp list`
+  // costs ~1.8 s here (measured: Hermes is a packaged Python executable with a
+  // slow cold start), and listHostCapabilities() calls this once per host, so
+  // the moment the spawn started working again `mio.host.capabilities` went
+  // from ~3 ms to ~2 s. Hermes records registered MCP servers in config.yaml
+  // under `mcp_servers:`, so the file answers the same question in ~1 ms.
+  //
+  // Anchored on the YAML key rather than a plain substring: the config also
+  // contains the server script path, which includes "mio-intelligence-mcp" and
+  // would make a loose contains() report "installed" for a config that merely
+  // mentions the path.
+  //
+  // Trade-off, deliberately accepted: if a future Hermes stores MCP config
+  // somewhere this does not read, this reports false. That is no worse than
+  // what it did for a long time -- the spawn used to fail silently, so the
+  // answer was always false regardless.
+  const cfg = hermesConfigPath()
+  if (!fs.existsSync(cfg)) return false
+  try {
+    return new RegExp('^\\s*' + SERVER_NAME + '\\s*:', 'm').test(fs.readFileSync(cfg, 'utf8'))
+  } catch (_) {
+    return false
+  }
+}
+
+function install({ node, serverScript, home, workspace, project }) {
+  if (!hermesBin() && !isHermesOnPath()) {
+    return {
+      changed: false,
+      message: 'Hermes binary not found. Install hermes-agent first.',
+      configPath: null,
+    }
+  }
+  if (isInstalled()) {
+    if (!configNeedsRepair()) {
+      return { changed: false, message: 'Hermes Mio MCP already installed.', configPath: null }
+    }
+    runHermes(['mcp', 'remove', SERVER_NAME])
+  }
+
+  const context = JSON.stringify({
+    agentId: 'hermes',
+    project: project || path.basename(workspace || process.cwd()),
+    workspace: workspace || process.cwd(),
+    sessionId: 'hermes-session',
+  })
+
+  // Hermes requires --env before --args: --args swallows everything after it.
+  const args = [
+    'mcp', 'add', SERVER_NAME,
+    '--command', node,
+    '--env', 'MIO_DATA_DIR=' + home, 'MIO_CONTEXT=' + context,
+    '--args', serverScript,
+  ]
+  const out = runHermes(args)
+  const installed = isInstalled()
+
+  let message
+  if (installed) {
+    message = 'Hermes Mio MCP installed (config: <LocalAppData>/hermes/config.yaml). Start a new Hermes session to load it.'
+  } else {
+    message = 'Hermes mcp add returned but Mio was not detected. Run `hermes mcp add ' + SERVER_NAME + ' --command ' + node + ' --args ' + serverScript + '` manually and answer Y to the prompt.'
+  }
+  return { changed: installed, configPath: null, message }
+}
+
+function isHermesOnPath() {
+  try {
+    // stdio[0] must not be a pipe: see spawnWithOptionalInput above.
+    const res = spawnSync('where', ['hermes'], {
+      encoding: 'utf8',
+      shell: true,
+      stdio: ['ignore', 'pipe', 'pipe'],
+    })
+    return res.status === 0 && /hermes/.test(res.stdout || '')
+  } catch (_) {
+    return false
+  }
+}
+
+module.exports = { hermesBin, isInstalled, install }