Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.Sign up
Bootstrap: make global.process, global.Buffer getters #26882
This implements just the semver major breaking changes from #26334 Restrict process and Buffer globals to CommonJS, which is to make
I've split this out because we need two TSC approvals for this to make the 12 major release, if this is to be considered a breaking change.
So this is the bare minimum prerequisite to ensure that #26334 remains possible in the design of ES Modules in Node.js. If we miss this target, it will be very difficult to implement these important security properties later on.
As well as the CIGTM run on that PR, I also ran these builds against Jest running on the ncc test suite, which tests a large number of ecosystem packages as well. These have all been passing fine.
It would be amazing to get those two TSC approvals to be able to ensure these important security approaches for modules remain possible.
Can confirm failing test is a flakey test case, as it passed on the rerun here - https://ci.nodejs.org/job/node-test-commit-linux/nodes=alpine-latest-x64/26361/.