refactor: consolidate .env use

Fix bug in script assuming same .env paths for new apps
novuhq · Jun 3, 2024 · 75803c9 · 75803c9
1 parent 1333cd0
commit 75803c9
Showing 1 changed file with 5 additions and 8 deletions.
diff --git a/scripts/setup-env-files.js b/scripts/setup-env-files.js
@@ -1,8 +1,9 @@
 const fs = require('fs');
 const path = require('path');
 
-const prePopulateEnv = (folders, folderBasePath, exampleEnvFilePath = 'src/.example.env', envFilePath = 'src/.env') => {
-  for (const folder of folders) {
+const prePopulateEnv = (apps, folderBasePath, exampleEnvFilePath = 'src/.example.env', envFilePath = 'src/.env') => {
+  console.log(`Pre-populating .env files from .example.env for [${apps.join(',')}]`);
+  for (const folder of apps) {
     const exists = fs.existsSync(path.resolve(`${folderBasePath}/${folder}/${envFilePath}`));
     if (!exists) {
       console.log(`Populating ${folderBasePath}/${folder} with .env file`);
@@ -15,14 +16,10 @@ const prePopulateEnv = (folders, folderBasePath, exampleEnvFilePath = 'src/.exam
 };
 
 (async () => {
-  const apps = ['api', 'ws', 'worker', 'web', 'widget'];
   const appsBasePath = `${__dirname}/../apps`;
-
   console.log('----------------------------------------');
-  console.log('Pre-populating .env files from .example.env');
-
-  prePopulateEnv(apps, appsBasePath);
-
+  prePopulateEnv(['api', 'ws', 'worker'], appsBasePath);
+  prePopulateEnv(['web', 'widget'], appsBasePath, '.example.env', '.env');
   console.log('Finished populating .env files');
   console.log('----------------------------------------');
 })();