knip 2.31.0 → 2.32.0

package/README.md

@@ -266,6 +266,7 @@ Knip contains a growing list of plugins:
 - [markdownlint][plugin-markdownlint]
 - [Mocha][plugin-mocha]
 - [Next.js][plugin-next]
+- [Node.js Test Runner][plugin-node-test-runner]
 - [npm-package-json-lint][plugin-npm-package-json-lint]
 - [Nx][plugin-nx]
 - [nyc][plugin-nyc]
@@ -310,35 +311,28 @@ unused or missing.
 Other configuration files use `require` or `import` statements to use dependencies, so they don't need special handing
 and can be analyzed like any other source file. That's why these configuration files are also used as `entry` files.
 
-#### Override plugin configuration
+#### Plugin configuration
 
-Usually, no custom configuration is required for plugins, but if your project uses custom file locations then Knip
-allows you to override any defaults. Let's take Cypress for example. By default it uses `cypress.config.js`, but your
-project uses `config/cypress.js`. Also, the default pattern for test files is `cypress/e2e/**/*.cy.js`, but your project
-has them at `e2e-tests/*.spec.ts`. Here's how to configure this:
+Is a plugin not behaving as expected? There are a few options to modify its behavior:
 
-```json
-{
-  "cypress": {
-    "entry": ["config/cypress.js", "e2e-tests/*.spec.js"]
-  }
-}
-```
-
-When overriding any plugin's configuration, the options object is fully replaced. Look at the plugin's page to find the
-default configuration and use that as a basis. Here's another example to override the test file pattern for Vitest:
+- Disable a plugin by setting its value to `false` (.e.g `"webpack": false`)
+- Force-enable a plugin by setting its value to `true`
+- Override a plugin's `config` or `entry` location, for example:
 
 ```json
 {
-  "vitest": {
-    "config": ["vitest.config.ts"],
-    "entry": ["**/*.vitest.ts"]
+  "mocha": {
+    "config": "config/mocha.config.js",
+    "entry": ["**/*.spec.js"]
   }
 }
 ```
 
-This plugin configuration can be set on root and workspace level. If set on root level, it can be disabled on workspace
-level by setting it to `false` there.
+It's rarely necessary to override the `entry` patterns, since plugins also read custom test file patterns from the
+tooling configuration (e.g. from the `include` option for Vitest or `testMatch` for Playwright).
+
+Plugin configuration can be set on root and workspace level. If enabled on root level, it can be disabled on workspace
+level by setting it to `false` there, and vice versa.
 
 #### Multi-project repositories
 
