@doist/todoist-ai 4.4.0 → 4.6.0

package/dist/tools/__tests__/add-projects.test.js

@@ -1,5 +1,5 @@
 import { jest } from '@jest/globals';
-import { TEST_IDS, createMockProject, extractStructuredContent, extractTextContent, } from '../../utils/test-helpers.js';
+import { createMockProject, extractStructuredContent, extractTextContent, TEST_IDS, } from '../../utils/test-helpers.js';
 import { ToolNames } from '../../utils/tool-names.js';
 import { addProjects } from '../add-projects.js';
 // Mock the Todoist API

package/dist/tools/__tests__/add-sections.test.js

@@ -1,5 +1,5 @@
 import { jest } from '@jest/globals';
-import { TEST_IDS, createMockSection, extractStructuredContent, extractTextContent, } from '../../utils/test-helpers.js';
+import { createMockSection, extractStructuredContent, extractTextContent, TEST_IDS, } from '../../utils/test-helpers.js';
 import { ToolNames } from '../../utils/tool-names.js';
 import { addSections } from '../add-sections.js';
 // Mock the Todoist API

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

@@ -1,5 +1,5 @@
 import { jest } from '@jest/globals';
-import { TODAY, createMockTask, extractStructuredContent, extractTextContent, } from '../../utils/test-helpers.js';
+import { createMockTask, extractStructuredContent, extractTextContent, TODAY, } from '../../utils/test-helpers.js';
 import { ToolNames } from '../../utils/tool-names.js';
 import { addTasks } from '../add-tasks.js';
 // Mock the Todoist API
