fix(assetDownload): handle race condition and add listener cleanup

- Remove early return that skipped completion/failure events for untracked tasks
- Add processedTaskIds set to deduplicate terminal events
- Add teardown() method to properly remove event listener on unmount
- Use strict equality operator in useUploadModelWizard

Author: luke-mino-altherr

src/platform/assets/composables/useUploadModelWizard.ts

@@ -233,7 +233,7 @@ export function useUploadModelWizard(modelTypes: Ref<ModelTypeOption[]>) {
           preview_id: previewId
         })
 
-        if (result.type === 'async' && result.task.status != 'completed') {
+        if (result.type === 'async' && result.task.status !== 'completed') {
           if (selectedModelType.value) {
             assetDownloadStore.trackDownload(
               result.task.task_id,

src/stores/assetDownloadStore.ts

@@ -29,6 +29,7 @@ export const useAssetDownloadStore = defineStore('assetDownload', () => {
   const activeDownloads = ref<Map<string, AssetDownload>>(new Map())
   const pendingModelTypes = new Map<string, string>()
   const lastToastTime = new Map<string, number>()
+  const processedTaskIds = new Set<string>()
 
   const hasActiveDownloads = computed(() => activeDownloads.value.size > 0)
   const downloadList = computed(() =>
@@ -61,12 +62,8 @@ export const useAssetDownloadStore = defineStore('assetDownload', () => {
     const data = e.detail
 
     if (data.status === 'completed' || data.status === 'failed') {
-      if (
-        !activeDownloads.value.has(data.task_id) &&
-        !pendingModelTypes.has(data.task_id)
-      ) {
-        return
-      }
+      if (processedTaskIds.has(data.task_id)) return
+      processedTaskIds.add(data.task_id)
     }
 
     const download: AssetDownload = {
@@ -125,15 +122,23 @@ export const useAssetDownloadStore = defineStore('assetDownload', () => {
     }
   }
 
+  let stopListener: (() => void) | undefined
+
   function setup() {
-    useEventListener(api, 'asset_download', handleAssetDownload)
+    stopListener = useEventListener(api, 'asset_download', handleAssetDownload)
+  }
+
+  function teardown() {
+    stopListener?.()
+    stopListener = undefined
   }
 
   return {
     activeDownloads,
     hasActiveDownloads,
     downloadList,
     trackDownload,
-    setup
+    setup,
+    teardown
   }
 })

src/views/GraphView.vue

@@ -273,6 +273,7 @@ onBeforeUnmount(() => {
   api.removeEventListener('reconnecting', onReconnecting)
   api.removeEventListener('reconnected', onReconnected)
   executionStore.unbindExecutionEvents()
+  assetDownloadStore.teardown()
 
   // Clean up page visibility listener
   if (visibilityListener) {