openvisio-agent 0.20.0 → 0.21.1

package/src/watch.mjs

@@ -11,7 +11,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 { agentAddedByName, blockingReplyMcpErrors, buildTaskCompletionReport, claudeEventEvidence, classifyConversationTarget, combineRuntimeWorkEvidence, conversationAsksPendingTickets, conversationNeedsCode, failedTaskRevisionIsCurrent, mentionDedupeKeys, messageParentId, messageSenderIsSelf, missingRuntimeWorkEvidence, normalizeRenderedMessageText, renderedAgentMessages, taskBelongsToAgent, taskFromEvent, taskIsAwaitingReview, taskIsCompleted, taskIsCoordinationOnly, taskRevision, ticketDisplaySlug } from './events.mjs'
+import { agentAddedByName, blockingReplyMcpErrors, buildTaskCompletionReport, claudeEventEvidence, classifyConversationTarget, combineRuntimeWorkEvidence, conversationAsksPendingTickets, conversationCreatesTicket, conversationNeedsCode, failedTaskRevisionIsCurrent, mentionDedupeKeys, messageParentId, messageSenderIsSelf, missingRuntimeWorkEvidence, normalizeRenderedMessageText, renderedAgentMessages, taskAgentId, taskAgentIdentifier, taskBelongsToAgent, taskFromEvent, taskIsAwaitingReview, taskIsCompleted, taskIsCoordinationOnly, taskRevision, ticketDisplaySlug } from './events.mjs'
 import { createMastraMemory } from './memory.mjs'
 import { repositoryHasPrPushAuthorization, configurePrPublishing } from './pr-push.mjs'
 import { createMcpHttpClient } from './mcp-http.mjs'
@@ -27,6 +27,8 @@ import { modelProcessOptions, stopModelProcess } from './process-lifecycle.mjs'
 import { createMastraAcpRunner } from './mastra-harness.mjs'
 import { claudeFallbackModels, resolveClaudeModel } from './model-selection.mjs'
 import { createAgentJournal } from './agent-journal.mjs'
