fix(codegen): print type arguments for JSXOpeningElement by camc314 · Pull Request #20127 · oxc-project/oxc

camc314 · 2026-03-08T19:12:01Z

No description provided.

camc314 · 2026-03-08T19:12:16Z

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

Fixes TSX code generation to preserve and print generic type arguments on JSXOpeningElement (e.g. <Foo<T> />), aligning output with the parsed AST and preventing loss of type information during codegen.

Changes:

Emit opening_element.type_arguments when generating JSX opening tags.
Add an integration test covering TSX JSX opening element type arguments.

Reviewed changes

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

File	Description
crates/oxc_codegen/src/gen.rs	Prints `JSXOpeningElement.type_arguments` immediately after the element name during JSX codegen.
crates/oxc_codegen/tests/integration/ts.rs	Adds a regression test asserting `<Foo<T> />` round-trips with type arguments intact.

You can also share your feedback on Copilot code review. Take the survey.

codspeed-hq · 2026-03-08T19:18:29Z

Merging this PR will not alter performance

✅ 53 untouched benchmarks
⏩ 3 skipped benchmarks¹

_{Comparing c/03-08-fix_codegen_print_type_arguments_for_jsxopeningelement (24faba2) with c/03-08-fix_codegen_preserve_parens_for_tsnonnullexpression (9081dc4)²}

3 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩
No successful run was found on c/03-08-fix_codegen_preserve_parens_for_tsnonnullexpression (8d813c8) during the generation of this report, so 261ff51 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

graphite-app · 2026-03-09T00:59:35Z

Merge activity

Mar 9, 12:59 AM UTC: Dunqing added this pull request to the Graphite merge queue.
Mar 9, 1:12 AM UTC: Merged by the Graphite merge queue.

…Gen impls We've had multiple bugs where new fields were added to AST structs but the codegen `Gen`/`GenExpr` implementations forgot to handle them (e.g., #20124 `declare` for TSInterfaceDeclaration, #20125 parens for TSNonNullExpression, #20127 type_arguments for JSXOpeningElement, #20128 type for TSImportEqualsDeclaration). These bugs are silent — the code compiles fine but produces incorrect output. This test uses `syn` to parse AST struct definitions and cross-reference them against the codegen `Gen`/`GenExpr` implementations, detecting any struct fields that are never accessed. It catches: - Direct `self.field` accesses in Gen/GenExpr impl blocks - Field accesses via helper functions (e.g., `print_if(stmt: &IfStatement)`) - Field accesses through enum wrapper match arms (e.g., `Binaryish::Binary(e) => e.left`) Fields that are intentionally not printed (internal metadata like `node_id`, `scope_id`, or fields accessed via methods like `as_str()`) are tracked in explicit skip lists with comments explaining why. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

### 🚀 Features - e8547cc parser: Report error for using declarations in ambient contexts (#19934) (camc314) - 8345318 allocator: Add methods for boxed slices `ArenaBox<[T]>` (#19968) (overlookmotel) - f83be30 allocator: Add `Vec::push_fast` method (#19959) (overlookmotel) ### 🐛 Bug Fixes - 291d867 transformer_plugins: Unwrap ChainExpression after define replacement removes optional markers (#20058) (IWANABETHATGUY) - 36b2e56 codegen: Print type for TSImportEqualsDeclaration (#20128) (camc314) - 5a246ec codegen: Print type arguments for JSXOpeningElement (#20127) (camc314) - a40870e codegen: Preserve parens for TSNonNullExpression (#20125) (camc314) - ae830b2 codegen: Print `declare` for `TSInterfaceDeclaration` (#20124) (camc314) - 92cfb14 linter/plugins: Fix types for `walkProgram` and `walkProgramWithCfg` (#20081) (overlookmotel) - ee0491e apps,napi: Explicitly specify libs in tsconfigs (#20071) (camc314) - 588009e codegen: Print `static` keyword for TSIndexSignature (#19755) (Dunqing) - 5a8799c codegen: Print `with_clause` for `ExportNamedDeclaration` (#20002) (Dunqing) - 7502afe parser: Correct capacity for tokens `Vec` (#19967) (overlookmotel) ### ⚡ Performance - 4ea8f9a napi: Remove `napi_build::setup()` from `oxc_napi` to avoid redundant rebuilds (#20094) (Boshen) - 2baa5fb napi: Unify build-test profile to coverage for cache sharing (#20090) (Boshen) - 8ba61dd parser: Make pushing tokens faster (#19960) (overlookmotel) Co-authored-by: Boshen <1430279+Boshen@users.noreply.github.com>

