refactor(formatter): Export unified way to get_parse_options

[leaysgur]

Export oxc_formatter::get_parse_options to get this options everywhere.

ParseOpitons {
    parse_regular_expression: false,
    // Enable all syntax features
    allow_v8_intrinsics: true,
    allow_return_outside_function: true,
    // `oxc_formatter` expects this to be false
    preserve_parens: false,
}

[leaysgur]

• fix(tasks/prettier_conformance): Skip tests which cannot parse #15025
• refactor(formatter): Export unified way to get_parse_options #15027 👈 (View in Graphite)
• 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.

[Copilot]

Pull Request Overview

This PR refactors the formatter codebase by centralizing parse options configuration. A new get_parse_options() function is introduced in oxc_formatter to provide a unified way to configure parser options for formatting operations, eliminating duplicated configuration blocks across multiple files.

Key changes:

• Introduced get_parse_options() function in crates/oxc_formatter/src/service/parse_utils.rs that returns standardized ParseOptions for formatting
• Replaced 11 instances of inline ParseOptions configuration with calls to the new helper function
• Moved oxc_parser from dev-dependencies to regular dependencies in oxc_formatter/Cargo.toml

Reviewed Changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
crates/oxc_formatter/src/service/parse_utils.rs	Defines new get_parse_options() function with standard parser configuration for formatting
crates/oxc_formatter/src/service/mod.rs	Updates module exports to include parse_utils
crates/oxc_formatter/src/lib.rs	Exports the new get_parse_options() function publicly
crates/oxc_formatter/Cargo.toml	Moves oxc_parser from dev-dependencies to dependencies
tasks/prettier_conformance/src/lib.rs	Replaces inline ParseOptions with get_parse_options() call
tasks/benchmark/benches/formatter.rs	Replaces inline ParseOptions with get_parse_options() call
napi/playground/src/lib.rs	Replaces inline ParseOptions with get_parse_options() call
crates/oxc_language_server/src/formatter/server_formatter.rs	Replaces inline ParseOptions with get_parse_options() call
crates/oxc_formatter/tests/ir_transform/mod.rs	Replaces inline ParseOptions with get_parse_options() call
crates/oxc_formatter/tests/fixtures/mod.rs	Replaces inline ParseOptions with get_parse_options() call
crates/oxc_formatter/examples/sort_imports.rs	Replaces inline ParseOptions with get_parse_options() call
crates/oxc_formatter/examples/formatter.rs	Replaces inline ParseOptions with get_parse_options() call
apps/oxfmt/src/service.rs	Replaces inline ParseOptions with get_parse_options() call

---

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

[codspeed-hq]

CodSpeed Performance Report

Merging #15027 will not alter performance

_{Comparing 10-28-refactor_formatter_export_unified_way_to_get_parse_options (cd0ef1d) with main (4904710)¹}

Summary

✅ 37 untouched

Footnotes

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

[graphite-app]

Merge activity

• Oct 28, 9:26 AM UTC: Dunqing added this pull request to the Graphite merge queue.
• Oct 28, 9:31 AM UTC: Merged by the Graphite merge queue.

[Dunqing]

Looks good to me! We probably need to export a parse method because we also need to align the SourceType everywhere as well.

[leaysgur]

We probably need to export a parse method

Ah~, make sense! I'll do it tomorrow.

[Dunqing]

Ah~, make sense! I'll do it tomorrow.

But it will bundle the parser, not sure if this should be done. 🥲

[leaysgur]

Now I just remembered that I had noticed that too and given up on doing that! 😅

Let's leave it for now.