knip 5.30.2 → 5.30.4

package/README.md

@@ -42,6 +42,6 @@ Special thanks to the wonderful people who have contributed to this project:
 [4]: https://discord.gg/r5uXTtbTpc
 [5]: https://github.com/webpro-nl/knip/blob/main/.github/CONTRIBUTING.md
 [6]: https://knip.dev/sponsors
-[7]: https://en.wiktionary.org/wiki/knip#Dutch
+[7]: https://www.youtube.com/watch?v=PE7h7KvQoUI&t=9s
 [8]: https://github.com/webpro-nl/knip/graphs/contributors
 [9]: https://contrib.rocks/image?repo=webpro-nl/knip

package/dist/ConfigurationChief.js

@@ -185,7 +185,7 @@ export class ConfigurationChief {
         const workspaces = this.manifest?.workspaces
             ? Array.isArray(this.manifest.workspaces)
                 ? this.manifest.workspaces
-                : this.manifest.workspaces.packages ?? []
+                : (this.manifest.workspaces.packages ?? [])
             : [];
         return workspaces.map(pattern => pattern.replace(/(?<=!?)\.\//, ''));
     }

package/dist/ConfigurationValidator.d.ts

@@ -2775,9 +2775,9 @@ export declare const ConfigurationValidator: z.ZodObject<z.objectUtil.extendShap
     ignoreExportsUsedInFile?: boolean | Partial<Record<"function" | "type" | "enum" | "class" | "interface" | "member", boolean>> | undefined;
     ignoreWorkspaces?: string[] | undefined;
     includeEntryExports?: boolean | undefined;
-    compilers?: Record<string, ((args_0: string, args_1: string, ...args_2: unknown[]) => string) | ((args_0: string, args_1: string, ...args_2: unknown[]) => Promise<string>)> | undefined;
-    syncCompilers?: Record<string, (args_0: string, args_1: string, ...args_2: unknown[]) => string> | undefined;
-    asyncCompilers?: Record<string, (args_0: string, args_1: string, ...args_2: unknown[]) => Promise<string>> | undefined;
+    compilers?: Record<string, ((args_0: string, args_1: string, ...args: unknown[]) => string) | ((args_0: string, args_1: string, ...args: unknown[]) => Promise<string>)> | undefined;
+    syncCompilers?: Record<string, (args_0: string, args_1: string, ...args: unknown[]) => string> | undefined;
+    asyncCompilers?: Record<string, (args_0: string, args_1: string, ...args: unknown[]) => Promise<string>> | undefined;
     astro?: string | boolean | string[] | {
         config?: string | string[] | undefined;
         entry?: string | string[] | undefined;
@@ -3552,9 +3552,9 @@ export declare const ConfigurationValidator: z.ZodObject<z.objectUtil.extendShap
     ignoreExportsUsedInFile?: boolean | Partial<Record<"function" | "type" | "enum" | "class" | "interface" | "member", boolean>> | undefined;
     ignoreWorkspaces?: string[] | undefined;
     includeEntryExports?: boolean | undefined;
-    compilers?: Record<string, ((args_0: string, args_1: string, ...args_2: unknown[]) => string) | ((args_0: string, args_1: string, ...args_2: unknown[]) => Promise<string>)> | undefined;
-    syncCompilers?: Record<string, (args_0: string, args_1: string, ...args_2: unknown[]) => string> | undefined;
-    asyncCompilers?: Record<string, (args_0: string, args_1: string, ...args_2: unknown[]) => Promise<string>> | undefined;
+    compilers?: Record<string, ((args_0: string, args_1: string, ...args: unknown[]) => string) | ((args_0: string, args_1: string, ...args: unknown[]) => Promise<string>)> | undefined;
+    syncCompilers?: Record<string, (args_0: string, args_1: string, ...args: unknown[]) => string> | undefined;
+    asyncCompilers?: Record<string, (args_0: string, args_1: string, ...args: unknown[]) => Promise<string>> | undefined;
     astro?: string | boolean | string[] | {
         config?: string | string[] | undefined;
         entry?: string | string[] | undefined;

package/dist/DependencyDeputy.js

@@ -175,11 +175,11 @@ export class DependencyDeputy {
                     return false;
                 const [scope, typedDependency] = dependency.split('/');
                 if (scope === DT_SCOPE) {
-                    if (hasTypesIncluded?.has(typedDependency))
-                        return false;
                     const typedPackageName = getPackageFromDefinitelyTyped(typedDependency);
-                    if (IGNORE_DEFINITELY_TYPED.includes(typedPackageName))
+                    if (IGNORE_DEFINITELY_TYPED.has(typedPackageName))
                         return true;
+                    if (hasTypesIncluded?.has(typedDependency))
+                        return false;
                     const hostDependencies = [
                         ...this.getHostDependenciesFor(workspace, dependency),
                         ...this.getHostDependenciesFor(workspace, typedPackageName),

package/dist/IssueFixer.js

@@ -65,8 +65,8 @@ export class IssueFixer {
         const filePaths = new Set([...this.unusedTypeNodes.keys(), ...this.unusedExportNodes.keys()]);
         for (const filePath of filePaths) {
             const exportPositions = [
-                ...(this.isFixUnusedTypes ? this.unusedTypeNodes.get(filePath) ?? [] : []),
-                ...(this.isFixUnusedExports ? this.unusedExportNodes.get(filePath) ?? [] : []),
+                ...(this.isFixUnusedTypes ? (this.unusedTypeNodes.get(filePath) ?? []) : []),
+                ...(this.isFixUnusedExports ? (this.unusedExportNodes.get(filePath) ?? []) : []),
             ].sort((a, b) => b[0] - a[0]);
             if (exportPositions.length > 0) {
                 const sourceFileText = exportPositions.reduce((text, [start, end]) => text.substring(0, start) + text.substring(end), await readFile(filePath, 'utf-8'));

package/dist/compilers/index.d.ts

@@ -17,7 +17,7 @@ export declare const partitionCompilers: (rawLocalConfig: RawConfiguration) => {
     ignoreExportsUsedInFile?: boolean | Partial<Record<"function" | "type" | "enum" | "class" | "interface" | "member", boolean>> | undefined;
     ignoreWorkspaces?: string[] | undefined;
     includeEntryExports?: boolean | undefined;
-    compilers?: Record<string, ((args_0: string, args_1: string, ...args_2: unknown[]) => string) | ((args_0: string, args_1: string, ...args_2: unknown[]) => Promise<string>)> | undefined;
+    compilers?: Record<string, ((args_0: string, args_1: string, ...args: unknown[]) => string) | ((args_0: string, args_1: string, ...args: unknown[]) => Promise<string>)> | undefined;
     astro?: string | boolean | string[] | {
         config?: string | string[] | undefined;
         entry?: string | string[] | undefined;

package/dist/constants.d.ts

@@ -14,6 +14,6 @@ export declare const IGNORED_GLOBAL_BINARIES: Set<string>;
 export declare const IGNORED_DEPENDENCIES: Set<string>;
 export declare const IGNORED_RUNTIME_DEPENDENCIES: Set<string>;
 export declare const FOREIGN_FILE_EXTENSIONS: Set<string>;
-export declare const IGNORE_DEFINITELY_TYPED: string[];
+export declare const IGNORE_DEFINITELY_TYPED: Set<string>;
 export declare const ISSUE_TYPES: IssueType[];
 export declare const ISSUE_TYPE_TITLE: Record<IssueType, string>;

package/dist/constants.js

@@ -142,7 +142,11 @@ export const FOREIGN_FILE_EXTENSIONS = new Set([
     '.yaml',
     '.yml',
 ]);
-export const IGNORE_DEFINITELY_TYPED = ['node', 'bun'];
+export const IGNORE_DEFINITELY_TYPED = new Set([
+    'node',
+    'bun',
+    'jest',
+]);
 export const ISSUE_TYPES = [
     'files',
     'dependencies',

package/dist/plugins/eslint/index.js

@@ -1,7 +1,7 @@
 import { hasDependency } from '#p/util/plugin.js';
 import { getDependenciesDeep } from './helpers.js';
 const title = 'ESLint';
-const enablers = ['eslint'];
+const enablers = ['eslint', '@eslint/js'];
 const isEnabled = ({ dependencies, manifest, config }) => hasDependency(dependencies, enablers) ||
     'eslint' in config ||
     Boolean(manifest.name && /(^eslint-config|\/eslint-config)/.test(manifest.name));

package/dist/plugins/vitest/index.js

@@ -5,7 +5,7 @@ import { getEnvPackageName, getExternalReporters } from './helpers.js';
 const title = 'Vitest';
 const enablers = ['vitest'];
 const isEnabled = ({ dependencies }) => hasDependency(dependencies, enablers);
-const config = ['vitest*.config.{js,mjs,ts,cjs,mts,cts}', 'vitest.{workspace,projects}.{ts,js,json}'];
+const config = ['vitest*.config.{js,mjs,ts,cjs,mts,cts}', 'vitest.{workspace,projects}.{js,mjs,ts,cjs,mts,cts,json}'];
 const entry = ['**/*.{bench,test,test-d,spec}.?(c|m)[jt]s?(x)'];
 const isVitestCoverageCommand = /vitest(.+)--coverage(?:\.enabled(?:=true)?)?/;
 const hasScriptWithCoverage = (scripts) => scripts

package/dist/util/to-source-path.js

@@ -9,7 +9,7 @@ const hasTSExt = /(?<!\.d)\.(m|c)?tsx?$/;
 const matchExt = /(\.d)?\.(m|c)?(j|t)s$/;
 export const augmentWorkspace = (workspace, dir, compilerOptions) => {
     const srcDir = join(dir, 'src');
-    workspace.srcDir = compilerOptions.rootDir ?? isDirectory(srcDir) ? srcDir : dir;
+    workspace.srcDir = (compilerOptions.rootDir ?? isDirectory(srcDir)) ? srcDir : dir;
     workspace.outDir = compilerOptions.outDir || workspace.srcDir;
 };
 export const getToSourcePathHandler = (chief) => {

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "5.30.2";
+export declare const version = "5.30.4";

package/dist/version.js

@@ -1 +1 @@
-export const version = '5.30.2';
+export const version = '5.30.4';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "5.30.2",
+  "version": "5.30.4",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://knip.dev",
   "repository": {
@@ -72,7 +72,7 @@
     "picocolors": "^1.0.0",
     "picomatch": "^4.0.1",
     "pretty-ms": "^9.0.0",
-    "smol-toml": "^1.1.4",
+    "smol-toml": "^1.3.0",
     "strip-json-comments": "5.0.1",
     "summary": "2.1.0",
     "zod": "^3.22.4",
@@ -85,14 +85,14 @@
   "devDependencies": {
     "@jest/types": "^29.6.3",
     "@release-it/bumper": "^6.0.1",
-    "@types/bun": "^1.1.6",
+    "@types/bun": "^1.1.10",
     "@types/js-yaml": "^4.0.9",
     "@types/minimist": "^1.2.5",
-    "@types/picomatch": "2.3.4",
+    "@types/picomatch": "3.0.1",
     "@types/webpack": "^5.28.5",
-    "@wdio/types": "^8.39.0",
+    "@wdio/types": "^9.0.8",
     "glob": "^10.4.2",
-    "release-it": "^17.4.1",
+    "release-it": "^17.6.0",
     "type-fest": "^4.20.0",
     "typescript": "^5.5.2"
   },