From 66c78f8ed566d39b64ef5d810fc80c14de355cfc Mon Sep 17 00:00:00 2001
From: Remy Sharp <remy@remysharp.com>
Date: Wed, 20 Dec 2017 22:53:47 +0000
Subject: [PATCH 1/2] fix: properly ignore defaults, don't match partial

---
 lib/config/defaults.js |  2 +-
 lib/monitor/watch.js   | 41 +++++++++++++++++++++++++----------------
 2 files changed, 26 insertions(+), 17 deletions(-)

diff --git a/lib/config/defaults.js b/lib/config/defaults.js
index 9a55b215..90c5d3fb 100644
--- a/lib/config/defaults.js
+++ b/lib/config/defaults.js
@@ -1,4 +1,4 @@
-var ignoreRoot = require('ignore-by-default').directories()
+var ignoreRoot = require('ignore-by-default').directories();
 
 // default options for config.options
 module.exports = {
diff --git a/lib/monitor/watch.js b/lib/monitor/watch.js
index bcd6d04d..5b4f95fb 100644
--- a/lib/monitor/watch.js
+++ b/lib/monitor/watch.js
@@ -32,19 +32,20 @@ function watch() {
   var dirs = [].slice.call(config.dirs);
 
   debugRoot('start watch on: %s', dirs.join(', '));
-  debugRoot('ignore dirs regex(%s)', config.options.ignore.re);
+  const rootIgnored = config.options.ignore.map(_ => `**/${_}`);
+  debugRoot('ignored', rootIgnored);
 
   var promises = [];
   var watchedFiles = [];
 
   dirs.forEach(function (dir) {
     var promise = new Promise(function (resolve) {
-      var ignored = config.options.ignore.re;
-      var dotFilePattern = /[\/\\]\./;
+      var dotFilePattern = /[/\\]\./;
+      var ignored = Array.from(rootIgnored);
 
       // don't ignore dotfiles if explicitly watched.
       if (!dir.match(dotFilePattern)) {
-        ignored = [ignored, dotFilePattern];
+        ignored.push(dotFilePattern);
       }
 
       var watchOptions = {
@@ -62,7 +63,8 @@ function watch() {
         watchOptions.useFsEvents = false;
       }
 
-      var watcher = chokidar.watch(dir,
+      var watcher = chokidar.watch(
+        dir,
         Object.assign({}, watchOptions, config.watchOptions || {})
       );
 
@@ -89,9 +91,11 @@ function watch() {
 
       watcher.on('error', function (error) {
         if (error.code === 'EINVAL') {
-          utils.log.error('Internal watch failed. Likely cause: too many ' +
+          utils.log.error(
+            'Internal watch failed. Likely cause: too many ' +
             'files being watched (perhaps from the root of a drive?\n' +
-            'See https://github.com/paulmillr/chokidar/issues/229 for details');
+            'See https://github.com/paulmillr/chokidar/issues/229 for details'
+          );
         } else {
           utils.log.error('Internal watch failed: ' + error.message);
           process.exit(1);
@@ -128,10 +132,14 @@ function filterAndRestart(files) {
     }
 
     var cwd = process.cwd();
-    utils.log.detail('files triggering change check: ' +
-      files.map(function (file) {
-        return path.relative(cwd, file);
-      }).join(', '));
+    utils.log.detail(
+      'files triggering change check: ' +
+      files
+        .map(function (file) {
+          return path.relative(cwd, file);
+        })
+        .join(', ')
+    );
 
     var matched = match(
       files,
@@ -150,15 +158,17 @@ function filterAndRestart(files) {
             matched = {
               result: [file],
               total: 1,
-            }
+            };
             return true;
           }
-        })
+        });
       }
     }
 
-    utils.log.detail('changes after filters (before/after): ' +
-      [files.length, matched.result.length].join('/'));
+    utils.log.detail(
+      'changes after filters (before/after): ' +
+      [files.length, matched.result.length].join('/')
+    );
 
     // reset the last check so we're only looking at recently modified files
     config.lastStarted = Date.now();
@@ -177,7 +187,6 @@ function filterAndRestart(files) {
   }
 }
 
-
 function restartBus(matched) {
   utils.log.status('restarting due to changes...');
   matched.result.map(function (file) {

From 03f8a39f22d7aa61c0435d0cf5c9dd912ca339f2 Mon Sep 17 00:00:00 2001
From: Remy Sharp <remy@remysharp.com>
Date: Wed, 20 Dec 2017 22:56:49 +0000
Subject: [PATCH 2/2] refactor: remove anymatch tweak to defaults.js

---
 lib/config/defaults.js | 2 +-
 lib/monitor/watch.js   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/config/defaults.js b/lib/config/defaults.js
index 90c5d3fb..092da50e 100644
--- a/lib/config/defaults.js
+++ b/lib/config/defaults.js
@@ -11,7 +11,7 @@ module.exports = {
     // compatible with linux, mac and windows, or make the default.js
     // dynamically append the `.cmd` for node based utilities
   },
-  ignoreRoot: ignoreRoot,
+  ignoreRoot: ignoreRoot.map(_ => `**/${_}`),
   watch: ['*.*'],
   stdin: true,
   runOnChangeOnly: false,
diff --git a/lib/monitor/watch.js b/lib/monitor/watch.js
index 5b4f95fb..67aba74f 100644
--- a/lib/monitor/watch.js
+++ b/lib/monitor/watch.js
@@ -32,7 +32,7 @@ function watch() {
   var dirs = [].slice.call(config.dirs);
 
   debugRoot('start watch on: %s', dirs.join(', '));
-  const rootIgnored = config.options.ignore.map(_ => `**/${_}`);
+  const rootIgnored = config.options.ignore;
   debugRoot('ignored', rootIgnored);
 
   var promises = [];