@critical-path/svelte 0.15.8 → 0.16.0

package/dist/kanban-state.svelte.d.ts

@@ -18,7 +18,7 @@ export declare class KanbanState {
     createTask(input: Omit<Task, 'id' | 'createdAt' | 'updatedAt'>): Promise<Task>;
     moveTask(taskId: string, targetStatus: TaskStatus): Promise<Task>;
     updateTaskStatus(taskId: string, status: TaskStatus): Promise<Task>;
-    deleteTask(taskId: string): Promise<boolean | undefined>;
+    deleteTask(taskId: string): Promise<void>;
 }
 export declare function createKanbanState(client: CriticalPathClient, projectId?: string, options?: KanbanStateOptions): KanbanState;
 //# sourceMappingURL=kanban-state.svelte.d.ts.map

package/dist/task-state.svelte.d.ts

@@ -1,17 +1,19 @@
 import type { CriticalPathClient } from '@critical-path/client';
-import type { Task, TaskStatus } from '@critical-path/core';
+import { type Task, type TaskStatus } from '@critical-path/core';
 export declare class TaskState {
     private client;
     projectId?: string | undefined;
     data: Task[];
     loading: boolean;
     error: Error | null;
+    private pendingCreations;
+    private tempToRealIdMap;
     constructor(client: CriticalPathClient, projectId?: string | undefined);
     fetch(): Promise<void>;
     createTask(input: Omit<Task, 'id' | 'createdAt' | 'updatedAt'>): Promise<Task>;
     updateTask(taskId: string, updates: Partial<Task>): Promise<Task>;
     updateTaskStatus(taskId: string, status: TaskStatus): Promise<Task>;
-    deleteTask(taskId: string): Promise<boolean | undefined>;
+    deleteTask(taskId: string): Promise<void>;
 }
 export declare function createTaskState(client: CriticalPathClient, projectId?: string): TaskState;
 //# sourceMappingURL=task-state.svelte.d.ts.map

