test: test fs patches via cjs and esm

jbedard · jbedard · commit 6cf39bb332be · 2025-10-07T11:35:43.000-07:00
diff --git a/.aspect/rules/external_repository_action_cache/npm_translate_lock_LTE4Nzc1MDcwNjU= b/.aspect/rules/external_repository_action_cache/npm_translate_lock_LTE4Nzc1MDcwNjU=
@@ -32,6 +32,6 @@ npm/private/test/package.json=322962683
 npm/private/test/vendored/is-odd/package.json=1041695223
 npm/private/test/vendored/lodash-4.17.21.tgz=-1206623349
 npm/private/test/vendored/semver-max/package.json=578664053
-package.json=1068891966
-pnpm-lock.yaml=1390701932
+package.json=234760749
+pnpm-lock.yaml=43430792
 pnpm-workspace.yaml=-2039776064
diff --git a/js/private/node-patches/fs.cjs b/js/private/node-patches/fs.cjs
@@ -47,6 +47,7 @@ const util = require("util");
 // es modules
 const fs = require('node:fs');
 const url = require('node:url');
+const esmModule = require('node:module');
 const HOP_NON_LINK = Symbol.for('HOP NON LINK');
 const HOP_NOT_FOUND = Symbol.for('HOP NOT FOUND');
 const PATCHED_FS_METHODS = [
@@ -747,12 +748,18 @@ function patcher(roots) {
             }
         }
     }
+    // Sync the esm modules to use the now patched fs cjs module.
+    // See: https://nodejs.org/api/esm.html#builtin-modules
+    esmModule.syncBuiltinESMExports();
     return function revertPatch() {
         Object.assign(fs, fs._unpatched);
         delete fs._unpatched;
         if (unpatchPromises) {
             unpatchPromises();
         }
+        // Sync the esm modules to use the now patched fs cjs module.
+        // See: https://nodejs.org/api/esm.html#builtin-modules
+        esmModule.syncBuiltinESMExports();
     };
 }
 // =========================================================================
diff --git a/js/private/node-patches/register.cjs b/js/private/node-patches/register.cjs
@@ -35,16 +35,11 @@ if (
     JS_BINARY__PATCH_NODE_FS != '0' &&
     JS_BINARY__FS_PATCH_ROOTS
 ) {
-    const module = require('node:module')
     const roots = JS_BINARY__FS_PATCH_ROOTS.split(':')
     if (JS_BINARY__LOG_DEBUG) {
         console.error(
             `DEBUG: ${JS_BINARY__LOG_PREFIX}: node fs patches will be applied with roots: ${roots}`
         )
     }
     patchfs(roots)
-
-    // Sync the esm modules to use the now patched fs cjs module.
-    // See: https://nodejs.org/api/esm.html#builtin-modules
-    module.syncBuiltinESMExports()
 }
diff --git a/js/private/node-patches/src/fs.cts b/js/private/node-patches/src/fs.cts
@@ -30,6 +30,7 @@ type Dirent = any
 // es modules
 const fs = require('node:fs') as any
 const url = require('node:url') as typeof UrlType
+const esmModule = require('node:module')
 
 const HOP_NON_LINK = Symbol.for('HOP NON LINK')
 const HOP_NOT_FOUND = Symbol.for('HOP NOT FOUND')
@@ -857,13 +858,21 @@ export function patcher(roots: string[]): () => void {
         }
     }
 
+    // Sync the esm modules to use the now patched fs cjs module.
+    // See: https://nodejs.org/api/esm.html#builtin-modules
+    esmModule.syncBuiltinESMExports()
+
     return function revertPatch() {
         Object.assign(fs, fs._unpatched)
         delete fs._unpatched
 
         if (unpatchPromises) {
             unpatchPromises()
         }
+
+        // Sync the esm modules to use the now patched fs cjs module.
+        // See: https://nodejs.org/api/esm.html#builtin-modules
+        esmModule.syncBuiltinESMExports()
     }
 }
 
