diff --git a/packages/solidstart/src/vite/sourceMaps.ts b/packages/solidstart/src/vite/sourceMaps.ts index 1228249c193d..285b3949bf93 100644 --- a/packages/solidstart/src/vite/sourceMaps.ts +++ b/packages/solidstart/src/vite/sourceMaps.ts @@ -18,7 +18,7 @@ export function makeAddSentryVitePlugin(options: SentrySolidStartPluginOptions, // Only if source maps were previously not set, we update the "filesToDeleteAfterUpload" (as we override the setting with "hidden") typeof viteConfig.build?.sourcemap === 'undefined' ) { - // This also works for adapters, as the source maps are also copied to e.g. the .vercel folder + // For .output, .vercel, .netlify etc. updatedFilesToDeleteAfterUpload = ['.*/**/*.map']; consoleSandbox(() => { diff --git a/packages/sveltekit/src/vite/sourceMaps.ts b/packages/sveltekit/src/vite/sourceMaps.ts index a59e9af19260..be0334348e70 100644 --- a/packages/sveltekit/src/vite/sourceMaps.ts +++ b/packages/sveltekit/src/vite/sourceMaps.ts @@ -1,11 +1,12 @@ +/* eslint-disable max-lines */ import * as child_process from 'child_process'; import * as fs from 'fs'; import * as path from 'path'; -import { escapeStringForRegex, uuid4 } from '@sentry/core'; +import { consoleSandbox, escapeStringForRegex, uuid4 } from '@sentry/core'; import { getSentryRelease } from '@sentry/node'; import type { SentryVitePluginOptions } from '@sentry/vite-plugin'; import { sentryVitePlugin } from '@sentry/vite-plugin'; -import type { Plugin } from 'vite'; +import { type Plugin, type UserConfig, loadConfigFromFile } from 'vite'; import MagicString from 'magic-string'; import { WRAPPED_MODULE_SUFFIX } from './autoInstrument'; @@ -23,6 +24,13 @@ type Sorcery = { load(filepath: string): Promise; }; +type GlobalWithSourceMapSetting = typeof globalThis & { + _sentry_sourceMapSetting?: { + updatedSourceMapSetting?: boolean | 'inline' | 'hidden'; + previousSourceMapSetting?: UserSourceMapSetting; + }; +}; + // storing this in the module scope because `makeCustomSentryVitePlugin` is called multiple times // and we only want to generate a uuid once in case we have to fall back to it. const releaseName = detectSentryRelease(); @@ -47,7 +55,9 @@ export async function makeCustomSentryVitePlugins(options?: CustomSentryVitePlug const svelteConfig = await loadSvelteConfig(); const usedAdapter = options?.adapter || 'other'; - const outputDir = await getAdapterOutputDir(svelteConfig, usedAdapter); + const adapterOutputDir = await getAdapterOutputDir(svelteConfig, usedAdapter); + + const globalWithSourceMapSetting = globalThis as GlobalWithSourceMapSetting; const defaultPluginOptions: SentryVitePluginOptions = { release: { @@ -60,6 +70,43 @@ export async function makeCustomSentryVitePlugins(options?: CustomSentryVitePlug }, }; + // Including all hidden (`.*`) directories by default so that folders like .vercel, + // .netlify, etc are also cleaned up. Additionally, we include the adapter output + // dir which could be a non-hidden directory, like `build` for the Node adapter. + const defaultFileDeletionGlob = ['./.*/**/*.map', `./${adapterOutputDir}/**/*.map`]; + + if (!globalWithSourceMapSetting._sentry_sourceMapSetting) { + const configFile = await loadConfigFromFile({ command: 'build', mode: 'production' }); + + if (configFile) { + globalWithSourceMapSetting._sentry_sourceMapSetting = getUpdatedSourceMapSetting(configFile.config); + } else { + if (options?.debug) { + consoleSandbox(() => { + // eslint-disable-next-line no-console + console.warn( + '[Sentry] Could not load Vite config with Vite "production" mode. This is needed for Sentry to automatically update source map settings.', + ); + }); + } + } + + if (options?.debug && globalWithSourceMapSetting._sentry_sourceMapSetting?.previousSourceMapSetting === 'unset') { + consoleSandbox(() => { + // eslint-disable-next-line no-console + console.warn( + `[Sentry] Automatically setting \`sourceMapsUploadOptions.sourcemaps.filesToDeleteAfterUpload: [${defaultFileDeletionGlob + .map(file => `"${file}"`) + .join(', ')}]\` to delete generated source maps after they were uploaded to Sentry.`, + ); + }); + } + } + + const shouldDeleteDefaultSourceMaps = + globalWithSourceMapSetting._sentry_sourceMapSetting?.previousSourceMapSetting === 'unset' && + !options?.sourcemaps?.filesToDeleteAfterUpload; + const mergedOptions = { ...defaultPluginOptions, ...options, @@ -67,7 +114,14 @@ export async function makeCustomSentryVitePlugins(options?: CustomSentryVitePlug ...defaultPluginOptions.release, ...options?.release, }, + sourcemaps: { + ...options?.sourcemaps, + filesToDeleteAfterUpload: shouldDeleteDefaultSourceMaps + ? defaultFileDeletionGlob + : options?.sourcemaps?.filesToDeleteAfterUpload, + }, }; + const { debug } = mergedOptions; const sentryPlugins: Plugin[] = await sentryVitePlugin(mergedOptions); @@ -126,37 +180,51 @@ export async function makeCustomSentryVitePlugins(options?: CustomSentryVitePlug const serverHooksFile = getHooksFileName(svelteConfig, 'server'); const globalSentryValues: GlobalSentryValues = { - __sentry_sveltekit_output_dir: outputDir, + __sentry_sveltekit_output_dir: adapterOutputDir, }; - const customDebugIdUploadPlugin: Plugin = { - name: 'sentry-sveltekit-debug-id-upload-plugin', + const sourceMapSettingsPlugin: Plugin = { + name: 'sentry-sveltekit-update-source-map-setting-plugin', apply: 'build', // only apply this plugin at build time - enforce: 'post', // this needs to be set to post, otherwise we don't pick up the output from the SvelteKit adapter + config: (config: UserConfig) => { + const settingKey = 'build.sourcemap'; - // Modify the config to generate source maps - config: config => { - const sourceMapsPreviouslyNotEnabled = !config.build?.sourcemap; - if (debug && sourceMapsPreviouslyNotEnabled) { - // eslint-disable-next-line no-console - console.log('[Source Maps Plugin] Enabling source map generation'); - if (!mergedOptions.sourcemaps?.filesToDeleteAfterUpload) { - // eslint-disable-next-line no-console + if (globalWithSourceMapSetting._sentry_sourceMapSetting?.previousSourceMapSetting === 'unset') { + consoleSandbox(() => { + // eslint-disable-next-line no-console + console.log(`[Sentry] Enabled source map generation in the build options with \`${settingKey}: "hidden"\`.`); + }); + + return { + ...config, + build: { ...config.build, sourcemap: 'hidden' }, + }; + } else if (globalWithSourceMapSetting._sentry_sourceMapSetting?.previousSourceMapSetting === 'disabled') { + consoleSandbox(() => { + // eslint-disable-next-line no-console console.warn( - `[Source Maps Plugin] We recommend setting the \`sourceMapsUploadOptions.sourcemaps.filesToDeleteAfterUpload\` option to clean up source maps after uploading. -[Source Maps Plugin] Otherwise, source maps might be deployed to production, depending on your configuration`, + `[Sentry] Parts of source map generation are currently disabled in your Vite configuration (\`${settingKey}: false\`). This setting is either a default setting or was explicitly set in your configuration. Sentry won't override this setting. Without source maps, code snippets on the Sentry Issues page will remain minified. To show unminified code, enable source maps in \`${settingKey}\` (e.g. by setting them to \`hidden\`).`, ); + }); + } else if (globalWithSourceMapSetting._sentry_sourceMapSetting?.previousSourceMapSetting === 'enabled') { + if (mergedOptions?.debug) { + consoleSandbox(() => { + // eslint-disable-next-line no-console + console.log( + `[Sentry] We discovered you enabled source map generation in your Vite configuration (\`${settingKey}\`). Sentry will keep this source map setting. This will un-minify the code snippet on the Sentry Issue page.`, + ); + }); } } - return { - ...config, - build: { - ...config.build, - sourcemap: true, - }, - }; + + return config; }, + }; + const customDebugIdUploadPlugin: Plugin = { + name: 'sentry-sveltekit-debug-id-upload-plugin', + apply: 'build', // only apply this plugin at build time + enforce: 'post', // this needs to be set to post, otherwise we don't pick up the output from the SvelteKit adapter resolveId: (id, _importer, _ref) => { if (id === VIRTUAL_GLOBAL_VALUES_FILE) { return { @@ -211,7 +279,7 @@ export async function makeCustomSentryVitePlugins(options?: CustomSentryVitePlug return; } - const outDir = path.resolve(process.cwd(), outputDir); + const outDir = path.resolve(process.cwd(), adapterOutputDir); // eslint-disable-next-line no-console debug && console.log('[Source Maps Plugin] Looking up source maps in', outDir); @@ -297,7 +365,7 @@ export async function makeCustomSentryVitePlugins(options?: CustomSentryVitePlug const writeBundleFn = sentryViteFileDeletionPlugin?.writeBundle; if (typeof writeBundleFn === 'function') { // This is fine though, because the original method doesn't consume any arguments in its `writeBundle` callback. - const outDir = path.resolve(process.cwd(), outputDir); + const outDir = path.resolve(process.cwd(), adapterOutputDir); try { // @ts-expect-error - the writeBundle hook expects two args we can't pass in here (they're only available in `writeBundle`) await writeBundleFn({ dir: outDir }); @@ -326,12 +394,59 @@ export async function makeCustomSentryVitePlugins(options?: CustomSentryVitePlug return [ ...unchangedSentryVitePlugins, + sourceMapSettingsPlugin, customReleaseManagementPlugin, customDebugIdUploadPlugin, customFileDeletionPlugin, ]; } +/** + * Whether the user enabled (true, 'hidden', 'inline') or disabled (false) source maps + */ +export type UserSourceMapSetting = 'enabled' | 'disabled' | 'unset' | undefined; + +/** There are 3 ways to set up source map generation (https://github.com/getsentry/sentry-javascript/issues/13993) + * + * 1. User explicitly disabled source maps + * - keep this setting (emit a warning that errors won't be unminified in Sentry) + * - We won't upload anything + * + * 2. Users enabled source map generation (true, 'hidden', 'inline'). + * - keep this setting (don't do anything - like deletion - besides uploading) + * + * 3. Users didn't set source maps generation + * - we enable 'hidden' source maps generation + * - configure `filesToDeleteAfterUpload` to delete all .map files (we emit a log about this) + * + * --> only exported for testing + */ +export function getUpdatedSourceMapSetting(viteConfig: { + build?: { + sourcemap?: boolean | 'inline' | 'hidden'; + }; +}): { updatedSourceMapSetting: boolean | 'inline' | 'hidden'; previousSourceMapSetting: UserSourceMapSetting } { + let previousSourceMapSetting: UserSourceMapSetting; + let updatedSourceMapSetting: boolean | 'inline' | 'hidden' | undefined; + + viteConfig.build = viteConfig.build || {}; + + const viteSourceMap = viteConfig.build.sourcemap; + + if (viteSourceMap === false) { + previousSourceMapSetting = 'disabled'; + updatedSourceMapSetting = viteSourceMap; + } else if (viteSourceMap && ['hidden', 'inline', true].includes(viteSourceMap)) { + previousSourceMapSetting = 'enabled'; + updatedSourceMapSetting = viteSourceMap; + } else { + previousSourceMapSetting = 'unset'; + updatedSourceMapSetting = 'hidden'; + } + + return { previousSourceMapSetting, updatedSourceMapSetting }; +} + function getFiles(dir: string): string[] { if (!fs.existsSync(dir)) { return []; diff --git a/packages/sveltekit/test/vite/sentrySvelteKitPlugins.test.ts b/packages/sveltekit/test/vite/sentrySvelteKitPlugins.test.ts index f5fa7327fe49..14977f6978d1 100644 --- a/packages/sveltekit/test/vite/sentrySvelteKitPlugins.test.ts +++ b/packages/sveltekit/test/vite/sentrySvelteKitPlugins.test.ts @@ -43,7 +43,7 @@ describe('sentrySvelteKit()', () => { expect(plugins).toBeInstanceOf(Array); // 1 auto instrument plugin + 5 source maps plugins - expect(plugins).toHaveLength(7); + expect(plugins).toHaveLength(8); }); it('returns the custom sentry source maps upload plugin, unmodified sourcemaps plugins and the auto-instrument plugin by default', async () => { @@ -56,6 +56,7 @@ describe('sentrySvelteKit()', () => { 'sentry-telemetry-plugin', 'sentry-vite-release-injection-plugin', 'sentry-vite-debug-id-injection-plugin', + 'sentry-sveltekit-update-source-map-setting-plugin', // custom release plugin: 'sentry-sveltekit-release-management-plugin', // custom source maps plugin: @@ -86,7 +87,7 @@ describe('sentrySvelteKit()', () => { it("doesn't return the auto instrument plugin if autoInstrument is `false`", async () => { const plugins = await getSentrySvelteKitPlugins({ autoInstrument: false }); const pluginNames = plugins.map(plugin => plugin.name); - expect(plugins).toHaveLength(6); + expect(plugins).toHaveLength(7); expect(pluginNames).not.toContain('sentry-upload-source-maps'); }); diff --git a/packages/sveltekit/test/vite/sourceMaps.test.ts b/packages/sveltekit/test/vite/sourceMaps.test.ts index 9837067ec643..378cbd2099e1 100644 --- a/packages/sveltekit/test/vite/sourceMaps.test.ts +++ b/packages/sveltekit/test/vite/sourceMaps.test.ts @@ -1,7 +1,9 @@ import { beforeEach, describe, expect, it, vi } from 'vitest'; +import { makeCustomSentryVitePlugins } from '../../src/vite/sourceMaps'; import type { Plugin } from 'vite'; -import { makeCustomSentryVitePlugins } from '../../src/vite/sourceMaps'; + +import * as vite from 'vite'; const mockedViteDebugIdUploadPlugin = { name: 'sentry-vite-debug-id-upload-plugin', @@ -18,6 +20,15 @@ const mockedFileDeletionPlugin = { writeBundle: vi.fn(), }; +vi.mock('vite', async () => { + const original = (await vi.importActual('vite')) as any; + + return { + ...original, + loadConfigFromFile: vi.fn(), + }; +}); + vi.mock('@sentry/vite-plugin', async () => { const original = (await vi.importActual('@sentry/vite-plugin')) as any; @@ -55,7 +66,7 @@ async function getSentryViteSubPlugin(name: string): Promise return plugins.find(plugin => plugin.name === name); } -describe('makeCustomSentryVitePlugin()', () => { +describe('makeCustomSentryVitePlugins()', () => { it('returns the custom sentry source maps plugin', async () => { const plugin = await getSentryViteSubPlugin('sentry-sveltekit-debug-id-upload-plugin'); @@ -66,7 +77,6 @@ describe('makeCustomSentryVitePlugin()', () => { expect(plugin?.resolveId).toBeInstanceOf(Function); expect(plugin?.transform).toBeInstanceOf(Function); - expect(plugin?.config).toBeInstanceOf(Function); expect(plugin?.configResolved).toBeInstanceOf(Function); // instead of writeBundle, this plugin uses closeBundle @@ -74,20 +84,89 @@ describe('makeCustomSentryVitePlugin()', () => { expect(plugin?.writeBundle).toBeUndefined(); }); - describe('Custom debug id source maps plugin plugin', () => { - it('enables source map generation', async () => { - const plugin = await getSentryViteSubPlugin('sentry-sveltekit-debug-id-upload-plugin'); + describe('Custom source map settings update plugin', () => { + beforeEach(() => { + // @ts-expect-error - this global variable is set/accessed in src/vite/sourceMaps.ts + globalThis._sentry_sourceMapSetting = undefined; + }); + it('returns the custom sentry source maps plugin', async () => { + const plugin = await getSentryViteSubPlugin('sentry-sveltekit-update-source-map-setting-plugin'); + + expect(plugin).toEqual({ + name: 'sentry-sveltekit-update-source-map-setting-plugin', + apply: 'build', + config: expect.any(Function), + }); + }); + + it('keeps source map generation settings when previously enabled', async () => { + const originalConfig = { + build: { sourcemap: true, assetsDir: 'assets' }, + }; + + vi.spyOn(vite, 'loadConfigFromFile').mockResolvedValueOnce({ + path: '', + config: originalConfig, + dependencies: [], + }); + + const plugin = await getSentryViteSubPlugin('sentry-sveltekit-update-source-map-setting-plugin'); + + // @ts-expect-error this function exists! + const sentryConfig = plugin.config(originalConfig); + + expect(sentryConfig).toEqual(originalConfig); + }); + + it('keeps source map generation settings when previously disabled', async () => { + const originalConfig = { + build: { sourcemap: false, assetsDir: 'assets' }, + }; + + vi.spyOn(vite, 'loadConfigFromFile').mockResolvedValueOnce({ + path: '', + config: originalConfig, + dependencies: [], + }); + + const plugin = await getSentryViteSubPlugin('sentry-sveltekit-update-source-map-setting-plugin'); + // @ts-expect-error this function exists! - const sentrifiedConfig = plugin.config({ build: { foo: {} }, test: {} }); - expect(sentrifiedConfig).toEqual({ + const sentryConfig = plugin.config(originalConfig); + + expect(sentryConfig).toEqual({ build: { - foo: {}, - sourcemap: true, + ...originalConfig.build, + sourcemap: false, }, - test: {}, }); }); + it('enables source map generation with "hidden" when unset', async () => { + const originalConfig = { + build: { assetsDir: 'assets' }, + }; + + vi.spyOn(vite, 'loadConfigFromFile').mockResolvedValueOnce({ + path: '', + config: originalConfig, + dependencies: [], + }); + + const plugin = await getSentryViteSubPlugin('sentry-sveltekit-update-source-map-setting-plugin'); + // @ts-expect-error this function exists! + const sentryConfig = plugin.config(originalConfig); + expect(sentryConfig).toEqual({ + ...originalConfig, + build: { + ...originalConfig.build, + sourcemap: 'hidden', + }, + }); + }); + }); + + describe('Custom debug id source maps plugin plugin', () => { it('injects the output dir into the server hooks file', async () => { const plugin = await getSentryViteSubPlugin('sentry-sveltekit-debug-id-upload-plugin'); // @ts-expect-error this function exists! @@ -237,3 +316,26 @@ describe('makeCustomSentryVitePlugin()', () => { }); }); }); + +describe('changeViteSourceMapSettings()', () => { + const cases = [ + { sourcemap: false, expectedSourcemap: false, expectedPrevious: 'disabled' }, + { sourcemap: 'hidden', expectedSourcemap: 'hidden', expectedPrevious: 'enabled' }, + { sourcemap: 'inline', expectedSourcemap: 'inline', expectedPrevious: 'enabled' }, + { sourcemap: true, expectedSourcemap: true, expectedPrevious: 'enabled' }, + { sourcemap: undefined, expectedSourcemap: 'hidden', expectedPrevious: 'unset' }, + ]; + + it.each(cases)('handles vite source map settings $1', async ({ sourcemap, expectedSourcemap, expectedPrevious }) => { + const viteConfig = { build: { sourcemap } }; + + const { getUpdatedSourceMapSetting } = await import('../../src/vite/sourceMaps'); + + const result = getUpdatedSourceMapSetting(viteConfig); + + expect(result).toEqual({ + updatedSourceMapSetting: expectedSourcemap, + previousSourceMapSetting: expectedPrevious, + }); + }); +});