refactor(ast): move StringLiteral definition higher up

[overlookmotel]

Pure refactor. StringLiteral definition was sandwiched in the middle of RegExp-related code. Move it higher up in literal.rs.

All the rest of the diff is just re-ordering generated code.

[graphite-app]

Your org has enabled the Graphite merge queue for merging into main

Add the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[overlookmotel]

• style(ast): add line break #7271
• feat(ast): serialize StringLiterals to ESTree without raw #7263
• refactor(ast): move StringLiteral definition higher up #7270 👈
• main

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

Join @overlookmotel and the rest of your teammates on Graphite

[overlookmotel]

Merge activity

• Nov 13, 8:35 AM EST: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Nov 13, 8:37 AM EST: A user added this pull request to the Graphite merge queue.
• Nov 13, 8:42 AM EST: A user merged this pull request with the Graphite merge queue.

[overlookmotel]

This change was originally part of #7263. Have split it out into a separate PR to make the substantive changes in #7263 easier to review.

[codspeed-hq]

CodSpeed Performance Report

Merging #7270 will not alter performance

_{Comparing 11-13-refactor_ast_move_stringliteral_definition_higher_up (de472ca) with main (834c94d)}

Summary

✅ 30 untouched benchmarks