@critical-path/svelte 0.5.3 → 0.5.4

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @critical-path/svelte
 
+## 0.5.4
+
+### Patch Changes
+
+- 9aa53f7: Optimistically remove comments and attachments upon delete with automatic rollback on network failure
+
 ## 0.5.3
 
 ### Patch Changes

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

@@ -1 +1 @@
-{"version":3,"file":"activity-state.svelte.d.ts","sourceRoot":"","sources":["../src/activity-state.svelte.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,uBAAuB,CAAC;AAChE,OAAO,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAC;AAE/D,MAAM,WAAW,8BAA+B,SAAQ,OAAO;IAC7D,WAAW,EAAE,UAAU,EAAE,CAAC;IAC1B,OAAO,EAAE,8BAA8B,EAAE,CAAC;CAC3C;AAED,qBAAa,iBAAiB;IA6ChB,OAAO,CAAC,MAAM;IAA6B,MAAM,CAAC,EAAE,MAAM;IA5CtE,QAAQ,YAAyB;IACjC,WAAW,eAA4B;IACvC,OAAO,UAA0B;IACjC,KAAK,eAA8B;IAEnC,OAAO,mCAiCJ;IAEH,qBAAqB,eAElB;gBAEiB,MAAM,EAAE,kBAAkB,EAAS,MAAM,CAAC,EAAE,MAAM,YAAA;IAEhE,KAAK,CAAC,MAAM,CAAC,EAAE,MAAM;IAwBrB,UAAU,CACd,KAAK,EAAE,IAAI,CAAC,OAAO,EAAE,IAAI,GAAG,QAAQ,GAAG,WAAW,GAAG,WAAW,CAAC,EACjE,gBAAgB,CAAC,EAAE,KAAK,CAAC,IAAI,CAAC,UAAU,EAAE,IAAI,GAAG,QAAQ,GAAG,WAAW,GAAG,WAAW,GAAG,WAAW,CAAC,CAAC;;;;IAiCjG,aAAa,CAAC,KAAK,EAAE,IAAI,CAAC,UAAU,EAAE,IAAI,GAAG,QAAQ,GAAG,WAAW,GAAG,WAAW,CAAC;IAelF,aAAa,CAAC,EAAE,EAAE,MAAM;IAWxB,gBAAgB,CAAC,EAAE,EAAE,MAAM;CAUlC;AAED,wBAAgB,uBAAuB,CAAC,MAAM,EAAE,kBAAkB,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,iBAAiB,CAEtG"}
+{"version":3,"file":"activity-state.svelte.d.ts","sourceRoot":"","sources":["../src/activity-state.svelte.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,uBAAuB,CAAC;AAChE,OAAO,KAAK,EAAE,OAAO,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAC;AAE/D,MAAM,WAAW,8BAA+B,SAAQ,OAAO;IAC7D,WAAW,EAAE,UAAU,EAAE,CAAC;IAC1B,OAAO,EAAE,8BAA8B,EAAE,CAAC;CAC3C;AAED,qBAAa,iBAAiB;IA6ChB,OAAO,CAAC,MAAM;IAA6B,MAAM,CAAC,EAAE,MAAM;IA5CtE,QAAQ,YAAyB;IACjC,WAAW,eAA4B;IACvC,OAAO,UAA0B;IACjC,KAAK,eAA8B;IAEnC,OAAO,mCAiCJ;IAEH,qBAAqB,eAElB;gBAEiB,MAAM,EAAE,kBAAkB,EAAS,MAAM,CAAC,EAAE,MAAM,YAAA;IAEhE,KAAK,CAAC,MAAM,CAAC,EAAE,MAAM;IAwBrB,UAAU,CACd,KAAK,EAAE,IAAI,CAAC,OAAO,EAAE,IAAI,GAAG,QAAQ,GAAG,WAAW,GAAG,WAAW,CAAC,EACjE,gBAAgB,CAAC,EAAE,KAAK,CAAC,IAAI,CAAC,UAAU,EAAE,IAAI,GAAG,QAAQ,GAAG,WAAW,GAAG,WAAW,GAAG,WAAW,CAAC,CAAC;;;;IAiCjG,aAAa,CAAC,KAAK,EAAE,IAAI,CAAC,UAAU,EAAE,IAAI,GAAG,QAAQ,GAAG,WAAW,GAAG,WAAW,CAAC;IAelF,aAAa,CAAC,EAAE,EAAE,MAAM;IAaxB,gBAAgB,CAAC,EAAE,EAAE,MAAM;CAYlC;AAED,wBAAgB,uBAAuB,CAAC,MAAM,EAAE,kBAAkB,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,iBAAiB,CAEtG"}

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

@@ -1 +1 @@
-{"version":3,"file":"attachment-state.svelte.d.ts","sourceRoot":"","sources":["../src/attachment-state.svelte.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,uBAAuB,CAAC;AAChE,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAC;AAEtD,MAAM,WAAW,gBAAgB;IAC/B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,qBAAa,eAAe;IAKd,OAAO,CAAC,MAAM;IAA6B,MAAM,CAAC,EAAE,gBAAgB;IAJhF,IAAI,eAA4B;IAChC,OAAO,UAA0B;IACjC,KAAK,eAA8B;gBAEf,MAAM,EAAE,kBAAkB,EAAS,MAAM,CAAC,EAAE,gBAAgB,YAAA;IAE1E,KAAK,CAAC,MAAM,CAAC,EAAE,gBAAgB;IAe/B,gBAAgB,CAAC,KAAK,EAAE,IAAI,CAAC,UAAU,EAAE,IAAI,GAAG,WAAW,GAAG,WAAW,CAAC;IAY1E,gBAAgB,CAAC,EAAE,EAAE,MAAM;CAUlC;AAED,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,kBAAkB,EAAE,aAAa,CAAC,EAAE,gBAAgB,GAAG,eAAe,CAEnH"}
+{"version":3,"file":"attachment-state.svelte.d.ts","sourceRoot":"","sources":["../src/attachment-state.svelte.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,uBAAuB,CAAC;AAChE,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,qBAAqB,CAAC;AAEtD,MAAM,WAAW,gBAAgB;IAC/B,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,qBAAa,eAAe;IAKd,OAAO,CAAC,MAAM;IAA6B,MAAM,CAAC,EAAE,gBAAgB;IAJhF,IAAI,eAA4B;IAChC,OAAO,UAA0B;IACjC,KAAK,eAA8B;gBAEf,MAAM,EAAE,kBAAkB,EAAS,MAAM,CAAC,EAAE,gBAAgB,YAAA;IAE1E,KAAK,CAAC,MAAM,CAAC,EAAE,gBAAgB;IAe/B,gBAAgB,CAAC,KAAK,EAAE,IAAI,CAAC,UAAU,EAAE,IAAI,GAAG,WAAW,GAAG,WAAW,CAAC;IAY1E,gBAAgB,CAAC,EAAE,EAAE,MAAM;CAYlC;AAED,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,kBAAkB,EAAE,aAAa,CAAC,EAAE,gBAAgB,GAAG,eAAe,CAEnH"}

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

@@ -1 +1 @@
-{"version":3,"file":"comment-state.svelte.d.ts","sourceRoot":"","sources":["../src/comment-state.svelte.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,uBAAuB,CAAC;AAChE,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,qBAAqB,CAAC;AAEnD,MAAM,WAAW,eAAgB,SAAQ,OAAO;IAC9C,OAAO,EAAE,eAAe,EAAE,CAAC;CAC5B;AAED,qBAAa,YAAY;IAyBX,OAAO,CAAC,MAAM;IAA6B,MAAM,CAAC,EAAE,MAAM;IAxBtE,IAAI,YAAyB;IAC7B,OAAO,UAA0B;IACjC,KAAK,eAA8B;IAEnC,OAAO,oBAkBJ;gBAEiB,MAAM,EAAE,kBAAkB,EAAS,MAAM,CAAC,EAAE,MAAM,YAAA;IAEhE,KAAK,CAAC,MAAM,CAAC,EAAE,MAAM;IAkBrB,UAAU,CAAC,KAAK,EAAE,IAAI,CAAC,OAAO,EAAE,IAAI,GAAG,QAAQ,GAAG,WAAW,GAAG,WAAW,CAAC;IAe5E,aAAa,CAAC,EAAE,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,CAAC,OAAO,CAAC;IAYnD,aAAa,CAAC,EAAE,EAAE,MAAM;CAU/B;AAED,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,kBAAkB,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,YAAY,CAE5F"}
+{"version":3,"file":"comment-state.svelte.d.ts","sourceRoot":"","sources":["../src/comment-state.svelte.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,uBAAuB,CAAC;AAChE,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,qBAAqB,CAAC;AAEnD,MAAM,WAAW,eAAgB,SAAQ,OAAO;IAC9C,OAAO,EAAE,eAAe,EAAE,CAAC;CAC5B;AAED,qBAAa,YAAY;IAyBX,OAAO,CAAC,MAAM;IAA6B,MAAM,CAAC,EAAE,MAAM;IAxBtE,IAAI,YAAyB;IAC7B,OAAO,UAA0B;IACjC,KAAK,eAA8B;IAEnC,OAAO,oBAkBJ;gBAEiB,MAAM,EAAE,kBAAkB,EAAS,MAAM,CAAC,EAAE,MAAM,YAAA;IAEhE,KAAK,CAAC,MAAM,CAAC,EAAE,MAAM;IAkBrB,UAAU,CAAC,KAAK,EAAE,IAAI,CAAC,OAAO,EAAE,IAAI,GAAG,QAAQ,GAAG,WAAW,GAAG,WAAW,CAAC;IAe5E,aAAa,CAAC,EAAE,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,CAAC,OAAO,CAAC;IAYnD,aAAa,CAAC,EAAE,EAAE,MAAM;CAY/B;AAED,wBAAgB,kBAAkB,CAAC,MAAM,EAAE,kBAAkB,EAAE,MAAM,CAAC,EAAE,MAAM,GAAG,YAAY,CAE5F"}

package/dist/index.js

@@ -311,11 +311,14 @@ var c = class {
 		}
 	}
 	async deleteComment(e) {
+		let t = this.data;
+		this.data = this.data.filter((t) => t.id !== e);
 		try {
-			await this.client.deleteComment(e), this.data = this.data.filter((t) => t.id !== e);
+			await this.client.deleteComment(e);
 		} catch (e) {
-			let t = e instanceof Error ? e : Error(String(e));
-			throw this.error = t, t;
+			this.data = t;
+			let n = e instanceof Error ? e : Error(String(e));
+			throw this.error = n, n;
 		}
 	}
 };
@@ -371,11 +374,14 @@ var u = class {
 		}
 	}
 	async deleteAttachment(e) {
+		let t = this.data;
+		this.data = this.data.filter((t) => t.id !== e);
 		try {
-			await this.client.deleteAttachment(e), this.data = this.data.filter((t) => t.id !== e);
+			await this.client.deleteAttachment(e);
 		} catch (e) {
-			let t = e instanceof Error ? e : Error(String(e));
-			throw this.error = t, t;
+			this.data = t;
+			let n = e instanceof Error ? e : Error(String(e));
+			throw this.error = n, n;
 		}
 	}
 };
@@ -495,19 +501,25 @@ var f = class {
 		}
 	}
 	async deleteComment(e) {
+		let t = this.comments;
+		this.comments = this.comments.filter((t) => t.id !== e);
 		try {
-			await this.client.deleteComment(e), this.comments = this.comments.filter((t) => t.id !== e);
+			await this.client.deleteComment(e);
 		} catch (e) {
-			let t = e instanceof Error ? e : Error(String(e));
-			throw this.error = t, t;
+			this.comments = t;
+			let n = e instanceof Error ? e : Error(String(e));
+			throw this.error = n, n;
 		}
 	}
 	async deleteAttachment(e) {
+		let t = this.attachments;
+		this.attachments = this.attachments.filter((t) => t.id !== e);
 		try {
-			await this.client.deleteAttachment(e), this.attachments = this.attachments.filter((t) => t.id !== e);
+			await this.client.deleteAttachment(e);
 		} catch (e) {
-			let t = e instanceof Error ? e : Error(String(e));
-			throw this.error = t, t;
+			this.attachments = t;
+			let n = e instanceof Error ? e : Error(String(e));
+			throw this.error = n, n;
 		}
 	}
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@critical-path/svelte",
-  "version": "0.5.3",
+  "version": "0.5.4",
   "description": "Svelte 5 Runes state and reactive integrations for Critical Path headless project management framework",
   "author": "Jack James",
   "license": "MIT",

package/src/activity-state.svelte.ts

@@ -129,10 +129,12 @@ export class TaskActivityState {
   }
 
   async deleteComment(id: string) {
+    const previous = this.comments;
+    this.comments = this.comments.filter((c) => c.id !== id);
     try {
       await this.client.deleteComment(id);
-      this.comments = this.comments.filter((c) => c.id !== id);
     } catch (err) {
+      this.comments = previous;
       const errorObj = err instanceof Error ? err : new Error(String(err));
       this.error = errorObj;
       throw errorObj;
@@ -140,10 +142,12 @@ export class TaskActivityState {
   }
 
   async deleteAttachment(id: string) {
+    const previous = this.attachments;
+    this.attachments = this.attachments.filter((a) => a.id !== id);
     try {
       await this.client.deleteAttachment(id);
-      this.attachments = this.attachments.filter((a) => a.id !== id);
     } catch (err) {
+      this.attachments = previous;
       const errorObj = err instanceof Error ? err : new Error(String(err));
       this.error = errorObj;
       throw errorObj;

package/src/attachment-state.svelte.ts

@@ -43,10 +43,12 @@ export class AttachmentState {
   }
 
   async deleteAttachment(id: string) {
+    const previous = this.data;
+    this.data = this.data.filter((a) => a.id !== id);
     try {
       await this.client.deleteAttachment(id);
-      this.data = this.data.filter((a) => a.id !== id);
     } catch (err) {
+      this.data = previous;
       const errorObj = err instanceof Error ? err : new Error(String(err));
       this.error = errorObj;
       throw errorObj;

package/src/comment-state.svelte.ts

@@ -79,10 +79,12 @@ export class CommentState {
   }
 
   async deleteComment(id: string) {
+    const previous = this.data;
+    this.data = this.data.filter((c) => c.id !== id);
     try {
       await this.client.deleteComment(id);
-      this.data = this.data.filter((c) => c.id !== id);
     } catch (err) {
+      this.data = previous;
       const errorObj = err instanceof Error ? err : new Error(String(err));
       this.error = errorObj;
       throw errorObj;

package/src/index.test.ts

@@ -259,6 +259,32 @@ describe('@critical-path/svelte Svelte 5 Runes Test Suite', () => {
       await attachmentState.deleteAttachment('att_1');
       expect(attachmentState.data).toEqual([]);
     });
+
+    it('rolls back optimistic deletion if deleteAttachment fails', async () => {
+      const mockAttachment = {
+        id: 'att_2',
+        filename: 'notes.txt',
+        mimeType: 'text/plain',
+        sizeBytes: 100,
+        url: 'https://storage.example.com/notes.txt',
+        uploaderId: 'u1',
+        uploaderType: 'user' as const,
+        createdAt: '2026-01-01',
+        updatedAt: '2026-01-01'
+      };
+
+      const mockClient = {
+        getAttachments: vi.fn().mockResolvedValue([mockAttachment]),
+        deleteAttachment: vi.fn().mockRejectedValue(new Error('Network error'))
+      } as unknown as CriticalPathClient;
+
+      const attachmentState = new AttachmentState(mockClient, { taskId: 'task_1' });
+      await attachmentState.fetch();
+
+      expect(attachmentState.data).toHaveLength(1);
+      await expect(attachmentState.deleteAttachment('att_2')).rejects.toThrow('Network error');
+      expect(attachmentState.data).toEqual([mockAttachment]);
+    });
   });
 
   describe('TaskActivityState', () => {
@@ -304,6 +330,10 @@ describe('@critical-path/svelte Svelte 5 Runes Test Suite', () => {
       expect(activityState.threads).toHaveLength(1);
       expect(activityState.threads[0].attachments).toEqual([mockAttachment]);
       expect(activityState.standaloneAttachments).toHaveLength(0);
+
+      await activityState.deleteAttachment('att_1');
+      expect(activityState.attachments).toEqual([]);
+      expect(activityState.threads[0].attachments).toEqual([]);
     });
   });
 });