Refactor environment variable handling in webpack configuration to return missing variables and log warnings, improving build resilience.

Author: kanushka

common/scripts/env-webpack-helper.js

@@ -23,7 +23,7 @@
  * 3. Only define variables that are explicitly declared in .env file
  * 
  * @param {Object} env - Parsed environment variables from .env file (from dotenv.config().parsed)
- * @returns {Object} Environment variables object ready for webpack.DefinePlugin
+ * @returns {Object} Object containing envKeys for webpack.DefinePlugin and missingVars array
  */
 function createEnvDefinePlugin(env) {
 
@@ -44,14 +44,7 @@ function createEnvDefinePlugin(env) {
       });
     }
 
-    if (missingVars.length > 0) {
-      throw new Error(
-        `Missing required environment variables: ${missingVars.join(', ')}\n` +
-        `Please provide values in either .env file or runtime environment.\n`
-      );
-    }
-  
-    return envKeys;
+    return { envKeys, missingVars };
   }
 
   module.exports = {

workspaces/ballerina/ballerina-extension/webpack.config.js

@@ -10,15 +10,14 @@ const { createEnvDefinePlugin } = require('../../../common/scripts/env-webpack-h
 
 const envPath = path.resolve(__dirname, '.env');
 const env = dotenv.config({ path: envPath }).parsed;
-
-let envKeys;
-try {
-  envKeys = createEnvDefinePlugin(env);
-} catch (error) {
-  console.warn('\n⚠️  Environment Variable Configuration Warning:');
-  console.warn(error.message);
-  console.warn('Continuing build with empty environment variables...');
-  envKeys = {};
+console.log("Fetching values for environment variables...");
+const { envKeys, missingVars } = createEnvDefinePlugin(env);
+if (missingVars.length > 0) {
+  console.warn(
+    '\n⚠️  Environment Variable Configuration Warning:\n' +
+    `Missing required environment variables: ${missingVars.join(', ')}\n` +
+    `Please provide values in either .env file or runtime environment.\n`
+  );
 }
 
 /** @type {import('webpack').Configuration} */

workspaces/mi/mi-extension/webpack.config.js

@@ -27,15 +27,14 @@ const { createEnvDefinePlugin } = require('../../../common/scripts/env-webpack-h
 
 const envPath = path.resolve(__dirname, '.env');
 const env = dotenv.config({ path: envPath }).parsed;
-
-let envKeys;
-try {
-  envKeys = createEnvDefinePlugin(env);
-} catch (error) {
-  console.warn('\n⚠️  Environment Variable Configuration Warning:');
-  console.warn(error.message);
-  console.warn('Continuing build with empty environment variables...');
-  envKeys = {};
+console.log("Fetching values for environment variables...");
+const { envKeys, missingVars } = createEnvDefinePlugin(env);
+if (missingVars.length > 0) {
+  console.warn(
+    '\n⚠️  Environment Variable Configuration Warning:\n' +
+    `Missing required environment variables: ${missingVars.join(', ')}\n` +
+    `Please provide values in either .env file or runtime environment.\n`
+  );
 }
 
 /** @type {import('webpack').Configuration} */

workspaces/wso2-platform/wso2-platform-extension/webpack.config.js

@@ -26,16 +26,16 @@ const { createEnvDefinePlugin } = require('../../../common/scripts/env-webpack-h
 
 const envPath = path.resolve(__dirname, ".env");
 const env = dotenv.config({ path: envPath }).parsed;
-
-let envKeys;
-try {
-  	envKeys = createEnvDefinePlugin(env);
-} catch (error) {
-	console.warn('\n⚠️  Environment Variable Configuration Warning:');
-	console.warn(error.message);
-	console.warn('Continuing build with empty environment variables...');
-	envKeys = {};
+console.log("Fetching values for environment variables...");
+const { envKeys, missingVars } = createEnvDefinePlugin(env);
+if (missingVars.length > 0) {
+  console.warn(
+    '\n⚠️  Environment Variable Configuration Warning:\n' +
+    `Missing required environment variables: ${missingVars.join(', ')}\n` +
+    `Please provide values in either .env file or runtime environment.\n`
+  );
 }
+
 //@ts-check
 /** @typedef {import('webpack').Configuration} WebpackConfig **/