Skip to content

Conversation

@clydin
Copy link
Member

@clydin clydin commented Dec 3, 2025

This commit replaces the tar package with tar-stream for file extraction in the e2e test utilities.

The primary advantage of this change is a significant reduction in dependency size. The tar package is approximately 2MB on disk, whereas tar-stream is only ~235KB. This improves installation speed and reduces the overall disk footprint of the project.

The implementation in extractFile has been updated to use the tar-stream event-based API and now includes zlib.createGunzip() to correctly handle compressed tarballs.

@clydin clydin added the target: minor This PR is targeted for the next minor release label Dec 3, 2025
This commit replaces the `tar` package with `tar-stream` for file extraction in the e2e test utilities.

The primary advantage of this change is a significant reduction in dependency size. The `tar` package is approximately 2MB on disk, whereas `tar-stream` is only ~235KB. This improves installation speed and reduces the overall disk footprint of the project.

The implementation in `extractFile` has been updated to use the `tar-stream` event-based API and now includes `zlib.createGunzip()` to correctly handle compressed tarballs.
@clydin clydin force-pushed the test/e2e-tar-stream branch from b39dd68 to 640124a Compare December 3, 2025 16:02
@clydin clydin added the action: review The PR is still awaiting reviews from at least one requested reviewer label Dec 3, 2025
@clydin clydin added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Dec 3, 2025
@clydin clydin merged commit 816b3d2 into angular:main Dec 3, 2025
36 checks passed
@clydin
Copy link
Member Author

clydin commented Dec 3, 2025

This PR was merged into the repository. The changes were merged into the following branches:

@clydin clydin deleted the test/e2e-tar-stream branch December 3, 2025 16:49
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jan 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants