build,win: fix dll build

[StefanStojanovic]

Fixes a linker error when building Node.js with ClangCL due to missing explicit template instantiation for ThreadsafeCopyOnWrite<std::map<std::string, UnionBytes>>.

Fixes: #58208

[nodejs-github-bot]

Review requested:

• @nodejs/startup

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

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

Additional details and impacted files

@@            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              

Files with missing lines	Coverage Δ
src/node_builtins.cc	79.17% <ø> (-0.25%)	⬇️

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

[aduh95]

Note that linter is failing

/cc @nodejs/platform-windows

[github-actions]

Failed to start CI

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

https://github.com/nodejs/node/actions/runs/14928236826

[joyeecheung]

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

node/src/node_builtins.h

Lines 130 to 132 in e74e065

	[[nodiscard]] auto GetBuiltinIds() const {
	return std::views::keys(*source_.read());
	}

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