Merge pull request #10 from BitcoinErrorLog/fix/phase1-critical-fixes

fix: Phase 1 Critical Fixes - Test Suite and Source Maps

Author: BitcoinErrorLog

src/popup/__tests__/App.integration.test.tsx

@@ -127,11 +127,14 @@ describe('Popup App Integration', () => {
 
   const renderPopup = async () => {
     const { SessionProvider } = await import('../../contexts/SessionContext');
+    const { ThemeProvider } = await import('../../contexts/ThemeContext');
     await act(async () => {
       render(
-        <SessionProvider>
-          <App />
-        </SessionProvider>
+        <ThemeProvider>
+          <SessionProvider>
+            <App />
+          </SessionProvider>
+        </ThemeProvider>
       );
     });
   };

src/sidepanel/__tests__/App.integration.test.tsx

@@ -175,11 +175,14 @@ describe('Sidepanel App Integration', () => {
 
   const renderSidepanel = async () => {
     const { SessionProvider } = await import('../../contexts/SessionContext');
+    const { ThemeProvider } = await import('../../contexts/ThemeContext');
     await act(async () => {
       render(
-        <SessionProvider>
-          <App />
-        </SessionProvider>
+        <ThemeProvider>
+          <SessionProvider>
+            <App />
+          </SessionProvider>
+        </ThemeProvider>
       );
     });
   };

vite.config.ts

@@ -39,7 +39,7 @@ export default defineConfig({
   plugins: [react(), copyStaticFiles()],
   build: {
     outDir: 'dist',
-    sourcemap: true,
+    sourcemap: process.env.NODE_ENV !== 'production',
     minify: 'esbuild',
     target: 'es2020',
     rollupOptions: {

vite.content.config.ts

@@ -5,7 +5,7 @@ import { resolve } from 'path';
 export default defineConfig({
   build: {
     outDir: 'dist',
-    sourcemap: true,
+    sourcemap: process.env.NODE_ENV !== 'production',
     minify: 'esbuild',
     target: 'es2020',
     emptyOutDir: false, // Don't clear the dist folder (main build already created it)