diff --git a/js/private/test/image/checksum_test.expected b/js/private/test/image/checksum_test.expected
@@ -1,4 +1,4 @@
-a4232a52470e03fc555dfc1bd5a6e83090a71bccad58eb45fbc55f780f84df1e  js/private/test/image/cksum_node
+41a1bc481e704b8776af2de8fd24cab6252e326096e84d27df15dd39dec727c3  js/private/test/image/cksum_node
 52836a988c8ac815b4a3b70fa3a3acec67b851699fa989694cef4cc1fa53de96  js/private/test/image/cksum_package_store_3p
 642b308a0561fb51dfd96d08d74a4ec419c9d2ca501cfa1002a49c8e25fbe4c2  js/private/test/image/cksum_package_store_1p
 5d45f32dacf0b83e26c33d4e1017c694e79eaff29b8ecc336f9ea8fdee870d45  js/private/test/image/cksum_node_modules
diff --git a/js/private/test/image/custom_layers_nomatch_test_node.listing b/js/private/test/image/custom_layers_nomatch_test_node.listing
@@ -8,8 +8,8 @@ drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/private/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/private/node-patches/
--r-xr-xr-x  0 0      0       34161 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/private/node-patches/fs.cjs
--r-xr-xr-x  0 0      0        1664 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/private/node-patches/register.cjs
+-r-xr-xr-x  0 0      0       34545 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/private/node-patches/fs.cjs
+-r-xr-xr-x  0 0      0        1460 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/private/node-patches/register.cjs
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/rules_nodejs~~node~nodejs_linux_amd64/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/rules_nodejs~~node~nodejs_linux_amd64/bin/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/rules_nodejs~~node~nodejs_linux_amd64/bin/nodejs/
diff --git a/js/private/test/image/custom_owner_test_node.listing b/js/private/test/image/custom_owner_test_node.listing
@@ -7,8 +7,8 @@ drwxr-xr-x  0 100    0           0 Jan  1  1970 ./js/private/test/image/bin.runf
 drwxr-xr-x  0 100    0           0 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/
 drwxr-xr-x  0 100    0           0 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/private/
 drwxr-xr-x  0 100    0           0 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/private/node-patches/
--r-xr-xr-x  0 100    0       34161 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/private/node-patches/fs.cjs
--r-xr-xr-x  0 100    0        1664 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/private/node-patches/register.cjs
+-r-xr-xr-x  0 100    0       34545 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/private/node-patches/fs.cjs
+-r-xr-xr-x  0 100    0        1460 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/private/node-patches/register.cjs
 drwxr-xr-x  0 100    0           0 Jan  1  1970 ./js/private/test/image/bin.runfiles/rules_nodejs~~node~nodejs_linux_amd64/
 drwxr-xr-x  0 100    0           0 Jan  1  1970 ./js/private/test/image/bin.runfiles/rules_nodejs~~node~nodejs_linux_amd64/bin/
 drwxr-xr-x  0 100    0           0 Jan  1  1970 ./js/private/test/image/bin.runfiles/rules_nodejs~~node~nodejs_linux_amd64/bin/nodejs/
