Not working in modules required by other modules

[inxilpro]

Not working in modules required by other modules

[inxilpro]

Can you give me a use case where you'd want to include it in a library that's required by something else? When I put it together, I considered doing a recursive loop up the path, but decided it was unnecessary because it was solving a problem that I didn't thin would ever come up. That said, I can be convinced.

[Californian]

Well to be honest, I won't be doing that any time soon. However, this is a very generally-useful module, and could potentially be used in a lot of others (which are used by still other modules), so I thought it would be a nice little contribution I could make.

[inxilpro]

Here's another possible solution:

var root = path.resolve(__dirname).split('node_modules', 1)[0];

That would take something like this:

/var/www/node_modules/mymod/node_modules/node-app-root-path/

And split it into ["/var/www/"] — then you'd just grab the first element in the array.

Can you see any issue with that?

[Californian]

Awesome, so clean and simple! That seems like a better solution even, in case it somehow has an extra folder between one node_modules and the next one.

[inxilpro]

I actually went ahead and rewrote a bunch of it to account for most edge cases. Take a look at:

https://github.com/inxilpro/node-app-root-path/tree/feature/comprehensive-resolution

[inxilpro]

0.1.0 has been released, which fixes this.