No successful run was found on main (32f5e6e) during the generation of this report, so af3c2d3 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.
fix(es/transforms/react): Fix all failing tests in VisitMutHook refactoring
Fixed remaining issues from VisitMutHook conversion:
1. jsx_self: Changed to use context stack instead of VisitMut for proper state management
- enter/exit hooks now properly track in_constructor and in_derived_class state
- Fixed duplicate __self attribute issue
2. jsx: Changed from enter_expr to exit_expr for transformation
- Allows jsx_src and jsx_self to run before jsx transformation
- Added special handling for attribute JSXElements in development mode
- Attribute JSXElements get __self fixed from `void 0` to `this`
3. refresh: Added visit_module() to collect @refresh reset comments
- Fixed missing shouldResetModule parameter in hook signatures
4. display_name, jsx_src, pure_annotations: Clean VisitMutHook implementation
All tests passing:
- swc_ecma_transforms_react: 268/268 (100%)
- swc_ecma_transforms_typescript: 190/190 (100%)
- swc_ecma_transforms: 5036/5036 (100%)
- swc projects/tsc: 861/861 (100%), 4580/4580 (100%)
Total: 10,935/10,935 tests passing
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Sonnet 4.5 (1M context) <noreply@anthropic.com>
661177f
2 days ago
by kdy1
-0.01%
refactor(es/transforms/react): Address code review feedback
Improvements based on code review:
1. Removed all debug eprintln! statements from jsx/mod.rs
- Cleaned up temporary debugging code
2. Extracted duplicated attribute processing logic
- Created process_attr_value() helper method
- Reduces code duplication between JSXAttrName::Ident and JSXNamespacedName branches
- Improves maintainability
3. Added comprehensive documentation for hook execution order
- Documented why Jsx implements both VisitMut and VisitMutHook
- Explained enter_expr/exit_expr split for proper hook composition
- Clarified that jsx_src and jsx_self must run before jsx transformation
4. Improved error messages
- Changed .unwrap() to .expect() in jsx_self/mod.rs with descriptive message
- Better error context for debugging if stack invariant is violated
All tests still passing (268/268).
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Sonnet 4.5 (1M context) <noreply@anthropic.com>
b604771
1 day ago
by kdy1
+0.01%
clippy
e3ec3dd
1 day ago
by kdy1
-0.02%
clippy
4b480ee
1 day ago
by kdy1
+0.01%
Change package version to major for swc_ecma_transforms_react
Updated package version from patch to major for swc_ecma_transforms_react.