@doist/todoist-ai 2.0.0 → 2.0.1

package/dist/tools/test-helpers.js

@@ -0,0 +1,139 @@
+/**
+ * Creates a mock Task with all required properties and sensible defaults.
+ * Pass only the properties you want to override for your specific test.
+ */
+export function createMockTask(overrides = {}) {
+    return {
+        id: '8485093748',
+        content: 'Test task content',
+        description: '',
+        completedAt: null,
+        labels: [],
+        childOrder: 1,
+        priority: 1,
+        projectId: '6cfCcrrCFg2xP94Q',
+        sectionId: null,
+        parentId: null,
+        url: 'https://todoist.com/showTask?id=8485093748',
+        // Use correct property names from Task schema
+        noteCount: 0,
+        addedByUid: '713437',
+        addedAt: '2025-08-13T22:09:56.123456Z',
+        deadline: null,
+        responsibleUid: null,
+        assignedByUid: null,
+        isCollapsed: false,
+        isDeleted: false,
+        duration: null,
+        checked: false,
+        updatedAt: '2025-08-13T22:09:56.123456Z',
+        due: null,
+        dayOrder: 0,
+        userId: '713437',
+        ...overrides,
+    };
+}
+/**
+ * Creates a mock Section with all required properties and sensible defaults.
+ * Pass only the properties you want to override for your specific test.
+ */
+export function createMockSection(overrides = {}) {
+    return {
+        id: 'section-123',
+        projectId: '6cfCcrrCFg2xP94Q',
+        sectionOrder: 1,
+        userId: 'test-user',
+        addedAt: '2024-01-01T00:00:00Z',
+        updatedAt: '2024-01-01T00:00:00Z',
+        archivedAt: null,
+        isArchived: false,
+        isDeleted: false,
+        isCollapsed: false,
+        name: 'Test Section',
+        ...overrides,
+    };
+}
+/**
+ * Creates a mock PersonalProject with all required properties and sensible defaults.
+ * Pass only the properties you want to override for your specific test.
+ */
+export function createMockProject(overrides = {}) {
+    return {
+        id: '6cfCcrrCFg2xP94Q',
+        name: 'Test Project',
+        color: 'charcoal',
+        isFavorite: false,
+        isShared: false,
+        parentId: null,
+        inboxProject: false,
+        viewStyle: 'list',
+        url: 'https://todoist.com/projects/6cfCcrrCFg2xP94Q',
+        isDeleted: false,
+        updatedAt: '2025-08-13T22:09:55.841800Z',
+        createdAt: '2025-08-13T22:09:55.841785Z',
+        childOrder: 1,
+        defaultOrder: 0,
+        description: '',
+        isCollapsed: false,
+        canAssignTasks: false,
+        isFrozen: false,
+        isArchived: false,
+        ...overrides,
+    };
+}
+/**
+ * Creates a mock API response object with results and nextCursor.
+ */
+export function createMockApiResponse(results, nextCursor = null) {
+    return {
+        results,
+        nextCursor,
+    };
+}
+/**
+ * Creates a simplified mapped task (matches mapTask output) for filter-based query tests.
+ */
+export function createMappedTask(overrides = {}) {
+    return {
+        id: TEST_IDS.TASK_1,
+        content: 'Test task content',
+        description: '',
+        dueDate: undefined,
+        recurring: false,
+        priority: 1,
+        projectId: TEST_IDS.PROJECT_TEST,
+        sectionId: null,
+        parentId: null,
+        labels: [],
+        ...overrides,
+    };
+}
+/**
+ * Common error messages used across tests.
+ */
+export const TEST_ERRORS = {
+    API_RATE_LIMIT: 'API Error: Rate limit exceeded',
+    API_UNAUTHORIZED: 'API Error: Unauthorized',
+    INVALID_CURSOR: 'Invalid cursor format',
+    INVALID_FILTER: 'Invalid filter query',
+};
+/**
+ * Creates multiple test cases for parameterized testing.
+ */
+export function createTestCases(cases) {
+    return cases;
+}
+/**
+ * Common mock IDs used across tests for consistency.
+ */
+export const TEST_IDS = {
+    TASK_1: '8485093748',
+    TASK_2: '8485093749',
+    TASK_3: '8485093750',
+    PROJECT_INBOX: 'inbox-project-id',
+    PROJECT_WORK: 'work-project-id',
+    PROJECT_TEST: '6cfCcrrCFg2xP94Q',
+    SECTION_1: 'section-123',
+    SECTION_2: 'section-456',
+    USER_ID: '713437',
+};

