Auto-merge for PR #675 via VersionBot

Ensure whoami failures (i.e. present but broken tokens) at startup don't break commands

CHANGELOG.md

@@ -4,6 +4,10 @@ All notable changes to this project will be documented in this file
 automatically by Versionist. DO NOT EDIT THIS FILE MANUALLY!
 This project adheres to [Semantic Versioning](http://semver.org/).
 
+## v6.6.8 - 2017-10-06
+
+* Ensure analytics failures (e.g. from broken tokens) at startup don't break commands #675 [Tim Perry]
+
 ## v6.6.7 - 2017-09-22
 
 * Update to resin-sync, which fixes local push on windows #666 [Tim Perry]

build/app.js

@@ -230,15 +230,17 @@ capitano.command(actions.deploy);
 update.notify();
 
 plugins.register(/^resin-plugin-(.+)$/).then(function() {
-  var cli;
+  var cli, runCommand;
   cli = capitano.parse(process.argv);
-  return events.trackCommand(cli).then(function() {
+  runCommand = function() {
     var ref, ref1;
     if ((ref = cli.global) != null ? ref.help : void 0) {
       return capitanoExecuteAsync({
         command: "help " + ((ref1 = cli.command) != null ? ref1 : '')
       });
+    } else {
+      return capitanoExecuteAsync(cli);
     }
-    return capitanoExecuteAsync(cli);
-  });
+  };
+  return Promise.all([events.trackCommand(cli), runCommand()]);
 })["catch"](errors.handle);

build/events.js

@@ -22,7 +22,7 @@ exports.trackCommand = function(capitanoCommand) {
   capitanoStateGetMatchCommandAsync = Promise.promisify(require('capitano').state.getMatchCommand);
   return Promise.props({
     resinUrl: resin.settings.get('resinUrl'),
-    username: resin.auth.whoami(),
+    username: resin.auth.whoami().catchReturn(void 0),
     mixpanel: exports.getLoggerInstance()
   }).then(function(arg) {
     var mixpanel, resinUrl, username;
@@ -45,5 +45,5 @@ exports.trackCommand = function(capitanoCommand) {
         command: capitanoCommand
       });
     });
-  });
+  }).timeout(100).catchReturn();
 };

lib/app.coffee

@@ -203,9 +203,12 @@ update.notify()
 plugins.register(/^resin-plugin-(.+)$/).then ->
 	cli = capitano.parse(process.argv)
 
-	events.trackCommand(cli).then ->
+	runCommand = ->
 		if cli.global?.help
-			return capitanoExecuteAsync(command: "help #{cli.command ? ''}")
-		capitanoExecuteAsync(cli)
+			capitanoExecuteAsync(command: "help #{cli.command ? ''}")
+		else
+			capitanoExecuteAsync(cli)
+
+	Promise.all([events.trackCommand(cli), runCommand()])
 
 .catch(errors.handle)

lib/events.coffee

@@ -13,7 +13,7 @@ exports.trackCommand = (capitanoCommand) ->
 
 	return Promise.props
 		resinUrl: resin.settings.get('resinUrl')
-		username: resin.auth.whoami()
+		username: resin.auth.whoami().catchReturn(undefined)
 		mixpanel: exports.getLoggerInstance()
 	.then ({ username, resinUrl, mixpanel }) ->
 		return capitanoStateGetMatchCommandAsync(capitanoCommand.command).then (command) ->
@@ -30,3 +30,5 @@ exports.trackCommand = (capitanoCommand) ->
 				resinUrl: resinUrl
 				platform: process.platform
 				command: capitanoCommand
+	.timeout(100)
+	.catchReturn()

package.json

@@ -1,6 +1,6 @@
 {
   "name": "resin-cli",
-  "version": "6.6.7",
+  "version": "6.6.8",
   "description": "The official resin.io CLI tool",
   "main": "./build/actions/index.js",
   "homepage": "https://github.com/resin-io/resin-cli",