openvisio-agent 0.10.0 → 0.10.1

package/bin/cli.mjs

@@ -91,7 +91,8 @@ function mcpReplace(claude, addArgs) {
 // background service) to work — an `npx` connect leaves nothing installed. Make
 // the global install part of setup.
 function ensureAgentInstalled() {
-  if (onPath('openvisio-agent')) return
+  const current = onPath('openvisio-agent')
+  if (current && !current.includes('/_npx/')) return
   info('Installing openvisio-agent globally (so `watch` and the always-on service have a stable command)…')
   spawnSync('npm', ['i', '-g', 'openvisio-agent@latest'], { stdio: 'inherit', shell: process.platform === 'win32' })
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openvisio-agent",
-  "version": "0.10.0",
+  "version": "0.10.1",
   "description": "Connect Claude Code, Codex, or OpenCode to an OpenVisio team — MCP tools + optional autonomy — in one command.",
   "type": "module",
   "bin": {

package/src/watch.mjs

@@ -217,9 +217,21 @@ export async function runWatch({ flags }) {
   if (!flags.install) {
     const lock = acquireSingleInstance(slug || 'openvisio')
     if (lock.conflict) {
-      fail(`Another openvisio-agent watcher for "${slug || 'openvisio'}" is already running (pid ${lock.conflict}).\n` +
+      const watcherName = slug || 'openvisio'
+      const stop = process.platform === 'darwin'
+        ? `launchctl unload ~/Library/LaunchAgents/io.openvisio.${watcherName}.plist`
+        : process.platform === 'win32'
+          ? 'Stop the existing openvisio-agent process in Task Manager.'
+          : `systemctl --user stop openvisio-${watcherName}.service`
+      const logs = process.platform === 'darwin'
+        ? `tail -f ~/.openvisio/${watcherName}.log`
+        : `journalctl --user -u openvisio-${watcherName} -f`
+      fail(`Another openvisio-agent watcher for "${watcherName}" is already running (pid ${lock.conflict}).\n` +
         `  Two watchers for the same agent BOTH reply to every mention — that is what causes duplicate/contradicting messages.\n` +
-        `  Stop the other one (kill ${lock.conflict}), or rely on ONLY the background service. Refusing to start a second.`)
+        `  This is usually the auto-restarting background service; killing its PID only makes it restart.\n` +
+        `  To run in this terminal instead, stop the service first:\n    ${stop}\n` +
+        `  Or keep the service and inspect its log:\n    ${logs}\n` +
+        `  Refusing to start a second watcher.`)
     }
     process.on('exit', () => { try { lock.release && lock.release() } catch { /* noop */ } })
   }
@@ -321,7 +333,7 @@ function createCodexRunner({ mcpUrl, mcpHeaders, workdir, canCode, maxCycleMs, l
         ? `mcp_servers={ openvisio-team = { url = ${tomlString(mcpUrl)}${headerEntries ? `, http_headers = { ${headerEntries} }` : ''} } }`
         : ''
       const args = ['exec', '--ignore-user-config', '--skip-git-repo-check', '--ephemeral', '--json', '--color', 'never',
-        '--sandbox', canCode ? 'workspace-write' : 'read-only', '--approve-for-me',
+        ...(canCode ? ['--approve-for-me'] : ['--sandbox', 'read-only']),
         ...(m ? ['--model', m] : []),
         ...(mcpOverride ? ['-c', mcpOverride] : []),
         full]
@@ -815,7 +827,7 @@ function loop({ host, key, slug, claude, agent, mcpConfig, mcpUrl, workdir, mode
 // ── background service install (launchd / systemd) ───────────────────────────
 export function installService({ slug, workdir }) {
   const binPath = onPath('openvisio-agent')
-  if (!binPath) {
+  if (!binPath || binPath.includes('/_npx/')) {
     info('Installing openvisio-agent globally so the background service has a stable path…')
     spawnSync('npm', ['i', '-g', 'openvisio-agent'], { stdio: 'inherit', shell: process.platform === 'win32' })
   }