fix: build (#188)

Author: bdougie

src/lib/env.ts

@@ -19,12 +19,10 @@ const isBrowser = typeof window !== 'undefined';
 function getEnvVar(viteKey: string, serverKey?: string): string {
   if (isBrowser) {
     // Browser: Only access VITE_* prefixed variables via import.meta.env
-    try {
-      return (import.meta as any).env?.[viteKey] || '';
-    } catch {
-      // Fallback if import.meta is not available
-      return '';
-    }
+    // Use optional chaining and fallback for production compatibility
+    const metaEnv = (typeof import.meta !== 'undefined' && (import.meta as any).env) || {};
+    const value = metaEnv[viteKey];
+    return value || '';
   } else {
     // Server: Use process.env only (import.meta.env not available in CommonJS/Netlify Functions)
     return process.env[viteKey] || (serverKey ? process.env[serverKey] : '') || '';
@@ -64,33 +62,24 @@ export const env = {
   // Development mode detection
   get DEV() {
     if (isBrowser) {
-      try {
-        return (import.meta as any).env?.DEV || false;
-      } catch {
-        return false;
-      }
+      const metaEnv = (typeof import.meta !== 'undefined' && (import.meta as any).env) || {};
+      return metaEnv.DEV || false;
     }
     return process.env.NODE_ENV === 'development';
   },
 
   get PROD() {
     if (isBrowser) {
-      try {
-        return (import.meta as any).env?.PROD || false;
-      } catch {
-        return false;
-      }
+      const metaEnv = (typeof import.meta !== 'undefined' && (import.meta as any).env) || {};
+      return metaEnv.PROD || false;
     }
     return process.env.NODE_ENV === 'production';
   },
 
   get MODE() {
     if (isBrowser) {
-      try {
-        return (import.meta as any).env?.MODE || 'development';
-      } catch {
-        return 'development';
-      }
+      const metaEnv = (typeof import.meta !== 'undefined' && (import.meta as any).env) || {};
+      return metaEnv.MODE || 'development';
     }
     return process.env.NODE_ENV || 'development';
   },

vite.config.ts

@@ -62,8 +62,15 @@ export default defineConfig({
       // Remove the external configuration as it's causing build issues
       output: {
         // Ensure proper file extensions for module recognition
-        entryFileNames: 'assets/[name]-[hash].js',
-        chunkFileNames: 'assets/[name]-[hash].js',
+        entryFileNames: (chunkInfo) => {
+          // Force .js extension for all entry files, including App
+          const name = chunkInfo.name?.replace(/\.tsx?$/, '') || 'chunk';
+          return `assets/${name}-[hash].js`;
+        },
+        chunkFileNames: (chunkInfo) => {
+          // Force .js extension for all chunk files
+          return `assets/${chunkInfo.name}-[hash].js`;
+        },
         assetFileNames: 'assets/[name]-[hash].[ext]',
         // Proven chunk splitting strategy from LIGHTHOUSE_OPTIMIZATIONS.md
         manualChunks: {