@mui/internal-babel-plugin-minify-errors 2.0.8-canary.15 → 2.0.8-canary.17

package/index.js

@@ -43,6 +43,69 @@ const COMMENT_OPT_OUT_MARKER = 'minify-error-disabled';
  * }} Options
  */
 
+/**
+ * Checks if a node is `process.env.NODE_ENV` using Babel types.
+ * @param {babel.types} t
+ * @param {babel.types.Node} node
+ * @returns {boolean}
+ */
+function isProcessEnvNodeEnv(t, node) {
+  return (
+    t.isMemberExpression(node) &&
+    t.isMemberExpression(node.object) &&
+    t.isIdentifier(node.object.object, { name: 'process' }) &&
+    t.isIdentifier(node.object.property, { name: 'env' }) &&
+    t.isIdentifier(node.property, { name: 'NODE_ENV' })
+  );
+}
+
+/**
+ * Checks if a binary expression compares `process.env.NODE_ENV` with a value using the given operator.
+ * Handles both `process.env.NODE_ENV op value` and `value op process.env.NODE_ENV`.
+ * @param {babel.types} t
+ * @param {babel.types.BinaryExpression} node
+ * @param {string} operator
+ * @param {string} value
+ * @returns {boolean}
+ */
+function isNodeEnvComparison(t, node, operator, value) {
+  if (node.operator !== operator) {
+    return false;
+  }
+  return (
+    (isProcessEnvNodeEnv(t, node.left) && t.isStringLiteral(node.right, { value })) ||
+    (t.isStringLiteral(node.left, { value }) && isProcessEnvNodeEnv(t, node.right))
+  );
+}
+
+/**
+ * Checks if the given path is inside a dev-only branch
+ * (e.g. `if (process.env.NODE_ENV !== 'production') { ... }`).
+ * Errors inside such branches are already stripped in production,
+ * so minification is unnecessary.
+ * @param {babel.types} t
+ * @param {babel.NodePath} path
+ * @returns {boolean}
+ */
+function isInsideDevOnlyBranch(t, path) {
+  let current = path;
+  while (current.parentPath) {
+    const parent = current.parentPath;
+    if (parent.isIfStatement()) {
+      const isInConsequent = current.key === 'consequent';
+      const isInAlternate = current.key === 'alternate';
+      if ((isInConsequent || isInAlternate) && t.isBinaryExpression(parent.node.test)) {
+        const operator = isInConsequent ? '!==' : '===';
+        if (isNodeEnvComparison(t, parent.node.test, operator, 'production')) {
+          return true;
+        }
+      }
+    }
+    current = current.parentPath;
+  }
+  return false;
+}
+
 /**
  * Extracts the message and expressions from a node.
  * @param {babel.types} t
@@ -113,6 +176,10 @@ function findMessageNode(t, newExpressionPath, { detection, missingError }) {
     return null;
   }
 
+  if (isInsideDevOnlyBranch(t, newExpressionPath)) {
+    return null;
+  }
+
   switch (detection) {
     case 'opt-in': {
       if (

package/index.test.js

@@ -167,5 +167,15 @@ pluginTester({
       fixture: path.join(fixturePath, 'custom-runtime-imports-recursive', 'input.js'),
       output: readOutputFixtureSync('custom-runtime-imports-recursive', 'output.js'),
     },
+    {
+      title: 'skips errors inside dev-only branches',
+      pluginOptions: {
+        errorCodesPath: path.join(fixturePath, 'dev-only-branch', 'error-codes.json'),
+        runtimeModule: '@mui/utils/formatMuiErrorMessage',
+        detection: 'opt-out',
+      },
+      fixture: path.join(fixturePath, 'dev-only-branch', 'input.js'),
+      output: readOutputFixtureSync('dev-only-branch', 'output.js'),
+    },
   ],
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mui/internal-babel-plugin-minify-errors",
-  "version": "2.0.8-canary.15",
+  "version": "2.0.8-canary.17",
   "author": "MUI Team",
   "description": "This is an internal package not meant for general use.",
   "repository": {
@@ -17,11 +17,11 @@
     "url": "https://opencollective.com/mui-org"
   },
   "dependencies": {
-    "@babel/helper-module-imports": "^7.27.1",
+    "@babel/helper-module-imports": "^7.28.6",
     "find-package-json": "^1.2.0"
   },
   "devDependencies": {
-    "@babel/core": "7.28.5",
+    "@babel/core": "7.28.6",
     "@types/babel__core": "7.20.5",
     "@types/babel__helper-module-imports": "7.18.3",
     "@types/find-package-json": "1.2.7",
@@ -42,6 +42,6 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitSha": "ccf82dfe8f0e2c810ad9c6b6e2810e4e37ed27a3",
+  "gitSha": "a38e519df883cb7e993dd0980712643a16e5f51a",
   "scripts": {}
 }