dbug - npm Package Compare versions

Comparing version 0.1.0 to 0.1.1

lib/dbug.js

@@ -7,21 +7,38 @@ /* This Source Code Form is subject to the terms of the Mozilla Public
 
-const names = [];
-const skips = [];
 const colors = [6, 2, 3, 4, 5, 1];
 var prevColor = 0;
-var useColor = tty.isatty(2) || process.env.DEBUG_COLOR;
 
-(process.env.DEBUG || '')
-  .split(/[\s,]+/)
-  .forEach(function(name){
-    name = name.replace('*', '.*?');
-    if (name[0] === '-') {
-      skips.push(new RegExp('^' + name.substr(1) + '$'));
-    } else {
-      names.push(new RegExp('^' + name + '(:.+)?$'));
-    }
-  });
+function envDebugColor() {
+  try {
+    return !!JSON.parse(process.env.DEBUG_COLOR);
+  } catch (ex) {
+    return false;
+  }
+}
 
+function environ() {
+  var useColor = process.env.DEBUG_COLOR ? envDebugColor() : tty.isatty(2);
+  var names = [];
+  var skips = [];
+  if (process.env.DEBUG) {
+    process.env.DEBUG
+      .split(/[\s,]+/)
+      .forEach(function(name){
+        name = name.replace('*', '.*?');
+        if (name[0] === '-') {
+          skips.push(new RegExp('^' + name.substr(1) + '$'));
+        } else {
+          names.push(new RegExp('^' + name + '(:.+)?$'));
+        }
+      });
+  }
 
+  return {
+    names: names,
+    skips: skips,
+    useColor: useColor
+  };
+}
 
+
 function color() {
@@ -50,3 +67,3 @@ return colors[prevColor++ % colors.length];
 
-function enable(name) {
+function enable(name, useColor) {
   var c;
@@ -79,2 +96,3 @@ var format = useColor ? function colored(level, text) {
   dbug.enabled = true;
+  dbug[format.name] = true;
   dbug.log = logAt('log');
@@ -90,3 +108,4 @@ dbug.debug = dbug.log;
 module.exports = function dbug(name) {
-  var match = skips.some(function(re){
+  var env = environ();
+  var match = env.skips.some(function(re){
     return re.test(name);
@@ -99,3 +118,3 @@ });
 
-  match = names.some(function(re){
+  match = env.names.some(function(re){
     return re.test(name);
@@ -109,3 +128,3 @@ });
 
-  return enable(name);
+  return enable(name, env.useColor);
 };

package.json

 {
   "name": "dbug",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "debug improvements",
@@ -5,0 +5,0 @@ "main": "./lib/dbug.js",

test/dbug.js

@@ -56,2 +56,8 @@ /* This Source Code Form is subject to the terms of the Mozilla Public
         assert(!food.enabled);
+      },
+      'should check process.env every time': function() {
+        process.env.DEBUG = 'derp';
+        assert(require('../')('derp').enabled);
+        process.env.DEBUG = 'foo';
+        assert(!require('../')('derp:herp').enabled);
       }
@@ -97,4 +103,3 @@ },
         process.stderr.write = err;
-      },
-
+      }
     },
@@ -127,2 +132,12 @@ 'disabled': {
 
+    'color': {
+      'should default to colored on tty': function() {
+        assert.equal(foo.colored, require('tty').isatty(2));
+      },
+      'should look for DEBUG_COLOR to override tty': function() {
+        process.env.DEBUG_COLOR = false;
+        assert(require('../')('foo').plain);
+      }
+    },
+
     'after': function() {
@@ -129,0 +144,0 @@ //console._stdout = process.stdout;

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

25700

increased by3.92%

Lines of code

233

increased by16.5%

Worsened metrics

Number of low supply chain risk alerts

8

increased by166.67%

No dependency changes