openvisio-agent 0.12.0 → 0.13.1

package/README.md

@@ -56,6 +56,8 @@ Then `watch --name ada` auto-detects the backend agent and runs a WebSocket loop
 
 Runs the **autonomy loop** — the agent replies to @mentions and picks up tickets on its own. It cheaply polls an inbox endpoint (no model spend when idle) and pokes a single warm Claude Code session only when something new arrives.
 
+Backend/BYO watchers reconcile immediately whenever the process starts or the WebSocket connects. A direct MCP session checks both `poll_inbox` and `get_marching_orders`, covering unanswered mentions, follow-ups, and assigned tasks missed while offline. The same zero-model check runs every five minutes as a safety net; a model starts only when pending work exists.
+
 Routine messages, triage, introductions, catch-up checks, and ticket movement use a lightweight model by default. Claude uses Haiku and Codex uses `gpt-5.6-luna`. Requests that clearly require repository work route to the coding model: Sonnet for Claude and `gpt-5.6-sol` for Codex. Override either lane with `--chat-model` and `--model`.
 
 ```bash

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openvisio-agent",
-  "version": "0.12.0",
+  "version": "0.13.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

@@ -550,6 +550,85 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
   // 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
+  let mcpSessionId = ''
+  let mcpRpcId = 0
+
+  const mcpPayload = async (res) => {
+    const body = await res.text()
+    const data = body.split(/\r?\n/).filter((line) => line.startsWith('data:')).map((line) => line.slice(5).trim()).pop()
+    return JSON.parse(data || body || '{}')
+  }
+  const mcpPost = async (message, withSession = true) => {
+    const res = await fetch(mcpUrl, {
+      method: 'POST',
+      headers: {
+        'content-type': 'application/json',
+        accept: 'application/json, text/event-stream',
+        'x-agent-api-key': apiKey,
+        'x-agent-identifier': identifier,
+        ...(withSession && mcpSessionId ? { 'mcp-session-id': mcpSessionId } : {}),
+      },
+      body: JSON.stringify(message),
+    })
+    return res
+  }
+  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.13.1' } } }, false)
+    if (!res.ok) throw new Error('MCP initialize HTTP ' + res.status)
+    await mcpPayload(res)
+    mcpSessionId = res.headers.get('mcp-session-id') || ''
+    if (!mcpSessionId) throw new Error('MCP initialize returned no session id')
+    const ready = await mcpPost({ jsonrpc: '2.0', method: 'notifications/initialized' })
+    if (!ready.ok) throw new Error('MCP initialized HTTP ' + ready.status)
+  }
+  const callMcpTool = async (name, args = {}, retried = false) => {
+    await ensureMcpSession()
+    const res = await mcpPost({ jsonrpc: '2.0', id: ++mcpRpcId, method: 'tools/call', params: { name, arguments: { ...args, agent_api_key: apiKey, agent_identifier: identifier } } })
+    if (!res.ok) {
+      if (!retried && (res.status === 400 || res.status === 404)) { mcpSessionId = ''; return callMcpTool(name, args, true) }
+      throw new Error(`MCP ${name} HTTP ${res.status}`)
+    }
+    const payload = await mcpPayload(res)
+    if (payload.error) throw new Error(`MCP ${name}: ${payload.error.message || 'tool error'}`)
+    return JSON.stringify(payload.result ?? payload)
+  }
+
+  // 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
+    backlogProbeBusy = true
+    try {
+      const orders = await callMcpTool('get_marching_orders')
+      const inbox = await callMcpTool('poll_inbox')
+      const normalized = String(orders).replace(/\\"/g, '"').replace(/\s+/g, ' ').trim()
+      const empty = !normalized || /\b(?:no|zero)\s+(?:open\s+|assigned\s+|pending\s+)?tasks?\b/i.test(normalized) || /"(?:tasks|assigned|marching_orders)"\s*:\s*\[\s*\]/i.test(normalized)
+      if (empty) lastTaskSignature = ''
+      else if (/\b(?:task|ticket|title|status|assigned)\b/i.test(normalized)) {
+        const signature = normalized.slice(0, 1200)
+        const retryDue = Date.now() - lastTaskTriggeredAt > 30 * 60 * 1000
+        if (signature !== lastTaskSignature || retryDue) {
+          lastTaskSignature = signature
+          lastTaskTriggeredAt = Date.now()
+          log('backlog reconciliation found pending tasks -> coding cycle')
+          void drain('full', 'Backlog reconciliation found one or more tasks in get_marching_orders. Call get_marching_orders now, select every open task assigned to YOU, and execute the oldest/highest-priority one immediately. Do not merely acknowledge it. Move it to the active column, complete the repository work, verify it, open the PR, move the ticket to done when appropriate, and report evidence. Ignore tasks assigned to anyone else.')
+        }
+      }
+      const inboxText = String(inbox).replace(/\\"/g, '"').replace(/\s+/g, ' ').trim()
+      const hasInbox = /"(?:mentions|messages|follow_?ups|items)"\s*:\s*\[\s*\{/i.test(inboxText) || /\b(?:pending|unanswered|new)\s+(?:mention|message|follow.?up)/i.test(inboxText)
+      if (hasInbox) {
+        log('backlog reconciliation found inbox activity -> reply cycle')
+        void drain('fast', 'Backlog reconciliation found pending inbox activity. Call poll_inbox now and handle every unanswered mention or follow-up directed to YOU. Do not respond to other agents\' messages and do not duplicate a reply already present.')
+      }
+    } catch (e) {
+      mcpSessionId = ''
+      log('backlog reconciliation failed: ' + (e && e.message ? e.message : e))
+    } finally { backlogProbeBusy = false }
+  }
 
   // Higher rank wins when coalescing cycles requested while one is running.
   const RANK = { fast: 0, intro: 1, coord: 2, sweep: 2, full: 3 }
@@ -577,9 +656,10 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
       // Codex can exit successfully after posting only "I'll do it". For coding
       // cycles, treat that as incomplete and give it one focused recovery turn.
       // Do not retry a real blocker, which would only create duplicate noise.
-      if (agent === 'codex' && kind === 'full' && result?.subtype === 'ok' && result.didMessage && !result.didCode && !/\b(?:blocked|cannot|can't|missing|need access|permission|unclear)\b/i.test(result.outputText || '')) {
-        log('coding cycle only acknowledged the task; running one completion recovery')
-        await runCycle('You posted an acknowledgement but performed no repository work. Do not post another acknowledgement. Resume the same assigned task now: inspect the workspace, make the required changes, verify them, commit and push an agent/* branch, open the PR, then post exactly one result update with evidence. If a real blocker appears, report that blocker once.', codeModel)
+      const legitimateNoWork = /\b(?:no (?:open |assigned |pending )?tasks?|not assigned to (?:me|you)|assigned to (?:another|someone else)|blocked|cannot|can't|missing|need access|permission|unclear)\b/i.test(result?.outputText || '')
+      if (agent === 'codex' && kind === 'full' && result?.subtype === 'ok' && !result.didCode && !legitimateNoWork) {
+        log('coding cycle exited without repository action; running one completion recovery')
+        await runCycle('You exited the assigned coding task without performing repository work. Do not post another acknowledgement or plan. Resume the same task now: inspect the workspace, make the required changes, verify them, commit and push an agent/* branch, open the PR, then post exactly one result update with evidence. If a real blocker appears, report that blocker once.', codeModel)
       }
     } finally {
       if (heartbeat) clearInterval(heartbeat)
@@ -639,12 +719,13 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
     // fanned out org-wide. Catch both, keep only agent-assigned tasks, and let the
     // cycle confirm ownership via get_marching_orders before acting.
     if (k === 'task:created' || k === 'task:updated') {
-      const t = raw.task && typeof raw.task === 'object' ? raw.task : {}
-      const agentId = t.agent_id != null ? t.agent_id : (t.agentId != null ? t.agentId : null)
+      const envelope = raw.data && typeof raw.data === 'object' ? raw.data : raw
+      const t = envelope.task && typeof envelope.task === 'object' ? envelope.task : envelope
+      const ag = (t.agent && typeof t.agent === 'object') ? t.agent : (t.assigned_agent && typeof t.assigned_agent === 'object') ? t.assigned_agent : null
+      const agentId = t.agent_id ?? t.agentId ?? t.assigned_agent_id ?? t.assignedAgentId ?? t.assignee_agent_id ?? t.assigneeAgentId ?? ag?.id ?? ag?.agent_id ?? null
       if (agentId == null) return // not assigned to an agent — ignore
       // If the payload carries the agent's identifier, filter precisely to US and skip
       // other agents' tasks entirely; otherwise let get_marching_orders confirm.
-      const ag = (t.agent && typeof t.agent === 'object') ? t.agent : null
       const agIdent = ag && (ag.identifier || ag.slug) ? String(ag.identifier || ag.slug) : null
       if (agIdent != null && agIdent !== identifier) return
       const key = `${t.id}:${agentId}`
@@ -719,10 +800,10 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
   // so anything assigned while the agent was offline — tasks especially — is still
   // picked up even though we don't hot-poll.
   log('up — backend WS watcher on ' + wsUrl + (canCode ? ' [code: ' + workdir + ']' : ''))
-  handle = connectAgentWs({ wsUrl, apiKey, identifier, onEvent, log })
+  handle = connectAgentWs({ wsUrl, apiKey, identifier, onEvent, onConnect: () => { setTimeout(() => void reconcileBacklog(), 250) }, log })
 
   const DAY_MS = 24 * 60 * 60 * 1000
-  let introTimer = null, sweepStartTimer = null, sweepTimer = null
+  let introTimer = null, sweepStartTimer = null, sweepTimer = null, taskProbeStartTimer = null, taskProbeTimer = null
   if (mcpConfig || mcpUrl) {
     // Workspace ethics: a one-time hello the FIRST time this agent ever connects.
     const introMarker = join(OV_DIR, 'intro-' + slugify(identifier) + '.done')
@@ -740,6 +821,10 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
       log('startup catch-up sweep skipped (ran within the last 6h)')
     }
     sweepTimer = setInterval(() => { log('daily catch-up sweep'); void drain('sweep', SWEEP) }, DAY_MS)
+    // Cheap reliability net: no model runs unless the MCP result actually contains
+    // assigned work. This also catches assignments created while the socket was down.
+    taskProbeStartTimer = setTimeout(() => void reconcileBacklog(), 3_000)
+    taskProbeTimer = setInterval(() => void reconcileBacklog(), 5 * 60 * 1000)
   } else {
     log('no MCP config — skipping intro + daily sweep (agent has no tools to post/act)')
   }
@@ -751,6 +836,8 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
       if (introTimer) clearTimeout(introTimer)
       if (sweepStartTimer) clearTimeout(sweepStartTimer)
       if (sweepTimer) clearInterval(sweepTimer)
+      if (taskProbeStartTimer) clearTimeout(taskProbeStartTimer)
+      if (taskProbeTimer) clearInterval(taskProbeTimer)
       try { handle && handle.close() } catch { /* noop */ }
       process.exit(0)
     }

package/src/ws.mjs

@@ -36,7 +36,7 @@ export function assertWebSocket(fail) {
  * @param {(kind: string, payload: any) => void} o.onEvent  targeted dispatch.
  * @param {(msg: string) => void} o.log
  */
-export function connectAgentWs({ wsUrl, apiKey, identifier, onEvent, log }) {
+export function connectAgentWs({ wsUrl, apiKey, identifier, onEvent, onConnect, log }) {
   const base = stripSlash(wsUrl)
   const url = `${base}?api_key=${encodeURIComponent(apiKey)}&identifier=${encodeURIComponent(identifier)}`
 
@@ -66,6 +66,7 @@ export function connectAgentWs({ wsUrl, apiKey, identifier, onEvent, log }) {
       if (sock !== ws) return
       backoff = BACKOFF_MIN_MS // healthy connection → reset the backoff ramp
       log(`connected as @${identifier}`)
+      try { onConnect && onConnect() } catch { /* reconciliation is best-effort */ }
       clearKeepalive()
       keepalive = setInterval(() => {
         try { sock.send(JSON.stringify({ type: 'keepalive' })) } catch { /* closing */ }