package/dist/task-state.svelte.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"task-state.svelte.d.ts","sourceRoot":"","sources":["../src/task-state.svelte.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,uBAAuB,CAAC;AAChE,OAAO,KAAK,EAAE,IAAI,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAC;AAE5D,qBAAa,SAAS;IAMlB,OAAO,CAAC,MAAM;IACP,SAAS,CAAC,EAAE,MAAM;IAN3B,IAAI,SAAsB;IAC1B,OAAO,UAA0B;IACjC,KAAK,eAA8B;gBAGzB,MAAM,EAAE,kBAAkB,EAC3B,SAAS,CAAC,EAAE,MAAM,YAAA;IAGrB,KAAK;IAYL,UAAU,CAAC,KAAK,EAAE,IAAI,CAAC,IAAI,EAAE,IAAI,GAAG,WAAW,GAAG,WAAW,CAAC;IA2B9D,UAAU,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,CAAC,IAAI,CAAC;IAyCjD,gBAAgB,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,UAAU;IAyCnD,UAAU,CAAC,MAAM,EAAE,MAAM;CAuBhC;AAED,wBAAgB,eAAe,CAC7B,MAAM,EAAE,kBAAkB,EAC1B,SAAS,CAAC,EAAE,MAAM,GACjB,SAAS,CAEX"}
+{"version":3,"file":"task-state.svelte.d.ts","sourceRoot":"","sources":["../src/task-state.svelte.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,uBAAuB,CAAC;AAChE,OAAO,EAAE,KAAK,IAAI,EAAE,KAAK,UAAU,EAAgB,MAAM,qBAAqB,CAAC;AAE/E,qBAAa,SAAS;IASlB,OAAO,CAAC,MAAM;IACP,SAAS,CAAC,EAAE,MAAM;IAT3B,IAAI,SAAsB;IAC1B,OAAO,UAA0B;IACjC,KAAK,eAA8B;IAEnC,OAAO,CAAC,gBAAgB,CAAoC;IAC5D,OAAO,CAAC,eAAe,CAA6B;gBAG1C,MAAM,EAAE,kBAAkB,EAC3B,SAAS,CAAC,EAAE,MAAM,YAAA;IAGrB,KAAK;IAYL,UAAU,CAAC,KAAK,EAAE,IAAI,CAAC,IAAI,EAAE,IAAI,GAAG,WAAW,GAAG,WAAW,CAAC;IAiC9D,UAAU,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,CAAC,IAAI,CAAC;IAyEjD,gBAAgB,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,UAAU;IAInD,UAAU,CAAC,MAAM,EAAE,MAAM;CA4ChC;AAED,wBAAgB,eAAe,CAC7B,MAAM,EAAE,kBAAkB,EAC1B,SAAS,CAAC,EAAE,MAAM,GACjB,SAAS,CAEX"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@critical-path/svelte",
-  "version": "0.15.8",
+  "version": "0.16.0",
   "description": "Svelte 5 Runes state and reactive integrations for Critical Path headless project management framework",
   "author": "Jack James",
   "license": "MIT",
@@ -18,9 +18,9 @@
     }
   },
   "dependencies": {
-    "@critical-path/core": "0.20.2",
-    "@critical-path/client": "0.14.4",
-    "@critical-path/mcp": "0.6.7"
+    "@critical-path/client": "0.14.6",
+    "@critical-path/core": "0.21.0",
+    "@critical-path/mcp": "0.6.9"
   },
   "peerDependencies": {
     "svelte": "^5.0.0"

package/src/index.test.ts

@@ -241,6 +241,67 @@ describe('@critical-path/svelte Svelte 5 Runes Test Suite', () => {
       // Should restore deleted task
       expect(taskState.data).toEqual([mockTask]);
     });
+
+    it('resolves in-flight temporary task when updated or deleted before creation completes', async () => {
+      let resolveCreate: (value: Task) => void;
+      const createPromise = new Promise<Task>((resolve) => {
+        resolveCreate = resolve;
+      });
+
+      const mockClient = {
+        getTasks: vi.fn().mockResolvedValue([]),
+        createTask: vi.fn().mockReturnValue(createPromise),
+        updateTask: vi.fn().mockImplementation((taskId: string, updates: Partial<Task>) =>
+          Promise.resolve({ id: taskId, projectId: 'proj_1', title: 'Real Task', status: 'todo', priority: 'medium', createdAt: '2026-01-01', updatedAt: '2026-01-01', ...updates } as Task)
+        ),
+        deleteTask: vi.fn().mockResolvedValue(undefined)
+      } as unknown as CriticalPathClient;
+
+      const taskState = createTaskState(mockClient, 'proj_1');
+      await taskState.fetch();
+
+      // Start creation
+      const creationAction = taskState.createTask({
+        projectId: 'proj_1',
+        title: 'Draft Task',
+        status: 'todo',
+        priority: 'medium'
+      });
+
+      // While in flight, optimistic task with temp_ ID is present
+      expect(taskState.data.length).toBe(1);
+      const tempId = taskState.data[0].id;
+      expect(tempId.startsWith('temp_')).toBe(true);
+
+      // Immediately update using tempId before creation resolves
+      const updateAction = taskState.updateTask(tempId, { title: 'Renamed Draft Task' });
+
+      // Local optimistic state is updated immediately
+      expect(taskState.data[0].title).toBe('Renamed Draft Task');
+
+      // Now resolve the server creation
+      const serverTask: Task = {
+        id: 'real_task_99',
+        projectId: 'proj_1',
+        title: 'Draft Task',
+        status: 'todo',
+        priority: 'medium',
+        createdAt: '2026-01-01',
+        updatedAt: '2026-01-01'
+      };
+      resolveCreate!(serverTask);
+
+      await creationAction;
+      await updateAction;
+
+      // Verify server update was called with the real task ID, not the temp ID
+      expect(mockClient.updateTask).toHaveBeenCalledWith('real_task_99', { title: 'Renamed Draft Task' });
+
+      // And deleting with the tempId resolves to real_task_99
+      await taskState.deleteTask(tempId);
+      expect(mockClient.deleteTask).toHaveBeenCalledWith('real_task_99');
+      expect(taskState.data.length).toBe(0);
+    });
   });
 
   describe('CommentState', () => {

package/src/task-state.svelte.ts

@@ -1,12 +1,15 @@
 /// <reference types="svelte" />
 import type { CriticalPathClient } from '@critical-path/client';
-import type { Task, TaskStatus } from '@critical-path/core';
+import { type Task, type TaskStatus, isTempTaskId } from '@critical-path/core';
 
 export class TaskState {
   data = $state<Task[]>([]);
   loading = $state<boolean>(false);
   error = $state<Error | null>(null);
 
+  private pendingCreations = new Map<string, Promise<Task>>();
+  private tempToRealIdMap = new Map<string, string>();
+
   constructor(
     private client: CriticalPathClient,
     public projectId?: string
@@ -37,8 +40,12 @@ export class TaskState {
     // Optimistically add temp task
     this.data = [...this.data, tempTask];
 
+    const promise = this.client.createTask(input);
+    this.pendingCreations.set(tempId, promise);
+
     try {
-      const created = await this.client.createTask(input);
+      const created = await promise;
+      this.tempToRealIdMap.set(tempId, created.id);
       // Replace temp task with created task from server
       this.data = this.data.map((t) => (t.id === tempId ? created : t));
       return created;
@@ -48,71 +55,62 @@ export class TaskState {
       const errorObj = err instanceof Error ? err : new Error(String(err));
       this.error = errorObj;
       throw errorObj;
+    } finally {
+      this.pendingCreations.delete(tempId);
     }
   }
 
   async updateTask(taskId: string, updates: Partial<Task>) {
-    const idx = this.data.findIndex((t) => t.id === taskId);
-    if (idx === -1) {
-      return this.client.updateTask(taskId, updates);
+    let targetId = taskId;
+    if (this.tempToRealIdMap.has(taskId)) {
+      targetId = this.tempToRealIdMap.get(taskId)!;
     }
 
-    const previousTask = this.data[idx];
-    const optimisticTask: Task = {
-      ...previousTask,
-      ...updates,
-      updatedAt: new Date().toISOString()
-    };
-
-    // Optimistically update local task
-    const newData = [...this.data];
-    newData[idx] = optimisticTask;
-    this.data = newData;
+    // 1. Immediately apply optimistic updates to the local task (temp or confirmed)
+    const idx = this.data.findIndex((t) => t.id === targetId || t.id === taskId);
+    let previousTask: Task | undefined;
+    if (idx !== -1) {
+      previousTask = this.data[idx];
+      const optimisticTask: Task = {
+        ...previousTask,
+        ...updates,
+        updatedAt: new Date().toISOString()
+      };
+      const newData = [...this.data];
+      newData[idx] = optimisticTask;
+      this.data = newData;
+    }
 
-    try {
-      const updated = await this.client.updateTask(taskId, updates);
-      const currentIdx = this.data.findIndex((t) => t.id === taskId);
-      if (currentIdx !== -1) {
-        const confirmedData = [...this.data];
-        confirmedData[currentIdx] = updated;
-        this.data = confirmedData;
-      }
-      return updated;
-    } catch (err) {
-      // Rollback to previous task on error
-      const rollbackIdx = this.data.findIndex((t) => t.id === taskId);
-      if (rollbackIdx !== -1) {
-        const rollbackData = [...this.data];
-        rollbackData[rollbackIdx] = previousTask;
-        this.data = rollbackData;
+    // 2. If it's a temporary task, await pending creation if in flight
+    if (isTempTaskId(targetId)) {
+      const pendingPromise =
+        this.pendingCreations.get(targetId) ||
+        (this.pendingCreations.size === 1
+          ? Array.from(this.pendingCreations.values())[0]
+          : undefined);
+
+      if (pendingPromise) {
+        try {
+          const created = await pendingPromise;
+          if (created && created.id) {
+            targetId = created.id;
+            this.tempToRealIdMap.set(taskId, targetId);
+          }
+        } catch {
+          throw new Error(`Failed to update task: task creation failed for "${taskId}"`);
+        }
       }
-      const errorObj = err instanceof Error ? err : new Error(String(err));
-      this.error = errorObj;
-      throw errorObj;
     }
-  }
 
-  async updateTaskStatus(taskId: string, status: TaskStatus) {
-    const idx = this.data.findIndex((t) => t.id === taskId);
-    if (idx === -1) {
-      return this.client.updateTask(taskId, { status });
+    // If still a temp ID, task exists purely in local client state; avoid sending unresolvable temp ID
+    if (isTempTaskId(targetId)) {
+      const currentTask = this.data.find((t) => t.id === targetId || t.id === taskId);
+      return (currentTask || { id: targetId, ...updates }) as Task;
     }
 
-    const previousTask = this.data[idx];
-    const optimisticTask: Task = {
-      ...previousTask,
-      status,
-      updatedAt: new Date().toISOString()
-    };
-
-    // Optimistically update local task status
-    const newData = [...this.data];
-    newData[idx] = optimisticTask;
-    this.data = newData;
-
     try {
-      const updated = await this.client.updateTask(taskId, { status });
-      const currentIdx = this.data.findIndex((t) => t.id === taskId);
+      const updated = await this.client.updateTask(targetId, updates);
+      const currentIdx = this.data.findIndex((t) => t.id === targetId || t.id === taskId);
       if (currentIdx !== -1) {
         const confirmedData = [...this.data];
         confirmedData[currentIdx] = updated;
@@ -121,11 +119,13 @@ export class TaskState {
       return updated;
     } catch (err) {
       // Rollback to previous task on error
-      const rollbackIdx = this.data.findIndex((t) => t.id === taskId);
-      if (rollbackIdx !== -1) {
-        const rollbackData = [...this.data];
-        rollbackData[rollbackIdx] = previousTask;
-        this.data = rollbackData;
+      if (previousTask) {
+        const rollbackIdx = this.data.findIndex((t) => t.id === targetId || t.id === taskId);
+        if (rollbackIdx !== -1) {
+          const rollbackData = [...this.data];
+          rollbackData[rollbackIdx] = previousTask;
+          this.data = rollbackData;
+        }
       }
       const errorObj = err instanceof Error ? err : new Error(String(err));
       this.error = errorObj;
@@ -133,24 +133,49 @@ export class TaskState {
     }
   }
 
+  async updateTaskStatus(taskId: string, status: TaskStatus) {
+    return this.updateTask(taskId, { status });
+  }
+
   async deleteTask(taskId: string) {
-    const idx = this.data.findIndex((t) => t.id === taskId);
-    if (idx === -1) {
-      return this.client.deleteTask(taskId);
+    let targetId = taskId;
+    if (this.tempToRealIdMap.has(taskId)) {
+      targetId = this.tempToRealIdMap.get(taskId)!;
     }
 
-    const previousTask = this.data[idx];
+    const idx = this.data.findIndex((t) => t.id === targetId || t.id === taskId);
+    const previousTask = idx !== -1 ? this.data[idx] : undefined;
 
-    // Optimistically remove task
-    this.data = this.data.filter((t) => t.id !== taskId);
+    // Optimistically remove task from local state
+    if (idx !== -1) {
+      this.data = this.data.filter((t) => t.id !== targetId && t.id !== taskId);
+    }
+
+    // If temporary task:
+    if (isTempTaskId(targetId)) {
+      const pendingPromise = this.pendingCreations.get(targetId);
+      if (pendingPromise) {
+        try {
+          const created = await pendingPromise;
+          if (created && created.id) {
+            await this.client.deleteTask(created.id);
+          }
+        } catch {
+          // If creation failed anyway, task is already gone
+        }
+      }
+      return;
+    }
 
     try {
-      await this.client.deleteTask(taskId);
+      await this.client.deleteTask(targetId);
     } catch (err) {
       // Rollback on error by re-inserting at previous index
-      const restoredData = [...this.data];
-      restoredData.splice(idx, 0, previousTask);
-      this.data = restoredData;
+      if (previousTask && idx !== -1) {
+        const restoredData = [...this.data];
+        restoredData.splice(idx, 0, previousTask);
+        this.data = restoredData;
+      }
       const errorObj = err instanceof Error ? err : new Error(String(err));
       this.error = errorObj;
       throw errorObj;