bug(codegen): missing `,` when generating type parameters with jsx

[camc314]

Playground link

input:

const genericFn = <T,>(foo: T) => {
	type Bar = {};
};

output:

const genericFn = <T>(foo: T) => {
	type Bar = {};
};

expected output:

const genericFn = <T,>(foo: T) => {
	type Bar = {};
};

[overlookmotel]

Please excuse my lack of TypeScript knowledge, but does the trailing comma in <T,> have any semantic meaning?

If not, does it matter that we miss it off?

[Boshen]

It breaks for .tsx.

[camc314]

iirc, the parser will interpret <T> as an opening jsx tag and not as a type parameter.

it's fine in non-jsx files because <T> cant be an opening jsx element.