feat: presign snapshot-version and paginated fork listing#110
Merged
Conversation
Add --snapshot-version (alias --snapshot) to `presign` for presigned GET URLs against a specific bucket snapshot; rejected for non-GET methods. Forward --limit / --page-token through `buckets list --forks-of` so fork listings paginate instead of fetching all forks. Bumps @tigrisdata/storage to ^3.14.0 to support both. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
MantasMiksys
approved these changes
Jun 19, 2026
|
🎉 This PR is included in version 3.2.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
presign --snapshot-version(alias--snapshot): generate presigned GET URLs against a specific bucket snapshot. Accepts a snapshot version string or a UNIX nanosecond timestamp. Rejected for non-GET methods with a clear error.buckets list --forks-ofpagination:--limit/--page-tokenare now forwarded tolistForks(and a pagination hint is printed) instead of being ignored with a warning.@tigrisdata/storageto^3.14.0to support both.Tests
Added to the integration suite (
test/cli.test.ts), in thepresign commandblock:--snapshot-version--snapshotalias for GET--snapshot-versionrejected with--method putThese run via
npm run test:integration(ortest:all) with real Tigris credentials and a rebuiltdist/.🤖 Generated with Claude Code
Note
Low Risk
User-facing CLI additions with validation and integration tests; dependency bump is the main behavioral surface beyond presign and fork pagination.
Overview
Adds
presign --snapshot-version(alias--snapshot) so GET presigned URLs can target a bucket snapshot (version string or nanosecond timestamp). Non-GET methods fail with an explicit error. The flag is documented inspecs.yamland passed through togetPresignedUrl.buckets list --forks-ofnow honors--limitand--page-tokenvialistForks, uses the same paginated output and pagination hint as the default bucket list, and drops the old warning that pagination was ignored.Bumps
@tigrisdata/storageto^3.14.0inpackage.json(lockfile refresh). Integration tests cover snapshot presign GET, the--snapshotalias, and rejecting snapshot with--method put.Reviewed by Cursor Bugbot for commit ab0890e. Bugbot is set up for automated code reviews on this repo. Configure here.