Conversation
Greptile SummaryThis PR adds a Key highlights:
Confidence Score: 5/5This PR is safe to merge; all remaining findings are P2 style suggestions. The only open finding is a P2 suggestion to enforce the documented 5,000-key limit client-side. All previously flagged P1 concerns (header constant usage, credentialProvider) have been resolved. The implementation is consistent with the existing codebase patterns and well-tested. No files require special attention. Important Files Changed
Reviews (2): Last reviewed commit: "feat(storage): add Bundle API support fo..." | Re-trigger Greptile |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 3 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 0ab6d53. Configure here.
…download
Add bundle() function that POSTs to /{bucket}?bundle with SigV4
signing and returns a streaming ReadableStream for tar consumption.
Includes unit tests and header constants.
Assisted-by: Claude Opus 4.6 via Claude Code
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Yevgeniy Firsov <firsov@tigrisdata.com>
|
@greptileai review and summarize |
|
🎉 This PR is included in version 2.16.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 2.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 2.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 2.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |

Summary
bundle()function for streaming multi-object tar download viaPOST /{bucket}?bundleReadableStream<Uint8Array>TigrisHeadersenumgenerateSignatureHeadersfrom shared for reuseTest plan
npm run testpasses (84 tests)npm run buildpasses🤖 Generated with Claude Code
Note
Medium Risk
Introduces a new storage client request path that builds auth headers (SigV4 vs session token) and streams response bodies, so mistakes could break authentication or large-download behavior. Scope is contained to a new API plus header/export wiring and unit tests.
Overview
Adds a new
bundle()storage client that POSTs to/{bucket}?bundlewith a JSON list of object keys and returns a streaming tar (optionally compressed) response, with configurablecompressionandonErrorrequest headers.Updates shared plumbing to support this: adds bundle-related
TigrisHeadersconstants, re-exportsgenerateSignatureHeadersfor reuse, and exposesbundle/types frompackages/storageserver entrypoint. Includes unit tests covering validation, request construction, auth modes (SigV4 vs session token/namespace), and HTTP/body error handling.Reviewed by Cursor Bugbot for commit 9d77718. Bugbot is set up for automated code reviews on this repo. Configure here.