Conversation
Greptile SummaryAdds Confidence Score: 5/5Safe to merge; the one remaining finding is a minor best-practice suggestion that doesn't affect correctness. All P0/P1 concerns from prior review rounds appear addressed (dedicated transport replacing http.DefaultClient, SDK pipeline helpers removed). The only new finding is a missing enforcement of the documented 5,000-key ceiling, which is P2 — the server will reject oversized requests anyway. bundle.go — missing guard for the documented 5,000-key limit. Important Files Changed
Reviews (2): Last reviewed commit: "feat: add Bundle API support for streami..." | Re-trigger Greptile |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 7fe70f2. Configure here.
Add BundleObjects method that POSTs to /{bucket}?bundle with SigV4
signing and returns a streaming io.ReadCloser for tar consumption.
Includes WithBundleCompression/WithBundleOnError header helpers,
unit tests, and godoc example.
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 0.6.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |

Summary
BundleObjectsmethod for streaming multi-object tar download viaPOST /{bucket}?bundleio.ReadCloserfor streaming tar consumptionWithBundleCompression/WithBundleOnErrorheader helpers intigrisheaderstar.NewReaderiterationTest plan
go test ./...passes🤖 Generated with Claude Code
Note
Medium Risk
Adds a new SigV4-signed HTTP streaming path (
POST /{bucket}?bundle) outside the AWS SDK, which could affect auth/signing correctness and introduces long-lived streaming connections. Surface area is limited to a new method plus tests and an example.Overview
Adds
Client.BundleObjects, a Tigris-specific API to fetch up to thousands of object keys in one request viaPOST /{bucket}?bundle, returning a streaming tario.ReadCloserwith configurable compression and missing-object behavior viaX-Tigris-Bundle-*headers.Includes unit tests for input validation, request/header/body construction (including SigV4 auth header), and HTTP error handling, plus a new godoc example demonstrating iterating the returned tar stream. Updates
go.modto makegithub.com/joho/godotenva direct dependency.Reviewed by Cursor Bugbot for commit 1a89e45. Bugbot is set up for automated code reviews on this repo. Configure here.