[code-infra] Lint rule to disallow guarded throw

[Janpot]

Implement #1069 (comment)

Make it a lint error to throw inside of a block that is branching into process.env.NODE_ENV one way or the other.

[netlify]

✅ Deploy Preview for mui-internal ready!

Name	Link
🔨 Latest commit	fe3d901
🔍 Latest deploy log	https://app.netlify.com/projects/mui-internal/deploys/6978d6e6c1fd080008ab9ad3
😎 Deploy Preview	https://deploy-preview-1074--mui-internal.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[mui-bot]

Bundle size report

Bundle	Parsed size	Gzip size
@base-ui-components/react	0B(0.00%)	0B(0.00%)
@mui/x-charts-pro	0B(0.00%)	0B(0.00%)

Details of bundle changes

---

Check out the code infra dashboard for more information about this PR.

[Copilot]

Pull request overview

Adds an ESLint rule to prevent throw statements from living inside branches whose execution depends on process.env.NODE_ENV, aligning with the goal of avoiding environment-dependent control-flow differences and supporting the earlier “skip error minification in dev-only branches” work.

Changes:

• Introduces mui/no-guarded-throw rule plus a dedicated RuleTester suite.
• Refactors require-dev-wrapper to reuse new shared NODE_ENV “inside if-check” utilities.
• Enables the new rule in the core ESLint config and wires it into the mui plugin export list.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
packages/test-utils/src/chaiPlugin.ts	Suppresses the new rule for a dev-only rethrow in an existing matcher implementation.
packages/code-infra/src/eslint/mui/rules/require-dev-wrapper.mjs	Uses a shared helper to detect NODE_ENV guard ancestry.
packages/code-infra/src/eslint/mui/rules/nodeEnvUtils.mjs	Adds shared utilities for detecting NODE_ENV binary comparisons and ancestry.
packages/code-infra/src/eslint/mui/rules/no-guarded-throw.mjs	Adds the new rule implementation that reports throws under NODE_ENV-dependent if branches.
packages/code-infra/src/eslint/mui/rules/no-guarded-throw.test.mjs	Adds tests covering common NODE_ENV-guard shapes (including composed conditions).
packages/code-infra/src/eslint/mui/index.mjs	Registers the new rule in the plugin entry point.
packages/code-infra/src/eslint/mui/config.mjs	Enables mui/no-guarded-throw as an error in the core config.

[code-infra-dashboard]

Deploy preview

https://deploy-preview-1074--mui-internal.netlify.app/

Bundle size

Total Size Change: 0B^(0.00%) - Total Gzip Change: 0B^(0.00%)
Files: 25 total (0 added, 0 removed, 0 changed)

Show details for 25 more bundles

@mui/internal-docs-infra/abstractCreateDemo parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/abstractCreateDemoClient parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/abstractCreateTypes parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/CodeControllerContext parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/CodeExternalsContext parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/CodeHighlighter parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/CodeHighlighter/errors parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/CodeHighlighter/types parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/CodeProvider parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/createDemoData parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/createDemoData/types parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/createSitemap parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/createSitemap/types parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/useCode parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/useCopier parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/useDemo parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/useErrors parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/useLocalStorageState parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/usePreference parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/useSearch parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/useSearch/types parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/useType parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/useTypes parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/useUrlHashState parsed: 0B^(0.00%) gzip: 0B^(0.00%)
@mui/internal-docs-infra/withDocsInfra parsed: 0B^(0.00%) gzip: 0B^(0.00%)

Details of bundle changes

Performance

Total duration: 16.91 ms -0.06 ms^(-0.4%) | Renders: 4 ⁽⁺⁰⁾ | Paint: 70.30 ms -3.50 ms^(-4.7%)

Test	Duration	Renders
DataGrid mount with paint timing	2.00 ms 🔺+0.09 ms(+4.7%)	1 (+0)
Counter click	4.15 ms ▼-0.09 ms(-2.0%)	2 (+0)
HeavyList mount	10.76 ms ▼-0.06 ms(-0.6%)	1 (+0)

details

---

Check out the code infra dashboard for more information about this PR.