Merge pull request #2854 from evidence-dev/hotfix/dev-server-logging

[Hotfix] dev server logging
evidence-dev · Nov 25, 2024 · b7088a6 · b7088a6
2 parents 251ec81 + f1f7d93
commit b7088a6
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 6 deletions.
diff --git a/.changeset/unlucky-ants-enjoy.md b/.changeset/unlucky-ants-enjoy.md
@@ -0,0 +1,5 @@
+---
+'@evidence-dev/evidence': patch
+---
+
+re-enable logging on dev mode
diff --git a/packages/evidence/cli.js b/packages/evidence/cli.js
@@ -173,7 +173,8 @@ const buildHelper = function (command, args) {
 			...process.env,
 			// used for source query HMR
 			EVIDENCE_DATA_URL_PREFIX: process.env.EVIDENCE_DATA_URL_PREFIX ?? 'static/data',
-			EVIDENCE_DATA_DIR: process.env.EVIDENCE_DATA_DIR ?? './static/data'
+			EVIDENCE_DATA_DIR: process.env.EVIDENCE_DATA_DIR ?? './static/data',
+			EVIDENCE_IS_BUILDING: 'true'
 		}
 	});
 	// Copy the outputs to the root of the project upon successful exit

diff --git a/packages/evidence/scripts/build-template.js b/packages/evidence/scripts/build-template.js
@@ -102,7 +102,14 @@ fsExtra.outputFileSync(
 		customLogger: logger
     }
 
-	if (isDebug()) {
+	// Suppress errors when building in non-debug mode
+	if (!isDebug() && process.env.EVIDENCE_IS_BUILDING === 'true') {
+		config.logLevel = 'silent';
+		logger.error = (msg) => log.error(msg);
+		logger.info = () => {};
+		logger.warn = () => {};
+		logger.warnOnce = () => {};
+	} else {
 		const loggerWarn = logger.warn;
 		const loggerOnce = logger.warnOnce
 
@@ -124,10 +131,6 @@ fsExtra.outputFileSync(
 
 			loggerWarn(msg, options);
 		};
-	} else {
-		config.logLevel = 'silent';
-		logger.error = (msg) => log.error(msg);
-		logger.info = logger.warn = logger.warnOnce = () => {};
 	}
 
     export default config`