knip 6.32.3 → 6.34.0

package/dist/typescript/visitors/walk.d.ts

@@ -1,4 +1,4 @@
-import { Visitor, type Class, type Function as FunctionNode, type Program, type Span, type VariableDeclarator } from 'oxc-parser';
+import { Visitor, type Class, type Function as FunctionNode, type Program, type Span, type TSInterfaceDeclaration, type VariableDeclarator } from 'oxc-parser';
 import type { PluginVisitorObject } from '../../types/config.ts';
 import type { GetImportsAndExportsOptions } from '../../types/config.ts';
 import type { Fix } from '../../types/exports.ts';
@@ -43,6 +43,7 @@ interface WalkContext {
     resolveModule: ResolveModule;
     programFiles: Set<string>;
     entryFiles: Set<string>;
+    handledImportExpressions: Set<number>;
     visitor: Visitor;
     getJSDocTags: (nodeStart: number) => Set<string>;
 }
@@ -66,10 +67,12 @@ export interface WalkState extends WalkContext {
     scopeEnds: number[];
     shadowScopes: Map<string, [number, number][]>;
     localDeclarations: Map<string, FunctionNode | Class | VariableDeclarator>;
+    localInterfaces: Map<string, TSInterfaceDeclaration | null>;
     pendingCallRefs: Array<{
         name: string;
         exportName: string;
         seen: Set<string>;
+        kind: 'callee' | 'argument';
     }>;
     pendingMemberCallRefs: Array<{
         objectName: string;

package/dist/typescript/visitors/walk.js

@@ -49,7 +49,7 @@ const _addExport = (identifier, type, pos, members, fix, isReExport, jsDocTags)
         });
     }
 };
-const _collectRefsInType = (node, exportName, signatureOnly, seen = new Set(), inBody = false) => {
+const _collectRefsInType = (node, exportName, signatureOnly, seen = new Set(), inBody = false, thisType) => {
     if (!node)
         return;
     const type = node.type;
@@ -64,10 +64,14 @@ const _collectRefsInType = (node, exportName, signatureOnly, seen = new Set(), i
             if (node.typeName?.type === 'Identifier')
                 _addRefInExport(node.typeName.name, exportName);
             break;
+        case 'TSThisType':
+            if (thisType)
+                _addRefInExport(thisType, exportName);
+            return;
         case 'CallExpression': {
             const callee = node.callee;
             if (callee?.type === 'Identifier') {
-                state.pendingCallRefs.push({ name: callee.name, exportName, seen });
+                state.pendingCallRefs.push({ name: callee.name, exportName, seen, kind: 'callee' });
             }
             else if (callee?.type === 'MemberExpression' &&
                 !callee.computed &&
@@ -84,8 +88,9 @@ const _collectRefsInType = (node, exportName, signatureOnly, seen = new Set(), i
                 const args = node.arguments;
                 if (args) {
                     for (const arg of args) {
-                        if (arg?.type === 'Identifier')
-                            state.pendingCallRefs.push({ name: arg.name, exportName, seen });
+                        if (arg?.type === 'Identifier') {
+                            state.pendingCallRefs.push({ name: arg.name, exportName, seen, kind: 'argument' });
+                        }
                     }
                 }
             }
@@ -101,14 +106,14 @@ const _collectRefsInType = (node, exportName, signatureOnly, seen = new Set(), i
         case 'TSSatisfiesExpression':
             if (inBody) {
                 if (node.expression)
-                    _collectRefsInType(node.expression, exportName, signatureOnly, seen, inBody);
+                    _collectRefsInType(node.expression, exportName, signatureOnly, seen, inBody, thisType);
                 return;
             }
             break;
         case 'VariableDeclarator':
             if (inBody) {
                 if (node.init)
-                    _collectRefsInType(node.init, exportName, signatureOnly, seen, inBody);
+                    _collectRefsInType(node.init, exportName, signatureOnly, seen, inBody, thisType);
                 return;
             }
             break;
@@ -124,14 +129,63 @@ const _collectRefsInType = (node, exportName, signatureOnly, seen = new Set(), i
         if (Array.isArray(val)) {
             for (const item of val) {
                 if (item)
-                    _collectRefsInType(item, exportName, signatureOnly, seen, childInBody);
+                    _collectRefsInType(item, exportName, signatureOnly, seen, childInBody, thisType);
             }
         }
         else {
-            _collectRefsInType(val, exportName, signatureOnly, seen, childInBody);
+            _collectRefsInType(val, exportName, signatureOnly, seen, childInBody, thisType);
         }
     }
 };
+const _collectRefsInDirectMemberResult = (fn, exportName, seen) => {
+    const body = fn.body;
+    if (body?.type !== 'MemberExpression' ||
+        body.computed ||
+        body.object?.type !== 'Identifier' ||
+        body.property?.type !== 'Identifier') {
+        return false;
+    }
+    const params = Array.isArray(fn.params) ? fn.params : (fn.params?.items ?? []);
+    const param = params.find((param) => param.type === 'Identifier' && param.name === body.object.name);
+    const paramType = param?.typeAnnotation?.typeAnnotation;
+    if (paramType?.type !== 'TSTypeReference' || paramType.typeName?.type !== 'Identifier' || paramType.typeArguments) {
+        return false;
+    }
+    const declaration = state.localInterfaces.get(paramType.typeName.name);
+    if (!declaration || declaration.typeParameters)
+        return false;
+    const members = declaration.body.body.filter((member) => member.type === 'TSPropertySignature' &&
+        !member.computed &&
+        member.key?.type === 'Identifier' &&
+        member.key.name === body.property.name &&
+        Boolean(member.typeAnnotation));
+    if (members.length !== 1)
+        return false;
+    _collectRefsInType(members[0].typeAnnotation, exportName, true, seen, false, paramType.typeName.name);
+    return true;
+};
+const _collectRefsInCallResult = (node, exportName, seen) => {
+    const annotatedType = node.type === 'VariableDeclarator' ? node.id.typeAnnotation?.typeAnnotation : undefined;
+    if (annotatedType?.type === 'TSFunctionType') {
+        _collectRefsInType(annotatedType.typeParameters, exportName, true, seen);
+        return _collectRefsInType(annotatedType.returnType, exportName, true, seen);
+    }
+    if (annotatedType)
+        return _collectRefsInType(annotatedType, exportName, true, seen);
+    const fn = node.type === 'VariableDeclarator' ? node.init : node;
+    if (fn?.type !== 'ArrowFunctionExpression' &&
+        fn?.type !== 'FunctionDeclaration' &&
+        fn?.type !== 'FunctionExpression') {
+        return _collectRefsInType(node, exportName, true, seen);
+    }
+    if (fn.returnType) {
+        _collectRefsInType(fn.typeParameters, exportName, true, seen);
+        return _collectRefsInType(fn.returnType, exportName, true, seen);
+    }
+    if (_collectRefsInDirectMemberResult(fn, exportName, seen))
+        return;
+    _collectRefsInType(fn, exportName, true, seen);
+};
 const _addRefInExport = (name, exportName) => {
     const refs = state.referencedInExport.get(name);
     if (refs)
@@ -221,6 +275,12 @@ const coreVisitorObject = {
                 state.addImport(specifier, name, undefined, undefined, node.id.start, IMPORT_FLAGS.TYPE_ONLY | IMPORT_FLAGS.AUGMENT);
         }
     },
+    TSInterfaceDeclaration(node) {
+        if (state.scopeDepth > 0 || state.isInNamespace(node))
+            return;
+        const name = node.id.name;
+        state.localInterfaces.set(name, state.localInterfaces.has(name) ? null : node);
+    },
     ClassDeclaration(node) {
         state.classNameStack.push(node.id?.name ?? '');
         if (node.id?.name) {
@@ -726,7 +786,6 @@ function walkAST(program, sourceText, filePath, hasModuleSyntax, ctx) {
         sourceText,
         isJS,
         isModuleFile: isExternalModule(program, hasModuleSyntax),
-        handledImportExpressions: new Set(),
         bareExprRefs: new Set(),
         accessedAliases: new Set(),
         nsContainers: new Map(),
@@ -741,6 +800,7 @@ function walkAST(program, sourceText, filePath, hasModuleSyntax, ctx) {
         scopeEnds: [],
         shadowScopes: new Map(),
         localDeclarations: new Map(),
+        localInterfaces: new Map(),
         pendingCallRefs: [],
         pendingMemberCallRefs: [],
         localToExports: new Map(),
@@ -757,14 +817,18 @@ function walkAST(program, sourceText, filePath, hasModuleSyntax, ctx) {
     ctx.visitor.visit(program);
     while (state.pendingCallRefs.length > 0 || state.pendingMemberCallRefs.length > 0) {
         while (state.pendingCallRefs.length > 0) {
-            const { name, exportName, seen } = state.pendingCallRefs.pop();
-            if (seen.has(name))
+            const { name, exportName, seen, kind } = state.pendingCallRefs.pop();
+            const key = `${kind}:${name}`;
+            if (seen.has(key))
                 continue;
             const decl = state.localDeclarations.get(name);
             if (!decl)
                 continue;
-            seen.add(name);
-            _collectRefsInType(decl, exportName, true, seen);
+            seen.add(key);
+            if (kind === 'callee')
+                _collectRefsInCallResult(decl, exportName, seen);
+            else
+                _collectRefsInType(decl, exportName, true, seen);
         }
         while (state.pendingMemberCallRefs.length > 0) {
             const { objectName, propertyName, exportName, seen } = state.pendingMemberCallRefs.pop();
@@ -781,7 +845,7 @@ function walkAST(program, sourceText, filePath, hasModuleSyntax, ctx) {
             if (fn.type !== 'ArrowFunctionExpression' && fn.type !== 'FunctionExpression')
                 continue;
             seen.add(key);
-            _collectRefsInType(fn, exportName, true, seen);
+            _collectRefsInCallResult(fn, exportName, seen);
         }
     }
     for (let i = 0; i < state.memberRefsInFile.length; i += 2) {

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

@@ -1,5 +1,6 @@
 export declare const helpText = "\u2702\uFE0F  Find unused dependencies, exports and files in your JavaScript and TypeScript projects\n\nUsage: knip [options]\n\nOptions:\n  -h, --help                   Print this help text\n  -V, --version                Print version\n  -n, --no-progress            Don't show dynamic progress updates (automatically enabled in CI environments)\n  -c, --config [file]          Configuration file path\n                               (default: [.]knip.json[c], knip.(js|ts), knip.config.(js|ts) or package.json#knip)\n      --use-tsconfig-files     Use tsconfig.json to define project files (override `project` patterns)\n  -t, --tsConfig [file]        TypeScript configuration path (default: tsconfig.json)\n\nMode\n      --cache                  Enable caching\n      --cache-location         Change cache location (default: node_modules/.cache/knip)\n      --include-entry-exports  Include entry files when reporting unused exports\n      --no-gitignore           Don't respect .gitignore\n  -p, --production             Analyze only production source files (e.g. no test files, devDependencies)\n  -s, --strict                 Consider only direct dependencies of workspace (not devDependencies, not other workspaces)\n  -w, --watch                  Watch mode\n\nScope\n  -W, --workspace [filter]     Filter workspaces by name, directory, or glob (can be repeated)\n  -D, --directory [dir]        Run process from a different directory (default: cwd)\n      --include                Include 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,catalog,catalogReferences\n      --exports                Shortcut for --include exports,nsExports,types,nsTypes,enumMembers,namespaceMembers,duplicates\n      --files                  Shortcut for --include files\n      --cycles                 Shortcut for --include cycles (circular dependencies)\n      --tags                   Include or exclude tagged exports\n\nFix\n  -f, --fix                    Fix issues (modifies files in your repo)\n      --fix-type               Fix only issues of type, can be comma-separated or repeated (2)\n      --allow-remove-files     Allow Knip to remove files (with --fix)\n  -F, --format                 Format modified files after --fix using the local formatter\n\nOutput\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 (default: symbols), can be repeated (3)\n      --reporter-options       Pass extra options to the reporter (as JSON string, see example)\n      --no-config-hints        Suppress configuration hints\n      --no-tag-hints           Suppress tag hints\n      --treat-config-hints-as-errors   Exit with non-zero code (1) if there are any configuration hints\n      --treat-tag-hints-as-errors      Exit with non-zero code (1) if there are any tag hints\n      --max-issues             Maximum number of total issues before non-zero exit code (default: 0)\n      --max-show-issues        Maximum number of issues to display per type\n      --no-exit-code           Always exit with code zero (0)\n\nTroubleshooting\n  -d, --debug                  Show debug output\n      --memory                 Measure memory usage and display data table\n      --memory-realtime        Log memory usage in realtime\n      --performance            Measure count and running time of key functions and display stats table\n      --performance-fn [name]  Measure only function [name]\n  -u, --duration               Print total running time (zero overhead, no instrumentation)\n      --trace                  Show trace output\n      --trace-dependency [name]  Show files that import the named dependency\n      --trace-export [name]    Show trace output for named export(s)\n      --trace-file [file]      Show trace output for exports in file\n\n(1) Issue types: files, dependencies, unlisted, unresolved, exports, nsExports, types, nsTypes, enumMembers, namespaceMembers, duplicates, catalog, catalogReferences, cycles\n(2) Fixable issue types: dependencies, exports, types, files, catalog\n(3) Built-in reporters: symbols (default), compact, codeowners, cycles, json, codeclimate, markdown, disclosure, github-actions, sarif\n\nExamples:\n\n$ knip\n$ knip --production\n$ knip --workspace packages/client --include files,dependencies\n$ knip --workspace @myorg/* --workspace '!@myorg/legacy'\n$ knip --workspace './apps/*' --workspace '@shared/utils'\n$ knip -c ./config/knip.json --reporter compact\n$ knip --reporter codeowners --reporter-options '{\"path\":\".github/CODEOWNERS\"}'\n$ knip --tags=-lintignore\n\nWebsite: https://knip.dev";
 export type ParsedCLIArgs = ReturnType<typeof parseCLIArgs>;
+export declare const parseNumericOption: (value: string | undefined, option: string) => number | undefined;
 export default function parseCLIArgs(): {
     cache?: boolean | undefined;
     'cache-location'?: string | undefined;

package/dist/util/cli-arguments.js

@@ -1,4 +1,5 @@
 import { parseArgs } from 'node:util';
+import { ConfigurationError } from './errors.js';
 export const helpText = `✂️  Find unused dependencies, exports and files in your JavaScript and TypeScript projects
 
 Usage: knip [options]
@@ -79,6 +80,14 @@ $ knip --reporter codeowners --reporter-options '{"path":".github/CODEOWNERS"}'
 $ knip --tags=-lintignore
 
 Website: https://knip.dev`;
+export const parseNumericOption = (value, option) => {
+    if (value === undefined)
+        return undefined;
+    if (!/^\d+$/.test(value)) {
+        throw new ConfigurationError(`Option --${option} expects a non-negative integer, got: ${value}`);
+    }
+    return Number(value);
+};
 export default function parseCLIArgs() {
     return parseArgs({
         options: {

package/dist/util/create-options.d.ts

@@ -1,5 +1,5 @@
 import type { Options } from '../types/options.ts';
-import type { ParsedCLIArgs } from './cli-arguments.ts';
+import { type ParsedCLIArgs } from './cli-arguments.ts';
 interface CreateOptions extends Partial<Options> {
     args?: ParsedCLIArgs;
 }
@@ -43,6 +43,7 @@ export declare const createOptions: (options: CreateOptions) => Promise<{
     isTreatTagHintsAsErrors: boolean;
     isUseTscFiles: boolean | undefined;
     isWatch: boolean;
+    maxIssues: number;
     maxShowIssues: number | undefined;
     parsedConfig: {
         angular?: string | boolean | string[] | {

package/dist/util/create-options.js

@@ -2,6 +2,7 @@ import { partitionCompilers } from '../compilers/index.js';
 import { ISSUE_TYPES, KNIP_CONFIG_LOCATIONS } from '../constants.js';
 import { knipConfigurationSchema } from '../schema/configuration.js';
 import { getCatalogContainer } from './catalog.js';
+import { parseNumericOption } from './cli-arguments.js';
 import { ConfigurationError } from './errors.js';
 import { findFile, loadJSON } from './fs.js';
 import { getIncludedIssueTypes, shorthandCycles, shorthandDeps, shorthandExports, shorthandFiles, } from './get-included-issue-types.js';
@@ -150,7 +151,8 @@ export const createOptions = async (options) => {
         isTreatTagHintsAsErrors: args['treat-tag-hints-as-errors'] ?? parsedConfig.treatTagHintsAsErrors ?? false,
         isUseTscFiles: options.isUseTscFiles ?? args['use-tsconfig-files'] ?? (options.isSession && !configFilePath),
         isWatch: args.watch ?? options.isWatch ?? false,
-        maxShowIssues: args['max-show-issues'] ? Number(args['max-show-issues']) : undefined,
+        maxIssues: parseNumericOption(args['max-issues'], 'max-issues') ?? 0,
+        maxShowIssues: parseNumericOption(args['max-show-issues'], 'max-show-issues'),
         parsedConfig,
         rules,
         tags,

package/dist/util/load-config.js

@@ -13,12 +13,17 @@ const unwrapFunction = async (maybeFunction, options) => {
     }
     return maybeFunction;
 };
+const isObject = (value) => typeof value === 'object' && value !== null && !Array.isArray(value);
 export async function loadResolvedConfigFile(configPath, options) {
     const loadedValue = await _load(configPath);
+    let config;
     try {
-        return await unwrapFunction(loadedValue, options);
+        config = await unwrapFunction(loadedValue, options);
     }
     catch (_error) {
         throw new ConfigurationError(`Error running the function from ${configPath}`);
     }
+    if (!isObject(config))
+        throw new ConfigurationError(`Expected an object as configuration from ${configPath}`);
+    return config;
 }

package/dist/util/resolve.d.ts

@@ -1,4 +1,5 @@
 export declare const extensionAlias: Record<string, string[]>;
+export declare const resolvePackageManifestPath: (packageName: string, baseDir: string) => string | undefined;
 export declare const _resolveModuleSync: (specifier: string, basePath: string) => string | undefined;
 declare const resolveDeclarationSync: (specifier: string, containingFile: string) => {
     path: string;

package/dist/util/resolve.js

@@ -23,6 +23,17 @@ const declarationResolver = new ResolverFactory({
     nodePath: false,
 });
 resolverInstances.push(declarationResolver);
+const packageManifestResolver = new ResolverFactory({
+    extensions: ['.json'],
+    exportsFields: [],
+    nodePath: false,
+});
+resolverInstances.push(packageManifestResolver);
+export const resolvePackageManifestPath = (packageName, baseDir) => {
+    const resolved = packageManifestResolver.sync(baseDir, `${packageName}/package.json`);
+    if (resolved.path)
+        return toPosix(resolved.path);
+};
 const createSyncModuleResolver = (extensions, tsConfigFile) => {
     const baseOptions = {
         extensions,

package/dist/util/scripts.d.ts

@@ -3,5 +3,6 @@ export interface ScriptCommand {
     binary: string;
     args: string[];
 }
+export declare const toShellCommand: (argv: string[]) => string;
 export declare function walkCommands(node: Node): Generator<Command>;
 export declare const getScriptCommands: (script: string) => ScriptCommand[];

package/dist/util/scripts.js

@@ -1,6 +1,7 @@
 import { parse } from 'unbash';
 import { extractBinary } from './modules.js';
 const spawningBinaries = new Set(['c8', 'cross-env', 'retry-cli']);
+export const toShellCommand = (argv) => argv.map(arg => `'${arg.replaceAll("'", `'\\''`)}'`).join(' ');
 export function* walkCommands(node) {
     switch (node.type) {
         case 'Command':

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "6.32.3";
+export declare const version = "6.34.0";

package/dist/version.js

@@ -1 +1 @@
-export const version = '6.32.3';
+export const version = '6.34.0';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "6.32.3",
+  "version": "6.34.0",
   "description": "Find and fix unused dependencies, exports and files in your TypeScript and JavaScript projects",
   "keywords": [
     "analysis",
@@ -71,6 +71,10 @@
       "types": "./dist/types.d.ts",
       "default": "./dist/index.js"
     },
+    "./config": {
+      "types": "./dist/config.d.ts",
+      "default": "./dist/config.js"
+    },
     "./session": {
       "types": "./dist/session/index.d.ts",
       "default": "./dist/session/index.js"
@@ -78,28 +82,28 @@
   },
   "dependencies": {
     "fdir": "^6.5.0",
-    "formatly": "^0.3.0",
-    "get-tsconfig": "4.14.1",
+    "formatly": "^0.7.0",
+    "get-tsconfig": "4.14.3",
     "jiti": "^2.7.0",
-    "oxc-parser": "^0.143.0",
+    "oxc-parser": "^0.147.0",
     "oxc-resolver": "11.24.2",
-    "picomatch": "^4.0.5",
-    "smol-toml": "^1.7.1",
+    "picomatch": "^4.0.7",
+    "smol-toml": "^1.8.0",
     "strip-json-comments": "5.0.3",
     "tinyglobby": "^0.2.17",
-    "unbash": "^4.0.9",
+    "unbash": "^4.0.10",
     "yaml": "^2.9.0",
     "zod": "^4.4.3"
   },
   "devDependencies": {
-    "@jest/types": "^30.4.1",
-    "@types/bun": "^1.3.14",
+    "@jest/types": "^30.5.0",
+    "@types/bun": "^1.4.0",
     "@types/picomatch": "^4.0.3",
-    "@types/webpack": "^5.28.5",
     "codeclimate-types": "^0.3.1",
     "prettier": "^3.9.6",
-    "tsx": "^4.23.11",
-    "typescript": "7.0.2"
+    "tsx": "^4.23.12",
+    "typescript": "7.0.2",
+    "webpack": "^5.110.1"
   },
   "engines": {
     "node": "^20.19.0 || >=22.12.0"