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

Comparing version 0.1.1 to 0.2.0

dist/index.d.ts

 import * as esbuild from 'esbuild';
-export declare function NodeGlobalsPolyfillPlugin({ buffer, define, process, }?: {
+export declare function NodeGlobalsPolyfillPlugin({ buffer, process, }?: {
     buffer?: boolean | undefined;
-    define?: {} | undefined;
     process?: boolean | undefined;
@@ -6,0 +5,0 @@ }): esbuild.Plugin;

dist/index.js

@@ -8,4 +8,5 @@ "use strict";
 const path_1 = __importDefault(require("path"));
-const fs_1 = __importDefault(require("fs"));
-function NodeGlobalsPolyfillPlugin({ buffer = false, define = {}, process = true, } = {}) {
+function NodeGlobalsPolyfillPlugin({ buffer = false, 
+// define = {},
+process = true, } = {}) {
     return {
@@ -24,16 +25,31 @@ name: 'node-globals-polyfill',
             });
-            onLoad({ filter: /_virtual-process-polyfill_\.js/ }, (arg) => {
-                const data = fs_1.default
-                    .readFileSync(path_1.default.resolve(__dirname, '../process.js'))
-                    .toString();
-                const keys = Object.keys(define);
+            // TODO esbuild cannot use virtual modules for inject: https://github.com/evanw/esbuild/issues/2762
+            // onLoad({ filter: /_virtual-process-polyfill_\.js/ }, (arg) => {
+            //     const data = fs
+            //         .readFileSync(path.resolve(__dirname, '../process.js'))
+            //         .toString()
+            //     const keys = Object.keys(define)
+            //     return {
+            //         loader: 'js',
+            //         contents: data.replace(
+            //             `const defines = {}`,
+            //             'const defines = {\n' +
+            //                 keys
+            //                     .filter((x) => x.startsWith('process.'))
+            //                     .sort((a, b) => a.length - b.length)
+            //                     .map(
+            //                         (k) =>
+            //                             `  ${JSON.stringify(k).replace(
+            //                                 'process.',
+            //                                 '',
+            //                             )}: ${define[k]},`,
+            //                     )
+            //                     .join('\n') +
+            //                 '\n}',
+            //         ),
+            //     }
+            // })
+            onResolve({ filter: /_virtual-process-polyfill_\.js/ }, () => {
                 return {
-                    loader: 'js',
-                    contents: data.replace(`const defines = {}`, 'const defines = {\n' +
-                        keys
-                            .filter((x) => x.startsWith('process.'))
-                            .sort((a, b) => a.length - b.length)
-                            .map((k) => `  ${JSON.stringify(k).replace('process.', '')}: ${define[k]},`)
-                            .join('\n') +
-                        '\n}'),
+                    path: path_1.default.resolve(__dirname, '../process.js'),
                 };
@@ -40,0 +56,0 @@ });

dist/index.test.js

@@ -49,2 +49,3 @@ "use strict";
 }));
+// TODO esbuild cannot use virtual modules for inject: https://github.com/evanw/esbuild/issues/2762
 test('process env vars are replaced with ones from define', () => __awaiter(void 0, void 0, void 0, function* () {
@@ -60,9 +61,6 @@ const { unlink, paths: [ENTRY], } = yield test_support_1.writeFiles({
         bundle: true,
-        plugins: [
-            _1.NodeGlobalsPolyfillPlugin({
-                define: {
-                    'process.env.VAR': '"hello"',
-                },
-            }),
-        ],
+        define: {
+            'process.env.VAR': '"hello"',
+        },
+        plugins: [_1.NodeGlobalsPolyfillPlugin({})],
     });
@@ -69,0 +67,0 @@ const output = res.outputFiles[0].text;

esm/index.d.ts

 import * as esbuild from 'esbuild';
-export declare function NodeGlobalsPolyfillPlugin({ buffer, define, process, }?: {
+export declare function NodeGlobalsPolyfillPlugin({ buffer, process, }?: {
     buffer?: boolean | undefined;
-    define?: {} | undefined;
     process?: boolean | undefined;
@@ -6,0 +5,0 @@ }): esbuild.Plugin;

esm/index.js

 import path from 'path';
-import fs from 'fs';
-export function NodeGlobalsPolyfillPlugin({ buffer = false, define = {}, process = true, } = {}) {
+export function NodeGlobalsPolyfillPlugin({ buffer = false, 
+// define = {},
+process = true, } = {}) {
     return {
@@ -17,16 +18,31 @@ name: 'node-globals-polyfill',
             });
-            onLoad({ filter: /_virtual-process-polyfill_\.js/ }, (arg) => {
-                const data = fs
-                    .readFileSync(path.resolve(__dirname, '../process.js'))
-                    .toString();
-                const keys = Object.keys(define);
+            // TODO esbuild cannot use virtual modules for inject: https://github.com/evanw/esbuild/issues/2762
+            // onLoad({ filter: /_virtual-process-polyfill_\.js/ }, (arg) => {
+            //     const data = fs
+            //         .readFileSync(path.resolve(__dirname, '../process.js'))
+            //         .toString()
+            //     const keys = Object.keys(define)
+            //     return {
+            //         loader: 'js',
+            //         contents: data.replace(
+            //             `const defines = {}`,
+            //             'const defines = {\n' +
+            //                 keys
+            //                     .filter((x) => x.startsWith('process.'))
+            //                     .sort((a, b) => a.length - b.length)
+            //                     .map(
+            //                         (k) =>
+            //                             `  ${JSON.stringify(k).replace(
+            //                                 'process.',
+            //                                 '',
+            //                             )}: ${define[k]},`,
+            //                     )
+            //                     .join('\n') +
+            //                 '\n}',
+            //         ),
+            //     }
+            // })
+            onResolve({ filter: /_virtual-process-polyfill_\.js/ }, () => {
                 return {
-                    loader: 'js',
-                    contents: data.replace(`const defines = {}`, 'const defines = {\n' +
-                        keys
-                            .filter((x) => x.startsWith('process.'))
-                            .sort((a, b) => a.length - b.length)
-                            .map((k) => `  ${JSON.stringify(k).replace('process.', '')}: ${define[k]},`)
-                            .join('\n') +
-                        '\n}'),
+                    path: path.resolve(__dirname, '../process.js'),
                 };
@@ -33,0 +49,0 @@ });

esm/index.test.js

@@ -47,2 +47,3 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
 }));
+// TODO esbuild cannot use virtual modules for inject: https://github.com/evanw/esbuild/issues/2762
 test('process env vars are replaced with ones from define', () => __awaiter(void 0, void 0, void 0, function* () {
@@ -58,9 +59,6 @@ const { unlink, paths: [ENTRY], } = yield writeFiles({
         bundle: true,
-        plugins: [
-            NodeGlobalsPolyfillPlugin({
-                define: {
-                    'process.env.VAR': '"hello"',
-                },
-            }),
-        ],
+        define: {
+            'process.env.VAR': '"hello"',
+        },
+        plugins: [NodeGlobalsPolyfillPlugin({})],
     });
@@ -67,0 +65,0 @@ const output = res.outputFiles[0].text;

package.json

 {
     "name": "@esbuild-plugins/node-globals-polyfill",
-    "version": "0.1.1",
+    "version": "0.2.0",
     "description": "",
@@ -5,0 +5,0 @@ "preferUnplugged": true,

src/index.test.ts

@@ -47,2 +47,4 @@ import { build } from 'esbuild'
 })
+
+// TODO esbuild cannot use virtual modules for inject: https://github.com/evanw/esbuild/issues/2762
 test('process env vars are replaced with ones from define', async () => {
@@ -61,9 +63,6 @@ const {
         bundle: true,
-        plugins: [
-            NodeGlobalsPolyfillPlugin({
-                define: {
-                    'process.env.VAR': '"hello"',
-                },
-            }),
-        ],
+        define: {
+            'process.env.VAR': '"hello"',
+        },
+        plugins: [NodeGlobalsPolyfillPlugin({})],
     })
@@ -70,0 +69,0 @@ const output = res.outputFiles[0].text

src/index.ts

@@ -7,3 +7,3 @@ import path from 'path'
     buffer = false,
-    define = {},
+    // define = {},
     process = true,
@@ -25,26 +25,32 @@ } = {}): esbuild.Plugin {
 
-            onLoad({ filter: /_virtual-process-polyfill_\.js/ }, (arg) => {
-                const data = fs
-                    .readFileSync(path.resolve(__dirname, '../process.js'))
-                    .toString()
+            // TODO esbuild cannot use virtual modules for inject: https://github.com/evanw/esbuild/issues/2762
+            // onLoad({ filter: /_virtual-process-polyfill_\.js/ }, (arg) => {
+            //     const data = fs
+            //         .readFileSync(path.resolve(__dirname, '../process.js'))
+            //         .toString()
 
-                const keys = Object.keys(define)
+            //     const keys = Object.keys(define)
+            //     return {
+            //         loader: 'js',
+            //         contents: data.replace(
+            //             `const defines = {}`,
+            //             'const defines = {\n' +
+            //                 keys
+            //                     .filter((x) => x.startsWith('process.'))
+            //                     .sort((a, b) => a.length - b.length)
+            //                     .map(
+            //                         (k) =>
+            //                             `  ${JSON.stringify(k).replace(
+            //                                 'process.',
+            //                                 '',
+            //                             )}: ${define[k]},`,
+            //                     )
+            //                     .join('\n') +
+            //                 '\n}',
+            //         ),
+            //     }
+            // })
+            onResolve({ filter: /_virtual-process-polyfill_\.js/ }, () => {
                 return {
-                    loader: 'js',
-                    contents: data.replace(
-                        `const defines = {}`,
-                        'const defines = {\n' +
-                            keys
-                                .filter((x) => x.startsWith('process.'))
-                                .sort((a, b) => a.length - b.length)
-                                .map(
-                                    (k) =>
-                                        `  ${JSON.stringify(k).replace(
-                                            'process.',
-                                            '',
-                                        )}: ${define[k]},`,
-                                )
-                                .join('\n') +
-                            '\n}',
-                    ),
+                    path: path.resolve(__dirname, '../process.js'),
                 }
@@ -51,0 +57,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

Filesystem access

Supply chain risk

Accesses the file system, and could potentially read sensitive data.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

103478

increased by0.72%

Lines of code

2758

increased by1.1%

Number of low supply chain risk alerts

1

decreased by-66.67%

Worsened metrics

Number of package files

23

decreased by-4.17%

No dependency changes