@critical-path/core 0.28.0 → 0.30.0

package/src/store/sqlite.ts

@@ -1,5 +1,6 @@
 import type { DatabaseSync } from 'node:sqlite';
 import type { StorageAdapter } from './index.js';
+import { decodeCursor, encodeCursor, pageSize, type ActivityQuery, type Page, type TaskQuery } from './query.js';
 import type {
   Project,
   Task,
@@ -18,6 +19,14 @@ import type {
 } from '../types/index.js';
 import { generateProjectKey } from '../utils/key.js';
 
+/**
+ * SQLite returns NULL columns as `null`, while domain objects leave unset optional fields out.
+ * Dropping nulls keeps SQLiteStore results identical to the other adapters.
+ */
+function dropNulls<T extends object>(row: T): T {
+  return Object.fromEntries(Object.entries(row).filter(([, value]) => value !== null)) as T;
+}
+
 export interface SQLiteStoreConfig {
   /**
    * Database file path (e.g., 'critical-path.db' or ':memory:').
@@ -280,6 +289,17 @@ export class SQLiteStore implements StorageAdapter {
       // Column may already exist
     }
 
+    // Indexes for filtered queries and keyset pagination
+    this.db.exec(`
+      CREATE INDEX IF NOT EXISTS idx_tasks_project_created ON tasks (projectId, createdAt, id);
+      CREATE INDEX IF NOT EXISTS idx_tasks_parent ON tasks (parentId);
+      CREATE INDEX IF NOT EXISTS idx_activities_project_created ON activities (projectId, createdAt, id);
+      CREATE INDEX IF NOT EXISTS idx_activities_task_created ON activities (taskId, createdAt, id);
+      CREATE INDEX IF NOT EXISTS idx_comments_task ON comments (taskId);
+      CREATE INDEX IF NOT EXISTS idx_dependencies_task ON dependencies (taskId);
+      CREATE INDEX IF NOT EXISTS idx_dependencies_upstream ON dependencies (dependsOnTaskId);
+    `);
+
     for (const column of ['name TEXT', 'tenantId TEXT']) {
       try {
         this.db.exec(`ALTER TABLE webhooks ADD COLUMN ${column}`);
@@ -516,13 +536,13 @@ export class SQLiteStore implements StorageAdapter {
   async getContainers(projectId: string): Promise<TaskContainer[]> {
     const stmt = this.db.prepare('SELECT * FROM containers WHERE projectId = ?');
     const rows = stmt.all(projectId) as any[];
-    return rows;
+    return rows.map(dropNulls);
   }
 
   async getContainer(id: string): Promise<TaskContainer | null> {
     const stmt = this.db.prepare('SELECT * FROM containers WHERE id = ?');
     const row = stmt.get(id) as any;
-    return row || null;
+    return row ? dropNulls(row) : null;
   }
 
   async createContainer(container: Omit<TaskContainer, 'id' | 'createdAt' | 'updatedAt'>): Promise<TaskContainer> {
@@ -680,6 +700,55 @@ export class SQLiteStore implements StorageAdapter {
     return rows.map((r) => this.mapTask(r));
   }
 
+  async queryTasks(query: TaskQuery): Promise<Page<Task>> {
+    const where: string[] = [];
+    const params: Array<string | number> = [];
+    const inList = (column: string, values: string[]) => {
+      where.push(`${column} IN (${values.map(() => '?').join(', ')})`);
+      params.push(...values);
+    };
+
+    if (query.projectId) {
+      where.push('projectId = ?');
+      params.push(query.projectId);
+    }
+    if (query.projectIds) {
+      if (query.projectIds.length === 0) return { items: [] };
+      inList('projectId', query.projectIds);
+    }
+    if (query.status?.length) inList('status', query.status);
+    if (query.priority?.length) inList('priority', query.priority);
+    for (const column of ['iterationId', 'deliverableId', 'containerId'] as const) {
+      if (query[column]) {
+        where.push(`${column} = ?`);
+        params.push(query[column]!);
+      }
+    }
+    if (query.parentId === null) where.push('parentId IS NULL');
+    else if (query.parentId) {
+      where.push('parentId = ?');
+      params.push(query.parentId);
+    }
+    if (query.assigneeId) {
+      where.push(
+        "(assigneeId = ? OR EXISTS (SELECT 1 FROM json_each(COALESCE(assignees, '[]')) WHERE json_extract(value, '$.id') = ?))"
+      );
+      params.push(query.assigneeId, query.assigneeId);
+    }
+    const after = decodeCursor(query.cursor);
+    if (after) {
+      where.push('(createdAt > ? OR (createdAt = ? AND id > ?))');
+      params.push(after[0], after[0], after[1]);
+    }
+
+    const size = pageSize(query.limit);
+    const sql = `SELECT * FROM tasks ${where.length ? `WHERE ${where.join(' AND ')}` : ''} ORDER BY createdAt ASC, id ASC LIMIT ?`;
+    const rows = this.db.prepare(sql).all(...params, size + 1) as any[];
+    const items = rows.slice(0, size).map((r) => this.mapTask(r));
+    const last = items[items.length - 1];
+    return { items, nextCursor: rows.length > size && last ? encodeCursor(last.createdAt, last.id) : undefined };
+  }
+
   async getTask(id: string): Promise<Task | null> {
     const stmt = this.db.prepare('SELECT * FROM tasks WHERE id = ?');
     const row = stmt.get(id) as any;
@@ -821,13 +890,13 @@ export class SQLiteStore implements StorageAdapter {
   async getIterations(projectId: string): Promise<Iteration[]> {
     const stmt = this.db.prepare('SELECT * FROM iterations WHERE projectId = ?');
     const rows = stmt.all(projectId) as any[];
-    return rows;
+    return rows.map(dropNulls);
   }
 
   async getIteration(id: string): Promise<Iteration | null> {
     const stmt = this.db.prepare('SELECT * FROM iterations WHERE id = ?');
     const row = stmt.get(id) as any;
-    return row || null;
+    return row ? dropNulls(row) : null;
   }
 
   async createIteration(iteration: Omit<Iteration, 'id' | 'createdAt'>): Promise<Iteration> {
@@ -883,7 +952,7 @@ export class SQLiteStore implements StorageAdapter {
 
   // --- Comments ---
   private mapComment(row: any): Comment {
-    return {
+    return dropNulls({
       id: row.id,
       taskId: row.taskId,
       authorId: row.authorId,
@@ -895,7 +964,7 @@ export class SQLiteStore implements StorageAdapter {
       metadata: row.metadata ? JSON.parse(row.metadata) : undefined,
       createdAt: row.createdAt,
       updatedAt: row.updatedAt
-    };
+    });
   }
 
   async getComments(taskId: string): Promise<Comment[]> {
@@ -1028,22 +1097,24 @@ export class SQLiteStore implements StorageAdapter {
 
     const stmt = this.db.prepare(sql);
     const rows = stmt.all(...params) as any[];
-    return rows.map((r) => ({
-      ...r,
-      sizeBytes: Number(r.sizeBytes),
-      metadata: r.metadata ? JSON.parse(r.metadata) : undefined
-    }));
+    return rows.map((r) =>
+      dropNulls({
+        ...r,
+        sizeBytes: Number(r.sizeBytes),
+        metadata: r.metadata ? JSON.parse(r.metadata) : undefined
+      })
+    );
   }
 
   async getAttachment(id: string): Promise<Attachment | null> {
     const stmt = this.db.prepare('SELECT * FROM attachments WHERE id = ?');
     const row = stmt.get(id) as any;
     if (!row) return null;
-    return {
+    return dropNulls({
       ...row,
       sizeBytes: Number(row.sizeBytes),
       metadata: row.metadata ? JSON.parse(row.metadata) : undefined
-    };
+    });
   }
 
   async createAttachment(attachment: Omit<Attachment, 'id' | 'createdAt' | 'updatedAt'>): Promise<Attachment> {
@@ -1106,10 +1177,44 @@ export class SQLiteStore implements StorageAdapter {
 
     const stmt = this.db.prepare(sql);
     const rows = stmt.all(...params) as any[];
-    return rows.map((r) => ({
-      ...r,
-      details: r.details ? JSON.parse(r.details) : undefined
-    }));
+    return rows.map((r) =>
+      dropNulls({
+        ...r,
+        details: r.details ? JSON.parse(r.details) : undefined
+      })
+    );
+  }
+
+  async queryActivities(query: ActivityQuery): Promise<Page<Activity>> {
+    const where: string[] = [];
+    const params: Array<string | number> = [];
+    if (query.projectId) {
+      where.push('projectId = ?');
+      params.push(query.projectId);
+    }
+    if (query.projectIds) {
+      if (query.projectIds.length === 0) return { items: [] };
+      where.push(`projectId IN (${query.projectIds.map(() => '?').join(', ')})`);
+      params.push(...query.projectIds);
+    }
+    if (query.taskId) {
+      where.push('taskId = ?');
+      params.push(query.taskId);
+    }
+    const after = decodeCursor(query.cursor);
+    if (after) {
+      where.push('(createdAt < ? OR (createdAt = ? AND id < ?))');
+      params.push(after[0], after[0], after[1]);
+    }
+
+    const size = pageSize(query.limit);
+    const sql = `SELECT * FROM activities ${where.length ? `WHERE ${where.join(' AND ')}` : ''} ORDER BY createdAt DESC, id DESC LIMIT ?`;
+    const rows = this.db.prepare(sql).all(...params, size + 1) as any[];
+    const items: Activity[] = rows
+      .slice(0, size)
+      .map((r) => dropNulls({ ...r, details: r.details ? JSON.parse(r.details) : undefined }));
+    const last = items[items.length - 1];
+    return { items, nextCursor: rows.length > size && last ? encodeCursor(last.createdAt, last.id) : undefined };
   }
 
   async logActivity(activity: Omit<Activity, 'id' | 'createdAt'>): Promise<Activity> {
@@ -1137,7 +1242,7 @@ export class SQLiteStore implements StorageAdapter {
   async getTimeEntries(taskId: string): Promise<TimeEntry[]> {
     const stmt = this.db.prepare('SELECT * FROM time_entries WHERE taskId = ?');
     const rows = stmt.all(taskId) as any[];
-    return rows.map((r) => ({ ...r, isBillable: r.isBillable !== null ? Boolean(r.isBillable) : undefined }));
+    return rows.map((r) => dropNulls({ ...r, isBillable: r.isBillable !== null ? Boolean(r.isBillable) : undefined }));
   }
 
   async logTime(entry: Omit<TimeEntry, 'id' | 'loggedAt'> & { loggedAt?: string }): Promise<TimeEntry> {
@@ -1164,7 +1269,7 @@ export class SQLiteStore implements StorageAdapter {
   // --- Dependencies ---
   async getDependencies(taskId: string): Promise<TaskDependency[]> {
     const stmt = this.db.prepare('SELECT * FROM dependencies WHERE taskId = ? OR dependsOnTaskId = ?');
-    return stmt.all(taskId, taskId) as any[];
+    return (stmt.all(taskId, taskId) as any[]).map(dropNulls);
   }
 
   async addDependency(dep: Omit<TaskDependency, 'id'>): Promise<TaskDependency> {
@@ -1179,6 +1284,19 @@ export class SQLiteStore implements StorageAdapter {
     return newDep;
   }
 
+  async getDependency(id: string): Promise<TaskDependency | null> {
+    const row = this.db.prepare('SELECT * FROM dependencies WHERE id = ?').get(id) as any;
+    return row ? { id: row.id, taskId: row.taskId, dependsOnTaskId: row.dependsOnTaskId, type: row.type } : null;
+  }
+
+  async removeDependency(id: string): Promise<boolean> {
+    return Number(this.db.prepare('DELETE FROM dependencies WHERE id = ?').run(id).changes) > 0;
+  }
+
+  async deleteTimeEntry(id: string): Promise<boolean> {
+    return Number(this.db.prepare('DELETE FROM time_entries WHERE id = ?').run(id).changes) > 0;
+  }
+
   // --- Webhooks ---
   async getWebhooks(): Promise<Webhook[]> {
     const stmt = this.db.prepare('SELECT * FROM webhooks');
@@ -1215,7 +1333,7 @@ export class SQLiteStore implements StorageAdapter {
   }
 
   private mapWebhook(row: any): Webhook {
-    return {
+    return dropNulls({
       id: row.id,
       name: row.name ?? '',
       url: row.url,
@@ -1224,7 +1342,7 @@ export class SQLiteStore implements StorageAdapter {
       active: Boolean(row.active),
       tenantId: row.tenantId ?? undefined,
       createdAt: row.createdAt
-    };
+    });
   }
 
   async addWebhook(webhook: Omit<Webhook, 'id' | 'createdAt'>): Promise<Webhook> {
@@ -1251,7 +1369,7 @@ export class SQLiteStore implements StorageAdapter {
 
   // Helper mappers
   private mapProject(row: any): Project {
-    return {
+    return dropNulls({
       ...row,
       tenantId: row.tenantId ?? undefined,
       workflowId: row.workflowId || undefined,
@@ -1261,30 +1379,30 @@ export class SQLiteStore implements StorageAdapter {
       statusDefinitions: row.statusDefinitions ? JSON.parse(row.statusDefinitions) : [],
       priorityDefinitions: row.priorityDefinitions ? JSON.parse(row.priorityDefinitions) : [],
       customFieldDefinitions: row.customFieldDefinitions ? JSON.parse(row.customFieldDefinitions) : []
-    };
+    });
   }
 
   private mapWorkflow(row: any): Workflow {
-    return {
+    return dropNulls({
       ...row,
       tenantId: row.tenantId ?? undefined,
       statuses: row.statuses ? JSON.parse(row.statuses) : [],
       transitions: row.transitions ? JSON.parse(row.transitions) : [],
       taskTypes: row.taskTypes ? JSON.parse(row.taskTypes) : undefined,
       isDefault: row.isDefault !== null && row.isDefault !== undefined ? Boolean(row.isDefault) : undefined
-    };
+    });
   }
 
   private mapTeam(row: any): Team {
-    return {
+    return dropNulls({
       ...row,
       tenantId: row.tenantId ?? undefined,
       memberIds: row.memberIds ? JSON.parse(row.memberIds) : []
-    };
+    });
   }
 
   private mapTask(row: any): Task {
-    return {
+    return dropNulls({
       ...row,
       actualDurationSeconds: row.actualDurationSeconds ?? undefined,
       inProgressSince: row.inProgressSince || undefined,
@@ -1299,15 +1417,15 @@ export class SQLiteStore implements StorageAdapter {
       tags: row.tags ? JSON.parse(row.tags) : [],
       assignees: row.assignees ? JSON.parse(row.assignees) : undefined,
       customFields: row.customFields ? JSON.parse(row.customFields) : {}
-    };
+    });
   }
 
   private mapDeliverable(row: any): Deliverable {
-    return {
+    return dropNulls({
       ...row,
       specs: row.specs ? JSON.parse(row.specs) : undefined,
       outputUrls: row.outputUrls ? JSON.parse(row.outputUrls) : [],
       customFields: row.customFields ? JSON.parse(row.customFields) : {}
-    };
+    });
   }
 }

package/src/webhooks/webhooks.test.ts

@@ -78,7 +78,8 @@ describe('webhook delivery', () => {
     await engine.createTask({ projectId: project.id, title: 'T' });
     await engine.webhooks.idle();
 
-    expect(deliveries.map((d) => d.body.event)).toEqual(['project.created', 'task.created']);
+    // Deliveries are asynchronous, so arrival order is not guaranteed
+    expect(deliveries.map((d) => d.body.event).sort()).toEqual(['project.created', 'task.created']);
     expect(deliveries.every((d) => d.url.endsWith('/all'))).toBe(true);
   });
 

package/tsconfig.tsbuildinfo

@@ -1 +1 @@
-{"root":["./src/deliverables.test.ts","./src/index.test.ts","./src/index.ts","./src/status.test.ts","./src/task-state.test.ts","./src/use-case-validation.test.ts","./src/domain/authorization.ts","./src/domain/calendar.test.ts","./src/domain/calendar.ts","./src/domain/cpm.test.ts","./src/domain/cpm.ts","./src/domain/custom-fields.ts","./src/domain/ddd.test.ts","./src/domain/entities.ts","./src/domain/errors.ts","./src/domain/events.ts","./src/domain/graph.ts","./src/domain/index.ts","./src/domain/ladder.test.ts","./src/domain/ladder.ts","./src/domain/metrics.test.ts","./src/domain/metrics.ts","./src/domain/task-duration.test.ts","./src/domain/workload.test.ts","./src/domain/workload.ts","./src/engine/actor.test.ts","./src/engine/authorization.test.ts","./src/engine/index.ts","./src/plugins/index.ts","./src/plugins/plugins.test.ts","./src/schemas/index.ts","./src/schemas/schemas.test.ts","./src/storage/file-storage.ts","./src/storage/firebase-storage.ts","./src/storage/index.ts","./src/storage/s3.ts","./src/storage/storage.test.ts","./src/store/firebase.test.ts","./src/store/firebase.ts","./src/store/index.ts","./src/store/sqlite.test.ts","./src/store/sqlite.ts","./src/types/index.ts","./src/utils/fractional-index.test.ts","./src/utils/fractional-index.ts","./src/utils/index.ts","./src/utils/key.test.ts","./src/utils/key.ts","./src/utils/mentions.test.ts","./src/utils/mentions.ts","./src/utils/status.ts","./src/utils/workflow.ts","./src/webhooks/dispatcher.ts","./src/webhooks/index.ts","./src/webhooks/signature.ts","./src/webhooks/webhooks.test.ts"],"version":"5.9.3"}
+{"root":["./src/deliverables.test.ts","./src/index.test.ts","./src/index.ts","./src/status.test.ts","./src/task-state.test.ts","./src/use-case-validation.test.ts","./src/domain/authorization.ts","./src/domain/calendar.test.ts","./src/domain/calendar.ts","./src/domain/cpm.test.ts","./src/domain/cpm.ts","./src/domain/custom-fields.ts","./src/domain/ddd.test.ts","./src/domain/entities.ts","./src/domain/errors.ts","./src/domain/events.ts","./src/domain/graph.ts","./src/domain/index.ts","./src/domain/ladder.test.ts","./src/domain/ladder.ts","./src/domain/metrics.test.ts","./src/domain/metrics.ts","./src/domain/task-duration.test.ts","./src/domain/workload.test.ts","./src/domain/workload.ts","./src/engine/actor.test.ts","./src/engine/authorization.test.ts","./src/engine/cascade.test.ts","./src/engine/index.ts","./src/plugins/index.ts","./src/plugins/plugins.test.ts","./src/schemas/index.ts","./src/schemas/schemas.test.ts","./src/storage/file-storage.ts","./src/storage/firebase-storage.ts","./src/storage/index.ts","./src/storage/s3.ts","./src/storage/storage.test.ts","./src/store/firebase.test.ts","./src/store/firebase.ts","./src/store/index.ts","./src/store/query.test.ts","./src/store/query.ts","./src/store/sqlite.test.ts","./src/store/sqlite.ts","./src/types/index.ts","./src/utils/fractional-index.test.ts","./src/utils/fractional-index.ts","./src/utils/index.ts","./src/utils/key.test.ts","./src/utils/key.ts","./src/utils/mentions.test.ts","./src/utils/mentions.ts","./src/utils/status.ts","./src/utils/workflow.ts","./src/webhooks/dispatcher.ts","./src/webhooks/index.ts","./src/webhooks/signature.ts","./src/webhooks/webhooks.test.ts"],"version":"5.9.3"}