zlib: avoid waiting for paused ZIP iterators

[trivikr]

Fixes: #65277

Track file-backed contentIterator() reads only while I/O is active. This lets ZipFile.close() finish when an iterator is paused after yielding a chunk, while still waiting for reads in flight.

---

Assisted-by: codex:gpt-5.6-sol

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.32%. Comparing base (91a99c5) to head (90480cd).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #65278      +/-   ##
==========================================
- Coverage   90.32%   90.32%   -0.01%     
==========================================
  Files         751      751              
  Lines      250000   250007       +7     
  Branches    47231    47231              
==========================================
+ Hits       225816   225821       +5     
+ Misses      15566    15565       -1     
- Partials     8618     8621       +3     

Files with missing lines	Coverage Δ
lib/internal/zip/entry.js	98.13% <100.00%> (+0.01%)	⬆️

... and 26 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.

[nodejs-github-bot]

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

[Copilot]

Pull request overview

Fixes a node:zlib ZipFile lifecycle bug where ZipFile.close() could hang if a file-backed ZipEntry.contentIterator() was paused after yielding a chunk, by only counting active I/O as “in-flight” work that close() must wait for.

Changes:

• Update file-backed ZIP entry streaming (ZipEntry.#rawChunks()) to decrement the shared handle’s reads counter before each yield, and re-increment only when resuming to perform the next read.
• Add a regression test covering close() resolving while a contentIterator() is paused, and ensuring further iteration fails with ERR_INVALID_STATE.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
test/parallel/test-zlib-zip-file-lifecycle.js	Adds regression coverage for ZipFile.close() behavior with a paused contentIterator().
lib/internal/zip/entry.js	Refines in-flight read tracking for file-backed iterator reads to avoid counting time spent paused at yield.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.