@startsimpli/ui 0.4.26 → 0.4.27

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@startsimpli/ui",
-  "version": "0.4.26",
+  "version": "0.4.27",
   "description": "Shared UI components package for StartSimpli applications",
   "main": "./src/index.ts",
   "types": "./src/index.ts",
@@ -32,7 +32,7 @@
     "next": "^14.0.0 || ^15.0.0 || ^16.0.0",
     "react": "^18.0.0 || ^19.0.0",
     "react-dom": "^18.0.0 || ^19.0.0",
-    "@startsimpli/auth": "0.4.22"
+    "@startsimpli/auth": "0.4.24"
   },
   "peerDependenciesMeta": {
     "@startsimpli/auth": {

package/src/components/chat/ChatComposer.tsx

@@ -345,13 +345,15 @@ function AttachmentChip({
           {name}
         </span>
         <span className="text-[10px] text-muted-foreground">
-          {status === 'uploading'
-            ? `Uploading… ${pct}%`
-            : status === 'error'
-              ? (error || 'Upload failed')
-              : sizeBytes != null
-                ? formatBytes(sizeBytes)
-                : null}
+          {status === 'queued'
+            ? 'Waiting for deck…'
+            : status === 'uploading'
+              ? `Uploading… ${pct}%`
+              : status === 'error'
+                ? (error || 'Upload failed')
+                : sizeBytes != null
+                  ? formatBytes(sizeBytes)
+                  : null}
         </span>
       </span>
       {status === 'error' && onRetry && (

package/src/components/chat/types.ts

@@ -40,7 +40,12 @@ export interface ChatMessageData {
 export interface PendingAttachment {
   id: string
   name: string
-  status: 'uploading' | 'ready' | 'error'
+  /**
+   * `queued` = parked client-side, waiting for a document to land on (a brand-new
+   * deck has no document until the first turn creates it). It is NOT actively
+   * uploading — surfacing it as `uploading` at 0% reads as a stuck upload.
+   */
+  status: 'queued' | 'uploading' | 'ready' | 'error'
   /** Upload progress 0..1 while `status === 'uploading'`. */
   progress?: number
   /** Object/asset url for an image preview thumbnail. */

package/src/components/chat/useComposerUploads.test.tsx

@@ -122,4 +122,50 @@ describe('useComposerUploads', () => {
       ])
     )
   })
+
+  // --- Regression: prod "dropped images never leave 0" (startsim-0fp3) ---
+  // When there is no document path (demo engine / brand-new deck whose document
+  // never arrives), a queued file used to surface as 'uploading' at progress 0
+  // indefinitely — no honest state, no way to know it's parked, no bound.
+  describe('no-document strand (startsim-0fp3)', () => {
+    it('surfaces a queued file with an honest non-uploading status, not 0%-uploading-forever', () => {
+      const uploader = fakeUploader()
+      const { result } = renderHook(() => useComposerUploads(uploader, undefined))
+      act(() => result.current.attachFiles([file('parked.png')]))
+      expect(result.current.attachments).toHaveLength(1)
+      // The user must be able to tell this is parked, not actively uploading.
+      expect(uploader.uploadAttachment).not.toHaveBeenCalled()
+      expect(result.current.attachments[0].status).toBe('queued')
+    })
+
+    it('removes a queued (never-uploaded) chip and revokes its object url', () => {
+      const revoke = jest.spyOn(URL, 'revokeObjectURL')
+      const uploader = fakeUploader()
+      const { result } = renderHook(() => useComposerUploads(uploader, undefined))
+      act(() => result.current.attachFiles([file('parked.png')]))
+      const chipId = result.current.attachments[0].id
+
+      act(() => result.current.removeAttachment(chipId))
+      expect(result.current.attachments).toHaveLength(0)
+      expect(revoke).toHaveBeenCalled()
+      // Never uploaded → no server-side delete attempted.
+      expect(uploader.deleteAttachment).not.toHaveBeenCalled()
+      revoke.mockRestore()
+    })
+  })
+
+  describe('clear() cleanup', () => {
+    it('empties all chips and revokes every object url', async () => {
+      const revoke = jest.spyOn(URL, 'revokeObjectURL')
+      const uploader = fakeUploader()
+      const { result } = renderHook(() => useComposerUploads(uploader, 'doc-1'))
+      act(() => result.current.attachFiles([file('a.png'), file('b.png')]))
+      await waitFor(() => expect(result.current.attachments).toHaveLength(2))
+
+      act(() => result.current.clear())
+      expect(result.current.attachments).toHaveLength(0)
+      expect(revoke).toHaveBeenCalledTimes(2)
+      revoke.mockRestore()
+    })
+  })
 })

package/src/components/chat/useComposerUploads.ts

@@ -255,9 +255,10 @@ export function useComposerUploads(
       items.map((it) => ({
         id: it.id,
         name: it.name,
-        // A queued chip reads as uploading (indeterminate) to the user.
-        status: it.status === 'queued' ? 'uploading' : it.status,
-        progress: it.status === 'queued' ? 0 : it.progress,
+        // A queued chip is parked (waiting for a document), NOT actively
+        // uploading — surface it honestly so it doesn't read as a stuck 0% upload.
+        status: it.status,
+        progress: it.progress,
         thumbnailUrl: it.thumbnailUrl,
         sizeBytes: it.sizeBytes,
         error: it.error,