Skip to content

perf: upgrade to jsonc-parser 0.32#32864

Merged
bartlomieju merged 4 commits intodenoland:mainfrom
dsherret:perf_jsonc_parser_0_32
Mar 20, 2026
Merged

perf: upgrade to jsonc-parser 0.32#32864
bartlomieju merged 4 commits intodenoland:mainfrom
dsherret:perf_jsonc_parser_0_32

Conversation

@dsherret
Copy link
Contributor

@dsherret dsherret commented Mar 20, 2026 •
edited
Loading

https://dprint.github.io/jsonc-parser/dev/bench/

Benchmark Old (ns/iter) New (ns/iter) Speedup
citm_catalog_json_large_ast 37,836,170 26,929,560 1.41x (29% faster)
citm_catalog_json_large_value 51,306,270 27,293,510 1.88x (47% faster)
package_json_ast 33,690 14,682 2.29x (56% faster)
package_json_value 44,507 22,956 1.94x (48% faster)
tsconfig_json_ast 10,318 2,368 4.36x (77% faster)
tsconfig_json_value 10,644 2,340 4.55x (78% faster)

bartlomieju
bartlomieju approved these changes Mar 20, 2026
View reviewed changes
Copy link
Member

@bartlomieju bartlomieju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:verycool:

@bartlomieju bartlomieju merged commit 63550f0 into denoland:main Mar 20, 2026
113 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bartlomieju bartlomieju bartlomieju approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dsherret @bartlomieju