src: simply uint32 to string as it must not fail

[legendecas]

v8::Uint32::ToString does not return an empty local handle. V8 internal NumberToString does not return an empty handle. It should be safe to assert in node::Uint32ToString that this function returns a non empty string handle, and remove the redundant error recovering logics.

Refs: #53959

[targos]

2025-11-25T00:39:41.4286597Z application.cc:(.text+0x0): multiple definition of `node::Uint32ToString(v8::Local<v8::Context>, unsigned int)'; /home/runner/work/node/node/node/out/Release/obj.target/embedtest/src/node_snapshot_stub.o:node_snapshot_stub.cc:(.text+0x0): first defined here 

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.57%. Comparing base (482b191) to head (86e3b6b).
⚠️ Report is 19 commits behind head on main.

Additional details and impacted files

@@ Coverage Diff @@## main #60846 +/- ## ========================================== + Coverage 88.02% 88.57% +0.54%  ========================================== Files 703 703 Lines 208268 208234 -34 Branches 40085 40158 +73 ========================================== + Hits 183337 184446 +1109 + Misses 16878 15810 -1068 + Partials 8053 7978 -75 

Files with missing lines	Coverage Δ
src/node_contextify.cc	82.45% <100.00%> (+0.66%)	⬆️
src/node_webstorage.cc	74.61% <100.00%> (+1.41%)	⬆️
src/util.h	90.09% <100.00%> (+0.37%)	⬆️

... and 103 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/70310/

[nodejs-github-bot]

Landed in 430db0d