openvisio-agent 0.17.3 → 0.17.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openvisio-agent",
-  "version": "0.17.3",
+  "version": "0.17.5",
   "description": "Connect Claude Code, Codex, or OpenCode to an OpenVisio team — MCP tools + optional autonomy — in one command.",
   "type": "module",
   "bin": {

package/scripts/certify.mjs

@@ -28,6 +28,12 @@ const spec = readFileSync(join(repo, 'docs', 'CODEX_BYO_AGENT_SPEC.md'), 'utf8')
 const assertions = [
   ['task:assigned is handled', watcher.includes("k === 'task:assigned'")],
   ['task signals are verified with get_ticket', watcher.includes("callMcpTool('get_ticket'")],
+  ['review and testing handoffs do not restart work', watcher.includes('taskIsAwaitingReview(task, reviewIds)') && watcher.includes('taskIsAwaitingReview(ticket)')],
+  ['review handoff releases the task key for future rework', watcher.includes('seenTasks.delete(taskKey)') && watcher.includes('seenTasks.delete(key)')],
+  ['assigned coding completion is posted by the watcher', watcher.includes('announceTaskCompletion') && watcher.includes("callMcpTool('post_message', { project_id: projectId, channel_id: channelId, content: report.content })")],
+  ['completion requires review/done state and PR evidence', watcher.includes('buildTaskCompletionReport') && watcher.includes('completion report deferred')],
+  ['completion delivery survives reconnect and deduplicates', watcher.includes('pendingCompletionReports: [...pendingCompletionReports]') && watcher.includes('reportedCompletions: [...reportedCompletions]') && watcher.includes('reportedCompletions.has(report.key)')],
+  ['ticket comments cannot masquerade as channel completion', watcher.includes('didChannelMessage') && watcher.includes("mcpCalls.includes('post_message')")],
   ['two internal lanes exist', watcher.includes('work: createCycleRunner') && watcher.includes('reply: createCycleRunner')],
   ['work and reply activity targets are isolated', watcher.includes("laneStatusTargets = { work: new Set(), reply: new Set() }") && watcher.includes("emitLaneStatus('work', 'typing')") && watcher.includes("emitLaneStatus('reply', 'typing')")],
   ['assigned task activity resolves a project channel', watcher.includes("callMcpTool('list_channels'") && watcher.includes('projectStatusChannel(projectId)')],

package/src/events.mjs

@@ -34,6 +34,38 @@ export function taskIsCompleted(task, completedTypeIds = new Set()) {
   return /\b(?:done|complete|completed|closed|cancelled|canceled|archived|resolved)\b/i.test(state)
 }
 
+// A coding agent has handed work off once the board reaches review, testing, QA,
+// or approval. These are not "completed" states: a reviewer may still send the
+// ticket back to an actionable column. Treating them as settled for the watcher,
+// however, prevents reconnect reconciliation from reopening the same PR every
+// 30 minutes while it waits for a human.
+export function taskIsAwaitingReview(task, reviewTypeIds = new Set()) {
+  if (!task || typeof task !== 'object') return false
+  if (reviewTypeIds.has(Number(task.type_id ?? task.typeId ?? task.status_id ?? task.statusId))) return true
+  const state = [task.status, task.state, task.type?.name, task.task_type?.name].filter(Boolean).join(' ')
+  return /\b(?:review|test|testing|qa|quality\s+assurance|verification|approval)\b/i.test(state)
+}
+
+export function buildTaskCompletionReport(task, { projectId, agentName, fallbackText = '' } = {}) {
+  if (!task || typeof task !== 'object' || task.id == null) return null
+  if (!taskIsCompleted(task) && !taskIsAwaitingReview(task)) return null
+
+  const evidence = [task.description, fallbackText].filter(Boolean).join('\n')
+  const prUrl = /https:\/\/github\.com\/[^\s)\]}>]+\/pull\/\d+/i.exec(evidence)?.[0]?.replace(/[.,;:]+$/, '') || ''
+  if (!prUrl) return null
+
+  const creator = task.creator || task.created_by_user || task.createdByUser || {}
+  const requester = (`${creator.first_name || creator.firstName || ''} ${creator.last_name || creator.lastName || ''}`.trim() || creator.name || '').trim()
+  const title = String(task.title || 'Untitled task').replace(/\s+/g, ' ').trim().slice(0, 180)
+  const status = String(task.type?.name || task.task_type?.name || task.status || task.state || 'review').replace(/\s+/g, ' ').trim()
+  const verification = /\bVerification:\s*([^\n]{1,180})/i.exec(evidence)?.[1]?.replace(/\s+/g, ' ').trim().replace(/[.]+$/, '') || ''
+  const who = String(agentName || 'The agent').trim()
+  const mention = requester ? `@${requester} ` : ''
+  const content = `${mention}${who} finished ticket #${task.id} “${title}” and moved it to ${status}. PR: ${prUrl}.${verification ? ` Verification: ${verification}.` : ''}`
+  const revision = task.updated_at ?? task.updatedAt ?? prUrl
+  return { key: `${projectId ?? task.project_id ?? task.projectId ?? '?'}:${task.id}:${revision}`, content, prUrl }
+}
+
 export function agentStateRequest(backend, channelId, state, apiKey, identifier) {
   if (!['thinking', 'working', 'typing'].includes(state)) throw new Error('invalid agent state')
   const id = Number(channelId)

package/src/watch.mjs

@@ -10,7 +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 { agentStateRequest, codexPolicyBlock, requestTargetsLaterAgent, taskAgentId, taskFromEvent, taskIsCompleted } from './events.mjs'
+import { agentStateRequest, buildTaskCompletionReport, codexPolicyBlock, requestTargetsLaterAgent, taskAgentId, taskFromEvent, taskIsAwaitingReview, taskIsCompleted } 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
@@ -73,7 +73,7 @@ const CODE_CHARTER = [
   '',
   'WORK ETHIC — how a reliable teammate behaves (this is the difference between useful and ignored):',
   '  1. CLOSE THE LOOP in THIS cycle. Never say "I\'ll do X" and stop. If you commit to something, do it NOW — the human must never have to remind you to circle back.',
-  '  2. FINISH, then REPORT. Always update/move the ticket with update_ticket. Post a channel result ONLY when this cycle includes a specific source channel/thread from a human request, and reply in that thread. Backlog-only work has no implied audience: update the ticket with evidence and do not announce it in an unrelated channel.',
+  '  2. FINISH, then REPORT. Always update/move the ticket with update_ticket. Reply in a supplied human source thread when one exists. For backlog-assigned work, do not call post_message yourself: the watcher publishes one evidence-verified result in the project channel after the PR and ticket handoff are confirmed.',
   '  3. Be honest and specific. Never invent progress. If you are genuinely blocked (missing repo, unclear spec, a failing tool), say exactly what you need in one message — that IS closing the loop.',
   '  4. One reply per channel per cycle; answer several nudges together.',
   '',
@@ -89,7 +89,7 @@ const CODE_FULL = [
   '  3. CHANGE + VERIFY: Read/Edit/Write the files; run the tests or build if the repo has them.',
   '  4. COMMIT + PUSH YOUR BRANCH: git add -A && git commit -m "…"; then git push -u origin agent/<slug>. Only ever push your own agent/* branch. Never --force, never push to main/master, never merge.',
   '  5. RAISE A PR: gh pr create --fill --base <default-branch> --head agent/<slug> (a clear title + a body summarizing the change and how you verified it). Never gh pr merge.',
-  '  6. CLOSE THE LOOP: move/update the ticket with update_ticket. Reply with the summary + PR link only in a source thread explicitly supplied by the event. For backlog-only tickets, do not post_message or seek a channel to announce completion.',
+  '  6. CLOSE THE LOOP: move/update the ticket with update_ticket. Reply with the summary + PR link in a source thread explicitly supplied by the event. For backlog-only tickets, do not call post_message yourself; the watcher sends one verified project-channel completion message and deduplicates it across reconnects.',
   'Bash is for git / gh / tests / clone ONLY — never to hunt for credentials (they are given to you above).',
 ].join('\n')
 
@@ -345,7 +345,7 @@ function createCodexRunner({ mcpUrl, mcpHeaders, workdir, canCode, maxCycleMs, l
         ...(m ? ['--model', m] : []),
         ...(mcpOverride ? ['-c', mcpOverride] : []),
         full]
-      let child = null, done = false, didCode = false, didRepoMutation = false, didMessage = false, outputText = '', jsonlBuffer = '', stderrBuffer = ''
+      let child = null, done = false, didCode = false, didRepoMutation = false, didMessage = false, didChannelMessage = false, outputText = '', jsonlBuffer = '', stderrBuffer = ''
       let policyBlock = null
       const mcpCalls = new Set(), mcpErrors = new Set()
       let didMcpTaskRead = false, didMcpTaskUpdate = false
@@ -355,7 +355,7 @@ function createCodexRunner({ mcpUrl, mcpHeaders, workdir, canCode, maxCycleMs, l
         clearTimeout(timer)
         const calls = [...mcpCalls]
         log('codex MCP calls: ' + (calls.length ? calls.join(', ') : 'none') + (mcpErrors.size ? ' (failed: ' + [...mcpErrors].join(', ') + ')' : ''))
-        resolve({ ...o, didCode, didRepoMutation, didMessage, didMcpTaskRead, didMcpTaskUpdate, mcpCalls: calls, mcpErrors: [...mcpErrors], outputText, policyBlock })
+        resolve({ ...o, didCode, didRepoMutation, didMessage, didChannelMessage, didMcpTaskRead, didMcpTaskUpdate, mcpCalls: calls, mcpErrors: [...mcpErrors], outputText, policyBlock })
       }
       const inspectDiagnostic = (value) => {
         const s = String(value || '')
@@ -379,6 +379,7 @@ function createCodexRunner({ mcpUrl, mcpHeaders, workdir, canCode, maxCycleMs, l
             if (/^(?:get_ticket|list_tasks|list_task_types)$/.test(tool)) didMcpTaskRead = true
             if (tool === 'update_ticket') didMcpTaskUpdate = true
             if (/post_message|comment_ticket/.test(tool)) didMessage = true
+            if (/post_message/.test(tool)) didChannelMessage = true
             if (/fail|error/i.test(String(item.status || '')) || item.error) mcpErrors.add(tool)
           }
         } catch { /* non-JSON diagnostic */ }
@@ -442,8 +443,15 @@ function createCycleRunner({ claude, agent, mcpUrl, mcpHeaders, cfgKey, mcpConfi
   let sessionStartedAt = 0
   let resolveTurn = null
   let cycleTimer = null
+  let turnToolCalls = new Set()
   const clearCycleTimer = () => { if (cycleTimer) { clearTimeout(cycleTimer); cycleTimer = null } }
-  const settleTurn = (o) => { clearCycleTimer(); const r = resolveTurn; resolveTurn = null; if (r) r(o) }
+  const settleTurn = (o) => {
+    clearCycleTimer()
+    const r = resolveTurn
+    resolveTurn = null
+    const mcpCalls = [...turnToolCalls]
+    if (r) r({ ...o, mcpCalls, didChannelMessage: mcpCalls.includes('post_message') })
+  }
 
   // --debug: surface what the cycle actually does (tool calls, tool errors, text)
   // so a silent/hung cycle is diagnosable.
@@ -483,7 +491,10 @@ function createCycleRunner({ claude, agent, mcpUrl, mcpHeaders, cfgKey, mcpConfi
         if (debug) logEvent(o)
         // Surface tool calls (e.g. post_message) so the loop can emit a live status.
         if (onTool && o.type === 'assistant' && o.message && Array.isArray(o.message.content)) {
-          for (const b of o.message.content) if (b.type === 'tool_use' && b.name) { try { onTool(b.name) } catch { /* status is best-effort */ } }
+          for (const b of o.message.content) if (b.type === 'tool_use' && b.name) {
+            turnToolCalls.add(String(b.name).replace(/^mcp__openvisio-team__/, ''))
+            try { onTool(b.name) } catch { /* status is best-effort */ }
+          }
         }
         if (o.type === 'result') {
           log('cycle done (' + (o.subtype || 'ok') + (o.is_error ? ' · ERROR' : '') + ')')
@@ -520,6 +531,7 @@ function createCycleRunner({ claude, agent, mcpUrl, mcpHeaders, cfgKey, mcpConfi
       }
       ensureSession()
       turnsThisSession++
+      turnToolCalls = new Set()
       resolveTurn = resolve
       // Backstop: abandon a cycle that never returns a result so `busy` is released
       // and queued mentions can proceed.
@@ -608,6 +620,11 @@ function loopBackendWs({ backend, wsUrl, apiKey, identifier, slug, claude, agent
   try { replayState = JSON.parse(readFileSync(replayPath, 'utf8')) } catch { /* first run */ }
   const seenMentions = new Set(Array.isArray(replayState.seenMentions) ? replayState.seenMentions : [])
   const seenActivities = new Set(Array.isArray(replayState.seenActivities) ? replayState.seenActivities : [])
+  // Completion delivery is runtime-owned for assigned coding work. Persist both
+  // pending and delivered keys so a reconnect can finish a missed notification
+  // without re-running the model or posting the same result twice.
+  const pendingCompletionReports = new Set(Array.isArray(replayState.pendingCompletionReports) ? replayState.pendingCompletionReports : [])
+  const reportedCompletions = new Set(Array.isArray(replayState.reportedCompletions) ? replayState.reportedCompletions : [])
   // A policy-blocked task stays paused across reconnects. It is released only
   // after the ticket itself carries explicit authorization or is completed/
   // unassigned. This prevents a 30-minute reconciliation retry from repeatedly
@@ -615,7 +632,15 @@ function loopBackendWs({ backend, wsUrl, apiKey, identifier, slug, claude, agent
   const blockedTasks = new Set(Array.isArray(replayState.blockedTasks) ? replayState.blockedTasks : [])
   const trimSeen = (set) => { while (set.size > 500) set.delete(set.values().next().value) }
   const persistReplay = () => {
-    try { writeJson(replayPath, { seenMentions: [...seenMentions], seenActivities: [...seenActivities], blockedTasks: [...blockedTasks] }, true) } catch { /* best-effort */ }
+    try {
+      writeJson(replayPath, {
+        seenMentions: [...seenMentions],
+        seenActivities: [...seenActivities],
+        blockedTasks: [...blockedTasks],
+        pendingCompletionReports: [...pendingCompletionReports],
+        reportedCompletions: [...reportedCompletions],
+      }, true)
+    } catch { /* best-effort */ }
   }
   // Context lines from the events themselves (the WS payload already carries the
   // channel + message / task), so the agent acts on THEM directly instead of
@@ -626,6 +651,7 @@ function loopBackendWs({ backend, wsUrl, apiKey, identifier, slug, claude, agent
   let lastTaskTriggeredAt = 0
   let lastInboxSignature = ''
   let selfAgentId = null
+  let selfAgentName = slug
   let mcpSessionId = ''
   let mcpRpcId = 0
 
@@ -650,7 +676,7 @@ function loopBackendWs({ backend, wsUrl, apiKey, identifier, slug, claude, agent
   }
   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.17.3' } } }, false)
+    const res = await mcpPost({ jsonrpc: '2.0', id: ++mcpRpcId, method: 'initialize', params: { protocolVersion: '2025-03-26', capabilities: {}, clientInfo: { name: 'openvisio-agent', version: '0.17.5' } } }, false)
     if (!res.ok) throw new Error('MCP initialize HTTP ' + res.status)
     await mcpPayload(res)
     mcpSessionId = res.headers.get('mcp-session-id') || ''
@@ -701,6 +727,39 @@ function loopBackendWs({ backend, wsUrl, apiKey, identifier, slug, claude, agent
     }
   }
 
+  const announceTaskCompletion = async (taskRef, result = {}) => {
+    const projectId = Number(taskRef?.projectId)
+    const ticketId = Number(taskRef?.ticketId)
+    if (!Number.isFinite(projectId) || !Number.isFinite(ticketId)) return false
+    const taskKey = `${projectId}:${ticketId}`
+    const current = toolData(await callMcpTool('get_ticket', { project_id: projectId, ticket_id: ticketId }))
+    const ticket = current.ticket ?? current.task ?? current
+    const report = buildTaskCompletionReport(ticket, { projectId, agentName: selfAgentName, fallbackText: result.outputText })
+    if (!report) return false
+    if (reportedCompletions.has(report.key)) {
+      pendingCompletionReports.delete(taskKey)
+      persistReplay()
+      return true
+    }
+    // A model may already have replied when the task originated in a real source
+    // thread. Mark that verified completion as delivered instead of posting a
+    // second top-level copy.
+    if (result.didChannelMessage) {
+      reportedCompletions.add(report.key); trimSeen(reportedCompletions)
+      pendingCompletionReports.delete(taskKey); persistReplay()
+      log('completion for ticket #' + ticketId + ' was already posted by the work cycle')
+      return true
+    }
+    const channelId = Number.isFinite(Number(taskRef.channelId)) ? Number(taskRef.channelId) : await projectStatusChannel(projectId)
+    if (!Number.isFinite(channelId)) return false
+    sendStatus(channelId, 'typing')
+    await callMcpTool('post_message', { project_id: projectId, channel_id: channelId, content: report.content })
+    reportedCompletions.add(report.key); trimSeen(reportedCompletions)
+    pendingCompletionReports.delete(taskKey); persistReplay()
+    log('posted verified completion for ticket #' + ticketId + ' in channel ' + channelId)
+    return true
+  }
+
   const publishBlocker = async ({ prompt, taskRef, notice, ticketNotice = notice, pause = false }) => {
     const sentenceTask = /ticket\s+#?(\d+).*?project\s+(\d+)/i.exec(prompt)
     const jsonTask = /"id"\s*:\s*(\d+)[\s\S]{0,300}?"projectId"\s*:\s*(\d+)/i.exec(prompt)
@@ -773,6 +832,7 @@ function loopBackendWs({ backend, wsUrl, apiKey, identifier, slug, claude, agent
       const self = agents.find((a) => String(a.identifier || a.slug || '') === identifier)
       if (!self?.id) throw new Error('list_agents did not return this BYO agent')
       selfAgentId = Number(self.id)
+      selfAgentName = String(self.name || selfAgentName)
       const projectsData = toolData(await callMcpTool('list_projects'))
       const projects = Array.isArray(projectsData.projects) ? projectsData.projects : []
       const assigned = []
@@ -783,21 +843,35 @@ function loopBackendWs({ backend, wsUrl, apiKey, identifier, slug, claude, agent
           callMcpTool('list_task_types', { project_id: project.id }).then(toolData),
           callMcpTool('list_activity', { project_id: project.id }).then(toolData),
         ])
-        const doneIds = new Set((Array.isArray(typesData.types) ? typesData.types : Array.isArray(typesData.task_types) ? typesData.task_types : Array.isArray(typesData.taskTypes) ? typesData.taskTypes : []).filter((t) => /\b(?:done|complete|completed|closed|cancelled|canceled|archived|resolved)\b/i.test(String(t.name || ''))).map((t) => Number(t.id)))
+        const taskTypes = Array.isArray(typesData.types) ? typesData.types : Array.isArray(typesData.task_types) ? typesData.task_types : Array.isArray(typesData.taskTypes) ? typesData.taskTypes : []
+        const doneIds = new Set(taskTypes.filter((t) => /\b(?:done|complete|completed|closed|cancelled|canceled|archived|resolved)\b/i.test(String(t.name || ''))).map((t) => Number(t.id)))
+        const reviewIds = new Set(taskTypes.filter((t) => /\b(?:review|test|testing|qa|quality\s+assurance|verification|approval)\b/i.test(String(t.name || ''))).map((t) => Number(t.id)))
         for (const task of Array.isArray(tasksData.tasks) ? tasksData.tasks : []) {
           const taskAgentId = Number(task.agent_id ?? task.agentId ?? task.agent?.id)
           const taskIdent = String(task.agent?.identifier ?? task.agent?.slug ?? '')
-          if (!taskIsCompleted(task, doneIds) && (taskAgentId === Number(self.id) || taskIdent === identifier)) {
-            const taskKey = `${project.id}:${task.id}`
-            if (blockedTasks.has(taskKey)) {
-              const approvalText = [task.title, task.description].filter(Boolean).join(' ')
-              if (/\b(?:i\s+)?(?:explicitly\s+)?(?:approve|authorize)\b[\s\S]{0,240}\b(?:git\s+push|push(?:ing)?\s+(?:the\s+)?(?:branch|code)|github|remote)\b/i.test(approvalText)) {
-                blockedTasks.delete(taskKey); seenTasks.delete(taskKey); persistReplay()
-                log('backlog ticket #' + task.id + ' now contains explicit push authorization — resuming')
-              } else continue
+          if (taskAgentId !== Number(self.id) && taskIdent !== identifier) continue
+          const taskKey = `${project.id}:${task.id}`
+          if (taskIsCompleted(task, doneIds) || taskIsAwaitingReview(task, reviewIds)) {
+            if (pendingCompletionReports.has(taskKey)) {
+              const activityChannel = await projectStatusChannel(project.id)
+              try { await announceTaskCompletion({ projectId: project.id, ticketId: task.id, channelId: activityChannel }) }
+              catch (e) { log('completion report retry failed for ticket #' + task.id + ': ' + (e?.message || e)) }
             }
-            assigned.push({ id: task.id, projectId: project.id, project: project.name, title: task.title, priority: task.priority, typeId: task.type_id ?? task.typeId, updatedAt: task.updated_at ?? task.updatedAt })
+            if (blockedTasks.delete(taskKey)) persistReplay()
+            // Release the in-flight de-dupe key at handoff. If a reviewer moves
+            // the ticket back to an actionable column, that update must start a
+            // fresh work cycle.
+            seenTasks.delete(taskKey)
+            continue
+          }
+          if (blockedTasks.has(taskKey)) {
+            const approvalText = [task.title, task.description].filter(Boolean).join(' ')
+            if (/\b(?:i\s+)?(?:explicitly\s+)?(?:approve|authorize)\b[\s\S]{0,240}\b(?:git\s+push|push(?:ing)?\s+(?:the\s+)?(?:branch|code)|github|remote)\b/i.test(approvalText)) {
+              blockedTasks.delete(taskKey); seenTasks.delete(taskKey); persistReplay()
+              log('backlog ticket #' + task.id + ' now contains explicit push authorization — resuming')
+            } else continue
           }
+          assigned.push({ id: task.id, projectId: project.id, project: project.name, title: task.title, priority: task.priority, typeId: task.type_id ?? task.typeId, updatedAt: task.updated_at ?? task.updatedAt })
         }
         const activities = Array.isArray(activityData.activities) ? activityData.activities : Array.isArray(activityData.activity) ? activityData.activity : []
         const mentionNeedles = [self.name, self.identifier, self.slug, identifier].filter(Boolean).map((s) => '@' + String(s).toLowerCase())
@@ -823,7 +897,8 @@ function loopBackendWs({ backend, wsUrl, apiKey, identifier, slug, claude, agent
           const next = [...assigned].sort((a, b) => (priorityRank[a.priority] ?? 9) - (priorityRank[b.priority] ?? 9) || String(a.updatedAt || '').localeCompare(String(b.updatedAt || '')))[0]
           log('backlog reconciliation found ' + assigned.length + ' assigned task(s); queueing only ticket #' + next.id)
           const activityChannel = await projectStatusChannel(next.projectId)
-          void drain('full', `Backlog reconciliation verified this open ticket is assigned to YOU: ${JSON.stringify(next)}. Process this ONE ticket only. This is backlog-only work with NO source message or requester thread: do NOT post_message and do not announce progress or completion in any channel. Call get_ticket, use list_task_types and update_ticket to move it active, complete and verify the work, open the PR, then update the ticket with evidence and move it to done when appropriate.`, activityChannel == null ? [] : [activityChannel], { projectId: next.projectId, ticketId: next.id, channelId: activityChannel })
+          pendingCompletionReports.add(`${next.projectId}:${next.id}`); trimSeen(pendingCompletionReports); persistReplay()
+          void drain('full', `Backlog reconciliation verified this open ticket is assigned to YOU: ${JSON.stringify(next)}. Process this ONE ticket only. This is backlog-only work with NO source message or requester thread: do NOT call post_message yourself. Call get_ticket, use list_task_types and update_ticket to move it active, complete and verify the work, open the PR, then update the ticket with evidence and move it to review or done when appropriate. The watcher will publish exactly one verified completion result to the project channel.`, activityChannel == null ? [] : [activityChannel], { projectId: next.projectId, ticketId: next.id, channelId: activityChannel })
         }
       }
       const inboxSignature = JSON.stringify(mentionActivity.slice(-20)).slice(0, 4000)
@@ -870,6 +945,7 @@ function loopBackendWs({ backend, wsUrl, apiKey, identifier, slug, claude, agent
     const heartbeat = targets.length ? setInterval(() => emitLaneStatus(laneName, 'working'), 20_000) : null
     try {
       const result = await runners[laneName].runCycle(prompt, useModel)
+      let completionResult = result
       if (agent === 'codex' && result?.subtype === 'blocked' && result?.policyBlock) {
         log('WORK_CYCLE_BLOCKED authorization required; pausing this ticket and publishing an action-required notice')
         try { await reportPolicyBlock(prompt, activeTaskRef, result.policyBlock) }
@@ -916,6 +992,16 @@ function loopBackendWs({ backend, wsUrl, apiKey, identifier, slug, claude, agent
           }
           lastTaskSignature = ''
           log('WORK_CYCLE_FAILED evidence gate still incomplete after one recovery; ticket retained for retry')
+          return
+        }
+        completionResult = recovery
+      }
+      if (kind === 'full' && activeTaskRef) {
+        try {
+          const delivered = await announceTaskCompletion(activeTaskRef, completionResult)
+          if (!delivered) log('completion report deferred for ticket #' + activeTaskRef.ticketId + '; waiting for verified review/done state and PR evidence')
+        } catch (e) {
+          log('completion report failed for ticket #' + activeTaskRef.ticketId + ': ' + (e?.message || e) + '; retained for reconnect retry')
         }
       }
     } finally {
@@ -979,6 +1065,7 @@ function loopBackendWs({ backend, wsUrl, apiKey, identifier, slug, claude, agent
         const agentsData = toolData(await callMcpTool('list_agents'))
         const self = (Array.isArray(agentsData.agents) ? agentsData.agents : []).find((a) => String(a.identifier || a.slug || '') === identifier)
         selfAgentId = self?.id != null ? Number(self.id) : null
+        selfAgentName = String(self?.name || selfAgentName)
       }
       const ticketData = toolData(await callMcpTool('get_ticket', { project_id: projectId, ticket_id: ticketId }))
       const ticket = ticketData.ticket ?? ticketData.task ?? ticketData
@@ -986,8 +1073,17 @@ function loopBackendWs({ backend, wsUrl, apiKey, identifier, slug, claude, agent
       const assignedIdentifier = String(ticket.agent?.identifier ?? ticket.assigned_agent?.identifier ?? '')
       const belongsToSelf = (selfAgentId != null && assignedId === selfAgentId) || assignedIdentifier === identifier
       const key = `${projectId}:${ticketId}`
-      if (!belongsToSelf) { blockedTasks.delete(key); persistReplay(); seenTasks.delete(key); log(kind + ' ticket #' + ticketId + ' is not assigned to this agent — ignored'); return }
-      if (taskIsCompleted(ticket)) { blockedTasks.delete(key); persistReplay(); seenTasks.add(key); log(kind + ' ticket #' + ticketId + ' is already complete — ignored'); return }
+      if (!belongsToSelf) { blockedTasks.delete(key); pendingCompletionReports.delete(key); persistReplay(); seenTasks.delete(key); log(kind + ' ticket #' + ticketId + ' is not assigned to this agent — ignored'); return }
+      if (taskIsCompleted(ticket) || taskIsAwaitingReview(ticket)) {
+        if (pendingCompletionReports.has(key)) {
+          const activityChannel = await projectStatusChannel(projectId)
+          try { await announceTaskCompletion({ projectId, ticketId, channelId: activityChannel }) }
+          catch (e) { log('completion report failed for ticket #' + ticketId + ': ' + (e?.message || e) + '; retained for reconnect retry') }
+        }
+        blockedTasks.delete(key); persistReplay(); seenTasks.delete(key)
+        log(kind + ' ticket #' + ticketId + ' is already complete or awaiting review — ignored')
+        return
+      }
       if (blockedTasks.has(key)) {
         const approvalText = [ticket.title, ticket.description].filter(Boolean).join(' ')
         if (/\b(?:i\s+)?(?:explicitly\s+)?(?:approve|authorize)\b[\s\S]{0,240}\b(?:git\s+push|push(?:ing)?\s+(?:the\s+)?(?:branch|code)|github|remote)\b/i.test(approvalText)) {
@@ -1006,7 +1102,8 @@ function loopBackendWs({ backend, wsUrl, apiKey, identifier, slug, claude, agent
       log(kind + ' verified ticket #' + ticketId + ' “' + title + '” -> ' + cycleKind + ' lane')
       const activityChannel = await projectStatusChannel(projectId)
       if (activityChannel != null) sendStatus(activityChannel, 'thinking')
-      void drain(cycleKind, `Authoritative get_ticket verification confirms ticket #${ticketId} in project ${projectId} is open and assigned to YOU: ${JSON.stringify({ id: ticketId, projectId, title, description: ticket.description, priority: ticket.priority, typeId: ticket.type_id ?? ticket.typeId })}. This assignment has no source channel: do not post_message. Use list_task_types and update_ticket to move it active, complete and verify the work, open the PR when applicable, then update/move the ticket with evidence. Never announce it in a channel.`, activityChannel == null ? [] : [activityChannel], { projectId, ticketId, channelId: activityChannel })
+      if (cycleKind === 'full') { pendingCompletionReports.add(key); trimSeen(pendingCompletionReports); persistReplay() }
+      void drain(cycleKind, `Authoritative get_ticket verification confirms ticket #${ticketId} in project ${projectId} is open and assigned to YOU: ${JSON.stringify({ id: ticketId, projectId, title, description: ticket.description, priority: ticket.priority, typeId: ticket.type_id ?? ticket.typeId })}. This assignment has no source thread: do not call post_message yourself. Use list_task_types and update_ticket to move it active, complete and verify the work, open the PR when applicable, then update/move the ticket with evidence. For coding work, the watcher will publish exactly one verified completion result in the project channel.`, activityChannel == null ? [] : [activityChannel], { projectId, ticketId, channelId: activityChannel })
     } catch (e) {
       log(kind + ' ticket verification failed for #' + ticketId + ': ' + (e?.message || e))
     }