Skip to content

chore: bump SDK version to 0.31.0#107

Merged
TorstenDittmann merged 1 commit into
mainfrom
concurrent-chunk-uploads-version-bump-1-9-x
May 21, 2026
Merged

chore: bump SDK version to 0.31.0#107
TorstenDittmann merged 1 commit into
mainfrom
concurrent-chunk-uploads-version-bump-1-9-x

Conversation

@TorstenDittmann
Copy link
Copy Markdown
Contributor

This PR bumps the SDK package version to 0.31.0 after the concurrent chunk upload update.

@TorstenDittmann
Copy link
Copy Markdown
Contributor Author

Closing this follow-up because the package version is already bumped on main and this generated diff only contains unrelated drift.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 21, 2026

Greptile Summary

This PR bumps the SDK to 0.31.0, introducing concurrent chunk uploads in storage.ts (up to 8 parallel workers) and a web-platform compatibility fix in realtime.ts that avoids passing custom headers to the browser WebSocket constructor.

  • storage.ts: Replaces the sequential chunk-upload loop with a worker-pool pattern. The first chunk is uploaded alone to obtain the upload ID, then remaining chunks are dispatched up to 8 at a time. Resume logic, content-range header math, and progress tracking are all correctly handled.
  • realtime.ts: Guards the WebSocketCtor call so the custom Origin header is only passed on native platforms, where the third argument is supported; web gets a bare two-argument constructor call.
  • docs/examples/avatars/get-screenshot.md: Switches the example timezone enum value to Timezone.AfricaAbidjan.

Confidence Score: 5/5

Safe to merge; the concurrent upload and web WebSocket changes are functionally correct and well-scoped.

The concurrent worker pool logic is sound — first-chunk sequencing, resume offset, content-range headers, and progress accounting are all correct. The realtime web-platform fix is a straightforward guard with no behavioural change on native. No data-loss or correctness defects are introduced by the changed code paths.

No files require special attention beyond what existing review threads have already flagged.

Important Files Changed

Filename Overview
src/services/storage.ts Replaces sequential chunk loop with a concurrent worker pool (up to 8 parallel uploads). Logic for first-chunk upload, resume offset, content-range headers, and progress tracking is correct; a minor concern exists around zombie workers after failure.
src/services/realtime.ts Adds a platform guard so the browser WebSocket constructor is called without the custom-headers third argument on web, fixing an incompatibility with the browser WebSocket API.
docs/examples/avatars/get-screenshot.md Updates the example timezone enum value from AmericaNewYork to AfricaAbidjan to reflect the new SDK defaults.

Reviews (2): Last reviewed commit: "chore: bump SDK version to 0.31.0" | Re-trigger Greptile

Comment thread src/services/storage.ts
Comment thread src/services/storage.ts
@TorstenDittmann TorstenDittmann merged commit ae369e8 into main May 21, 2026
3 checks passed
@TorstenDittmann TorstenDittmann deleted the concurrent-chunk-uploads-version-bump-1-9-x branch May 21, 2026 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants