@hasna/todos 0.9.28 → 0.9.29

package/dist/cli/index.js

@@ -3198,7 +3198,18 @@ function updateTask(id, input, db) {
     logTaskChange(id, "update", "assigned_to", task.assigned_to, input.assigned_to, agentId, d);
   if (input.approved_by !== undefined)
     logTaskChange(id, "approve", "approved_by", null, input.approved_by, agentId, d);
-  return getTask(id, d);
+  return {
+    ...task,
+    ...Object.fromEntries(Object.entries(input).filter(([, v]) => v !== undefined)),
+    tags: input.tags ?? task.tags,
+    metadata: input.metadata ?? task.metadata,
+    version: task.version + 1,
+    updated_at: now(),
+    completed_at: input.status === "completed" ? now() : task.completed_at,
+    requires_approval: input.requires_approval !== undefined ? input.requires_approval : task.requires_approval,
+    approved_by: input.approved_by ?? task.approved_by,
+    approved_at: input.approved_by ? now() : task.approved_at
+  };
 }
 function deleteTask(id, db) {
   const d = db || getDatabase();
@@ -3220,6 +3231,9 @@ function getBlockingDeps(id, db) {
 }
 function startTask(id, agentId, db) {
   const d = db || getDatabase();
+  const task = getTask(id, d);
+  if (!task)
+    throw new TaskNotFoundError(id);
   const blocking = getBlockingDeps(id, d);
   if (blocking.length > 0) {
     const blockerIds = blocking.map((b) => b.id.slice(0, 8)).join(", ");
@@ -3230,15 +3244,12 @@ function startTask(id, agentId, db) {
   const result = d.run(`UPDATE tasks SET status = 'in_progress', assigned_to = ?, locked_by = ?, locked_at = ?, version = version + 1, updated_at = ?
      WHERE id = ? AND (locked_by IS NULL OR locked_by = ? OR locked_at < ?)`, [agentId, agentId, timestamp, timestamp, id, agentId, cutoff]);
   if (result.changes === 0) {
-    const current = getTask(id, d);
-    if (!current)
-      throw new TaskNotFoundError(id);
-    if (current.locked_by && current.locked_by !== agentId && !isLockExpired(current.locked_at)) {
-      throw new LockError(id, current.locked_by);
+    if (task.locked_by && task.locked_by !== agentId && !isLockExpired(task.locked_at)) {
+      throw new LockError(id, task.locked_by);
     }
   }
   logTaskChange(id, "start", "status", "pending", "in_progress", agentId, d);
-  return getTask(id, d);
+  return { ...task, status: "in_progress", assigned_to: agentId, locked_by: agentId, locked_at: timestamp, version: task.version + 1, updated_at: timestamp };
 }
 function completeTask(id, agentId, db, evidence) {
   const d = db || getDatabase();
@@ -3250,14 +3261,15 @@ function completeTask(id, agentId, db, evidence) {
   }
   checkCompletionGuard(task, agentId || null, d);
   if (evidence) {
-    const meta = { ...task.metadata, _evidence: evidence };
-    d.run("UPDATE tasks SET metadata = ? WHERE id = ?", [JSON.stringify(meta), id]);
+    const meta2 = { ...task.metadata, _evidence: evidence };
+    d.run("UPDATE tasks SET metadata = ? WHERE id = ?", [JSON.stringify(meta2), id]);
   }
   const timestamp = now();
   d.run(`UPDATE tasks SET status = 'completed', locked_by = NULL, locked_at = NULL, completed_at = ?, version = version + 1, updated_at = ?
      WHERE id = ?`, [timestamp, timestamp, id]);
   logTaskChange(id, "complete", "status", task.status, "completed", agentId || null, d);
-  return getTask(id, d);
+  const meta = evidence ? { ...task.metadata, _evidence: evidence } : task.metadata;
+  return { ...task, status: "completed", locked_by: null, locked_at: null, completed_at: timestamp, version: task.version + 1, updated_at: timestamp, metadata: meta };
 }
 function lockTask(id, agentId, db) {
   const d = db || getDatabase();
@@ -8363,22 +8375,22 @@ var init_mcp = __esm(() => {
     version: "0.9.15"
   });
   server.tool("create_task", "Create a new task", {
-    title: exports_external.string().describe("Task title"),
-    description: exports_external.string().optional().describe("Task description"),
-    project_id: exports_external.string().optional().describe("Project ID"),
-    parent_id: exports_external.string().optional().describe("Parent task ID (for subtasks)"),
-    priority: exports_external.enum(["low", "medium", "high", "critical"]).optional().describe("Task priority"),
-    status: exports_external.enum(["pending", "in_progress", "completed", "failed", "cancelled"]).optional().describe("Initial status"),
-    agent_id: exports_external.string().optional().describe("Creator agent ID"),
-    assigned_to: exports_external.string().optional().describe("Assigned agent ID"),
-    session_id: exports_external.string().optional().describe("Session ID"),
-    working_dir: exports_external.string().optional().describe("Working directory context"),
-    plan_id: exports_external.string().optional().describe("Plan ID to assign task to"),
-    task_list_id: exports_external.string().optional().describe("Task list ID to assign task to"),
-    tags: exports_external.array(exports_external.string()).optional().describe("Task tags"),
-    metadata: exports_external.record(exports_external.unknown()).optional().describe("Arbitrary metadata"),
-    estimated_minutes: exports_external.number().optional().describe("Estimated time in minutes"),
-    requires_approval: exports_external.boolean().optional().describe("Require approval before completion")
+    title: exports_external.string(),
+    description: exports_external.string().optional(),
+    project_id: exports_external.string().optional(),
+    parent_id: exports_external.string().optional(),
+    priority: exports_external.enum(["low", "medium", "high", "critical"]).optional(),
+    status: exports_external.enum(["pending", "in_progress", "completed", "failed", "cancelled"]).optional(),
+    agent_id: exports_external.string().optional(),
+    assigned_to: exports_external.string().optional(),
+    session_id: exports_external.string().optional(),
+    working_dir: exports_external.string().optional(),
+    plan_id: exports_external.string().optional(),
+    task_list_id: exports_external.string().optional(),
+    tags: exports_external.array(exports_external.string()).optional(),
+    metadata: exports_external.record(exports_external.unknown()).optional(),
+    estimated_minutes: exports_external.number().optional(),
+    requires_approval: exports_external.boolean().optional()
   }, async (params) => {
     try {
       const resolved = { ...params };
@@ -8397,19 +8409,19 @@ var init_mcp = __esm(() => {
     }
   });
   server.tool("list_tasks", "List tasks with optional filters", {
-    project_id: exports_external.string().optional().describe("Filter by project"),
+    project_id: exports_external.string().optional(),
     status: exports_external.union([
       exports_external.enum(["pending", "in_progress", "completed", "failed", "cancelled"]),
       exports_external.array(exports_external.enum(["pending", "in_progress", "completed", "failed", "cancelled"]))
-    ]).optional().describe("Filter by status"),
+    ]).optional(),
     priority: exports_external.union([
       exports_external.enum(["low", "medium", "high", "critical"]),
       exports_external.array(exports_external.enum(["low", "medium", "high", "critical"]))
-    ]).optional().describe("Filter by priority"),
-    assigned_to: exports_external.string().optional().describe("Filter by assigned agent"),
-    tags: exports_external.array(exports_external.string()).optional().describe("Filter by tags (any match)"),
-    plan_id: exports_external.string().optional().describe("Filter by plan"),
-    task_list_id: exports_external.string().optional().describe("Filter by task list")
+    ]).optional(),
+    assigned_to: exports_external.string().optional(),
+    tags: exports_external.array(exports_external.string()).optional(),
+    plan_id: exports_external.string().optional(),
+    task_list_id: exports_external.string().optional()
   }, async (params) => {
     try {
       const resolved = { ...params };
@@ -8436,7 +8448,7 @@ ${text}` }] };
     }
   });
   server.tool("get_task", "Get full task details with relations", {
-    id: exports_external.string().describe("Task ID (full or partial)")
+    id: exports_external.string()
   }, async ({ id }) => {
     try {
       const resolvedId = resolveId(id);
@@ -8484,17 +8496,17 @@ Parent: ${task.parent.id.slice(0, 8)} | ${task.parent.title}`);
     }
   });
   server.tool("update_task", "Update task fields. Version required for optimistic locking.", {
-    id: exports_external.string().describe("Task ID (full or partial)"),
-    version: exports_external.number().describe("Current version (for optimistic locking)"),
-    title: exports_external.string().optional().describe("New title"),
-    description: exports_external.string().optional().describe("New description"),
-    status: exports_external.enum(["pending", "in_progress", "completed", "failed", "cancelled"]).optional().describe("New status"),
-    priority: exports_external.enum(["low", "medium", "high", "critical"]).optional().describe("New priority"),
-    assigned_to: exports_external.string().optional().describe("Assign to agent"),
-    tags: exports_external.array(exports_external.string()).optional().describe("New tags"),
-    metadata: exports_external.record(exports_external.unknown()).optional().describe("New metadata"),
-    plan_id: exports_external.string().optional().describe("Plan ID to assign task to"),
-    task_list_id: exports_external.string().optional().describe("Task list ID")
+    id: exports_external.string(),
+    version: exports_external.number(),
+    title: exports_external.string().optional(),
+    description: exports_external.string().optional(),
+    status: exports_external.enum(["pending", "in_progress", "completed", "failed", "cancelled"]).optional(),
+    priority: exports_external.enum(["low", "medium", "high", "critical"]).optional(),
+    assigned_to: exports_external.string().optional(),
+    tags: exports_external.array(exports_external.string()).optional(),
+    metadata: exports_external.record(exports_external.unknown()).optional(),
+    plan_id: exports_external.string().optional(),
+    task_list_id: exports_external.string().optional()
   }, async ({ id, ...rest }) => {
     try {
       const resolvedId = resolveId(id);
@@ -8505,7 +8517,7 @@ Parent: ${task.parent.id.slice(0, 8)} | ${task.parent.title}`);
     }
   });
   server.tool("delete_task", "Delete a task permanently", {
-    id: exports_external.string().describe("Task ID (full or partial)")
+    id: exports_external.string()
   }, async ({ id }) => {
     try {
       const resolvedId = resolveId(id);
@@ -8521,8 +8533,8 @@ Parent: ${task.parent.id.slice(0, 8)} | ${task.parent.title}`);
     }
   });
   server.tool("start_task", "Claim, lock, and set task status to in_progress.", {
-    id: exports_external.string().describe("Task ID (full or partial)"),
-    agent_id: exports_external.string().describe("Agent claiming the task")
+    id: exports_external.string(),
+    agent_id: exports_external.string()
   }, async ({ id, agent_id }) => {
     try {
       const resolvedId = resolveId(id);
@@ -8533,8 +8545,8 @@ Parent: ${task.parent.id.slice(0, 8)} | ${task.parent.title}`);
     }
   });
   server.tool("complete_task", "Mark task completed and release lock.", {
-    id: exports_external.string().describe("Task ID (full or partial)"),
-    agent_id: exports_external.string().optional().describe("Agent completing the task")
+    id: exports_external.string(),
+    agent_id: exports_external.string().optional()
   }, async ({ id, agent_id }) => {
     try {
       const resolvedId = resolveId(id);
@@ -8545,8 +8557,8 @@ Parent: ${task.parent.id.slice(0, 8)} | ${task.parent.title}`);
     }
   });
   server.tool("lock_task", "Acquire exclusive lock on a task", {
-    id: exports_external.string().describe("Task ID (full or partial)"),
-    agent_id: exports_external.string().describe("Agent acquiring lock")
+    id: exports_external.string(),
+    agent_id: exports_external.string()
   }, async ({ id, agent_id }) => {
     try {
       const resolvedId = resolveId(id);
@@ -8560,8 +8572,8 @@ Parent: ${task.parent.id.slice(0, 8)} | ${task.parent.title}`);
     }
   });
   server.tool("unlock_task", "Release exclusive lock on a task", {
-    id: exports_external.string().describe("Task ID (full or partial)"),
-    agent_id: exports_external.string().optional().describe("Agent releasing lock (omit to force)")
+    id: exports_external.string(),
+    agent_id: exports_external.string().optional()
   }, async ({ id, agent_id }) => {
     try {
       const resolvedId = resolveId(id);
@@ -8572,8 +8584,8 @@ Parent: ${task.parent.id.slice(0, 8)} | ${task.parent.title}`);
     }
   });
   server.tool("add_dependency", "Add a dependency: task_id depends on depends_on.", {
-    task_id: exports_external.string().describe("Task that depends on another"),
-    depends_on: exports_external.string().describe("Task that must complete first")
+    task_id: exports_external.string(),
+    depends_on: exports_external.string()
   }, async ({ task_id, depends_on }) => {
     try {
       const resolvedTaskId = resolveId(task_id);
@@ -8585,8 +8597,8 @@ Parent: ${task.parent.id.slice(0, 8)} | ${task.parent.title}`);
     }
   });
   server.tool("remove_dependency", "Remove a dependency between tasks", {
-    task_id: exports_external.string().describe("Task ID"),
-    depends_on: exports_external.string().describe("Dependency to remove")
+    task_id: exports_external.string(),
+    depends_on: exports_external.string()
   }, async ({ task_id, depends_on }) => {
     try {
       const resolvedTaskId = resolveId(task_id);
@@ -8603,10 +8615,10 @@ Parent: ${task.parent.id.slice(0, 8)} | ${task.parent.title}`);
     }
   });
   server.tool("add_comment", "Add a comment/note to a task", {
-    task_id: exports_external.string().describe("Task ID (full or partial)"),
-    content: exports_external.string().describe("Comment content"),
-    agent_id: exports_external.string().optional().describe("Agent adding comment"),
-    session_id: exports_external.string().optional().describe("Session ID")
+    task_id: exports_external.string(),
+    content: exports_external.string(),
+    agent_id: exports_external.string().optional(),
+    session_id: exports_external.string().optional()
   }, async ({ task_id, ...rest }) => {
     try {
       const resolvedId = resolveId(task_id);
@@ -8634,10 +8646,10 @@ ${text}` }] };
     }
   });
   server.tool("create_project", "Register a new project", {
-    name: exports_external.string().describe("Project name"),
-    path: exports_external.string().describe("Absolute path to project"),
-    description: exports_external.string().optional().describe("Project description"),
-    task_list_id: exports_external.string().optional().describe("Custom task list ID for Claude Code sync (defaults to todos-<slugified-name>)")
+    name: exports_external.string(),
+    path: exports_external.string(),
+    description: exports_external.string().optional(),
+    task_list_id: exports_external.string().optional()
   }, async (params) => {
     try {
       const project = createProject(params);
@@ -8653,12 +8665,12 @@ ${text}` }] };
     }
   });
   server.tool("create_plan", "Create a new plan", {
-    name: exports_external.string().describe("Plan name"),
-    project_id: exports_external.string().optional().describe("Project ID"),
-    description: exports_external.string().optional().describe("Plan description"),
-    status: exports_external.enum(["active", "completed", "archived"]).optional().describe("Plan status"),
-    task_list_id: exports_external.string().optional().describe("Task list ID"),
-    agent_id: exports_external.string().optional().describe("Owner agent ID")
+    name: exports_external.string(),
+    project_id: exports_external.string().optional(),
+    description: exports_external.string().optional(),
+    status: exports_external.enum(["active", "completed", "archived"]).optional(),
+    task_list_id: exports_external.string().optional(),
+    agent_id: exports_external.string().optional()
   }, async (params) => {
     try {
       const resolved = { ...params };
@@ -8678,7 +8690,7 @@ ${text}` }] };
     }
   });
   server.tool("list_plans", "List plans with optional project filter", {
-    project_id: exports_external.string().optional().describe("Filter by project")
+    project_id: exports_external.string().optional()
   }, async ({ project_id }) => {
     try {
       const resolvedProjectId = project_id ? resolveId(project_id, "projects") : undefined;
@@ -8698,7 +8710,7 @@ ${text}` }] };
     }
   });
   server.tool("get_plan", "Get plan details", {
-    id: exports_external.string().describe("Plan ID (full or partial)")
+    id: exports_external.string()
   }, async ({ id }) => {
     try {
       const resolvedId = resolveId(id, "plans");
@@ -8723,12 +8735,12 @@ ${text}` }] };
     }
   });
   server.tool("update_plan", "Update a plan", {
-    id: exports_external.string().describe("Plan ID (full or partial)"),
-    name: exports_external.string().optional().describe("New name"),
-    description: exports_external.string().optional().describe("New description"),
-    status: exports_external.enum(["active", "completed", "archived"]).optional().describe("New status"),
-    task_list_id: exports_external.string().optional().describe("Task list ID"),
-    agent_id: exports_external.string().optional().describe("Owner agent ID")
+    id: exports_external.string(),
+    name: exports_external.string().optional(),
+    description: exports_external.string().optional(),
+    status: exports_external.enum(["active", "completed", "archived"]).optional(),
+    task_list_id: exports_external.string().optional(),
+    agent_id: exports_external.string().optional()
   }, async ({ id, ...rest }) => {
     try {
       const resolvedId = resolveId(id, "plans");
@@ -8747,7 +8759,7 @@ ${text}` }] };
     }
   });
   server.tool("delete_plan", "Delete a plan", {
-    id: exports_external.string().describe("Plan ID (full or partial)")
+    id: exports_external.string()
   }, async ({ id }) => {
     try {
       const resolvedId = resolveId(id, "plans");
@@ -8763,9 +8775,9 @@ ${text}` }] };
     }
   });
   server.tool("search_tasks", "Full-text search across task titles, descriptions, tags.", {
-    query: exports_external.string().describe("Search query"),
-    project_id: exports_external.string().optional().describe("Limit to project"),
-    task_list_id: exports_external.string().optional().describe("Filter by task list")
+    query: exports_external.string(),
+    project_id: exports_external.string().optional(),
+    task_list_id: exports_external.string().optional()
   }, async ({ query, project_id, task_list_id }) => {
     try {
       const resolvedProjectId = project_id ? resolveId(project_id, "projects") : undefined;
@@ -8783,12 +8795,12 @@ ${text}` }] };
     }
   });
   server.tool("sync", "Sync tasks with an agent task list.", {
-    task_list_id: exports_external.string().optional().describe("Task list ID (required for Claude)"),
-    agent: exports_external.string().optional().describe("Agent/provider name (default: claude)"),
-    all_agents: exports_external.boolean().optional().describe("Sync across all configured agents"),
-    project_id: exports_external.string().optional().describe("Project ID \u2014 its task_list_id will be used for Claude if task_list_id is not provided"),
-    direction: exports_external.enum(["push", "pull", "both"]).optional().describe("Sync direction: push (SQLite->agent), pull (agent->SQLite), or both (default)"),
-    prefer: exports_external.enum(["local", "remote"]).optional().describe("Conflict strategy")
+    task_list_id: exports_external.string().optional(),
+    agent: exports_external.string().optional(),
+    all_agents: exports_external.boolean().optional(),
+    project_id: exports_external.string().optional(),
+    direction: exports_external.enum(["push", "pull", "both"]).optional(),
+    prefer: exports_external.enum(["local", "remote"]).optional()
   }, async ({ task_list_id, agent, all_agents, project_id, direction, prefer }) => {
     try {
       const resolvedProjectId = project_id ? resolveId(project_id, "projects") : undefined;
@@ -8831,8 +8843,8 @@ ${text}` }] };
     }
   });
   server.tool("register_agent", "Register an agent (idempotent by name).", {
-    name: exports_external.string().describe("Agent name"),
-    description: exports_external.string().optional().describe("Agent description")
+    name: exports_external.string(),
+    description: exports_external.string().optional()
   }, async ({ name, description }) => {
     try {
       const agent = registerAgent({ name, description });
@@ -8868,8 +8880,8 @@ ${text}` }] };
     }
   });
   server.tool("get_agent", "Get agent details by ID or name", {
-    id: exports_external.string().optional().describe("Agent ID"),
-    name: exports_external.string().optional().describe("Agent name")
+    id: exports_external.string().optional(),
+    name: exports_external.string().optional()
   }, async ({ id, name }) => {
     try {
       if (!id && !name) {
@@ -8896,10 +8908,10 @@ ${text}` }] };
     }
   });
   server.tool("create_task_list", "Create a new task list", {
-    name: exports_external.string().describe("Task list name"),
-    slug: exports_external.string().optional().describe("URL-friendly slug (auto-generated from name if omitted)"),
-    project_id: exports_external.string().optional().describe("Project ID to associate with"),
-    description: exports_external.string().optional().describe("Task list description")
+    name: exports_external.string(),
+    slug: exports_external.string().optional(),
+    project_id: exports_external.string().optional(),
+    description: exports_external.string().optional()
   }, async (params) => {
     try {
       const resolved = { ...params };
@@ -8922,7 +8934,7 @@ Description: ${list.description}` : ""}`
     }
   });
   server.tool("list_task_lists", "List task lists, optionally filtered by project", {
-    project_id: exports_external.string().optional().describe("Filter by project")
+    project_id: exports_external.string().optional()
   }, async ({ project_id }) => {
     try {
       const resolvedProjectId = project_id ? resolveId(project_id, "projects") : undefined;
@@ -8942,7 +8954,7 @@ ${text}` }] };
     }
   });
   server.tool("get_task_list", "Get task list details", {
-    id: exports_external.string().describe("Task list ID (full or partial)")
+    id: exports_external.string()
   }, async ({ id }) => {
     try {
       const resolvedId = resolveId(id, "task_lists");
@@ -8970,9 +8982,9 @@ ${text}` }] };
     }
   });
   server.tool("update_task_list", "Update a task list", {
-    id: exports_external.string().describe("Task list ID (full or partial)"),
-    name: exports_external.string().optional().describe("New name"),
-    description: exports_external.string().optional().describe("New description")
+    id: exports_external.string(),
+    name: exports_external.string().optional(),
+    description: exports_external.string().optional()
   }, async ({ id, ...rest }) => {
     try {
       const resolvedId = resolveId(id, "task_lists");
@@ -8991,7 +9003,7 @@ Slug: ${list.slug}`
     }
   });
   server.tool("delete_task_list", "Delete a task list. Tasks lose association but keep data.", {
-    id: exports_external.string().describe("Task list ID (full or partial)")
+    id: exports_external.string()
   }, async ({ id }) => {
     try {
       const resolvedId = resolveId(id, "task_lists");
@@ -9007,7 +9019,7 @@ Slug: ${list.slug}`
     }
   });
   server.tool("get_task_history", "Get audit log for a task.", {
-    task_id: exports_external.string().describe("Task ID (full or partial)")
+    task_id: exports_external.string()
   }, async ({ task_id }) => {
     try {
       const resolvedId = resolveId(task_id);
@@ -9024,7 +9036,7 @@ ${text}` }] };
     }
   });
   server.tool("get_recent_activity", "Get recent task changes across all tasks.", {
-    limit: exports_external.number().optional().describe("Max entries (default 50)")
+    limit: exports_external.number().optional()
   }, async ({ limit }) => {
     try {
       const { getRecentActivity: getRecentActivity2 } = await Promise.resolve().then(() => (init_audit(), exports_audit));
@@ -9040,9 +9052,9 @@ ${text}` }] };
     }
   });
   server.tool("create_webhook", "Register a webhook to receive task change events.", {
-    url: exports_external.string().describe("Webhook URL"),
-    events: exports_external.array(exports_external.string()).optional().describe("Event types to subscribe to (empty = all)"),
-    secret: exports_external.string().optional().describe("HMAC secret for signature verification")
+    url: exports_external.string(),
+    events: exports_external.array(exports_external.string()).optional(),
+    secret: exports_external.string().optional()
   }, async (params) => {
     try {
       const { createWebhook: createWebhook2 } = await Promise.resolve().then(() => (init_webhooks(), exports_webhooks));
@@ -9067,7 +9079,7 @@ ${text}` }] };
     }
   });
   server.tool("delete_webhook", "Delete a webhook", {
-    id: exports_external.string().describe("Webhook ID")
+    id: exports_external.string()
   }, async ({ id }) => {
     try {
       const { deleteWebhook: deleteWebhook2 } = await Promise.resolve().then(() => (init_webhooks(), exports_webhooks));
@@ -9078,13 +9090,13 @@ ${text}` }] };
     }
   });
   server.tool("create_template", "Create a reusable task template.", {
-    name: exports_external.string().describe("Template name"),
-    title_pattern: exports_external.string().describe("Title pattern for tasks created from this template"),
-    description: exports_external.string().optional().describe("Default description"),
-    priority: exports_external.enum(["low", "medium", "high", "critical"]).optional().describe("Default priority"),
-    tags: exports_external.array(exports_external.string()).optional().describe("Default tags"),
-    project_id: exports_external.string().optional().describe("Default project"),
-    plan_id: exports_external.string().optional().describe("Default plan")
+    name: exports_external.string(),
+    title_pattern: exports_external.string(),
+    description: exports_external.string().optional(),
+    priority: exports_external.enum(["low", "medium", "high", "critical"]).optional(),
+    tags: exports_external.array(exports_external.string()).optional(),
+    project_id: exports_external.string().optional(),
+    plan_id: exports_external.string().optional()
   }, async (params) => {
     try {
       const { createTemplate: createTemplate2 } = await Promise.resolve().then(() => (init_templates(), exports_templates));
@@ -9109,12 +9121,12 @@ ${text}` }] };
     }
   });
   server.tool("create_task_from_template", "Create a task from a template with optional overrides.", {
-    template_id: exports_external.string().describe("Template ID"),
-    title: exports_external.string().optional().describe("Override title"),
-    description: exports_external.string().optional().describe("Override description"),
-    priority: exports_external.enum(["low", "medium", "high", "critical"]).optional().describe("Override priority"),
-    assigned_to: exports_external.string().optional().describe("Assign to agent"),
-    project_id: exports_external.string().optional().describe("Override project")
+    template_id: exports_external.string(),
+    title: exports_external.string().optional(),
+    description: exports_external.string().optional(),
+    priority: exports_external.enum(["low", "medium", "high", "critical"]).optional(),
+    assigned_to: exports_external.string().optional(),
+    project_id: exports_external.string().optional()
   }, async (params) => {
     try {
       const { taskFromTemplate: taskFromTemplate2 } = await Promise.resolve().then(() => (init_templates(), exports_templates));
@@ -9132,7 +9144,7 @@ ${task.id.slice(0, 8)} | ${task.priority} | ${task.title}` }] };
       return { content: [{ type: "text", text: formatError(e) }], isError: true };
     }
   });
-  server.tool("delete_template", "Delete a task template", { id: exports_external.string().describe("Template ID") }, async ({ id }) => {
+  server.tool("delete_template", "Delete a task template", { id: exports_external.string() }, async ({ id }) => {
     try {
       const { deleteTemplate: deleteTemplate2 } = await Promise.resolve().then(() => (init_templates(), exports_templates));
       const deleted = deleteTemplate2(id);
@@ -9142,8 +9154,8 @@ ${task.id.slice(0, 8)} | ${task.priority} | ${task.title}` }] };
     }
   });
   server.tool("approve_task", "Approve a task that requires approval.", {
-    id: exports_external.string().describe("Task ID (full or partial)"),
-    agent_id: exports_external.string().optional().describe("Agent approving the task")
+    id: exports_external.string(),
+    agent_id: exports_external.string().optional()
   }, async ({ id, agent_id }) => {
     try {
       const resolvedId = resolveId(id);
@@ -9161,7 +9173,7 @@ ${task.id.slice(0, 8)} | ${task.priority} | ${task.title}` }] };
     }
   });
   server.tool("get_my_tasks", "Get assigned tasks and stats for an agent.", {
-    agent_name: exports_external.string().describe("Your agent name")
+    agent_name: exports_external.string()
   }, async ({ agent_name }) => {
     try {
       const agent = registerAgent({ name: agent_name });
@@ -9193,7 +9205,7 @@ In Progress:`);
       return { content: [{ type: "text", text: formatError(e) }], isError: true };
     }
   });
