openvisio-agent 0.3.6 → 0.3.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openvisio-agent",
-  "version": "0.3.6",
+  "version": "0.3.7",
   "description": "Connect your coding agent (Claude Code) to an OpenVisio team — MCP tools + optional autonomy — in one command. No shell scripts.",
   "type": "module",
   "bin": {

package/src/watch.mjs

@@ -158,7 +158,6 @@ function loopBackendWs({ wsUrl, apiKey, identifier, claude, mcpConfig, workdir,
   let busy = false
   let queued = null // 'full' | 'fast' — a cycle requested while one was running
   let handle = null
-  let lastActivityAt = Date.now()
   // Context lines from the events themselves (the WS payload already carries the
   // channel + message / task), so the agent acts on THEM directly instead of
   // hoping poll_inbox re-surfaces the same item. Accumulated across coalesced
@@ -196,7 +195,6 @@ function loopBackendWs({ wsUrl, apiKey, identifier, claude, mcpConfig, workdir,
 
   function onEvent(k, d) {
     const raw = d && typeof d === 'object' ? d : {}
-    lastActivityAt = Date.now()
     if (k === 'task:assigned') {
       const t = raw.task || {}
       log('task:assigned ' + (t.id != null ? '#' + t.id + ' “' + (t.title || '') + '”' : ''))
@@ -207,34 +205,36 @@ function loopBackendWs({ wsUrl, apiKey, identifier, claude, mcpConfig, workdir,
       const msg = raw.message && typeof raw.message === 'object' ? raw.message : {}
       const text = String(msg.content || msg.body || msg.text || '').replace(/\s+/g, ' ').slice(0, 600)
       const who = senderName(msg)
-      log('agent:mention in channel ' + (cid != null ? cid : '?'))
+      // Reply IN THE SAME THREAD: parent is the thread root (the message's own id
+      // for a top-level mention, or its parent when the mention is itself a reply).
+      const mid = msg.id != null ? msg.id : (msg.message_id != null ? msg.message_id : null)
+      const parent = msg.parent_id != null ? msg.parent_id : (msg.parentId != null ? msg.parentId : null)
+      const threadRoot = parent != null ? parent : mid
+      log('agent:mention in channel ' + (cid != null ? cid : '?') + (threadRoot != null ? ' (thread ' + threadRoot + ')' : ''))
       const ctx = cid != null
-        ? `You were @mentioned in OpenVisio channel ${cid}${who ? ` by ${who}` : ''}: "${text}". Your FIRST action must be mcp__openvisio-team__post_message with channel_id ${cid}, a 1-3 sentence reply, and the agent_identifier + agent_api_key from the AUTH line above. You already have the message here — do NOT poll_inbox to find it. After posting, you MAY poll_inbox once for other .followUps.`
+        ? `You were @mentioned in OpenVisio channel ${cid}${who ? ` by "${who}"` : ''}: "${text}". Reply with mcp__openvisio-team__post_message as your FIRST action — 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.${who ? ` To @mention them back, write their EXACT full name "@${who}" (a mention only links when the name matches exactly — "@${who.split(' ')[0]}" alone will NOT).` : ''} You already have the message here — do NOT poll_inbox to find it.`
         : undefined
       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))
     } else {
       log('event ' + k)
     }
   }
 
-  // Follow-up sweep: thread replies you are part of get NO @mention (no WS event
-  // fires), so poll the inbox on a slow cadence to catch .followUps — but ONLY
-  // within a short window after real activity, so an idle agent costs nothing.
-  const FOLLOWUP_MS = 60_000
-  const ACTIVE_WINDOW_MS = 5 * 60_000
-  const sweep = setInterval(() => {
-    if (busy) return
-    if (Date.now() - lastActivityAt > ACTIVE_WINDOW_MS) return // idle → don't spend a cycle
-    void drain('fast', 'Follow-up check: call poll_inbox and reply to any NEW .followUps (thread replies you are part of, even with no @mention) or unanswered .mentions. If there is nothing new, do nothing and stop — do NOT post.')
-  }, FOLLOWUP_MS)
-
+  // NO polling — the WebSocket is the only trigger (task:assigned / agent:mention).
+  // A thread reply that @mentions the agent fires agent:mention and is handled
+  // in-thread above; genuinely un-mentioned thread activity has no WS event, so
+  // it's a backend concern (dispatch a thread event to participant agents), not a
+  // reason to poll.
   log('up — backend WS watcher on ' + wsUrl + (canCode ? ' [code: ' + workdir + ']' : ''))
   handle = connectAgentWs({ wsUrl, apiKey, identifier, onEvent, log })
 
   return new Promise(() => {
     // Run until killed. Tidy up the socket on termination so a restarting
     // service doesn't leak a half-open connection.
-    const bye = () => { clearInterval(sweep); try { handle && handle.close() } catch { /* noop */ } process.exit(0) }
+    const bye = () => { try { handle && handle.close() } catch { /* noop */ } process.exit(0) }
     process.on('SIGTERM', bye)
     process.on('SIGINT', bye)
   })