chore: migrate to oxfmt (#19200)

logaretm · web-flow · commit 6fb1ee139a92 · 2026-02-13T17:23:14.000+01:00
This PR migrates our formatting tool from `prettier` to `oxfmt` which is part of the oxc toolchain and offers faster checking and format fixing speeds while [maintaining the same coverage](https://x.com/boshen_c/status/2018329440607203471). I created a follow up PR in #19311 to unignore a few rules and fix the associated snapshot tests affected by it. ### Benchmarks Benchmark | Prettier | oxfmt | Speedup -- | -- | -- | -- CI | 45s | 6.0s-7.0s | ~5x-7.5× Local M3 Pro | 22s | 1.22s-1.98s | ~11× --- closes #19223
diff --git a/.cursor/rules/sdk_development.mdc b/.cursor/rules/sdk_development.mdc
@@ -34,6 +34,8 @@ You are working on the Sentry JavaScript SDK, a critical production SDK used by
 
 - `yarn lint` - Run ESLint and Prettier checks
 - `yarn fix` - Auto-fix linting and formatting issues
+- `yarn format:check` - Check file formatting only
+- `yarn format` - Auto-fix formatting issues
 - `yarn lint:es-compatibility` - Check ES compatibility
 
 ## Git Flow Branching Strategy
diff --git a/.editorconfig b/.editorconfig
@@ -8,4 +8,4 @@ trim_trailing_whitespace = true
 insert_final_newline = true
 
 [*.md]
-trim_trailing_whitespace = false
+trim_trailing_whitespace = false
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -339,7 +339,7 @@ jobs:
         id: install_dependencies
 
       - name: Check file formatting
-        run: yarn lint:prettier
+        run: yarn format:check
 
   job_circular_dep_check:
     name: Circular Dependency Check
diff --git a/.oxfmtrc.json b/.oxfmtrc.json
@@ -0,0 +1,29 @@
+{
+  "$schema": "./node_modules/oxfmt/configuration_schema.json",
+  "arrowParens": "avoid",
+  "printWidth": 120,
+  "proseWrap": "always",
+  "singleQuote": true,
+  "trailingComma": "all",
+  "experimentalSortPackageJson": false,
+  "ignorePatterns": [
+    "packages/browser/test/loader.js",
+    "packages/replay-worker/examples/worker.min.js",
+    "dev-packages/browser-integration-tests/fixtures",
+    "**/test.ts-snapshots/**",
+    "/.nx/cache",
+    "/.nx/workspace-data",
+    "dev-packages/**/*.html",
+    "dev-packages/**/*.hbs",
+    "packages/ember/**/*.hbs",
+    "packages/ember/**/*.html"
+  ],
+  "overrides": [
+    {
+      "files": ["*.md", "*.mdc"],
+      "options": {
+        "proseWrap": "preserve"
+      }
+    }
+  ]
+}
diff --git a/.prettierignore b/.prettierignore
diff --git a/.vscode/extensions.json b/.vscode/extensions.json
@@ -2,7 +2,7 @@
   // See http://go.microsoft.com/fwlink/?LinkId=827846
   // for the documentation about the extensions.json format
   "recommendations": [
-    "esbenp.prettier-vscode",
+    "oxc.oxc-vscode",
     "dbaeumer.vscode-eslint",
     "augustocdias.tasks-shell-input",
     "denoland.vscode-deno"
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -11,10 +11,10 @@
   },
   "typescript.tsdk": "./node_modules/typescript/lib",
   "[markdown]": {
-    "editor.defaultFormatter": "esbenp.prettier-vscode"
+    "editor.defaultFormatter": "oxc.oxc-vscode"
   },
   "[css]": {
-    "editor.defaultFormatter": "esbenp.prettier-vscode"
+    "editor.defaultFormatter": "oxc.oxc-vscode"
   },
   "yaml.schemas": {
     "https://json.schemastore.org/github-workflow.json": ".github/workflows/**.yml"
@@ -25,8 +25,8 @@
     }
   ],
   "deno.enablePaths": ["packages/deno/test"],
-  "editor.defaultFormatter": "esbenp.prettier-vscode",
+  "editor.defaultFormatter": "oxc.oxc-vscode",
   "[typescript]": {
-    "editor.defaultFormatter": "esbenp.prettier-vscode"
+    "editor.defaultFormatter": "oxc.oxc-vscode"
   }
 }
diff --git a/CLAUDE.md b/CLAUDE.md
@@ -31,8 +31,10 @@ You are working on the Sentry JavaScript SDK, a critical production SDK used by
 
 ### Linting and Formatting
 
-- `yarn lint` - Run ESLint and Prettier checks
+- `yarn lint` - Run ESLint and Oxfmt checks
 - `yarn fix` - Auto-fix linting and formatting issues
