knip 5.41.1 → 5.42.0

package/dist/typescript/get-imports-and-exports.js

@@ -6,7 +6,7 @@ import { addNsValue, addValue, createImports } from '../util/dependency-graph.js
 import { getPackageNameFromFilePath, isStartsLikePackageName, sanitizeSpecifier } from '../util/modules.js';
 import { isInNodeModules } from '../util/path.js';
 import { shouldIgnore } from '../util/tag.js';
-import { getAccessMembers, getDestructuredIds, getJSDocTags, getLineAndCharacterOfPosition, getTypeName, isAccessExpression, isConsiderReferencedNS, isDestructuring, isImportSpecifier, isObjectEnumerationCallExpressionArgument, isReferencedInExport, } from './ast-helpers.js';
+import { getAccessMembers, getDestructuredIds, getJSDocTags, getLineAndCharacterOfPosition, getTypeName, isAccessExpression, isConsiderReferencedNS, isDestructuring, isImportSpecifier, isInForIteration, isObjectEnumerationCallExpressionArgument, isReferencedInExport, } from './ast-helpers.js';
 import { findInternalReferences, isType } from './find-internal-references.js';
 import getDynamicImportVisitors from './visitors/dynamic-imports/index.js';
 import getExportVisitors from './visitors/exports/index.js';
@@ -35,9 +35,9 @@ const createMember = (node, member, pos) => {
 };
 const getImportsAndExports = (sourceFile, resolveModule, typeChecker, options) => {
     const { skipTypeOnly, tags, ignoreExportsUsedInFile } = options;
-    const internalImports = new Map();
-    const externalImports = new Set();
-    const unresolvedImports = new Set();
+    const internal = new Map();
+    const external = new Set();
+    const unresolved = new Set();
     const resolved = new Set();
     const specifiers = new Set();
     const exports = new Map();
@@ -80,10 +80,10 @@ const getImportsAndExports = (sourceFile, resolveModule, typeChecker, options) =
         const { identifier, symbol, filePath, namespace, alias, specifier, isReExport } = options;
         const isStar = identifier === IMPORT_STAR;
         specifiers.add([specifier, filePath]);
-        const file = internalImports.get(filePath);
+        const file = internal.get(filePath);
         const imports = file ?? createImports();
         if (!file)
-            internalImports.set(filePath, imports);
+            internal.set(filePath, imports);
         const nsOrAlias = symbol ? String(symbol.escapedName) : alias;
         if (isReExport) {
             if (isStar && namespace) {
@@ -134,7 +134,7 @@ const getImportsAndExports = (sourceFile, resolveModule, typeChecker, options) =
                     if (!isStartsLikePackageName(sanitizedSpecifier)) {
                         return;
                     }
-                    externalImports.add(sanitizedSpecifier);
+                    external.add(sanitizedSpecifier);
                 }
             }
         }
@@ -147,10 +147,10 @@ const getImportsAndExports = (sourceFile, resolveModule, typeChecker, options) =
                 return;
             if (typeof pos === 'number') {
                 const { line, character } = sourceFile.getLineAndCharacterOfPosition(pos);
-                unresolvedImports.add({ specifier, pos, line: line + 1, col: character + 1 });
+                unresolved.add({ specifier, pos, line: line + 1, col: character + 1 });
             }
             else {
-                unresolvedImports.add({ specifier });
+                unresolved.add({ specifier });
             }
         }
     };
@@ -161,7 +161,7 @@ const getImportsAndExports = (sourceFile, resolveModule, typeChecker, options) =
             const importedSymbolFilePath = importedInternalSymbols.get(symbol);
             if (importedSymbolFilePath) {
                 const importId = String(symbol.escapedName);
-                const internalImport = internalImports.get(importedSymbolFilePath);
+                const internalImport = internal.get(importedSymbolFilePath);
                 if (internalImport) {
                     if (importId !== identifier) {
                         addNsValue(internalImport.reExportedAs, importId, identifier, sourceFile.fileName);
@@ -242,7 +242,7 @@ const getImportsAndExports = (sourceFile, resolveModule, typeChecker, options) =
             if (symbol) {
                 if (filePath) {
                     if (!isImportSpecifier(node)) {
-                        const imports = internalImports.get(filePath);
+                        const imports = internal.get(filePath);
                         if (imports) {
                             traceRefs.add(id);
                             if (isAccessExpression(node.parent)) {
@@ -277,6 +277,9 @@ const getImportsAndExports = (sourceFile, resolveModule, typeChecker, options) =
                                 else if (isObjectEnumerationCallExpressionArgument(node)) {
                                     imports.refs.add(id);
                                 }
+                                else if (isInForIteration(node)) {
+                                    imports.refs.add(id);
+                                }
                             }
                         }
                     }
@@ -294,7 +297,7 @@ const getImportsAndExports = (sourceFile, resolveModule, typeChecker, options) =
             const namespace = left.text;
             const { filePath } = getImport(namespace, node);
             if (filePath) {
-                const internalImport = internalImports.get(filePath);
+                const internalImport = internal.get(filePath);
                 if (internalImport)
                     addNsMemberRefs(internalImport, namespace, right.text);
             }
@@ -327,17 +330,9 @@ const getImportsAndExports = (sourceFile, resolveModule, typeChecker, options) =
         item.symbol = undefined;
     }
     return {
-        imports: {
-            internal: internalImports,
-            external: externalImports,
-            resolved,
-            specifiers,
-            unresolved: unresolvedImports,
-        },
-        exports: {
-            exported: exports,
-            duplicate: [...aliasedExports.values()],
-        },
+        imports: { internal, external, resolved, specifiers, unresolved },
+        exports,
+        duplicates: [...aliasedExports.values()],
         scripts,
         traceRefs,
     };

package/dist/typescript/visitors/exports/exportAssignment.js

@@ -1,8 +1,9 @@
 import ts from 'typescript';
 import { FIX_FLAGS } from '../../../constants.js';
 import { SymbolType } from '../../../types/issues.js';
+import { isModule } from '../helpers.js';
 import { exportVisitor as visit } from '../index.js';
-export default visit(() => true, (node, { isFixExports }) => {
+export default visit(isModule, (node, { isFixExports }) => {
     if (ts.isExportAssignment(node)) {
         const pos = node.getChildAt(1).getStart();
         const fix = isFixExports ? [node.getStart(), node.getEnd() + 1, FIX_FLAGS.NONE] : undefined;

package/dist/typescript/visitors/exports/exportDeclaration.js

@@ -1,8 +1,9 @@
 import ts from 'typescript';
 import { FIX_FLAGS } from '../../../constants.js';
 import { SymbolType } from '../../../types/issues.js';
+import { isModule } from '../helpers.js';
 import { exportVisitor as visit } from '../index.js';
-export default visit(() => true, (node, { isFixExports, isFixTypes }) => {
+export default visit(isModule, (node, { isFixExports, isFixTypes }) => {
     if (ts.isExportDeclaration(node)) {
         if (node.exportClause && ts.isNamedExports(node.exportClause)) {
             const nodeType = node.isTypeOnly ? SymbolType.TYPE : SymbolType.UNKNOWN;

package/dist/typescript/visitors/exports/exportKeyword.js

@@ -3,8 +3,9 @@ import { FIX_FLAGS } from '../../../constants.js';
 import { SymbolType } from '../../../types/issues.js';
 import { compact } from '../../../util/array.js';
 import { getDefaultKeywordNode, getExportKeywordNode, isGetOrSetAccessorDeclaration, isPrivateMember, stripQuotes, } from '../../ast-helpers.js';
+import { isModule } from '../helpers.js';
 import { exportVisitor as visit } from '../index.js';
-export default visit(() => true, (node, { isFixExports, isFixTypes, isReportClassMembers }) => {
+export default visit(isModule, (node, { isFixExports, isFixTypes, isReportClassMembers }) => {
     const exportKeyword = getExportKeywordNode(node);
     if (exportKeyword) {
         const getFix = (node, defaultKeyword) => isFixExports ? [node.getStart(), (defaultKeyword ?? node).getEnd() + 1, FIX_FLAGS.NONE] : undefined;
@@ -14,9 +15,7 @@ export default visit(() => true, (node, { isFixExports, isFixTypes, isReportClas
                 if (ts.isObjectBindingPattern(declaration.name)) {
                     return compact(declaration.name.elements.map(element => {
                         if (ts.isIdentifier(element.name)) {
-                            const fix = isFixExports
-                                ? [element.getStart(), element.getEnd(), FIX_FLAGS.OBJECT_BINDING]
-                                : undefined;
+                            const fix = isFixExports ? [element.getStart(), element.getEnd(), FIX_FLAGS.OBJECT_BINDING] : undefined;
                             return {
                                 node: element,
                                 symbol: element.symbol,

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

@@ -3,5 +3,6 @@ import type { ImportNode } from '../../types/imports.js';
 import type { BoundSourceFile } from '../SourceFile.js';
 export declare const isNotJS: (sourceFile: BoundSourceFile) => boolean;
 export declare const isJS: (sourceFile: BoundSourceFile) => boolean;
+export declare const isModule: (sourceFile: BoundSourceFile) => boolean;
 export declare function getImportsFromPragmas(sourceFile: BoundSourceFile): ImportNode[];
 export declare function hasImportSpecifier(node: ts.Statement, name: string): boolean;

package/dist/typescript/visitors/helpers.js

@@ -1,6 +1,7 @@
 import ts from 'typescript';
 export const isNotJS = (sourceFile) => !isJS(sourceFile);
 export const isJS = (sourceFile) => sourceFile.scriptKind === ts.ScriptKind.JS || sourceFile.scriptKind === ts.ScriptKind.JSX;
+export const isModule = (sourceFile) => ts.isExternalModule(sourceFile);
 export function getImportsFromPragmas(sourceFile) {
     const importNodes = [];
     if (sourceFile.pragmas) {

package/dist/util/dependency-graph.js

@@ -36,10 +36,8 @@ const createFileNode = () => ({
         external: new Set(),
         unresolved: new Set(),
     },
-    exports: {
-        exported: new Map(),
-        duplicate: new Set(),
-    },
+    exports: new Map(),
+    duplicates: new Set(),
     scripts: new Set(),
     traceRefs: new Set(),
 });

package/dist/util/file-entry-cache.js

@@ -1,12 +1,10 @@
 import fs from 'node:fs';
 import path from 'node:path';
 import { timerify } from './Performance.js';
-import parsedArgValues from './cli-arguments.js';
 import { debugLog } from './debug.js';
 import { isDirectory, isFile } from './fs.js';
-import { dirname, isAbsolute, resolve } from './path.js';
+import { cwd, dirname, isAbsolute, resolve } from './path.js';
 import { deserialize, serialize } from './serialize.js';
-const cwd = process.cwd();
 const createCache = (filePath) => {
     try {
         return deserialize(fs.readFileSync(filePath));
@@ -21,9 +19,7 @@ export class FileEntryCache {
     cache = new Map();
     normalizedEntries = new Map();
     constructor(cacheId, _path) {
-        const { 'cache-location': cacheLocation } = parsedArgValues;
-        const currentPath = cacheLocation ? path.resolve(cacheLocation) : path.resolve(cwd);
-        this.filePath = isAbsolute(currentPath) ? path.join(currentPath, cacheId) : path.join(currentPath, _path, cacheId);
+        this.filePath = isAbsolute(_path) ? path.resolve(_path, cacheId) : path.resolve(cwd, _path, cacheId);
         if (isFile(this.filePath))
             this.cache = create(this.filePath);
         this.removeNotFoundFiles();

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "5.41.1";
+export declare const version = "5.42.0";

package/dist/version.js

@@ -1 +1 @@
-export const version = '5.41.1';
+export const version = '5.42.0';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "5.41.1",
+  "version": "5.42.0",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://knip.dev",
   "repository": {
@@ -48,7 +48,7 @@
     "qa": "bun lint && bun run build && bun knip && bun knip:production && bun run test",
     "release": "release-it",
     "create-plugin": "bun ./scripts/create-new-plugin.ts",
-    "postcreate-plugin": "biome format --write schema.json schema-jsonc.json src/ConfigurationValidator.ts",
+    "postcreate-plugin": "bun run build && biome format --write schema.json schema-jsonc.json src/ConfigurationValidator.ts",
     "generate-plugin-defs": "node ./scripts/generate-plugin-defs.js && biome check --write src/plugins/index.ts src/types/PluginNames.ts src/schema/plugins.ts"
   },
   "files": [
@@ -58,12 +58,12 @@
     "schema-jsonc.json"
   ],
   "dependencies": {
-    "@nodelib/fs.walk": "1.2.8",
+    "@nodelib/fs.walk": "3.0.1",
     "@snyk/github-codeowners": "1.1.0",
     "easy-table": "1.2.0",
-    "enhanced-resolve": "^5.17.1",
-    "fast-glob": "^3.3.2",
-    "jiti": "^2.4.0",
+    "enhanced-resolve": "^5.18.0",
+    "fast-glob": "^3.3.3",
+    "jiti": "^2.4.2",
     "js-yaml": "^4.1.0",
     "minimist": "^1.2.8",
     "picocolors": "^1.1.0",
@@ -81,20 +81,20 @@
   },
   "devDependencies": {
     "@jest/types": "^29.6.3",
-    "@release-it/bumper": "^6.0.1",
-    "@types/bun": "^1.1.13",
+    "@release-it/bumper": "^7.0.0",
+    "@types/bun": "^1.1.15",
     "@types/js-yaml": "^4.0.9",
     "@types/minimist": "^1.2.5",
     "@types/picomatch": "3.0.1",
     "@types/webpack": "^5.28.5",
-    "@wdio/types": "^9.2.2",
+    "@wdio/types": "^9.5.0",
     "glob": "^10.4.2",
-    "release-it": "^17.10.0",
-    "type-fest": "^4.26.1",
+    "release-it": "^18.1.1",
+    "type-fest": "^4.31.0",
     "typescript": "^5.5.2"
   },
   "engines": {
-    "node": ">=18.6.0"
+    "node": ">=18.18.0"
   },
   "engineStrict": true,
   "keywords": [

package/schema.json

@@ -400,6 +400,10 @@
           "title": "markdownlint plugin configuration (https://knip.dev/reference/plugins/markdownlint)",
           "$ref": "#/definitions/plugin"
         },
+        "metro": {
+          "title": "metro plugin configuration (https://knip.dev/reference/plugins/metro)",
+          "$ref": "#/definitions/plugin"
+        },
         "mocha": {
           "title": "Mocha plugin configuration (https://knip.dev/reference/plugins/mocha)",
           "$ref": "#/definitions/plugin"