Fix case of node ../../node_modules/.bin/executable (resolves #908)

Author: webpro

packages/knip/fixtures/workspaces-tooling/node_modules/@swc/register/index.js

@@ -1,10 +1,13 @@
 {
   "name": "@workspaces-tooling/backend",
   "scripts": {
-    "lint": "eslint"
+    "lint": "eslint",
+    "typeorm": "node --require @swc/register ../../node_modules/.bin/typeorm"
   },
   "devDependencies": {
     "eslint": "*",
-    "eslint-config-custom": "*"
+    "eslint-config-custom": "*",
+    "typeorm": "*",
+    "@swc/register": "*"
   }
 }

packages/knip/fixtures/workspaces-tooling/node_modules/@swc/register/package.json

@@ -3,6 +3,7 @@ import { pluginArgsMap } from '../plugins.js';
 import type { BinaryResolver } from '../types/config.js';
 import { compact } from '../util/array.js';
 import { toBinary, toConfig, toDeferResolve, toDeferResolveEntry, toEntry } from '../util/input.js';
+import { extractBinary } from '../util/modules.js';
 import { resolve as fallbackResolve } from './fallback.js';
 
 const isGlobLikeMatch = /(^!|[*+\\(|{^$])/;
@@ -38,7 +39,10 @@ export const resolve: BinaryResolver = (binary, _args, options) => {
   if (opts.positional && parsed._[0]) {
     const id = parsed._[0]; // let's start out safe, but sometimes we'll want more
     if (isGlobLike(id)) positionals.push(toEntry(id));
-    else positionals.push(toDeferResolveEntry(id));
+    else {
+      if (id.includes('node_modules/.bin/')) positionals.push(toBinary(extractBinary(id)));
+      else positionals.push(toDeferResolveEntry(id));
+    }
   }
 
   const mapToParsedKey = (id: string) => parsed[id];

packages/knip/fixtures/workspaces-tooling/node_modules/typeorm/index.js

@@ -23,7 +23,7 @@ export const isStartsLikePackageName = (specifier: string) => /^(@[a-z0-9._]|[a-
 
 export const stripVersionFromSpecifier = (specifier: string) => specifier.replace(/(\S+)@.*/, '$1');
 
-const stripNodeModulesFromPath = (command: string) => command.replace(/^(\.\/)?node_modules\//, '');
+const stripNodeModulesFromPath = (command: string) => command.replace(/(?:\.{0,2}\/)*node_modules\//, '');
 
 export const extractBinary = (command: string) =>
   stripVersionFromSpecifier(

packages/knip/fixtures/workspaces-tooling/node_modules/typeorm/package.json

@@ -63,8 +63,8 @@ test('getInputsFromScripts (node -r)', () => {
   t('node --experimental-loader ts-node/esm/transpile-only script.js', [toBinary('node'), toDeferResolveEntry('script.js'), toDeferResolve('ts-node/esm/transpile-only')]);
   t('node -r @scope/package/register ./dir', [toBinary('node'), toDeferResolveEntry('./dir'), toDeferResolve('@scope/package/register')]);
   t('node -r @scope/package/register ./dir/index', [toBinary('node'), toDeferResolveEntry('./dir/index'), toDeferResolve('@scope/package/register')]);
-  t('node --inspect-brk -r pkg/register node_modules/.bin/exec --runInBand', [toBinary('node'), toDeferResolveEntry('node_modules/.bin/exec'), toDeferResolve('pkg/register')]);
-  t('node -r ts-node/register node_modules/.bin/jest', [toBinary('node'), toDeferResolveEntry('node_modules/.bin/jest'), toDeferResolve('ts-node/register')]);
+  t('node --inspect-brk -r pkg/register node_modules/.bin/exec --runInBand', [toBinary('node'), toBinary('exec'), toDeferResolve('pkg/register')]);
+  t('node -r ts-node/register node_modules/.bin/jest', [toBinary('node'), toBinary('jest'), toDeferResolve('ts-node/register')]);
 });
 
 test('getInputsFromScripts (ts-node)', () => {