-  server.tool("search_tools", "List tool names matching a query.", { query: exports_external.string().optional().describe("Keyword to filter tools") }, async ({ query }) => {
+  server.tool("search_tools", "List tool names matching a query.", { query: exports_external.string().optional() }, async ({ query }) => {
     const all = [
       "create_task",
       "list_tasks",
@@ -9242,7 +9254,7 @@ In Progress:`);
     const matches = q ? all.filter((n) => n.includes(q)) : all;
     return { content: [{ type: "text", text: matches.join(", ") }] };
   });
-  server.tool("describe_tools", "Get descriptions for specific tools by name.", { names: exports_external.array(exports_external.string()).describe("Tool names from search_tools") }, async ({ names }) => {
+  server.tool("describe_tools", "Get descriptions for specific tools by name.", { names: exports_external.array(exports_external.string()) }, async ({ names }) => {
     const descriptions = {
       create_task: "Create a task. Params: title(req), description, priority, project_id, plan_id, tags, assigned_to, estimated_minutes, requires_approval",
       list_tasks: "List tasks. Params: status, priority, project_id, plan_id, assigned_to, tags, limit",

package/dist/index.js

@@ -1115,7 +1115,18 @@ function updateTask(id, input, db) {
     logTaskChange(id, "update", "assigned_to", task.assigned_to, input.assigned_to, agentId, d);
   if (input.approved_by !== undefined)
     logTaskChange(id, "approve", "approved_by", null, input.approved_by, agentId, d);
-  return getTask(id, d);
+  return {
+    ...task,
+    ...Object.fromEntries(Object.entries(input).filter(([, v]) => v !== undefined)),
+    tags: input.tags ?? task.tags,
+    metadata: input.metadata ?? task.metadata,
+    version: task.version + 1,
+    updated_at: now(),
+    completed_at: input.status === "completed" ? now() : task.completed_at,
+    requires_approval: input.requires_approval !== undefined ? input.requires_approval : task.requires_approval,
+    approved_by: input.approved_by ?? task.approved_by,
+    approved_at: input.approved_by ? now() : task.approved_at
+  };
 }
 function deleteTask(id, db) {
   const d = db || getDatabase();
@@ -1137,6 +1148,9 @@ function getBlockingDeps(id, db) {
 }
 function startTask(id, agentId, db) {
   const d = db || getDatabase();
+  const task = getTask(id, d);
+  if (!task)
+    throw new TaskNotFoundError(id);
   const blocking = getBlockingDeps(id, d);
   if (blocking.length > 0) {
     const blockerIds = blocking.map((b) => b.id.slice(0, 8)).join(", ");
@@ -1147,15 +1161,12 @@ function startTask(id, agentId, db) {
   const result = d.run(`UPDATE tasks SET status = 'in_progress', assigned_to = ?, locked_by = ?, locked_at = ?, version = version + 1, updated_at = ?
      WHERE id = ? AND (locked_by IS NULL OR locked_by = ? OR locked_at < ?)`, [agentId, agentId, timestamp, timestamp, id, agentId, cutoff]);
   if (result.changes === 0) {
-    const current = getTask(id, d);
-    if (!current)
-      throw new TaskNotFoundError(id);
-    if (current.locked_by && current.locked_by !== agentId && !isLockExpired(current.locked_at)) {
-      throw new LockError(id, current.locked_by);
+    if (task.locked_by && task.locked_by !== agentId && !isLockExpired(task.locked_at)) {
+      throw new LockError(id, task.locked_by);
     }
   }
   logTaskChange(id, "start", "status", "pending", "in_progress", agentId, d);
-  return getTask(id, d);
+  return { ...task, status: "in_progress", assigned_to: agentId, locked_by: agentId, locked_at: timestamp, version: task.version + 1, updated_at: timestamp };
 }
 function completeTask(id, agentId, db, evidence) {
   const d = db || getDatabase();
@@ -1167,14 +1178,15 @@ function completeTask(id, agentId, db, evidence) {
   }
   checkCompletionGuard(task, agentId || null, d);
   if (evidence) {
-    const meta = { ...task.metadata, _evidence: evidence };
-    d.run("UPDATE tasks SET metadata = ? WHERE id = ?", [JSON.stringify(meta), id]);
+    const meta2 = { ...task.metadata, _evidence: evidence };
+    d.run("UPDATE tasks SET metadata = ? WHERE id = ?", [JSON.stringify(meta2), id]);
   }
   const timestamp = now();
   d.run(`UPDATE tasks SET status = 'completed', locked_by = NULL, locked_at = NULL, completed_at = ?, version = version + 1, updated_at = ?
      WHERE id = ?`, [timestamp, timestamp, id]);
   logTaskChange(id, "complete", "status", task.status, "completed", agentId || null, d);
-  return getTask(id, d);
+  const meta = evidence ? { ...task.metadata, _evidence: evidence } : task.metadata;
+  return { ...task, status: "completed", locked_by: null, locked_at: null, completed_at: timestamp, version: task.version + 1, updated_at: timestamp, metadata: meta };
 }
 function lockTask(id, agentId, db) {
   const d = db || getDatabase();

package/dist/mcp/index.js

@@ -5154,7 +5154,18 @@ function updateTask(id, input, db) {
     logTaskChange(id, "update", "assigned_to", task.assigned_to, input.assigned_to, agentId, d);
   if (input.approved_by !== undefined)
     logTaskChange(id, "approve", "approved_by", null, input.approved_by, agentId, d);
-  return getTask(id, d);
+  return {
+    ...task,
+    ...Object.fromEntries(Object.entries(input).filter(([, v]) => v !== undefined)),
+    tags: input.tags ?? task.tags,
+    metadata: input.metadata ?? task.metadata,
+    version: task.version + 1,
+    updated_at: now(),
+    completed_at: input.status === "completed" ? now() : task.completed_at,
+    requires_approval: input.requires_approval !== undefined ? input.requires_approval : task.requires_approval,
+    approved_by: input.approved_by ?? task.approved_by,
+    approved_at: input.approved_by ? now() : task.approved_at
+  };
 }
 function deleteTask(id, db) {
   const d = db || getDatabase();
@@ -5176,6 +5187,9 @@ function getBlockingDeps(id, db) {
 }
 function startTask(id, agentId, db) {
   const d = db || getDatabase();
+  const task = getTask(id, d);
+  if (!task)
+    throw new TaskNotFoundError(id);
   const blocking = getBlockingDeps(id, d);
   if (blocking.length > 0) {
     const blockerIds = blocking.map((b) => b.id.slice(0, 8)).join(", ");
@@ -5186,15 +5200,12 @@ function startTask(id, agentId, db) {
   const result = d.run(`UPDATE tasks SET status = 'in_progress', assigned_to = ?, locked_by = ?, locked_at = ?, version = version + 1, updated_at = ?
      WHERE id = ? AND (locked_by IS NULL OR locked_by = ? OR locked_at < ?)`, [agentId, agentId, timestamp, timestamp, id, agentId, cutoff]);
   if (result.changes === 0) {
-    const current = getTask(id, d);
-    if (!current)
-      throw new TaskNotFoundError(id);
-    if (current.locked_by && current.locked_by !== agentId && !isLockExpired(current.locked_at)) {
-      throw new LockError(id, current.locked_by);
+    if (task.locked_by && task.locked_by !== agentId && !isLockExpired(task.locked_at)) {
+      throw new LockError(id, task.locked_by);
     }
   }
   logTaskChange(id, "start", "status", "pending", "in_progress", agentId, d);
-  return getTask(id, d);
+  return { ...task, status: "in_progress", assigned_to: agentId, locked_by: agentId, locked_at: timestamp, version: task.version + 1, updated_at: timestamp };
 }
 function completeTask(id, agentId, db, evidence) {
   const d = db || getDatabase();
@@ -5206,14 +5217,15 @@ function completeTask(id, agentId, db, evidence) {
   }
   checkCompletionGuard(task, agentId || null, d);
   if (evidence) {
-    const meta = { ...task.metadata, _evidence: evidence };
-    d.run("UPDATE tasks SET metadata = ? WHERE id = ?", [JSON.stringify(meta), id]);
+    const meta2 = { ...task.metadata, _evidence: evidence };
+    d.run("UPDATE tasks SET metadata = ? WHERE id = ?", [JSON.stringify(meta2), id]);
   }
   const timestamp = now();
   d.run(`UPDATE tasks SET status = 'completed', locked_by = NULL, locked_at = NULL, completed_at = ?, version = version + 1, updated_at = ?
      WHERE id = ?`, [timestamp, timestamp, id]);
   logTaskChange(id, "complete", "status", task.status, "completed", agentId || null, d);
-  return getTask(id, d);
+  const meta = evidence ? { ...task.metadata, _evidence: evidence } : task.metadata;
+  return { ...task, status: "completed", locked_by: null, locked_at: null, completed_at: timestamp, version: task.version + 1, updated_at: timestamp, metadata: meta };
 }
 function lockTask(id, agentId, db) {
   const d = db || getDatabase();

package/dist/server/index.js

@@ -1233,7 +1233,18 @@ function updateTask(id, input, db) {
     logTaskChange(id, "update", "assigned_to", task.assigned_to, input.assigned_to, agentId, d);
   if (input.approved_by !== undefined)
     logTaskChange(id, "approve", "approved_by", null, input.approved_by, agentId, d);
-  return getTask(id, d);
+  return {
+    ...task,
+    ...Object.fromEntries(Object.entries(input).filter(([, v]) => v !== undefined)),
+    tags: input.tags ?? task.tags,
+    metadata: input.metadata ?? task.metadata,
+    version: task.version + 1,
+    updated_at: now(),
+    completed_at: input.status === "completed" ? now() : task.completed_at,
+    requires_approval: input.requires_approval !== undefined ? input.requires_approval : task.requires_approval,
+    approved_by: input.approved_by ?? task.approved_by,
+    approved_at: input.approved_by ? now() : task.approved_at
+  };
 }
 function deleteTask(id, db) {
   const d = db || getDatabase();
@@ -1255,6 +1266,9 @@ function getBlockingDeps(id, db) {
 }
 function startTask(id, agentId, db) {
   const d = db || getDatabase();
+  const task = getTask(id, d);
+  if (!task)
+    throw new TaskNotFoundError(id);
   const blocking = getBlockingDeps(id, d);
   if (blocking.length > 0) {
     const blockerIds = blocking.map((b) => b.id.slice(0, 8)).join(", ");
@@ -1265,15 +1279,12 @@ function startTask(id, agentId, db) {
   const result = d.run(`UPDATE tasks SET status = 'in_progress', assigned_to = ?, locked_by = ?, locked_at = ?, version = version + 1, updated_at = ?
      WHERE id = ? AND (locked_by IS NULL OR locked_by = ? OR locked_at < ?)`, [agentId, agentId, timestamp, timestamp, id, agentId, cutoff]);
   if (result.changes === 0) {
-    const current = getTask(id, d);
-    if (!current)
-      throw new TaskNotFoundError(id);
-    if (current.locked_by && current.locked_by !== agentId && !isLockExpired(current.locked_at)) {
-      throw new LockError(id, current.locked_by);
+    if (task.locked_by && task.locked_by !== agentId && !isLockExpired(task.locked_at)) {
+      throw new LockError(id, task.locked_by);
     }
   }
   logTaskChange(id, "start", "status", "pending", "in_progress", agentId, d);
-  return getTask(id, d);
+  return { ...task, status: "in_progress", assigned_to: agentId, locked_by: agentId, locked_at: timestamp, version: task.version + 1, updated_at: timestamp };
 }
 function completeTask(id, agentId, db, evidence) {
   const d = db || getDatabase();
@@ -1285,14 +1296,15 @@ function completeTask(id, agentId, db, evidence) {
   }
   checkCompletionGuard(task, agentId || null, d);
   if (evidence) {
-    const meta = { ...task.metadata, _evidence: evidence };
-    d.run("UPDATE tasks SET metadata = ? WHERE id = ?", [JSON.stringify(meta), id]);
+    const meta2 = { ...task.metadata, _evidence: evidence };
+    d.run("UPDATE tasks SET metadata = ? WHERE id = ?", [JSON.stringify(meta2), id]);
   }
   const timestamp = now();
   d.run(`UPDATE tasks SET status = 'completed', locked_by = NULL, locked_at = NULL, completed_at = ?, version = version + 1, updated_at = ?
      WHERE id = ?`, [timestamp, timestamp, id]);
   logTaskChange(id, "complete", "status", task.status, "completed", agentId || null, d);
-  return getTask(id, d);
+  const meta = evidence ? { ...task.metadata, _evidence: evidence } : task.metadata;
+  return { ...task, status: "completed", locked_by: null, locked_at: null, completed_at: timestamp, version: task.version + 1, updated_at: timestamp, metadata: meta };
 }
 function getTaskDependencies(taskId, db) {
   const d = db || getDatabase();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hasna/todos",
-  "version": "0.9.28",
+  "version": "0.9.29",
   "description": "Universal task management for AI coding agents - CLI + MCP server + interactive TUI",
   "type": "module",
   "main": "dist/index.js",