fix tests

Author: BYK

dev-packages/e2e-tests/test-applications/SPOTLIGHT_ENV_TESTS.md

@@ -21,6 +21,7 @@ Tests the **CJS build** scenario where `import.meta` must be stripped to avoid s
 **Source**: `nextjs-15/app/spotlight-env-test/page.tsx`
 
 #### Tests:
+
 1. **`respects NEXT_PUBLIC_SENTRY_SPOTLIGHT environment variable`**
    - Verifies that `NEXT_PUBLIC_SENTRY_SPOTLIGHT=true` enables Spotlight
    - Checks that the Spotlight integration is registered
@@ -39,6 +40,7 @@ Tests the **CJS build** scenario where `import.meta` must be stripped to avoid s
    - Monitors console for syntax errors
 
 #### Environment Setup:
+
 ```bash
 NEXT_PUBLIC_SENTRY_SPOTLIGHT=true
 # SENTRY_SPOTLIGHT can be set for backend, but won't be exposed to browser
@@ -52,6 +54,7 @@ Tests the **ESM build** scenario where `import.meta` should be present and funct
 **Source**: `browser-webworker-vite/src/spotlight-env-test.ts`
 
 #### Tests:
+
 1. **`respects VITE_SENTRY_SPOTLIGHT environment variable`**
    - Verifies that `VITE_SENTRY_SPOTLIGHT=true` enables Spotlight
    - Checks that the Spotlight integration is registered
@@ -78,6 +81,7 @@ Tests the **ESM build** scenario where `import.meta` should be present and funct
    - Verifies it successfully reads from `import.meta.env`
 
 #### Environment Setup:
+
 ```bash
 VITE_SENTRY_SPOTLIGHT=true
 ```
@@ -119,6 +123,7 @@ try {
 ```
 
 This code is:
+
 - **Included** in ESM builds (Vite, Astro, SvelteKit)
 - **Stripped** from CJS builds (Next.js, Webpack, etc.)
 
@@ -130,25 +135,26 @@ The shared `resolveSpotlightOptions` function filters empty/whitespace strings:
 
 ```typescript
 // Treat empty/whitespace-only strings as undefined
-const envUrl = typeof envSpotlight === 'string' && envSpotlight.trim() !== ''
-  ? envSpotlight
-  : undefined;
+const envUrl = typeof envSpotlight === 'string' && envSpotlight.trim() !== '' ? envSpotlight : undefined;
 ```
 
 This ensures:
+
 - Empty strings never enable Spotlight
 - Whitespace-only strings are treated as undefined
 - No invalid URL connections are attempted
 
 ## Running the Tests
 
 ### Next.js Tests
+
 ```bash
 cd dev-packages/e2e-tests/test-applications/nextjs-15
 NEXT_PUBLIC_SENTRY_SPOTLIGHT=true pnpm test tests/spotlight-env.test.ts
 ```
 
 ### Vite Tests
