feat: Add user-info tool and upgrade Todoist API (#82)

Author: amix

package-lock.json

@@ -44,7 +44,7 @@
         "prepare": "husky"
     },
     "dependencies": {
-        "@doist/todoist-api-typescript": "5.1.2",
+        "@doist/todoist-api-typescript": "5.4.0",
         "@modelcontextprotocol/sdk": "^1.11.1",
         "date-fns": "^4.1.0",
         "dotenv": "^16.5.0",

package.json

@@ -26,6 +26,7 @@ import { updateComments } from './tools/update-comments.js'
 // General tools
 import { deleteObject } from './tools/delete-object.js'
 import { getOverview } from './tools/get-overview.js'
+import { userInfo } from './tools/user-info.js'
 
 const tools = {
     // Task management tools
@@ -50,6 +51,7 @@ const tools = {
     // General tools
     getOverview,
     deleteObject,
+    userInfo,
 }
 
 export { tools, getMcpServer }
@@ -77,4 +79,5 @@ export {
     // General tools
     getOverview,
     deleteObject,
+    userInfo,
 }

src/index.ts

@@ -28,6 +28,7 @@ import { updateComments } from './tools/update-comments.js'
 // General tools
 import { deleteObject } from './tools/delete-object.js'
 import { getOverview } from './tools/get-overview.js'
+import { userInfo } from './tools/user-info.js'
 
 const instructions = `
 Tools to help you manage your todoist tasks.
@@ -78,6 +79,7 @@ function getMcpServer({ todoistApiKey, baseUrl }: { todoistApiKey: string; baseU
     // General tools
     registerTool(getOverview, server, todoist)
     registerTool(deleteObject, server, todoist)
+    registerTool(userInfo, server, todoist)
 
     return server
 }

src/mcp-server.ts

@@ -34,6 +34,7 @@ describe(`${UPDATE_SECTIONS} tool`, () => {
                 isDeleted: false,
                 isCollapsed: false,
                 name: 'Updated Section Name',
+                url: 'https://todoist.com/sections/existing-section-123',
             }
 
             mockTodoistApi.updateSection.mockResolvedValue(mockApiResponse)

src/tools/__tests__/update-sections.test.ts

@@ -0,0 +1,173 @@
+import type { CurrentUser, TodoistApi } from '@doist/todoist-api-typescript'
+import { jest } from '@jest/globals'
+import {
+    TEST_ERRORS,
+    extractStructuredContent,
+    extractTextContent,
+} from '../../utils/test-helpers.js'
+import { ToolNames } from '../../utils/tool-names.js'
+import { userInfo } from '../user-info.js'
+
+// Mock the Todoist API
+const mockTodoistApi = {
+    getUser: jest.fn(),
+} as unknown as jest.Mocked<TodoistApi>
+
+const { USER_INFO } = ToolNames
+
+// Helper function to create a mock user with default values that can be overridden
+function createMockUser(overrides: Partial<CurrentUser> = {}): CurrentUser {
+    return {
+        id: '123',
+        fullName: 'Test User',
+        email: '[email protected]',
+        isPremium: true,
+        completedToday: 12,
+        dailyGoal: 10,
+        weeklyGoal: 100,
+        startDay: 1, // Monday
+        tzInfo: {
+            timezone: 'Europe/Madrid',
+            gmtString: '+02:00',
+            hours: 2,
+            minutes: 0,
+            isDst: 1,
+        },
+        lang: 'en',
+        avatarBig: 'https://example.com/avatar.jpg',
+        avatarMedium: null,
+        avatarS640: null,
+        avatarSmall: null,
+        karma: 86394.0,
+        karmaTrend: 'up',
+        nextWeek: 1,
+        weekendStartDay: 6,
+        timeFormat: 0,
+        dateFormat: 0,
+        daysOff: [6, 7],
+        businessAccountId: null,
+        completedCount: 102920,
+        inboxProjectId: '6PVw8cMf7m8fWwRp',
+        startPage: 'overdue',
+        ...overrides,
+    }
+}
+
+describe(`${USER_INFO} tool`, () => {
+    beforeEach(() => {
+        jest.clearAllMocks()
+    })
+
+    it('should generate user info with all required fields', async () => {
+        const mockUser = createMockUser()
+
+        mockTodoistApi.getUser.mockResolvedValue(mockUser)
+
+        const result = await userInfo.execute({}, mockTodoistApi)
+
+        expect(mockTodoistApi.getUser).toHaveBeenCalledWith()
+
+        // Test text content contains expected information
+        const textContent = extractTextContent(result)
+        expect(textContent).toContain('Test User')
+        expect(textContent).toContain('[email protected]')
+        expect(textContent).toContain('Europe/Madrid')
+        expect(textContent).toContain('Monday (1)')
+        expect(textContent).toContain('Completed Today:** 12')
+        expect(textContent).toContain('Plan:** Todoist Pro')
+
+        // Test structured content
+        const structuredContent = extractStructuredContent(result)
+        expect(structuredContent).toEqual(
+            expect.objectContaining({
+                type: 'user_info',
+                fullName: 'Test User',
+                email: '[email protected]',
+                timezone: 'Europe/Madrid',
+                startDay: 1,
+                startDayName: 'Monday',
+                completedToday: 12,
+                dailyGoal: 10,
+                weeklyGoal: 100,
+                plan: 'Todoist Pro',
+                currentLocalTime: expect.any(String),
+                weekStartDate: expect.any(String),
+                weekEndDate: expect.any(String),
+                currentWeekNumber: expect.any(Number),
+            }),
+        )
+
+        // Verify date formats
+        expect(structuredContent.weekStartDate).toMatch(/^\d{4}-\d{2}-\d{2}$/)
+        expect(structuredContent.weekEndDate).toMatch(/^\d{4}-\d{2}-\d{2}$/)
+        expect(structuredContent.currentLocalTime).toMatch(
+            /^\d{2}\/\d{2}\/\d{4}, \d{2}:\d{2}:\d{2}$/,
+        )
+    })
+
+    it('should handle missing timezone info', async () => {
+        const mockUser = createMockUser({
+            isPremium: false,
+            tzInfo: {
+                timezone: 'UTC',
+                gmtString: '+00:00',
+                hours: 0,
+                minutes: 0,
+                isDst: 0,
+            },
+        })
+
+        mockTodoistApi.getUser.mockResolvedValue(mockUser)
+
+        const result = await userInfo.execute({}, mockTodoistApi)
+
+        const textContent = extractTextContent(result)
+        expect(textContent).toContain('UTC') // Should default to UTC
+        expect(textContent).toContain('Monday (1)') // Should default to Monday
+        expect(textContent).toContain('Plan:** Todoist Free')
+
+        const structuredContent = extractStructuredContent(result)
+        expect(structuredContent.timezone).toBe('UTC')
+        expect(structuredContent.startDay).toBe(1)
+        expect(structuredContent.startDayName).toBe('Monday')
+        expect(structuredContent.plan).toBe('Todoist Free')
+    })
+
+    it('should handle invalid timezone and fallback to UTC', async () => {
+        const mockUser = createMockUser({
+            startDay: 2, // Tuesday
+            tzInfo: {
+                timezone: 'Invalid/Timezone',
+                gmtString: '+05:30',
+                hours: 5,
+                minutes: 30,
+                isDst: 0,
+            },
+        })
+
+        mockTodoistApi.getUser.mockResolvedValue(mockUser)
+
+        const result = await userInfo.execute({}, mockTodoistApi)
+
+        const textContent = extractTextContent(result)
+        expect(textContent).toContain('UTC') // Should fallback to UTC
+        expect(textContent).toContain('Tuesday (2)')
+
+        const structuredContent = extractStructuredContent(result)
+        expect(structuredContent.timezone).toBe('UTC') // Should be UTC, not the invalid timezone
+        expect(structuredContent.startDay).toBe(2)
+        expect(structuredContent.startDayName).toBe('Tuesday')
+        expect(structuredContent.currentLocalTime).toMatch(
+            /^\d{2}\/\d{2}\/\d{4}, \d{2}:\d{2}:\d{2}$/,
+        )
+    })
+
+    it('should propagate API errors', async () => {
+        const apiError = new Error(TEST_ERRORS.API_UNAUTHORIZED)
+        mockTodoistApi.getUser.mockRejectedValue(apiError)
+
+        await expect(userInfo.execute({}, mockTodoistApi)).rejects.toThrow(
+            TEST_ERRORS.API_UNAUTHORIZED,
+        )
+    })
+})