@@ -42,11 +42,14 @@ describe(`${ADD_TASKS} tool`, () => {
                 .mockResolvedValueOnce(mockApiResponse2);
             const result = await addTasks.execute({
                 tasks: [
-                    { content: 'First task content', projectId: '6cfCcrrCFg2xP94Q' },
+                    {
+                        content: 'First task content',
+                        projectId: '6cfCcrrCFg2xP94Q',
+                    },
                     {
                         content: 'Second task content',
                         description: 'Task description',
-                        priority: 2,
+                        priority: 'p2',
                         dueString: 'Aug 15',
                         projectId: '6cfCcrrCFg2xP94Q',
                     },
@@ -63,7 +66,7 @@ describe(`${ADD_TASKS} tool`, () => {
             expect(mockTodoistApi.addTask).toHaveBeenNthCalledWith(2, {
                 content: 'Second task content',
                 description: 'Task description',
-                priority: 2,
+                priority: 3,
                 dueString: 'Aug 15',
                 projectId: '6cfCcrrCFg2xP94Q',
                 sectionId: undefined,
@@ -99,7 +102,7 @@ describe(`${ADD_TASKS} tool`, () => {
                     {
                         content: 'Subtask content',
                         description: 'Subtask description',
-                        priority: 3,
+                        priority: 'p3',
                         projectId: '6cfCcrrCFg2xP94Q',
                         sectionId: 'section-123',
                         parentId: 'parent-task-456',
@@ -109,7 +112,7 @@ describe(`${ADD_TASKS} tool`, () => {
             expect(mockTodoistApi.addTask).toHaveBeenCalledWith({
                 content: 'Subtask content',
                 description: 'Subtask description',
-                priority: 3,
+                priority: 2,
                 projectId: '6cfCcrrCFg2xP94Q',
                 sectionId: 'section-123',
                 parentId: 'parent-task-456',
@@ -203,25 +206,179 @@ describe(`${ADD_TASKS} tool`, () => {
             ];
             for (const testCase of testCases) {
                 mockTodoistApi.addTask.mockClear();
-                await addTasks.execute({ tasks: [{ content: 'Test task', duration: testCase.input }] }, mockTodoistApi);
+                await addTasks.execute({
+                    tasks: [
+                        {
+                            content: 'Test task',
+                            duration: testCase.input,
+                            projectId: '6cfCcrrCFg2xP94Q',
+                        },
+                    ],
+                }, mockTodoistApi);
                 expect(mockTodoistApi.addTask).toHaveBeenCalledWith(expect.objectContaining({
                     duration: testCase.expectedMinutes,
                     durationUnit: 'minute',
                 }));
             }
         });
+        it('should add task with labels', async () => {
+            const mockApiResponse = createMockTask({
+                id: '8485093755',
+                content: 'Task with labels',
+                labels: ['urgent', 'work'],
+                url: 'https://todoist.com/showTask?id=8485093755',
+                addedAt: '2025-08-13T22:09:56.123456Z',
+            });
+            mockTodoistApi.addTask.mockResolvedValue(mockApiResponse);
+            const result = await addTasks.execute({
+                tasks: [
+                    {
+                        content: 'Task with labels',
+                        labels: ['urgent', 'work'],
+                        projectId: '6cfCcrrCFg2xP94Q',
+                    },
+                ],
+            }, mockTodoistApi);
+            expect(mockTodoistApi.addTask).toHaveBeenCalledWith({
+                content: 'Task with labels',
+                labels: ['urgent', 'work'],
+                projectId: '6cfCcrrCFg2xP94Q',
+                sectionId: undefined,
+                parentId: undefined,
+            });
+            // Verify structured content includes labels
+            const structuredContent = extractStructuredContent(result);
+            expect(structuredContent.tasks).toHaveLength(1);
+            expect(structuredContent.tasks[0]).toEqual(expect.objectContaining({
+                labels: ['urgent', 'work'],
+            }));
+        });
+        it('should add task with empty labels array', async () => {
+            const mockApiResponse = createMockTask({
+                id: '8485093756',
+                content: 'Task with empty labels',
+                labels: [],
+                url: 'https://todoist.com/showTask?id=8485093756',
+                addedAt: '2025-08-13T22:09:56.123456Z',
+            });
+            mockTodoistApi.addTask.mockResolvedValue(mockApiResponse);
+            await addTasks.execute({
+                tasks: [
+                    {
+                        content: 'Task with empty labels',
+                        labels: [],
+                        projectId: '6cfCcrrCFg2xP94Q',
+                    },
+                ],
+            }, mockTodoistApi);
+            expect(mockTodoistApi.addTask).toHaveBeenCalledWith({
+                content: 'Task with empty labels',
+                labels: [],
+                projectId: '6cfCcrrCFg2xP94Q',
+                sectionId: undefined,
+                parentId: undefined,
+            });
+        });
+        it('should add task without labels field', async () => {
+            const mockApiResponse = createMockTask({
+                id: '8485093757',
+                content: 'Task without labels',
+                url: 'https://todoist.com/showTask?id=8485093757',
+                addedAt: '2025-08-13T22:09:56.123456Z',
+            });
+            mockTodoistApi.addTask.mockResolvedValue(mockApiResponse);
+            await addTasks.execute({
+                tasks: [
+                    {
+                        content: 'Task without labels',
+                        projectId: '6cfCcrrCFg2xP94Q',
+                    },
+                ],
+            }, mockTodoistApi);
+            expect(mockTodoistApi.addTask).toHaveBeenCalledWith({
+                content: 'Task without labels',
+                labels: undefined,
+                projectId: '6cfCcrrCFg2xP94Q',
+                sectionId: undefined,
+                parentId: undefined,
+            });
+        });
+        it('should add multiple tasks with different label configurations', async () => {
+            const mockApiResponse1 = createMockTask({
+                id: '8485093758',
+                content: 'Task with labels',
+                labels: ['personal'],
+            });
+            const mockApiResponse2 = createMockTask({
+                id: '8485093759',
+                content: 'Task without labels',
+            });
+            const mockApiResponse3 = createMockTask({
+                id: '8485093760',
+                content: 'Task with multiple labels',
+                labels: ['work', 'urgent', 'review'],
+            });
+            mockTodoistApi.addTask
+                .mockResolvedValueOnce(mockApiResponse1)
+                .mockResolvedValueOnce(mockApiResponse2)
+                .mockResolvedValueOnce(mockApiResponse3);
+            await addTasks.execute({
+                tasks: [
+                    {
+                        content: 'Task with labels',
+                        labels: ['personal'],
+                        projectId: '6cfCcrrCFg2xP94Q',
+                    },
+                    {
+                        content: 'Task without labels',
+                        projectId: '6cfCcrrCFg2xP94Q',
+                    },
+                    {
+                        content: 'Task with multiple labels',
+                        labels: ['work', 'urgent', 'review'],
+                        projectId: '6cfCcrrCFg2xP94Q',
+                    },
+                ],
+            }, mockTodoistApi);
+            expect(mockTodoistApi.addTask).toHaveBeenCalledTimes(3);
+            expect(mockTodoistApi.addTask).toHaveBeenNthCalledWith(1, expect.objectContaining({
+                labels: ['personal'],
+            }));
+            expect(mockTodoistApi.addTask).toHaveBeenNthCalledWith(2, expect.objectContaining({
+                labels: undefined,
+            }));
+            expect(mockTodoistApi.addTask).toHaveBeenNthCalledWith(3, expect.objectContaining({
+                labels: ['work', 'urgent', 'review'],
+            }));
+        });
     });
     describe('error handling', () => {
         it('should throw error for invalid duration format', async () => {
-            await expect(addTasks.execute({ tasks: [{ content: 'Task with invalid duration', duration: 'invalid' }] }, mockTodoistApi)).rejects.toThrow('Task "Task with invalid duration": Invalid duration format "invalid"');
+            await expect(addTasks.execute({
+                tasks: [
+                    {
+                        content: 'Task with invalid duration',
+                        duration: 'invalid',
+                        projectId: '6cfCcrrCFg2xP94Q',
+                    },
+                ],
+            }, mockTodoistApi)).rejects.toThrow('Task "Task with invalid duration": Invalid duration format "invalid"');
         });
         it('should throw error for duration exceeding 24 hours', async () => {
-            await expect(addTasks.execute({ tasks: [{ content: 'Task with too long duration', duration: '25h' }] }, mockTodoistApi)).rejects.toThrow('Task "Task with too long duration": Invalid duration format "25h": Duration cannot exceed 24 hours (1440 minutes)');
+            await expect(addTasks.execute({
+                tasks: [
+                    {
+                        content: 'Task with too long duration',
+                        duration: '25h',
+                        projectId: '6cfCcrrCFg2xP94Q',
+                    },
+                ],
+            }, mockTodoistApi)).rejects.toThrow('Task "Task with too long duration": Invalid duration format "25h": Duration cannot exceed 24 hours (1440 minutes)');
         });
         it('should propagate API errors', async () => {
             const apiError = new Error('API Error: Task content is required');
             mockTodoistApi.addTask.mockRejectedValue(apiError);
-            await expect(addTasks.execute({ tasks: [{ content: '' }] }, mockTodoistApi)).rejects.toThrow(apiError.message);
+            await expect(addTasks.execute({ tasks: [{ content: '', projectId: '6cfCcrrCFg2xP94Q' }] }, mockTodoistApi)).rejects.toThrow(apiError.message);
         });
         it('should handle partial failures when adding multiple tasks', async () => {
             const mockApiResponse = createMockTask({
@@ -236,8 +393,8 @@ describe(`${ADD_TASKS} tool`, () => {
                 .mockRejectedValueOnce(apiError);
             await expect(addTasks.execute({
                 tasks: [
-                    { content: 'First task content' },
-                    { content: 'Second task content' },
+                    { content: 'First task content', projectId: '6cfCcrrCFg2xP94Q' },
+                    { content: 'Second task content', projectId: '6cfCcrrCFg2xP94Q' },
                 ],
             }, mockTodoistApi)).rejects.toThrow('API Error: Second task failed');
             // Verify first task was attempted
@@ -246,6 +403,8 @@ describe(`${ADD_TASKS} tool`, () => {
     });
     describe('next steps logic', () => {
         it('should suggest find-tasks-by-date for today when hasToday is true', async () => {
+            // Clear any leftover mocks from previous tests
+            mockTodoistApi.addTask.mockClear();
             const mockApiResponse = createMockTask({
                 id: '8485093755',
                 content: 'Task due today',
@@ -260,12 +419,22 @@ describe(`${ADD_TASKS} tool`, () => {
                 },
             });
             mockTodoistApi.addTask.mockResolvedValue(mockApiResponse);
-            const result = await addTasks.execute({ tasks: [{ content: 'Task due today', dueString: 'today' }] }, mockTodoistApi);
+            const result = await addTasks.execute({
+                tasks: [
+                    {
+                        content: 'Task due today',
+                        dueString: 'today',
+                        projectId: '6cfCcrrCFg2xP94Q',
+                    },
+                ],
+            }, mockTodoistApi);
             const textContent = extractTextContent(result);
             expect(textContent).toMatchSnapshot();
             expect(textContent).toContain(`Use ${GET_OVERVIEW} to see your updated project organization`);
         });
         it('should suggest overview tool when no hasToday context', async () => {
+            // Clear any leftover mocks from previous tests
+            mockTodoistApi.addTask.mockClear();
             const mockApiResponse = createMockTask({
                 id: '8485093756',
                 content: 'Regular task',

package/dist/tools/__tests__/assignment-integration.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=assignment-integration.test.d.ts.map

package/dist/tools/__tests__/assignment-integration.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"assignment-integration.test.d.ts","sourceRoot":"","sources":["../../../src/tools/__tests__/assignment-integration.test.ts"],"names":[],"mappings":""}