+import { readModelSettings } from './model-settings.mjs'
+import { startStudioInBackground } from './studio-cli.mjs'
 
 function runtimeJournal(options, supplied) {
   let journal
@@ -320,6 +322,7 @@ export async function runWatch({ flags }) {
     if (!apiKey || !identifier) fail('No saved backend credentials for that agent.\n  Run `openvisio-agent connect --backend …` first, or pass --key and --id.')
     assertWebSocket(fail)
     if (flags.install) return installService({ slug: slug || 'openvisio', claude, mcpConfig, workdir })
+    startStudioInBackground()
     return loopBackendWs({ backend, wsUrl, apiKey, identifier, slug: slug || 'openvisio', claude, agent, mcpConfig, mcpUrl, workdir, model, chatModel, debug: !!flags.debug })
   }
 
@@ -593,9 +596,9 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
   // The Mastra bridge authenticates per-CALL: every openvisio-team tool needs
   // agent_identifier + agent_api_key as arguments. Hand them over up front.
   const authRule = agent !== 'claude'
-    ? 'AUTH: the local OpenVisio MCP bridge injects authentication outside the model. Never add, request, print, or search for agent credentials in tool arguments.'
+    ? 'AUTH: the local OpenVisio MCP bridge injects authentication outside the model. Use only this session’s openvisio-team-watcher server for OpenVisio actions; other installed OpenVisio servers do not receive this bridge’s credentials. If a discovered action requires agent_api_key or agent_identifier, it is not the authenticated watcher action: search for the openvisio-team-watcher version. Never add, request, print, or search for agent credentials in tool arguments.'
     : `AUTH: the openvisio-team tools REQUIRE two arguments on EVERY call — agent_identifier: "${identifier}" and agent_api_key: "${apiKey}". Include BOTH on every openvisio-team tool call. The credentials are given here; do NOT hunt for them.`
-  const credNote = `${authRule} Codex may defer MCP actions instead of placing them in the initial tool list. If a native tool_search is available and an expected openvisio-team action is not initially visible, use tool_search to find that exact action, then call it; an initial-list miss is NOT evidence that the server is disconnected. Do not ask a teammate to reconnect or provide a slug until tool_search and the applicable live lookup have actually failed. On backend MCP, discover and update work with list_agents, list_projects, list_tasks, list_task_types, get_ticket, update_ticket, list_channels, list_message_thread, and list_activity as applicable. Ticket comments are optional: discover an advertised ticket-comment action and follow its input schema; never invent a tool name or call an unadvertised action. get_marching_orders, poll_inbox, get_resource, list_mcp_resources, and list_mcp_resource_templates are NOT team-action tools here; never call them. Tools may be namespaced — call whichever names actually appear. Bash/git/gh ARE for code work; this rule only forbids searching for keys.`
+  const credNote = `${authRule} Codex may defer MCP actions instead of placing them in the initial tool list. If a native tool_search is available and an expected openvisio-team action is not initially visible, use tool_search to find that exact action, then call it; an initial-list miss is NOT evidence that the server is disconnected. Do not ask a teammate to reconnect or provide a slug until tool_search and the applicable live lookup have actually failed. On backend MCP, discover and update work with list_agents, list_projects, list_tasks, list_task_types, get_ticket, update_ticket, list_channels, list_message_thread, and list_activity as applicable. Ticket comments are optional: discover an advertised ticket-comment action and follow its input schema; never invent a tool name or call an unadvertised action. get_marching_orders, poll_inbox, get_resource, list_mcp_resources, and list_mcp_resource_templates are NOT team-action tools here; never call them. Tools may be namespaced; use advertised actions on the authenticated session server only. Bash/git/gh ARE for code work; this rule only forbids searching for keys.`
   // Keep the static charter stable so provider prompt caching can reuse it.
   // Actual cache hits and billing depend on the runtime and model provider.
   const systemPrompt = (canCode ? CODE_CHARTER : CHAT_CHARTER) + '\n\n' + credNote
@@ -630,15 +633,29 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
     ? '\n\nCODEX PR DELIVERY: when the repository exists in the local workspace, use that clone for branch creation, edits, tests, and commits; do not inspect or mutate it through linked-codebase MCP tools. To publish the local agent/* branch, run `openvisio-agent push-pr-branch` from the repository, then open the PR with `gh pr create`. The helper can only push HEAD to the matching agent/* branch on the exact authorized origin. If it reports OPENVISIO_PR_PUSH_AUTH_REQUIRED, do not retry or route around it. PR publishing is enabled during connection setup for the coding workspace and its worktrees. If authorization is missing outside that workspace, report the exact repository configuration issue once; never tell the user to restart the watcher. Use list_codebases/create_codebase_branch/create_codebase_commit/create_pull_request only as a fallback when the repository cannot be obtained locally.'
     : ''
   const backendToolRule = 'BACKEND MCP RULE: the event and watcher already provide the work source. Never call get_marching_orders, poll_inbox, get_resource, list_mcp_resources, or list_mcp_resource_templates; they are not backend team actions. Codex can defer MCP actions: native tool_search is allowed and must be used to find an expected openvisio-team action that is absent from the initial list. An initial-list miss does not mean the MCP server is unavailable. For live discovery use list_agents, list_projects, list_tasks, list_task_types, list_activity, get_ticket, and list_channels as applicable. For ticket comments, discover an advertised action and follow its input schema; never invent a tool name or call an unadvertised action.'
-  const fullPrompt = (canCode ? CODE_FULL + codexPushGuide : 'Handle the supplied verified backend ticket with the available OpenVisio tools. Update or comment on the ticket as requested, do not claim repository work in chat-only mode, and stop after the verified action.') + '\n\n' + backendToolRule
-  const fastPrompt = (canCode ? CODE_FAST : CYCLE_FAST) + '\n\n' + backendToolRule
-  const coordinatePrompt = COORDINATE + '\n\n' + backendToolRule
-  const guardedReplyPrompt = `WATCHER-DELIVERED REPLY: the watcher has already verified that the current source message is addressed to you. Your current OpenVisio agent identifier is "${identifier}"; do not call list_agents merely to rediscover yourself. Do not call post_message, relay inbox tools, or MCP resource APIs. OpenVisio team-state tools are allowed: when the answer depends on live projects, assignments, tickets, or status, use list_projects/list_tasks/get_ticket as needed and never ask the teammate for a slug that those tools can resolve. A failed read-only discovery call is not a completed action and must not be described as an intervention-level blocker; continue with available live data or state the narrow fact you could not verify. Return only one natural, context-specific reply of 1-3 sentences. Do not echo the request, announce a plan, or add a generic acknowledgement. If the supplied context says the work was completed, lead with the verified result; if it is blocked, name only the real blocker and next action.` + '\n\n' + backendToolRule
+  const studioGuide = `\n\nAGENT STUDIO: When a teammate asks to inspect agent activity or change model settings, offer [Open Agent Studio](http://127.0.0.1:4317/#agent=${encodeURIComponent(identifier)}&settings=1). The app renders this link as a button. Studio manages agents on the teammate’s own computer; it does not connect to another teammate’s machine. Prefer this UI over asking people to run model-setting CLI commands.`
+  const fullPrompt = (canCode ? CODE_FULL + codexPushGuide : 'Handle the supplied verified backend ticket with the available OpenVisio tools. Update or comment on the ticket as requested, do not claim repository work in chat-only mode, and stop after the verified action.') + '\n\n' + backendToolRule + studioGuide
+  const fastPrompt = (canCode ? CODE_FAST : CYCLE_FAST) + '\n\n' + backendToolRule + studioGuide
+  const coordinatePrompt = COORDINATE + '\n\n' + backendToolRule + studioGuide
+  const guardedReplyPrompt = `WATCHER-DELIVERED REPLY: the watcher has already verified that the current source message is addressed to you. Your current OpenVisio agent identifier is "${identifier}"; do not call list_agents merely to rediscover yourself. Do not call post_message, relay inbox tools, or MCP resource APIs. OpenVisio team-state tools are allowed: when the answer depends on live projects, assignments, tickets, or status, use list_projects/list_tasks/get_ticket as needed and never ask the teammate for a slug that those tools can resolve. A failed read-only discovery call is not a completed action and must not be described as an intervention-level blocker; continue with available live data or state the narrow fact you could not verify. Return only one natural, context-specific reply of 1-3 sentences. Do not echo the request, announce a plan, or add a generic acknowledgement. If the supplied context says the work was completed, lead with the verified result; if it is blocked, name only the real blocker and next action.` + '\n\n' + backendToolRule + studioGuide
   // Live model state — changeable at runtime by the in-chat `/model` command.
   // codeModel drives full/sweep cycles; chatModel (if set) the lighter fast/intro
   // ones, so routine chatter can run cheaper than real code work.
   let codeModel = model
   let liteModel = chatModel || model
+  let modelSettingsRevision = ''
+  const refreshModelSettings = () => {
+    try {
+      const settings = readModelSettings(stateDir, slug)
+      if (!settings.revision || settings.revision === modelSettingsRevision) return
+      codeModel = settings.model
+      liteModel = settings.chatModel || settings.model
+      modelSettingsRevision = settings.revision
+      emit('models.updated', { model: codeModel, chatModel: liteModel, modelSettingsRevision })
+      log('Model settings applied from Agent Studio; new requests will use the selected models')
+    } catch { /* absent, legacy, or concurrently replaced configuration */ }
+  }
+  refreshModelSettings()
 
   const queues = {
     work: createCycleQueue({
@@ -808,7 +825,7 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
   const selfAliases = new Set([slug, identifier].map((value) => String(value || '').toLowerCase()).filter(Boolean))
   const rememberSelfAgent = (self) => {
     if (!self || typeof self !== 'object' || String(self.identifier || self.slug || '') !== identifier) return
-    if (self.id != null) selfAgentId = Number(self.id)
+    if (validBackendId(self.id)) selfAgentId = Number(self.id)
     for (const alias of [self.name, self.identifier, self.slug]) if (alias) selfAliases.add(String(alias).toLowerCase())
     const addedBy = agentAddedByName(self)
     if (addedBy) completionRecipientName = addedBy
@@ -833,15 +850,24 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
     }))
   }
   let selfRead = null
+  let selfReadAfter = 0
   const ensureSelfAgent = async () => {
     if (selfAgentId != null) return
+    if (!selfRead && Date.now() < selfReadAfter) return
     if (!selfRead) selfRead = callMcpTool('list_agents').then(toolData).then((data) => {
-      const self = (Array.isArray(data.agents) ? data.agents : []).find((row) => String(row.identifier || row.slug || '') === identifier)
-      if (!self?.id) throw new Error('list_agents did not return this BYO agent')
+      const self = (Array.isArray(data.agents) ? data.agents : []).find((row) => String(row?.identifier || row?.slug || '') === identifier)
       rememberSelfAgent(self)
-    }).finally(() => { selfRead = null })
+    }).catch((error) => {
+      log('Agent profile unavailable; continuing with task ownership and configured aliases: ' + (error?.message || error))
+    }).finally(() => { selfRead = null; selfReadAfter = Date.now() + 60_000 })
     await selfRead
   }