@@ -354,10 +348,6 @@ own Cypress configuration and test files. In that case, we could configure the C
 }
 ```
 
-#### Disable a plugin
-
-In case a plugin causes issues, it can be disabled by using `false` as its value (e.g. `"webpack": false`).
-
 #### Create a new plugin
 
 Getting false positives because a plugin is missing? Want to help out? Please read more at [writing a plugin][47]. This
@@ -920,6 +910,7 @@ Special thanks to the wonderful people who have contributed to this project:
 [plugin-markdownlint]: ./src/plugins/markdownlint
 [plugin-mocha]: ./src/plugins/mocha
 [plugin-next]: ./src/plugins/next
+[plugin-node-test-runner]: ./src/plugins/node-test-runner
 [plugin-npm-package-json-lint]: ./src/plugins/npm-package-json-lint
 [plugin-nx]: ./src/plugins/nx
 [plugin-nyc]: ./src/plugins/nyc

package/dist/ConfigurationValidator.d.ts

@@ -281,6 +281,19 @@ export declare const ConfigurationValidator: z.ZodObject<{
             entry?: string | string[] | undefined;
             project?: string | string[] | undefined;
         }>]>>;
+        'node-test-runner': z.ZodOptional<z.ZodUnion<[z.ZodBoolean, z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>, z.ZodObject<{
+            config: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
+            entry: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
+            project: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
+        }, "strip", z.ZodTypeAny, {
+            config?: string | string[] | undefined;
+            entry?: string | string[] | undefined;
+            project?: string | string[] | undefined;
+        }, {
+            config?: string | string[] | undefined;
+            entry?: string | string[] | undefined;
+            project?: string | string[] | undefined;
+        }>]>>;
         'npm-package-json-lint': z.ZodOptional<z.ZodUnion<[z.ZodBoolean, z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>, z.ZodObject<{
             config: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
             entry: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
@@ -656,6 +669,11 @@ export declare const ConfigurationValidator: z.ZodObject<{
             entry?: string | string[] | undefined;
             project?: string | string[] | undefined;
         } | undefined;
+        'node-test-runner'?: string | boolean | string[] | {
+            config?: string | string[] | undefined;
+            entry?: string | string[] | undefined;
+            project?: string | string[] | undefined;
+        } | undefined;
         'npm-package-json-lint'?: string | boolean | string[] | {
             config?: string | string[] | undefined;
             entry?: string | string[] | undefined;
@@ -863,6 +881,11 @@ export declare const ConfigurationValidator: z.ZodObject<{
             entry?: string | string[] | undefined;
             project?: string | string[] | undefined;
         } | undefined;
+        'node-test-runner'?: string | boolean | string[] | {
+            config?: string | string[] | undefined;
+            entry?: string | string[] | undefined;
+            project?: string | string[] | undefined;
+        } | undefined;
         'npm-package-json-lint'?: string | boolean | string[] | {
             config?: string | string[] | undefined;
             entry?: string | string[] | undefined;
@@ -1216,6 +1239,19 @@ export declare const ConfigurationValidator: z.ZodObject<{
         entry?: string | string[] | undefined;
         project?: string | string[] | undefined;
     }>]>>;
+    'node-test-runner': z.ZodOptional<z.ZodUnion<[z.ZodBoolean, z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>, z.ZodObject<{
+        config: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
+        entry: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
+        project: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
+    }, "strip", z.ZodTypeAny, {
+        config?: string | string[] | undefined;
+        entry?: string | string[] | undefined;
+        project?: string | string[] | undefined;
+    }, {
+        config?: string | string[] | undefined;
+        entry?: string | string[] | undefined;
+        project?: string | string[] | undefined;
+    }>]>>;
     'npm-package-json-lint': z.ZodOptional<z.ZodUnion<[z.ZodBoolean, z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>, z.ZodObject<{
         config: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
         entry: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
@@ -1606,6 +1642,11 @@ export declare const ConfigurationValidator: z.ZodObject<{
             entry?: string | string[] | undefined;
             project?: string | string[] | undefined;
         } | undefined;
+        'node-test-runner'?: string | boolean | string[] | {
+            config?: string | string[] | undefined;
+            entry?: string | string[] | undefined;
+            project?: string | string[] | undefined;
+        } | undefined;
         'npm-package-json-lint'?: string | boolean | string[] | {
             config?: string | string[] | undefined;
             entry?: string | string[] | undefined;
@@ -1807,6 +1848,11 @@ export declare const ConfigurationValidator: z.ZodObject<{
         entry?: string | string[] | undefined;
         project?: string | string[] | undefined;
     } | undefined;
+    'node-test-runner'?: string | boolean | string[] | {
+        config?: string | string[] | undefined;
+        entry?: string | string[] | undefined;
+        project?: string | string[] | undefined;
+    } | undefined;
     'npm-package-json-lint'?: string | boolean | string[] | {
         config?: string | string[] | undefined;
         entry?: string | string[] | undefined;
@@ -2029,6 +2075,11 @@ export declare const ConfigurationValidator: z.ZodObject<{
             entry?: string | string[] | undefined;
             project?: string | string[] | undefined;
         } | undefined;
+        'node-test-runner'?: string | boolean | string[] | {
+            config?: string | string[] | undefined;
+            entry?: string | string[] | undefined;
+            project?: string | string[] | undefined;
+        } | undefined;
         'npm-package-json-lint'?: string | boolean | string[] | {
             config?: string | string[] | undefined;
             entry?: string | string[] | undefined;
@@ -2230,6 +2281,11 @@ export declare const ConfigurationValidator: z.ZodObject<{
         entry?: string | string[] | undefined;
         project?: string | string[] | undefined;
     } | undefined;
+    'node-test-runner'?: string | boolean | string[] | {
+        config?: string | string[] | undefined;
+        entry?: string | string[] | undefined;
+        project?: string | string[] | undefined;
+    } | undefined;
     'npm-package-json-lint'?: string | boolean | string[] | {
         config?: string | string[] | undefined;
         entry?: string | string[] | undefined;

package/dist/ConfigurationValidator.js

@@ -79,6 +79,7 @@ const pluginsSchema = z.object({
     markdownlint: pluginSchema,
     mocha: pluginSchema,
     next: pluginSchema,
+    'node-test-runner': pluginSchema,
     'npm-package-json-lint': pluginSchema,
     nx: pluginSchema,
     nyc: pluginSchema,

package/dist/WorkspaceWorker.d.ts

@@ -31,19 +31,19 @@ export declare class WorkspaceWorker {
     hostDependencies: HostDependencies;
     installedBinaries: InstalledBinaries;
     hasTypesIncluded: Set<string>;
+    entryFilePatterns: Set<string>;
+    productionEntryFilePatterns: Set<string>;
     constructor({ name, dir, cwd, config, manifest, isProduction, isStrict, rootIgnore, negatedWorkspacePatterns, enabledPluginsInAncestors, }: WorkspaceManagerOptions);
     init(): Promise<void>;
     private setEnabledPlugins;
     private initReferencedDependencies;
     private getConfigForPlugin;
     getEntryFilePatterns(): string[];
-    getProjectFilePatterns(): string[];
-    getPluginEntryFilePatterns(isIncludeProductionEntryFiles?: boolean): string[];
+    getProjectFilePatterns(testFilePatterns: string[]): string[];
     getPluginProjectFilePatterns(): string[];
     getPluginConfigPatterns(): string[];
-    getProductionEntryFilePatterns(): string[];
-    getProductionProjectFilePatterns(): string[];
-    getProductionPluginEntryFilePatterns(): string[];
+    getProductionEntryFilePatterns(negatedTestFilePatterns: string[]): string[];
+    getProductionProjectFilePatterns(negatedTestFilePatterns: string[]): string[];
     private getConfigurationFilePatterns;
     getIgnorePatterns(): string[];
     private findDependenciesByPlugins;
@@ -52,7 +52,9 @@ export declare class WorkspaceWorker {
         installedBinaries: InstalledBinaries;
         referencedDependencies: ReferencedDependencies;
         hasTypesIncluded: Set<string>;
-        enabledPlugins: ("angular" | "ava" | "babel" | "capacitor" | "changesets" | "commitizen" | "commitlint" | "cspell" | "cypress" | "eslint" | "gatsby" | "husky" | "jest" | "lefthook" | "markdownlint" | "mocha" | "next" | "nx" | "nyc" | "playwright" | "postcss" | "prettier" | "remark" | "remix" | "rollup" | "sentry" | "storybook" | "stryker" | "stylelint" | "tailwind" | "typedoc" | "typescript" | "vite" | "vitest" | "webpack" | "drizzle" | "githubActions" | "lintStaged" | "npmPackageJsonLint" | "playwrightCt" | "releaseIt" | "semanticRelease" | "svelte")[];
+        enabledPlugins: ("angular" | "ava" | "babel" | "capacitor" | "changesets" | "commitizen" | "commitlint" | "cspell" | "cypress" | "eslint" | "gatsby" | "husky" | "jest" | "lefthook" | "markdownlint" | "mocha" | "next" | "nx" | "nyc" | "playwright" | "postcss" | "prettier" | "remark" | "remix" | "rollup" | "sentry" | "storybook" | "stryker" | "stylelint" | "tailwind" | "typedoc" | "typescript" | "vite" | "vitest" | "webpack" | "drizzle" | "githubActions" | "lintStaged" | "nodeTestRunner" | "npmPackageJsonLint" | "playwrightCt" | "releaseIt" | "semanticRelease" | "svelte")[];
+        entryFilePatterns: string[];
+        productionEntryFilePatterns: string[];
     }>;
 }
 export {};

package/dist/WorkspaceWorker.js

@@ -1,11 +1,10 @@
-import { TEST_FILE_PATTERNS } from './constants.js';
 import * as npm from './manifest/index.js';
 import * as plugins from './plugins/index.js';
 import { debugLogArray, debugLogObject } from './util/debug.js';
 import { _pureGlob, negate, hasProductionSuffix, hasNoProductionSuffix, prependDirToPattern } from './util/glob.js';
-import { getKeysByValue } from './util/object.js';
+import { get, getKeysByValue } from './util/object.js';
 import { join, toPosix } from './util/path.js';
-const negatedTestFilePatterns = TEST_FILE_PATTERNS.map(negate);
+import { fromEntryPattern, fromProductionEntryPattern, isEntryPattern, isProductionEntryPattern, } from './util/protocols.js';
 const nullConfig = { config: null, entry: null, project: null };
 export class WorkspaceWorker {
     name;
@@ -24,6 +23,8 @@ export class WorkspaceWorker {
     hostDependencies = new Map();
     installedBinaries = new Map();
     hasTypesIncluded = new Set();
+    entryFilePatterns = new Set();
+    productionEntryFilePatterns = new Set();
     constructor({ name, dir, cwd, config, manifest, isProduction, isStrict, rootIgnore, negatedWorkspacePatterns, enabledPluginsInAncestors, }) {
         this.name = name;
         this.dir = dir;
@@ -77,61 +78,38 @@ export class WorkspaceWorker {
         this.hostDependencies = hostDependencies;
         this.installedBinaries = installedBinaries;
         this.hasTypesIncluded = hasTypesIncluded;
+        this.hasTypesIncluded = hasTypesIncluded;
     }
     getConfigForPlugin(pluginName) {
-        return this.config[pluginName] ?? nullConfig;
+        return this.config[pluginName] !== true ? this.config[pluginName] ?? nullConfig : nullConfig;
     }
     getEntryFilePatterns() {
         const { entry } = this.config;
         if (entry.length === 0)
             return [];
-        return [entry, TEST_FILE_PATTERNS, this.negatedWorkspacePatterns].flat();
+        return [entry, this.negatedWorkspacePatterns].flat();
     }
-    getProjectFilePatterns() {
+    getProjectFilePatterns(testFilePatterns) {
         const { project } = this.config;
         if (project.length === 0)
             return [];
         const negatedPluginConfigPatterns = this.getPluginConfigPatterns().map(negate);
-        const negatedPluginEntryFilePatterns = this.getPluginEntryFilePatterns(false).map(negate);
         const negatedPluginProjectFilePatterns = this.getPluginProjectFilePatterns().map(negate);
         return [
             project,
             negatedPluginConfigPatterns,
-            negatedPluginEntryFilePatterns,
             negatedPluginProjectFilePatterns,
-            TEST_FILE_PATTERNS,
+            testFilePatterns,
             this.negatedWorkspacePatterns,
         ].flat();
     }
-    getPluginEntryFilePatterns(isIncludeProductionEntryFiles = true) {
-        const patterns = [];
-        for (const [pluginName, plugin] of Object.entries(plugins)) {
-            const pluginConfig = this.getConfigForPlugin(pluginName);
-            if (this.enabled[pluginName] && pluginConfig) {
-                const { entry } = pluginConfig === true ? nullConfig : pluginConfig;
-                const defaultEntryFiles = 'ENTRY_FILE_PATTERNS' in plugin ? plugin.ENTRY_FILE_PATTERNS : [];
-                patterns.push(...(entry ?? defaultEntryFiles));
-                if (isIncludeProductionEntryFiles) {
-                    const entry = 'PRODUCTION_ENTRY_FILE_PATTERNS' in plugin ? plugin.PRODUCTION_ENTRY_FILE_PATTERNS : [];
-                    patterns.push(...entry);
-                }
-            }
-        }
-        return [patterns, this.negatedWorkspacePatterns].flat();
-    }
     getPluginProjectFilePatterns() {
         const patterns = [];
         for (const [pluginName, plugin] of Object.entries(plugins)) {
             const pluginConfig = this.getConfigForPlugin(pluginName);
             if (this.enabled[pluginName] && pluginConfig) {
                 const { entry, project } = pluginConfig === true ? nullConfig : pluginConfig;
-                patterns.push(...(project ??
-                    entry ??
-                    ('PROJECT_FILE_PATTERNS' in plugin
-                        ? plugin.PROJECT_FILE_PATTERNS
-                        : 'ENTRY_FILE_PATTERNS' in plugin
-                            ? plugin.ENTRY_FILE_PATTERNS
-                            : [])));
+                patterns.push(...(project ?? entry ?? ('PROJECT_FILE_PATTERNS' in plugin ? plugin.PROJECT_FILE_PATTERNS : [])));
             }
         }
         return [patterns, this.negatedWorkspacePatterns].flat();
@@ -148,17 +126,17 @@ export class WorkspaceWorker {
         }
         return patterns;
     }
-    getProductionEntryFilePatterns() {
+    getProductionEntryFilePatterns(negatedTestFilePatterns) {
         const entry = this.config.entry.filter(hasProductionSuffix);
         if (entry.length === 0)
             return [];
         const negatedEntryFiles = this.config.entry.filter(hasNoProductionSuffix).map(negate);
         return [entry, negatedEntryFiles, negatedTestFilePatterns, this.negatedWorkspacePatterns].flat();
     }
-    getProductionProjectFilePatterns() {
+    getProductionProjectFilePatterns(negatedTestFilePatterns) {
         const project = this.config.project;
         if (project.length === 0)
-            return this.getProductionEntryFilePatterns();
+            return this.getProductionEntryFilePatterns(negatedTestFilePatterns);
         const _project = this.config.project.map(pattern => {
             if (!pattern.endsWith('!') && !pattern.startsWith('!'))
                 return negate(pattern);
@@ -166,32 +144,16 @@ export class WorkspaceWorker {
         });
         const negatedEntryFiles = this.config.entry.filter(hasNoProductionSuffix).map(negate);
         const negatedPluginConfigPatterns = this.getPluginConfigPatterns().map(negate);
-        const negatedPluginEntryFilePatterns = this.getPluginEntryFilePatterns(false).map(negate);
         const negatedPluginProjectFilePatterns = this.getPluginProjectFilePatterns().map(negate);
         return [
             _project,
             negatedEntryFiles,
             negatedPluginConfigPatterns,
-            negatedPluginEntryFilePatterns,
             negatedPluginProjectFilePatterns,
             negatedTestFilePatterns,
             this.negatedWorkspacePatterns,
         ].flat();
     }
-    getProductionPluginEntryFilePatterns() {
-        const patterns = [];
-        for (const [pluginName, plugin] of Object.entries(plugins)) {
-            const pluginConfig = this.getConfigForPlugin(pluginName);
-            if (this.enabled[pluginName] && pluginConfig) {
-                if ('PRODUCTION_ENTRY_FILE_PATTERNS' in plugin) {
-                    patterns.push(...((pluginConfig === true ? null : pluginConfig.entry) ?? plugin.PRODUCTION_ENTRY_FILE_PATTERNS));
-                }
-            }
-        }
-        if (patterns.length === 0)
-            return [];
-        return [patterns.flat(), negatedTestFilePatterns].flat();
-    }
     getConfigurationFilePatterns(pluginName) {
         const plugin = plugins[pluginName];
         const pluginConfig = this.getConfigForPlugin(pluginName);
@@ -208,31 +170,41 @@ export class WorkspaceWorker {
         const cwd = this.dir;
         const ignore = this.getIgnorePatterns();
         for (const [pluginName, plugin] of Object.entries(plugins)) {
-            const isIncludePlugin = this.isProduction ? `PRODUCTION_ENTRY_FILE_PATTERNS` in plugin : true;
-            if (this.enabled[pluginName] && isIncludePlugin) {
+            if (this.enabled[pluginName]) {
                 const hasDependencyFinder = 'findDependencies' in plugin && typeof plugin.findDependencies === 'function';
                 if (hasDependencyFinder) {
                     const pluginConfig = this.getConfigForPlugin(pluginName);
                     if (!pluginConfig)
                         continue;
                     const patterns = this.getConfigurationFilePatterns(pluginName);
-                    const configFilePaths = await _pureGlob({ patterns, cwd, ignore, gitignore: false });
+                    const allConfigFilePaths = await _pureGlob({ patterns, cwd, ignore, gitignore: false });
+                    const configFilePaths = allConfigFilePaths.filter(filePath => !filePath.endsWith('package.json') ||
+                        get(this.manifest, 'PACKAGE_JSON_PATH' in plugin ? plugin.PACKAGE_JSON_PATH : pluginName));
                     debugLogArray(`Found ${plugin.NAME} config file paths`, configFilePaths);
-                    if (configFilePaths.length === 0)
+                    if (patterns.length > 0 && configFilePaths.length === 0)
                         continue;
+                    if (patterns.length === 0)
+                        configFilePaths.push('__FAKE__');
                     const pluginDependencies = new Set();
                     for (const configFilePath of configFilePaths) {
                         const dependencies = await plugin.findDependencies(configFilePath, {
                             cwd,
                             manifest: this.manifest,
-                            config: pluginConfig,
+                            config: pluginConfig === true ? nullConfig : pluginConfig,
                             isProduction: this.isProduction,
                         });
-                        dependencies.map(toPosix).forEach(specifier => {
+                        dependencies.forEach(specifier => {
                             pluginDependencies.add(specifier);
-                            this.referencedDependencies.add([configFilePath, specifier]);
+                            if (isEntryPattern(specifier)) {
+                                this.entryFilePatterns.add(fromEntryPattern(specifier));
+                            }
+                            else if (isProductionEntryPattern(specifier)) {
+                                this.productionEntryFilePatterns.add(fromProductionEntryPattern(specifier));
+                            }
+                            else {
+                                this.referencedDependencies.add([configFilePath, toPosix(specifier)]);
+                            }
                         });
-                        dependencies.forEach(dependency => pluginDependencies.add(dependency));
                     }
                     debugLogArray(`Dependencies referenced in ${plugin.NAME}`, pluginDependencies);
                 }
@@ -247,6 +219,8 @@ export class WorkspaceWorker {
             referencedDependencies: this.referencedDependencies,
             hasTypesIncluded: this.hasTypesIncluded,
             enabledPlugins: this.enabledPlugins,
+            entryFilePatterns: Array.from(this.entryFilePatterns),
+            productionEntryFilePatterns: Array.from(this.productionEntryFilePatterns),
         };
     }
 }

package/dist/binaries/index.js

@@ -2,8 +2,8 @@ import { compact } from '../util/array.js';
 import { getPackageNameFromModuleSpecifier } from '../util/modules.js';
 import { isInternal } from '../util/path.js';
 import { timerify } from '../util/Performance.js';
+import { isBinary } from '../util/protocols.js';
 import { getBinariesFromScript } from './bash-parser.js';
-import { isBinary } from './util.js';
 const defaultCwd = process.cwd();
 const getDependenciesFromScripts = (npmScripts, options = {}) => {
     const { cwd = defaultCwd, manifest = {}, knownGlobalsOnly = false } = options;

package/dist/binaries/resolvers/c8.js

@@ -1,5 +1,6 @@
 import parseArgs from 'minimist';
-import { toBinary, argsFrom } from '../util.js';
+import { toBinary } from '../../util/protocols.js';
+import { argsFrom } from '../util.js';
 export const resolve = (binary, args, { fromArgs }) => {
     const parsed = parseArgs(args, {
         boolean: ['all', 'check-coverage', 'clean', 'exclude-after-remap', 'per-file', 'skip-full'],

package/dist/binaries/resolvers/dotenv.js

@@ -1,5 +1,6 @@
 import parseArgs from 'minimist';
-import { toBinary, argsFrom } from '../util.js';
+import { toBinary } from '../../util/protocols.js';
+import { argsFrom } from '../util.js';
 export const resolve = (binary, args, { fromArgs }) => {
     const parsed = parseArgs(args);
     return [toBinary(binary), ...fromArgs(argsFrom(args, parsed._[0]))];

package/dist/binaries/resolvers/fallback.js

@@ -1,6 +1,7 @@
 import parseArgs from 'minimist';
 import { compact } from '../../util/array.js';
-import { toBinary, tryResolveFilePath, tryResolveSpecifiers } from '../util.js';
+import { toBinary } from '../../util/protocols.js';
+import { tryResolveFilePath, tryResolveSpecifiers } from '../util.js';
 const withPositional = parsed => [parsed._[0], parsed.require].flat();
 const withoutPositional = parsed => [parsed.require].flat();
 const argFilters = {

package/dist/binaries/resolvers/nodemon.js

@@ -1,6 +1,7 @@
 import parseArgs from 'minimist';
 import { compact } from '../../util/array.js';
-import { toBinary, tryResolveFilePath, tryResolveSpecifiers } from '../util.js';
+import { toBinary } from '../../util/protocols.js';
+import { tryResolveFilePath, tryResolveSpecifiers } from '../util.js';
 export const resolve = (binary, args, { cwd, fromArgs }) => {
     const parsed = parseArgs(args, {
         string: ['r', 'exec'],

package/dist/binaries/resolvers/npx.js

@@ -1,7 +1,8 @@
 import parseArgs from 'minimist';
 import { isInternal } from '../../util/path.js';
+import { toBinary } from '../../util/protocols.js';
 import { getBinariesFromScript } from '../bash-parser.js';
-import { argsFrom, stripVersionFromSpecifier, toBinary } from '../util.js';
+import { argsFrom, stripVersionFromSpecifier } from '../util.js';
 export const resolve = (binary, args, { cwd, fromArgs, manifest }) => {
     const parsed = parseArgs(args, {
         boolean: ['yes', 'no'],

package/dist/binaries/resolvers/nx.js

@@ -1,5 +1,5 @@
 import parseArgs from 'minimist';
-import { toBinary } from '../util.js';
+import { toBinary } from '../../util/protocols.js';
 export const resolve = (binary, args, { fromArgs }) => {
     const parsed = parseArgs(args);
     const [command] = parsed._;

package/dist/binaries/resolvers/pnpm.js

@@ -1,5 +1,5 @@
 import parseArgs from 'minimist';
-import { toBinary } from '../util.js';
+import { toBinary } from '../../util/protocols.js';
 const commands = [
     'add',
     'audit',

package/dist/binaries/resolvers/rollup.js

@@ -1,6 +1,7 @@
 import parseArgs from 'minimist';
 import { compact } from '../../util/array.js';
-import { toBinary, tryResolveSpecifiers } from '../util.js';
+import { toBinary } from '../../util/protocols.js';
+import { tryResolveSpecifiers } from '../util.js';
 export const resolve = (binary, args, { cwd, fromArgs }) => {
     const safeArgs = args.filter(arg => arg !== '--watch');
     const parsed = parseArgs(safeArgs, { alias: { plugin: 'p' } });

package/dist/binaries/resolvers/yarn.js

@@ -1,5 +1,5 @@
 import parseArgs from 'minimist';
-import { toBinary } from '../util.js';
+import { toBinary } from '../../util/protocols.js';
 const commands = [
     'add',
     'bin',

package/dist/binaries/util.d.ts

@@ -1,8 +1,5 @@
 export declare const tryResolveFilePath: (cwd: string, specifier: string, acceptModuleSpecifier?: boolean) => string | undefined;
 export declare const tryResolveSpecifiers: (cwd: string, specifiers: string[]) => (string | undefined)[];
-export declare const toBinary: (specifier: string) => string;
-export declare const fromBinary: (specifier: string) => string;
-export declare const isBinary: (specifier: string) => boolean;
 export declare const stripVersionFromSpecifier: (specifier: string) => string;
 export declare const stripBinaryPath: (command: string) => string;
 export declare const argsFrom: (args: string[], from: string) => string[];

package/dist/binaries/util.js

@@ -1,5 +1,6 @@
 import { getPackageNameFromFilePath, getPackageNameFromModuleSpecifier } from '../util/modules.js';
 import { isInNodeModules, join } from '../util/path.js';
+import { toBinary } from '../util/protocols.js';
 import { _tryResolve } from '../util/require.js';
 export const tryResolveFilePath = (cwd, specifier, acceptModuleSpecifier) => {
     if (specifier) {
@@ -22,9 +23,6 @@ export const tryResolveFilePath = (cwd, specifier, acceptModuleSpecifier) => {
     }
 };
 export const tryResolveSpecifiers = (cwd, specifiers) => specifiers.map(specifier => tryResolveFilePath(cwd, specifier, true));
-export const toBinary = (specifier) => specifier.replace(/^(bin:)?/, 'bin:');
-export const fromBinary = (specifier) => specifier.replace(/^(bin:)?/, '');
-export const isBinary = (specifier) => specifier.startsWith('bin:');
 export const stripVersionFromSpecifier = (specifier) => specifier.replace(/(\S+)@.*/, '$1');
 const stripNodeModulesFromPath = (command) => command.replace(/^(\.\/)?node_modules\//, '');
 export const stripBinaryPath = (command) => stripVersionFromSpecifier(stripNodeModulesFromPath(command)

package/dist/constants.d.ts

@@ -2,7 +2,6 @@ import type { IssueType } from './types/issues.js';
 export declare const ROOT_WORKSPACE_NAME = ".";
 export declare const KNIP_CONFIG_LOCATIONS: string[];
 export declare const DEFAULT_EXTENSIONS: string[];
-export declare const TEST_FILE_PATTERNS: string[];
 export declare const GLOBAL_IGNORE_PATTERNS: string[];
 export declare const IGNORED_GLOBAL_BINARIES: string[];
 export declare const IGNORED_DEPENDENCIES: string[];

package/dist/constants.js

@@ -1,10 +1,6 @@
 export const ROOT_WORKSPACE_NAME = '.';
 export const KNIP_CONFIG_LOCATIONS = ['knip.json', 'knip.jsonc', '.knip.json', '.knip.jsonc', 'knip.ts', 'knip.js'];
 export const DEFAULT_EXTENSIONS = ['.js', '.mjs', '.cjs', '.jsx', '.ts', '.tsx', '.mts', '.cts'];
-export const TEST_FILE_PATTERNS = [
-    '**/*{.,-}{test,spec}.{js,jsx,ts,tsx,mjs,cjs}',
-    '**/{test,__tests__}/**/*.{js,jsx,ts,tsx,mjs,cjs}',
-];
 export const GLOBAL_IGNORE_PATTERNS = ['**/node_modules/**', '.yarn'];
 export const IGNORED_GLOBAL_BINARIES = [
     'bash',