Skip to content

perf(estree/tokens): serialize with ESTree not serde#19725

Merged
graphite-app[bot] merged 1 commit intomainfrom
om/02-22-perf_estree_tokens_serialize_with_estree_not_serde_
Feb 25, 2026
Merged

perf(estree/tokens): serialize with ESTree not serde#19725
graphite-app[bot] merged 1 commit intomainfrom
om/02-22-perf_estree_tokens_serialize_with_estree_not_serde_

Conversation

@overlookmotel
Copy link
Member

@overlookmotel overlookmotel commented Feb 25, 2026

Serialize tokens using ESTree trait instead of serde. It's faster. +37% on ESTree tokens benchmark.

@github-actions github-actions bot added the C-performance Category - Solution not expected to change functional behavior, only performance label Feb 25, 2026
This was referenced Feb 25, 2026
@graphite-app graphite-app bot added the 0-merge Merge with Graphite Merge Queue label Feb 25, 2026
@overlookmotel overlookmotel force-pushed the om/02-22-feat_estree_oxc_estree_crate_export_config_and_formatter_types branch from 17a331c to 85a976b Compare February 25, 2026 19:40
@overlookmotel overlookmotel force-pushed the om/02-22-perf_estree_tokens_serialize_with_estree_not_serde_ branch from 71a5f51 to 8bf8df6 Compare February 25, 2026 19:40
@graphite-app graphite-app bot force-pushed the om/02-22-feat_estree_oxc_estree_crate_export_config_and_formatter_types branch from 85a976b to 7400512 Compare February 25, 2026 19:59
@graphite-app graphite-app bot requested a review from camc314 as a code owner February 25, 2026 19:59
@graphite-app graphite-app bot force-pushed the om/02-22-perf_estree_tokens_serialize_with_estree_not_serde_ branch from 8bf8df6 to e091eda Compare February 25, 2026 20:00
@graphite-app
Copy link
Contributor

graphite-app bot commented Feb 25, 2026

Merge activity

@overlookmotel overlookmotel force-pushed the om/02-22-perf_estree_tokens_serialize_with_estree_not_serde_ branch from e091eda to eddb96f Compare February 25, 2026 20:25
@overlookmotel overlookmotel force-pushed the om/02-22-feat_estree_oxc_estree_crate_export_config_and_formatter_types branch from 7400512 to 1680c60 Compare February 25, 2026 20:25
Serialize tokens using `ESTree` trait instead of `serde`. It's faster. +37% on ESTree tokens benchmark.
@graphite-app graphite-app bot force-pushed the om/02-22-feat_estree_oxc_estree_crate_export_config_and_formatter_types branch from 1680c60 to 233f947 Compare February 25, 2026 20:34
@graphite-app graphite-app bot force-pushed the om/02-22-perf_estree_tokens_serialize_with_estree_not_serde_ branch from eddb96f to bc6507f Compare February 25, 2026 20:35
@graphite-app graphite-app bot removed the 0-merge Merge with Graphite Merge Queue label Feb 25, 2026
Base automatically changed from om/02-22-feat_estree_oxc_estree_crate_export_config_and_formatter_types to main February 25, 2026 20:51
@graphite-app graphite-app bot merged commit bc6507f into main Feb 25, 2026
23 checks passed
@graphite-app graphite-app bot deleted the om/02-22-perf_estree_tokens_serialize_with_estree_not_serde_ branch February 25, 2026 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-linter-plugins Area - Linter JS plugins A-parser Area - Parser C-performance Category - Solution not expected to change functional behavior, only performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants