refactor(oxlint/lsp): move lsp code to `apps/oxlint`, remove lsp linter feature flag by Sysix · Pull Request #17463 · oxc-project/oxc

Sysix · 2025-12-29T19:18:45Z

Sysix · 2025-12-29T19:19:04Z

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 · 2025-12-29T19:25:44Z

CodSpeed Performance Report

Merging #17463 will not alter performance

_{Comparing 12-29-refactor_oxlint_lsp_move_lsp_code_to_apps_oxlint_remove_lsp_linter_feature_flag (eb8bf94) with main (5cc2db6)¹}

Summary

✅ 42 untouched
⏩ 3 skipped²

No successful run was found on main (9ad0f29) during the generation of this report, so 5cc2db6 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩
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. ↩

apps/oxlint/src/snapshots/fixtures_--tsconfig oxc__tsconfig.json@oxlint.snap

…ter is passed (#17498) somehow the refactoring in #17463 shows some js plugin errors for CLI tests. See https://github.com/oxc-project/oxc/actions/runs/20587373437/job/59126059421 The reason was this code part: https://github.com/oxc-project/oxc/blob/0222857e629cc286aabb388adf418c24299b1896/crates/oxc_linter/src/config/config_builder.rs#L166-L176

camc314 · 2025-12-31T21:09:05Z

Merge activity

Dec 31, 9:09 PM UTC: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
Dec 31, 9:12 PM UTC: camc314 added this pull request to the Graphite merge queue.
Dec 31, 9:18 PM UTC: Merged by the Graphite merge queue.

…er feature flag (#17463) closes #15119

github-actions bot added A-linter Area - Linter A-cli Area - CLI A-editor Area - Editor and Language Server A-formatter Area - Formatter C-cleanup Category - technical debt or refactoring. Solution not expected to change behavior labels Dec 29, 2025

Sysix mentioned this pull request Dec 29, 2025

refactor(oxfmt/lsp): move lsp code to apps/oxfmt, remove lsp formatter feature flag #17462

Merged

Sysix commented Dec 29, 2025

View reviewed changes

apps/oxlint/src/snapshots/fixtures_--tsconfig oxc__tsconfig.json@oxlint.snap Outdated Show resolved Hide resolved

graphite-app bot changed the base branch from 12-29-refactor_oxfmt_lsp_move_lsp_code_to_apps_oxfmt_remove_lsp_formatter_feature_flag to graphite-base/17463 December 30, 2025 02:18

graphite-app bot force-pushed the graphite-base/17463 branch from d404ffa to 435e955 Compare December 30, 2025 02:29

graphite-app bot force-pushed the 12-29-refactor_oxlint_lsp_move_lsp_code_to_apps_oxlint_remove_lsp_linter_feature_flag branch from f110194 to 4608d36 Compare December 30, 2025 02:29

graphite-app bot changed the base branch from graphite-base/17463 to main December 30, 2025 02:30

graphite-app bot force-pushed the 12-29-refactor_oxlint_lsp_move_lsp_code_to_apps_oxlint_remove_lsp_linter_feature_flag branch from 4608d36 to a68da80 Compare December 30, 2025 02:30

Sysix changed the base branch from main to graphite-base/17463 December 30, 2025 16:39

Sysix force-pushed the 12-29-refactor_oxlint_lsp_move_lsp_code_to_apps_oxlint_remove_lsp_linter_feature_flag branch from a68da80 to 0f201d0 Compare December 30, 2025 16:39

Sysix changed the base branch from graphite-base/17463 to 12-30-fix_oxlint_do_notenable_external_plugin_store_when_no_external_linter_is_passed December 30, 2025 16:39

Sysix mentioned this pull request Dec 30, 2025

fix(oxlint): do not enable external plugin store when no external linter is passed #17498

Merged

Sysix marked this pull request as ready for review December 30, 2025 16:53

Sysix requested a review from camc314 as a code owner December 30, 2025 16:53

Sysix force-pushed the 12-30-fix_oxlint_do_notenable_external_plugin_store_when_no_external_linter_is_passed branch from bd672ad to 36e7a62 Compare December 31, 2025 20:52

Sysix force-pushed the 12-29-refactor_oxlint_lsp_move_lsp_code_to_apps_oxlint_remove_lsp_linter_feature_flag branch from 0f201d0 to 25698dd Compare December 31, 2025 20:52

graphite-app bot changed the base branch from 12-30-fix_oxlint_do_notenable_external_plugin_store_when_no_external_linter_is_passed to graphite-base/17463 December 31, 2025 20:59

graphite-app bot force-pushed the graphite-base/17463 branch from 36e7a62 to 9ad0f29 Compare December 31, 2025 21:05

graphite-app bot force-pushed the 12-29-refactor_oxlint_lsp_move_lsp_code_to_apps_oxlint_remove_lsp_linter_feature_flag branch from 25698dd to 52117f8 Compare December 31, 2025 21:05

graphite-app bot changed the base branch from graphite-base/17463 to main December 31, 2025 21:06

graphite-app bot force-pushed the 12-29-refactor_oxlint_lsp_move_lsp_code_to_apps_oxlint_remove_lsp_linter_feature_flag branch from 52117f8 to eb8bf94 Compare December 31, 2025 21:06

camc314 approved these changes Dec 31, 2025

View reviewed changes

camc314 self-assigned this Dec 31, 2025

camc314 added the 0-merge Merge with Graphite Merge Queue label Dec 31, 2025

refactor(oxlint/lsp): move lsp code to apps/oxlint, remove lsp lint…

99b0834

…er feature flag (#17463) closes #15119

graphite-app bot force-pushed the 12-29-refactor_oxlint_lsp_move_lsp_code_to_apps_oxlint_remove_lsp_linter_feature_flag branch from eb8bf94 to 99b0834 Compare December 31, 2025 21:12

graphite-app bot merged commit 99b0834 into main Dec 31, 2025
24 checks passed

graphite-app bot deleted the 12-29-refactor_oxlint_lsp_move_lsp_code_to_apps_oxlint_remove_lsp_linter_feature_flag branch December 31, 2025 21:18

graphite-app bot removed the 0-merge Merge with Graphite Merge Queue label Dec 31, 2025

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

refactor(oxlint/lsp): move lsp code to `apps/oxlint`, remove lsp linter feature flag#17463

refactor(oxlint/lsp): move lsp code to `apps/oxlint`, remove lsp linter feature flag#17463
graphite-app[bot] merged 1 commit intomainfrom
12-29-refactor_oxlint_lsp_move_lsp_code_to_apps_oxlint_remove_lsp_linter_feature_flag

Sysix commented Dec 29, 2025 •

edited

Loading

Uh oh!

Sysix commented Dec 29, 2025 •

edited

Loading

Uh oh!

codspeed-hq bot commented Dec 29, 2025 •

edited

Loading

Uh oh!

Uh oh!

camc314 commented Dec 31, 2025 •

edited by graphite-app bot

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

Sysix commented Dec 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Sysix commented Dec 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

How to use the Graphite Merge Queue

Uh oh!

codspeed-hq bot commented Dec 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CodSpeed Performance Report

Merging #17463 will not alter performance

Summary

Footnotes

Uh oh!

Uh oh!

camc314 commented Dec 31, 2025 • edited by graphite-app bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge activity

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Sysix commented Dec 29, 2025 •

edited

Loading

Sysix commented Dec 29, 2025 •

edited

Loading

codspeed-hq bot commented Dec 29, 2025 •

edited

Loading

camc314 commented Dec 31, 2025 •

edited by graphite-app bot

Loading