openvisio-agent 0.6.0 → 0.6.2

package/package.json

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

@@ -40,7 +40,12 @@ const CHAT_CHARTER = [
 ].join('\n')
 
 const CYCLE = CHAT_CHARTER + '\n\nRun one OpenVisio autonomy cycle: call poll_inbox and handle mentions + follow-ups. Reply in 1-3 sentences, @mention people by their EXACT full name, at most one reply per channel. Then stop.'
-const CYCLE_FAST = CHAT_CHARTER + '\n\nNew chat activity. If a specific mention FOR YOU is given above, reply to it with mcp__openvisio-team__post_message (once). Then call poll_inbox and look at .followUps — but reply ONLY to the ones actually directed at YOU (a question to you, or a reply to your own message); SKIP thread chatter aimed at someone else or another agent. Ignore .tasks/.claimable. AT MOST ONE reply per channel; answer several nudges together in ONE post; never repeat a reply you already sent. If asked for work you have no tool for, say so plainly and offer to file a ticket. Never invent progress. Reply in 1-3 sentences, no summary. Then stop.'
+const CYCLE_FAST = CHAT_CHARTER + '\n\n' + [
+  'New chat activity. Do EXACTLY ONE of these:',
+  '  • IF a specific mention/message FOR YOU is given above: reply to THAT ONE message exactly once with mcp__openvisio-team__post_message, then STOP. Do NOT call poll_inbox and do NOT answer anything else this cycle — you already have the message; polling would make you re-answer it and double-post.',
+  '  • IF NO specific mention is given above: call poll_inbox and reply only to items truly directed at YOU (a question to you, or a reply to your own message) — SKIP chatter aimed at someone else / another agent, ignore .tasks/.claimable, at most one reply per channel.',
+  'Post ONE message total for the thing you are answering — compose it fully, then send once. Never send a reply and then a "better" version; never repeat a reply you already sent. Be sure of your answer before sending. If asked for work you have no tool for, say so plainly and offer to file a ticket. 1-3 sentences, no summary. Then stop.',
+].join('\n')
 
 // ── CODE agents (--workdir given): full file + Bash + git/gh surface. ─────────
 // A stable "who you are / how you work" charter prepended to every code cycle.
@@ -75,10 +80,11 @@ const CODE_FULL = CODE_CHARTER + '\n\n' + [
 ].join('\n')
 
 const CODE_FAST = CODE_CHARTER + '\n\n' + [
-  'New chat activity. If a specific mention FOR YOU is given above, reply to it FIRST with mcp__openvisio-team__post_message (once) — before any Bash.',
-  'Then poll_inbox and handle .followUps ONLY when the reply is directed at YOU (asks you something, or responds to your own message) — SKIP thread chatter aimed at someone else / another agent. At most one reply per channel, and never repeat a reply you already sent.',
-  'If a message asks YOU for real CODE work, do the WHOLE job now — branch (checkout -B agent/<slug>), edit, run tests, commit, push your branch, open a PR (gh pr create), then reply with the PR link and @mention the requester by exact full name. Never push to main, never --force, never merge.',
-  'Do NOT promise and stop — finish and report in THIS cycle. Reply 1-3 sentences, no summary. Then stop.',
+  'New chat activity. Do EXACTLY ONE of these:',
+  '  • IF a specific mention/message FOR YOU is given above: reply to THAT ONE message exactly once with mcp__openvisio-team__post_message, then STOP. Do NOT call poll_inbox and do NOT answer anything else this cycle — polling would re-surface the same message and make you double-post.',
+  '  • IF NO specific mention is given above: call poll_inbox and reply only to items directed at YOU (asks you something, or responds to your own message) — SKIP chatter aimed at someone else / another agent; at most one reply per channel.',
+  'If the message asks YOU for real CODE work, do the WHOLE job now — branch (checkout -B agent/<slug>), edit, run tests, commit, push your branch, open a PR (gh pr create), then reply once with the PR link and @mention the requester by exact full name. Never push to main, never --force, never merge.',
+  'Post ONE message for the thing you are answering — send it once; never a reply then a "better" version; never repeat a reply you already sent; be sure before you send. Do NOT promise and stop — finish and report in THIS cycle. 1-3 sentences, no summary. Then stop.',
 ].join('\n')
 
 // ── Workspace-ethics cycles (both chat-only + code agents) ───────────────────
