Skip to content

Allow skipping unread ZIP entries after size rejection - #62

Merged
andrew merged 2 commits into
mainfrom
feat/skip-unread-zip
Oct 21, 2026
Merged

andrew merged 2 commits into
mainfrom
feat/skip-unread-zip

Conversation

@andrew

@andrew andrew commented Oct 21, 2026

Copy link
Copy Markdown
Contributor

An oversized ZIP entry currently prevents callers from reaching smaller entries that fit the remaining expansion budget. Add Stream.Skip() to abandon an unread ZIP body without decompressing or validating it, including after Next rejects its size.

Next returns rejected entry metadata alongside limit errors so callers can report the affected path. Skipped entries still count toward the entry limit, while their unread bodies do not consume expanded bytes. Previously read data remains charged. Partial reads, cancellation, input limits and entry-count limits cannot be bypassed; TAR-based formats and conda remain unsupported by Skip.

Copilot AI balanced review requested due to automatic review settings October 21, 2026 10:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

@andrew
andrew merged commit 6526bdf into main Oct 21, 2026
6 checks passed
@andrew
andrew deleted the feat/skip-unread-zip branch October 21, 2026 10: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