github-actions bot added the A-codegen Area - Code Generation label Mar 8, 2026

camc314 mentioned this pull request Mar 8, 2026

fix(codegen): print declare for TSInterfaceDeclaration #20124

Merged

github-actions bot added the C-bug Category - Bug label Mar 8, 2026

camc314 mentioned this pull request Mar 8, 2026

fix(codegen): preserve parens for TSNonNullExpression #20125

Merged

camc314 marked this pull request as ready for review March 8, 2026 19:13

Copilot AI review requested due to automatic review settings March 8, 2026 19:13

Copilot started reviewing on behalf of camc314 March 8, 2026 19:13 View session

camc314 mentioned this pull request Mar 8, 2026

fix(codegen): print type for TSImportEqualsDeclaration #20128

Merged

Copilot AI reviewed Mar 8, 2026

View reviewed changes

camc314 force-pushed the c/03-08-fix_codegen_preserve_parens_for_tsnonnullexpression branch from 9081dc4 to 8d813c8 Compare March 8, 2026 21:49

camc314 force-pushed the c/03-08-fix_codegen_print_type_arguments_for_jsxopeningelement branch from 7b10e62 to 24faba2 Compare March 8, 2026 21:49

Dunqing approved these changes Mar 9, 2026

View reviewed changes

graphite-app bot added the 0-merge Merge with Graphite Merge Queue label Mar 9, 2026

fix(codegen): print type arguments for JSXOpeningElement (#20127)

5a246ec

graphite-app bot force-pushed the c/03-08-fix_codegen_preserve_parens_for_tsnonnullexpression branch from 8d813c8 to a40870e Compare March 9, 2026 01:00

graphite-app bot force-pushed the c/03-08-fix_codegen_print_type_arguments_for_jsxopeningelement branch from 24faba2 to 5a246ec Compare March 9, 2026 01:01

Base automatically changed from c/03-08-fix_codegen_preserve_parens_for_tsnonnullexpression to main March 9, 2026 01:12

graphite-app bot removed the 0-merge Merge with Graphite Merge Queue label Mar 9, 2026

graphite-app bot merged commit 5a246ec into main Mar 9, 2026
22 checks passed

graphite-app bot deleted the c/03-08-fix_codegen_print_type_arguments_for_jsxopeningelement branch March 9, 2026 01:12

Dunqing mentioned this pull request Mar 9, 2026

test(codegen): add lint test to detect missing AST field handling #20131

Open

3 tasks

Boshen mentioned this pull request Mar 9, 2026

release(crates): oxc v0.117.0 #20142

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix(codegen): print type arguments for JSXOpeningElement#20127

fix(codegen): print type arguments for JSXOpeningElement#20127
graphite-app[bot] merged 1 commit intomainfrom
c/03-08-fix_codegen_print_type_arguments_for_jsxopeningelement

camc314 commented Mar 8, 2026

Uh oh!

camc314 commented Mar 8, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

codspeed-hq bot commented Mar 8, 2026 •

edited

Loading

Uh oh!

graphite-app bot commented Mar 9, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

camc314 commented Mar 8, 2026

Uh oh!

camc314 commented Mar 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

How to use the Graphite Merge Queue

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

codspeed-hq bot commented Mar 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merging this PR will not alter performance

Footnotes

Uh oh!

graphite-app bot commented Mar 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

camc314 commented Mar 8, 2026 •

edited

Loading

codspeed-hq bot commented Mar 8, 2026 •

edited

Loading

graphite-app bot commented Mar 9, 2026 •

edited

Loading