Skip to content

Commit f9ee046

Browse files
committed
fix: Error: Dynamic require of "os" is not supported
Needed as esbuild does not provide require shims / proxy from ESModules.
1 parent dadb2ea commit f9ee046

File tree

2 files changed

+15
-2
lines changed

2 files changed

+15
-2
lines changed

github-actions/post-approval-changes/main.js

Lines changed: 2 additions & 1 deletion
Large diffs are not rendered by default.

tools/defaults2.bzl

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,9 +84,21 @@ def jasmine_test(name, **kwargs):
8484
**kwargs
8585
)
8686

87-
def esbuild_checked_in(name, **kwargs):
87+
def esbuild_checked_in(name, platform, **kwargs):
88+
config = None
89+
if platform == "node":
90+
config = {
91+
"banner": {
92+
# Note: Needed as esbuild does not provide require shims / proxy from ESModules.
93+
# See: https://github.com/evanw/esbuild/issues/1921.
94+
"js": "import { createRequire } from 'node:module';globalThis['require'] ??= createRequire(import.meta.url);",
95+
}
96+
}
97+
8898
_esbuild(
8999
name = "%s_generated" % name,
100+
platform = platform,
101+
config = config,
90102
sourcemap = "inline",
91103
**kwargs
92104
)

0 commit comments

Comments
 (0)