fix: better readability on monitor output

Mattlk13 · Sep 9, 2018 · f455f65 · f455f65
1 parent 1a30329
commit f455f65
Showing 1 changed file with 16 additions and 12 deletions.
diff --git a/cli/commands/monitor.js b/cli/commands/monitor.js
@@ -14,6 +14,8 @@ var detect = require('../../lib/detect');
 var plugins = require('../../lib/plugins');
 var ModuleInfo = require('../../lib/module-info');
 
+var SEPARATOR = '\n-------------------------------------------------------\n';
+
 function monitor() {
   var args = [].slice.call(arguments, 0);
   var options = {};
@@ -150,11 +152,14 @@ function monitor() {
             if (res.ok) {
               return res.data;
             }
-            if (res.data && res.data.cliMessage) {
-              return chalk.bold.red(res.data.cliMessage);
-            }
-            return '\nFor path `' + res.path + '`, ' + res.data.message;
-          }).join('\n');
+
+            var errorMessage = (res.data && res.data.cliMessage) ?
+              chalk.bold.red(res.data.cliMessage) :
+              (res.data ? res.data.message : 'Unknown error occurred.');
+
+            return chalk.bold.white('\nMonitoring ' + res.path + '...\n\n') +
+              errorMessage;
+          }).join('\n' + SEPARATOR);
 
           if (results.every(function (res) {
             return res.ok;
@@ -169,14 +174,13 @@ function monitor() {
 
 function formatMonitorOutput(packageManager, res, manageUrl, isJson) {
   var issues = res.licensesPolicy ? 'issues' : 'vulnerabilities';
-  var strOutput = (packageManager === 'yarn' ?
-    'A yarn.lock file was detected - continuing as a Yarn project.\n\n' : '') +
-    '\n\nProject path: ' + res.path +
-    '\nCaptured a snapshot of this project\'s dependencies.\n' +
-    'Explore this snapshot at ' + res.uri + '\n\n' +
+  var strOutput = chalk.bold.white('\nMonitoring ' + res.path + '...\n\n') +
+    (packageManager === 'yarn' ?
+      'A yarn.lock file was detected - continuing as a Yarn project.\n' : '') +
+      'Explore this snapshot at ' + res.uri + '\n\n' +
     (res.isMonitored ?
-      'Notifications about newly disclosed ' + issues + ' related\n' +
-      'to these dependencies will be emailed to you.' :
+      'Notifications about newly disclosed ' + issues + ' related ' +
+      'to these dependencies will be emailed to you.\n' :
       chalk.bold.red('Project is inactive, so notifications are turned ' +
         'off.\nActivate this project here: ' + manageUrl + '\n\n')) +
     (res.trialStarted ?