Skip to content

build: stop declaring @babel/plugin-syntax-jsx directly - #828

Merged
cigamit merged 1 commit into
ctrliq:mainfrom
blaipr:build/drop-babel-plugin-syntax-jsx
Sep 7, 2026
Merged

cigamit merged 1 commit into
ctrliq:mainfrom
blaipr:build/drop-babel-plugin-syntax-jsx

Conversation

@blaipr

@blaipr blaipr commented Sep 6, 2026

Copy link
Copy Markdown
Contributor
SUMMARY

Nothing here references @babel/plugin-syntax-jsx. The babel configuration lives inline in webpack.config.js and names @babel/preset-env, @babel/preset-react, @babel/plugin-transform-runtime and the local jsx-compat-plugin. JSX syntax support arrives with preset-react, not from a plugin listed here.

It is a regular dependency of @babel/plugin-transform-react-jsx, babel-plugin-styled-components and jest-snapshot, so this changes package.json only: the package stays in node_modules at 8.0.1 and the install is identical.

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • UI
ADDITIONAL INFORMATION

Verified with a production build rather than lint. The whole UI is JSX, so a syntax plugin that actually mattered would fail compilation outright rather than subtly:

  • npm run build: succeeds.
  • make ui-lint: clean.
  • make ui-test-general: 1038 tests, all passing.
  • make ui-test-screens: 1941 tests, all passing.

@cigamit cigamit self-assigned this Sep 7, 2026
@cigamit cigamit added the dependencies Pull requests that update a dependency file label Sep 7, 2026
cigamit
cigamit previously approved these changes Sep 7, 2026
@cigamit cigamit added the Needs triage When a Issue needs to be researched or a PR has an issue that needs fixing before merging label Sep 7, 2026
Nothing here references it. The babel configuration lives inline in
webpack.config.js and names @babel/preset-env, @babel/preset-react,
@babel/plugin-transform-runtime and the local jsx-compat-plugin. JSX syntax
support arrives with preset-react rather than from a plugin listed here.

It is a regular dependency of @babel/plugin-transform-react-jsx,
babel-plugin-styled-components and jest-snapshot, so this changes
package.json only: the package stays in node_modules at 8.0.1 and the install
is identical.

Verified with a production build rather than lint, since the whole UI is JSX
and a syntax plugin that mattered would fail compilation outright.
@blaipr
blaipr force-pushed the build/drop-babel-plugin-syntax-jsx branch from 2de13c0 to 63fec15 Compare September 7, 2026 06:42
@cigamit cigamit removed the Needs triage When a Issue needs to be researched or a PR has an issue that needs fixing before merging label Sep 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Development

Successfully merging this pull request may close these issues.

2 participants