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

Comparing version 1.4.2-next.8681bbb.0 to 1.5.0-next.c02060d.0

dist/index.d.ts

@@ -10,3 +10,3 @@ import { Plugin } from 'esbuild';
 interface NodePolyfillsOptions {
-    fallback?: 'empty' | 'none';
+    fallback?: 'empty' | 'error' | 'none';
     globals?: {
@@ -16,3 +16,3 @@ Buffer?: boolean;
     };
-    modules?: string[] | Record<string, boolean | 'empty'>;
+    modules?: string[] | Record<string, boolean | 'empty' | 'error'>;
     name?: string;
@@ -19,0 +19,0 @@ namespace?: string;

dist/index.js

@@ -5,2 +5,3 @@ 'use strict';
 var path = require('path');
+var process = require('process');
 var localPkg = require('local-pkg');
@@ -13,2 +14,3 @@ var esbuild = require('esbuild');
 var path__default = /*#__PURE__*/_interopDefault(path);
+var process__default = /*#__PURE__*/_interopDefault(process);
 
@@ -120,3 +122,9 @@ var __defProp = Object.defineProperty;
 var nodeModulesPolyfillPlugin = /* @__PURE__ */ __name((options = {}) => {
-  const { globals = {}, modules: modulesOption = module$1.builtinModules, fallback, namespace = NAME, name = NAME } = options;
+  const {
+    globals = {},
+    modules: modulesOption = module$1.builtinModules,
+    fallback = "none",
+    namespace = NAME,
+    name = NAME
+  } = options;
   if (namespace.endsWith("commonjs")) {
@@ -128,8 +136,17 @@ throw new Error(`namespace ${namespace} must not end with commonjs`);
   }
+  if (namespace.endsWith("error")) {
+    throw new Error(`namespace ${namespace} must not end with error`);
+  }
   const modules = Array.isArray(modulesOption) ? Object.fromEntries(modulesOption.map((mod) => [mod, true])) : modulesOption;
   const commonjsNamespace = `${namespace}-commonjs`;
   const emptyNamespace = `${namespace}-empty`;
+  const errorNamespace = `${namespace}-error`;
+  const shouldDetectErrorModules = fallback === "error" || Object.values(modules).includes("error");
   return {
     name,
-    setup: ({ onLoad, onResolve, initialOptions }) => {
+    setup: ({ onLoad, onResolve, onEnd, initialOptions }) => {
+      if (shouldDetectErrorModules && initialOptions.write !== false) {
+        throw new Error(`The "write" build option must be set to false when using the "error" polyfill type`);
+      }
+      const root = initialOptions.absWorkingDir ?? process__default.default.cwd();
       if (initialOptions.define && !initialOptions.define.global) {
@@ -156,11 +173,31 @@ initialOptions.define.global = "globalThis";
       });
+      onLoad({ filter: /.*/, namespace: errorNamespace }, (args) => {
+        return {
+          loader: "js",
+          contents: `module.exports = ${JSON.stringify(
+            // This encoded string is detected and parsed at the end of the build to report errors
+            `__POLYFILL_ERROR_START__::MODULE::${args.path}::IMPORTER::${args.pluginData.importer}::__POLYFILL_ERROR_END__`
+          )}`
+        };
+      });
       onLoad({ filter: /.*/, namespace }, loader);
       onLoad({ filter: /.*/, namespace: commonjsNamespace }, loader);
-      const bundledModules = fallback === "empty" ? module$1.builtinModules : Object.keys(modules).filter((moduleName) => module$1.builtinModules.includes(moduleName));
+      const bundledModules = fallback === "none" ? Object.keys(modules).filter((moduleName) => module$1.builtinModules.includes(moduleName)) : module$1.builtinModules;
       const filter = new RegExp(`^(?:node:)?(?:${bundledModules.map(escapeRegex).join("|")})$`);
       const resolver = /* @__PURE__ */ __name(async (args) => {
-        const emptyResult = {
-          namespace: emptyNamespace,
-          path: args.path,
-          sideEffects: false
+        const result = {
+          empty: {
+            namespace: emptyNamespace,
+            path: args.path,
+            sideEffects: false
+          },
+          error: {
+            namespace: errorNamespace,
+            path: args.path,
+            sideEffects: false,
+            pluginData: {
+              importer: path__default.default.relative(root, args.importer).replace(/\\/g, "/")
+            }
+          },
+          none: void 0
         };
@@ -170,16 +207,20 @@ if (initialOptions.platform === "browser") {
           const browserFieldValue = packageJson?.browser?.[args.path];
-          if (browserFieldValue !== void 0)
+          if (browserFieldValue === false) {
+            return result.empty;
+          }
+          if (browserFieldValue !== void 0) {
             return;
+          }
         }
         const moduleName = normalizeNodeBuiltinPath(args.path);
-        const fallbackResult = fallback === "empty" ? emptyResult : void 0;
-        if (!modules[moduleName]) {
-          return fallbackResult;
+        const polyfillOption = modules[moduleName];
+        if (!polyfillOption) {
+          return result[fallback];
         }
-        if (modules[moduleName] === "empty") {
-          return emptyResult;
+        if (polyfillOption === "error" || polyfillOption === "empty") {
+          return result[polyfillOption];
         }
-        const polyfill = await getCachedPolyfillPath(moduleName).catch(() => null);
-        if (!polyfill) {
-          return fallbackResult;
+        const polyfillPath2 = await getCachedPolyfillPath(moduleName).catch(() => null);
+        if (!polyfillPath2) {
+          return result[fallback];
         }
@@ -195,2 +236,22 @@ const ignoreRequire = args.namespace === commonjsNamespace;
       onResolve({ filter }, resolver);
+      onEnd(({ outputFiles = [] }) => {
+        if (!shouldDetectErrorModules)
+          return;
+        const errors = [];
+        const { outfile, outExtension = {} } = initialOptions;
+        const jsExtension = outfile ? path__default.default.extname(outfile) : outExtension[".js"] || ".js";
+        const jsFiles = outputFiles.filter((file) => path__default.default.extname(file.path) === jsExtension);
+        for (const file of jsFiles) {
+          const matches = file.text.matchAll(
+            /__POLYFILL_ERROR_START__::MODULE::(?<module>.+?)::IMPORTER::(?<importer>.+?)::__POLYFILL_ERROR_END__/g
+          );
+          for (const { groups } of matches) {
+            errors.push({
+              pluginName: name,
+              text: `Module "${groups.module}" is not polyfilled, imported by "${groups.importer}"`
+            });
+          }
+        }
+        return { errors };
+      });
     }
@@ -197,0 +258,0 @@ };

package.json

 {
   "name": "esbuild-plugins-node-modules-polyfill",
-  "version": "1.4.2-next.8681bbb.0",
+  "version": "1.5.0-next.c02060d.0",
   "description": "Polyfills nodejs builtin modules and globals for the browser.",
@@ -5,0 +5,0 @@ "main": "dist/index.js",

README.md

@@ -100,2 +100,34 @@ <div align="center">
 
+Optionally fail the build when certain modules are used (note that the `write` build option must be `false` to support this):
+
+```ts
+import { nodeModulesPolyfillPlugin } from 'esbuild-plugins-node-modules-polyfill';
+import { build } from 'esbuild';
+const buildResult = await build({
+	write: false,
+	plugins: [nodeModulesPolyfillPlugin({
+		modules: {
+			crypto: 'error',
+			path: true,
+		},
+	})],
+});
+```
+
+Optionally fail the build when a module is not polyfilled or configured (note that the `write` build option must be `false` to support this):
+
+```ts
+import { nodeModulesPolyfillPlugin } from 'esbuild-plugins-node-modules-polyfill';
+import { build } from 'esbuild';
+const buildResult = await build({
+	write: false,
+	plugins: [nodeModulesPolyfillPlugin({
+		fallback: 'error',
+		modules: {
+			path: true,
+		}
+	})],
+});
+```
+
 ## Buy me some doughnuts
@@ -102,0 +134,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

45621

increased by21.63%

Lines of code

277

increased by28.24%

Number of lines in readme file

146

increased by28.07%

No dependency changes