@@ -344,21 +350,24 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, mcpConfig, wor
     if (x === 'opus' || x === 'sonnet' || x === 'haiku') return x
     return /^claude-[a-z0-9.\-\[\]]+$/i.test(x) ? x : null
   }
+  // Optional target tier: "chat"/"lite" → chat cycles only, "code"/"full" → code
+  // cycles only, absent → both.
+  const parseTarget = (t) => t && /^(chat|lite)$/i.test(t) ? 'chat' : t && /^(code|full)$/i.test(t) ? 'code' : 'both'
   const parseModelCmd = (text) => {
-    // Slash form is unambiguous: "/model" (report) or "/model <name>".
-    const slash = /(?:^|\s)\/model\b(?:\s+(\S+))?/i.exec(text)
-    if (slash) { const a = slash[1]; if (!a) return { report: true }; const n = normalizeModel(a); return n ? { set: n } : { invalid: a } }
+    // Slash form is unambiguous: "/model" (report), "/model <name>", "/model chat <name>".
+    const slash = /(?:^|\s)\/model\b(?:\s+(chat|lite|code|full))?(?:\s+(\S+))?/i.exec(text)
+    if (slash) { const name = slash[2]; if (!name) return { report: true }; const n = normalizeModel(name); return n ? { set: n, target: parseTarget(slash[1]) } : { invalid: name } }
     // Natural language: only when a VALID model name is explicitly given after the
     // word "model" — so "use the model file in the repo" does NOT switch anything.
-    const nl = /\b(?:set|switch|change|use)\s+(?:the\s+|your\s+)?model\s+(?:to\s+)?(\S+)/i.exec(text)
-    if (nl) { const n = normalizeModel(nl[1]); if (n) return { set: n } }
+    const nl = /\b(?:set|switch|change|use)\s+(?:the\s+|your\s+)?(chat|code)?\s*model\s+(?:to\s+)?(\S+)/i.exec(text)
+    if (nl) { const n = normalizeModel(nl[2]); if (n) return { set: n, target: parseTarget(nl[1]) } }
     if (/\b(?:which|what)\s+model\s+(?:are\s+you|do\s+you|you\b)/i.test(text)) return { report: true }
     return null
   }
