chore: Migrate to oxfmt (#880)

Author: timfish

.devcontainer/devcontainer.json

@@ -30,7 +30,7 @@
   },
   "customizations": {
     "vscode": {
-      "extensions": ["esbenp.prettier-vscode", "SonarSource.sonarlint-vscode"]
+      "extensions": ["oxc.oxc-vscode", "SonarSource.sonarlint-vscode"]
     }
   }
 }

.oxfmtrc.json

@@ -0,0 +1,11 @@
+{
+  "$schema": "./node_modules/oxfmt/configuration_schema.json",
+  "printWidth": 100,
+  "experimentalSortPackageJson": false,
+  "trailingComma": "es5",
+  "ignorePatterns": [
+    "packages/e2e-tests/scenarios/*/ref/**/*",
+    "packages/bundler-plugin-core/test/fixtures",
+    ".nxcache"
+  ]
+}

.prettierignore

@@ -0,0 +1,3 @@
+{
+  "editor.defaultFormatter": "oxc.oxc-vscode",
+}

.prettierrc.json

@@ -22,15 +22,15 @@
     "test:integration": "nx run @sentry-internal/integration-tests:test",
     "test:e2e": "nx run @sentry-internal/bundler-plugin-e2e-tests:test",
     "lint": "nx run-many --target=lint --all",
-    "check:formatting": "prettier --check .",
-    "fix:formatting": "prettier --write ."
+    "check:formatting": "oxfmt --check .",
+    "fix:formatting": "oxfmt ."
   },
   "devDependencies": {
     "@nrwl/cli": "14.5.10",
     "@nrwl/workspace": "14.5.10",
     "lerna": "^6.6.2",
     "nx": "14.5.10",
-    "prettier": "^2.7.1",
+    "oxfmt": "^0.33.0",
     "pretty-quick": "^3.1.3",
     "npm-run-all": "^4.1.5",
     "ts-node": "^10.9.2"

.vscode/settings.json

@@ -3,12 +3,12 @@ try {
     "undefined" != typeof window
       ? window
       : "undefined" != typeof global
-      ? global
-      : "undefined" != typeof globalThis
-      ? global
-      : "undefined" != typeof self
-      ? self
-      : {};
+        ? global
+        : "undefined" != typeof globalThis
+          ? global
+          : "undefined" != typeof self
+            ? self
+            : {};
 
   let stack = new globalObject.Error().stack;
 

package.json

@@ -110,7 +110,7 @@ export function setTelemetryDataOnScope(
 
   scope.setTags({
     organization: org,
-    project: Array.isArray(project) ? project.join(", ") : project ?? "undefined",
+    project: Array.isArray(project) ? project.join(", ") : (project ?? "undefined"),
     bundler: buildTool,
   });
 

packages/bundler-plugin-core/sentry-esbuild-debugid-injection-file.js

@@ -120,16 +120,19 @@ export function getDependencies(packageJson: PackageJson): {
 
   const deps = Object.keys(dependencies).sort();
 
-  const depsVersions: Record<string, number> = deps.reduce((depsVersions, depName) => {
-    if (PACKAGES_TO_INCLUDE_VERSION.includes(depName)) {
-      const version = dependencies[depName] as string;
-      const majorVersion = parseMajorVersion(version);
-      if (majorVersion) {
-        depsVersions[depName] = majorVersion;
+  const depsVersions: Record<string, number> = deps.reduce(
+    (depsVersions, depName) => {
+      if (PACKAGES_TO_INCLUDE_VERSION.includes(depName)) {
+        const version = dependencies[depName] as string;
+        const majorVersion = parseMajorVersion(version);
+        if (majorVersion) {
+          depsVersions[depName] = majorVersion;
+        }
       }
-    }
-    return depsVersions;
-  }, {} as Record<string, number>);
+      return depsVersions;
+    },
+    {} as Record<string, number>
+  );
 
   return { deps, depsVersions };
 }
@@ -414,8 +417,8 @@ export function serializeIgnoreOptions(ignoreValue: string | string[] | undefine
   const ignoreOptions: string[] = Array.isArray(ignoreValue)
     ? ignoreValue
     : typeof ignoreValue === "string"
-    ? [ignoreValue]
-    : DEFAULT_IGNORE;
+      ? [ignoreValue]
+      : DEFAULT_IGNORE;
 
   return ignoreOptions.reduce(
     (acc, value) => acc.concat(["--ignore", String(value)]),

packages/bundler-plugin-core/src/sentry/telemetry.ts

@@ -2,8 +2,7 @@ import childProcess from "child_process";
 import path from "path";
 import { testIfNodeMajorVersionIsLessThan18 } from "../../utils/testIf";
 
-// prettier-ignore
-const SNAPSHOT = `"<div><span data-sentry-component=\\"ComponentA\\">Component A</span></div>"`
+const SNAPSHOT = `"<div><span data-sentry-component=\\"ComponentA\\">Component A</span></div>"`;
 const ESBUILD_SNAPSHOT = `"<div><span>Component A</span></div>"`;
 
 function checkBundle(bundlePath: string, snapshot = SNAPSHOT): void {