+  const resolveTaskOwner = async (task) => {
+    // An identifier on the authoritative task is sufficient. Older backends
+    // emit only a numeric assignee, which needs the optional profile lookup.
+    if (selfAgentId == null && !taskAgentIdentifier(task) && taskAgentId(task) != null) await ensureSelfAgent()
+    return taskBelongsToAgent(task, { id: selfAgentId, identifier })
+  }
   let mcpToolNames = null
   let mcpToolDiscoveryPromise = null
   let mcpToolDiscoveryWarned = false
@@ -901,12 +927,6 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
   }
 
   const loadPendingTickets = async () => {
-    if (selfAgentId == null) {
-      const agentsData = toolData(await callMcpReadWithRetry('list_agents'))
-      const self = (Array.isArray(agentsData.agents) ? agentsData.agents : []).find((a) => String(a.identifier || a.slug || '') === identifier)
-      if (!self?.id) throw new Error('list_agents did not return this BYO agent')
-      rememberSelfAgent(self)
-    }
     const projectsData = toolData(await callMcpReadWithRetry('list_projects'))
     const projects = Array.isArray(projectsData.projects) ? projectsData.projects : []
     const groups = await mapConcurrent(projects.filter((project) => project?.id != null), 3, async (project) => {
@@ -916,6 +936,7 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
       ])
       const tasksData = toolData(tasksResult)
       const { done: doneIds, review: reviewIds } = typesResult
+      await Promise.all((Array.isArray(tasksData.tasks) ? tasksData.tasks : []).map(resolveTaskOwner))
       return (Array.isArray(tasksData.tasks) ? tasksData.tasks : []).filter((task) => {
         const assignedHere = taskBelongsToAgent(task, { id: selfAgentId, identifier })
         return assignedHere && !taskIsCompleted(task, doneIds)
@@ -1103,10 +1124,7 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
     if (dedicatedChannelRead) return dedicatedChannelRead
     dedicatedChannelRead = (async () => {
       try {
-        if (selfAgentId == null) {
-          const agents = toolData(await callMcpTool('list_agents')).agents || []
-          rememberSelfAgent(agents.find((entry) => String(entry.identifier || entry.slug || '') === identifier))
-        }
+        await ensureSelfAgent()
         const data = toolData(await callMcpTool('list_channels'))
         const channels = data.channels ?? data.data?.channels ?? data.data ?? data.items ?? []
         const channel = dedicatedChannel(Array.isArray(channels) ? channels : [], { id: selfAgentId, identifier, aliases: [...selfAliases], channelId: readConfig(slug)?.dedicatedChannelId })
@@ -1278,11 +1296,6 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
     if (!mcpUrl || backlogProbeBusy) return
     backlogProbeBusy = true
     try {
-      const agentsData = toolData(await callMcpTool('list_agents'))
-      const agents = Array.isArray(agentsData.agents) ? agentsData.agents : []
-      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')
-      rememberSelfAgent(self)
       const projectsData = toolData(await callMcpTool('list_projects'))
       const projects = Array.isArray(projectsData.projects) ? projectsData.projects : []
       const mentionActivity = []
@@ -1296,7 +1309,7 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
           ])
           const { done: doneIds, review: reviewIds } = typesData
           for (const task of Array.isArray(tasksData.tasks) ? tasksData.tasks : []) {
-            if (!taskBelongsToAgent(task, { id: selfAgentId, identifier })) continue
+            if (!await resolveTaskOwner(task)) continue
             const taskKey = `${project.id}:${task.id}`
             if (taskIsCompleted(task, doneIds) || taskIsAwaitingReview(task, reviewIds)) {
               memory.remember({ key: `ticket:${project.id}:${task.id}`, kind: 'ticket', state: 'handoff', summary: task.title, refs: { projectId: project.id, ticketId: task.id } })
@@ -1337,7 +1350,8 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
             return {}
           })
           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())
+          await ensureSelfAgent()
+          const mentionNeedles = [...selfAliases].map((alias) => '@' + alias)
           for (const item of activities) {
             const text = JSON.stringify(item)
             const activityKey = String(project.id) + ':' + String(item.id ?? item.message_id ?? item.messageId ?? text.slice(0, 500))
@@ -1437,6 +1451,7 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
   }
 
   async function executeCycle(kind, context, targetChannels = [], taskRef = null, delivery = null, runner, control, preparedWorkdir = '') {
+    refreshModelSettings()
     const laneName = kind === 'full' ? 'work' : 'reply'
     const cycleControl = control || { cancelled: false, runner, statusTargets: new Set() }
     const cycleStartedAt = performance.now()
@@ -1467,6 +1482,8 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
     const prompt = (ctx.length ? ctx.join('\n') + '\n\n' : '') + (recalled ? recalled + '\n\n' : '') + baseFor(kind, delivery)
     // Chat-shaped cycles (mentions/intro) may run on the cheaper chat model; code
     // work (full/sweep) uses the main model.
+    refreshModelSettings()
+    const cycleModelRevision = modelSettingsRevision
     const useModel = kind === 'full' ? codeModel : liteModel
     log('running ' + kind + ' cycle…' + (ctx.length ? ' (' + ctx.length + ' event' + (ctx.length === 1 ? '' : 's') + ')' : '') + (useModel ? ' [' + useModel + ']' : ''))
     // Presence notifications are intentionally disabled; durable messages and
@@ -1492,7 +1509,8 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
       const result = await runner.runCycle(prompt, useModel, runnerOptions)
       cycleControl.outcome = result?.subtype || 'ok'
       if (result?.timings) log(`cycle timing runtime=${JSON.stringify(result.timings)} kind=${kind}`)
-      if (result?.model && result.model !== useModel) {
+      refreshModelSettings()
+      if (cycleModelRevision === modelSettingsRevision && result?.model && result.model !== useModel) {
         // Persist only a successfully selected model and only if the user has
         // not changed that tier while this cycle was running.
         let changed = false
@@ -1513,7 +1531,7 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
       }
       if (!cycleSucceeded(result)) {
         const outcome = result?.subtype || 'an unknown runtime error'
-        const notice = `I'm blocked because the ${kind === 'full' ? 'coding' : 'reply'} cycle ended with ${outcome}. I'm not claiming completion.${activeTaskRef ? " I've paused this revision until the ticket changes." : ''}`
+        const notice = `I'm blocked because the ${kind === 'full' ? 'coding' : 'reply'} cycle ended with ${outcome}.${result?.subtype === 'rate_limited' ? ` The model provider is rate-limiting this request. Choose a model with available quota in Agent Studio, or retry once the limit resets.\n\n[Open Agent Studio](http://127.0.0.1:4317/#agent=${encodeURIComponent(identifier)}&settings=1)\n\n` : ' '}I'm not claiming completion.${activeTaskRef ? " I've paused this revision until the ticket changes." : ''}`
         log('WORK_CYCLE_BLOCKED ' + outcome + '; publishing blocker')
         if (activeTaskRef) pauseFailedTask(activeTaskRef)
         try { await publishBlocker({ prompt, taskRef: activeTaskRef, delivery, notice }) }
@@ -1581,8 +1599,10 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
         if (!reply) {
           log('guarded reply produced no final text; leaving delivery unrecorded for retry')
         } else {
+          const deliveryStartedAt = performance.now()
           try { await postMessageOnce({ ...delivery, content: reply }) }
           catch (e) { log('guarded reply delivery failed closed: ' + (e?.message || e)) }
+          finally { log(`cycle timing delivery=${Math.round(performance.now() - deliveryStartedAt)}ms kind=${kind}`) }
         }
       }
     } finally {
@@ -1637,17 +1657,11 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
     const projectId = hinted && (hinted.project_id ?? hinted.projectId ?? raw.project_id ?? raw.projectId)
     if (!validBackendId(ticketId) || !validBackendId(projectId)) { log(kind + ' missing ticket/project identity — ignored'); return }
     try {
-      if (selfAgentId == null) {
-        const agentsData = toolData(await callMcpTool('list_agents'))
-        const self = (Array.isArray(agentsData.agents) ? agentsData.agents : []).find((a) => String(a.identifier || a.slug || '') === identifier)
-        selfAgentId = null
-        rememberSelfAgent(self)
-      }
       const ticketData = toolData(await callMcpTool('get_ticket', { project_id: projectId, ticket_id: ticketId }))
       const ticket = ticketData.ticket ?? ticketData.task ?? ticketData
       if (String(ticket.id) !== String(ticketId) || (ticket.project_id != null && String(ticket.project_id) !== String(projectId))) { log('get_ticket returned mismatched identity; ignored'); return }
       const typeSets = await taskTypeSets(projectId)
-      const belongsToSelf = taskBelongsToAgent(ticket, { id: selfAgentId, identifier })
+      const belongsToSelf = await resolveTaskOwner(ticket)
       const key = `${projectId}:${ticketId}`
       if (!belongsToSelf) {
         for (const [name, queue] of Object.entries(queues)) {
@@ -1713,6 +1727,15 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
     // Keep redirects ordered while a legacy root is being verified. Other
     // conversations can resolve and run independently.
     const next = (mentionIntake.get(key) || Promise.resolve()).then(async () => {
+      const { idKey } = mentionDedupeKeys(message, cid)
+      const sourceKey = `mention:${cid}:${idKey}`
+      // A confirmed received ledger entry needs no identity/root network read.
+      // Preserve the legacy replay recovery path when only the seen key exists.
+      if (idKey && seenMentions.has(idKey) && memory.has(sourceKey, 'received')) {
+        log(`${k} duplicate channel=${cid} message=${messageId} — original already accepted; no new cycle`)
+        return { status: 'duplicate' }
+      }
+      const intakeStartedAt = performance.now()
       try { await ensureSelfAgent() }
       catch (error) { log('agent identity lookup unavailable; using configured aliases: ' + (error?.message || error)) }
       if (k === 'channel:message' && selfOrganizationId == null) return { status: 'ignored-unknown-organization' }
@@ -1723,13 +1746,30 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
       const irrelevant = preliminary.action === 'ignore' && preliminary.reason !== 'unaddressed-thread-activity'
       let rootContent = ''
       if (parent != null && !irrelevant && preliminary.action !== 'stand_down') {
-        try { rootContent = await resolveOwnThread(cid, parent) }
+        const optional = preliminary.explicitSelf || memory.has(threadControlKey(cid, parent), 'active')
+        let contextTimer
+        try {
+          const context = resolveOwnThread(cid, parent)
+          // An explicit recipient or durable ownership already authorizes intake.
+          // Slow optional context must not consume the transport's 20s deadline
+          // before a model starts. Delivery still re-reads the authoritative thread.
+          rootContent = optional ? await Promise.race([context, new Promise((resolve) => {
+            contextTimer = setTimeout(() => {
+              log(`mention optional context pending channel=${cid} message=${messageId} — continuing with verified recipient`)
+              resolve('')
+            }, 1000)
+          })]) : await context
+        }
         catch (error) {
           if (!preliminary.explicitSelf && !memory.has(threadControlKey(cid, parent), 'active')) throw error
           log('optional root context unavailable; preserving verified recipient: ' + (error?.message || error))
         }
+        finally { clearTimeout(contextTimer) }
+      }
+      if (!stopping) {
+        if (!irrelevant) log(`mention timing intake=${Math.round(performance.now() - intakeStartedAt)}ms source=${k} channel=${cid} message=${messageId}`)
+        return routeEvent('agent:mention', { ...d, _threadRootContent: rootContent })
       }
-      if (!stopping) return routeEvent('agent:mention', { ...d, _threadRootContent: rootContent })
     }).catch((error) => { log('mention thread verification failed: ' + (error?.message || error)); return { status: 'retry' } })
       .finally(() => { if (mentionIntake.get(key) === next) mentionIntake.delete(key) })
     mentionIntake.set(key, next)
@@ -1817,7 +1857,8 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
         }
         return
       }
-      const assignment = assignmentRequest(text)
+      const creatingTicket = conversationCreatesTicket(text)
+      const assignment = creatingTicket ? null : assignmentRequest(text)
       if (assignment) {
         const delivery = conversationDelivery('assignment-routing')
         const isCancelled = () => !!controlKey && memory.has(controlKey, 'cancelled')
@@ -1837,7 +1878,7 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
         })().catch((error) => log('assignment routing delivery failed: ' + (error?.message || error)))
         return
       }
-      if (cid != null && conversationAsksPendingTickets(text)) {
+      if (cid != null && !creatingTicket && conversationAsksPendingTickets(text)) {
         log('pending-ticket question -> watcher-owned MCP lookup')
         void answerPendingTickets({ delivery: conversationDelivery('pending-tickets'), who, text })
         return
@@ -1846,20 +1887,21 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
       // Light up the live status the instant we pick this up; drain owns the
       // subsequent working/typing heartbeat for its lane.
       if (cid != null) sendStatus(cid, 'thinking')
-      const codingMention = canCode && conversationNeedsCode(text, { rootContent: raw._threadRootContent })
+      const codingMention = !creatingTicket && canCode && conversationNeedsCode(text, { rootContent: raw._threadRootContent })
       const replyDelivery = conversationDelivery(codingMention ? 'result' : 'reply', !codingMention)
       let ctx = cid != null
         ? replyDelivery?.watcherOwned
           ? `The watcher verified this source message is addressed to you in OpenVisio channel ${cid}${who ? ` by "${who}"` : ''}: "${text}". ${codingMention ? 'Complete the concrete repository work and verification first.' : 'Answer the request directly.'} Do NOT call post_message; it is intentionally unavailable. Return only the final reply as your final answer. The watcher will re-check the live thread and render it at most once.${who ? ` To mention the requester, use their exact full name "@${who}".` : ''} The complete message is already here; do not call get_resource, get_marching_orders, poll_inbox, list_mcp_resources, or list_mcp_resource_templates.`
           : `The watcher verified this source message is addressed to you in OpenVisio channel ${cid}${who ? ` by "${who}"` : ''}: "${text}". ${codingMention ? `This is concrete repository work. Start the coding flow now. You may send at most one concrete progress update after work begins, but do not stop there: continue the work and then send one distinct final result with evidence or a real blocker. Use post_message for the final result even if you already sent a progress message` : '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. Never repeat the same message.${who ? ` To @mention them back, write their EXACT full name "@${who}". A mention only links when the name matches exactly.` : ''} The complete message is already here. Do not call get_resource, get_marching_orders, poll_inbox, list_mcp_resources, or list_mcp_resource_templates.${codingMention ? ' Stop only after the work and final result/blocker.' : ' After your single reply, STOP.'}`
         : undefined
+      if (ctx && creatingTicket) ctx += ' This is a ticket-creation request, not repository implementation. Discover create_ticket on the authenticated watcher server, resolve the project and valid type using list_projects and list_task_types, then create the requested ticket once. Follow its actual schema. If a required detail is ambiguous, ask for that detail rather than inventing it. Report the returned ticket reference. Never replay a creation after a timeout or uncertain response.'
       if (ctx && raw._threadRootContent) ctx += ` Verified thread root authored by you (context, not a new request): ${JSON.stringify(raw._threadRootContent)}. For requested repairs, inspect the referenced PR and its branch, run the failing verification, and fix that existing work. A review/testing handoff does not prevent an explicit human repair request.`
       if (codingMention) {
         // Activity indicators make accepted work visible. Do not add a canned
         // pickup message; the first channel message is the verified result or a
         // concrete blocker from the work lane.
         void drain('full', ctx, cid == null ? [] : [cid], null, replyDelivery)
-      } else void drain('fast', ctx, cid == null ? [] : [cid], null, replyDelivery)
+      } else void drain(creatingTicket ? 'coord' : 'fast', ctx, cid == null ? [] : [cid], null, replyDelivery)
     } else if (k === 'error') {
       const detail = raw && (raw.message || raw.error || raw.reason || raw.code || raw.d?.message || raw.d?.error)
       log('error event: ' + (detail ? String(detail) : JSON.stringify(raw)).slice(0, 220))
@@ -1911,7 +1953,10 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
   deliveryTimer?.unref?.()
   const authorizationTimer = autoStart ? setInterval(() => void resumeReadyAuthorizations(), 2_000) : null
   authorizationTimer?.unref?.()
-  const heartbeatTimer = autoStart ? setInterval(() => emit('watcher.heartbeat', { status: 'running', active: queues.work.activeSize + queues.reply.activeSize, pending: queues.work.size + queues.reply.size, model: codeModel, chatModel: liteModel, workdir: workdir || '' }), 15_000) : null
+  const modelSettingsTimer = autoStart ? setInterval(refreshModelSettings, 2_000) : null
+  modelSettingsTimer?.unref?.()
+  emit('models.updated', { model: codeModel, chatModel: liteModel, modelSettingsRevision })
+  const heartbeatTimer = autoStart ? setInterval(() => emit('watcher.heartbeat', { status: 'running', active: queues.work.activeSize + queues.reply.activeSize, pending: queues.work.size + queues.reply.size, model: codeModel, chatModel: liteModel, modelSettingsRevision, workdir: workdir || '' }), 15_000) : null
   heartbeatTimer?.unref?.()
   let introTimer = null, taskProbeStartTimer = null, taskProbeTimer = null
   if (autoStart && (mcpConfig || mcpUrl)) {
@@ -1945,6 +1990,7 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
       clearInterval(authorizationTimer)
       clearInterval(deliveryTimer)
       clearInterval(heartbeatTimer)
+      clearInterval(modelSettingsTimer)
       if (introTimer) clearTimeout(introTimer)
       if (taskProbeStartTimer) clearTimeout(taskProbeStartTimer)
       if (taskProbeTimer) clearInterval(taskProbeTimer)