when running a coffee file, use --nodejs correctly

Previously, the --nodejs flag was being prepended if you had any
arguments prefixed with '--', even if they were intended for your
application. There is an additional problem where arguments intended for
node js need to be grouped into a single argument passed after --nodejs.

After this patch, any flag before your script is considered a nodejs
argument. These, along with any other exec arguments are bundled into a
single argument separated by spaces.

Fixes #530

lib/config/exec.js

@@ -121,15 +121,18 @@ function exec(nodemonOptions, execMap) {
   // note: indexOf('coffee') handles both .coffee and .litcoffee
   if (!execDefined && options.exec === 'node' && scriptExt.indexOf('coffee') !== -1) {
     options.exec = 'coffee';
-    // ensure that we call: `coffee --nodejs ...`
-    if (options.execArgs === undefined) {
-      options.execArgs = [];
-    }
-    // if there's a leading argument to the exec that starts with `--` then
-    // it could be --debug or --debug-brk or something else intended for node
-    // so we'll add the --nodejs switch.
-    if ((options.args || []).join(' ').indexOf('--') === 0) {
-      options.execArgs.unshift('--nodejs');
+
+    // we need to get execArgs set before the script
+    // for example, in `nodemon --debug my-script.coffee --my-flag`, debug is an
+    // execArg, while my-flag is a script arg
+    var leadingArgs = (options.args || []).splice(0, options.scriptPosition);
+    options.execArgs = options.execArgs.concat(leadingArgs);
+    options.scriptPosition = 0;
+
+    if (options.execArgs.length > 0) {
+      // because this is the coffee executable, we need to combine the exec args
+      // into a single argument after the nodejs flag
+      options.execArgs = ['--nodejs', options.execArgs.join(' ')];
     }
   }
 

test/cli/exec.test.js

@@ -87,7 +87,7 @@ describe('nodemon exec', function () {
     var options = exec({ script: 'app.coffee', nodeArgs: [ '--debug' ] });
 
     assert(options.exec.indexOf('coffee') === 0, 'using coffeescript to execute');
-    assert(options.execArgs.indexOf('--debug') !== -1);
+    assert(options.execArgs[1].indexOf('--debug') !== -1);
     assert(options.ext.indexOf('coffee') !== -1);
   });