openvisio-agent 0.15.1 → 0.16.0

package/package.json

@@ -1,11 +1,14 @@
 {
   "name": "openvisio-agent",
-  "version": "0.15.1",
+  "version": "0.16.0",
   "description": "Connect Claude Code, Codex, or OpenCode to an OpenVisio team — MCP tools + optional autonomy — in one command.",
   "type": "module",
   "bin": {
     "openvisio-agent": "bin/cli.mjs"
   },
+  "scripts": {
+    "test": "node --test"
+  },
   "files": [
     "bin",
     "src",

package/src/events.mjs

@@ -0,0 +1,27 @@
+// Backend deployments have emitted task bodies in a few compatible envelope
+// shapes over time. Keep that transport detail out of the watcher so an
+// assignment cannot be silently dropped during a rolling backend/client update.
+export function taskFromEvent(payload) {
+  if (!payload || typeof payload !== 'object') return null
+
+  const candidates = [
+    payload.task,
+    payload.data && payload.data.task,
+    payload.payload && payload.payload.task,
+    payload.data,
+    payload.payload,
+    payload,
+  ]
+
+  for (const candidate of candidates) {
+    if (!candidate || typeof candidate !== 'object') continue
+    const hasTaskIdentity = candidate.id != null || candidate.task_id != null || candidate.taskId != null
+    if (hasTaskIdentity) return candidate
+  }
+  return null
+}
+
+export function taskAgentId(task) {
+  if (!task || typeof task !== 'object') return null
+  return task.agent_id != null ? task.agent_id : (task.agentId != null ? task.agentId : null)
+}

package/src/watch.mjs

@@ -10,6 +10,7 @@ import { homedir } from 'node:os'
 import { join, dirname } from 'node:path'
 import { OV_DIR, DEFAULT_WORKSPACE, readConfig, writeJson, configPath, onPath, fail, ok, info, slugify, stripSlash, chmodSafe } from './lib.mjs'
 import { connectAgentWs, assertWebSocket } from './ws.mjs'
+import { taskAgentId, taskFromEvent } from './events.mjs'
 
 // Behaviour prompts. The openvisio-team MCP bridge requires the agent's
 // credentials as ARGUMENTS on every tool call — those are injected at runtime by
@@ -528,10 +529,16 @@ function createCycleRunner({ claude, agent, mcpUrl, mcpHeaders, cfgKey, mcpConfi
 function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConfig, mcpUrl, workdir, model, chatModel, debug }) {
   const log = (m) => process.stdout.write('[ws ' + new Date().toISOString() + '] ' + m + '\n')
   let handle = null
+  let statusEnabled = true
+  let lastStatusSentAt = 0
   // Channels the agent is actively working in this cycle — drives the live
   // agent:status broadcast (thinking → working → typing → done).
   const statusTargets = new Set()
-  const emitStatus = (state) => { for (const c of statusTargets) { try { handle && handle.sendStatus(c, state) } catch { /* best-effort */ } } }
+  const sendStatus = (channelId, state) => {
+    if (!statusEnabled) return
+    try { if (handle) { lastStatusSentAt = Date.now(); handle.sendStatus(channelId, state) } } catch { /* best-effort */ }
+  }
+  const emitStatus = (state) => { for (const c of statusTargets) sendStatus(c, state) }
   const canCode = !!workdir
   // The Mastra bridge authenticates per-CALL: every openvisio-team tool needs
   // agent_identifier + agent_api_key as arguments. Hand them over up front.
@@ -539,12 +546,18 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
   // The STATIC charter + creds are the session system prompt (cached, billed once),
   // NOT re-sent in every cycle's user message — the big token saving.
   const systemPrompt = (canCode ? CODE_CHARTER : CHAT_CHARTER) + '\n\n' + credNote
-  const { runCycle } = createCycleRunner({
+  const runnerOptions = {
     claude, agent, mcpUrl, mcpHeaders: { 'x-agent-api-key': apiKey, 'x-agent-identifier': identifier },
     cfgKey: identifier, mcpConfig, workdir, log, debug, model, systemPrompt,
     // The moment the agent calls post_message it is about to speak → "typing".
     onTool: (name) => { if (/post_message/.test(name)) emitStatus('typing') },
-  })
+  }
+  // One watcher and one WS subscription, but two independent model lanes. This
+  // avoids duplicate event delivery while mentions can be answered during code.
+  const runners = {
+    work: createCycleRunner(runnerOptions),
+    reply: createCycleRunner({ ...runnerOptions, cfgKey: identifier + '-reply' }),
+  }
   const fullPrompt = canCode ? CODE_FULL : CYCLE
   const fastPrompt = canCode ? CODE_FAST : CYCLE_FAST
   // Live model state — changeable at runtime by the in-chat `/model` command.
@@ -553,8 +566,10 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
   let codeModel = model
   let liteModel = chatModel || model
 
-  let busy = false
-  let queued = null // 'full' | 'coord' | 'fast' | 'sweep' | 'intro' — a cycle requested while one was running
+  const lanes = {
+    work: { busy: false, queued: null, pending: [] },
+    reply: { busy: false, queued: null, pending: [] },
+  }
   // Tasks we've already reacted to (keyed task-id:agent — so a REASSIGNMENT to a
   // different agent re-triggers), so a noisy stream of task:updated events doesn't
   // re-acknowledge the same assignment.
@@ -567,7 +582,6 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
   // channel + message / task), so the agent acts on THEM directly instead of
   // hoping poll_inbox re-surfaces the same item. Accumulated across coalesced
   // events and drained into the next cycle's prompt.
-  const pending = []
   let backlogProbeBusy = false
   let lastTaskSignature = ''
   let lastTaskTriggeredAt = 0
@@ -596,7 +610,7 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
   }
   const ensureMcpSession = async () => {
     if (mcpSessionId) return
-    const res = await mcpPost({ jsonrpc: '2.0', id: ++mcpRpcId, method: 'initialize', params: { protocolVersion: '2025-03-26', capabilities: {}, clientInfo: { name: 'openvisio-agent', version: '0.15.1' } } }, false)
+    const res = await mcpPost({ jsonrpc: '2.0', id: ++mcpRpcId, method: 'initialize', params: { protocolVersion: '2025-03-26', capabilities: {}, clientInfo: { name: 'openvisio-agent', version: '0.16.0' } } }, false)
     if (!res.ok) throw new Error('MCP initialize HTTP ' + res.status)
     await mcpPayload(res)
     mcpSessionId = res.headers.get('mcp-session-id') || ''
@@ -629,7 +643,7 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
   // Reconcile everything that may have arrived while disconnected. This spends no
   // model tokens unless the tools actually report pending work.
   const reconcileBacklog = async () => {
-    if (!mcpUrl || backlogProbeBusy || busy) return
+    if (!mcpUrl || backlogProbeBusy) return
     backlogProbeBusy = true
     try {
       const agentsData = toolData(await callMcpTool('list_agents'))
@@ -690,10 +704,12 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
   const baseFor = (kind) => kind === 'intro' ? INTRO : kind === 'full' ? fullPrompt : (kind === 'coord' || kind === 'sweep') ? COORDINATE : fastPrompt
 
   async function drain(kind, context) {
-    if (context) pending.push(context)
-    if (busy) { queued = (RANK[kind] ?? 0) >= (RANK[queued] ?? 0) ? kind : queued; log('busy — queued a ' + kind + ' follow-up cycle'); return }
-    busy = true
-    const ctx = pending.splice(0) // take everything accumulated so far
+    const laneName = kind === 'full' ? 'work' : 'reply'
+    const lane = lanes[laneName]
+    if (context) lane.pending.push(context)
+    if (lane.busy) { lane.queued = (RANK[kind] ?? 0) >= (RANK[lane.queued] ?? 0) ? kind : lane.queued; log(laneName + ' lane busy — queued a ' + kind + ' follow-up cycle'); return }
+    lane.busy = true
+    const ctx = lane.pending.splice(0)
     // credNote + charter live in the cached system prompt now — the per-cycle
     // message is just the event context + the small base instruction.
     const prompt = (ctx.length ? ctx.join('\n') + '\n\n' : '') + baseFor(kind)
@@ -707,7 +723,7 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
     emitStatus('working')
     const heartbeat = targets.length ? setInterval(() => emitStatus('working'), 9000) : null
     try {
-      const result = await runCycle(prompt, useModel)
+      const result = await runners[laneName].runCycle(prompt, useModel)
       // A zero exit is not proof of work. Assigned-ticket cycles must both use
       // the task MCP and leave repository evidence. A simple `ls` no longer
       // counts as completion. Do not retry a verified real blocker.
@@ -716,13 +732,13 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
       if (agent === 'codex' && kind === 'full' && result?.subtype === 'ok' && incomplete && !legitimateNoWork) {
         const missing = [!result.didMcpTaskRead && 'read the ticket through get_ticket/list_tasks', !result.didRepoMutation && 'perform and verify the repository change', !result.didMcpTaskUpdate && 'update the ticket through update_ticket'].filter(Boolean)
         log('coding cycle incomplete; recovery requires: ' + missing.join(', '))
-        await runCycle(`The assigned task is NOT complete. Missing evidence: ${missing.join('; ')}. Do not post another acknowledgement or plan. Resume now. First use get_ticket/list_tasks and list_task_types, then do the repository work, verify it, commit and push an agent/* branch, open the PR, call update_ticket with the correct board column, and post exactly one result update with evidence. If a real blocker appears, report it once.`, codeModel)
+        await runners.work.runCycle(`The assigned task is NOT complete. Missing evidence: ${missing.join('; ')}. Do not post another acknowledgement or plan. Resume now. First use get_ticket/list_tasks and list_task_types, then do the repository work, verify it, commit and push an agent/* branch, open the PR, call update_ticket with the correct board column, and post exactly one result update with evidence. If a real blocker appears, report it once.`, codeModel)
       }
     } finally {
       if (heartbeat) clearInterval(heartbeat)
-      for (const c of targets) { try { handle && handle.sendStatus(c, 'done') } catch { /* noop */ } statusTargets.delete(c) }
-      busy = false
-      if (queued || pending.length) { const next = queued || 'fast'; queued = null; void drain(next) }
+      for (const c of targets) { sendStatus(c, 'done'); statusTargets.delete(c) }
+      lane.busy = false
+      if (lane.queued || lane.pending.length) { const next = lane.queued || (laneName === 'work' ? 'full' : 'fast'); lane.queued = null; void drain(next) }
     }
   }
 
@@ -837,15 +853,28 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
       log('agent:mention in channel ' + (cid != null ? cid : '?') + (threadRoot != null ? ' (thread ' + threadRoot + ')' : ''))
       // Light up the live status the instant we pick this up (thinking → the cycle
       // takes it to working → typing → done).
-      if (cid != null) { statusTargets.add(cid); try { handle && handle.sendStatus(cid, 'thinking') } catch { /* best-effort */ } }
+      if (cid != null) { statusTargets.add(cid); sendStatus(cid, 'thinking') }
       const codingMention = canCode && needsCode(text)
       const ctx = cid != null
         ? `You were @mentioned in OpenVisio channel ${cid}${who ? ` by "${who}"` : ''}: "${text}". This mention is FOR YOU. ${codingMention ? 'This is repository work: complete the coding flow first, then send' : 'Send'} EXACTLY ONE reply with post_message: arguments: channel_id ${cid}${threadRoot != null ? `, parent_id ${threadRoot} (reply IN THAT THREAD, do not start a new top-level message)` : ''}, plus agent_identifier + agent_api_key from the AUTH line above, and a 1-3 sentence reply. Compose the whole answer, then post it ONCE. Do not post a first reply and then a revised version. FIRST read the recent messages in this thread: if you already answered this, or another agent was the one addressed, do NOT post at all. Be sure of your answer before sending.${who ? ` To @mention them back, write their EXACT full name "@${who}". A mention only links when the name matches exactly.` : ''} You ALREADY have the message here. Do not poll_inbox, and after your single reply, STOP.`
         : undefined
-      void drain(codingMention ? 'full' : 'fast', ctx)
+      if (codingMention) {
+        const ack = cid != null
+          ? `You were asked for repository work in channel ${cid}${threadRoot != null ? `, thread ${threadRoot}` : ''}. The dedicated work lane has accepted it. Post exactly one short reply with post_message in that same thread saying you have picked it up and will return there with the verified result. Include agent_identifier + agent_api_key. Do not inspect or edit code in this reply lane.`
+          : undefined
+        void drain('coord', ack)
+        void drain('full', ctx)
+      } else void drain('fast', ctx)
     } else if (k === 'error') {
-      // Surface the backend's rejection detail instead of a bare "event error".
-      log('error event: ' + JSON.stringify(raw).slice(0, 220))
+      const detail = raw && (raw.message || raw.error || raw.reason || raw.code || raw.d?.message || raw.d?.error)
+      // Older backend stages reject agent_status. Stop its heartbeat after the
+      // first immediate rejection instead of producing an error every nine seconds.
+      if (statusEnabled && lastStatusSentAt && Date.now() - lastStatusSentAt < 3000) {
+        statusEnabled = false
+        log('live agent status unsupported by this backend; disabling status heartbeat' + (detail ? ': ' + String(detail).slice(0, 160) : ''))
+      } else {
+        log('error event: ' + (detail ? String(detail) : JSON.stringify(raw)).slice(0, 220))
+      }
     } else {
       log('event ' + k)
     }

package/src/ws.mjs

@@ -79,7 +79,9 @@ export function connectAgentWs({ wsUrl, apiKey, identifier, onEvent, onConnect,
       try { msg = JSON.parse(typeof ev.data === 'string' ? ev.data : String(ev.data)) } catch { return }
       const kind = msg && msg.k
       if (!kind || kind === 'keepalive:ack') return // keepalive round-trip, nothing to do
-      try { onEvent(kind, msg.d) } catch (e) { log('event handler error: ' + (e && e.message ? e.message : e)) }
+      // Error metadata often lives on the JSON envelope rather than `d`. Preserve
+      // it so callers do not receive an unhelpful empty object.
+      try { onEvent(kind, kind === 'error' ? msg : msg.d) } catch (e) { log('event handler error: ' + (e && e.message ? e.message : e)) }
     })
 
     const down = (why) => {