Skip to content

feat: support concurrent chunk uploads#106

Merged
TorstenDittmann merged 3 commits into
mainfrom
concurrent-chunk-uploads-1-9-x-minimal
May 21, 2026
Merged

feat: support concurrent chunk uploads#106
TorstenDittmann merged 3 commits into
mainfrom
concurrent-chunk-uploads-1-9-x-minimal

Conversation

@TorstenDittmann
Copy link
Copy Markdown
Contributor

This PR updates the SDK to support concurrent chunk uploads.

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 21, 2026

Greptile Summary

This PR bumps the SDK to version 0.31.0 and adds a ping() helper method to the Client class, along with a ws security override and a refined glob override in package.json. The storage.ts chunk-upload path is unchanged; the sequential while-loop upload loop remains as-is, so the "concurrent chunk uploads" support appears to be a server-side capability that this SDK version enables rather than a client-side concurrency change.

  • Client.ping() is added with Promise<unknown> as its return type, which correctly reflects that call() may return a parsed JSON object, a { message: string } wrapper for plain-text responses, or throw an AppwriteException — consistent with every other call site in the SDK.
  • package.json overrides are updated: glob is restructured from a flat version string to a nested object using the valid . self-reference to pin both glob itself and its brace-expansion dependency; ws is added to force a patched version.

Confidence Score: 5/5

Safe to merge — the only behavioral change is a new ping() method and dependency overrides; no existing upload or auth paths are touched.

The diff is small and isolated: a new ping() method that delegates to the existing call() helper with an appropriate Promise<unknown> return type, a version bump, and dependency pin updates. No existing logic was modified, no edge cases were introduced, and the package override changes follow documented npm syntax.

No files require special attention.

Important Files Changed

Filename Overview
src/client.ts Version header bumped to 0.31.0; new ping() method added with Promise<unknown> return type, consistent with how call() wraps non-JSON responses as { message: string }
package.json Version bumped to 0.31.0; glob override restructured to nested format (using valid . self-reference syntax) to also pin brace-expansion; ws security override added
package-lock.json Lockfile regenerated to reflect updated overrides and minor transitive dependency version bumps

Reviews (3): Last reviewed commit: "feat: support concurrent chunk uploads" | Re-trigger Greptile

Comment thread src/client.ts Outdated
@TorstenDittmann TorstenDittmann merged commit 884afab into main May 21, 2026
1 check passed
@TorstenDittmann TorstenDittmann deleted the concurrent-chunk-uploads-1-9-x-minimal branch May 21, 2026 17:51
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