openvisio-agent 0.25.0 → 0.25.1

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## [0.25.1] — 2026-09-19
+
+- Discover task columns by board when required by the live MCP schema; remove the incorrect project-only instruction that caused failed calls and assignment lookups.
+- Stop surviving watcher descendants before deleting agent data, preserve shared Studio processes, and explain when an older Studio needs restarting to enable removal.
+
 ## [0.25.0] — 2026-09-18
 
 - Reduce Studio journal-processing CPU by caching unchanged files, refreshing every two seconds, and disconnecting hidden or paused views.

package/package.json

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

package/scenarios/workspace.scenarios.mjs

@@ -23,14 +23,15 @@ export function fixture({ agent = 'codex', task = true, run, threadError = false
   let watcher, connection, messageId = 100, closed = 0
   const memory = createByoMemoryGraph({ path: join(dir, 'ledger.json') })
   const mcp = {
-    listTools: async () => ['list_agents', 'list_projects', 'list_tasks', 'get_ticket', 'update_ticket', 'post_message', 'list_message_thread', 'list_channels', 'list_task_types', 'list_activity', ...commentTools].map((name) => ({ name })),
+    listTools: async () => ['list_agents', 'list_projects', 'list_boards', 'list_tasks', 'get_ticket', 'update_ticket', 'post_message', 'list_message_thread', 'list_channels', 'list_task_types', 'list_activity', ...commentTools].map((name) => ({ name, ...(name === 'list_task_types' ? { inputSchema: { required: ['project_id', 'board_id'], properties: { project_id: {}, board_id: {} } } } : {}) })),
     callTool: async (name, args = {}) => {
       calls.push({ name, args })
       if (commentTools.includes(name)) return onComment ? await onComment(name, args) : { id: 500 }
       if (name === 'list_agents') return listAgents ? await listAgents() : { agents: [selfProfile] }
       if (name === 'list_projects') return { projects: [{ id: 1, name: 'Workspace' }] }
       if (name === 'list_tasks') return { tasks: task ? [{ ...ticket }] : [] }
-      if (name === 'list_task_types') return { types: [{ id: 1, name: 'In Progress' }, { id: 2, name: 'Testing' }, { id: 3, name: 'Done' }] }
+      if (name === 'list_boards') return { boards: [{ id: 4 }] }
+      if (name === 'list_task_types') { assert.equal(args.board_id, 4); return { types: [{ id: 1, name: 'In Progress' }, { id: 2, name: 'Testing' }, { id: 3, name: 'Done' }] } }
       if (name === 'get_ticket') { assert.equal(args.project_id, 1); assert.equal(args.ticket_id, 77); return { ticket: getTicket ? getTicket(ticket) : { ...ticket } } }
       if (name === 'update_ticket') { Object.assign(ticket, args); return { ticket: { ...ticket } } }
       if (name === 'list_channels' && listChannels) return { channels: await listChannels() }

package/src/agent-processes.mjs

@@ -0,0 +1,46 @@
+import { execFile } from 'node:child_process'
+import { promisify } from 'node:util'
+import { setTimeout as delay } from 'node:timers/promises'
+
+const exec = promisify(execFile)
+export async function agentProcessSnapshot() {
+  const { stdout } = await exec('ps', ['-axo', 'pid=,ppid=,stat=,command='], { maxBuffer: 4 * 1024 * 1024 })
+  return stdout.split('\n').flatMap(line => {
+    const match = /^\s*(\d+)\s+(\d+)\s+(\S+)\s+(.+)$/.exec(line)
+    return match && !match[3].startsWith('Z') ? [{ pid: Number(match[1]), parent: Number(match[2]), command: match[4] }] : []
+  })
+}
+
+export function agentProcessTree(rows, slug) {
+  const selected = new Map()
+  for (const row of rows) {
+    const name = /(?:^|\s)--name(?:=|\s+)["']?([^"'\s]+)/.exec(row.command)?.[1]
+    if (/\bopenvisio-agent\b/.test(row.command) && /\bwatch\b/.test(row.command) && name === slug) selected.set(row.pid, row)
+  }
+  let added
+  do {
+    added = false
+    for (const row of rows) {
+      // Studio is shared by all agents, even when a watcher launched it.
+      if (selected.has(row.pid) || !selected.has(row.parent) || /\bcli\.mjs\s+studio\b/.test(row.command)) continue
+      selected.set(row.pid, row); added = true
+    }
+  } while (added)
+  return [...selected.values()]
+}
+
+export async function stopAgentProcessTree(captured, { snapshot = agentProcessSnapshot, signal = process.kill.bind(process), wait = delay } = {}) {
+  const remaining = async () => (await snapshot()).filter(row => captured.some(old => old.pid === row.pid && old.command === row.command))
+  for (const name of ['SIGTERM', 'SIGKILL']) {
+    const rows = await remaining()
+    if (!rows.length) return
+    for (const row of rows.reverse()) {
+      try { signal(row.pid, name) } catch (error) { if (error.code !== 'ESRCH') throw error }
+    }
+    for (let attempt = 0; attempt < 20; attempt++) {
+      await wait(100)
+      if (!(await remaining()).length) return
+    }
+  }
+  throw new Error('Agent processes are still running. Local data has been kept; retry removal after they stop.')
+}

package/src/agent-removal.mjs

@@ -7,6 +7,7 @@ import { homedir } from 'node:os'
 import { join, resolve } from 'node:path'
 import { fileURLToPath } from 'node:url'
 import { OV_DIR, slugify } from './lib.mjs'
+import { agentProcessSnapshot, agentProcessTree, stopAgentProcessTree } from './agent-processes.mjs'
 
 const exec = promisify(execFile)
 const hash = value => createHash('sha256').update(value).digest('hex')
@@ -74,8 +75,13 @@ async function stopLocalAgent({ stateDir, slug }) {
   if (resolve(stateDir) !== resolve(OV_DIR)) throw problem('Stopping agents is supported only in this computer’s agent storage.')
   if (slugify(slug) !== slug) throw problem('This legacy local name cannot be stopped safely from Studio.')
   if (process.platform === 'win32') throw problem('Stop the watcher manually before removal on Windows; automatic removal is not supported yet.')
+  // Capture children before unloading the service, which can orphan runtimes.
+  // A stale lock PID is not proof of ownership and must never target another app.
+  const rows = await agentProcessSnapshot()
+  const captured = agentProcessTree(rows, slug)
   const lock = await readFile(join(stateDir, `watch-${slug}.lock`), 'utf8').catch(() => '')
-  const pid = /^\d+\s*$/.test(lock) ? Number(lock.trim()) : null
+  const lockedPid = /^\d+\s*$/.test(lock) ? Number(lock.trim()) : null
+  if (rows.some(row => row.pid === lockedPid) && !captured.some(row => row.pid === lockedPid)) throw problem('The recorded watcher uses an unrecognized command. Stop it before removing its data.')
   const service = process.platform === 'darwin'
     ? join(homedir(), 'Library', 'LaunchAgents', `io.openvisio.${slug}.plist`)
     : join(homedir(), '.config', 'systemd', 'user', `openvisio-${slug}.service`)
@@ -83,11 +89,8 @@ async function stopLocalAgent({ stateDir, slug }) {
     await exec('systemctl', ['--user', 'disable', '--now', `openvisio-${slug}.service`], { timeout: 15000 })
   }
   await exec(process.execPath, [fileURLToPath(new URL('../bin/cli.mjs', import.meta.url)), 'stop', '--name', slug], { timeout: 20000, maxBuffer: 65536 })
-  if (Number.isSafeInteger(pid) && pid > 0) {
-    let alive = false
-    try { process.kill(pid, 0); alive = true } catch (error) { alive = error.code === 'EPERM' }
-    if (alive) throw problem('The recorded watcher process is still running. Stop it before removing its data.')
-  }
+  await stopAgentProcessTree(captured)
+  if (agentProcessTree(await agentProcessSnapshot(), slug).length) throw problem('The watcher restarted. Stop its background service before removing data.')
   if (process.platform === 'darwin') {
     const loaded = await exec('launchctl', ['print', `gui/${process.getuid()}/io.openvisio.${slug}`], { timeout: 5000 }).then(() => true, () => false)
     if (loaded) throw problem('The background service is still loaded. Stop it before removing the agent.')

package/src/mastra-harness.mjs

@@ -12,7 +12,7 @@ import { opencodeProviderFailure } from './opencode-failure.mjs'
 import { isTransportInterruption } from './transport-recovery.mjs'
 
 const MCP_TOOL_NAMES = [
-  WORK_SESSION_TOOL, ...contextToolNames, 'list_agents', 'list_projects', 'list_tasks', 'list_task_types', 'get_ticket',
+  WORK_SESSION_TOOL, ...contextToolNames, 'list_agents', 'list_projects', 'list_boards', 'list_tasks', 'list_task_types', 'get_ticket',
   'create_ticket', 'update_ticket', 'list_channels', 'list_message_thread', 'list_activity',
   'post_message', 'react_message', 'list_codebases', 'get_codebase',
   'codebase_tree', 'create_codebase_branch', 'create_codebase_commit',

package/src/task-types.mjs

@@ -1,3 +1,21 @@
+export async function loadProjectTaskTypes({ projectId, listTools, callTool }) {
+  const tools = await listTools()
+  const tool = tools.find(tool => tool.name === 'list_task_types')
+  const schema = tool?.inputSchema || tool?.input_schema
+  if (!schema?.properties?.board_id && !schema?.required?.includes('board_id')) return callTool('list_task_types', { project_id: projectId })
+  const result = await callTool('list_boards', { project_id: projectId })
+  if (!Array.isArray(result?.boards)) throw new Error('Cannot verify project boards')
+  const types = []
+  for (const board of result.boards) {
+    if (board?.id == null) throw new Error('Cannot verify board identity')
+    const data = await callTool('list_task_types', { project_id: projectId, board_id: board.id })
+    const rows = data?.types ?? data?.task_types ?? data?.taskTypes
+    if (!Array.isArray(rows)) throw new Error('Cannot verify task columns')
+    types.push(...rows)
+  }
+  return { types }
+}
+
 export function createTaskTypeLookup({ load, now = Date.now, ttlMs = 60_000 }) {
   if (!Number.isFinite(ttlMs) || ttlMs < 0) throw new Error('Invalid task column cache lifetime')
   const cache = new Map(), pending = new Map()

package/src/watch.mjs

@@ -27,7 +27,7 @@ import { interruptedRuntimeResult, isTransportInterruption, transportRecoveryDel
 import { mapConcurrent } from './concurrency.mjs'
 import { releaseAuthorizedPause } from './authorization-resume.mjs'
 import { dedicatedChannel, repliesAfterSource } from './channel-routing.mjs'
-import { createTaskTypeLookup } from './task-types.mjs'
+import { createTaskTypeLookup, loadProjectTaskTypes } from './task-types.mjs'
 import { createThreadContextResolver } from './thread-context.mjs'
 import { assignmentRequest, assignmentStatusOnly, routeAssignments } from './assignment-routing.mjs'
 import { modelProcessOptions, stopModelProcess } from './process-lifecycle.mjs'
@@ -993,7 +993,7 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
     try { return JSON.parse(text) } catch { return { text } }
   }
 
-  const taskTypeSets = createTaskTypeLookup({ load: (projectId) => callMcpTool('list_task_types', { project_id: projectId }).then(toolData) })
+  const taskTypeSets = createTaskTypeLookup({ load: (projectId) => loadProjectTaskTypes({ projectId, listTools: () => mcpClient.listTools(), callTool: (name, args) => callMcpTool(name, args).then(toolData) }) })
 
   const callMcpReadWithRetry = async (name, args = {}) => {
     try { return await callMcpTool(name, args) }
@@ -1883,7 +1883,7 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
       if (activityChannel != null) sendStatus(activityChannel, 'thinking')
       const humanTicketRef = ticketSlug ? `ticket ${ticketSlug}` : `the ticket “${title}”`
       const workspaceDirective = preparedWorktree ? ` A prepared local git worktree for this ticket exists at ${JSON.stringify(preparedWorktree)}. Start repository inspection and edits there. Do not call list_codebases, codebase_tree, or get_codebase while this local worktree is available.` : ''
-      void drain(cycleKind, `Authoritative get_ticket verification confirms ${humanTicketRef} is open and assigned to YOU. Internal tool identity: project_id ${projectId}, ticket_id ${ticketId}. For every get_ticket call pass exactly { project_id: ${projectId}, ticket_id: ${ticketId} }; for list_task_types pass exactly { project_id: ${projectId} }. Numeric ids are MCP arguments only and must never appear in human-facing text; use ${ticketSlug || 'the ticket title'} instead. Ticket details: ${JSON.stringify({ slug: ticketSlug, title, description: ticket.description, priority: ticket.priority, typeId: ticket.type_id ?? ticket.typeId })}.${workspaceDirective} This assignment has no source thread: do not call post_message yourself. Choose the tools, context, validation, and ticket state appropriate to this request. Return your final response when you decide this turn is finished; the watcher will publish it once in the dedicated agent channel without changing ticket status.`, activityChannel == null ? [] : [activityChannel], { projectId, ticketId, channelId: activityChannel })
+      void drain(cycleKind, `Authoritative get_ticket verification confirms ${humanTicketRef} is open and assigned to YOU. Internal tool identity: project_id ${projectId}, ticket_id ${ticketId}. For every get_ticket call pass exactly { project_id: ${projectId}, ticket_id: ${ticketId} }; For list_task_types, follow its advertised schema: resolve the ticket’s board_id from the ticket or list_boards and include it when required. Never guess a board id. Numeric ids are MCP arguments only and must never appear in human-facing text; use ${ticketSlug || 'the ticket title'} instead. Ticket details: ${JSON.stringify({ slug: ticketSlug, title, description: ticket.description, priority: ticket.priority, typeId: ticket.type_id ?? ticket.typeId })}.${workspaceDirective} This assignment has no source thread: do not call post_message yourself. Choose the tools, context, validation, and ticket state appropriate to this request. Return your final response when you decide this turn is finished; the watcher will publish it once in the dedicated agent channel without changing ticket status.`, activityChannel == null ? [] : [activityChannel], { projectId, ticketId, channelId: activityChannel })
     } catch (e) {
       log(kind + ' ticket verification failed for #' + ticketId + ': ' + (e?.message || e))
     }
@@ -2079,7 +2079,7 @@ export function createBackendWatcher({ backend, wsUrl, apiKey, identifier, slug,
           ? `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}".` : ''} Use additional discovery, resources, and history when they help answer this request.`
           : `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)` : ''}, using the authenticated session bridge. 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.` : ''} Use additional discovery, resources, and history when they help answer this request.${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 && creatingTicket) ctx += ' This is a ticket-creation request, not repository implementation. Discover create_ticket on the authenticated watcher server, resolve the project, board, and valid type using list_projects, list_boards, 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

package/studio/app.mjs

@@ -391,11 +391,13 @@ function initializeStudio() {
     $('model-settings-agent').textContent = `${agentName(agent)} · Models`;
     $('open-model-settings').disabled = !agent.settingsEditable;
     $('open-agent-removal').disabled = !agent.removable;
+    $('open-agent-removal').title = agent.removable == null ? 'Restart Agent Studio with the updated openvisio-agent package to enable removal.' : !agent.removable ? 'Automatic removal is unavailable on this platform.' : '';
     const settings = agent.modelSettings || {};
     const pending = settings.revision && agent.appliedModelRevision !== settings.revision;
     $('model-settings-state').textContent = !agent.settingsEditable ? 'Model controls are available for agents configured on this computer.'
       : pending ? agent.modelControlsSupported && agent.status === 'online' ? 'Saved · waiting for the watcher to apply your choices.' : 'Saved for the next watcher connection. Older watchers need an update to apply changes live.'
       : `Replies: ${settings.chatModel || settings.model || 'Runtime default'} · Coding: ${settings.model || 'Runtime default'}`;
+    if (agent.removable == null) $('model-settings-state').textContent += ' Restart Agent Studio with the updated package to enable agent removal.';
     if (launchSettings && agent.settingsEditable) { launchSettings = false; void openModelSettings(agent); }
   }
   async function settingsFetch(path, options) {