@esbuild-plugins/node-modules-polyfill - npm Package Compare versions

Comparing version 0.1.3 to 0.1.4

dist/index.js

@@ -44,6 +44,6 @@ "use strict";
             if ((initialOptions === null || initialOptions === void 0 ? void 0 : initialOptions.define) && !((_a = initialOptions.define) === null || _a === void 0 ? void 0 : _a.global)) {
-                initialOptions.define['global'] = '{}';
+                initialOptions.define['global'] = 'globalThis';
             }
             else if (!(initialOptions === null || initialOptions === void 0 ? void 0 : initialOptions.define)) {
-                initialOptions.define = { global: '{}' };
+                initialOptions.define = { global: 'globalThis' };
             }
@@ -50,0 +50,0 @@ // TODO these polyfill module cannot import anything, is that ok?

dist/index.test.js

@@ -19,2 +19,3 @@ "use strict";
 const _1 = __importDefault(require("."));
+const node_globals_polyfill_1 = __importDefault(require("@esbuild-plugins/node-globals-polyfill"));
 require('debug').enable(require('../package.json').name);
@@ -159,3 +160,2 @@ test('works', () => __awaiter(void 0, void 0, void 0, function* () {
     eval(text);
-    console.log(text);
     expect(text).not.toContain(/\bglobal\b/);
@@ -165,2 +165,22 @@ // console.log(res.outputFiles[0].text)
 }));
+test('works with globals polyfills', () => __awaiter(void 0, void 0, void 0, function* () {
+    const { unlink, paths: [ENTRY], } = yield test_support_1.writeFiles({
+        'entry.ts': `import {x} from './utils'; console.log(x);`,
+        'utils.ts': `import path from 'path'; import { Buffer } from 'buffer'; export const x = path.resolve(Buffer.from('x').toString());`,
+    });
+    // const outfile = randomOutputFile()
+    const res = yield esbuild_1.build({
+        entryPoints: [ENTRY],
+        write: false,
+        format: 'esm',
+        target: 'es2017',
+        bundle: true,
+        plugins: [_1.default(), node_globals_polyfill_1.default()],
+    });
+    const text = res.outputFiles[0].text;
+    eval(text);
+    console.log(text);
+    // console.log(res.outputFiles[0].text)
+    unlink();
+}));
 //# sourceMappingURL=index.test.js.map

package.json

 {
     "name": "@esbuild-plugins/node-modules-polyfill",
-    "version": "0.1.3",
+    "version": "0.1.4",
     "description": "",
@@ -5,0 +5,0 @@ "main": "dist/index.js",

src/index.test.ts

@@ -5,2 +5,3 @@ import { build } from 'esbuild'
 import NodeModulesPolyfillsPlugin from '.'
+import NodeGlobalsPolyfillsPlugin from '@esbuild-plugins/node-globals-polyfill'
 
@@ -173,3 +174,2 @@ require('debug').enable(require('../package.json').name)
     eval(text)
-    console.log(text)
     expect(text).not.toContain(/\bglobal\b/)
@@ -179,1 +179,25 @@ // console.log(res.outputFiles[0].text)
 })
+
+test('works with globals polyfills', async () => {
+    const {
+        unlink,
+        paths: [ENTRY],
+    } = await writeFiles({
+        'entry.ts': `import {x} from './utils'; console.log(x);`,
+        'utils.ts': `import path from 'path'; import { Buffer } from 'buffer'; export const x = path.resolve(Buffer.from('x').toString());`,
+    })
+    // const outfile = randomOutputFile()
+    const res = await build({
+        entryPoints: [ENTRY],
+        write: false,
+        format: 'esm',
+        target: 'es2017',
+        bundle: true,
+        plugins: [NodeModulesPolyfillsPlugin(), NodeGlobalsPolyfillsPlugin()],
+    })
+    const text = res.outputFiles[0].text
+    eval(text)
+    console.log(text)
+    // console.log(res.outputFiles[0].text)
+    unlink()
+})

src/index.ts

@@ -41,5 +41,5 @@ import { OnResolveArgs, Plugin } from 'esbuild'
             if (initialOptions?.define && !initialOptions.define?.global) {
-                initialOptions.define['global'] = '{}'
+                initialOptions.define['global'] = 'globalThis'
             } else if (!initialOptions?.define) {
-                initialOptions.define = { global: '{}' }
+                initialOptions.define = { global: 'globalThis' }
             }
@@ -46,0 +46,0 @@

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

68553

increased by3.49%

Lines of code

1140

increased by3.92%

Worsened metrics

Number of medium supply chain risk alerts

15

increased by7.14%

No dependency changes