perf: Remove dynamic imports of Sentry (#20545)

This commit is contained in:
Keith Williams
2025-04-04 13:22:27 +00:00
committed by GitHub
parent ddde762f9b
commit 3d583678cf
6 changed files with 10 additions and 11 deletions
@@ -1,4 +1,5 @@
import { wrapApiHandlerWithSentry } from "@sentry/nextjs";
import { captureException } from "@sentry/nextjs";
import type { Params } from "app/_types";
import { ApiError } from "next/dist/server/api-utils";
import type { NextRequest } from "next/server";
@@ -48,7 +49,6 @@ export const defaultResponderForAppDir = <T extends NextResponse | Response = Ne
// Don't report 400-499 errors to Sentry/console
if (!(serverError.statusCode >= 400 && serverError.statusCode < 500)) {
console.error(serverError);
const captureException = (await import("@sentry/nextjs")).captureException;
captureException(error);
}
@@ -1,3 +1,5 @@
import { captureException } from "@sentry/nextjs";
import db from "@calcom/prisma";
import type { AppFlags } from "./config";
@@ -9,7 +11,6 @@ export class FeaturesRepository implements IFeaturesRepository {
try {
return await getFeatureFlag(db, slug);
} catch (err) {
const captureException = (await import("@sentry/nextjs")).captureException;
captureException(err);
throw err;
}
@@ -34,7 +35,6 @@ export class FeaturesRepository implements IFeaturesRepository {
if (userBelongsToTeamWithFeature) return true;
return false;
} catch (err) {
const captureException = (await import("@sentry/nextjs")).captureException;
captureException(err);
throw err;
}
@@ -61,7 +61,6 @@ export class FeaturesRepository implements IFeaturesRepository {
if (user) return true;
return false;
} catch (err) {
const captureException = (await import("@sentry/nextjs")).captureException;
captureException(err);
throw err;
}
@@ -73,7 +72,6 @@ export class FeaturesRepository implements IFeaturesRepository {
});
return !!teamFeature;
} catch (err) {
const captureException = (await import("@sentry/nextjs")).captureException;
captureException(err);
throw err;
}
+2 -1
View File
@@ -1,3 +1,5 @@
import { captureException } from "@sentry/nextjs";
import db from "@calcom/prisma";
import type { IUsersRepository } from "./users.repository.interface";
@@ -11,7 +13,6 @@ export class UsersRepository implements IUsersRepository {
});
return user;
} catch (err) {
const captureException = (await import("@sentry/nextjs")).captureException;
captureException(err);
throw err;
}
@@ -1,3 +1,5 @@
import { captureException } from "@sentry/nextjs";
import db from "@calcom/prisma";
import { WatchlistType, WatchlistSeverity } from "@calcom/prisma/enums";
@@ -18,7 +20,6 @@ export class WatchlistRepository implements IWatchlistRepository {
});
return emailInWatchlist;
} catch (err) {
const captureException = (await import("@sentry/nextjs")).captureException;
captureException(err);
throw err;
}
@@ -34,7 +35,6 @@ export class WatchlistRepository implements IWatchlistRepository {
});
return domainInWatchWatchlist;
} catch (err) {
const captureException = (await import("@sentry/nextjs")).captureException;
captureException(err);
throw err;
}
@@ -90,7 +90,6 @@ export class WatchlistRepository implements IWatchlistRepository {
});
return blockedRecords;
} catch (err) {
const captureException = (await import("@sentry/nextjs")).captureException;
captureException(err);
throw err;
}
+1 -1
View File
@@ -1,4 +1,5 @@
import { wrapApiHandlerWithSentry } from "@sentry/nextjs";
import { captureException } from "@sentry/nextjs";
import type { NextApiRequest, NextApiResponse } from "next";
import { getServerErrorFromUnknown } from "./getServerErrorFromUnknown";
@@ -28,7 +29,6 @@ export function defaultResponder<T>(
// we don't want to report Bad Request errors to Sentry / console
if (!(error.statusCode >= 400 && error.statusCode < 500)) {
console.error(error);
const captureException = (await import("@sentry/nextjs")).captureException;
captureException(error);
}
return res
@@ -1,3 +1,5 @@
import { captureException } from "@sentry/nextjs";
import { redactError } from "@calcom/lib/redactError";
import { middleware } from "../trpc";
@@ -9,7 +11,6 @@ const captureErrorsMiddleware = middleware(async ({ next }) => {
if (!cause) {
return result;
}
const { captureException } = await import("@sentry/nextjs");
captureException(cause);
throw redactError(cause);
}