diff --git a/js/private/test/image/default_test_node.listing b/js/private/test/image/default_test_node.listing
@@ -7,8 +7,8 @@ drwxr-xr-x  0 0      0           0 Jan  1  1970 ./js/private/test/image/bin.runf
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/private/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/private/node-patches/
--r-xr-xr-x  0 0      0       34161 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/private/node-patches/fs.cjs
--r-xr-xr-x  0 0      0        1664 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/private/node-patches/register.cjs
+-r-xr-xr-x  0 0      0       34545 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/private/node-patches/fs.cjs
+-r-xr-xr-x  0 0      0        1460 Jan  1  1970 ./js/private/test/image/bin.runfiles/_main/js/private/node-patches/register.cjs
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./js/private/test/image/bin.runfiles/rules_nodejs~~node~nodejs_linux_amd64/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./js/private/test/image/bin.runfiles/rules_nodejs~~node~nodejs_linux_amd64/bin/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./js/private/test/image/bin.runfiles/rules_nodejs~~node~nodejs_linux_amd64/bin/nodejs/
diff --git a/js/private/test/image/non_ascii/custom_layer_groups_test_just_the_fs_patch.listing b/js/private/test/image/non_ascii/custom_layer_groups_test_just_the_fs_patch.listing
@@ -9,4 +9,4 @@ drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/non_
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/non_ascii/bin2.runfiles/_main/js/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/non_ascii/bin2.runfiles/_main/js/private/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/non_ascii/bin2.runfiles/_main/js/private/node-patches/
--r-xr-xr-x  0 0      0       34161 Jan  1  1970 ./app/js/private/test/image/non_ascii/bin2.runfiles/_main/js/private/node-patches/fs.cjs
+-r-xr-xr-x  0 0      0       34545 Jan  1  1970 ./app/js/private/test/image/non_ascii/bin2.runfiles/_main/js/private/node-patches/fs.cjs
diff --git a/js/private/test/image/non_ascii/custom_layer_groups_test_node.listing b/js/private/test/image/non_ascii/custom_layer_groups_test_node.listing
@@ -9,7 +9,7 @@ drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/non_
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/non_ascii/bin2.runfiles/_main/js/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/non_ascii/bin2.runfiles/_main/js/private/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/non_ascii/bin2.runfiles/_main/js/private/node-patches/
--r-xr-xr-x  0 0      0        1664 Jan  1  1970 ./app/js/private/test/image/non_ascii/bin2.runfiles/_main/js/private/node-patches/register.cjs
+-r-xr-xr-x  0 0      0        1460 Jan  1  1970 ./app/js/private/test/image/non_ascii/bin2.runfiles/_main/js/private/node-patches/register.cjs
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/non_ascii/bin2.runfiles/rules_nodejs~~node~nodejs_linux_amd64/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/non_ascii/bin2.runfiles/rules_nodejs~~node~nodejs_linux_amd64/bin/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/non_ascii/bin2.runfiles/rules_nodejs~~node~nodejs_linux_amd64/bin/nodejs/
diff --git a/js/private/test/image/regex_edge_cases_test_node.listing b/js/private/test/image/regex_edge_cases_test_node.listing
@@ -8,8 +8,8 @@ drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/private/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/private/node-patches/
--r-xr-xr-x  0 0      0       34161 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/private/node-patches/fs.cjs
--r-xr-xr-x  0 0      0        1664 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/private/node-patches/register.cjs
+-r-xr-xr-x  0 0      0       34545 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/private/node-patches/fs.cjs
+-r-xr-xr-x  0 0      0        1460 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/_main/js/private/node-patches/register.cjs
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/rules_nodejs~~node~nodejs_linux_amd64/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/rules_nodejs~~node~nodejs_linux_amd64/bin/
 drwxr-xr-x  0 0      0           0 Jan  1  1970 ./app/js/private/test/image/bin.runfiles/rules_nodejs~~node~nodejs_linux_amd64/bin/nodejs/
diff --git a/js/private/test/node-patches/BUILD.bazel b/js/private/test/node-patches/BUILD.bazel
@@ -1,13 +1,20 @@
 load("@aspect_bazel_lib//lib:copy_to_bin.bzl", "copy_to_bin")
+load("@npm//:@babel/cli/package_json.bzl", babel_bin = "bin")
 load("//js:defs.bzl", "js_test")
 
 TESTS = [
     "escape.js",
-    "lstat.js",
-    "opendir.js",
-    "readdir.js",
-    "readlink.js",
-    "realpath.js",
+    "lstat.mjs",
+    "opendir.mjs",
+    "readdir.mjs",
+    "readlink.mjs",
+    "realpath.mjs",
+]
+
+CJS_TESTS = [
+    t.replace(".mjs", ".cjs")
+    for t in TESTS
+    if t.endswith(".mjs")
 ]
 
 # Multiple node toolchains for testing across versions
@@ -39,26 +46,65 @@ TOOLCHAINS_VERSIONS = [
     for t in TESTS
 ]
 
