youngkingdom/dify
1
0
Fork 0
mirror of https://github.com/langgenius/dify.git synced 2026-05-04 17:38:04 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(skill): resolve race condition in upload progress tracking

Browse Source 
Use shared object reference instead of separate variables to track
upload progress across concurrent Promise.all operations, preventing
progress bar from showing incorrect or regressing values.
This commit is contained in:
yyh
2026-01-28 20:25:12 +08:00
parent 76c4d7f62c
commit acec271e88
1 changed files with 6 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
web/app/components/workflow/skill/hooks/use-create-operations.ts
View File

@ -53,8 +53,7 @@ export function useCreateOperations({
}
const total = files.length
let uploaded = 0
let failed = 0
const progress = { uploaded: 0, failed: 0 }
storeApi.getState().setUploadStatus('uploading')
storeApi.getState().setUploadProgress({ uploaded: 0, total, failed: 0 })
@ -65,17 +64,17 @@ export function useCreateOperations({
uploadFiles.map(async (file) => {
try {
await uploadFile.mutateAsync({ appId, file, parentId })
uploaded++
progress.uploaded++
}
catch {
failed++
progress.failed++
}
storeApi.getState().setUploadProgress({ uploaded, total, failed })
storeApi.getState().setUploadProgress({ uploaded: progress.uploaded, total, failed: progress.failed })
}),
)
storeApi.getState().setUploadStatus(failed > 0 ? 'partial_error' : 'success')
storeApi.getState().setUploadProgress({ uploaded, total, failed })
storeApi.getState().setUploadStatus(progress.failed > 0 ? 'partial_error' : 'success')
storeApi.getState().setUploadProgress({ uploaded: progress.uploaded, total, failed: progress.failed })
}
catch {
storeApi.getState().setUploadStatus('partial_error')