fix: resolve variable

huozhi · Nov 24, 2024 · 409e4ad · 409e4ad
1 parent a13e2b5
commit 409e4ad
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/plugins/esm-shim.test.ts b/src/plugins/esm-shim.test.ts
@@ -7,6 +7,7 @@ describe('require regex', () => {
     expect(match('require(\n"foo"\n)')).toBe(true)
     expect(match(`require(\`mod\`)`)).toBe(true)
     expect(match(`require.resolve('mod')`)).toBe(true)
+    expect(match(`\nrequire.resolve(value)`)).toBe(true)
 
     expect(match('foo.require("foo")')).toBe(false)
     expect(match(`"require('module')"`)).toBe(false)

diff --git a/src/plugins/esm-shim.ts b/src/plugins/esm-shim.ts
@@ -7,7 +7,7 @@ const FILENAME_REGEX = /__filename/
 const DIRNAME_REGEX = /__dirname/
 // not char, or space before require(.resolve)?(
 export const GLOBAL_REQUIRE_REGEX =
-  /(?:^|[^.\w'"`])\brequire(\.resolve)?\(\s*[\r\n]*(['"`])/
+  /(?:^|[^.\w'"`])\brequire(\.resolve)?\(\s*[\r\n]*(\w|['"`])/
 
 const PolyfillComment = '/** rollup-private-do-not-use-esm-shim-polyfill */'
 const createESMShim = ({