deps: update V8 to 13.6 #58070

targos · 2025-04-29T06:11:23Z

Same as #57753, without compatibility patches for MSVC and armv7.

Notable changes:

nodejs-github-bot · 2025-04-29T06:11:28Z

targos · 2025-04-29T08:06:30Z

We still have this consistent macOS test failure on GitHub actions:

 Path: sequential/test-error-serdes
Error: --- stderr ---
node:internal/streams/readable:90
const FastBuffer = Buffer[SymbolSpecies];
                         ^

RangeError: Maximum call stack size exceeded
    at node:internal/streams/readable:90:26
    at BuiltinModule.compileForInternalLoader (node:internal/bootstrap/realm:398:7)
    at requireBuiltin (node:internal/bootstrap/realm:429:14)
    at node:internal/streams/duplex:39:18
    at BuiltinModule.compileForInternalLoader (node:internal/bootstrap/realm:398:7)
    at requireBuiltin (node:internal/bootstrap/realm:429:14)
    at node:internal/streams/pipeline:16:16
    at BuiltinModule.compileForInternalLoader (node:internal/bootstrap/realm:398:7)
    at requireBuiltin (node:internal/bootstrap/realm:429:14)
    at node:internal/streams/compose:7:22

Node.js v24.0.0-pre
Command: out/Release/node --expose-internals --stack-size=64 --test-reporter=./test/common/test-error-reporter.js --test-reporter-destination=stdout /Users/runner/work/node/node/node/test/sequential/test-error-serdes.js

codecov · 2025-04-29T09:24:54Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.63%. Comparing base (102d8cf) to head (fb09a88).
Report is 45 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #58070      +/-   ##
==========================================
- Coverage   90.20%   89.63%   -0.57%     
==========================================
  Files         630      630              
  Lines      186446   186470      +24     
  Branches    36622    36306     -316     
==========================================
- Hits       168189   167149    -1040     
- Misses      11051    12088    +1037     
- Partials     7206     7233      +27

see 118 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.

legendecas · 2025-04-29T11:06:38Z

@targos I think we can safely increase the --stack-size flag in the test from 64 to some value like 128 or 256 to quickly remediate the failure. However, I did not find it is necessary to rely on the stack size in this test at all: #58075.

nodejs-github-bot · 2025-04-29T15:03:20Z

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

nodejs-github-bot · 2025-04-29T15:03:29Z

V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=benchmark-ubuntu2204-intel-64,v8test=v8test/6520/

nodejs-github-bot · 2025-04-29T15:05:05Z

V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=rhel8-ppc64le,v8test=v8test/6520/

nodejs-github-bot · 2025-04-29T15:06:40Z

V8 CI: https://ci.nodejs.org/job/node-test-commit-v8-linux/nodes=rhel8-s390x,v8test=v8test/6520/

targos · 2025-04-29T20:13:47Z

Now there are failing pummel/test-heapdump-... tests 😞

https://ci.nodejs.org/job/node-test-commit-linux-containered/50344/

targos · 2025-04-29T20:15:11Z

And timeouts too while building addons on Windows

joyeecheung · 2025-04-29T20:24:29Z

It passed in https://ci.nodejs.org/job/node-test-pull-request/66495/ and https://ci.nodejs.org/job/node-test-commit/79461/ so could come from the diff?

targos · 2025-04-29T20:26:54Z

It's probably me being too optimistic when I removed #47450 (df034a3). I'll add it back when I rebase to fix the node_buffer.cc conflict.

joyeecheung · 2025-04-30T01:16:57Z

If it's caused by enabling concurrent sparkplug it's likely #54918 (which might be fixed by #58047 but I will need more time to reason it out). For unblocking 24 it's better to keep it disabled for now.

nodejs-github-bot · 2025-04-30T05:39:51Z

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

targos · 2025-04-30T07:45:21Z

CI is looking better.

targos · 2025-04-30T09:39:08Z

There are still issues with Windows tests. /cc @StefanStojanovic

aduh95 · 2025-04-30T11:04:59Z

pummel/test-heapdump-zlib is quite flaky on GHA test-linux (failed 7/20 times)

nodejs-github-bot · 2025-04-30T14:37:00Z

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

targos · 2025-04-30T14:38:20Z

Another pummel/test-heapdump-shadow-realm failed in one of the containered jobs https://ci.nodejs.org/job/node-test-commit-linux-containered/50349/

GCC emits warnings because of the trailing backslashes. PR-URL: nodejs/node#58070 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

PR-URL: nodejs/node#58070 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

nodejs/node#58070

GCC emits warnings because of the trailing backslashes. PR-URL: nodejs/node#58070 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

PR-URL: nodejs/node#58070 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

nodejs/node#58070

GCC emits warnings because of the trailing backslashes. PR-URL: nodejs/node#58070 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

PR-URL: nodejs/node#58070 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

GCC emits warnings because of the trailing backslashes. PR-URL: nodejs#58070 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

