Fix resolveBin for windows about kcd-scripts

[kentcdodds]

🎉 This issue has been resolved in version 1.3.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[adamdharrington]

nice one @kentcdodds !

[rossipedia]

Ok I'm not sure I understand what the problem is. The executable for Windows is the *.cmd file, which is a thin wrapper around invoking node.exe with pkg.bin.

For example, the contents of node_modules\.bin\webpack.cmd is:

@IF EXIST "%~dp0\node.exe" (
  "%~dp0\node.exe"  "%~dp0\..\webpack\bin\webpack.js" %*
) ELSE (
  @SETLOCAL
  @SET PATHEXT=%PATHEXT:;.JS;=;%
  node  "%~dp0\..\webpack\bin\webpack.js" %*
)

and likewise, the contents of which.cmd is:

@IF EXIST "%~dp0\node.exe" (
  "%~dp0\node.exe"  "%~dp0\..\which\bin\which" %*
) ELSE (
  @SETLOCAL
  @SET PATHEXT=%PATHEXT:;.JS;=;%
  node  "%~dp0\..\which\bin\which" %*
)

As you can see, they follow the same general format, the only thing differing is the path to the *.js file that node will execute.

This is required because in Windows, executable files are determined by file extension rather than a permissions bit as they are in *nix land.

The *.cmd file can be treated as the executable, and can be started via child_process.exec or child_process.spawn and it should just do the right thing.

[rossipedia]

Ok, I see what the issue is (I cloned the repo and ran the test suite). You might have to rethink the tests here, since the way Windows handles command lines / dir separators / executables is so different from *nix. Gotta run to the store, but I might take a crack at it later tonight.

[kentcdodds]

Thanks for looking into it! If you have another suggestion let me know! I'm thinking that with the serializers I add we can normalize the paths so it's not an issue for tests, but I definitely want it to work the same as far as the implementation goes. I'd really love it to not have the full path to the executable if the executable exists in the PATH by its name already.

[alexandernanberg]

I might be missing something, but why not use npx to run the scripts instead? From what I've seen that should remove the need for the resolveBin util altogether.

[kentcdodds]

Hmmm... Interesting thought. That might work! Would you like to give it a try?

[alexandernanberg]

Sure, I'll give it a go!