vfs: add ZipProvider

[pipobscure]

Add a node:vfs provider backed by a node:zlib ZIP archive - a ZipBuffer
held in memory or a ZipFile on disk - that exposes the archive's members
as a virtual filesystem tree. Directories are recognized both explicitly
(a "name/" entry) and implicitly (any entry under "name/"), and a file
opened for writing commits its content as a new archive entry when its
handle is closed. The provider is read-only unless the backing archive
is writable, and offers both asynchronous and synchronous operations.
Available as vfs.ZipProvider.

[codecov]

Codecov Report

❌ Patch coverage is 97.93814% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.07%. Comparing base (4d9cb71) to head (d6766c6).
⚠️ Report is 11 commits behind head on main.

Files with missing lines	Patch %	Lines
lib/internal/vfs/providers/ziparchive.js	97.93%	7 Missing and 5 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #64915      +/-   ##
==========================================
+ Coverage   90.05%   90.07%   +0.01%     
==========================================
  Files         754      755       +1     
  Lines      256308   256899     +591     
  Branches    48464    48684     +220     
==========================================
+ Hits       230817   231399     +582     
- Misses      16593    16605      +12     
+ Partials     8898     8895       -3     

Files with missing lines	Coverage Δ
lib/vfs.js	100.00% <100.00%> (ø)
lib/internal/vfs/providers/ziparchive.js	97.93% <97.93%> (ø)

... and 34 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[pipobscure]

Note: the failing mac-os test seems to be a flakiness issue with that test. I've seen it before and it seems to be entirely unrelated.

[pipobscure]

@mcollina given you are the VFS man, this one is for you.

Note:

Next up will be a commandline option --vfs-mount which allows mounting a VFS mount from the CLI for the app to use. This is useful for example to provide an app with an archive of files it can use, like bundled resources. This can be mounted by either mounting a directory (development) or a bundled archive (deployments)

Following on from that is a --vfs-load flag that will set the entrypoint to a vfs mounted volume. In the full combination it's possible to have bundled node apps (zip archives) that get loaded. --vfs-mount already allows registering custom vfs-providers (via --import or -r). So what the vfs-provider does (signed code checking, etc...) is up to the user. In the simplest case, you just have an app as a zip archive and it works out of the box.

I have all the code already and it's working quite nicely.

[mcollina]

I'm doing significant surgery on the vfs machinery in #63653.

I need to land that before this.

[pipobscure]

I'm doing significant surgery on the vfs machinery in #63653.

I need to land that before this.

Have a look at my branch pipobscure#8 here which does all the module loader work for VFS already. And I've been testing it like crazy.

Though from what I see in your PR you're going way deeper, especially the fact that you have an O(1) identification of vfs based modules is a much better solution. So I'm EXTREMELY happy to hold on with this and then rework my other PRs based on your stuff 😄 (starting work based of your solution)

Note: It looks like #63653 is entirely orthogonal to this PR as that doesn't alter the Provider surface contract at all. But for sanity sake it's still better to hold this until #63653 is landed and rebase this on that then.

[mcollina]

lgtm

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/75379/

[trivikr]

Added blocked label since we're waiting for #63653 to land first.

[mcollina]

lgtm

[nodejs-github-bot]

CI: https://ci.nodejs.org/job/node-test-pull-request/76771/