+
 ```bash
 cd dev-packages/e2e-tests/test-applications/browser-webworker-vite
 VITE_SENTRY_SPOTLIGHT=true pnpm test tests/spotlight-env.test.ts
@@ -157,13 +163,15 @@ VITE_SENTRY_SPOTLIGHT=true pnpm test tests/spotlight-env.test.ts
 ## Expected Outcomes
 
 ### Next.js (CJS)
+
 - ✅ `process.env.NEXT_PUBLIC_SENTRY_SPOTLIGHT` accessible
 - ✅ `SENTRY_SPOTLIGHT` NOT accessible (backend-only)
 - ✅ No `import.meta` syntax in output
 - ✅ No syntax errors
 - ✅ Spotlight integration enabled
 
 ### Vite (ESM)
+
 - ✅ `import.meta.env.VITE_SENTRY_SPOTLIGHT` accessible
 - ✅ `process.env.VITE_SENTRY_SPOTLIGHT` accessible (transformed)
 - ✅ `import.meta` syntax present in output
@@ -173,16 +181,19 @@ VITE_SENTRY_SPOTLIGHT=true pnpm test tests/spotlight-env.test.ts
 ## Troubleshooting
 
 ### Syntax Error: Cannot use import.meta outside a module
+
 - **Cause**: `import.meta` code not stripped from CJS build
 - **Fix**: Verify `makeStripEsmPlugin()` is applied to CJS builds
 - **Check**: Look for `/* rollup-esm-only */` comments in source
 
 ### Spotlight not enabled despite env var set
+
 - **Cause**: Empty string or wrong prefix
 - **Fix**: Use correct prefix (`NEXT_PUBLIC_*` for Next.js, `VITE_*` for Vite)
 - **Check**: Verify `resolveSpotlightOptions` receives non-empty string
 
 ### import.meta.env returns undefined in Vite
+
 - **Cause**: `import.meta` code stripped from ESM build
 - **Fix**: Verify `makeStripEsmPlugin()` is NOT applied to ESM builds
 - **Check**: ESM builds should only use `makeStripCjsPlugin()`

dev-packages/e2e-tests/test-applications/browser-webworker-vite/tests/spotlight-env.test.ts

@@ -83,10 +83,8 @@ test.describe('Spotlight environment variable handling in Vite (ESM)', () => {
     await page.waitForTimeout(1000);
 
     // Should have no syntax errors
-    const syntaxErrors = [...consoleErrors, ...pageErrors].filter(err =>
-      err.includes('SyntaxError') ||
-      err.includes('Unexpected token') ||
-      err.includes('Cannot use import.meta')
+    const syntaxErrors = [...consoleErrors, ...pageErrors].filter(
+      err => err.includes('SyntaxError') || err.includes('Unexpected token') || err.includes('Cannot use import.meta'),
     );
 
     expect(syntaxErrors).toHaveLength(0);

dev-packages/e2e-tests/test-applications/nextjs-15/app/spotlight-env-test/page.tsx

@@ -32,9 +32,7 @@ export default function SpotlightEnvTestPage() {
         <div data-testid="next-public-spotlight">
           NEXT_PUBLIC_SENTRY_SPOTLIGHT: {envVars.NEXT_PUBLIC_SENTRY_SPOTLIGHT || 'undefined'}
         </div>
-        <div data-testid="sentry-spotlight">
-          SENTRY_SPOTLIGHT: {envVars.SENTRY_SPOTLIGHT || 'undefined'}
-        </div>
+        <div data-testid="sentry-spotlight">SENTRY_SPOTLIGHT: {envVars.SENTRY_SPOTLIGHT || 'undefined'}</div>
       </div>
       <div id="spotlight-status" data-testid="spotlight-status">
         <h2>Spotlight Integration Status</h2>

dev-packages/e2e-tests/test-applications/nextjs-15/tests/spotlight-env.test.ts

@@ -59,10 +59,8 @@ test.describe('Spotlight environment variable handling', () => {
     await page.waitForTimeout(1000);
 
     // Should have no syntax errors from import.meta in CJS build
-    const syntaxErrors = consoleErrors.filter(err =>
-      err.includes('import.meta') ||
-      err.includes('SyntaxError') ||
-      err.includes('Unexpected token')
+    const syntaxErrors = consoleErrors.filter(
+      err => err.includes('import.meta') || err.includes('SyntaxError') || err.includes('Unexpected token'),
     );
 
     expect(syntaxErrors).toHaveLength(0);

packages/browser/test/build-artifacts.test.ts

@@ -1,6 +1,6 @@
 /**
  * Tests to verify build artifacts are correctly formatted for their module type.
- * 
+ *
  * These tests verify that:
  * - CJS builds don't contain import.meta (would cause syntax errors)
  * - ESM builds DO contain import.meta.env checks
@@ -22,21 +22,25 @@ describe('Build Artifacts - CJS vs ESM', () => {
 
       if (fs.existsSync(cjsDevPath)) {
         const cjsDevContent = fs.readFileSync(cjsDevPath, 'utf-8');
-        
+
         // Should NOT contain actual import.meta code (only in comments is ok)
-        const lines = cjsDevContent.split('\n').filter(line => !line.trim().startsWith('//') && !line.trim().startsWith('*'));
+        const lines = cjsDevContent
+          .split('\n')
+          .filter(line => !line.trim().startsWith('//') && !line.trim().startsWith('*'));
         const codeOnly = lines.join('\n');
-        
+
         expect(codeOnly).not.toContain('import.meta.env');
         expect(codeOnly).not.toContain('typeof import.meta');
       }
 
       if (fs.existsSync(cjsProdPath)) {
         const cjsProdContent = fs.readFileSync(cjsProdPath, 'utf-8');
-        
-        const lines = cjsProdContent.split('\n').filter(line => !line.trim().startsWith('//') && !line.trim().startsWith('*'));
+
+        const lines = cjsProdContent
+          .split('\n')
+          .filter(line => !line.trim().startsWith('//') && !line.trim().startsWith('*'));
         const codeOnly = lines.join('\n');
-        
+
         expect(codeOnly).not.toContain('import.meta.env');
         expect(codeOnly).not.toContain('typeof import.meta');
       }
@@ -47,7 +51,7 @@ describe('Build Artifacts - CJS vs ESM', () => {
 
       if (fs.existsSync(cjsDevPath)) {
         const content = fs.readFileSync(cjsDevPath, 'utf-8');
-        
+
         // SHOULD contain process.env check
         expect(content).toContain('process.env');
       }
@@ -61,15 +65,15 @@ describe('Build Artifacts - CJS vs ESM', () => {
 
       if (fs.existsSync(esmDevPath)) {
         const content = fs.readFileSync(esmDevPath, 'utf-8');
-        
+
         // SHOULD contain import.meta checks
         expect(content).toContain('import.meta');
         expect(content).toContain('import.meta.env');
       }
 
       if (fs.existsSync(esmProdPath)) {
         const content = fs.readFileSync(esmProdPath, 'utf-8');
-        
+
         expect(content).toContain('import.meta');
         expect(content).toContain('import.meta.env');
       }
@@ -80,7 +84,7 @@ describe('Build Artifacts - CJS vs ESM', () => {
 
       if (fs.existsSync(esmDevPath)) {
         const content = fs.readFileSync(esmDevPath, 'utf-8');
-        
+
         // SHOULD contain the rollup markers (they're kept in output as comments)
         expect(content).toContain('rollup-esm-only');
       }
@@ -91,7 +95,7 @@ describe('Build Artifacts - CJS vs ESM', () => {
 
       if (fs.existsSync(esmDevPath)) {
         const content = fs.readFileSync(esmDevPath, 'utf-8');
-        
+
         // SHOULD contain process.env check too (for compatibility)
         expect(content).toContain('process.env');
       }
@@ -115,4 +119,3 @@ describe('Build Artifacts - CJS vs ESM', () => {
     });
   });
 });
-