2025-01-07, Version 23.6.0 (Current) #56450
Open
+3,739
−1,311
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
PR-URL: #56276 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Ruben Bridgewater <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]>
V8 should be built with -fvisibility=hidden, otherwise the resulting binary would contain unnecessary symbols. In particular, on macOS, this leads to 5000+ weak symbols resolved at runtime, leading to a startup regression. On macOS this also reduces the binary size about ~10MB. It's only enabled on macOS in this patch as gcc can time out or run out of memory on some machines in the CI with -fvisibility=hidden. PR-URL: #56275 Fixes: nodejs/performance#180 Reviewed-By: Juan José Arboleda <[email protected]> Reviewed-By: Daniel Lemire <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Marco Ippolito <[email protected]>
PR-URL: #56283 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Jacob Smith <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
PR-URL: #56295 Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Xuguang Mei <[email protected]>
This reverts commit bdb6d12. PR-URL: #56282 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Jason Zhang <[email protected]> Reviewed-By: Jake Yuesong Li <[email protected]>
PR-URL: #56282 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Jason Zhang <[email protected]> Reviewed-By: Jake Yuesong Li <[email protected]>
Refs: #54827 Refs: #51560 PR-URL: #56050 Reviewed-By: Antoine du Hamel <[email protected]>
When the source map data are invalid json strings, skip construct `SourceMap` on it. Additionally, suppress exceptions on source map lookups and fix test runners crash on invalid source maps. PR-URL: #56299 Refs: #56296 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Xuguang Mei <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Chemi Atlow <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
Only invalidates source map lookup cache when a new source map is found. This improves when user codes interleave with builtin functions, like `array.map`. PR-URL: #56299 Refs: #56296 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Xuguang Mei <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Chemi Atlow <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
This commit stabilizes test isolation configuration in the test runner. PR-URL: #56298 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Pietro Marchini <[email protected]> Reviewed-By: Chemi Atlow <[email protected]> Reviewed-By: Jake Yuesong Li <[email protected]>
The issue is the same as other tests that time out. Refs: #54918 Refs: #54802 PR-URL: #56048 Reviewed-By: Gerhard Stöbich <[email protected]>
PR-URL: #56067 Fixes: #56014 Refs: whatwg/streams#1333 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Jake Yuesong Li <[email protected]>
PR-URL: #55844 Fixes: #40541 Fixes: #55821 Reviewed-By: Guy Bedford <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Jacob Smith <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
PR-URL: #56162 Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Gabriel Schulhof <[email protected]>
We are in the process of updating macOS to version 13 in the Jenkins CI, but unfortunately this is taking longer than expected. Add it to the GitHub actions test matrix so that we have some coverage. Refs: nodejs/build#3686 PR-URL: #56307 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Joyee Cheung <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
`btoa` only supports latin-1 charset and produces invalid source mapping urls. PR-URL: #56315 Refs: #56296 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
PR-URL: #56317 Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]>
PR-URL: #56318 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]> Reviewed-By: Ulises Gascón <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
PR-URL: #56300 Reviewed-By: Juan José Arboleda <[email protected]> Reviewed-By: Xuguang Mei <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
PR-URL: #56325 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: LiviaMedeiros <[email protected]>
PR-URL: #54657 Refs: #54645 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]>
PR-URL: #56332 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
PR-URL: #56334 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]>
Migrate the eval tests in the `test/message` folder from Python to JS. PR-URL: #50482 Reviewed-By: Juan José Arboleda <[email protected]>
co-authored-by: jazelly <[email protected]> PR-URL: #56278 Reviewed-By: Matteo Collina <[email protected]>
It was excluded as it was failing intermittently. Likely that s390 was just so fast times were rounded down to 0. Increase the spin time on s390x only. Signed-off-by: Michael Dawson <[email protected]> PR-URL: #56228 Refs: #41286 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Marco Ippolito <[email protected]>
PR-URL: #56285 Refs: nodejs/typescript#17 Reviewed-By: Pietro Marchini <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]>
PR-URL: #56341 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Michaël Zasso <[email protected]>
PR-URL: #55898 Fixes: #55800 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
PR-URL: #55811 Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Jason Zhang <[email protected]> Reviewed-By: Deokjin Kim <[email protected]>
PR-URL: #56390 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Jacob Smith <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
PR-URL: #56382 Reviewed-By: Jordan Harband <[email protected]> Reviewed-By: Geoffrey Booth <[email protected]>
Notable runtime changes to existing APIs: - returning `node_api_cannot_run_js` instead of `napi_pending_exception`. - allow creating references to objects, functions, and symbols. PR-URL: #55676 Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: James M Snell <[email protected]>
The `process.ref(...)` and `process.unref(...)` methods are intended to replace the use of `ref()` and `unref()` methods defined directly on individual API objects. The existing `ref()` and `unref()` methods will be marked as legacy and won't be removed but new APIs should use `process.ref()` and `process.unref()` instead. Refs: #53266 PR-URL: #56400 Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Chemi Atlow <[email protected]>
When piping a `ReadableStream` created from an `Iterable` into a `WritableStream`, the sequence of objects in the `Iterable` must consist of either `Buffer`s, `TypedArray`s, or `DataView`s. Re: #56297 PR-URL: #56415 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Chemi Atlow <[email protected]> Reviewed-By: Ulises Gascón <[email protected]>
V8 introduced `v8::LocalVector` somewhat recently as an alternative to using `std::vector<v8::Local<T>>` to help ensure that Local handles are handled correctly. This updates most (but not all) of our uses of `std::vector<v8::Local<T>>` to use `v8::LocalVector<T>` with a few other minor cleanups encountered along the way. PR-URL: #56417 Reviewed-By: Juan José Arboleda <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Chengzhong Wu <[email protected]>
PR-URL: #56388 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: James M Snell <[email protected]>
PR-URL: #56394 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Paolo Insogna <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
nodejs-github-bot
added a commit
that referenced
this pull request
Jan 3, 2025
Notable changes: lib: * (SEMVER-MINOR) add typescript support to STDIN eval (Marco Ippolito) #56359 module: * (SEMVER-MINOR) unflag --experimental-strip-types (Marco Ippolito) #56350 process: * (SEMVER-MINOR) add process.ref() and process.unref() methods (James M Snell) #56400 worker: * (SEMVER-MINOR) add eval ts input (Marco Ippolito) #56394 PR-URL: #56450
github-actions
bot
added
release
|
Review requested:
|
Notable changes: lib: * (SEMVER-MINOR) add typescript support to STDIN eval (Marco Ippolito) #56359 module: * (SEMVER-MINOR) unflag --experimental-strip-types (Marco Ippolito) #56350 process: * (SEMVER-MINOR) add process.ref() and process.unref() methods (James M Snell) #56400 worker: * (SEMVER-MINOR) add eval ts input (Marco Ippolito) #56394 PR-URL: #56450
marco-ippolito
force-pushed
the
v23.6.0-proposal
branch
from
3062e40 to
9fa6aeb
Compare
|
It will be magnificent ✨ to see 👀 #56274 in this release 🙄 |
I see its a semver major change, so I don't know if it's possible to include it. @nodejs/releasers wdyt |
As Richard said in #56274 (comment), Node.js follows a semver-compatible release process, and therefor semver-major PRs are only included in the next major release. Now there seems to be discussions in the PR thread on whether that update should be considered semver-major, so if the discussion settles on "it is not semver-major", it could be included in the next release, but it wouldn't make sense to delay this release until a consensus is found IMO. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## v23.x #56450 +/- ##
==========================================
- Coverage 88.55% 88.55% -0.01%
==========================================
Files 657 657
Lines 190225 190741 +516
Branches 36545 36622 +77
==========================================
+ Hits 168458 168909 +451
- Misses 14954 15003 +49
- Partials 6813 6829 +16
|
github-actions
bot
removed
the
request-ci
RafaelGSS
approved these changes
Jan 3, 2025
|
I see a babel failure in the CITGM (cc @nicolo-ribaudo): It seems the error changed |
|
Will it be backported down to v22? |
What commit? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
2025-01-07, Version 23.6.0 (Current), @marco-ippolito
Notable Changes
Unflagging --experimental-strip-types
This release enables the flag
--experimental-strip-typesby default.Node.js will be able to execute TypeScript files without additional configuration:
There are some limitations in the supported syntax documented at https://nodejs.org/api/typescript.html#type-stripping
This feature is experimental and is subject to change.
Contributed by Marco Ippolito in #56350
Other Notable Changes
c1023284c3] - (SEMVER-MINOR) lib: add typescript support to STDIN eval (Marco Ippolito) #563598dc39e5e2e] - (SEMVER-MINOR) process: add process.ref() and process.unref() methods (James M Snell) #564008b20cc212b] - (SEMVER-MINOR) worker: add eval ts input (Marco Ippolito) #56394Commits
7b4d288116] - assert: make partialDeepStrictEqual throw when comparing [0] with [-0] (Giovanni) #562370ec2ed0a0b] - build: fix GN build for ngtcp2 (Cheng) #56300ab3e64630b] - build: test macos-13 on GitHub actions (Michaël Zasso) #5630746fb69daca] - build: build v8 with -fvisibility=hidden on macOS (Joyee Cheung) #562759d4930b993] - deps: update simdutf to 5.7.2 (Node.js GitHub Bot) #563886afe36397e] - deps: update amaro to 0.2.1 (Node.js GitHub Bot) #56390195990a0ee] - deps: update googletest to 7d76a23 (Node.js GitHub Bot) #56387b9c0852fc6] - deps: update googletest to e54519b (Node.js GitHub Bot) #56370eaefd90128] - deps: update ngtcp2 to 1.10.0 (Node.js GitHub Bot) #5633406de0c65cf] - deps: update simdutf to 5.7.0 (Node.js GitHub Bot) #5633203df76cdec] - doc: add example for piping ReadableStream (Gabriel Schulhof) #5641538ce249b07] - doc: expand description ofparseArg'sdefault(Kevin Gibbons) #54431ecc718cef2] - doc: use<ul>instead of<ol>inSECURITY.md(Antoine du Hamel) #563463db4809130] - doc: clarify that WASM is trusted (Matteo Collina) #56345384ccbacd5] - doc: update macOS and Xcode versions for releases (Michaël Zasso) #563373943986e88] - doc: fix thecrc32documentation (Kevin Toshihiro Uehara) #55898710b8fc6ed] - doc: add entry to changelog about SQLite Session Extension (Bart Louwers) #563184c978b4d77] - doc: fix links inmodule.md(Antoine du Hamel) #56283cdb631efe7] - esm: add experimental support for addon modules (Chengzhong Wu) #55844db83d2f0ee] - Revert "events: add hasEventListener util for validate" (origranot) #56282c2baae84ce] - lib: refactor execution.js (Marco Ippolito) #56358c1023284c3] - (SEMVER-MINOR) lib: add typescript support to STDIN eval (Marco Ippolito) #56359e4b795ec4a] - lib: optimizeprepareStackTraceon builtin frames (Chengzhong Wu) #56299d1b009b623] - lib: suppress source map lookup exceptions (Chengzhong Wu) #56299c2837f0805] - meta: move one or more collaborators to emeritus (Node.js GitHub Bot) #5634272336233f2] - meta: move MoLow to TSC regular member (Moshe Atlow) #562764f77920a9d] - module: fix async resolution error within the syncfindPackageJSON(Jacob Smith) #56382e5ba216501] - (SEMVER-MINOR) module: unflag --experimental-strip-types (Marco Ippolito) #56350959f133a22] - module: support eval with ts syntax detection (Marco Ippolito) #56285717cfa4fac] - module: use buffer.toString base64 (Chengzhong Wu) #56315c2f4d8d688] - node-api: define version 10 (Gabriel Schulhof) #55676417a8ebdec] - node-api: remove deprecated attribute from napi_module_register (Vladimir Morozov) #561628dc39e5e2e] - (SEMVER-MINOR) process: add process.ref() and process.unref() methods (James M Snell) #56400d194f1ab5f] - sqlite: pass conflict type to conflict resolution handler (Bart Louwers) #5635229f5d70452] - src: use v8::LocalVector consistently with other minor cleanups (James M Snell) #564172a5543b78e] - src: use starts_with in fs_permission.cc (ishabi) #558113a3f5c9a64] - stream: validate undefined sizeAlgorithm in WritableStream (Jason Zhang) #560676e6f6b071a] - test: add ts eval snapshots (Marco Ippolito) #563588a87e39052] - test: remove empty lines from snapshots (Marco Ippolito) #56358510649f617] - test: use unusual chars in the path to ensure our tests are robust (Antoine du Hamel) #4840954f6d681a0] - test: remove flaky designation (Luigi Pinca) #5636920ace0bb01] - test: remove test-worker-arraybuffer-zerofill flaky designation (Luigi Pinca) #56364b757e40525] - test: remove test-net-write-fully-async-hex-string flaky designation (Luigi Pinca) #5636564556baddc] - test: improve abort signal dropping test (Edy Silva) #56339accbdad329] - test: enable ts test on win arm64 (Marco Ippolito) #563494188ee00d1] - test: deflake test-watch-file-shared-dependency (Luigi Pinca) #56344079cee0609] - test: skiptest-sqlite-extensionswhen SQLite is not built by us (Antoine du Hamel) #5634196a38044ee] - test: increase spin for eventloop test on s390 (Michael Dawson) #56228c062ffc242] - test: add coverage for pipeline (jakecastelli) #56278d4404f0d0e] - test: migrate message eval tests from Python to JS (Yiyun Lei) #504829369942745] - test: check typescript loader (Marco Ippolito) #546574930244484] - test: remove async-hooks/test-writewrap flaky designation (Luigi Pinca) #560487819bfec69] - test: deflake test-esm-loader-hooks-inspect-brk (Luigi Pinca) #56050e9762bf005] - test: add test case for listeners (origranot) #56282c1627e9d19] - test: maketest-permission-sqlite-load-extensionmore robust (Antoine du Hamel) #5629597d854e1d5] - test_runner,cli: mark test isolation as stable (Colin Ihrig) #56298a4f336fdd4] - tools: fixrequire-common-firstlint rule from subfolder (Antoine du Hamel) #56325dc3dafcb50] - tools: add release line label when opening release proposal (Antoine du Hamel) #563172a5ac932ac] - url: use resolved path to convert UNC paths to URL (Antoine du Hamel) #563028b20cc212b] - (SEMVER-MINOR) worker: add eval ts input (Marco Ippolito) #56394