yarn run with bin in local node_modules

[billxinli]

Do you want to request a feature or report a bug?
Report a bug

What is the current behavior?
From npm with respect to npm run: In addition to the shell's pre-existing PATH, npm run adds node_modules/.bin to the PATH provided to scripts. Any binaries provided by locally-installed dependencies can be used without the node_modules/.bin prefix.

If the current behavior is a bug, please provide the steps to reproduce.

Add additional entry to the scripts section of the package.json without installing the dependency globally (in this case mocha).

  "scripts": {
    "test-api": "mocha test/api"
  },

What is the expected behavior?

If I run npm run test-api it will resolve mocha to the local copy. However yarn run test-api will produce:

$ "mocha test/api" 
sh: 1: mocha test/api: not found

Please mention your node.js, yarn and operating system version.
node: 6.7.0
yarn: 0.15
os: Ubuntu 16.04

[Daniel15]

Thanks for the bug report! This is pretty important. I'll try to take a look tonight.

[notahat]

Duplicate with more info here: #735

[Daniel15]

@notahat - That's actually a separate issue. This issue is saying that yarn run is not running executables in the node_modules/.bin directory (eg. mocha test/api in a script should essentially run node_modules/.bin/mocha test/api if mocha is an npm dependency of the app), whereas your issue #735 is about running shell commands (I think?)

[notahat]

@Daniel15 I think it's the same thing.

It's not that yarn isn't finding stuff in node_modules/.bin, it's that yarn is shelling out in such a way that the command is mocha test/api, rather than mocha with an argument of test/api. Not surprisingly, the shell can't find an executable called mocha test/api.

[Daniel15]

Ohh I see! That might be it. I'll take a look tonight 😄

[slmgc]

@Daniel15 @notahat I think it's the root of the issue:

echo `npm bin` #path_to_the_project/node_modules/.bin
echo `yarn bin` #path_to_the_project/node_modules/.bin/node_modules/.bin

yarn duplicates /node_modules/.bin part.

Edit: see my next comment.

[notahat]

@slmgc I don't see that when I try it.

$ cd /path/to/project
$ yarn bin
/path/to/project/node_modules/.bin

BUT, if I do this:

$ cd /path/to/project/node_modules/.bin
$ yarn bin
/path/to/project/node_modules/.bin/node_modules/.bin

That definitely isn't right, but I don't think it's the source of the problem as I'm seeing it.

[f0rr0]

@slmgc I don't see that either. Both paths are identical

[nodesocket]

Both look identical for me as well.

➜  js git:(master) npm bin
/Users/justin/Sites/my-project/js/node_modules/.bin
➜  js git:(master) yarn bin
/Users/justin/Sites/my-project/js/node_modules/.bin

Though I am encountering this error (see issue 782).