Microsoft/TypeScript

Comment like jsx text emitted twice when preserve enabled

Open

#61.860 geöffnet am 13. Juni 2025

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (48.455 Stars) (6.726 Forks)batch import
BugDomain: JSX/TSXHelp Wanted

Beschreibung

🔎 Search Terms

comment like jsx text, preserve jsx

🕗 Version & Regression Information

  • This is the behavior in every version I tried, and I reviewed the FAQ for entries about _________

⏯ Playground Link

https://www.typescriptlang.org/play/?jsx=1#code/GYVwdgxgLglg9mABAQQA6oBQEpEG8C+AUMRAgM5SIBWZAHogLyIA8aqAfAPQBUiYcibp1w9EARkGdxAJgDMiUdMn5R-Sc05t2AbkJA

💻 Code

function App() {}
const jsx = <App>/* no */{/* 1 */ 123 /* 2 */}/* no */</App>;

🙁 Actual behavior

These duplicate sections will be rendered in the JSX text output:

"use strict";
function App() { }
const jsx = <App> /* no *//* no */{/* 1 */123 /* 2 */} /* no *//* no */</App>
//                ~~~~~~~                              ~~~~~~~

🙂 Expected behavior

"use strict";
function App() { }
- const jsx = <App> /* no *//* no */{/* 1 */123 /* 2 */} /* no *//* no */</App>
+ const jsx = <App> /* no */{/* 1 */123 /* 2 */} /* no */</App>

Additional information about the issue

No response

Contributor Guide

Tech Stack
typescriptreact
Domain
tooling
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
basic understanding of JSXTypeScript compiler architecture
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
Start by reading the TypeScript compiler source, specifically the JSX emitter logic in 'src/compiler/emitter.ts'. Look for how text nodes inside JSX are handled when 'jsx: preserve' is set. The issue shows that text before and after expression braces is emitted twice. The fix likely involves deduplicating text emission for JSX children. The playground link (https://www.typescriptlang.org/play/?jsx=1#code=...) reproduces the bug. Check for existing comments or PRs linked to issue #61860; as of now there are none. Confirm the expected behavior by comparing the actual output with the diff provided.