ref(sveltekit): Inject init plugin only if sentry config files exist

packages/sveltekit/src/config/vitePlugins.ts

@@ -60,7 +60,11 @@ function addSentryConfigFileImport(
   return { code: ms.toString(), map: ms.generateMap() };
 }
 
-function getUserConfigFile(projectDir: string, platform: 'server' | 'client'): string | undefined {
+/**
+ * Looks up the sentry.{@param platform}.config.(ts|js) file
+ * @returns the file path to the file or undefined if it doesn't exist
+ */
+export function getUserConfigFile(projectDir: string, platform: 'server' | 'client'): string | undefined {
   const possibilities = [`sentry.${platform}.config.ts`, `sentry.${platform}.config.js`];
 
   for (const filename of possibilities) {
@@ -69,5 +73,5 @@ function getUserConfigFile(projectDir: string, platform: 'server' | 'client'): s
     }
   }
 
-  throw new Error(`Cannot find '${possibilities[0]}' or '${possibilities[1]}' in '${projectDir}'.`);
+  return undefined;
 }

packages/sveltekit/src/config/withSentryViteConfig.ts

@@ -1,6 +1,6 @@
 import type { UserConfig, UserConfigExport } from 'vite';
 
-import { injectSentryInitPlugin } from './vitePlugins';
+import { getUserConfigFile, injectSentryInitPlugin } from './vitePlugins';
 
 /**
  * This function adds Sentry-specific configuration to your Vite config.
@@ -31,17 +31,27 @@ export function withSentryViteConfig(originalConfig: UserConfigExport): UserConf
 }
 
 function addSentryConfig(originalConfig: UserConfig): UserConfig {
+  const sentryPlugins = [];
+
+  const shouldAddInjectInitPlugin = hasSentryInitFiles();
+
+  if (shouldAddInjectInitPlugin) {
+    sentryPlugins.push(injectSentryInitPlugin);
+  }
+
   const config = {
     ...originalConfig,
-    plugins: originalConfig.plugins ? [injectSentryInitPlugin, ...originalConfig.plugins] : [injectSentryInitPlugin],
+    plugins: originalConfig.plugins ? [...sentryPlugins, ...originalConfig.plugins] : [...sentryPlugins],
   };
 
-  const mergedDevServerFileSystemConfig: UserConfig['server'] = {
-    fs: {
-      ...(config.server && config.server.fs),
-      allow: [...((config.server && config.server.fs && config.server.fs.allow) || []), '.'],
-    },
-  };
+  const mergedDevServerFileSystemConfig: UserConfig['server'] = shouldAddInjectInitPlugin
+    ? {
+        fs: {
+          ...(config.server && config.server.fs),
+          allow: [...((config.server && config.server.fs && config.server.fs.allow) || []), '.'],
+        },
+      }
+    : {};
 
   config.server = {
     ...config.server,
@@ -50,3 +60,9 @@ function addSentryConfig(originalConfig: UserConfig): UserConfig {
 
   return config;
 }
+
+function hasSentryInitFiles(): boolean {
+  const hasSentryServerInit = !!getUserConfigFile(process.cwd(), 'server');
+  const hasSentryClientInit = !!getUserConfigFile(process.cwd(), 'client');
+  return hasSentryServerInit || hasSentryClientInit;
+}

packages/sveltekit/test/config/withSentryViteConfig.test.ts

@@ -1,7 +1,17 @@
+import type fs from 'fs';
 import type { Plugin, UserConfig } from 'vite';
+import { vi } from 'vitest';
 
 import { withSentryViteConfig } from '../../src/config/withSentryViteConfig';
 
+let existsFile = true;
+vi.mock('fs', async () => {
+  const original = await vi.importActual<typeof fs>('fs');
+  return {
+    ...original,
+    existsSync: vi.fn().mockImplementation(() => existsFile),
+  };
+});
 describe('withSentryViteConfig', () => {
   const originalConfig = {
     plugins: [{ name: 'foo' }],
@@ -115,4 +125,28 @@ describe('withSentryViteConfig', () => {
 
     expect((sentrifiedConfig as UserConfig).server?.fs?.allow).toStrictEqual(['.']);
   });
+
+  it("doesn't add the inject init plugin or the server config if sentry config files don't exist", () => {
+    existsFile = false;
+
+    const sentrifiedConfig = withSentryViteConfig({
+      plugins: [{ name: 'some plugin' }],
+      test: {
+        include: ['src/**/*.{test,spec}.{js,ts}'],
+      },
+      server: {
+        fs: {
+          allow: ['./bar'],
+        },
+      },
+    } as UserConfig);
+
+    expect(typeof sentrifiedConfig).toBe('object');
+    const plugins = (sentrifiedConfig as UserConfig).plugins as Plugin[];
+    expect(plugins).toHaveLength(1);
+    expect(plugins[0].name).toBe('some plugin');
+    expect((sentrifiedConfig as UserConfig).server?.fs?.allow).toStrictEqual(['./bar']);
+
+    existsFile = true;
+  });
 });