knip 3.7.1 → 3.8.1

package/dist/ConfigurationChief.js

@@ -334,8 +334,8 @@ export class ConfigurationChief {
         const project = workspaceConfig.project ? arrayify(workspaceConfig.project) : defaultConfig.project;
         const paths = workspaceConfig.paths ?? defaultConfig.paths;
         const ignore = arrayify(workspaceConfig.ignore);
-        const ignoreBinaries = arrayify(workspaceConfig.ignoreBinaries);
-        const ignoreDependencies = arrayify(workspaceConfig.ignoreDependencies);
+        const ignoreBinaries = arrayify(workspaceConfig.ignoreBinaries).map(toRegexOrString);
+        const ignoreDependencies = arrayify(workspaceConfig.ignoreDependencies).map(toRegexOrString);
         const isIncludeEntryExports = workspaceConfig.includeEntryExports ?? this.config.isIncludeEntryExports;
         const plugins = {};
         for (const [name, pluginConfig] of Object.entries(this.config.rootPluginConfigs)) {

package/dist/DependencyDeputy.js

@@ -1,7 +1,7 @@
 import { isBuiltin } from 'node:module';
 import { IGNORE_DEFINITELY_TYPED, IGNORED_DEPENDENCIES, IGNORED_GLOBAL_BINARIES, ROOT_WORKSPACE_NAME, } from './constants.js';
 import { isDefinitelyTyped, getDefinitelyTypedFor, getPackageFromDefinitelyTyped } from './util/modules.js';
-import { hasMatch, hasMatchInArray, hasMatchInSet } from './util/regex.js';
+import { hasMatch, hasMatchInArray, hasMatchInSet, toRegexOrString, findKey } from './util/regex.js';
 export class DependencyDeputy {
     isProduction;
     isStrict;
@@ -49,8 +49,8 @@ export class DependencyDeputy {
         });
     }
     addIgnored(ignoreBinaries, ignoreDependencies) {
-        this.ignoreBinaries = ignoreBinaries;
-        this.ignoreDependencies = ignoreDependencies;
+        this.ignoreBinaries = ignoreBinaries.map(toRegexOrString);
+        this.ignoreDependencies = ignoreDependencies.map(toRegexOrString);
     }
     getWorkspaceManifest(workspaceName) {
         return this._manifests.get(workspaceName);
@@ -236,22 +236,32 @@ export class DependencyDeputy {
             const referencedDependencies = this.referencedDependencies.get(workspaceName);
             const referencedBinaries = this.referencedBinaries.get(workspaceName);
             const installedBinaries = this.getInstalledBinaries(workspaceName);
+            const ignoredBinaries = this._manifests.get(workspaceName)?.ignoreBinaries ?? [];
+            const ignoredDependencies = this._manifests.get(workspaceName)?.ignoreDependencies ?? [];
             referencedDependencies?.forEach(pkg => {
-                for (const key of rootIgnoreDependencies.keys()) {
+                for (const key of [...ignoredDependencies, ...rootIgnoreDependencies.keys()]) {
                     if ((typeof key === 'string' && key === pkg) || (key instanceof RegExp && key.test(pkg))) {
-                        rootIgnoreDependencies.set(key, rootIgnoreDependencies.get(key) + 1);
-                        return;
+                        const rootKey = typeof key === 'string' ? key : findKey(rootIgnoreDependencies, key);
+                        if (rootKey && rootIgnoreDependencies.has(rootKey)) {
+                            rootIgnoreDependencies.set(rootKey, rootIgnoreDependencies.get(rootKey) + 1);
+                            return;
+                        }
                     }
                 }
             });
             referencedBinaries?.forEach(binaryName => {
-                for (const key of rootIgnoreBinaries.keys()) {
+                for (const key of [...ignoredBinaries, ...rootIgnoreBinaries.keys()]) {
                     if ((typeof key === 'string' && key === binaryName) || (key instanceof RegExp && key.test(binaryName))) {
-                        rootIgnoreBinaries.set(key, rootIgnoreBinaries.get(key) + 1);
-                        return;
+                        const rootKey = typeof key === 'string' ? key : findKey(rootIgnoreBinaries, key);
+                        if (rootKey && rootIgnoreBinaries.has(rootKey)) {
+                            rootIgnoreBinaries.set(rootKey, rootIgnoreBinaries.get(rootKey) + 1);
+                            return;
+                        }
                     }
                 }
             });
+            if (workspaceName === ROOT_WORKSPACE_NAME)
+                continue;
             const dependencies = [
                 ...this.getProductionDependencies(workspaceName),
                 ...this.getDevDependencies(workspaceName),
@@ -261,11 +271,11 @@ export class DependencyDeputy {
                 .filter(packageName => {
                 if (hasMatchInArray(IGNORED_DEPENDENCIES, packageName))
                     return true;
-                if (workspaceName !== ROOT_WORKSPACE_NAME && this.ignoreDependencies.includes(packageName))
+                if (this.ignoreDependencies.includes(packageName))
                     return true;
                 const isReferenced = hasMatchInSet(referencedDependencies, packageName);
                 const isListed = hasMatchInArray(dependencies, packageName) && !hasMatchInArray(peerDependencies, packageName);
-                return isListed && isReferenced;
+                return (isListed && isReferenced) || (!isReferenced && !isListed);
             })
                 .forEach(identifier => {
                 configurationHints.add({ workspaceName, identifier, type: 'ignoreDependencies' });
@@ -274,11 +284,11 @@ export class DependencyDeputy {
                 .filter(binaryName => {
                 if (hasMatchInArray(IGNORED_GLOBAL_BINARIES, binaryName))
                     return true;
-                if (workspaceName !== ROOT_WORKSPACE_NAME && this.ignoreBinaries.includes(binaryName))
+                if (this.ignoreBinaries.includes(binaryName))
                     return true;
                 const isReferenced = hasMatchInSet(referencedBinaries, binaryName);
                 const isInstalled = hasMatchInArray(Array.from(installedBinaries?.keys() ?? []), binaryName);
-                return isReferenced && isInstalled;
+                return (isReferenced && isInstalled) || (!isInstalled && !isReferenced);
             })
                 .forEach(identifier => configurationHints.add({ workspaceName, identifier, type: 'ignoreBinaries' }));
         }

package/dist/cli.js

@@ -52,7 +52,8 @@ const run = async () => {
         if (isObservePerf) {
             await perfObserver.finalize();
             console.log('\n' + perfObserver.getTable());
-            console.log('\nTotal running time:', prettyMilliseconds(perfObserver.getTotalTime()));
+            const mem = Math.round((perfObserver.getMemHeapUsage() / 1024 / 1024) * 100) / 100;
+            console.log('\nTotal running time:', prettyMilliseconds(perfObserver.getTotalTime()), `(mem: ${mem}MB)`);
             perfObserver.reset();
         }
         if (!noExitCode && totalErrorCount > Number(maxIssues)) {

package/dist/index.js

@@ -183,7 +183,7 @@ export const main = async (unresolvedConfiguration) => {
         }
     }
     const principals = factory.getPrincipals();
-    debugLog('*', `Installed ${principals.length} principals for ${workspaces.length} workspaces`);
+    debugLog('*', `Created ${principals.length} programs for ${workspaces.length} workspaces`);
     const analyzedFiles = new Set();
     const exportedSymbols = new Map();
     const importedSymbols = new Map();

package/dist/types/config.d.ts

@@ -6,6 +6,7 @@ import type { Rules } from './issues.js';
 export type RawConfiguration = z.infer<typeof ConfigurationValidator>;
 export type RawPluginConfiguration = z.infer<typeof pluginSchema>;
 type NormalizedGlob = string[];
+type IgnorePatterns = (string | RegExp)[];
 export type PluginName = keyof typeof Plugins;
 export type PluginMap = typeof Plugins;
 export type EnsuredPluginConfiguration = {
@@ -20,8 +21,8 @@ interface BaseWorkspaceConfiguration {
     project: NormalizedGlob;
     paths: Record<string, string[]>;
     ignore: NormalizedGlob;
-    ignoreBinaries: string[];
-    ignoreDependencies: string[];
+    ignoreBinaries: IgnorePatterns;
+    ignoreDependencies: IgnorePatterns;
     isIncludeEntryExports: boolean;
 }
 export interface WorkspaceConfiguration extends BaseWorkspaceConfiguration, Partial<PluginsConfiguration> {
@@ -32,8 +33,8 @@ export interface Configuration {
     include: string[];
     exclude: string[];
     ignore: NormalizedGlob;
-    ignoreBinaries: (string | RegExp)[];
-    ignoreDependencies: (string | RegExp)[];
+    ignoreBinaries: IgnorePatterns;
+    ignoreDependencies: IgnorePatterns;
     ignoreExportsUsedInFile: boolean | Partial<Record<IgnorableExport, boolean>>;
     ignoreWorkspaces: string[];
     isIncludeEntryExports: boolean;

package/dist/util/Performance.d.ts

@@ -1,5 +1,7 @@
 /// <reference types="node" resolution-mode="require"/>
+/// <reference types="node" resolution-mode="require"/>
 import { PerformanceObserver, PerformanceEntry } from 'node:perf_hooks';
+import { memoryUsage } from 'node:process';
 import type { TimerifyOptions } from 'node:perf_hooks';
 type Timerify = <T extends (...params: any[]) => any>(fn: T, options?: TimerifyOptions) => T;
 export declare const timerify: Timerify;
@@ -9,7 +11,10 @@ export declare class Performance {
     endTime: number;
     entries: PerformanceEntry[];
     instanceId?: number;
-    observer?: PerformanceObserver;
+    fnObserver?: PerformanceObserver;
+    gcObserver?: PerformanceObserver;
+    memoryUsageStart?: ReturnType<typeof memoryUsage>;
+    memoryUsageEnd?: ReturnType<typeof memoryUsage>;
     constructor(isEnabled: boolean);
     private setMark;
     private clearMark;
@@ -17,6 +22,7 @@ export declare class Performance {
     private getEntriesByName;
     getTable(): string;
     getTotalTime(): number;
+    getMemHeapUsage(): number;
     finalize(): Promise<void>;
     reset(): void;
 }

package/dist/util/Performance.js

@@ -1,7 +1,11 @@
 import { performance, PerformanceObserver, PerformanceEntry } from 'node:perf_hooks';
+import { constants } from 'node:perf_hooks';
+import { memoryUsage } from 'node:process';
 import EasyTable from 'easy-table';
+import prettyMilliseconds from 'pretty-ms';
 import Summary from 'summary';
 import parsedArgValues from './cli-arguments.js';
+import { debugLog } from './debug.js';
 const { performance: isEnabled = false } = parsedArgValues;
 export const timerify = fn => (isEnabled ? performance.timerify(fn) : fn);
 export class Performance {
@@ -10,17 +14,29 @@ export class Performance {
     endTime = 0;
     entries = [];
     instanceId;
-    observer;
+    fnObserver;
+    gcObserver;
+    memoryUsageStart;
+    memoryUsageEnd;
     constructor(isEnabled) {
         if (isEnabled) {
             this.startTime = performance.now();
             this.instanceId = Math.floor(performance.now() * 100);
-            this.observer = new PerformanceObserver(items => {
+            this.fnObserver = new PerformanceObserver(items => {
                 items.getEntries().forEach(entry => {
                     this.entries.push(entry);
                 });
             });
-            this.observer.observe({ entryTypes: ['function'] });
+            this.fnObserver.observe({ entryTypes: ['function'] });
+            this.gcObserver = new PerformanceObserver(items => {
+                for (const item of items.getEntries()) {
+                    if (item.detail?.kind === constants.NODE_PERFORMANCE_GC_MAJOR) {
+                        debugLog('*', `GC (after ${prettyMilliseconds(item.startTime)} in ${prettyMilliseconds(item.duration)})`);
+                    }
+                }
+            });
+            this.gcObserver.observe({ entryTypes: ['gc'] });
+            this.memoryUsageStart = memoryUsage();
         }
         this.isEnabled = isEnabled;
     }
@@ -67,14 +83,18 @@ export class Performance {
     getTotalTime() {
         return this.endTime - this.startTime;
     }
+    getMemHeapUsage() {
+        return (this.memoryUsageEnd?.heapUsed ?? 0) - (this.memoryUsageStart?.heapUsed ?? 0);
+    }
     async finalize() {
         if (!this.isEnabled)
             return;
         this.endTime = performance.now();
+        this.memoryUsageEnd = memoryUsage();
         await this.flush();
     }
     reset() {
         this.entries = [];
-        this.observer?.disconnect();
+        this.fnObserver?.disconnect();
     }
 }

package/dist/util/cli-arguments.d.ts

@@ -1,4 +1,4 @@
-export declare const helpText = "\u2702\uFE0F  Find unused files, dependencies and exports in your JavaScript and TypeScript projects\n\nUsage: knip [options]\n\nOptions:\n  -c, --config [file]      Configuration file path (default: [.]knip.json[c], knip.js, knip.ts or package.json#knip)\n  -t, --tsConfig [file]    TypeScript configuration path (default: tsconfig.json)\n  --production             Analyze only production source files (e.g. no tests, devDependencies, exported types)\n  --strict                 Consider only direct dependencies of workspace (not devDependencies, not other workspaces)\n  -W, --workspace [dir]    Analyze a single workspace (default: analyze all configured workspaces)\n  --directory [dir]        Run process from a different directory (default: cwd)\n  --no-gitignore           Don't use .gitignore\n  --include                Report only provided issue type(s), can be comma-separated or repeated (1)\n  --exclude                Exclude provided issue type(s) from report, can be comma-separated or repeated (1)\n  --dependencies           Shortcut for --include dependencies,unlisted,binaries,unresolved\n  --exports                Shortcut for --include exports,nsExports,classMembers,types,nsTypes,enumMembers,duplicates\n  --fix                    Fix issues\n  --fix-type               Fix only issues of type, can be comma-separated or repeated (2)\n  --include-entry-exports  Include entry files when reporting unused exports\n  --isolate-workspaces     Isolated workspaces in monorepo\n  -n, --no-progress        Don't show dynamic progress updates (automatically enabled in CI environments)\n  --preprocessor           Preprocess the results before providing it to the reporter(s), can be repeated\n  --preprocessor-options   Pass extra options to the preprocessor (as JSON string, see --reporter-options example)\n  --reporter               Select reporter: symbols, compact, codeowners, json, can be repeated (default: symbols)\n  --reporter-options       Pass extra options to the reporter (as JSON string, see example)\n  --no-config-hints        Suppress configuration hints\n  --no-exit-code           Always exit with code zero (0)\n  --max-issues             Maximum number of issues before non-zero exit code (default: 0)\n  -d, --debug              Show debug output\n  --performance            Measure count and running time of expensive functions and display stats table\n  -h, --help               Print this help text\n  -V, --version            Print version\n\n(1) Issue types: files, dependencies, unlisted, unresolved, exports, nsExports, classMembers, types, nsTypes, enumMembers, duplicates\n(2) Fixable issue types: dependencies, exports, types\n\nExamples:\n\n$ knip\n$ knip --production\n$ knip --workspace packages/client --include files,dependencies\n$ knip -c ./config/knip.json --reporter compact\n$ knip --reporter codeowners --reporter-options '{\"path\":\".github/CODEOWNERS\"}'\n\nWebsite: https://knip.dev";
+export declare const helpText = "\u2702\uFE0F  Find unused files, dependencies and exports in your JavaScript and TypeScript projects\n\nUsage: knip [options]\n\nOptions:\n  -c, --config [file]      Configuration file path (default: [.]knip.json[c], knip.js, knip.ts or package.json#knip)\n  -t, --tsConfig [file]    TypeScript configuration path (default: tsconfig.json)\n  --production             Analyze only production source files (e.g. no tests, devDependencies, exported types)\n  --strict                 Consider only direct dependencies of workspace (not devDependencies, not other workspaces)\n  -W, --workspace [dir]    Analyze a single workspace (default: analyze all configured workspaces)\n  --directory [dir]        Run process from a different directory (default: cwd)\n  --no-gitignore           Don't use .gitignore\n  --include                Report only provided issue type(s), can be comma-separated or repeated (1)\n  --exclude                Exclude provided issue type(s) from report, can be comma-separated or repeated (1)\n  --dependencies           Shortcut for --include dependencies,unlisted,binaries,unresolved\n  --exports                Shortcut for --include exports,nsExports,classMembers,types,nsTypes,enumMembers,duplicates\n  --fix                    Fix issues\n  --fix-type               Fix only issues of type, can be comma-separated or repeated (2)\n  --include-entry-exports  Include entry files when reporting unused exports\n  --isolate-workspaces     Isolate workspaces into separate programs (default: false)\n  -n, --no-progress        Don't show dynamic progress updates (automatically enabled in CI environments)\n  --preprocessor           Preprocess the results before providing it to the reporter(s), can be repeated\n  --preprocessor-options   Pass extra options to the preprocessor (as JSON string, see --reporter-options example)\n  --reporter               Select reporter: symbols, compact, codeowners, json, can be repeated (default: symbols)\n  --reporter-options       Pass extra options to the reporter (as JSON string, see example)\n  --no-config-hints        Suppress configuration hints\n  --no-exit-code           Always exit with code zero (0)\n  --max-issues             Maximum number of issues before non-zero exit code (default: 0)\n  -d, --debug              Show debug output\n  --performance            Measure count and running time of expensive functions and display stats table\n  -h, --help               Print this help text\n  -V, --version            Print version\n\n(1) Issue types: files, dependencies, unlisted, unresolved, exports, nsExports, classMembers, types, nsTypes, enumMembers, duplicates\n(2) Fixable issue types: dependencies, exports, types\n\nExamples:\n\n$ knip\n$ knip --production\n$ knip --workspace packages/client --include files,dependencies\n$ knip -c ./config/knip.json --reporter compact\n$ knip --reporter codeowners --reporter-options '{\"path\":\".github/CODEOWNERS\"}'\n\nWebsite: https://knip.dev";
 declare const _default: {
     config: string | undefined;
     debug: boolean | undefined;

package/dist/util/cli-arguments.js

@@ -18,7 +18,7 @@ Options:
   --fix                    Fix issues
   --fix-type               Fix only issues of type, can be comma-separated or repeated (2)
   --include-entry-exports  Include entry files when reporting unused exports
-  --isolate-workspaces     Isolated workspaces in monorepo
+  --isolate-workspaces     Isolate workspaces into separate programs (default: false)
   -n, --no-progress        Don't show dynamic progress updates (automatically enabled in CI environments)
   --preprocessor           Preprocess the results before providing it to the reporter(s), can be repeated
   --preprocessor-options   Pass extra options to the preprocessor (as JSON string, see --reporter-options example)

package/dist/util/regex.d.ts

@@ -2,3 +2,4 @@ export declare const toRegexOrString: (value: string | RegExp) => string | RegEx
 export declare const hasMatch: (haystack: undefined | (string | RegExp)[], needle: string) => boolean | undefined;
 export declare const hasMatchInSet: (haystack: undefined | Set<string>, needle: string | RegExp) => boolean | undefined;
 export declare const hasMatchInArray: (haystack: string[], needle: string | RegExp) => boolean;
+export declare const findKey: (map: Map<string | RegExp, unknown>, key: RegExp) => string | RegExp | undefined;

package/dist/util/regex.js

@@ -2,3 +2,4 @@ export const toRegexOrString = (value) => typeof value === 'string' && /[*+\\(|{
 export const hasMatch = (haystack, needle) => haystack && haystack.some(n => (typeof n === 'string' ? n === needle : n.test(needle)));
 export const hasMatchInSet = (haystack, needle) => haystack && (typeof needle === 'string' ? haystack.has(needle) : [...haystack].some(n => needle.test(n)));
 export const hasMatchInArray = (haystack, needle) => typeof needle === 'string' ? haystack.includes(needle) : haystack.some(n => needle.test(n));
+export const findKey = (map, key) => [...map.keys()].find(k => k instanceof RegExp && k.source === key.source);

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "3.7.1";
+export declare const version = "3.8.1";

package/dist/version.js

@@ -1 +1 @@
-export const version = '3.7.1';
+export const version = '3.8.1';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "3.7.1",
+  "version": "3.8.1",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://knip.dev",
   "repository": {
@@ -58,7 +58,7 @@
     "@npmcli/package-json": "5.0.0",
     "@pkgjs/parseargs": "0.11.0",
     "@pnpm/logger": "5.0.0",
-    "@pnpm/workspace.pkgs-graph": "^2.0.11",
+    "@pnpm/workspace.pkgs-graph": "^2.0.12",
     "@snyk/github-codeowners": "1.1.0",
     "chalk": "^5.3.0",
     "easy-table": "1.2.0",
@@ -94,8 +94,8 @@
     "c8": "8.0.1",
     "eslint": "^8.55.0",
     "playwright": "^1.40.1",
-    "prettier": "^3.1.0",
-    "release-it": "^17.0.0",
+    "prettier": "^3.1.1",
+    "release-it": "^17.0.1",
     "tsx": "^4.6.2",
     "type-fest": "^4.8.3",
     "typescript": "5.3.3"