+- `yarn format:check` - Check file formatting only
+- `yarn format` - Auto-fix formatting issues
 - `yarn lint:es-compatibility` - Check ES compatibility
 
 ## Git Flow Branching Strategy
diff --git a/dev-packages/cloudflare-integration-tests/suites/basic/wrangler.jsonc b/dev-packages/cloudflare-integration-tests/suites/basic/wrangler.jsonc
@@ -2,5 +2,5 @@
   "name": "worker-name",
   "compatibility_date": "2025-06-17",
   "main": "index.ts",
-  "compatibility_flags": ["nodejs_compat"]
+  "compatibility_flags": ["nodejs_compat"],
 }
diff --git a/dev-packages/cloudflare-integration-tests/suites/hono-integration/wrangler.jsonc b/dev-packages/cloudflare-integration-tests/suites/hono-integration/wrangler.jsonc
@@ -2,6 +2,5 @@
   "name": "hono-basic-worker",
   "compatibility_date": "2025-06-17",
   "main": "index.ts",
-  "compatibility_flags": ["nodejs_compat"]
+  "compatibility_flags": ["nodejs_compat"],
 }
-
diff --git a/dev-packages/cloudflare-integration-tests/suites/hono-sdk/wrangler.jsonc b/dev-packages/cloudflare-integration-tests/suites/hono-sdk/wrangler.jsonc
@@ -2,6 +2,5 @@
   "name": "hono-sdk-worker",
   "compatibility_date": "2025-06-17",
   "main": "index.ts",
-  "compatibility_flags": ["nodejs_compat"]
+  "compatibility_flags": ["nodejs_compat"],
 }
-
diff --git a/dev-packages/cloudflare-integration-tests/suites/tracing/durableobject/wrangler.jsonc b/dev-packages/cloudflare-integration-tests/suites/tracing/durableobject/wrangler.jsonc
@@ -5,19 +5,19 @@
   "migrations": [
     {
       "new_sqlite_classes": ["TestDurableObject"],
-      "tag": "v1"
-    }
+      "tag": "v1",
+    },
   ],
   "durable_objects": {
     "bindings": [
       {
         "class_name": "TestDurableObject",
-        "name": "TEST_DURABLE_OBJECT"
-      }
-    ]
+        "name": "TEST_DURABLE_OBJECT",
+      },
+    ],
   },
   "compatibility_flags": ["nodejs_als"],
   "vars": {
-    "SENTRY_DSN": "https://932e620ee3921c3b4a61c72558ad88ce@o447951.ingest.us.sentry.io/4509553159831552"
-  }
+    "SENTRY_DSN": "https://932e620ee3921c3b4a61c72558ad88ce@o447951.ingest.us.sentry.io/4509553159831552",
+  },
 }
diff --git a/dev-packages/cloudflare-integration-tests/suites/tracing/headers/wrangler.jsonc b/dev-packages/cloudflare-integration-tests/suites/tracing/headers/wrangler.jsonc
@@ -2,5 +2,5 @@
   "name": "worker-name",
   "compatibility_date": "2025-06-17",
   "main": "index.ts",
-  "compatibility_flags": ["nodejs_compat"]
+  "compatibility_flags": ["nodejs_compat"],
 }
diff --git a/dev-packages/cloudflare-integration-tests/suites/tracing/openai/wrangler.jsonc b/dev-packages/cloudflare-integration-tests/suites/tracing/openai/wrangler.jsonc
@@ -2,5 +2,5 @@
   "name": "worker-name",
   "compatibility_date": "2025-06-17",
   "main": "index.ts",
-  "compatibility_flags": ["nodejs_compat"]
+  "compatibility_flags": ["nodejs_compat"],
 }
diff --git a/dev-packages/e2e-tests/test-applications/angular-17/src/app/app.component.ts b/dev-packages/e2e-tests/test-applications/angular-17/src/app/app.component.ts
@@ -5,7 +5,9 @@ import { RouterOutlet } from '@angular/router';
   selector: 'app-root',
   standalone: true,
   imports: [RouterOutlet],
