build(frontend): determine minification based on build mode

hotosm · Feb 25, 2024 · e81724e · e81724e
1 parent 328c4d5
commit e81724e
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 19 deletions.
diff --git a/docker-compose.development.yml b/docker-compose.development.yml
@@ -115,6 +115,7 @@ services:
       args:
         APP_VERSION: ${GIT_BRANCH}
         VITE_API_URL: https://${FMTM_API_DOMAIN:-api.${FMTM_DOMAIN}}
+        NODE_ENV: development
     volumes:
       - fmtm_frontend:/frontend
     network_mode: none

diff --git a/src/frontend/prod.dockerfile b/src/frontend/prod.dockerfile
@@ -10,9 +10,10 @@ ENV PATH="$PNPM_HOME:$PATH"
 RUN corepack enable
 RUN pnpm install
 
-ENV NODE_ENV production
+ARG NODE_ENV=production
+ENV NODE_ENV ${NODE_ENV}
 COPY . .
-RUN pnpm run build
+RUN pnpm run build --mode ${NODE_ENV}
 
 
 

diff --git a/src/frontend/vite.config.ts b/src/frontend/vite.config.ts
@@ -5,23 +5,28 @@ import react from '@vitejs/plugin-react';
 import { VitePWA } from 'vite-plugin-pwa';
 
 // https://vitejs.dev/config/
-export default defineConfig({
-  plugins: [react(), VitePWA({ registerType: 'autoUpdate' })],
-  server: {
-    port: 7051,
-    host: '0.0.0.0',
-    watch: {
-      usePolling: true,
+export default defineConfig(({ mode }) => {
+  return {
+    plugins: [react(), VitePWA({ registerType: 'autoUpdate' })],
+    server: {
+      port: 7051,
+      host: '0.0.0.0',
+      watch: {
+        usePolling: true,
+      },
     },
-  },
-  resolve: {
-    alias: {
-      '@': path.resolve(__dirname, 'src'),
+    build: {
+      minify: mode === 'development' ? false : 'esbuild',
     },
-  },
-  test: {
-    globals: true,
-    environment: 'jsdom',
-    setupFiles: './setupTests.ts',
-  },
+    resolve: {
+      alias: {
+        '@': path.resolve(__dirname, 'src'),
+      },
+    },
+    test: {
+      globals: true,
+      environment: 'jsdom',
+      setupFiles: './setupTests.ts',
+    },
+  };
 });