patch for properly removing deleted files during watch

lib/commands/watch.js

@@ -31,39 +31,33 @@ var _watch = function(){
 
     server.compiling();
 
-    // if there was an error, the whole project needs to be recompiled to
-    // get rid of the error message
-    if (global.options.error) {
-      options.debug.log('error, reloading project')
-      global.options.error = false;
-      return roots.compile_project(current_directory, server.reload);
-    }
+    // make sure the file wasn't deleted
+    if (fs.existsSync(file.fullPath)){
 
-    // if it's a dynamic file, the entire project needs to be recompiled
-    // so that references to it show up in other files
-    if (yaml_parser.detect(file.fullPath)){
-      options.debug.log('dynamic file changed, reloading project')
-      return roots.compile_project(current_directory, server.reload);
-    }
+      // if there was an error, the whole project needs to be recompiled to
+      // get rid of the error message
+      if (global.options.error) return compile_project('error');
+
+      // if it's a dynamic file, the entire project needs to be recompiled
+      // so that references to it show up in other files
+      if (yaml_parser.detect(file.fullPath)) return compile_project('dynamic file')
 
-    // ignored files that are modified actively are often dependencies
-    // for another non-ignored file. Until we have something like assetgraph
-    // in this project, the safest approach is to recompile the whole project
-    // when an ignored file is modified.
-    var ignored = global.options.ignore_files;
+      // ignored files that are modified are often dependencies
+      // for another non-ignored file. Until we have an asset graph
+      // in this project, the safest approach is to recompile the
+      // whole project when an ignored file is modified.
+      var ignored = global.options.ignore_files;
 
-    for (var i = 0; i < ignored.length; i++){
-      if (minimatch(path.basename(file.path), ignored[i].slice(1))) {
-        options.debug.log('ignored file changed, reloading project')
-        return roots.compile_project(current_directory, server.reload);
+      for (var i = 0; i < ignored.length; i++){
+        if (minimatch(path.basename(file.path), ignored[i].slice(1))) {
+          options.debug.log('ignored file changed, reloading project')
+          return roots.compile_project(current_directory, server.reload);
+        }
       }
-    }
 
-    // otherwise, just compile the single file that was changed
-    if (fs.existsSync(file.fullPath)){
-      options.debug.log('single file compile')
-      roots.compile_project(file.fullPath, server.reload);
+      compile_single_file(file.fullPath);
     } else {
+      // if the changed file was deleted, just remove it in the public folder
       try {
         fs.unlinkSync(output_path(file.fullPath));
       } catch(e) {
@@ -77,3 +71,14 @@ var _watch = function(){
 };
 
 module.exports = { execute: _watch, needs_config: true };
+
+function compile_single_file(file_path){
+  options.debug.log('single file compile');
+  roots.compile_project(file_path, server.reload);
+}
+
+function compile_project(reason){
+  options.debug.log(reason + ": full project compile");
+  global.options.error = false;
+  return roots.compile_project(current_directory, server.reload);
+}

package.json

@@ -1,6 +1,6 @@
 {
   "name": "roots",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "description": "roots is a toolbox for building simple, beautiful, and efficient products for the web",
   "keywords": [
     "front-end",