+babel_bin.babel(
+    name = "babel_mjs2js",
+    srcs = [t for t in TESTS if t.endswith(".mjs")] + [
+        "babel.config.js",
+        "//:node_modules/@babel/plugin-transform-modules-commonjs",
+    ],
+    outs = [t.replace(".mjs", ".cjs") for t in TESTS if t.endswith(".mjs")],
+    args = [
+        "--extensions",
+        ".mjs",
+        "--out-file-extension",
+        ".cjs",
+        "--out-dir",
+        ".",
+    ] + [t for t in TESTS if t.endswith(".mjs")],
+    chdir = package_name(),  # to automatically pickup babel.config.js
+)
+
 # Basic tests
 [
     [
-        js_test(
-            name = "{}_{}_test".format(
-                t.replace(".js", ""),
-                toolchain_name,
-            ),
-            data = [
-                "//:node_modules/inline-fixtures",
-                "//js/private/node-patches/src:compile",
-            ],
-            entry_point = "copy_entry_{}".format(t),
-            node_toolchain = toolchain,
-            patch_node_fs = False,
-            # Without node patches on for these tests, the program is going to escape the sandbox if it
-            # is on since the fs patches are not on for the tests as they are the code under test
-            tags = ["no-sandbox"],
-        )
-        for t in TESTS
+        [
+            js_test(
+                name = "{}_{}_test".format(
+                    t.replace(".mjs", "").replace(".js", ""),
+                    toolchain_name,
+                ),
+                data = [
+                    "//:node_modules/inline-fixtures",
+                    "//js/private/node-patches/src:compile",
+                ],
+                entry_point = "copy_entry_{}".format(t) if (t in TESTS) else t,
+                node_toolchain = toolchain,
+                patch_node_fs = False,
+                # Without node patches on for these tests, the program is going to escape the sandbox if it
+                # is on since the fs patches are not on for the tests as they are the code under test
+                tags = ["no-sandbox"],
+            )
+            for t in TESTS
+        ],
+        [
+            js_test(
+                name = "{}_{}_cjs_test".format(
+                    t.replace(".cjs", ""),
+                    toolchain_name,
+                ),
+                data = [
+                    "//:node_modules/inline-fixtures",
+                    "//js/private/node-patches/src:compile",
+                ],
+                entry_point = "copy_entry_{}".format(t) if (t in TESTS) else t,
+                node_toolchain = toolchain,
+                patch_node_fs = False,
+                # Without node patches on for these tests, the program is going to escape the sandbox if it
+                # is on since the fs patches are not on for the tests as they are the code under test
+                tags = ["no-sandbox"],
+            )
+            for t in CJS_TESTS
+        ],
     ]
     for toolchain_name, toolchain in zip(
         TOOLCHAINS_NAMES,
diff --git a/js/private/test/node-patches/babel.config.js b/js/private/test/node-patches/babel.config.js
@@ -0,0 +1,10 @@
+module.exports = {
+    plugins: [
+        [
+            '@babel/plugin-transform-modules-commonjs',
+            {
+                importInterop: 'none',
+            },
+        ],
+    ],
+}
diff --git a/js/private/test/node-patches/lstat.mjs b/js/private/test/node-patches/lstat.mjs
@@ -14,13 +14,13 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-const assert = require('assert')
-const fs = require('fs')
-const withFixtures = require('inline-fixtures').withFixtures
-const path = require('path')
-const util = require('util')
+import * as assert from 'node:assert'
+import * as fs from 'node:fs'
+import { withFixtures } from 'inline-fixtures'
+import * as path from 'node:path'
+import * as util from 'node:util'
 
-const patcher = require('../../node-patches/src/fs.cjs').patcher
+import { patcher } from '../../node-patches/src/fs.cjs'
 
 // We don't want to bring jest into this repo so we just fake the describe and it functions here
 async function describe(_, fn) {
@@ -197,7 +197,7 @@ describe('testing lstat', async () => {
                     brokenLinkPath
                 )
 
-                stat = await fs.promises.lstat(brokenLinkPath)
+                const stat = await fs.promises.lstat(brokenLinkPath)
                 assert.ok(
                     stat.isSymbolicLink(),
                     'if a symlink is broken but not escaping return it as a link.'
diff --git a/js/private/test/node-patches/opendir.mjs b/js/private/test/node-patches/opendir.mjs
@@ -14,13 +14,13 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-const assert = require('assert')
-const fs = require('fs')
-const withFixtures = require('inline-fixtures').withFixtures
-const path = require('path')
-const util = require('util')
+import * as assert from 'node:assert'
+import * as fs from 'node:fs'
+import { withFixtures } from 'inline-fixtures'
+import * as path from 'node:path'
+import * as util from 'node:util'
 
-const patcher = require('../../node-patches/src/fs.cjs').patcher
+import { patcher } from '../../node-patches/src/fs.cjs'
 
 // We don't want to bring jest into this repo so we just fake the describe and it functions here
 async function describe(_, fn) {
diff --git a/js/private/test/node-patches/readdir.mjs b/js/private/test/node-patches/readdir.mjs
@@ -14,13 +14,13 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-const assert = require('assert')
-const fs = require('fs')
-const withFixtures = require('inline-fixtures').withFixtures
-const path = require('path')
-const util = require('util')
+import * as assert from 'node:assert'
+import * as fs from 'node:fs'
+import { withFixtures } from 'inline-fixtures'
+import * as path from 'node:path'
+import * as util from 'node:util'
 
-const patcher = require('../../node-patches/src/fs.cjs').patcher
+import { patcher } from '../../node-patches/src/fs.cjs'
 
 // We don't want to bring jest into this repo so we just fake the describe and it functions here
 async function describe(_, fn) {
diff --git a/js/private/test/node-patches/readlink.mjs b/js/private/test/node-patches/readlink.mjs
@@ -14,13 +14,13 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-const assert = require('assert')
-const fs = require('fs')
-const withFixtures = require('inline-fixtures').withFixtures
-const path = require('path')
-const util = require('util')
+import * as assert from 'node:assert'
+import * as fs from 'node:fs'
+import { withFixtures } from 'inline-fixtures'
+import * as path from 'node:path'
+import * as util from 'node:util'
 
-const patcher = require('../../node-patches/src/fs.cjs').patcher
+import { patcher } from '../../node-patches/src/fs.cjs'
 
 // We don't want to bring jest into this repo so we just fake the describe and it functions here
 async function describe(_, fn) {
diff --git a/js/private/test/node-patches/realpath.mjs b/js/private/test/node-patches/realpath.mjs
@@ -14,13 +14,13 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-const assert = require('assert')
-const fs = require('fs')
-const withFixtures = require('inline-fixtures').withFixtures
-const path = require('path')
-const util = require('util')
+import * as assert from 'node:assert'
+import * as fs from 'node:fs'
+import { withFixtures } from 'inline-fixtures'
+import * as path from 'node:path'
+import * as util from 'node:util'
 
-const patcher = require('../../node-patches/src/fs.cjs').patcher
+import { patcher } from '../../node-patches/src/fs.cjs'
 
 // We don't want to bring jest into this repo so we just fake the describe and it functions here
 async function describe(_, fn) {
diff --git a/package.json b/package.json
@@ -1,6 +1,9 @@
 {
     "private": true,
     "devDependencies": {
+        "@babel/cli": "^7.23.0",
+        "@babel/core": "^7.23.0",
+        "@babel/plugin-transform-modules-commonjs": "^7.23.0",
         "@types/node": "18.19.54",
         "chalk": "5.1.1",
         "inline-fixtures": "1.1.0",
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml

Original file line number	Diff line number	Diff line change
`@@ -47,6 +47,7 @@ const util = require("util");`
`47`	`47`	`// es modules`
`48`	`48`	`const fs = require('node:fs');`
`49`	`49`	`const url = require('node:url');`
	`50`	`+const esmModule = require('node:module');`
`50`	`51`	`const HOP_NON_LINK = Symbol.for('HOP NON LINK');`
`51`	`52`	`const HOP_NOT_FOUND = Symbol.for('HOP NOT FOUND');`
`52`	`53`	`const PATCHED_FS_METHODS = [`
`@@ -747,12 +748,18 @@ function patcher(roots) {`
`747`	`748`	`}`
`748`	`749`	`}`
`749`	`750`	`}`
	`751`	`+ // Sync the esm modules to use the now patched fs cjs module.`
	`752`	`+ // See: https://nodejs.org/api/esm.html#builtin-modules`
	`753`	`+ esmModule.syncBuiltinESMExports();`
`750`	`754`	`return function revertPatch() {`
`751`	`755`	`Object.assign(fs, fs._unpatched);`
`752`	`756`	`delete fs._unpatched;`
`753`	`757`	`if (unpatchPromises) {`
`754`	`758`	`unpatchPromises();`
`755`	`759`	`}`
	`760`	`+ // Sync the esm modules to use the now patched fs cjs module.`
	`761`	`+ // See: https://nodejs.org/api/esm.html#builtin-modules`
	`762`	`+ esmModule.syncBuiltinESMExports();`
`756`	`763`	`};`
`757`	`764`	`}`
`758`	`765`	`// =========================================================================`
Original file line number	Diff line number	Diff line change
`@@ -35,16 +35,11 @@ if (`
`35`	`35`	`JS_BINARY__PATCH_NODE_FS != '0' &&`
`36`	`36`	`JS_BINARY__FS_PATCH_ROOTS`
`37`	`37`	`) {`
`38`		`- const module = require('node:module')`
`39`	`38`	`const roots = JS_BINARY__FS_PATCH_ROOTS.split(':')`
`40`	`39`	`if (JS_BINARY__LOG_DEBUG) {`
`41`	`40`	`console.error(`
`42`	`41`	`DEBUG: ${JS_BINARY__LOG_PREFIX}: node fs patches will be applied with roots: ${roots}`
`43`	`42`	`)`
`44`	`43`	`}`
`45`	`44`	`patchfs(roots)`
`46`		`-`
`47`		`- // Sync the esm modules to use the now patched fs cjs module.`
`48`		`- // See: https://nodejs.org/api/esm.html#builtin-modules`
`49`		`- module.syncBuiltinESMExports()`
`50`	`45`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,6 +30,7 @@ type Dirent = any`
`30`	`30`	`// es modules`
`31`	`31`	`const fs = require('node:fs') as any`
`32`	`32`	`const url = require('node:url') as typeof UrlType`
	`33`	`+const esmModule = require('node:module')`
`33`	`34`
`34`	`35`	`const HOP_NON_LINK = Symbol.for('HOP NON LINK')`
`35`	`36`	`const HOP_NOT_FOUND = Symbol.for('HOP NOT FOUND')`
`@@ -857,13 +858,21 @@ export function patcher(roots: string[]): () => void {`
`857`	`858`	`}`
`858`	`859`	`}`
`859`	`860`
	`861`	`+ // Sync the esm modules to use the now patched fs cjs module.`
	`862`	`+ // See: https://nodejs.org/api/esm.html#builtin-modules`
	`863`	`+ esmModule.syncBuiltinESMExports()`
	`864`	`+`
`860`	`865`	`return function revertPatch() {`
`861`	`866`	`Object.assign(fs, fs._unpatched)`
`862`	`867`	`delete fs._unpatched`
`863`	`868`
`864`	`869`	`if (unpatchPromises) {`
`865`	`870`	`unpatchPromises()`
`866`	`871`	`}`
	`872`	`+`
	`873`	`+ // Sync the esm modules to use the now patched fs cjs module.`
	`874`	`+ // See: https://nodejs.org/api/esm.html#builtin-modules`
	`875`	`+ esmModule.syncBuiltinESMExports()`
`867`	`876`	`}`
`868`	`877`	`}`
`869`	`878`