Skip to content

Drop Biome.js for oxfmt and oxlint#3333

Merged
Kocal merged 1 commit intosymfony:2.xfrom
Kocal:switch-from-biomejs-to-oxc
Feb 3, 2026
Merged

Drop Biome.js for oxfmt and oxlint#3333
Kocal merged 1 commit intosymfony:2.xfrom
Kocal:switch-from-biomejs-to-oxc

Conversation

@Kocal
Copy link
Member

@Kocal Kocal commented Feb 3, 2026

Q A
Bug fix? no
New feature? no
Deprecations? no
Documentation? no
Issues Fix #...
License MIT

More powerful, more languages supported, and faster

@Kocal Kocal self-assigned this Feb 3, 2026
@Kocal Kocal force-pushed the switch-from-biomejs-to-oxc branch from 924fe4b to 4cd9032 Compare February 3, 2026 22:03
@Kocal Kocal requested a review from Copilot February 3, 2026 22:03
@carsonbot carsonbot added the Status: Needs Review Needs to be reviewed label Feb 3, 2026
@carsonbot carsonbot changed the title Drop Biome.js for oxc (format & lint) Drop Biome.js for oxc (format & lint) Feb 3, 2026
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request replaces Biome.js with oxc (oxlint and oxfmt) as the formatting and linting toolchain for the JavaScript/TypeScript codebase. The change is motivated by oxc's superior performance according to its benchmarks.

Changes:

  • Replaced @biomejs/biome with oxfmt and oxlint packages in the root package.json
  • Added .oxlintrc.json and .oxfmtrc.json configuration files for the new tooling
  • Removed check and ci scripts from all UX package.json files across the monorepo
  • Applied automated formatting changes throughout the codebase using oxc's formatter
  • Updated CONTRIBUTING.md documentation to reflect the new format and lint commands

Reviewed changes

Copilot reviewed 131 out of 203 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
package.json Replaced Biome dependency with oxfmt and oxlint; updated scripts
.oxlintrc.json New linting configuration for oxc
.oxfmtrc.json New formatting configuration for oxc
src/*/assets/package.json Removed Biome-specific scripts from all UX packages
test/playwright-helpers.ts Fixed missing semicolons and removed extra parentheses
Multiple files Applied consistent formatting changes (quotes, spacing, indentation)
CONTRIBUTING.md Updated documentation to reflect new formatter and linter commands
src/Translator/CHANGELOG.md Formatting issue with line 55 combining multi-line bullet points
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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

@Kocal Kocal force-pushed the switch-from-biomejs-to-oxc branch from f20456f to 36412c2 Compare February 3, 2026 22:09
@Kocal Kocal changed the title Drop Biome.js for oxc (format & lint) Drop Biome.js for oxfmt and oxlint Feb 3, 2026
@Kocal Kocal marked this pull request as ready for review February 3, 2026 22:09
@Kocal Kocal force-pushed the switch-from-biomejs-to-oxc branch from 36412c2 to dded126 Compare February 3, 2026 22:14
@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2026

📊 Packages dist files size difference

Thanks for the PR! Here is the difference in size of the packages dist files between the base branch and the PR.
Please review the changes and make sure they are expected.

FileBefore (Size / Gzip)After (Size / Gzip)
LiveComponent
live_controller.js 99.07 kB / 21.35 kB 99.07 kB0% / 21.35 kB0%
Map
abstract_map_controller.d.ts 7.96 kB / 1.51 kB 7.97 kB0% / 1.51 kB0%
abstract_map_controller.js 5.45 kB / 1.47 kB 5.47 kB0% / 1.48 kB+1% 📈
Map (Bridge Google)
map_controller.d.ts 3.42 kB / 900 B 3.43 kB0% / 894 B-1% 📉
map_controller.js 14.02 kB / 3.29 kB 14.11 kB+1% 📈 / 3.29 kB0%
Map (Bridge Leaflet)
map_controller.d.ts 2.84 kB / 793 B 2.85 kB0% / 794 B0%
map_controller.js 13.77 kB / 3.52 kB 13.85 kB+1% 📈 / 3.54 kB0%
Translator
translator_controller.d.ts 2.01 kB / 696 B 2.03 kB+1% 📈 / 701 B+1% 📈

@Kocal Kocal merged commit 7554741 into symfony:2.x Feb 3, 2026
32 of 36 checks passed
@Kocal Kocal deleted the switch-from-biomejs-to-oxc branch February 3, 2026 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Needs Review Needs to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants