Implement .done() without bluebird

Author: mwilliamson

.eslintrc.json

@@ -68,8 +68,10 @@
     "console": true,
     "exports": true,
     "module": true,
+    "process": false,
     "Promise": false,
     "require": false,
+    "setTimeout": false,
     "TextDecoder": false,
     "Uint8Array": false
   }

lib/promises.js

@@ -1,5 +1,4 @@
 var _ = require("underscore");
-var bluebird = require("bluebird/js/release/promise")();
 
 exports.defer = defer;
 exports.extend = extend;
@@ -115,6 +114,33 @@ function forEachSeries(array, func) {
  * added.
  */
 function toExternalPromise(promise) {
-    // TODO: stop using bluebird.
-    return bluebird.resolve(promise);
+    var externalPromise = Promise.resolve(promise);
+
+    externalPromise.done = function(resolve, reject) {
+        return promise.then(resolve, reject).catch(function(error) {
+            if (isNode()) {
+                var stack = error instanceof Error ? error.stack : error;
+                process.stderr.write("Fatal " + stack + "\n");
+                process.exit(2);
+            } else {
+                setTimeout(function() {
+                    throw error;
+                }, 0);
+            }
+        });
+    };
+
+    return externalPromise;
+}
+
+/**
+ * Detect whether we're running in node.js. This uses the same logic as in
+ * bluebird for consistency with previous versions of Mammoth.
+ */
+function isNode() {
+    if (typeof process === "undefined") {
+        return false;
+    }
+
+    return {}.toString.call(process).toLowerCase() === "[object process]";
 }

test/.eslintrc.json

@@ -2,7 +2,6 @@
   "globals": {
     "__dirname": false,
     "describe": false,
-    "it": false,
-    "setTimeout": false
+    "it": false
   }
 }