watcher: adding cwd + ignore defaults to all

Author: shakyShane

examples/server.watch.js

@@ -0,0 +1,20 @@
+/**
+ *
+ * Install:
+ *      npm install browser-sync
+ *
+ * Run:
+ *      node <yourfile.js>
+ *
+ * This example will create a server using https using the default information & use the `app` directory as the root
+ *
+ */
+
+"use strict";
+
+var browserSync = require("browser-sync").create();
+
+browserSync.init({
+    server: "test/fixtures",
+    watch: true
+});

lib/cli/cli-options.ts

@@ -10,6 +10,7 @@ import { handleProxyOption } from "./transforms/handleProxyOption";
 import { handleServerOption } from "./transforms/handleServerOption";
 import { appendServerIndexOption } from "./transforms/appendServerIndexOption";
 import { appendServerDirectoryOption } from "./transforms/appendServerDirectoryOption";
+import {addCwdToWatchOptions} from "./transforms/addCwdToWatchOptions";
 
 const _ = require("../lodash.custom");
 const defaultConfig = require("../default-config");
@@ -23,6 +24,7 @@ export function merge(input) {
     const merged = immDefs.mergeDeep(input);
     const transforms = [
         addToFilesOption,
+        addCwdToWatchOptions,
         addDefaultIgnorePatterns,
         copyCLIIgnoreToWatchOptions,
         handleServerOption,

lib/cli/transforms/addCwdToWatchOptions.ts

@@ -0,0 +1,5 @@
+export function addCwdToWatchOptions(incoming) {
+    return incoming.updateIn(['watchOptions', 'cwd'], (watchCwd) => {
+        return watchCwd || incoming.get('cwd');
+    })
+}

lib/cli/transforms/addDefaultIgnorePatterns.ts

@@ -1,6 +1,6 @@
 import {List} from "immutable";
 
-const defaultIgorePatterns = [
+const defaultIgnorePatterns = [
     /node_modules/,
     /bower_components/,
     /\.sass-cache/,
@@ -10,21 +10,16 @@ const defaultIgorePatterns = [
 ];
 
 export function addDefaultIgnorePatterns(incoming) {
-    if (!incoming.get("watch")) {
-        return incoming;
-    }
-
     return incoming.update("watchOptions", watchOptions => {
         const userIgnored = List([])
             .concat(watchOptions.get("ignored"))
             .filter(Boolean)
             .toSet();
 
-        const merged = userIgnored.merge(defaultIgorePatterns);
+        const merged = userIgnored.merge(defaultIgnorePatterns);
 
         return watchOptions.merge({
             ignored: merged.toList(),
-            cwd: incoming.get("cwd")
         });
     });
 }

lib/file-watcher.js

@@ -79,6 +79,10 @@ function watch(patterns, opts, cb) {
         watcher.on("all", cb);
     }
 
+    // watcher.on('ready', () => {
+    //     console.log(watcher.getWatched());
+    // });
+
     return watcher;
 }
 

test/specs/cli/cli.options.ignore.js

@@ -0,0 +1,13 @@
+var cli = require("../../../dist/cli/cli-options");
+var merge = cli.merge;
+var assert = require("chai").assert;
+
+describe("CLI: Options: dealing with 'ignore' option", function() {
+    it("watches in server mode (no files given)", function() {
+        var cwd = '/Users/shakyshane/app';
+        var input = { server: true, files: ['**/*'], ignore: ['**/*.php'], cwd: cwd };
+        var config = merge(input).toJS();
+        assert.deepEqual(config.files, { core: { globs: ["**/*"], objs: [] } });
+        assert.ok(config.watchOptions.ignored.indexOf('**/*.php') > -1);
+    });
+});

test/specs/cli/cli.options.watchOptions.cwd.js

@@ -0,0 +1,12 @@
+var cli = require("../../../dist/cli/cli-options");
+var merge = cli.merge;
+var assert = require("chai").assert;
+
+describe("ensures the CWD is transferred to the watchOptions.cwd ", function() {
+    it("add cwd with files option", function() {
+        var cwd = '/Users/shakyshane/app';
+        var input = { server: true, files: "**/*.html", cwd: cwd };
+        var config = merge(input).toJS();
+        assert.deepEqual(config.watchOptions.cwd, '/Users/shakyshane/app');
+    });
+});