-  template: `<router-outlet></router-outlet>`,
+  template: `
+    <router-outlet></router-outlet>
+  `,
 })
 export class AppComponent {
   title = 'angular-17';
diff --git a/dev-packages/e2e-tests/test-applications/angular-17/src/app/cancel/cancel.components.ts b/dev-packages/e2e-tests/test-applications/angular-17/src/app/cancel/cancel.components.ts
@@ -3,6 +3,8 @@ import { Component } from '@angular/core';
 @Component({
   selector: 'app-cancel',
   standalone: true,
-  template: `<div></div>`,
+  template: `
+    <div></div>
+  `,
 })
 export class CancelComponent {}
diff --git a/dev-packages/e2e-tests/test-applications/angular-17/src/app/sample-component/sample-component.components.ts b/dev-packages/e2e-tests/test-applications/angular-17/src/app/sample-component/sample-component.components.ts
@@ -3,7 +3,9 @@ import { Component, OnInit } from '@angular/core';
 @Component({
   selector: 'app-sample-component',
   standalone: true,
-  template: `<div></div>`,
+  template: `
+    <div></div>
+  `,
 })
 export class SampleComponent implements OnInit {
   ngOnInit() {
diff --git a/dev-packages/e2e-tests/test-applications/angular-18/src/app/app.component.ts b/dev-packages/e2e-tests/test-applications/angular-18/src/app/app.component.ts
@@ -5,7 +5,9 @@ import { RouterOutlet } from '@angular/router';
   selector: 'app-root',
   standalone: true,
   imports: [RouterOutlet],
-  template: `<router-outlet></router-outlet>`,
+  template: `
+    <router-outlet></router-outlet>
+  `,
 })
 export class AppComponent {
   title = 'angular-18';
diff --git a/dev-packages/e2e-tests/test-applications/angular-18/src/app/cancel/cancel.components.ts b/dev-packages/e2e-tests/test-applications/angular-18/src/app/cancel/cancel.components.ts
@@ -3,6 +3,8 @@ import { Component } from '@angular/core';
 @Component({
   selector: 'app-cancel',
   standalone: true,
-  template: `<div></div>`,
+  template: `
+    <div></div>
+  `,
 })
 export class CancelComponent {}
diff --git a/dev-packages/e2e-tests/test-applications/angular-18/src/app/component-tracking/component-tracking.components.ts b/dev-packages/e2e-tests/test-applications/angular-18/src/app/component-tracking/component-tracking.components.ts
@@ -6,7 +6,9 @@ import { SampleComponent } from '../sample-component/sample-component.components
   selector: 'app-cancel',
   standalone: true,
   imports: [TraceModule, SampleComponent],
-  template: `<app-sample-component [trace]="'sample-component'"></app-sample-component>`,
+  template: `
+    <app-sample-component [trace]="'sample-component'"></app-sample-component>
+  `,
 })
 @TraceClass({ name: 'ComponentTrackingComponent' })
 export class ComponentTrackingComponent implements OnInit, AfterViewInit {
diff --git a/dev-packages/e2e-tests/test-applications/angular-18/src/app/sample-component/sample-component.components.ts b/dev-packages/e2e-tests/test-applications/angular-18/src/app/sample-component/sample-component.components.ts
@@ -3,7 +3,9 @@ import { Component, OnInit } from '@angular/core';
 @Component({
   selector: 'app-sample-component',
   standalone: true,
-  template: `<div>Component</div>`,
+  template: `
+    <div>Component</div>
+  `,
 })
 export class SampleComponent implements OnInit {
   ngOnInit() {
diff --git a/dev-packages/e2e-tests/test-applications/angular-19/src/app/app.component.ts b/dev-packages/e2e-tests/test-applications/angular-19/src/app/app.component.ts
@@ -5,7 +5,9 @@ import { RouterOutlet } from '@angular/router';
   selector: 'app-root',
   standalone: true,
   imports: [RouterOutlet],
-  template: `<router-outlet></router-outlet>`,
+  template: `
+    <router-outlet></router-outlet>
+  `,
 })
 export class AppComponent {
   title = 'angular-19';
diff --git a/dev-packages/e2e-tests/test-applications/angular-19/src/app/cancel/cancel.components.ts b/dev-packages/e2e-tests/test-applications/angular-19/src/app/cancel/cancel.components.ts
@@ -3,6 +3,8 @@ import { Component } from '@angular/core';
 @Component({
   selector: 'app-cancel',
   standalone: true,
-  template: `<div></div>`,
+  template: `
+    <div></div>
+  `,
 })
 export class CancelComponent {}
diff --git a/dev-packages/e2e-tests/test-applications/angular-19/src/app/sample-component/sample-component.components.ts b/dev-packages/e2e-tests/test-applications/angular-19/src/app/sample-component/sample-component.components.ts
@@ -3,7 +3,9 @@ import { Component, OnInit } from '@angular/core';
 @Component({
   selector: 'app-sample-component',
   standalone: true,
-  template: `<div>Component</div>`,
+  template: `
+    <div>Component</div>
+  `,
 })
 export class SampleComponent implements OnInit {
   ngOnInit() {
diff --git a/dev-packages/e2e-tests/test-applications/angular-20/src/app/app.component.ts b/dev-packages/e2e-tests/test-applications/angular-20/src/app/app.component.ts
@@ -5,7 +5,9 @@ import { RouterOutlet } from '@angular/router';
   selector: 'app-root',
   standalone: true,
   imports: [RouterOutlet],
-  template: `<router-outlet></router-outlet>`,
+  template: `
+    <router-outlet></router-outlet>
+  `,
 })
 export class AppComponent {
   title = 'angular-20';
diff --git a/dev-packages/e2e-tests/test-applications/angular-20/src/app/cancel/cancel.components.ts b/dev-packages/e2e-tests/test-applications/angular-20/src/app/cancel/cancel.components.ts
@@ -3,6 +3,8 @@ import { Component } from '@angular/core';
 @Component({
   selector: 'app-cancel',
   standalone: true,
-  template: `<div></div>`,
+  template: `
+    <div></div>
+  `,
 })
 export class CancelComponent {}
diff --git a/dev-packages/e2e-tests/test-applications/angular-20/src/app/sample-component/sample-component.components.ts b/dev-packages/e2e-tests/test-applications/angular-20/src/app/sample-component/sample-component.components.ts
@@ -3,7 +3,9 @@ import { Component, OnInit } from '@angular/core';
 @Component({
   selector: 'app-sample-component',
   standalone: true,
-  template: `<div>Component</div>`,
+  template: `
+    <div>Component</div>
+  `,
 })
 export class SampleComponent implements OnInit {
   ngOnInit() {
diff --git a/dev-packages/e2e-tests/test-applications/angular-21/src/app/app.component.ts b/dev-packages/e2e-tests/test-applications/angular-21/src/app/app.component.ts
@@ -5,7 +5,9 @@ import { RouterOutlet } from '@angular/router';
   selector: 'app-root',
   standalone: true,
   imports: [RouterOutlet],
-  template: `<router-outlet></router-outlet>`,
+  template: `
+    <router-outlet></router-outlet>
+  `,
 })
 export class AppComponent {
   title = 'angular-21';
diff --git a/dev-packages/e2e-tests/test-applications/angular-21/src/app/cancel/cancel.components.ts b/dev-packages/e2e-tests/test-applications/angular-21/src/app/cancel/cancel.components.ts
@@ -3,6 +3,8 @@ import { Component } from '@angular/core';
 @Component({
   selector: 'app-cancel',
   standalone: true,
-  template: `<div></div>`,
+  template: `
+    <div></div>
+  `,
 })
 export class CancelComponent {}
diff --git a/dev-packages/e2e-tests/test-applications/angular-21/src/app/sample-component/sample-component.components.ts b/dev-packages/e2e-tests/test-applications/angular-21/src/app/sample-component/sample-component.components.ts
@@ -3,7 +3,9 @@ import { Component, OnInit } from '@angular/core';
 @Component({
   selector: 'app-sample-component',
   standalone: true,
-  template: `<div>Component</div>`,
+  template: `
+    <div>Component</div>
+  `,
 })
 export class SampleComponent implements OnInit {
   ngOnInit() {
diff --git a/dev-packages/e2e-tests/test-applications/nextjs-16-cf-workers/wrangler.jsonc b/dev-packages/e2e-tests/test-applications/nextjs-16-cf-workers/wrangler.jsonc
diff --git a/dev-packages/node-integration-tests/suites/tracing/prisma-orm-v5/prisma/migrations/migration_lock.toml b/dev-packages/node-integration-tests/suites/tracing/prisma-orm-v5/prisma/migrations/migration_lock.toml
diff --git a/dev-packages/node-integration-tests/suites/tracing/prisma-orm-v6/prisma/migrations/migration_lock.toml b/dev-packages/node-integration-tests/suites/tracing/prisma-orm-v6/prisma/migrations/migration_lock.toml
diff --git a/package.json b/package.json
diff --git a/packages/replay-internal/package.json b/packages/replay-internal/package.json
diff --git a/packages/sveltekit/test/server-common/serverRoute.test.ts b/packages/sveltekit/test/server-common/serverRoute.test.ts
diff --git a/yarn.lock b/yarn.lock

Original file line number	Diff line number	Diff line change
`@@ -2,5 +2,5 @@`
`2`	`2`	`"name": "worker-name",`
`3`	`3`	`"compatibility_date": "2025-06-17",`
`4`	`4`	`"main": "index.ts",`
`5`		`- "compatibility_flags": ["nodejs_compat"]`
	`5`	`+ "compatibility_flags": ["nodejs_compat"],`
`6`	`6`	`}`
Original file line number	Diff line number	Diff line change
`@@ -2,6 +2,5 @@`
`2`	`2`	`"name": "hono-basic-worker",`
`3`	`3`	`"compatibility_date": "2025-06-17",`
`4`	`4`	`"main": "index.ts",`
`5`		`- "compatibility_flags": ["nodejs_compat"]`
	`5`	`+ "compatibility_flags": ["nodejs_compat"],`
`6`	`6`	`}`
`7`		`-`