GeezerVison

Copy link

Copy Markdown

Member

StefanStojanovic approved these changes Oct 20, 2023

View reviewed changes

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

H4ad approved these changes Oct 20, 2023

View reviewed changes

mhdawson approved these changes Oct 20, 2023

View reviewed changes

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

PR-URL: #50296
Refs: #50295
Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>

PR-URL: #50296
Refs: #50295
Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>

PR-URL: #50296
Refs: #50295
Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>

PR-URL: #50296
Refs: #50295
Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>

nodejs/node#50296

* chore: bump node in DEPS to v20.10.0

* chore: update feat_initialize_asar_support.patch

no code changes; patch just needed an update due to nearby upstream changes

Xref: nodejs/node#49986

* chore: update pass_all_globals_through_require.patch

no manual changes; patch applied with fuzz

Xref: nodejs/node#49657

* chore: update refactor_allow_embedder_overriding_of_internal_fs_calls

Xref: nodejs/node#49912

no code changes; patch just needed an update due to nearby upstream changes

* chore: update chore_allow_the_node_entrypoint_to_be_a_builtin_module.patch

Xref: nodejs/node#49986

minor manual changes needed to sync with upstream change

* update fix_expose_the_built-in_electron_module_via_the_esm_loader.patch

Xref: nodejs/node#50096
Xref: nodejs/node#50314
in lib/internal/modules/esm/load.js, update the code that checks for
`format === 'electron'`. I'd like 👀 on this

Xref: nodejs/node#49657
add braces in lib/internal/modules/esm/translators.js to sync with upstream

* fix: lazyload fs in esm loaders to apply asar patches

* nodejs/node#50127
* nodejs/node#50096

* esm: jsdoc for modules code

nodejs/node#49523

* test: set test-cli-node-options as flaky

nodejs/node#50296

* deps: update c-ares to 1.20.1

nodejs/node#50082

* esm: bypass CommonJS loader under --default-type=module

nodejs/node#49986

* deps: update uvwasi to 0.0.19

nodejs/node#49908

* lib,test: do not hardcode Buffer.kMaxLength

nodejs/node#49876

* crypto: account for disabled SharedArrayBuffer

nodejs/node#50034

* test: fix edge snapshot stack traces

nodejs/node#49659

* src: generate snapshot with --predictable

nodejs/node#48749

* chore: fixup patch indices

* fs: throw errors from sync branches instead of separate implementations

nodejs/node#49913

* crypto: ensure valid point on elliptic curve in SubtleCrypto.importKey

nodejs/node#50234

* esm: detect ESM syntax in ambiguous JavaScrip

nodejs/node#50096

* fixup! test: fix edge snapshot stack traces

* esm: unflag extensionless ES module JavaScript and Wasm in module scope

nodejs/node#49974

* [tagged-ptr] Arrowify objects

https://chromium-review.googlesource.com/c/v8/v8/+/4705331

---------

Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>
Co-authored-by: Charles Kerr <[email protected]>
Co-authored-by: Shelley Vohr <[email protected]>

PR-URL: nodejs/node#50296
Refs: nodejs/node#50295
Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>

PR-URL: nodejs/node#50296
Refs: nodejs/node#50295
Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>