From ce0ac2c77b571fa6ae21bb0fff6d9605b7ec68bc Mon Sep 17 00:00:00 2001
From: Charles de Dreuille <charles.dedreuille@gmail.com>
Date: Sun, 14 Sep 2025 22:25:06 +0100
Subject: [PATCH] Update main.ts

Signed-off-by: Charles de Dreuille <charles.dedreuille@gmail.com>
---
 .storybook/main.ts | 26 +++++++++++++++++++-------
 1 file changed, 19 insertions(+), 7 deletions(-)

diff --git a/.storybook/main.ts b/.storybook/main.ts
index 74d84b55ab..1b9e4e2bde 100644
--- a/.storybook/main.ts
+++ b/.storybook/main.ts
@@ -42,7 +42,7 @@ const config: StorybookConfig = {
     name: getAbsolutePath('@storybook/react-vite'),
     options: {},
   },
-  viteFinal: async config => {
+  viteFinal: async (config, { configType }) => {
     // Add Node.js polyfills for browser compatibility
     //
     // When upgrading from Storybook 8 to 9 with the react-vite framework,
@@ -61,12 +61,24 @@ const config: StorybookConfig = {
     //
     // Without these, Backstage components that rely on Node.js APIs will fail
     // to load in Storybook's browser environment.
-    config.define = {
-      ...config.define,
-      global: 'globalThis',
-      'process.env': '{}',
-      process: '{ env: {}, browser: true }',
-    };
+    // Different configurations for development vs production
+    if (configType === 'DEVELOPMENT') {
+      // Development: Include process polyfill to prevent runtime errors
+      config.define = {
+        ...config.define,
+        global: 'globalThis',
+        'process.env': {},
+        process: '({ env: {}, browser: true })',
+      };
+    } else if (configType === 'PRODUCTION') {
+      // Production: Minimal define to avoid esbuild errors
+      config.define = {
+        ...config.define,
+        global: 'globalThis',
+        'process.env': {},
+        // No process polyfill in production build
+      };
+    }
 
     config.resolve = {
       ...config.resolve,