@yeaft/webchat-agent 0.1.486 → 0.1.488

package/unify/session.js

@@ -27,7 +27,10 @@ import { initThreadStore } from './threads/store.js';
 import { Engine } from './engine.js';
 import { createThreadEngineRegistry } from './threads/engine-registry.js';
 import { MAIN_THREAD_ID } from './threads/store.js';
+import { getThreadStore } from './threads/store.js';
 import { createIntentClassifier } from './router/intent-classifier.js';
+import { initInputQueueStore } from './input-queue/store.js';
+import { createDispatcher } from './pipeline/dispatcher.js';
 import { join } from 'path';
 
 /**
@@ -193,6 +196,22 @@ export async function loadSession(options = {}) {
   // config as the engines so it can use primaryModel for classification.
   const router = createIntentClassifier({ adapter, trace, config });
 
+  // task-310 Phase 2 integration: wire InputQueue + Dispatcher so the
+  // web-bridge can submit `unify_chat` inputs through the unified pipeline
+  // (queue → router → engineRegistry → EngineInstance). In read-only mode
+  // the queue is memory-only (no disk writes).
+  const inputQueue = initInputQueueStore({
+    yeaftDir: config._readOnly ? null : yeaftDir,
+    force: true,
+  });
+  const dispatcher = createDispatcher({
+    inputQueue,
+    router,
+    engineRegistry,
+    threadStore: getThreadStore(),
+    trace,
+  });
+
   // ─── 10. Build session ─────────────────────────────────
   const status = {
     skills: skillManager.size,
@@ -224,6 +243,8 @@ export async function loadSession(options = {}) {
     engine,
     engineRegistry,
     router,
+    inputQueue,
+    dispatcher,
     adapter,
     config,
     conversationStore,

package/unify/skills.js

@@ -381,17 +381,21 @@ export class SkillManager {
   }
 
   /**
-   * List all skills (metadata only — no content), optionally filtered by mode.
+   * List all skills (metadata only — no content).
    * This is the "progressive disclosure" list tier.
    *
-   * @param {string} [mode] — 'chat' | 'work' | undefined (all)
+   * task-311: the legacy `mode` parameter (chat/work filter) is accepted but
+   * ignored — Unify no longer has mode distinction, so every skill is treated
+   * as universally applicable. The `mode` field on each record is still
+   * surfaced for historic YAML compatibility.
+   *
+   * @param {string} [_mode] — deprecated, ignored
    * @returns {Array<{ name: string, description: string, trigger: string, mode: string, category?: string, platforms?: string[], keywords?: string[], source: string, hasReferences: boolean, hasTemplates: boolean }>}
    */