-  // Persist a model change so a restart / the background service keeps it.
-  const persistModel = (mdl) => {
+  // Persist the current model tiers so a restart / the background service keeps them.
+  const persistModel = () => {
     if (!slug) return
-    try { writeJson(configPath(slug), { ...(readConfig(slug) || {}), model: mdl, chatModel: mdl }, true) } catch { /* best-effort */ }
+    try { writeJson(configPath(slug), { ...(readConfig(slug) || {}), model: codeModel, chatModel: liteModel !== codeModel ? liteModel : '' }, true) } catch { /* best-effort */ }
   }
 
   function onEvent(k, d) {
@@ -393,32 +402,36 @@ function loopBackendWs({ wsUrl, apiKey, identifier, slug, claude, mcpConfig, wor
       const parent = msg.parent_id != null ? msg.parent_id : (msg.parentId != null ? msg.parentId : null)
       const threadRoot = parent != null ? parent : mid
       // De-dupe: the same mention re-delivered (reconnect replay / dup fan-out) must
-      // NOT trigger a second reply to the same message.
-      if (mid != null) {
-        if (seenMentions.has(mid)) { log('agent:mention (dup ' + mid + ') — skipped'); return }
-        seenMentions.add(mid); if (seenMentions.size > 500) seenMentions.clear()
-      }
+      // NOT trigger a second reply. Key by message id, or a channel+text signature
+      // when the payload carries no id.
+      const dedupeKey = mid != null ? 'id:' + mid : 'sig:' + (cid != null ? cid : '?') + '|' + text.slice(0, 100)
+      if (seenMentions.has(dedupeKey)) { log('agent:mention (dup) — skipped'); return }
+      seenMentions.add(dedupeKey); if (seenMentions.size > 500) seenMentions.clear()
       // Under-the-hood model control from chat (view / switch the model the agent runs).
       const mcmd = cid != null ? parseModelCmd(text) : null
       if (mcmd) {
         const thread = threadRoot != null ? `, parent_id ${threadRoot}` : ''
         if (mcmd.report) {
-          log('model query → ' + codeModel + (liteModel !== codeModel ? ' / chat ' + liteModel : ''))
-          void drain('fast', `An engineer asked which model you're running. Reply once in channel ${cid}${thread} (with your agent creds): "I'm currently running on ${codeModel}${liteModel !== codeModel ? ` (chat replies on ${liteModel})` : ''}." One line. Then stop.`)
+          log('model query → code ' + codeModel + ' / chat ' + liteModel)
+          void drain('fast', `An engineer asked which model you're running. Reply once in channel ${cid}${thread} (with your agent creds): "I'm on ${codeModel} for code work${liteModel !== codeModel ? ` and ${liteModel} for chat replies` : ' (and chat)'}." One line. Then stop.`)
         } else if (mcmd.invalid) {
           void drain('fast', `An engineer tried to switch your model to "${mcmd.invalid}", which isn't one you recognize. Reply once in channel ${cid}${thread} (with your agent creds): say you support "opus", "sonnet", "haiku", or a full "claude-…" id, and ask which they meant. One line. Then stop.`)
         } else {
-          const prev = codeModel
-          codeModel = mcmd.set; liteModel = mcmd.set
-          persistModel(mcmd.set)
-          log('model switched ' + prev + ' → ' + codeModel + (who ? ' (by ' + who + ')' : ''))
-          void drain('fast', `An engineer switched your underlying model to "${codeModel}" — it is now active for your next actions. Post ONE short confirmation in channel ${cid}${thread} (with your agent creds): e.g. "Switched to ${codeModel} — I'll run on it from here." Then stop.`)
+          const tgt = mcmd.target // 'chat' | 'code' | 'both'
+          const prev = `code ${codeModel}/chat ${liteModel}`
+          if (tgt === 'chat') liteModel = mcmd.set
+          else if (tgt === 'code') codeModel = mcmd.set
+          else { codeModel = mcmd.set; liteModel = mcmd.set }
+          persistModel()
+          const label = tgt === 'chat' ? 'chat model' : tgt === 'code' ? 'code model' : 'model'
+          log('model switched (' + tgt + ') ' + prev + ' → code ' + codeModel + '/chat ' + liteModel + (who ? ' (by ' + who + ')' : ''))
+          void drain('fast', `An engineer switched your ${label} to "${mcmd.set}" — active for your next ${tgt === 'chat' ? 'chat replies' : tgt === 'code' ? 'code cycles' : 'actions'}. Post ONE short confirmation in channel ${cid}${thread} (with your agent creds): e.g. "Switched my ${label} to ${mcmd.set} — I'll use it from here." Then stop.`)
         }
         return
       }
       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}". This mention is FOR YOU — reply once 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. FIRST read the recent messages in this thread: if you already answered this, or it turns out another agent was the one addressed, do NOT post. Be sure of your answer before sending — do not reply then correct yourself.${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.`
+        ? `You were @mentioned in OpenVisio channel ${cid}${who ? ` by "${who}"` : ''}: "${text}". This mention is FOR YOU. Send EXACTLY ONE reply with mcp__openvisio-team__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/"better" one. 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 — "@${who.split(' ')[0]}" alone will NOT).` : ''} You ALREADY have the message here — do NOT poll_inbox, and after your single reply, STOP.`
         : undefined
       void drain('fast', ctx)
     } else if (k === 'error') {