PR-URL: nodejs#58070 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

GCC emits warnings because of the trailing backslashes. PR-URL: nodejs/node#58070 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

PR-URL: nodejs/node#58070 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>

nodejs/node#58070

* chore: upgrade Node.js to v24.10.0 * chore: fixup crypto patch * chore: fixup crypto test patch * src: prepare for v8 sandboxing nodejs/node#58376 * esm: fix module.exports export on CJS modules nodejs/node#57366 * chore: fixup lazyload fs patch * esm: Source Phase Imports for WebAssembly nodejs/node#56919 * module: remove --experimental-default-type nodejs/node#56092 * lib,src: refactor assert to load error source from memory nodejs/node#59751 * src: add source location to v8::TaskRunner nodejs/node#54077 * src: remove dependency on wrapper-descriptor-based CppHeap nodejs/node#54077 * src: do not use soon-to-be-deprecated V8 API nodejs/node#53174 * src: stop using deprecated fields of v8::FastApiCallbackOptions nodejs/node#54077 * test: update v8-stats test for V8 12.6 nodejs/node#54077 * esm: unflag --experimental-wasm-modules nodejs/node#57038 * test: adapt assert tests to stack trace changes nodejs/node#58070 * src,test: unregister the isolate after disposal and before freeing nodejs/node#58070 * src: use cppgc to manage ContextifyContext nodejs/node#56522 * src: replace uses of FastApiTypedArray nodejs/node#58070 * module: integrate TypeScript into compile cache nodejs/node#56629 * deps: update ada to 3.2.7 nodejs/node#59336 * src: make minor cleanups in encoding_binding.cc nodejs/node#57448 * src: switch from `Get/SetPrototype` to `Get/SetPrototypeV2` nodejs/node#55453 * src: use non-deprecated Get/SetPrototype methods nodejs/node#59671 * src: simplify string_bytes with views nodejs/node#54876 * src: improve utf8 string generation performance nodejs/node#54873 * src: use non-deprecated Utf8LengthV2() method nodejs/node#58070 * src: use non-deprecated WriteUtf8V2() method nodejs/node#58070 * src: refactor WriteUCS2 and remove flags argument nodejs/node#58163 * src: use String::WriteV2() in TwoByteValue nodejs/node#58164 * node-api: use WriteV2 in napi_get_value_string_utf16 nodejs/node#58165 * node-api: use WriteOneByteV2 in napi_get_value_string_latin1 nodejs/node#58325 * src: migrate WriteOneByte to WriteOneByteV2 nodejs/node#59634 * fs: introduce dirent\.parentPath nodejs/node#50976 * src: avoid copy by using std::views::keys nodejs/node#56080 * chore: fixup patch indices * fix: errant use of context->GetIsolate() * fix: tweak BoringSSL compat patch for new changes * fix: add back missing isolate dtor declaration * fixup! esm: fix module.exports export on CJS modules * cli: remove --no-experimental-fetch flag https://github.com/nodejs/node/pull/52611/files * esm: Source Phase Imports for WebAssembly nodejs/node#56919 * fixup! src: prepare for v8 sandboxing * chore: bump @types/node to v24 * chore: fix const assignment in crypto test * fix: sandbox pointer patch issues * chore: rework source phase import patch * src: add percentage support to --max-old-space-size nodejs/node#59082 * chore: fixup crypto tests * chore: HostImportModuleWithPhaseDynamically todo * fix: cjs esm failures * fix: v8::Object::Wrappable issues - v8/node@b72a615 - v8/node@490bac2 - v8/node@4896a0d * chore: remove deleted specs * src: use v8::ExternalMemoryAccounter nodejs/node#58070 * fs: port SonicBoom module to fs module as FastUtf8Stream nodejs/node#58897 * chore: tweak sandboxed pr patch * test: disable parallel/test-os-checked-function * test: use WHATWG URL instead of url.parse * fix: OPENSSL_secure_zalloc doesn't work in BoringSSL * chore: fix accidental extra line * 7017517: [defer-import-eval] Parse import defer syntax https://chromium-review.googlesource.com/c/v8/v8/+/7017517

nodejs/node#58070 Co-authored-by: Shelley Vohr <[email protected]>

targos force-pushed the v8-136-min branch from b37633b to 10bf2eb Compare April 29, 2025 06:39

aduh95 added the semver-major PRs that contain breaking changes and should be released in the next major version. label Apr 29, 2025

targos mentioned this pull request Apr 29, 2025

[help wanted] deps: update V8 to 13.6 #57753

Closed

legendecas mentioned this pull request Apr 29, 2025

lib: avoid StackOverflow on serializeError #58075

Merged

targos mentioned this pull request Apr 29, 2025

2025-05-06, Version 24.0.0 (Current) #57609

Merged

targos force-pushed the v8-136-min branch from 10bf2eb to 47ad83e Compare April 30, 2025 05:30

codebytere added a commit to electron/electron that referenced this pull request Oct 27, 2025

