omnifocus-mcp-enhanced 1.6.7 → 1.6.8

package/README.md

@@ -17,6 +17,7 @@ Enhanced Model Context Protocol (MCP) server for OmniFocus featuring **native cu
 
 ## 🆕 Latest Release
 
+- **v1.6.8** - Added stable task move support via `move_task` and `edit_item` (`newProjectId/newProjectName/newParentTaskId/newParentTaskName/moveToInbox`) with duplicate-name protection and cycle-prevention checks.
 - **v1.6.6** - Added full Planned Date support (create/edit/read/filter/sort/export), including `plannedDate`/`newPlannedDate` and updated task displays.
 
 ## ✨ Key Features
@@ -221,24 +222,27 @@ Efficiently manage multiple tasks:
 2. **add_omnifocus_task** - Create tasks (enhanced with subtask support)
 3. **add_project** - Create projects
 4. **remove_item** - Delete tasks or projects
-5. **edit_item** - Edit tasks or projects
-6. **batch_add_items** - Bulk add (enhanced with subtask support)
-7. **batch_remove_items** - Bulk remove
-8. **get_task_by_id** - Query task information
+5. **edit_item** - Edit tasks or projects (now supports task moves: project/parent/inbox)
+6. **move_task** - Move an existing task to project/parent task/inbox
+7. **batch_add_items** - Bulk add (enhanced with subtask support)
+8. **batch_remove_items** - Bulk remove
+9. **get_task_by_id** - Query task information
 
 ### 🔍 Built-in Perspective Tools
-9. **get_inbox_tasks** - Inbox perspective
-10. **get_flagged_tasks** - Flagged perspective
-11. **get_forecast_tasks** - Forecast perspective (due/deferred/planned task data included)
-12. **get_tasks_by_tag** - Tag-based filtering
-13. **filter_tasks** - Ultimate filtering with unlimited combinations
+10. **get_inbox_tasks** - Inbox perspective
+11. **get_flagged_tasks** - Flagged perspective
+12. **get_forecast_tasks** - Forecast perspective (due/deferred/planned task data included)
+13. **get_tasks_by_tag** - Tag-based filtering
+14. **filter_tasks** - Ultimate filtering with unlimited combinations
 
 ### 🌟 Custom Perspective Tools (NEW)
-14. **list_custom_perspectives** - 🌟 **NEW**: List all custom perspectives with details
-15. **get_custom_perspective_tasks** - 🌟 **NEW**: Access custom perspective with hierarchical display
+15. **list_custom_perspectives** - 🌟 **NEW**: List all custom perspectives with details
+16. **get_custom_perspective_tasks** - 🌟 **NEW**: Access custom perspective with hierarchical display
 
 ### 📊 Analytics & Tracking
-16. **get_today_completed_tasks** - View today's completed tasks
+17. **get_today_completed_tasks** - View today's completed tasks
+
+Batch move feature roadmap (future): [docs/roadmap/2026-02-25-batch-move-tasks-plan.md](docs/roadmap/2026-02-25-batch-move-tasks-plan.md)
 
 ## 🚀 Quick Start Examples
 
@@ -272,6 +276,43 @@ add_omnifocus_task {
 }
 ```
 
+### Task Move Operations
+```bash
+# Move task to a project
+move_task {
+  "id": "task-id-123",
+  "targetProjectName": "Planning"
+}
+
+# Move task under another task
+move_task {
+  "id": "task-id-123",
+  "targetParentTaskId": "parent-task-id-456"
+}
+
+# Move task back to inbox
+move_task {
+  "id": "task-id-123",
+  "targetInbox": true
+}
+```
+
+Task move safety rules:
+- Name lookups fail fast on duplicates and ask you to use IDs.
+- Destination must be exactly one type: project OR parent task OR inbox.
+- Moving a task into itself/its descendants is blocked to prevent cycles.
+
+You can also move with `edit_item` and combine move + field updates:
+```bash
+edit_item {
+  "itemType": "task",
+  "id": "task-id-123",
+  "newProjectName": "Planning",
+  "newName": "Review tmux workflow",
+  "newFlagged": true
+}
+```
+
 ### Smart Task Discovery
 ```bash
 # Find high-priority work

package/README.zh.md

