fix: only set env.DEBUG_COLORS='true' for the plugin (#51)

Author: erezrokah

src/lighthouse.js

@@ -1,11 +1,25 @@
-// prevent logger from prefixing a date when running in tty
+// we set DEBUG_COLORS = 'true' to prevent the logger from prefixing a date when running in tty
+// keep the old DEBUG_COLORS value so we can return it to the original value
+let debugColorsSet = false;
+let debugColorsOriginalValue;
+if ('DEBUG_COLORS' in process.env) {
+  debugColorsSet = true;
+  debugColorsOriginalValue = process.env.DEBUG_COLORS;
+}
 process.env.DEBUG_COLORS = 'true';
 
 const puppeteer = require('puppeteer');
 const lighthouse = require('lighthouse');
 const log = require('lighthouse-logger');
 const chromeLauncher = require('chrome-launcher');
 
+// we can return the original value after requiring the dependencies
+if (debugColorsSet) {
+  process.env.DEBUG_COLORS = debugColorsOriginalValue;
+} else {
+  delete process.env.DEBUG_COLORS;
+}
+
 const getBrowserPath = async () => {
   const browserFetcher = puppeteer.createBrowserFetcher();
   const revisions = await browserFetcher.localRevisions();