chore: improve instrumentation deps (#16899)

* Remove Sentry from direct import in instrumentation file * chore: Exclude Sentry configs when not needed
calcom · Oct 1, 2024 · 0d7f560 · 0d7f560
1 parent 1377f1a
commit 0d7f560
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 11 deletions.
diff --git a/apps/web/instrumentation.ts b/apps/web/instrumentation.ts
@@ -1,15 +1,9 @@
-import * as Sentry from "@sentry/nextjs";
-
-export function register() {
-  if (process.env.NEXT_RUNTIME === "nodejs") {
-    Sentry.init({
-      dsn: process.env.NEXT_PUBLIC_SENTRY_DSN,
-    });
+export async function register() {
+  if (process.env.NEXT_PUBLIC_SENTRY_DSN && process.env.NEXT_RUNTIME === "nodejs") {
+    await import("./sentry.server.config");
   }
 
-  if (process.env.NEXT_RUNTIME === "edge") {
-    Sentry.init({
-      dsn: process.env.NEXT_PUBLIC_SENTRY_DSN,
-    });
+  if (process.env.NEXT_PUBLIC_SENTRY_DSN && process.env.NEXT_RUNTIME === "edge") {
+    await import("./sentry.edge.config");
   }
 }
diff --git a/apps/web/sentry.edge.config.ts b/apps/web/sentry.edge.config.ts
@@ -0,0 +1,5 @@
+import * as Sentry from "@sentry/nextjs";
+
+Sentry.init({
+  dsn: process.env.NEXT_PUBLIC_SENTRY_DSN,
+});
diff --git a/apps/web/sentry.server.config.ts b/apps/web/sentry.server.config.ts
@@ -0,0 +1,5 @@
+import * as Sentry from "@sentry/nextjs";
+
+Sentry.init({
+  dsn: process.env.NEXT_PUBLIC_SENTRY_DSN,
+});