package/package.json

@@ -1,10 +1,12 @@
 {
     "name": "@doist/todoist-ai",
-    "version": "2.0.0",
+    "version": "2.0.1",
+    "type": "module",
     "main": "./dist/index.js",
     "types": "./dist/index.d.ts",
     "files": [
         "dist",
+        "scripts",
         "package.json",
         "LICENSE.txt",
         "README.md"
@@ -23,8 +25,10 @@
     "scripts": {
         "test": "jest",
         "build": "rimraf dist && npx tsc --project tsconfig.json",
-        "dev": "concurrently \"npx tsc --watch\" \"nodemon --watch dist --ext js --exec 'npx -y @modelcontextprotocol/inspector npx node dist/main.js\"",
+        "start": "npm run build && npx @modelcontextprotocol/inspector node dist/main.js",
+        "dev": "concurrently \"npx tsc --watch\" \"nodemon --watch dist --ext js --exec 'npx @modelcontextprotocol/inspector node dist/main.js'\"",
         "setup": "cp .env.example .env && npm install && npm run build",
+        "test:executable": "npm run build && node scripts/test-executable.cjs",
         "type-check": "npx tsc --noEmit",
         "biome:sort-imports": "biome check --formatter-enabled=false --linter-enabled=false --organize-imports-enabled=true --write .",
         "lint:check": "biome lint",

package/scripts/test-executable.cjs

@@ -0,0 +1,69 @@
+#!/usr/bin/env node
+
+const { spawn } = require('node:child_process')
+const path = require('node:path')
+
+console.log('Testing MCP server executable...')
+
+const mainJs = path.join(__dirname, '..', 'dist', 'main.js')
+const child = spawn('node', [mainJs], {
+    stdio: ['pipe', 'pipe', 'pipe'],
+})
+
+let _stdoutOutput = ''
+let stderrOutput = ''
+let hasError = false
+
+child.stdout.on('data', (data) => {
+    _stdoutOutput += data.toString()
+})
+
+child.stderr.on('data', (data) => {
+    const output = data.toString()
+    stderrOutput += output
+
+    // Only consider it an error if it's not related to graceful shutdown
+    if (output.includes('Error:') && !output.includes('SIGTERM') && !output.includes('SIGKILL')) {
+        console.error('Server startup error detected:', output)
+        hasError = true
+    }
+})
+
+child.on('error', (error) => {
+    console.error('Failed to start MCP server:', error.message)
+    hasError = true
+    process.exit(1)
+})
+
+child.on('exit', (code, signal) => {
+    // Expected signals when we kill the process
+    if (signal === 'SIGTERM' || signal === 'SIGKILL') {
+        return // This is expected
+    }
+
+    // Unexpected exit codes during startup
+    if (code !== null && code !== 0) {
+        console.error(`Server exited unexpectedly with code ${code}`)
+        hasError = true
+    }
+})
+
+// Kill the process after 2 seconds (MCP server should start successfully)
+setTimeout(() => {
+    if (hasError) {
+        console.error('❌ MCP server failed to start properly')
+        if (stderrOutput.trim()) {
+            console.error('Error output:', stderrOutput.trim())
+        }
+        process.exit(1)
+    }
+
+    // Gracefully terminate
+    child.kill('SIGTERM')
+
+    setTimeout(() => {
+        console.log('✅ MCP server executable test passed')
+        console.log('Server started successfully and is ready to accept connections')
+        process.exit(0)
+    }, 200) // Give it a moment to clean up
+}, 2000)