Use plugin names as binary name fallback (fixes #743)

packages/knip/fixtures/plugins/husky-v9-1/.husky/pre-commit

@@ -0,0 +1 @@
+lint-staged

packages/knip/fixtures/plugins/husky-v9-1/package.json

@@ -0,0 +1,10 @@
+{
+  "name": "@fixtures/husky-v9-1",
+  "scripts": {
+    "prepare": "husky"
+  },
+  "devDependencies": {
+    "husky": "^9.1.3",
+    "lint-staged": "^15.2.7"
+  }
+}

packages/knip/src/binaries/bash-parser.ts

@@ -1,5 +1,5 @@
 import parse, { type Assignment, type ExpansionNode, type Node, type Prefix } from '../../vendor/bash-parser/index.js';
-import { pluginArgsMap } from '../plugins.js';
+import { Plugins, pluginArgsMap } from '../plugins.js';
 import type { GetInputsFromScriptsOptions } from '../types/config.js';
 import { debugLogObject } from '../util/debug.js';
 import { type Input, toBinary, toDeferResolve } from '../util/input.js';
@@ -84,6 +84,10 @@ export const getDependenciesFromScript = (script: string, options: GetInputsFrom
             return [toBinary(binary), ...getDependenciesFromScript(command, options)];
           }
 
+          if (binary in Plugins) {
+            return [...fallbackResolve(binary, args, { ...options, fromArgs }), ...fromNodeOptions];
+          }
+
           // Before using the fallback resolver, we need a way to bail out for scripts in CI environments like GitHub
           // Actions, which are provisioned with lots of unknown global binaries.
           if (options.knownBinsOnly && !text?.startsWith('.')) return [];

packages/knip/test/plugins/husky-v9-1.test.ts

@@ -0,0 +1,19 @@
+import { test } from 'bun:test';
+import assert from 'node:assert/strict';
+import { main } from '../../src/index.js';
+import { resolve } from '../../src/util/path.js';
+import baseArguments from '../helpers/baseArguments.js';
+import baseCounters from '../helpers/baseCounters.js';
+
+const cwd = resolve('fixtures/plugins/husky-v9-1');
+
+test('Find dependencies with husky plugin (v9.1)', async () => {
+  const { counters } = await main({
+    ...baseArguments,
+    cwd,
+  });
+
+  assert.deepEqual(counters, {
+    ...baseCounters,
+  });
+});