fix(transformer/react): spread props optimization breaks __self and __source injection (#13009)

[Dunqing]

• Fixes transformer: classic runtime jsx with a spread prop is transformed incorrectly #13009

The optimization that converts {...props} to props when it's the only attribute was incorrectly applied even when jsx_self or jsx_source plugins are enabled. This caused __self and __source props to not be merged properly,
resulting in incorrect transformation output in development mode.

This fix disables the optimization when these dev plugins are active, ensuring spread props are preserved for proper merging.

[Dunqing]

• fix(transformer/react): spread props optimization breaks __self and __source injection (#13009) #13020 👈 (View in Graphite)
• main

---

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

• 0-merge - adds this PR to the back of the merge queue
• hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

[Copilot]

Pull Request Overview

This PR fixes a bug where the JSX transformer's spread props optimization was incorrectly applied when development plugins (jsx_self and jsx_source) are enabled. The optimization that converts {...props} to props when it's the only attribute was preventing proper merging of injected __self and __source props in development mode.

Key changes:

• Added condition to disable spread props optimization when dev plugins are active
• Added test case to verify correct behavior with classic runtime and development plugins
• Updated test snapshots to reflect the new passing test

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
crates/oxc_transformer/src/jsx/jsx_impl.rs	Added condition to preserve spread syntax when jsx_self or jsx_source plugins are enabled
tasks/transform_conformance/tests/babel-plugin-transform-react-jsx/test/fixtures/spread-props-classic/input.jsx	Test input with spread props in JSX element
tasks/transform_conformance/tests/babel-plugin-transform-react-jsx/test/fixtures/spread-props-classic/options.json	Test configuration enabling development transform with classic runtime
tasks/transform_conformance/tests/babel-plugin-transform-react-jsx/test/fixtures/spread-props-classic/output.js	Expected output showing preserved spread syntax with injected __self and __source props
tasks/transform_conformance/snapshots/oxc.snap.md	Updated test count from 182/302 to 183/303 passing tests

[codspeed-hq]

CodSpeed Instrumentation Performance Report

Merging #13020 will not alter performance

_{Comparing 08-12-fix_transformer_react_spread_props_optimization_breaks___self_and___source_injection_13009_ (6fe0bb5) with main (c58a699)}

Summary

✅ 34 untouched benchmarks

[overlookmotel]

Makes sense!

Pushed a commit to update conformance snapshots (just the usual churn in TS semantic). Hopefully it'll merge now.

[overlookmotel]

Merge activity

• Aug 12, 11:54 AM UTC: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Aug 12, 11:59 AM UTC: overlookmotel added this pull request to the Graphite merge queue.
• Aug 12, 12:05 PM UTC: Merged by the Graphite merge queue.