@doist/todoist-ai 4.13.2 → 4.13.3

package/dist/tools/__tests__/update-tasks.test.js

@@ -5,7 +5,7 @@ import { updateTasks } from '../update-tasks.js';
 // Mock the Todoist API
 const mockTodoistApi = {
     updateTask: jest.fn(),
-    moveTasks: jest.fn(),
+    moveTask: jest.fn(),
 };
 const { UPDATE_TASKS } = ToolNames;
 describe(`${UPDATE_TASKS} tool`, () => {
@@ -132,7 +132,7 @@ describe(`${UPDATE_TASKS} tool`, () => {
                 url: 'https://todoist.com/showTask?id=8485093750',
                 addedAt: '2025-08-13T22:09:56.123456Z',
             });
-            mockTodoistApi.moveTasks.mockResolvedValue([mockApiResponse]);
+            mockTodoistApi.moveTask.mockResolvedValue(mockApiResponse);
             const result = await updateTasks.execute({
                 tasks: [
                     {
@@ -141,7 +141,7 @@ describe(`${UPDATE_TASKS} tool`, () => {
                     },
                 ],
             }, mockTodoistApi);
-            expect(mockTodoistApi.moveTasks).toHaveBeenCalledWith(['8485093750'], {
+            expect(mockTodoistApi.moveTask).toHaveBeenCalledWith('8485093750', {
                 projectId: 'new-project-id',
             });
             expect(mockTodoistApi.updateTask).not.toHaveBeenCalled();
@@ -159,7 +159,7 @@ describe(`${UPDATE_TASKS} tool`, () => {
                 url: 'https://todoist.com/showTask?id=8485093751',
                 addedAt: '2025-08-13T22:09:56.123456Z',
             });
-            mockTodoistApi.moveTasks.mockResolvedValue([mockApiResponse]);
+            mockTodoistApi.moveTask.mockResolvedValue(mockApiResponse);
             const result = await updateTasks.execute({
                 tasks: [
                     {
@@ -168,7 +168,7 @@ describe(`${UPDATE_TASKS} tool`, () => {
                     },
                 ],
             }, mockTodoistApi);
-            expect(mockTodoistApi.moveTasks).toHaveBeenCalledWith(['8485093751'], {
+            expect(mockTodoistApi.moveTask).toHaveBeenCalledWith('8485093751', {
                 parentId: 'parent-task-123',
             });
             expect(mockTodoistApi.updateTask).not.toHaveBeenCalled();
@@ -200,7 +200,7 @@ describe(`${UPDATE_TASKS} tool`, () => {
                     timezone: null,
                 },
             });
-            mockTodoistApi.moveTasks.mockResolvedValue([movedTask]);
+            mockTodoistApi.moveTask.mockResolvedValue(movedTask);
             mockTodoistApi.updateTask.mockResolvedValue(updatedTask);
             const result = await updateTasks.execute({
                 tasks: [
@@ -214,8 +214,8 @@ describe(`${UPDATE_TASKS} tool`, () => {
                     },
                 ],
             }, mockTodoistApi);
-            // Should call moveTasks first for the projectId
-            expect(mockTodoistApi.moveTasks).toHaveBeenCalledWith(['8485093752'], {
+            // Should call moveTask first for the projectId
+            expect(mockTodoistApi.moveTask).toHaveBeenCalledWith('8485093752', {
                 projectId: 'different-project-id',
             });
             // Then call updateTask for the other properties
@@ -307,7 +307,7 @@ describe(`${UPDATE_TASKS} tool`, () => {
                 duration: { amount: 120, unit: 'minute' },
                 projectId: 'new-project-id',
             });
-            mockTodoistApi.moveTasks.mockResolvedValue([movedTask]);
+            mockTodoistApi.moveTask.mockResolvedValue(movedTask);
             mockTodoistApi.updateTask.mockResolvedValue(updatedTask);
             const result = await updateTasks.execute({
                 tasks: [
@@ -319,8 +319,8 @@ describe(`${UPDATE_TASKS} tool`, () => {
                     },
                 ],
             }, mockTodoistApi);
-            // Should call moveTasks first
-            expect(mockTodoistApi.moveTasks).toHaveBeenCalledWith(['8485093755'], {
+            // Should call moveTask first
+            expect(mockTodoistApi.moveTask).toHaveBeenCalledWith('8485093755', {
                 projectId: 'new-project-id',
             });
             // Then call updateTask with duration
@@ -485,21 +485,21 @@ describe(`${UPDATE_TASKS} tool`, () => {
                     createMockTask({ id: '6cPHJj2MV4HMj92W', content: 'Second task', sectionId }),
                 ];
                 // Each task should be moved individually to avoid bulk operation issues
-                mockTodoistApi.moveTasks
-                    .mockResolvedValueOnce([mockResponses[0]])
-                    .mockResolvedValueOnce([mockResponses[1]]);
+                mockTodoistApi.moveTask
+                    .mockResolvedValueOnce(mockResponses[0])
+                    .mockResolvedValueOnce(mockResponses[1]);
                 const result = await updateTasks.execute({
                     tasks: [
                         { id: '6cPHJm59x4WhMwR4', sectionId },
                         { id: '6cPHJj2MV4HMj92W', sectionId },
                     ],
                 }, mockTodoistApi);
-                // Should call moveTasks twice, once for each task individually
-                expect(mockTodoistApi.moveTasks).toHaveBeenCalledTimes(2);
-                expect(mockTodoistApi.moveTasks).toHaveBeenNthCalledWith(1, ['6cPHJm59x4WhMwR4'], {
+                // Should call moveTask twice, once for each task individually
+                expect(mockTodoistApi.moveTask).toHaveBeenCalledTimes(2);
+                expect(mockTodoistApi.moveTask).toHaveBeenNthCalledWith(1, '6cPHJm59x4WhMwR4', {
                     sectionId,
                 });
-                expect(mockTodoistApi.moveTasks).toHaveBeenNthCalledWith(2, ['6cPHJj2MV4HMj92W'], {
+                expect(mockTodoistApi.moveTask).toHaveBeenNthCalledWith(2, '6cPHJj2MV4HMj92W', {
                     sectionId,
                 });
                 expect(mockTodoistApi.updateTask).not.toHaveBeenCalled();
@@ -518,10 +518,10 @@ describe(`${UPDATE_TASKS} tool`, () => {
                     createMockTask({ id: TASK_3, content: 'Task 3', parentId: 'parent-task-123' }),
                 ];
                 // Each task should be moved individually
-                mockTodoistApi.moveTasks
-                    .mockResolvedValueOnce([mockResponses[0]])
-                    .mockResolvedValueOnce([mockResponses[1]])
-                    .mockResolvedValueOnce([mockResponses[2]]);
+                mockTodoistApi.moveTask
+                    .mockResolvedValueOnce(mockResponses[0])
+                    .mockResolvedValueOnce(mockResponses[1])
+                    .mockResolvedValueOnce(mockResponses[2]);
                 const result = await updateTasks.execute({
                     tasks: [
                         { id: '8485093748', projectId: 'new-project-id' },
@@ -530,14 +530,14 @@ describe(`${UPDATE_TASKS} tool`, () => {
                     ],
                 }, mockTodoistApi);
                 // Verify API was called correctly - 3 individual move calls
-                expect(mockTodoistApi.moveTasks).toHaveBeenCalledTimes(3);
-                expect(mockTodoistApi.moveTasks).toHaveBeenNthCalledWith(1, ['8485093748'], {
+                expect(mockTodoistApi.moveTask).toHaveBeenCalledTimes(3);
+                expect(mockTodoistApi.moveTask).toHaveBeenNthCalledWith(1, '8485093748', {
                     projectId: 'new-project-id',
                 });
-                expect(mockTodoistApi.moveTasks).toHaveBeenNthCalledWith(2, ['8485093749'], {
+                expect(mockTodoistApi.moveTask).toHaveBeenNthCalledWith(2, '8485093749', {
                     sectionId: 'new-section-id',
                 });
-                expect(mockTodoistApi.moveTasks).toHaveBeenNthCalledWith(3, ['8485093750'], {
+                expect(mockTodoistApi.moveTask).toHaveBeenNthCalledWith(3, '8485093750', {
                     parentId: 'parent-task-123',
                 });
                 expect(mockTodoistApi.updateTask).not.toHaveBeenCalled();
@@ -556,10 +556,10 @@ describe(`${UPDATE_TASKS} tool`, () => {
                     url: 'https://todoist.com/showTask?id=8485093751',
                     addedAt: '2025-08-13T22:09:59.123456Z',
                 });
-                mockTodoistApi.moveTasks.mockResolvedValue([mockTaskResponse]);
+                mockTodoistApi.moveTask.mockResolvedValue(mockTaskResponse);
                 const result = await updateTasks.execute({ tasks: [{ id: '8485093751', sectionId: 'target-section' }] }, mockTodoistApi);
-                expect(mockTodoistApi.moveTasks).toHaveBeenCalledTimes(1);
-                expect(mockTodoistApi.moveTasks).toHaveBeenCalledWith(['8485093751'], {
+                expect(mockTodoistApi.moveTask).toHaveBeenCalledTimes(1);
+                expect(mockTodoistApi.moveTask).toHaveBeenCalledWith('8485093751', {
                     sectionId: 'target-section',
                 });
                 expect(mockTodoistApi.updateTask).not.toHaveBeenCalled();
@@ -600,10 +600,10 @@ describe(`${UPDATE_TASKS} tool`, () => {
                     }),
                 ];
                 // Each task should be moved individually
-                mockTodoistApi.moveTasks
-                    .mockResolvedValueOnce([mockResponses[0]])
-                    .mockResolvedValueOnce([mockResponses[1]])
-                    .mockResolvedValueOnce([mockResponses[2]]);
+                mockTodoistApi.moveTask
+                    .mockResolvedValueOnce(mockResponses[0])
+                    .mockResolvedValueOnce(mockResponses[1])
+                    .mockResolvedValueOnce(mockResponses[2]);
                 const result = await updateTasks.execute({
                     tasks: [
                         { id: 'task-1', projectId: 'project-new' },
@@ -612,14 +612,14 @@ describe(`${UPDATE_TASKS} tool`, () => {
                     ],
                 }, mockTodoistApi);
                 // Verify API was called correctly - 3 individual move calls
-                expect(mockTodoistApi.moveTasks).toHaveBeenCalledTimes(3);
-                expect(mockTodoistApi.moveTasks).toHaveBeenNthCalledWith(1, ['task-1'], {
+                expect(mockTodoistApi.moveTask).toHaveBeenCalledTimes(3);
+                expect(mockTodoistApi.moveTask).toHaveBeenNthCalledWith(1, 'task-1', {
                     projectId: 'project-new',
                 });
-                expect(mockTodoistApi.moveTasks).toHaveBeenNthCalledWith(2, ['task-2'], {
+                expect(mockTodoistApi.moveTask).toHaveBeenNthCalledWith(2, 'task-2', {
                     parentId: 'task-1',
                 });
-                expect(mockTodoistApi.moveTasks).toHaveBeenNthCalledWith(3, ['task-3'], {
+                expect(mockTodoistApi.moveTask).toHaveBeenNthCalledWith(3, 'task-3', {
                     sectionId: 'section-new',
                 });
                 expect(mockTodoistApi.updateTask).not.toHaveBeenCalled();
@@ -640,7 +640,7 @@ describe(`${UPDATE_TASKS} tool`, () => {
                     url: 'https://todoist.com/showTask?id=8485093752',
                     addedAt: '2025-08-13T22:10:07.123456Z',
                 });
-                mockTodoistApi.moveTasks.mockResolvedValue([mockResponse]);
+                mockTodoistApi.moveTask.mockResolvedValue(mockResponse);
                 const result = await updateTasks.execute({
                     tasks: [
                         {
@@ -650,7 +650,7 @@ describe(`${UPDATE_TASKS} tool`, () => {
                         },
                     ],
                 }, mockTodoistApi);
-                expect(mockTodoistApi.moveTasks).toHaveBeenCalledWith(['8485093752'], {
+                expect(mockTodoistApi.moveTask).toHaveBeenCalledWith('8485093752', {
                     projectId: 'new-project-only',
                 });
                 expect(mockTodoistApi.updateTask).not.toHaveBeenCalled();
@@ -667,7 +667,7 @@ describe(`${UPDATE_TASKS} tool`, () => {
             it('should handle empty updates (only id provided)', async () => {
                 const result = await updateTasks.execute({ tasks: [{ id: '8485093753' }] }, mockTodoistApi);
                 // No API calls should be made since no move parameters are provided
-                expect(mockTodoistApi.moveTasks).not.toHaveBeenCalled();
+                expect(mockTodoistApi.moveTask).not.toHaveBeenCalled();
                 expect(mockTodoistApi.updateTask).not.toHaveBeenCalled();
                 // Returns empty results since no moves were processed
                 const textContent = extractTextContent(result);
@@ -691,22 +691,22 @@ describe(`${UPDATE_TASKS} tool`, () => {
             });
             it('should propagate API errors for individual task moves', async () => {
                 const apiError = new Error('API Error: Task not found');
-                mockTodoistApi.moveTasks.mockRejectedValue(apiError);
+                mockTodoistApi.moveTask.mockRejectedValue(apiError);
                 await expect(updateTasks.execute({ tasks: [{ id: 'non-existent-task', projectId: 'some-project' }] }, mockTodoistApi)).rejects.toThrow('API Error: Task not found');
             });
             it('should handle validation errors', async () => {
                 const validationError = new Error('API Error: Invalid section ID');
-                mockTodoistApi.moveTasks.mockRejectedValue(validationError);
+                mockTodoistApi.moveTask.mockRejectedValue(validationError);
                 await expect(updateTasks.execute({ tasks: [{ id: 'task-1', sectionId: 'invalid-section-format' }] }, mockTodoistApi)).rejects.toThrow('API Error: Invalid section ID');
             });
             it('should handle permission errors', async () => {
                 const permissionError = new Error('API Error: Insufficient permissions to move task');
-                mockTodoistApi.moveTasks.mockRejectedValue(permissionError);
+                mockTodoistApi.moveTask.mockRejectedValue(permissionError);
                 await expect(updateTasks.execute({ tasks: [{ id: 'restricted-task', projectId: 'restricted-project' }] }, mockTodoistApi)).rejects.toThrow('API Error: Insufficient permissions to move task');
             });
             it('should handle circular parent dependency errors', async () => {
                 const circularError = new Error('API Error: Circular dependency detected');
-                mockTodoistApi.moveTasks.mockRejectedValue(circularError);
+                mockTodoistApi.moveTask.mockRejectedValue(circularError);
                 await expect(updateTasks.execute({
                     tasks: [
                         {

package/dist/tools/update-tasks.js

@@ -97,16 +97,16 @@ const updateTasks = {
                     updateArgs = { ...updateArgs, assigneeId: validation.resolvedUser?.userId };
                 }
             }
-            // If no move parameters are provided, use updateTask without moveTasks
+            // If no move parameters are provided, use updateTask without moveTask
             if (!projectId && !sectionId && !parentId) {
                 return await client.updateTask(id, updateArgs);
             }
             const moveArgs = createMoveTaskArgs(id, projectId, sectionId, parentId);
-            const movedTasks = await client.moveTasks([id], moveArgs);
+            const movedTask = await client.moveTask(id, moveArgs);
             if (Object.keys(updateArgs).length > 0) {
                 return await client.updateTask(id, updateArgs);
             }
-            return movedTasks[0];
+            return movedTask;
         });
         const updatedTasks = (await Promise.all(updateTasksPromises)).filter((task) => task !== undefined);
         const mappedTasks = updatedTasks.map(mapTask);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@doist/todoist-ai",
-    "version": "4.13.2",
+    "version": "4.13.3",
     "type": "module",
     "main": "./dist/index.js",
     "types": "./dist/index.d.ts",
@@ -45,7 +45,7 @@
         "prepare": "husky"
     },
     "dependencies": {
-        "@doist/todoist-api-typescript": "5.6.4",
+        "@doist/todoist-api-typescript": "5.7.0",
         "@modelcontextprotocol/sdk": "^1.11.1",
         "date-fns": "^4.1.0",
         "dotenv": "^16.5.0",