refactor: Update MCP server configuration and add SQLite error handling; introduce test script for MCP functionality

Author: Marvin Zhang

.vscode/mcp.json

@@ -9,7 +9,7 @@
             "args": [
                 "--filter",
                 "@devlog/mcp-server",
-                "dev"
+                "start"
             ]
         },
         "playwright": {

packages/core/package.json

@@ -4,6 +4,7 @@
   "description": "Core devlog management functionality",
   "main": "build/index.js",
   "types": "build/index.d.ts",
+  "type": "module",
   "files": [
     "build/**/*",
     "README.md",
@@ -39,7 +40,9 @@
   },
   "license": "MIT",
   "dependencies": {
-    "@devlog/types": "workspace:*",
+    "@devlog/types": "workspace:*"
+  },
+  "peerDependencies": {
     "better-sqlite3": "^11.0.0",
     "pg": "^8.12.0",
     "mysql2": "^3.11.0"

packages/core/src/storage/sqlite-storage.ts

@@ -18,12 +18,15 @@ export class SQLiteStorageProvider implements StorageProvider {
   async initialize(): Promise<void> {
     // Dynamic import to make better-sqlite3 optional
     try {
-      const sqlite3Module = await import("better-sqlite3" as any);
+      // Use eval to prevent TypeScript from transforming the import
+      const dynamicImport = eval('(specifier) => import(specifier)');
+      const sqlite3Module = await dynamicImport("better-sqlite3");
       const Database = sqlite3Module.default;
 
       this.db = new Database(this.filePath, this.options);
-    } catch (error) {
-      throw new Error("better-sqlite3 is required for SQLite storage. Install it with: npm install better-sqlite3");
+    } catch (error: any) {
+      console.error("Failed to initialize SQLite storage:", error);
+      throw new Error("Failed to initialize SQLite storage: " + error.message);
     }
 
     // Create tables

packages/mcp-server/package.json

@@ -4,7 +4,6 @@
   "description": "MCP server for managing development logs and working notes",
   "main": "build/index.js",
   "types": "build/index.d.ts",
-  "type": "module",
   "files": [
     "build/**/*",
     "README.md",
@@ -45,7 +44,8 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.0.0",
     "@devlog/types": "workspace:*",
-    "@devlog/core": "workspace:*"
+    "@devlog/core": "workspace:*",
+    "better-sqlite3": "^11.10.0"
   },
   "devDependencies": {
     "@devlog/types": "workspace:*",

packages/mcp-server/tsconfig.json

@@ -2,7 +2,7 @@
   "compilerOptions": {
     "target": "ES2022",
     "module": "ESNext",
-    "moduleResolution": "bundler",
+    "moduleResolution": "node",
     "allowSyntheticDefaultImports": true,
     "esModuleInterop": true,
     "allowImportingTsExtensions": false,
@@ -13,17 +13,8 @@
     "rootDir": "./src",
     "declaration": true,
     "declarationMap": true,
-    "sourceMap": true,
-    "composite": true
+    "sourceMap": true
   },
   "include": ["src/**/*"],
-  "exclude": ["node_modules", "build", "src/**/*.test.ts", "src/__tests__"],
-  "references": [
-    {
-      "path": "../types"
-    },
-    {
-      "path": "../core"
-    }
-  ]
+  "exclude": ["node_modules", "build", "src/**/*.test.ts", "src/__tests__"]
 }

pnpm-lock.yaml

@@ -0,0 +1,37 @@
+#!/usr/bin/env node
+
+/**
+ * Test script to verify MCP functionality works correctly
+ */
+
+import { MCPDevlogAdapter } from './packages/mcp-server/build/mcp-adapter.js';
+
+async function testMCPFunctionality() {
+  console.log('Testing MCP functionality...');
+  
+  const adapter = new MCPDevlogAdapter(process.cwd());
+  
+  try {
+    console.log('Creating test devlog entry...');
+    const result = await adapter.findOrCreateDevlog({
+      title: 'Test MCP Server Functionality',
+      type: 'task',
+      description: 'Testing the MCP server to ensure it can create and manage devlog entries after the cleanup and build process',
+      priority: 'medium'
+    });
+    
+    console.log('Success!');
+    console.log(result);
+    
+    console.log('\nListing active devlogs...');
+    const activeResult = await adapter.getActiveContext();
+    console.log(activeResult);
+    
+  } catch (error) {
+    console.error('Error:', error.message);
+  } finally {
+    await adapter.dispose();
+  }
+}
+
+testMCPFunctionality().catch(console.error);