refactor(ast): remove Serialize impls for Identifier types

[overlookmotel]

Serialize can be derived for IdentifierName etc without explicit impl Serialize, as they just need serde to skip some fields. No changes to the JSON output, just a bit simpler.

[overlookmotel]

• refactor(ast): remove Serialize impls for Identifier types #2651 👈
• main

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

Join @overlookmotel and the rest of your teammates on Graphite

[codspeed-hq]

CodSpeed Performance Report

Merging #2651 will not alter performance

_{Comparing 03-09-refactor_ast_remove_Serialize_impls_for_Identifier_types (f42fe25) with main (32303b2)}

Summary

✅ 29 untouched benchmarks

[ArnaudBarre]

For info TSIndexSignatureName also need to be renamed to Identifier:

Example code: type C = {[key: string]: number}
oxc vs Babel

[overlookmotel]

I've opened a separate issue #2656 for your point above, Arnaud. Should not be hard to fix.

I don't believe it's directly relevant to this PR - does not block merging this if you agree it's a good change, Boshen.