refactor: Enhance DevlogManager and SQLiteStorageProvider with improved devlog creation and dynamic import handling

Author: tikazyq

packages/core/src/__tests__/devlog-manager.test.ts

@@ -8,9 +8,18 @@ describe('DevlogManager', () => {
   let testWorkspace: string;
 
   beforeEach(async () => {
-    // Create a temporary test workspace
-    testWorkspace = path.join(process.cwd(), 'test-workspace-' + Date.now());
-    manager = new DevlogManager({ workspaceRoot: testWorkspace });
+    // Create a temporary test workspace with unique name
+    testWorkspace = path.join(process.cwd(), 'test-workspace-' + Date.now() + '-' + Math.random().toString(36).substr(2, 9));
+    
+    // Create isolated storage config for each test
+    const sqliteDbPath = path.join(testWorkspace, 'test.db');
+    manager = new DevlogManager({ 
+      workspaceRoot: testWorkspace,
+      storage: {
+        type: 'sqlite',
+        filePath: sqliteDbPath
+      }
+    });
   });
 
   afterEach(async () => {
@@ -22,9 +31,9 @@ describe('DevlogManager', () => {
     }
   });
 
-  describe('createDevlog', () => {
+  describe('findOrCreateDevlog', () => {
     it('should create a new devlog entry', async () => {
-      const result = await manager.createDevlog({
+      const result = await manager.findOrCreateDevlog({
         title: 'Test Feature',
         type: 'feature',
         description: 'A test feature',
@@ -44,7 +53,7 @@ describe('DevlogManager', () => {
 
     it('should create a devlog with custom ID', async () => {
       const customId = 'my-custom-id';
-      const result = await manager.createDevlog({
+      const result = await manager.findOrCreateDevlog({
         id: customId,
         title: 'Custom ID Test',
         type: 'task',
@@ -230,10 +239,7 @@ describe('DevlogManager', () => {
         description: 'Testing notes'
       });
 
-      const result = await manager.addNote(created.id, {
-        category: 'progress',
-        content: 'Made some progress on this task'
-      });
+      const result = await manager.addNote(created.id, 'Made some progress on this task', 'progress');
 
       expect(result.notes).toHaveLength(1);
       expect(result.notes[0].content).toBe('Made some progress on this task');

packages/core/src/devlog-manager.ts

@@ -72,13 +72,22 @@ export class DevlogManager {
     this.initialized = true;
   }
 
+  /**
+   * Create a new devlog entry or find existing one with same title and type
+   * Alias for findOrCreateDevlog to provide a cleaner API
+   */
+  async createDevlog(request: CreateDevlogRequest): Promise<DevlogEntry> {
+    return this.findOrCreateDevlog(request);
+  }
+
   /**
    * Create a new devlog entry or find existing one with same title and type
    */
   async findOrCreateDevlog(request: CreateDevlogRequest): Promise<DevlogEntry> {
     await this.ensureInitialized();
 
-    const id = this.generateId(request.title, request.type);
+    // Use provided ID if available, otherwise generate one
+    const id = request.id || this.generateId(request.title, request.type);
 
     // Check if entry already exists
     const existing = await this.storageProvider.get(id);
@@ -235,6 +244,12 @@ export class DevlogManager {
    */
   async deleteDevlog(id: string): Promise<void> {
     await this.ensureInitialized();
+    
+    const existing = await this.storageProvider.get(id);
+    if (!existing) {
+      throw new Error(`Devlog entry with ID '${id}' not found`);
+    }
+    
     await this.storageProvider.delete(id);
   }
 
@@ -262,6 +277,20 @@ export class DevlogManager {
     return await this.storageProvider.get(id);
   }
 
+  /**
+   * Get active devlog entries for AI context
+   */
+  async getActiveContext(limit?: number): Promise<DevlogEntry[]> {
+    await this.ensureInitialized();
+    
+    const filter = {
+      status: ["todo", "in-progress", "review", "testing"] as any[]
+    };
+    
+    const entries = await this.storageProvider.list(filter);
+    return entries.slice(0, limit || 10);
+  }
+
   /**
    * Update AI context for a devlog entry
    */

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

@@ -45,10 +45,20 @@ export class SQLiteStorageProvider implements StorageProvider {
     // Dynamic import to make better-sqlite3 optional
     try {
       console.log(`[SQLiteStorage] Attempting to import better-sqlite3...`);
-      // Use eval to prevent TypeScript from transforming the import
-      const dynamicImport = eval('(specifier) => import(specifier)');
-      const sqlite3Module = await dynamicImport("better-sqlite3");
-      console.log(`[SQLiteStorage] Successfully imported better-sqlite3`);
+      
+      // Try different import approaches for better compatibility
+      let sqlite3Module;
+      try {
+        // Standard dynamic import
+        sqlite3Module = await import("better-sqlite3");
+        console.log(`[SQLiteStorage] Successfully imported better-sqlite3 via standard import`);
+      } catch (importError) {
+        console.log(`[SQLiteStorage] Standard import failed, trying eval approach...`);
+        // Fallback to eval approach for environments that require it
+        const dynamicImport = eval('(specifier) => import(specifier)');
+        sqlite3Module = await dynamicImport("better-sqlite3");
+        console.log(`[SQLiteStorage] Successfully imported better-sqlite3 via eval import`);
+      }
 
       const Database = sqlite3Module.default;
       console.log(`[SQLiteStorage] Creating database instance...`);

packages/core/src/types/better-sqlite3.d.ts

@@ -0,0 +1 @@
+declare module 'better-sqlite3';

packages/core/vitest.config.ts

@@ -5,5 +5,9 @@ export default defineConfig({
     environment: 'node',
     globals: true,
     testTimeout: 30000,
+    // Handle dynamic imports better
+    deps: {
+      external: ['better-sqlite3']
+    }
   },
 });