I just gave the code a quick look and I find this `eval` usage not too cool:

https://github.com/harthur/replace/blob/master/replace.js#L4...

It relies on the fact that you can provide javascripts String#replace a "replace function" (1). The author just evals the contents of a command line provided file into a variable called `replaceFunc`.

Beside that, there's not a lot that can be pointed out as "bad", at least by just reading it lightly (5-10 mins).

[1] https://developer.mozilla.org/en-US/docs/JavaScript/Referenc...