Stabilize CI mocks Sentry init

Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>

Author: cursoragent

server/index.ts

@@ -87,11 +87,17 @@ const getHost = (req: { get: (key: string) => string | undefined }) =>
 
 const MODE = process.env.NODE_ENV
 
-if (MODE === 'production' && process.env.SENTRY_DSN) {
+const SHOULD_INIT_SENTRY =
+	MODE === 'production' &&
+	Boolean(process.env.SENTRY_DSN) &&
+	// `start:mocks` (used in CI + local e2e) runs with `MOCKS=true`.
+	process.env.MOCKS !== 'true'
+
+if (SHOULD_INIT_SENTRY) {
 	void import('./utils/monitoring.js').then(({ init }) => init())
 }
 
-if (MODE === 'production') {
+if (SHOULD_INIT_SENTRY) {
 	sentryInit({
 		dsn: process.env.SENTRY_DSN,
 		tracesSampleRate: 0.3,

vite.config.ts

@@ -15,6 +15,8 @@ if (SENTRY_UPLOAD && MODE === 'production') {
 	const authToken = process.env.SENTRY_AUTH_TOKEN
 	const project = process.env.SENTRY_PROJECT
 	const org = process.env.SENTRY_ORG
+	// New-style Sentry auth tokens (prefix "sntrys_") embed the org, so SENTRY_ORG
+	// is not required when using one.
 	const tokenImpliesOrg = Boolean(authToken?.startsWith('sntrys_'))
 
 	// If upload is "on" but required settings are missing, the Sentry plugin will
@@ -66,6 +68,10 @@ export default defineConfig(async () => {
 				: null,
 		],
 		build: {
+			// This is an OSS project, so it's fine to generate "regular" sourcemaps.
+			// If we ever want sourcemaps upload without exposing them publicly, switch
+			// to `sourcemap: 'hidden'` (and keep uploading to Sentry).
+			sourcemap: true,
 			cssMinify: MODE === 'production',
 			rollupOptions: {
 				external: [/node:.*/, 'stream', 'crypto'],