feat(commands): Add reload command for http-protocol comms

Author: shakyShane

bin/browser-sync.js

@@ -6,9 +6,7 @@ var fs            = require("fs");
 var path          = require("path");
 var compile       = require("eazy-logger").compile;
 var utils         = require("../lib/utils");
-var flags         = require("../lib/cli/opts");
-var flagKeys      = Object.keys(flags);
-
+var logger        = require("../lib/logger").logger;
 var cmdWhitelist  = ["start", "init", "reload"];
 
 var cli = meow({
@@ -29,20 +27,23 @@ if (!module.parent) {
  * @returns {String}
  */
 function getHelpText(filepath) {
-    return compile(
-        fs.readFileSync(
-            filepath,
-            "utf8"
-        ).replace(
-            "%flags%",
-            listFlags(
-                flags,
-                longest(
-                    flagKeys
-                ).length
-            )
-        )
-    );
+
+    /**
+     * Help text template
+     */
+    var template = fs.readFileSync(filepath, "utf8");
+
+    cmdWhitelist.forEach(function (command) {
+
+        var flags = require("../lib/cli/opts." + command + ".json");
+
+        template = template.replace(
+            ["%", command, "flags%"].join(""),
+            listFlags(flags)
+        );
+    });
+
+    return compile(template);
 }
 
 /**
@@ -54,7 +55,6 @@ function handleCli (opts) {
     opts.cb = opts.cb || utils.defaultCallback;
 
     var input = opts.cli.input;
-    var flags = opts.cli.flags;
 
     if (!opts.whitelist) {
         opts.whitelist = cmdWhitelist;
@@ -64,24 +64,28 @@ function handleCli (opts) {
         return console.log(opts.cli.help);
     }
 
-    if (input[0] === "start") {
-        require("../lib/cli/command.start")(opts);
-    }
-
-    if (input[0] === "init") {
-        require("../lib/cli/command.init")(opts);
+    if (!require("../lib/cli/cli-utils").verifyOpts(input[0], opts.cli.flags)) {
+        logger.info("For help, run: {cyan:browser-sync --help}");
+        return opts.cb(new Error("Unknown flag given. Please refer to the documentation for help."));
     }
 
+    return require("../lib/cli/command." + input[0])(opts);
 }
 
 /**
  * @param {Object} flags
- * @param {Number} longest
- * @returns {String}
  */
-function listFlags (flags, longest) {
+function listFlags (flags) {
+
+    var flagKeys = Object.keys(flags);
+    var longest = getLongest(Object.keys(flags));
+
+    if (!longest || !longest.length) {
+        return;
+    }
+
     return flagKeys.reduce(function (all, item) {
-        return all + "    {bold:--" + item + "}" + getPadding(item.length, longest + 8) + flags[item] + "\n";
+        return all + "    {bold:--" + item + "}" + getPadding(item.length, longest.length + 8) + flags[item] + "\n";
     }, "");
 }
 
@@ -98,7 +102,7 @@ function getPadding (len, max) {
  * @param {Array} arr
  * @returns {String}
  */
-function longest (arr) {
+function getLongest (arr) {
     return arr.sort(function (a, b) { return b.length - a.length; })[0];
 }
 

lib/async.js

@@ -234,16 +234,12 @@ module.exports = {
      */
     addHttpProtocol: function (bs, done) {
 
-        if (!bs.options.get("httpProtocol")) {
-            return done(null);
-        }
-
         /**
          * Add a middleware to listen to http
          * requests in the BrowserSync http protocol namespace
          */
         bs.addMiddleware(
-            bs.options.getIn(["httpProtocol", "path"]),
+            require("./config").httpProtocol.path,
             require("./http-protocol").middleware(bs),
             {override: true}
         );

lib/cli/cli-utils.js

@@ -2,11 +2,13 @@
 
 var utils    = exports;
 var logger   = require("../logger").logger;
-var flags    = require("./opts.json");
-var flagKeys = Object.keys(flags);
 var _        = require("lodash");
 
-utils.verifyOpts = function (flagWhitelist, cliFlags) {
+utils.verifyOpts = function (optskey, cliFlags) {
+
+    var flags         = require("./opts." + optskey + ".json");
+    var flagKeys      = Object.keys(flags);
+    var flagWhitelist = flagKeys.map(dropPrefix).map(_.camelCase);
 
     return Object.keys(cliFlags).every(function (key) {
 
@@ -19,3 +21,11 @@ utils.verifyOpts = function (flagWhitelist, cliFlags) {
         return false;
     });
 };
+
+/**
+ * @param {String} item
+ * @returns {String}
+ */
+function dropPrefix (item) {
+    return item.replace("no-", "");
+}

lib/cli/command.reload.js

@@ -0,0 +1,29 @@
+"use strict";
+
+var error = "Could not contact BrowserSync server.";
+
+/**
+ * $ browser-sync reload <options>
+ *
+ * This commands starts the BrowserSync servers
+ * & Optionally UI.
+ *
+ * @param opts
+ * @returns {Function}
+ */
+module.exports = function (opts) {
+
+    var flags = opts.cli.flags;
+    var proto = require("../http-protocol");
+
+    var url   = proto.getUrl({method: "reload", args: flags.files}, "http://localhost:" + (flags.port || 3000));
+
+    require("http").get(url, function (res) {
+        if (res.statusCode !== 200) {
+            require("logger").logger.error(error);
+            return opts.cb(new Error(error));
+        } else {
+            opts.cb(null, res);
+        }
+    });
+};

lib/cli/command.start.js

@@ -1,11 +1,6 @@
 "use strict";
 
-var logger        = require("../logger").logger;
-var info          = require("./cli-info");
-var flags         = require("./opts.json");
-var _             = require("lodash");
-var flagKeys      = Object.keys(flags);
-var flagWhitelist = flagKeys.map(dropPrefix).map(_.camelCase);
+var info = require("./cli-info");
 
 /**
  * $ browser-sync start <options>
@@ -20,20 +15,7 @@ module.exports = function (opts) {
 
     var flags = opts.cli.flags;
 
-    if (!require("./cli-utils").verifyOpts(flagWhitelist, flags)) {
-        logger.info("For help, run: {cyan:browser-sync --help}");
-        return opts.cb(new Error("Unknown flag given. Please refer to the documentation for help."));
-    }
-
     return require("../../")
         .create("cli")
         .init(flags.config ? info.getConfigFile(flags.config) : flags, opts.cb);
 };
-
-/**
- * @param {String} item
- * @returns {String}
- */
-function dropPrefix (item) {
-    return item.replace("no-", "");
-}

lib/cli/help.txt

@@ -8,11 +8,17 @@
 
     {bold:init}    Creates a default config file
     {bold:start}   Start Browser Sync
+    {bold:reload}  Send a reload event over HTTP protocol
 
-{bold:Options:}
+{bold:`start` Options:}
 {gray:--------}
 
-%flags%
+%startflags%
+
+{bold:`reload` Options:}
+{gray:--------}
+
+%reloadflags%
 
 {cyan:Server Example:}
 {gray:---------------}

lib/cli/opts.init.json

@@ -0,0 +1 @@
+{}

lib/cli/opts.reload.json

@@ -0,0 +1,4 @@
+{
+    "files": "File paths to reload",
+    "port":  "Target a running instance by port number"
+}

lib/cli/opts.json renamed to lib/cli/opts.start.json

@@ -17,6 +17,9 @@ module.exports = {
     configFile: "/default-config.js",
     userFile: "/bs-config.js",
     template: "/cli-template.js",
+    httpProtocol: {
+        path: "/__browser_sync__"
+    },
     client: {
         shims: "/client/client-shims.js"
     },