openvisio-agent 0.15.0 → 0.15.2

package/README.md

@@ -58,7 +58,7 @@ Runs the **autonomy loop** — the agent replies to @mentions and picks up ticke
 
 Backend/BYO watchers reconcile immediately whenever the process starts or the WebSocket connects. A direct MCP session uses the backend's actual tools (`list_agents`, `list_projects`, `list_tasks`, `list_task_types`, and `list_activity`) to recover assigned tasks and recent mention activity 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 lower-cost model by default. Claude uses Haiku and Codex uses `gpt-5.6-terra`; Luna is intentionally not used because watcher coordination depends on reliable MCP tool use. Requests that 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`.
+Claude uses Haiku for routine coordination and Sonnet for repository work. Codex uses `gpt-5.6-sol` for every cycle, including messages, mentions, triage, board movement, MCP calls, and coding. This intentionally favors reliability and consistent tool use over the cheaper Codex tiers.
 
 ```bash
 openvisio-agent watch --name ada                 # run in this terminal

package/bin/cli.mjs

@@ -59,8 +59,8 @@ connect --backend
                      gpt-5.6-sol; OpenCode uses its configured default.
                      Engineers can also change it live from chat: "@agent /model …".
     --chat-model <m> coordination model for chat, triage, and board movement.
-                     Defaults: Claude haiku, Codex gpt-5.6-terra.
-                     Coding work stays on --model.
+                     Claude defaults to haiku. Codex uses gpt-5.6-sol for every
+                     cycle, including coordination and messaging.
     --no-service     skip the background service — just save config + print the
                      watch commands to run yourself.
 

package/package.json

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

@@ -215,11 +215,10 @@ export async function runWatch({ flags }) {
   const workdir = chatOnly ? '' : (explicitWorkdir || (saved && (saved.workspace || saved.workdir)) || DEFAULT_WORKSPACE)
   if (workdir) { try { mkdirSync(workdir, { recursive: true }) } catch { /* best-effort; spawn will surface a real problem */ } }
 
-  // Keep routine coordination on a balanced tool-capable tier. Luna proved too
-  // willing to narrate instead of acting in MCP-heavy watcher cycles, so Codex
-  // defaults to Terra for coordination and reserves Sol for repository work.
+  // Codex uses Sol for every watcher cycle. Reliability and consistent MCP/tool
+  // behaviour take priority over maintaining a cheaper coordination lane.
   const defaultModel = agent === 'claude' ? 'sonnet' : agent === 'codex' ? 'gpt-5.6-sol' : ''
-  const defaultChatModel = agent === 'claude' ? 'haiku' : agent === 'codex' ? 'gpt-5.6-terra' : ''
+  const defaultChatModel = agent === 'claude' ? 'haiku' : agent === 'codex' ? 'gpt-5.6-sol' : ''
   const model = String(flags.model || (saved && saved.model) || defaultModel)
   const chatModel = String(flags['chat-model'] || (saved && saved.chatModel) || defaultChatModel)
 
@@ -529,10 +528,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.
@@ -597,7 +602,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.0' } } }, false)
+    const res = await mcpPost({ jsonrpc: '2.0', id: ++mcpRpcId, method: 'initialize', params: { protocolVersion: '2025-03-26', capabilities: {}, clientInfo: { name: 'openvisio-agent', version: '0.15.2' } } }, false)
     if (!res.ok) throw new Error('MCP initialize HTTP ' + res.status)
     await mcpPayload(res)
     mcpSessionId = res.headers.get('mcp-session-id') || ''
@@ -700,7 +705,7 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
     const prompt = (ctx.length ? ctx.join('\n') + '\n\n' : '') + baseFor(kind)
     // Chat-shaped cycles (mentions/intro) may run on the cheaper chat model; code
     // work (full/sweep) uses the main model.
-    const useModel = kind === 'full' ? codeModel : liteModel
+    const useModel = agent === 'codex' ? codeModel : kind === 'full' ? codeModel : liteModel
     log('running ' + kind + ' cycle…' + (ctx.length ? ' (' + ctx.length + ' event' + (ctx.length === 1 ? '' : 's') + ')' : '') + (useModel ? ' [' + useModel + ']' : ''))
     // Live status: "working" now + a heartbeat so the UI (and its TTL) stays lit
     // through a long cycle; onTool flips it to "typing" when post_message fires.
@@ -721,7 +726,7 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, agent, mcpConf
       }
     } finally {
       if (heartbeat) clearInterval(heartbeat)
-      for (const c of targets) { try { handle && handle.sendStatus(c, 'done') } catch { /* noop */ } statusTargets.delete(c) }
+      for (const c of targets) { sendStatus(c, 'done'); statusTargets.delete(c) }
       busy = false
       if (queued || pending.length) { const next = queued || 'fast'; queued = null; void drain(next) }
     }
@@ -838,15 +843,22 @@ 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)
     } 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) => {