Skip to content

[v22.x backport] Wasm ESM Integration PRs & Unflagging - #59179

Closed
guybedford wants to merge 2 commits into
nodejs:v22.x-stagingfrom
guybedford:esm-integration-unflag-backport
Closed

[v22.x backport] Wasm ESM Integration PRs & Unflagging#59179
guybedford wants to merge 2 commits into
nodejs:v22.x-stagingfrom
guybedford:esm-integration-unflag-backport

Conversation

@guybedford

@guybedfordguybedford commented Jul 22, 2025

Copy link
Copy Markdown
Contributor

Backports:

This does not include source phase imports support as this was a later v8 feature.

This only reflects the unflagging of the experimental instance phase, and the backport is updated to reflect that.

JS String Builtins is possible to support with a very small polyfill layer allowing the implementation to match and pass the tests from v24.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/config
  • @nodejs/loaders

@nodejs-github-botnodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. lib / src Issues and PRs related to general changes in the lib or src directory. needs-ci PRs that need a full CI run. v22.x Issues that can be reproduced on v22.x or PRs targeting the v22.x-staging branch. labels Jul 22, 2025
@guybedford
guybedfordforce-pushed the esm-integration-unflag-backport branch from b6d4790 to 8a5a312CompareJuly 23, 2025 18:52
@guybedfordguybedford changed the title [v22.x backport] esm: unflag --experimental-wasm-modules[v22.x backport] Wasm ESM Integration PRs & UnflaggingJul 23, 2025
@guybedford
guybedfordforce-pushed the esm-integration-unflag-backport branch from 8a5a312 to d069903CompareJuly 23, 2025 21:16
@guybedford
guybedfordforce-pushed the esm-integration-unflag-backport branch from d069903 to d5ff7eaCompareJuly 25, 2025 16:25
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@ryanaslett

Copy link
Copy Markdown

This is the first PR to run after re-enabling OSX testing.

@aduh95

aduh95 commented Jul 28, 2025

Copy link
Copy Markdown
Contributor

Could you please rebase? Also, #59020 is labeled dont-land-on-v22.xPRs that should not land on the v22.x-staging branch and should not be released in v22.x. , do we actually want to include it?

@guybedford
guybedfordforce-pushed the esm-integration-unflag-backport branch from d5ff7ea to fc09828CompareJuly 28, 2025 19:56
@guybedford

guybedford commented Jul 28, 2025

Copy link
Copy Markdown
ContributorAuthor

@aduh95 sure, this is now rebased. I initially added that label when I thought string builtins was not implementable, but it was trivial to include the polyfill. Aligning the semantics for instances between 22 and 24 feels like a win to me here, so I think it could be worthwhile.

@guybedford
guybedfordforce-pushed the esm-integration-unflag-backport branch from fc09828 to cb08197CompareJuly 30, 2025 02:01
@guybedford

Copy link
Copy Markdown
ContributorAuthor

It actually turns out we can use the V8 flag here instead of including the polyfill for string builtins, so rebasing #59020 was actually much easier than I originally implemented too.

@guybedford
guybedfordforce-pushed the esm-integration-unflag-backport branch from cb08197 to c25ba79CompareJuly 30, 2025 02:04
@nodejs-github-bot

This comment was marked as outdated.

@guybedford
guybedfordforce-pushed the esm-integration-unflag-backport branch from c25ba79 to 208e26dCompareJuly 30, 2025 03:55
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@guybedford
guybedfordforce-pushed the esm-integration-unflag-backport branch from 208e26d to a9736daCompareAugust 5, 2025 18:08
PR-URL: nodejs#59020
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
@guybedford
guybedfordforce-pushed the esm-integration-unflag-backport branch 2 times, most recently from b2ebc0d to d6de6cdCompareAugust 6, 2025 00:46
PR-URL: nodejs#57038
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
@guybedford
guybedfordforce-pushed the esm-integration-unflag-backport branch from d6de6cd to 6f42a72CompareAugust 6, 2025 01:43
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

aduh95 pushed a commit that referenced this pull request Aug 26, 2025
PR-URL: #59020
Backport-PR-URL: #59179
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 26, 2025
PR-URL: #57038
Backport-PR-URL: #59179
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Stephen Belanger <admin@stephenbelanger.com>
Reviewed-By: Geoffrey Booth <webadmin@geoffreybooth.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
@aduh95

Copy link
Copy Markdown
Contributor

Landed in f64f5df...fa9a9e9

@aduh95aduh95 closed this Aug 26, 2025
codebytere added a commit to electron/electron that referenced this pull request Sep 5, 2025
jkleinsc pushed a commit to electron/electron that referenced this pull request Sep 5, 2025
* chore: bump node in DEPS to v22.19.0
* chore: fixup patch indices
* crypto: add tls.setDefaultCACertificates()
nodejs/node#58822
* esm: js-string Wasm builtins in ESM Integration
nodejs/node#59179
---------
Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
dodolalorc pushed a commit to dodolalorc/electron that referenced this pull request Sep 11, 2025
* chore: bump node in DEPS to v22.19.0
* chore: fixup patch indices
* crypto: add tls.setDefaultCACertificates()
nodejs/node#58822
* esm: js-string Wasm builtins in ESM Integration
nodejs/node#59179
---------
Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
kigh-ota pushed a commit to kigh-ota/electron that referenced this pull request Sep 30, 2025
* chore: bump node in DEPS to v22.19.0
* chore: fixup patch indices
* crypto: add tls.setDefaultCACertificates()
nodejs/node#58822
* esm: js-string Wasm builtins in ESM Integration
nodejs/node#59179
---------
Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
nilayarya pushed a commit to nilayarya/electron that referenced this pull request Nov 21, 2025
* chore: bump node in DEPS to v22.19.0
* chore: fixup patch indices
* crypto: add tls.setDefaultCACertificates()
nodejs/node#58822
* esm: js-string Wasm builtins in ESM Integration
nodejs/node#59179
---------
Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
nilayarya added a commit to nilayarya/electron that referenced this pull request Nov 21, 2025
* chore: bump node in DEPS to v22.19.0
* chore: fixup patch indices
* crypto: add tls.setDefaultCACertificates()
nodejs/node#58822
* esm: js-string Wasm builtins in ESM Integration
nodejs/node#59179
---------
Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
nilayarya added a commit to nilayarya/electron that referenced this pull request Nov 21, 2025
* chore: bump node in DEPS to v22.19.0
* chore: fixup patch indices
* crypto: add tls.setDefaultCACertificates()
nodejs/node#58822
* esm: js-string Wasm builtins in ESM Integration
nodejs/node#59179
---------
Co-authored-by: electron-roller[bot] <84116207+electron-roller[bot]@users.noreply.github.com>
Co-authored-by: Shelley Vohr <shelley.vohr@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++Issues and PRs that require attention from people who are familiar with C++.lib / srcIssues and PRs related to general changes in the lib or src directory.needs-ciPRs that need a full CI run.v22.xIssues that can be reproduced on v22.x or PRs targeting the v22.x-staging branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@guybedford@nodejs-github-bot@ryanaslett@aduh95