-  list(mode) {
+  list(_mode) {
     const skills = [...this.#skills.values()];
-    const filtered = mode ? skills.filter(s => s.mode === 'both' || s.mode === mode) : skills;
 
-    return filtered.map(s => ({
+    return skills.map(s => ({
       name: s.name,
       description: s.description || '',
       trigger: s.trigger || '',
@@ -447,20 +451,22 @@ export class SkillManager {
    * Find skills relevant to a prompt.
    * Enhanced matching: regex triggers, keyword lists, name/description match.
    *
+   * task-311: the `mode` parameter is accepted but ignored (all skills are
+   * considered universally applicable since mode distinction was removed).
+   *
    * @param {string} prompt — user's prompt
-   * @param {string} [mode] — filter by mode
+   * @param {string} [_mode] — deprecated, ignored
    * @returns {Skill[]}
    */
-  findRelevant(prompt, mode) {
+  findRelevant(prompt, _mode) {
     if (!prompt) return [];
 
     const lowerPrompt = prompt.toLowerCase();
     const cleanPrompt = lowerPrompt.replace(/[^\w\s]/g, '');
     const promptWords = cleanPrompt.split(/\s+/).filter(w => w.length > 2);
     const allSkills = [...this.#skills.values()];
-    const filtered = mode ? allSkills.filter(s => s.mode === 'both' || s.mode === mode) : allSkills;
 
-    return filtered.filter(skill => {
+    return allSkills.filter(skill => {
       // 1. Regex or keyword trigger match
       if (skill.trigger && matchTrigger(skill.trigger, lowerPrompt, promptWords)) {
         return true;

package/unify/tools/agent.js

@@ -229,7 +229,6 @@ Guidelines:
     },
     required: ['name'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   async execute(input, ctx) {

package/unify/tools/apply-patch.js

@@ -115,7 +115,6 @@ Guidelines:
     },
     required: ['patch'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   isDestructive: () => false,

package/unify/tools/ask-user.js

@@ -38,7 +38,6 @@ Guidelines:
     },
     required: ['question'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => true,
   async execute(input, ctx) {

package/unify/tools/bash.js

@@ -137,7 +137,6 @@ Guidelines:
     },
     required: ['command'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   isDestructive: (input) => {

package/unify/tools/close-agent.js

@@ -25,7 +25,6 @@ The agent's result (if any) is returned before closing.`,
     },
     required: ['agent_id'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   async execute(input, ctx) {

package/unify/tools/enter-worktree.js

@@ -39,7 +39,6 @@ Returns the worktree path and branch name.`,
       },
     },
   },
-  modes: ['chat', 'work'],
   isDestructive: () => false,
   async execute(input, ctx) {
     const cwd = ctx?.cwd || process.cwd();

package/unify/tools/exit-worktree.js

@@ -41,7 +41,6 @@ unless discard_changes is set to true.`,
     },
     required: ['path', 'action'],
   },
-  modes: ['chat', 'work'],
   isDestructive: (input) => input?.action === 'remove',
   async execute(input, ctx) {
     const worktreePath = resolve(input.path);

package/unify/tools/file-edit.js

@@ -46,7 +46,6 @@ Guidelines:
     },
     required: ['file_path', 'old_string', 'new_string'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   isDestructive: () => false,

package/unify/tools/file-read.js

@@ -60,7 +60,6 @@ Guidelines:
     },
     required: ['file_path'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {

package/unify/tools/file-write.js

@@ -34,7 +34,6 @@ Guidelines:
     },
     required: ['file_path', 'content'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   isDestructive: () => false,

package/unify/tools/glob.js

@@ -95,7 +95,6 @@ Guidelines:
     },
     required: ['pattern'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {

package/unify/tools/grep.js

@@ -207,7 +207,6 @@ Guidelines:
     },
     required: ['pattern'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {

package/unify/tools/history-search.js

@@ -30,7 +30,6 @@ Results are returned newest-first with message role and content.`,
     },
     required: ['keyword'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {

package/unify/tools/image-generation.js

@@ -36,7 +36,6 @@ Guidelines:
     },
     required: ['prompt'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => false,
   async execute(input, ctx) {

package/unify/tools/js-repl.js

@@ -69,7 +69,6 @@ Guidelines:
     },
     required: ['code'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => true,
   async execute(input, ctx) {
@@ -112,7 +111,6 @@ Use when you want a clean slate.`,
     type: 'object',
     properties: {},
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   async execute(input, ctx) {

package/unify/tools/list-agents.js

@@ -20,7 +20,6 @@ and message counts. Use to monitor parallel task progress.`,
       },
     },
   },
-  modes: ['work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {

package/unify/tools/list-dir.js

@@ -37,7 +37,6 @@ This is better than using Bash with 'ls' because it provides structured output.`
       },
     },
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {

package/unify/tools/mcp-tools.js

@@ -31,7 +31,6 @@ Usage guidelines:
       },
     },
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {
@@ -88,7 +87,6 @@ Usage guidelines:
     },
     required: ['tool_name'],
   },
-  modes: ['chat', 'work'],
   async execute(input, ctx) {
     const mcpManager = ctx?.mcpManager;
 

package/unify/tools/memory-query.js

@@ -57,7 +57,6 @@ results sorted by score descending.`,
     },
     required: ['keywords'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {

package/unify/tools/memory-read.js

@@ -32,7 +32,6 @@ Actions:
     },
     required: ['action'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {

package/unify/tools/memory-search.js

@@ -48,7 +48,6 @@ Up to ${MAX_FILES_PER_CALL} files per call. Each file is capped at ${MAX_BYTES_P
     },
     required: ['paths'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {

package/unify/tools/memory-write.js

@@ -51,7 +51,6 @@ Importance levels: low, normal, high, critical`,
     },
     required: ['action'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   async execute(input, ctx) {

package/unify/tools/notebook-edit.js

@@ -48,7 +48,6 @@ Cell types: "code" or "markdown"`,
     },
     required: ['notebook_path'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => false,
   isReadOnly: (input) => input?.action === 'read',
   async execute(input, ctx) {

package/unify/tools/request-permissions.js

@@ -37,7 +37,6 @@ The user must explicitly approve before you proceed.`,
     },
     required: ['operation'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => true,
   async execute(input, ctx) {

package/unify/tools/send-message.js

@@ -25,7 +25,6 @@ The message is queued for the agent to process.`,
     },
     required: ['agent_id', 'message'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   async execute(input, ctx) {

package/unify/tools/skill.js

@@ -56,7 +56,6 @@ Actions:
     },
     required: ['action'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {

package/unify/tools/task-tools.js

@@ -72,7 +72,6 @@ Use this to break down complex work into trackable items.`,
     },
     required: ['title'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   async execute(input, ctx) {
@@ -143,7 +142,6 @@ Status values: pending, in_progress, completed, blocked, cancelled`,
     },
     required: ['task_id'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   async execute(input, ctx) {
@@ -192,7 +190,6 @@ Shows task IDs, titles, status, and priority. Filter by status if needed.`,
       },
     },
   },
-  modes: ['work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {
@@ -247,7 +244,6 @@ export const taskGet = defineTool({
     },
     required: ['task_id'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {
@@ -302,7 +298,6 @@ Use "view" to see the full log, or "append" to add a new entry.`,
     },
     required: ['task_id', 'action'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => false,
   isReadOnly: (input) => input?.action === 'view',
   async execute(input, ctx) {
@@ -358,7 +353,6 @@ memory can be rewritten to keep it current.`,
     },
     required: ['task_id', 'action'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => false,
   isReadOnly: (input) => input?.action === 'view',
   async execute(input, ctx) {
@@ -416,7 +410,6 @@ The new task is linked as a child of the original.`,
     },
     required: ['parent_task_id', 'title'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   async execute(input, ctx) {
@@ -475,7 +468,6 @@ approach, steps, and status of the current work.`,
     },
     required: ['action'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => false,
   isReadOnly: (input) => input?.action === 'view',
   async execute(input, ctx) {

package/unify/tools/thread-tools.js

@@ -46,7 +46,6 @@ engine to the new thread — call SwitchThread to activate it.`,
     },
     required: ['name'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   async execute(input) {
@@ -89,7 +88,6 @@ thread.`,
     },
     required: ['thread_id'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   async execute(input) {
@@ -120,7 +118,6 @@ exposes id, name, goal, parentThreadId, status ('active'|'idle'|'archived'),
 messageCount, lastMessageAt, archived, attachedTaskId. Reads only cached
 fields — does not scan messages.`,
   parameters: { type: 'object', properties: {} },
-  modes: ['work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute() {
@@ -168,7 +165,6 @@ the same thread.`,
     },
     required: ['thread_id', 'task_id'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   async execute(input) {
@@ -227,7 +223,6 @@ This replaces the deprecated SpawnSubtask tool.`,
     },
     required: ['title'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => false,
   isReadOnly: () => false,
   async execute(input) {
@@ -287,7 +282,6 @@ Use this to cross-reference work on another thread without switching.`,
     },
     required: ['thread_id'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input) {
@@ -333,7 +327,6 @@ Use this to review another thread's recent activity without switching.`,
     },
     required: ['thread_id'],
   },
-  modes: ['work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
   async execute(input, ctx) {

package/unify/tools/tool-search.js

@@ -1,5 +1,9 @@
 /**
  * tool-search.js — Search available tools by name or description.
+ *
+ * task-311: chat/work mode was removed in task-297; this tool no longer
+ * accepts or reports a `modes` filter. Results come back as plain
+ * { name, description } pairs.
  */
 
 import { defineTool } from './types.js';
@@ -9,7 +13,7 @@ export default defineTool({
   description: `Search available tools by name or description keyword.
 
 Use when you're unsure which tool to use for a task.
-Returns matching tools with their descriptions and parameters.`,
+Returns matching tools with their descriptions.`,
   parameters: {
     type: 'object',
     properties: {
@@ -17,77 +21,64 @@ Returns matching tools with their descriptions and parameters.`,
         type: 'string',
         description: 'Search keyword to match against tool names and descriptions',
       },
-      mode: {
-        type: 'string',
-        enum: ['chat', 'work'],
-        description: 'Filter by mode (optional)',
-      },
     },
     required: ['query'],
   },
-  modes: ['chat', 'work'],
   isConcurrencySafe: () => true,
   isReadOnly: () => true,
-  async execute(input, ctx) {
-    const { query, mode } = input;
+  async execute(input, _ctx) {
+    const { query } = input;
     if (!query) return JSON.stringify({ error: 'query is required' });
 
-    // Access the tool registry through the engine context
-    // Since we don't have direct registry access, list what we know
     const lowerQuery = query.toLowerCase();
 
-    // Get all tool definitions from the registry if available
-    // This is a self-referential tool — it describes the tools available to this engine
+    // Self-referential catalogue of tools available to the engine.
     const toolList = [
-      { name: 'AskUser', description: 'Ask the user a question', modes: ['chat', 'work'] },
-      { name: 'MemoryRead', description: 'Read from memory system', modes: ['chat', 'work'] },
-      { name: 'MemoryWrite', description: 'Write to memory system', modes: ['chat', 'work'] },
-      { name: 'MemorySearch', description: 'Search memory entries', modes: ['chat', 'work'] },
-      { name: 'WebSearch', description: 'Search the web', modes: ['chat', 'work'] },
-      { name: 'WebFetch', description: 'Fetch web page content', modes: ['chat', 'work'] },
-      { name: 'HistorySearch', description: 'Search conversation history', modes: ['chat', 'work'] },
-      { name: 'Bash', description: 'Execute shell commands', modes: ['chat', 'work'] },
-      { name: 'FileRead', description: 'Read file with line numbers', modes: ['chat', 'work'] },
-      { name: 'FileWrite', description: 'Write/create files', modes: ['chat', 'work'] },
-      { name: 'FileEdit', description: 'Surgical string replacement in files', modes: ['chat', 'work'] },
-      { name: 'Glob', description: 'Find files by pattern', modes: ['chat', 'work'] },
-      { name: 'Grep', description: 'Search file contents', modes: ['chat', 'work'] },
-      { name: 'ListDir', description: 'List directory contents', modes: ['chat', 'work'] },
-      { name: 'ApplyPatch', description: 'Apply unified diff patches', modes: ['chat', 'work'] },
-      { name: 'Agent', description: 'Create sub-agents', modes: ['work'] },
-      { name: 'SendMessage', description: 'Send message to sub-agent', modes: ['work'] },
-      { name: 'WaitAgent', description: 'Wait for sub-agent result', modes: ['work'] },
-      { name: 'CloseAgent', description: 'Close a sub-agent', modes: ['work'] },
-      { name: 'ListAgents', description: 'List all sub-agents', modes: ['work'] },
-      { name: 'TaskCreate', description: 'Create a task', modes: ['work'] },
-      { name: 'TaskUpdate', description: 'Update task status', modes: ['work'] },
-      { name: 'TaskList', description: 'List all tasks', modes: ['work'] },
-      { name: 'TaskGet', description: 'Get task details', modes: ['work'] },
-      { name: 'FollowupTask', description: 'Create follow-up task', modes: ['work'] },
-      { name: 'UpdatePlan', description: 'View/update execution plan', modes: ['work'] },
-      { name: 'JsRepl', description: 'JavaScript REPL evaluation', modes: ['chat', 'work'] },
-      { name: 'JsReplReset', description: 'Reset REPL state', modes: ['chat', 'work'] },
-      { name: 'NotebookEdit', description: 'Edit Jupyter notebooks', modes: ['chat', 'work'] },
-      { name: 'ImageGeneration', description: 'Generate images from text', modes: ['chat', 'work'] },
-      { name: 'ViewImage', description: 'View image metadata', modes: ['chat', 'work'] },
-      { name: 'RequestPermissions', description: 'Request dangerous operation permissions', modes: ['chat', 'work'] },
-      { name: 'WriteStdin', description: 'Write to running process stdin', modes: ['chat', 'work'] },
-      { name: 'Skill', description: 'Load skills from library', modes: ['chat', 'work'] },
-      { name: 'EnterWorktree', description: 'Create git worktree', modes: ['chat', 'work'] },
-      { name: 'ExitWorktree', description: 'Exit git worktree', modes: ['chat', 'work'] },
-      { name: 'mcp_list_tools', description: 'List MCP server tools', modes: ['chat', 'work'] },
-      { name: 'mcp_call_tool', description: 'Call MCP server tool', modes: ['chat', 'work'] },
+      { name: 'AskUser', description: 'Ask the user a question' },
+      { name: 'MemoryRead', description: 'Read from memory system' },
+      { name: 'MemoryWrite', description: 'Write to memory system' },
+      { name: 'MemorySearch', description: 'Search memory entries' },
+      { name: 'WebSearch', description: 'Search the web' },
+      { name: 'WebFetch', description: 'Fetch web page content' },
+      { name: 'HistorySearch', description: 'Search conversation history' },
+      { name: 'Bash', description: 'Execute shell commands' },
+      { name: 'FileRead', description: 'Read file with line numbers' },
+      { name: 'FileWrite', description: 'Write/create files' },
+      { name: 'FileEdit', description: 'Surgical string replacement in files' },
+      { name: 'Glob', description: 'Find files by pattern' },
+      { name: 'Grep', description: 'Search file contents' },
+      { name: 'ListDir', description: 'List directory contents' },
+      { name: 'ApplyPatch', description: 'Apply unified diff patches' },
+      { name: 'Agent', description: 'Create sub-agents' },
+      { name: 'SendMessage', description: 'Send message to sub-agent' },
+      { name: 'WaitAgent', description: 'Wait for sub-agent result' },
+      { name: 'CloseAgent', description: 'Close a sub-agent' },
+      { name: 'ListAgents', description: 'List all sub-agents' },
+      { name: 'TaskCreate', description: 'Create a task' },
+      { name: 'TaskUpdate', description: 'Update task status' },
+      { name: 'TaskList', description: 'List all tasks' },
+      { name: 'TaskGet', description: 'Get task details' },
+      { name: 'FollowupTask', description: 'Create follow-up task' },
+      { name: 'UpdatePlan', description: 'View/update execution plan' },
+      { name: 'JsRepl', description: 'JavaScript REPL evaluation' },
+      { name: 'JsReplReset', description: 'Reset REPL state' },
+      { name: 'NotebookEdit', description: 'Edit Jupyter notebooks' },
+      { name: 'ImageGeneration', description: 'Generate images from text' },
+      { name: 'ViewImage', description: 'View image metadata' },
+      { name: 'RequestPermissions', description: 'Request dangerous operation permissions' },
+      { name: 'WriteStdin', description: 'Write to running process stdin' },
+      { name: 'Skill', description: 'Load skills from library' },
+      { name: 'EnterWorktree', description: 'Create git worktree' },
+      { name: 'ExitWorktree', description: 'Exit git worktree' },
+      { name: 'mcp_list_tools', description: 'List MCP server tools' },
+      { name: 'mcp_call_tool', description: 'Call MCP server tool' },
     ];
 
-    let results = toolList.filter(t =>
+    const results = toolList.filter(t =>
       t.name.toLowerCase().includes(lowerQuery) ||
       t.description.toLowerCase().includes(lowerQuery)
     );
 
-    if (mode) {
-      results = results.filter(t => t.modes.includes(mode));
-    }
-
     return JSON.stringify({
       results,
       totalResults: results.length,

package/unify/tools/types.js

@@ -5,6 +5,9 @@
  * This ensures consistent shape and API-format conversion.
  *
  * Reference: yeaft-unify-core-systems.md §3.1
+ *
+ * task-311: the legacy `modes` field (task-297 deprecated) is now fully
+ * removed — Unify runs in a single unified mode.
  */
 
 /**
@@ -24,9 +27,6 @@
  * @property {string} description — LLM-facing description
  * @property {object} parameters — JSON Schema for input
  * @property {(input: object, ctx?: ToolContext) => Promise<string>} execute — execution function
- * @property {string[]} [modes] — @deprecated since task-297. Legacy mode filter (['chat', 'work']).
- *   Unify no longer has mode distinction; the ToolRegistry ignores this field and exposes every
- *   registered tool to the engine. Retained only so existing tool definitions keep loading.
  * @property {(input?: object) => boolean} [isConcurrencySafe] — can run in parallel?
  * @property {(input?: object) => boolean} [isReadOnly] — read-only operation?
  * @property {(input?: object) => boolean} [isDestructive] — destructive operation?
@@ -40,7 +40,6 @@
  *   description: string,
  *   parameters: object,
  *   execute: (input: object, ctx?: ToolContext) => Promise<string>,
- *   modes?: string[],  // @deprecated since task-297 — ignored by ToolRegistry
  *   isConcurrencySafe?: (input?: object) => boolean,
  *   isReadOnly?: (input?: object) => boolean,
  *   isDestructive?: (input?: object) => boolean,
@@ -52,7 +51,6 @@ export function defineTool({
   description,
   parameters,
   execute,
-  modes = ['chat', 'work'],
   isConcurrencySafe = () => false,
   isReadOnly = () => false,
   isDestructive = () => false,
@@ -65,7 +63,6 @@ export function defineTool({
     description: description || `Tool: ${name}`,
     parameters: parameters || { type: 'object', properties: {} },
     execute,
-    modes,
     isConcurrencySafe,
     isReadOnly,
     isDestructive,