Idea: watch only require()'d files about node-supervisor HOT 8 CLOSED

A requirement from the node-supervisor executable for this would be actually the opposite of issue #14: instead of an option to restart only on watch changes (not on crashes), this would need an option to restart only on crashes (and not watch anything, except I suppose the entry file, e.g. app.js).

from node-supervisor.

FYI, I prototyped this in our current project and it seems to work great. Here's the full source code to the helper script that we require() before requiring any of our regular source code:

https://gist.github.com/1053401

And we changed our node-supervisor usage to now specify that only the entry file (e.g. app.js in our case) should be watched, instead of the entire current directory.

The result: all false positives and false negatives eliminated, and a slight startup performance boost. Suh-weet!

Would be awesome to get this into node-supervisor directly. Let me know what you think of this.

from node-supervisor.

+1

from node-supervisor.

I think perhaps we should default to only watching to app.js or server.js file passed in... but I'm not sure. Should /node_modules also be watched?

That means doing supervisor app.js will default to watching only that file

I'm going to do this on my fork

from node-supervisor.

I think it would be very cool if we can do this without making folks change their source in any way. Maybe supervisor could sneakily prepend some code to what it is running. Or maybe we could use something like node-detective to analyse the file prior to runtime.

from node-supervisor.

FWIW, we long ago switched to @fgnass's excellent node-dev, which does this perfectly. No code changes, just node-dev foo.js arg1 arg2 instead of node foo.js arg1 arg2, and it watches only and all require()'d files.

from node-supervisor.

This ticket should be closed because node-dev already does it.

from node-supervisor.

Closing as this is probably unlikely to happen. It would a great feature to have if it didn't involve a lot of added complication and extra code, but I'll leave the messy stuff to other packages that want to deal with it. Supervisor is and will remain fairly simple.

from node-supervisor.