Incorrect path when used within both project and its dependency on npm v3

[inxilpro]

Incorrect path when used within both project and its dependency on npm v3

[inxilpro]

Can you give me a better idea of your directory structure? Like this?

app/
    node_modules/
    webapp/
        node_modules/

Or, even better, can you send me a code sample the demonstrates this behavior?

[doublemarked]

My directory structure looks like this,

   my-web-app/
      node_modules/
         app-root-path/
         my-module/

Normally, if I were to clone and npm install inside my-module, the directory structure would look like,

   my-module/
      node_modules/
         app-root-path/

However, npm v3 sees that the same version of app-root-path is used both by my-web-app and my-module and so it moves it up to flatten the dependency hierarchy.

I can put together a code sample!

[doublemarked]

Here you go:
https://github.com/doublemarked/app-root-issue-17
https://github.com/doublemarked/app-root-issue-17-module

To reproduce,

1. Clone the module by itself, npm install then node index.js
2. Then, clone the base project separately (which depends on the module), npm install and node index.js

Output is like:

Scallop:app-root-issue-17-module heath$ node index.js
MODULE APPROOT: /Users/heath/Projects/app-root-issue-17-module

Scallop:app-root-issue-17 heath$ node index.js 
BASE APPROOT:/Users/heath/Projects/app-root-issue-17
MODULE APPROOT: /Users/heath/Projects/app-root-issue-17
MYMOD: { appRootPath: '/Users/heath/Projects/app-root-issue-17' }

[inxilpro]

Oh, I inderstand. Yeah--this module isn't meant to solve that problem. app-root-path is meant to find the app's root path, regardless of the directory structure, so what you're seeing is exactly the expected behavior.

I'll try to post a code sample later tonight that would do what you're trying to do.

[inxilpro]

Here's a code sample that can get you the path to the current module, no matter where it's called:

function moduleDirectory(dir, pathSeparator) {
  var sep = pathSeparator || '/';
  var search = 'node_modules' + sep;
  var moduleIndex = dir.lastIndexOf(search);
  var sepIndex = dir.indexOf(sep, moduleIndex + search.length);

  if (-1 === sepIndex) {
    return dir;
  }

  return dir.substring(0, sepIndex);
}

See this JSBin for a demo:
http://jsbin.com/qazukeluve/1/edit?js,console