Skip to content

feat: Drop requirement to have a user-provided babel config#15

Merged
frankieyan merged 3 commits into
mainfrom
frankie/use-standalone-babel-config
Jan 16, 2026
Merged

feat: Drop requirement to have a user-provided babel config#15
frankieyan merged 3 commits into
mainfrom
frankie/use-standalone-babel-config

Conversation

@frankieyan
Copy link
Copy Markdown
Member

This PR removes the need for users to provide a Babel config, as there is a wide range of ways the React Compiler babel plugin can be set up. Instead, this sets up a standalone Babel config solely focused on detecting compiler errors.

References

Closes #7

Test plan

Run the tool against the included fixtures:

  • Run npm run build
  • Navigate to src/__fixtures__/sample-project
  • Run node ../../../dist/index.mjs --overwrite
  • It reports errors in the files and creates a .react-compiler.rec.json in src/__fixtures__/sample-project/src

@frankieyan frankieyan self-assigned this Jan 16, 2026
Copy link
Copy Markdown

@doistbot-app doistbot-app Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes successfully implement the goal of making the tool standalone by internalizing Babel dependencies and removing the need for a user-provided configuration. The documentation and lockfile updates are consistent with this architectural shift.

@frankieyan frankieyan added the 👀 Show PR PR must be reviewed before or after merging label Jan 16, 2026
@frankieyan frankieyan requested review from a team and engfragui and removed request for a team January 16, 2026 07:33
@frankieyan frankieyan merged commit 82ecfd4 into main Jan 16, 2026
2 checks passed
@frankieyan frankieyan deleted the frankie/use-standalone-babel-config branch January 16, 2026 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👀 Show PR PR must be reviewed before or after merging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support wider range of babel configs

1 participant