@@ -217,24 +217,27 @@ get_custom_perspective_tasks {
 2. **add_omnifocus_task** - 创建任务（增强子任务支持）
 3. **add_project** - 创建项目
 4. **remove_item** - 删除任务或项目
-5. **edit_item** - 编辑任务或项目
-6. **batch_add_items** - 批量添加（增强子任务支持）
-7. **batch_remove_items** - 批量删除
-8. **get_task_by_id** - 查询任务信息
+5. **edit_item** - 编辑任务或项目（现已支持任务转移：项目/父任务/Inbox）
+6. **move_task** - 将已有任务转移到项目/父任务/Inbox
+7. **batch_add_items** - 批量添加（增强子任务支持）
+8. **batch_remove_items** - 批量删除
+9. **get_task_by_id** - 查询任务信息
 
 ### 🔍 内置透视工具
-9. **get_inbox_tasks** - 收件箱透视
-10. **get_flagged_tasks** - 已标记透视
-11. **get_forecast_tasks** - 预测透视（包含截止/推迟/计划日期任务数据）
-12. **get_tasks_by_tag** - 基于标签的过滤
-13. **filter_tasks** - 无限组合的终极过滤
+10. **get_inbox_tasks** - 收件箱透视
+11. **get_flagged_tasks** - 已标记透视
+12. **get_forecast_tasks** - 预测透视（包含截止/推迟/计划日期任务数据）
+13. **get_tasks_by_tag** - 基于标签的过滤
+14. **filter_tasks** - 无限组合的终极过滤
 
 ### 🌟 自定义透视工具（新功能）
-14. **list_custom_perspectives** - 🌟 **新功能**：列出所有自定义透视及详情
-15. **get_custom_perspective_tasks** - 🌟 **新功能**：访问自定义透视，支持层级显示
+15. **list_custom_perspectives** - 🌟 **新功能**：列出所有自定义透视及详情
+16. **get_custom_perspective_tasks** - 🌟 **新功能**：访问自定义透视，支持层级显示
 
 ### 📊 分析与跟踪
-16. **get_today_completed_tasks** - 查看今日完成的任务
+17. **get_today_completed_tasks** - 查看今日完成的任务
+
+批量转移功能后续计划（Roadmap）：[docs/roadmap/2026-02-25-batch-move-tasks-plan.zh.md](docs/roadmap/2026-02-25-batch-move-tasks-plan.zh.md)
 
 ## 🚀 快速开始示例
 
@@ -268,6 +271,27 @@ add_omnifocus_task {
 }
 ```
 
+### 任务转移操作
+```bash
+# 转移到项目
+move_task {
+  "id": "task-id-123",
+  "targetProjectName": "规划"
+}
+
+# 转移到父任务下
+move_task {
+  "id": "task-id-123",
+  "targetParentTaskId": "parent-task-id-456"
+}
+
+# 转移回 Inbox
+move_task {
+  "id": "task-id-123",
+  "targetInbox": true
+}
+```
+
 ### 智能任务发现
 ```bash
 # 找到高优先级工作

package/dist/server.js

@@ -7,6 +7,7 @@ import * as addOmniFocusTaskTool from './tools/definitions/addOmniFocusTask.js';
 import * as addProjectTool from './tools/definitions/addProject.js';
 import * as removeItemTool from './tools/definitions/removeItem.js';
 import * as editItemTool from './tools/definitions/editItem.js';
+import * as moveTaskTool from './tools/definitions/moveTask.js';
 import * as batchAddItemsTool from './tools/definitions/batchAddItems.js';
 import * as batchRemoveItemsTool from './tools/definitions/batchRemoveItems.js';
 import * as getTaskByIdTool from './tools/definitions/getTaskById.js';
@@ -32,6 +33,7 @@ server.tool("add_omnifocus_task", "Add a new task to OmniFocus", addOmniFocusTas
 server.tool("add_project", "Add a new project to OmniFocus", addProjectTool.schema.shape, addProjectTool.handler);
 server.tool("remove_item", "Remove a task or project from OmniFocus", removeItemTool.schema.shape, removeItemTool.handler);
 server.tool("edit_item", "Edit a task or project in OmniFocus", editItemTool.schema.shape, editItemTool.handler);
+server.tool("move_task", "Move an existing task to a project, parent task, or inbox", moveTaskTool.schema.shape, moveTaskTool.handler);
 server.tool("batch_add_items", "Add multiple tasks or projects to OmniFocus in a single operation", batchAddItemsTool.schema.shape, batchAddItemsTool.handler);
 server.tool("batch_remove_items", "Remove multiple tasks or projects from OmniFocus in a single operation", batchRemoveItemsTool.schema.shape, batchRemoveItemsTool.handler);
 server.tool("get_task_by_id", "Get information about a specific task by ID or name", getTaskByIdTool.schema.shape, getTaskByIdTool.handler);

package/dist/tools/definitions/editItem.js

@@ -17,6 +17,11 @@ export const schema = z.object({
     addTags: z.array(z.string()).optional().describe("Tags to add to the task"),
     removeTags: z.array(z.string()).optional().describe("Tags to remove from the task"),
     replaceTags: z.array(z.string()).optional().describe("Tags to replace all existing tags with"),
+    newProjectId: z.string().optional().describe("For tasks: move task to this project ID"),
+    newProjectName: z.string().optional().describe("For tasks: move task to this project name (errors on duplicate names)"),
+    newParentTaskId: z.string().optional().describe("For tasks: move task under this parent task ID"),
+    newParentTaskName: z.string().optional().describe("For tasks: move task under this parent task name (errors on duplicate names)"),
+    moveToInbox: z.boolean().optional().describe("For tasks: move task to inbox"),
     // Project-specific fields
     newSequential: z.boolean().optional().describe("Whether the project should be sequential"),
     newFolderName: z.string().optional().describe("New folder to move the project to"),

package/dist/tools/definitions/moveTask.js

@@ -0,0 +1,54 @@
+import { z } from 'zod';
+import { moveTask } from '../primitives/moveTask.js';
+export const schema = z.object({
+    id: z.string().optional().describe('The ID of the task to move'),
+    name: z.string().optional().describe('The name of the task to move (fallback if ID not provided)'),
+    targetProjectId: z.string().optional().describe('Destination project ID'),
+    targetProjectName: z.string().optional().describe('Destination project name (errors on duplicate names)'),
+    targetParentTaskId: z.string().optional().describe('Destination parent task ID'),
+    targetParentTaskName: z.string().optional().describe('Destination parent task name (errors on duplicate names)'),
+    targetInbox: z.boolean().optional().describe('Move task to inbox')
+});
+function formatDestination(args) {
+    if (args.targetInbox) {
+        return 'inbox';
+    }
+    if (args.targetProjectId || args.targetProjectName) {
+        return `project "${args.targetProjectId || args.targetProjectName}"`;
+    }
+    if (args.targetParentTaskId || args.targetParentTaskName) {
+        return `parent task "${args.targetParentTaskId || args.targetParentTaskName}"`;
+    }
+    return 'destination';
+}
+export async function handler(args, extra) {
+    try {
+        const result = await moveTask(args);
+        if (result.success) {
+            return {
+                content: [{
+                        type: 'text',
+                        text: `✅ Task "${result.name || args.id || args.name}" moved successfully to ${formatDestination(args)}.`
+                    }]
+            };
+        }
+        return {
+            content: [{
+                    type: 'text',
+                    text: `Failed to move task: ${result.error}`
+                }],
+            isError: true
+        };
+    }
+    catch (err) {
+        const error = err;
+        console.error(`Tool execution error: ${error.message}`);
+        return {
+            content: [{
+                    type: 'text',
+                    text: `Error moving task: ${error.message}`
+                }],
+            isError: true
+        };
+    }
+}

package/dist/tools/definitions/moveTask.test.js

@@ -0,0 +1,27 @@
+import assert from 'node:assert/strict';
+import test from 'node:test';
+import { handler, schema } from './moveTask.js';
+test('move_task schema accepts source and destination fields', () => {
+    const parsed = schema.parse({
+        id: 'task-1',
+        targetProjectId: 'project-1'
+    });
+    assert.equal(parsed.id, 'task-1');
+    assert.equal(parsed.targetProjectId, 'project-1');
+});
+test('move_task schema preserves targetInbox boolean', () => {
+    const parsed = schema.parse({
+        name: 'My Task',
+        targetInbox: true
+    });
+    assert.equal(parsed.targetInbox, true);
+});
+test('move_task handler returns validation errors for conflicting destinations', async () => {
+    const result = await handler({
+        id: 'task-1',
+        targetProjectId: 'project-1',
+        targetInbox: true
+    }, {});
+    assert.equal(result.isError, true);
+    assert.match(result.content[0].text, /Exactly one destination/);
+});

package/dist/tools/definitions/plannedDateSchemas.test.js

@@ -45,6 +45,16 @@ test('edit_item schema preserves newPlannedDate', () => {
     });
     assert.equal(parsed.newPlannedDate, '2026-02-14');
 });
+test('edit_item schema supports task move destination fields', () => {
+    const parsed = editItemSchema.parse({
+        itemType: 'task',
+        id: 'abc',
+        newProjectId: 'project-1',
+        moveToInbox: false
+    });
+    assert.equal(parsed.newProjectId, 'project-1');
+    assert.equal(parsed.moveToInbox, false);
+});
 test('filter_tasks schema supports planned date filters and sorting', () => {
     const parsed = filterTasksSchema.parse({
         plannedToday: true,

package/dist/tools/primitives/addOmniFocusTask.js

@@ -1,5 +1,5 @@
 import { executeAppleScript } from '../../utils/scriptExecution.js';
-import { formatDateForAppleScript } from '../../utils/dateFormatter.js';
+import { appleScriptDateCode } from '../../utils/dateFormatter.js';
 export function buildTagAssignmentScript(tags, targetVar) {
     if (!tags || tags.length === 0) {
         return '';
@@ -24,14 +24,15 @@ export function buildTagAssignmentScript(tags, targetVar) {
 /**
  * Generate pure AppleScript for task creation
  */
-function generateAppleScript(params) {
+export function generateAppleScript(params) {
     // Sanitize and prepare parameters for AppleScript
     const name = params.name.replace(/['"\\]/g, '\\$&'); // Escape quotes and backslashes
     const note = params.note?.replace(/['"\\]/g, '\\$&') || '';
-    // Convert ISO dates to AppleScript format
-    const dueDate = params.dueDate ? formatDateForAppleScript(params.dueDate) : '';
-    const deferDate = params.deferDate ? formatDateForAppleScript(params.deferDate) : '';
-    const plannedDate = params.plannedDate ? formatDateForAppleScript(params.plannedDate) : '';
+    // Build date variables outside OmniFocus tell block to avoid locale parsing issues.
+    const dueDateCode = params.dueDate ? appleScriptDateCode(params.dueDate, 'dueDateValue') : '';
+    const deferDateCode = params.deferDate ? appleScriptDateCode(params.deferDate, 'deferDateValue') : '';
+    const plannedDateCode = params.plannedDate ? appleScriptDateCode(params.plannedDate, 'plannedDateValue') : '';
+    const datePreamble = [dueDateCode, deferDateCode, plannedDateCode].filter(Boolean).join('\n');
     const flagged = params.flagged === true;
     const estimatedMinutes = params.estimatedMinutes?.toString() || '';
     const tags = params.tags || [];
@@ -42,6 +43,7 @@ function generateAppleScript(params) {
     // Construct AppleScript with error handling
     let script = `
   try
+${datePreamble}
     tell application "OmniFocus"
       tell front document
         -- Determine the container (parent task, project, or inbox)
@@ -76,9 +78,9 @@ function generateAppleScript(params) {
         
         -- Set task properties
         ${note ? `set note of newTask to "${note}"` : ''}
-        ${dueDate ? `set due date of newTask to date "${dueDate}"` : ''}
-        ${deferDate ? `set defer date of newTask to date "${deferDate}"` : ''}
-        ${plannedDate ? `set planned date of newTask to date "${plannedDate}"` : ''}
+        ${params.dueDate ? `set due date of newTask to dueDateValue` : ''}
+        ${params.deferDate ? `set defer date of newTask to deferDateValue` : ''}
+        ${params.plannedDate ? `set planned date of newTask to plannedDateValue` : ''}
         ${flagged ? `set flagged of newTask to true` : ''}
         ${estimatedMinutes ? `set estimated minutes of newTask to ${estimatedMinutes}` : ''}
         

package/dist/tools/primitives/addOmniFocusTask.test.js

@@ -1,6 +1,6 @@
 import assert from 'node:assert/strict';
 import test from 'node:test';
-import { buildTagAssignmentScript } from './addOmniFocusTask.js';
+import { buildTagAssignmentScript, generateAppleScript } from './addOmniFocusTask.js';
 test('buildTagAssignmentScript creates missing tags before assignment', () => {
     const script = buildTagAssignmentScript(['mcp-test-tag'], 'newTask');
     assert.match(script, /set theTag to first flattened tag where name = "mcp-test-tag"/);
@@ -8,3 +8,20 @@ test('buildTagAssignmentScript creates missing tags before assignment', () => {
     assert.match(script, /set theTag to make new tag with properties \{name:"mcp-test-tag"\}/);
     assert.match(script, /add theTag to tags of newTask/);
 });
+test('generateAppleScript builds date variables before OmniFocus tell block', () => {
+    const script = generateAppleScript({
+        name: 'Task with dates',
+        dueDate: '2026-02-27',
+        deferDate: '2026-02-25',
+        plannedDate: '2026-02-24'
+    });
+    const tellIndex = script.indexOf('tell application "OmniFocus"');
+    const preambleIndex = script.indexOf('set dueDateValue to current date');
+    assert.ok(preambleIndex > -1 && preambleIndex < tellIndex);
+    assert.match(script, /set due date of newTask to dueDateValue/);
+    assert.match(script, /set defer date of newTask to deferDateValue/);
+    assert.match(script, /set planned date of newTask to plannedDateValue/);
+    assert.doesNotMatch(script, /set due date of newTask to date "/);
+    assert.doesNotMatch(script, /set defer date of newTask to date "/);
+    assert.doesNotMatch(script, /set planned date of newTask to date "/);
+});

package/dist/tools/primitives/addProject.js

@@ -1,16 +1,17 @@
 import { executeAppleScript } from '../../utils/scriptExecution.js';
-import { formatDateForAppleScript } from '../../utils/dateFormatter.js';
+import { appleScriptDateCode } from '../../utils/dateFormatter.js';
 /**
  * Generate pure AppleScript for project creation
  */
-function generateAppleScript(params) {
+export function generateAppleScript(params) {
     // Sanitize and prepare parameters for AppleScript
     const name = params.name.replace(/['"\\]/g, '\\$&'); // Escape quotes and backslashes
     const note = params.note?.replace(/['"\\]/g, '\\$&') || '';
-    // Convert ISO dates to AppleScript format
-    const dueDate = params.dueDate ? formatDateForAppleScript(params.dueDate) : '';
-    const deferDate = params.deferDate ? formatDateForAppleScript(params.deferDate) : '';
-    const plannedDate = params.plannedDate ? formatDateForAppleScript(params.plannedDate) : '';
+    // Build date variables outside OmniFocus tell block to avoid locale parsing issues.
+    const dueDateCode = params.dueDate ? appleScriptDateCode(params.dueDate, 'dueDateValue') : '';
+    const deferDateCode = params.deferDate ? appleScriptDateCode(params.deferDate, 'deferDateValue') : '';
+    const plannedDateCode = params.plannedDate ? appleScriptDateCode(params.plannedDate, 'plannedDateValue') : '';
+    const datePreamble = [dueDateCode, deferDateCode, plannedDateCode].filter(Boolean).join('\n');
     const flagged = params.flagged === true;
     const estimatedMinutes = params.estimatedMinutes?.toString() || '';
     const tags = params.tags || [];
@@ -37,6 +38,7 @@ function generateAppleScript(params) {
     // Construct AppleScript with error handling
     let script = `
   try
+${datePreamble}
     tell application "OmniFocus"
       tell front document
         -- Determine the container (root or folder)
@@ -55,9 +57,9 @@ function generateAppleScript(params) {
         
         -- Set project properties
         ${note ? `set note of newProject to "${note}"` : ''}
-        ${dueDate ? `set due date of newProject to date "${dueDate}"` : ''}
-        ${deferDate ? `set defer date of newProject to date "${deferDate}"` : ''}
-        ${plannedDate ? `set planned date of newProject to date "${plannedDate}"` : ''}
+        ${params.dueDate ? `set due date of newProject to dueDateValue` : ''}
+        ${params.deferDate ? `set defer date of newProject to deferDateValue` : ''}
+        ${params.plannedDate ? `set planned date of newProject to plannedDateValue` : ''}
         ${flagged ? `set flagged of newProject to true` : ''}
         ${estimatedMinutes ? `set estimated minutes of newProject to ${estimatedMinutes}` : ''}
         ${`set sequential of newProject to ${sequential}`}

package/dist/tools/primitives/addProject.test.js

@@ -0,0 +1,20 @@
+import assert from 'node:assert/strict';
+import test from 'node:test';
+import { generateAppleScript } from './addProject.js';
+test('addProject date handling uses preamble variables outside OmniFocus tell block', () => {
+    const script = generateAppleScript({
+        name: 'Project with dates',
+        dueDate: '2026-02-27',
+        deferDate: '2026-02-25',
+        plannedDate: '2026-02-24'
+    });
+    const tellIndex = script.indexOf('tell application "OmniFocus"');
+    const preambleIndex = script.indexOf('set dueDateValue to current date');
+    assert.ok(preambleIndex > -1 && preambleIndex < tellIndex);
+    assert.match(script, /set due date of newProject to dueDateValue/);
+    assert.match(script, /set defer date of newProject to deferDateValue/);
+    assert.match(script, /set planned date of newProject to plannedDateValue/);
+    assert.doesNotMatch(script, /set due date of newProject to date "/);
+    assert.doesNotMatch(script, /set defer date of newProject to date "/);
+    assert.doesNotMatch(script, /set planned date of newProject to date "/);
+});