src: use v8::ExternalMemoryAccounter

2b086ed

nodejs/node#58070

codebytere added a commit to electron/electron that referenced this pull request Oct 28, 2025

test: adapt assert tests to stack trace changes

17eae68

nodejs/node#58070

codebytere added a commit to electron/electron that referenced this pull request Oct 28, 2025

src,test: unregister the isolate after disposal and before freeing

61dcc6e

nodejs/node#58070

codebytere added a commit to electron/electron that referenced this pull request Oct 28, 2025

src: replace uses of FastApiTypedArray

c31dfd5

nodejs/node#58070

codebytere added a commit to electron/electron that referenced this pull request Oct 28, 2025

src: use non-deprecated Utf8LengthV2() method

0d488d8

nodejs/node#58070

codebytere added a commit to electron/electron that referenced this pull request Oct 28, 2025

src: use non-deprecated WriteUtf8V2() method

374bc8b

nodejs/node#58070

codebytere added a commit to electron/electron that referenced this pull request Oct 28, 2025

src: use v8::ExternalMemoryAccounter

335208f

nodejs/node#58070

codebytere added a commit to electron/electron that referenced this pull request Oct 30, 2025

test: adapt assert tests to stack trace changes

9035346

nodejs/node#58070

codebytere added a commit to electron/electron that referenced this pull request Oct 30, 2025

src,test: unregister the isolate after disposal and before freeing

75b004c

nodejs/node#58070

codebytere added a commit to electron/electron that referenced this pull request Oct 30, 2025

src: replace uses of FastApiTypedArray

4112756

nodejs/node#58070

codebytere added a commit to electron/electron that referenced this pull request Oct 30, 2025

src: use non-deprecated Utf8LengthV2() method

d3892b1

nodejs/node#58070

codebytere added a commit to electron/electron that referenced this pull request Oct 30, 2025

src: use non-deprecated WriteUtf8V2() method

5c6ee4d

nodejs/node#58070

codebytere added a commit to electron/electron that referenced this pull request Oct 30, 2025

src: use v8::ExternalMemoryAccounter

795f523

nodejs/node#58070

trop bot added a commit to electron/electron that referenced this pull request Oct 30, 2025

test: adapt assert tests to stack trace changes

434d481

nodejs/node#58070 Co-authored-by: Shelley Vohr <[email protected]>

trop bot added a commit to electron/electron that referenced this pull request Oct 30, 2025

src,test: unregister the isolate after disposal and before freeing

03ec0b3

nodejs/node#58070 Co-authored-by: Shelley Vohr <[email protected]>

trop bot added a commit to electron/electron that referenced this pull request Oct 30, 2025

src: replace uses of FastApiTypedArray

8e369d5

nodejs/node#58070 Co-authored-by: Shelley Vohr <[email protected]>

trop bot added a commit to electron/electron that referenced this pull request Oct 30, 2025

src: use non-deprecated Utf8LengthV2() method

a2fb0e2

nodejs/node#58070 Co-authored-by: Shelley Vohr <[email protected]>

trop bot added a commit to electron/electron that referenced this pull request Oct 30, 2025

src: use non-deprecated WriteUtf8V2() method

dca0704

nodejs/node#58070 Co-authored-by: Shelley Vohr <[email protected]>

trop bot added a commit to electron/electron that referenced this pull request Oct 30, 2025

src: use v8::ExternalMemoryAccounter

b333364

nodejs/node#58070 Co-authored-by: Shelley Vohr <[email protected]>

Uh oh!

deps: update V8 to 13.6 #58070

deps: update V8 to 13.6 #58070

Conversation

targos commented Apr 29, 2025

Uh oh!

nodejs-github-bot commented Apr 29, 2025

Uh oh!

targos commented Apr 29, 2025

Uh oh!

codecov bot commented Apr 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

legendecas commented Apr 29, 2025

Uh oh!

nodejs-github-bot commented Apr 29, 2025

Uh oh!

nodejs-github-bot commented Apr 29, 2025

Uh oh!

nodejs-github-bot commented Apr 29, 2025

Uh oh!

nodejs-github-bot commented Apr 29, 2025

Uh oh!

targos commented Apr 29, 2025

Uh oh!

targos commented Apr 29, 2025

Uh oh!

joyeecheung commented Apr 29, 2025

Uh oh!

targos commented Apr 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

joyeecheung commented Apr 30, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

nodejs-github-bot commented Apr 30, 2025

Uh oh!

targos commented Apr 30, 2025

Uh oh!

targos commented Apr 30, 2025

Uh oh!

aduh95 commented Apr 30, 2025

Uh oh!

nodejs-github-bot commented Apr 30, 2025

Uh oh!

targos commented Apr 30, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

15 participants

codecov bot commented Apr 29, 2025 •

edited

Loading

targos commented Apr 29, 2025 •

edited

Loading

joyeecheung commented Apr 30, 2025 •

edited

Loading