feat: support concurrent chunk uploads#314
Conversation
Greptile SummaryThis PR adds concurrent chunk uploading to both the browser and native IO clients, replacing the sequential
Confidence Score: 4/5The concurrent upload logic is sound for Dart's event-loop model, but the first-chunk response is never tested for upload completion, leaving a gap when the resumed first chunk is also the final chunk of the file. The worker-pool design is correct and shared-state mutations are safe under Dart's cooperative scheduler. The first-chunk response omits the isUploadComplete check that every subsequent chunk gets inside uploadNext, meaning finalResponse stays null in that edge case and the caller silently receives the lastResponse fallback. Both lib/src/client_browser.dart and lib/src/client_io.dart share the same first-chunk isUploadComplete omission and the lastResponse/finalResponse fallback behavior. Important Files Changed
Reviews (4): Last reviewed commit: "Commit from GitHub Actions (Format and p..." | Re-trigger Greptile |
This PR updates the SDK to support concurrent chunk uploads.