fix: resolve review feedback

Author: mattmillerai

src/stores/assetsStore.test.ts

@@ -1741,6 +1741,7 @@ describe('assetsStore - Flat Output Assets (cloud-only)', () => {
       await Promise.all([loadMoreResult, refreshResult])
 
       expect(store.flatOutputAssets.map((a) => a.id)).toContain('fresh-1')
+      expect(store.flatOutputAssets.map((a) => a.id)).not.toContain('extra-1')
     })
 
     it('a second concurrent refresh coalesces into the first refresh promise', async () => {

src/stores/assetsStore.ts

@@ -122,7 +122,6 @@ export const useAssetsStore = defineStore('assets', () => {
   const allHistoryItems = ref<AssetItem[]>([])
 
   const loadedIds = shallowReactive(new Set<string>())
-  const loadedJobIds = new Set<string>()
 
   const fetchInputFiles = isCloud
     ? fetchInputFilesFromCloud
@@ -157,17 +156,12 @@ export const useAssetsStore = defineStore('assets', () => {
       hasMoreHistory.value = true
       allHistoryItems.value = []
       loadedIds.clear()
-      loadedJobIds.clear()
     }
 
     const history = await api.getHistory(BATCH_SIZE, {
       offset: historyOffset.value
     })
 
-    for (const job of history) {
-      loadedJobIds.add(job.id)
-    }
-
     const newAssets = mapHistoryToAssets(history)
 
     if (loadMore) {
@@ -267,6 +261,7 @@ export const useAssetsStore = defineStore('assets', () => {
   const flatOutputSeenIds = new Set<string>()
   let flatOutputRefreshInFlight: Promise<AssetItem[]> | null = null
   let flatOutputLoadMoreInFlight: Promise<AssetItem[]> | null = null
+  let flatOutputGeneration = 0
 
   async function fetchFlatOutputs(loadMore: boolean): Promise<AssetItem[]> {
     if (loadMore) {
@@ -275,19 +270,25 @@ export const useAssetsStore = defineStore('assets', () => {
       flatOutputIsLoadingMore.value = true
     } else {
       if (flatOutputRefreshInFlight) return flatOutputRefreshInFlight
+      flatOutputGeneration++
       flatOutputLoading.value = true
       flatOutputOffset.value = 0
       flatOutputHasMore.value = true
       flatOutputSeenIds.clear()
     }
     flatOutputError.value = null
 
+    const generation = flatOutputGeneration
+
     const inFlight = (async () => {
       try {
         const page = await assetService.getAssetsByTag(OUTPUT_TAG, true, {
           limit: FLAT_OUTPUT_PAGE_SIZE,
           offset: flatOutputOffset.value
         })
+        if (loadMore && generation !== flatOutputGeneration) {
+          return flatOutputAssets.value
+        }
         const fresh = loadMore
           ? page.filter((asset) => !flatOutputSeenIds.has(asset.id))
           : page