knip 5.43.2 → 5.43.4

package/dist/index.js

@@ -13,7 +13,7 @@ import { getOrCreateFileNode, updateImportMap } from './util/dependency-graph.js
 import { getReferencedInputsHandler } from './util/get-referenced-inputs.js';
 import { getGitIgnoredHandler } from './util/glob-core.js';
 import { _glob, negate } from './util/glob.js';
-import { getHasStrictlyNsReferences, getType } from './util/has-strictly-ns-references.js';
+import { getType, hasStrictlyEnumReferences, hasStrictlyNsReferences } from './util/has-strictly-ns-references.js';
 import { isConfigPattern, isEntry, isProductionEntry, toProductionEntry } from './util/input.js';
 import { getIsIdentifierReferencedHandler } from './util/is-identifier-referenced.js';
 import { getPackageNameFromModuleSpecifier } from './util/modules.js';
@@ -358,7 +358,7 @@ export const main = async (unresolvedConfiguration) => {
                                 printTrace(traceNode, filePath, identifier);
                             if (isReferenced) {
                                 if (report.enumMembers && exportedItem.type === 'enum') {
-                                    if (importsForExport.refs.has(identifier))
+                                    if (hasStrictlyEnumReferences(importsForExport, identifier))
                                         continue;
                                     for (const member of exportedItem.members) {
                                         if (findMatch(workspace.ignoreMembers, member.identifier))
@@ -424,7 +424,7 @@ export const main = async (unresolvedConfiguration) => {
                                 continue;
                             }
                         }
-                        const [hasStrictlyNsRefs, namespace] = getHasStrictlyNsReferences(graph, importsForExport, identifier);
+                        const [hasStrictlyNsRefs, namespace] = hasStrictlyNsReferences(graph, importsForExport, identifier);
                         const isType = ['enum', 'type', 'interface'].includes(exportedItem.type);
                         if (hasStrictlyNsRefs && ((!report.nsTypes && isType) || !(report.nsExports || isType)))
                             continue;

package/dist/plugins/jest/index.js

@@ -6,7 +6,7 @@ const title = 'Jest';
 const enablers = ['jest'];
 const isEnabled = ({ dependencies, manifest }) => hasDependency(dependencies, enablers) || Boolean(manifest.name?.startsWith('jest-presets'));
 const config = ['jest.config.{js,ts,mjs,cjs,json}', 'package.json'];
-const entry = ['**/__tests__/**/*.[jt]s?(x)', '**/?(*.)+(spec|test).[jt]s?(x)'];
+const entry = ['**/__tests__/**/*.[jt]s?(x)', '**/?(*.)+(spec|test).[jt]s?(x)', '**/__mocks__/**/*.[jt]s'];
 const resolveDependencies = async (config, options) => {
     const { configFileDir } = options;
     if (config?.preset) {

package/dist/typescript/ast-helpers.d.ts

@@ -23,7 +23,7 @@ export declare const isConsiderReferencedNS: (node: ts.Identifier) => boolean;
 export declare const isObjectEnumerationCallExpressionArgument: (node: ts.Identifier) => boolean;
 export declare const isInForIteration: (node: ts.Node) => boolean;
 export declare const isTopLevel: (node: ts.Node) => boolean;
-export declare const getTypeName: (node: ts.Identifier) => ts.QualifiedName | undefined;
+export declare const getTypeRef: (node: ts.Identifier) => ts.TypeReferenceNode | undefined;
 export declare const isImportSpecifier: (node: ts.Node) => boolean;
 export declare const isReferencedInExport: (node: ts.Node) => boolean;
 export declare const getExportKeywordNode: (node: ts.Node) => ts.ExportKeyword | undefined;

package/dist/typescript/ast-helpers.js

@@ -150,12 +150,10 @@ export const isObjectEnumerationCallExpressionArgument = (node) => ts.isCallExpr
     objectEnumerationMethods.has(String(node.parent.expression.name.escapedText));
 export const isInForIteration = (node) => node.parent && (ts.isForInStatement(node.parent) || ts.isForOfStatement(node.parent));
 export const isTopLevel = (node) => ts.isSourceFile(node.parent) || (node.parent && ts.isSourceFile(node.parent.parent));
-export const getTypeName = (node) => {
+export const getTypeRef = (node) => {
     if (!node.parent?.parent)
         return;
-    const typeRef = findAncestor(node, _node => ts.isTypeReferenceNode(_node));
-    if (typeRef && ts.isQualifiedName(typeRef.typeName))
-        return typeRef.typeName;
+    return findAncestor(node, _node => ts.isTypeReferenceNode(_node));
 };
 export const isImportSpecifier = (node) => ts.isImportSpecifier(node.parent) ||
     ts.isImportEqualsDeclaration(node.parent) ||

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, isInForIteration, isObjectEnumerationCallExpressionArgument, isReferencedInExport, } from './ast-helpers.js';
+import { getAccessMembers, getDestructuredIds, getJSDocTags, getLineAndCharacterOfPosition, getTypeRef, 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';
@@ -265,23 +265,27 @@ const getImportsAndExports = (sourceFile, resolveModule, typeChecker, options) =
                                 addNsMemberRefs(imports, id, members);
                             }
                             else {
-                                const typeName = getTypeName(node);
-                                if (typeName) {
-                                    const [ns, ...right] = [typeName.left.getText(), typeName.right.getText()].join('.').split('.');
-                                    const members = right.map((_r, index) => right.slice(0, index + 1).join('.'));
-                                    addNsMemberRefs(imports, ns, members);
-                                }
-                                else if (imports.importedNs.has(id)) {
-                                    if (isConsiderReferencedNS(node)) {
-                                        imports.refs.add(id);
-                                    }
-                                    else if (isObjectEnumerationCallExpressionArgument(node)) {
-                                        imports.refs.add(id);
+                                const typeRef = getTypeRef(node);
+                                if (typeRef) {
+                                    if (ts.isQualifiedName(typeRef.typeName)) {
+                                        const typeName = typeRef.typeName;
+                                        const [ns, ...right] = [typeName.left.getText(), typeName.right.getText()].join('.').split('.');
+                                        const members = right.map((_r, index) => right.slice(0, index + 1).join('.'));
+                                        addNsMemberRefs(imports, ns, members);
                                     }
-                                    else if (isInForIteration(node)) {
+                                    else {
                                         imports.refs.add(id);
                                     }
                                 }
+                                else if (imports.importedNs.has(id) && isConsiderReferencedNS(node)) {
+                                    imports.refs.add(id);
+                                }
+                                else if (isObjectEnumerationCallExpressionArgument(node)) {
+                                    imports.refs.add(id);
+                                }
+                                else if (isInForIteration(node)) {
+                                    imports.refs.add(id);
+                                }
                             }
                         }
                     }

package/dist/util/has-strictly-ns-references.d.ts

@@ -1,3 +1,4 @@
 import type { DependencyGraph, ImportDetails } from '../types/dependency-graph.js';
-export declare const getHasStrictlyNsReferences: (graph: DependencyGraph, importsForExport: ImportDetails | undefined, id: string) => [boolean, string?];
+export declare const hasStrictlyEnumReferences: (importsForExport: ImportDetails | undefined, id: string) => boolean;
+export declare const hasStrictlyNsReferences: (graph: DependencyGraph, importsForExport: ImportDetails | undefined, id: string) => [boolean, string?];
 export declare const getType: (hasOnlyNsReference: boolean, isType: boolean) => "exports" | "types" | "nsExports" | "nsTypes";

package/dist/util/has-strictly-ns-references.js

@@ -1,5 +1,13 @@
 import { IMPORT_STAR } from '../constants.js';
-export const getHasStrictlyNsReferences = (graph, importsForExport, id) => {
+export const hasStrictlyEnumReferences = (importsForExport, id) => {
+    if (!importsForExport || !importsForExport.refs.has(id))
+        return false;
+    for (const ref of importsForExport.refs)
+        if (ref.startsWith(`${id}.`))
+            return false;
+    return true;
+};
+export const hasStrictlyNsReferences = (graph, importsForExport, id) => {
     if (!importsForExport)
         return [false];
     let namespace;
@@ -15,9 +23,9 @@ export const getHasStrictlyNsReferences = (graph, importsForExport, id) => {
             for (const filePath of byFilePaths) {
                 const file = graph.get(filePath);
                 if (file?.imported) {
-                    const hasStrictlyNsReferences = getHasStrictlyNsReferences(graph, file.imported, id);
-                    if (hasStrictlyNsReferences[0] === false)
-                        return hasStrictlyNsReferences;
+                    const hasStrictlyNsRefs = hasStrictlyNsReferences(graph, file.imported, id);
+                    if (hasStrictlyNsRefs[0] === false)
+                        return hasStrictlyNsRefs;
                 }
             }
         }
@@ -27,9 +35,9 @@ export const getHasStrictlyNsReferences = (graph, importsForExport, id) => {
                 for (const filePath of byFilePaths) {
                     const file = graph.get(filePath);
                     if (file?.imported) {
-                        const hasStrictlyNsReferences = getHasStrictlyNsReferences(graph, file.imported, id);
-                        if (hasStrictlyNsReferences[0] === false)
-                            return hasStrictlyNsReferences;
+                        const hasStrictlyNsRefs = hasStrictlyNsReferences(graph, file.imported, id);
+                        if (hasStrictlyNsRefs[0] === false)
+                            return hasStrictlyNsRefs;
                     }
                 }
             }
@@ -41,10 +49,10 @@ export const getHasStrictlyNsReferences = (graph, importsForExport, id) => {
         for (const filePath of byFilePaths) {
             const file = graph.get(filePath);
             if (file?.imported) {
-                const hasStrictlyNsReferences = getHasStrictlyNsReferences(graph, file.imported, id);
-                if (hasStrictlyNsReferences[0] === false)
-                    return hasStrictlyNsReferences;
-                namespace = hasStrictlyNsReferences[1];
+                const hasStrictlyNsRefs = hasStrictlyNsReferences(graph, file.imported, id);
+                if (hasStrictlyNsRefs[0] === false)
+                    return hasStrictlyNsRefs;
+                namespace = hasStrictlyNsRefs[1];
             }
         }
     }
@@ -54,10 +62,10 @@ export const getHasStrictlyNsReferences = (graph, importsForExport, id) => {
             for (const filePath of byFilePaths) {
                 const file = graph.get(filePath);
                 if (file?.imported) {
-                    const hasStrictlyNsReferences = getHasStrictlyNsReferences(graph, file.imported, id);
-                    if (hasStrictlyNsReferences[0] === false)
-                        return hasStrictlyNsReferences;
-                    namespace = hasStrictlyNsReferences[1];
+                    const hasStrictlyNsRefs = hasStrictlyNsReferences(graph, file.imported, id);
+                    if (hasStrictlyNsRefs[0] === false)
+                        return hasStrictlyNsRefs;
+                    namespace = hasStrictlyNsRefs[1];
                 }
             }
         }
@@ -69,10 +77,10 @@ export const getHasStrictlyNsReferences = (graph, importsForExport, id) => {
             for (const filePath of filePaths) {
                 const file = graph.get(filePath);
                 if (file?.imported) {
-                    const hasStrictlyNsReferences = getHasStrictlyNsReferences(graph, file.imported, [alias, ...rest].join('.'));
-                    if (hasStrictlyNsReferences[0] === false)
-                        return hasStrictlyNsReferences;
-                    namespace = hasStrictlyNsReferences[1];
+                    const hasStrictlyNsRefs = hasStrictlyNsReferences(graph, file.imported, [alias, ...rest].join('.'));
+                    if (hasStrictlyNsRefs[0] === false)
+                        return hasStrictlyNsRefs;
+                    namespace = hasStrictlyNsRefs[1];
                 }
             }
         }
@@ -81,10 +89,10 @@ export const getHasStrictlyNsReferences = (graph, importsForExport, id) => {
         for (const filePath of filePaths) {
             const file = graph.get(filePath);
             if (file?.imported) {
-                const hasStrictlyNsReferences = getHasStrictlyNsReferences(graph, file.imported, `${ns}.${id}`);
-                if (hasStrictlyNsReferences[0] === false)
-                    return hasStrictlyNsReferences;
-                namespace = hasStrictlyNsReferences[1];
+                const hasStrictlyNsRefs = hasStrictlyNsReferences(graph, file.imported, `${ns}.${id}`);
+                if (hasStrictlyNsRefs[0] === false)
+                    return hasStrictlyNsRefs;
+                namespace = hasStrictlyNsRefs[1];
             }
         }
     }

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "5.43.2";
+export declare const version = "5.43.4";

package/dist/version.js

@@ -1 +1 @@
-export const version = '5.43.2';
+export const version = '5.43.4';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "5.43.2",
+  "version": "5.43.4",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://knip.dev",
   "repository": {