Review requested:

• @nodejs/startup

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.18%. Comparing base (e38ce27) to head (78225e7).
Report is 205 commits behind head on main.

@@            Coverage Diff             @@
##             main   #58235      +/-   ##
==========================================
+ Coverage   90.14%   90.18%   +0.04%     
==========================================
  Files         629      629              
  Lines      186634   186660      +26     
  Branches    36616    36658      +42     
==========================================
+ Hits       168236   168347     +111     
+ Misses      11204    11119      -85     
  Partials     7194     7194              

... and 50 files with indirect coverage changes

• ❄️ 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.

Note that linter is failing

/cc @nodejs/platform-windows

   ⚠  No approving reviews found
   ✘  Refusing to run CI on potentially unsafe PR

I think a more correct solution is to avoid inlining this in the class body?

Otherwise, node_builtins.h needs to include node_threadsafe_cow-inl.h to have the implementation of the read method.

Closing this one since the other PR landed.