feat(oxfmt): Enable experimental package.json sorting by default

[leaysgur]

Fixes #16525, closes #16541

• Use sort-package-json: 0.0.2
• Enable by default, can be disabled by experimentalSortPackageJson: false

[leaysgur]

• refactor(oxfmt): Rename FormatFileSource > FormatFileStrategy #16635
• feat(oxfmt): Enable experimental package.json sorting by default #16593 👈 (View in Graphite)
• feat(oxfmt): Pass filepath field to prettier formatting #16591
• feat(oxfmt): Pass populated config to prettier formatting #16584
• feat(oxfmt): Pass raw config to prettier formatting #16582
• refactor(oxfmt): Extract read_to_string as core/utils #16581
• feat(oxfmt): Expose setupConfig(configJSON: string) napi callback #16579
• main

---

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

• 0-merge - adds this PR to the back of the merge queue
• hotfix - for urgent hot fixes, skip the queue and merge this PR next

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

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

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

[codspeed-hq]

CodSpeed Performance Report

Merging #16593 will not alter performance

_{Comparing 12-08-feat_oxfmt_enable_experimental_package.json_sorting_by_default (8649cfa) with main (0d5da26)¹}

Summary

✅ 42 untouched
⏩ 3 skipped²

Footnotes

1. No successful run was found on main (02f59ba) during the generation of this report, so 0d5da26 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

2. 3 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[leaysgur]

NOTE: Hold off for a bit.

Because, as per #16606, the infrastructure might change, and if we only pass the path to Prettier, we'd need to prepare a separate place for this feature to intervene.

[Copilot]

Pull request overview

This PR enables experimental package.json sorting by default using the sort-package-json crate version 0.0.2. The feature can be disabled via the experimentalSortPackageJson: false configuration option. The implementation integrates package.json sorting into the formatting pipeline before passing files to the external formatter (Prettier).

Key Changes:

• Added experimentalSortPackageJson configuration option (defaults to true)
• Integrated sort-package-json crate for automatic key sorting
• Refactored error handling in format.rs to use Result types for better composability

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
tasks/website_formatter/src/snapshots/schema_markdown.snap	Added documentation for the new experimentalSortPackageJson configuration field
npm/oxfmt/configuration_schema.json	Added JSON schema definition for experimentalSortPackageJson with default value true
crates/oxc_formatter/tests/snapshots/schema_json.snap	Updated schema snapshot to include the new configuration field
crates/oxc_formatter/src/service/oxfmtrc.rs	Added experimental_sort_package_json field with default true, removed it from Prettier config passthrough
apps/oxfmt/test/__snapshots__/external_formatter.test.ts.snap	Updated test snapshot showing package.json keys sorted (engines moved after dependencies)
apps/oxfmt/src/core/support.rs	Added is_package_json() method to detect package.json files by checking parser name and filename
apps/oxfmt/src/core/format.rs	Integrated sort-package-json before Prettier formatting, refactored to use Result-based error handling
apps/oxfmt/src/cli/format.rs	Plumbed through is_sort_package_json configuration from config loading to formatter initialization
apps/oxfmt/Cargo.toml	Added sort-package-json = "0.0.2" dependency
Cargo.lock	Updated lockfile with sort-package-json dependency and its transitive dependencies

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[leaysgur]

I will make some minor adjustments to improve readability.

[leaysgur]

Reviews will be covered by: #16635

[graphite-app]

Merge activity

• Dec 10, 1:47 AM UTC: Dunqing added this pull request to the Graphite merge queue.
• Dec 10, 1:53 AM UTC: Merged by the Graphite merge queue.