feat: support using ‘nodemonConfig’ in package.json

Implements loading configuration options from the `nodemonConfig` value in the package.json, nodemon.json is still preferred before looking at the new option.
Also includes tests.

Closes remy#873

lib/config/load.js

@@ -171,6 +171,13 @@ function loadFile(options, config, dir, ready) {
   var filename = options.configFile || path.join(dir, 'nodemon.json');
   fs.readFile(filename, 'utf8', function (err, data) {
     if (err) {
+      if (err.code === 'ENOENT') {
+        if (!options.configFile && dir !== utils.home) {
+          // if no specified local config file and local nodemon.json
+          // doesn't exist, try the package.json
+          return loadPackageJSON(config, callback);
+        }
+      }
       return callback({});
     }
 
@@ -189,4 +196,19 @@ function loadFile(options, config, dir, ready) {
   });
 
 
+}
+
+function loadPackageJSON(config, ready) {
+  if (!ready) {
+    ready = function () {};
+  }
+
+  utils.log.detail('Looking in package.json for nodemonConfig');
+
+  var dir = process.cwd();
+  var filename = path.join(dir, 'package.json');
+  var packageLoadOptions = { configFile: filename };
+  return loadFile(packageLoadOptions, config, dir, function (settings) {
+    ready(settings.nodemonConfig || {});
+  });
 }

test/config/load.test.js

@@ -107,6 +107,19 @@ describe('config load', function () {
     });
   });
 
+  it('should read package.json config', function (done) {
+    var dir = path.resolve(pwd, 'test/fixtures/packages/package-json-settings');
+    process.chdir(dir);
+
+    var config = {},
+        settings = { quiet: true },
+        options = {};
+    load(settings, options, config, function (config) {
+      assert.deepEqual(config.exec, 'foo', 'exec is "foo": ' + config.exec);
+      done();
+    });
+  });
+
   it('should give local files preference', function (done) {
     var config = {},
         settings = { quiet: true },
@@ -120,6 +133,36 @@ describe('config load', function () {
     });
   });
 
+  it('should give local files preference over package.json config', function (done) {
+    var dir = path.resolve(pwd, 'test/fixtures/packages/nodemon-settings-and-package-json-settings');
+    process.chdir(dir);
+
+    var config = {},
+        settings = { quiet: true },
+        options = {};
+    load(settings, options, config, function (config) {
+      assert.deepEqual(config.exec, 'foo', 'exec is "foo": ' + config.exec);
+      done();
+    });
+  });
+
+  it('should give package.json config preference', function (done) {
+    var dir = path.resolve(pwd, 'test/fixtures/packages/package-json-settings');
+    process.chdir(dir);
+
+    var config = {},
+        settings = { quiet: true },
+        options = {};
+    load(settings, options, config, function (config) {
+      removeRegExp(config);
+      assert.deepEqual(config.exec, 'foo', 'exec is "foo": ' + config.exec);
+      assert.ok(config.ignore.indexOf('one') !== -1, 'ignore contains "one": ' + config.ignore);
+      assert.ok(config.ignore.indexOf('three') !== -1, 'ignore contains "three": ' + config.ignore);
+      assert.deepEqual(config.watch, ['four'], 'watch is "four": ' + config.watch);
+      done();
+    });
+  });
+
   it('should give user specified settings preference', function (done) {
     var config = {},
         settings = { ignore: ['one'], watch: ['one'], quiet: true },
@@ -132,6 +175,19 @@ describe('config load', function () {
     });
   });
 
+  it('should give user specified settings preference over package.json config', function (done) {
+    var dir = path.resolve(pwd, 'test/fixtures/packages/package-json-settings');
+    process.chdir(dir);
+
+    var config = {},
+        settings = { exec: 'foo-user', quiet: true },
+        options = {};
+    load(settings, options, config, function (config) {
+      assert.deepEqual(config.exec, 'foo-user', 'exec is "foo-user": ' + config.exec);
+      done();
+    });
+  });
+
   it('should give user specified exec preference over package.scripts.start', function (done) {
     var dir = path.resolve(pwd, 'test/fixtures/packages/start-and-settings');
     process.chdir(dir);
@@ -146,6 +202,20 @@ describe('config load', function () {
     });
   });
 
+  it('should give package.json specified exec config over package.scripts.start', function (done) {
+    var dir = path.resolve(pwd, 'test/fixtures/packages/start-and-package-json-settings');
+    process.chdir(dir);
+
+    var config = {},
+        settings = {},
+        options = {};
+
+    load(settings, options, config, function (config) {
+      assert.deepEqual(config.exec, 'foo', 'exec is "foo": ' + config.exec);
+      done();
+    });
+  });
+
   // it('should put the script at the end if found in package.scripts.start', function (done) {
   //   process.chdir(path.resolve(pwd, 'test/fixtures/packages/start')); // allows us to load text/fixtures/package.json
   //   var settings = cli.parse(asCLI('--harmony'));

test/fixtures/packages/nodemon-settings-and-package-json-settings/nodemon.json

@@ -0,0 +1,3 @@
+{
+  "exec": "foo"
+}

test/fixtures/packages/nodemon-settings-and-package-json-settings/package.json

@@ -0,0 +1,5 @@
+{
+  "nodemonConfig": {
+    "exec": "foo-ignored"
+  }
+}

test/fixtures/packages/package-json-settings/package.json

@@ -0,0 +1,7 @@
+{
+  "nodemonConfig": {
+    "exec": "foo",
+    "ignore": ["one", "three"],
+    "watch": ["four"]
+  }
+}

test/fixtures/packages/start-and-package-json-settings/package.json

@@ -0,0 +1,8 @@
+{
+  "scripts": {
+    "start": "node app.js"
+  },
+  "nodemonConfig": {
+    "exec": "foo"
+  }
+}