chore(CI): Remove Biome in favour of prettier (#15461)

Author: timfish

.github/workflows/build.yml

@@ -263,8 +263,6 @@ jobs:
           dependency_cache_key: ${{ needs.job_build.outputs.dependency_cache_key }}
       - name: Lint source files
         run: yarn lint:lerna
-      - name: Lint C++ files
-        run: yarn lint:clang
       - name: Lint for ES compatibility
         run: yarn lint:es-compatibility
 
@@ -289,7 +287,7 @@ jobs:
         id: install_dependencies
 
       - name: Check file formatting
-        run: yarn lint:prettier && yarn lint:biome
+        run: yarn lint:prettier
 
   job_circular_dep_check:
     name: Circular Dependency Check

.prettierignore

@@ -0,0 +1,2 @@
+packages/browser/test/loader.js
+packages/replay-worker/examples/worker.min.js

.prettierrc.json

@@ -0,0 +1,7 @@
+{
+  "arrowParens": "avoid",
+  "printWidth": 120,
+  "proseWrap": "always",
+  "singleQuote": true,
+  "trailingComma": "all"
+}

.vscode/extensions.json

@@ -3,7 +3,6 @@
   // for the documentation about the extensions.json format
   "recommendations": [
     "esbenp.prettier-vscode",
-    "biomejs.biome",
     "dbaeumer.vscode-eslint",
     "augustocdias.tasks-shell-input",
     "denoland.vscode-deno"

.vscode/settings.json

@@ -25,8 +25,8 @@
     }
   ],
   "deno.enablePaths": ["packages/deno/test"],
-  "editor.defaultFormatter": "biomejs.biome",
+  "editor.defaultFormatter": "esbenp.prettier-vscode",
   "[typescript]": {
-    "editor.defaultFormatter": "biomejs.biome"
+    "editor.defaultFormatter": "esbenp.prettier-vscode"
   }
 }

biome.json

@@ -20,17 +20,14 @@
     "clean:tarballs": "rimraf {packages,dev-packages}/*/*.tgz",
     "clean:watchman": "watchman watch-del \".\"",
     "clean:all": "run-s clean:build clean:tarballs clean:caches clean:deps clean:watchman",
-    "fix": "run-s fix:biome fix:prettier fix:lerna",
+    "fix": "run-s fix:prettier fix:lerna",
     "fix:lerna": "lerna run fix",
-    "fix:biome": "biome check --apply .",
-    "fix:prettier": "prettier \"**/*.md\" \"**/*.css\" --write",
+    "fix:prettier": "prettier \"**/*.md\" \"**/*.css\" \"packages/**/**.{ts,js}\" --write",
     "changelog": "ts-node ./scripts/get-commit-list.ts",
     "link:yarn": "lerna exec yarn link",
-    "lint": "run-s lint:lerna lint:biome lint:prettier",
-    "lint:clang": "lerna run lint:clang",
+    "lint": "run-s lint:prettier lint:lerna",
     "lint:lerna": "lerna run lint",
-    "lint:biome": "biome check .",
-    "lint:prettier": "prettier \"**/*.md\" \"**/*.css\" --check",
+    "lint:prettier": "prettier \"**/*.md\" \"**/*.css\" \"packages/**/**.{ts,js}\" --check",
     "lint:es-compatibility": "es-check es2020 ./packages/*/build/{bundles,npm/cjs,cjs}/*.js && es-check es2020 ./packages/*/build/{npm/esm,esm}/*.js --module",
     "postpublish": "lerna run --stream --concurrency 1 postpublish",
     "test": "lerna run --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests}\" test",
@@ -98,7 +95,6 @@
     "dev-packages/rollup-utils"
   ],
   "devDependencies": {
-    "@biomejs/biome": "^1.5.2",
     "@rollup/plugin-commonjs": "^25.0.7",
     "@rollup/plugin-esm-shim": "^0.1.5",
     "@rollup/plugin-json": "^6.1.0",

package.json

@@ -82,13 +82,7 @@ export {
 } from './stack-parsers';
 export { eventFromException, eventFromMessage, exceptionFromError } from './eventbuilder';
 export { createUserFeedbackEnvelope } from './userfeedback';
-export {
-  getDefaultIntegrations,
-  forceLoad,
-  init,
-  onLoad,
-  showReportDialog,
-} from './sdk';
+export { getDefaultIntegrations, forceLoad, init, onLoad, showReportDialog } from './sdk';
 
 export { breadcrumbsIntegration } from './integrations/breadcrumbs';
 export { globalHandlersIntegration } from './integrations/globalhandlers';

packages/browser/src/exports.ts

@@ -62,10 +62,7 @@ export { makeBrowserOfflineTransport } from './transports/offline';
 export { browserProfilingIntegration } from './profiling/integration';
 export { spotlightBrowserIntegration } from './integrations/spotlight';
 export { browserSessionIntegration } from './integrations/browsersession';
-export {
-  featureFlagsIntegration,
-  type FeatureFlagsIntegration,
-} from './integrations/featureFlags';
+export { featureFlagsIntegration, type FeatureFlagsIntegration } from './integrations/featureFlags';
 export { launchDarklyIntegration, buildLaunchDarklyFlagUsedHandler } from './integrations/featureFlags/launchdarkly';
 export { openFeatureIntegration, OpenFeatureIntegrationHook } from './integrations/featureFlags/openfeature';
 export { unleashIntegration } from './integrations/featureFlags/unleash';

packages/browser/src/index.ts

@@ -39,10 +39,7 @@ export {
   getClient,
   getTraceContextFromScope,
 } from './currentScopes';
-export {
-  getDefaultCurrentScope,
-  getDefaultIsolationScope,
-} from './defaultScopes';
+export { getDefaultCurrentScope, getDefaultIsolationScope } from './defaultScopes';
 export { setAsyncContextStrategy } from './asyncContext';
 export { getGlobalSingleton, getMainCarrier } from './carrier';
 export { makeSession, closeSession, updateSession } from './session';
@@ -60,11 +57,7 @@ export { initAndBind, setCurrentClient } from './sdk';
 export { createTransport } from './transports/base';
 export { makeOfflineTransport } from './transports/offline';
 export { makeMultiplexedTransport } from './transports/multiplexed';
-export {
-  getIntegrationsToSetup,
-  addIntegration,
-  defineIntegration,
-} from './integration';
+export { getIntegrationsToSetup, addIntegration, defineIntegration } from './integration';
 export { applyScopeDataToEvent, mergeScopeData } from './utils/applyScopeDataToEvent';
 export { prepareEvent } from './utils/prepareEvent';
 export { createCheckInEnvelope } from './checkin';