Passing a directory path implicitly watches directory contents, undoes accidental breaking change and fixes #229

Also refactor event filtering and add logging for unexpected watched files.

Author: UltCombo

index.js

@@ -74,8 +74,11 @@ module.exports = function (globs, opts, cb) {
 
 	outputStream._read = function _read() { };
 
-	var watcher = chokidar.watch(globs, opts)
-		.on('all', processEvent);
+	var watcher = chokidar.watch(globs, opts);
+
+	opts.events.forEach(function (ev) {
+		watcher.on(ev, processEvent.bind(undefined, ev));
+	});
 
 	['add', 'change', 'unlink', 'addDir', 'unlinkDir', 'error', 'ready', 'raw']
 		.forEach(function (ev) {
@@ -95,21 +98,25 @@ module.exports = function (globs, opts, cb) {
 	};
 
 	function processEvent(event, filepath) {
-		var glob = globs[anymatch(globs, filepath, true)];
+		var glob;
+		var currentFilepath = filepath;
+		while (!(glob = globs[anymatch(globs, currentFilepath, true)]) && currentFilepath !== (currentFilepath = path.resolve(currentFilepath, '..'))) {} // eslint-disable-line no-empty-blocks/no-empty-blocks
 
 		if (!glob) {
+			util.log(
+				util.colors.cyan('[gulp-watch]'),
+				util.colors.yellow('wut? This shouldn\'t happen. Please open this link to report the issue:\n') +
+				'https://github.com/floatdrop/gulp-watch/issues/new?title=' +
+				encodeURIComponent('Watched unexpected filepath') + '&body=' +
+				encodeURIComponent('Globs: `' + JSON.stringify(globs) + '`\nFilepath: `' + filepath + '`\nOptions:\n```js\n' + JSON.stringify(opts, null, 2) + '\n```')
+			);
 			return;
 		}
 
 		if (!baseForced) {
 			opts.base = glob2base(new Glob(glob));
 		}
 
-		// React only on opts.events
-		if (opts.events.indexOf(event) === -1) {
-			return;
-		}
-
 		// Do not stat deleted files
 		if (event === 'unlink' || event === 'unlinkDir') {
 			opts.path = pathIsAbsolute(filepath) ? filepath : path.join(opts.cwd || process.cwd(), filepath);

package.json

@@ -40,8 +40,8 @@
   },
   "dependencies": {
     "anymatch": "^1.3.0",
-    "chokidar": "^1.0.3",
-    "glob": "^5.0.13",
+    "chokidar": "^1.5.2",
+    "glob": "^7.0.3",
     "glob2base": "0.0.12",
     "gulp-util": "^3.0.6",
     "path-is-absolute": "^1.0.0",

test/test-dir.js

@@ -20,14 +20,14 @@ describe('dir', function () {
 		w.close();
 	});
 
-	it('should not watch files inside directory', function (done) {
+	it('should watch files inside directory', function (done) {
 		fs.mkdirSync(fixtures('newDir'));
 		touch(fixtures('newDir/index.js'))();
-		w = watch(fixtures('newDir'), function () {
-			done('Watched unexpected file.');
+		w = watch(fixtures('newDir'), function (file) {
+			file.relative.should.eql(path.normalize('newDir/index.js'));
+			done();
 		}).on('ready', function () {
 			touch(fixtures('newDir/index.js'))('new content');
-			setTimeout(done, 200);
 		});
 	});