knip 5.30.4 → 5.30.5

package/dist/plugins/playwright/index.js

@@ -6,10 +6,11 @@ const enablers = ['@playwright/test'];
 const isEnabled = ({ dependencies }) => hasDependency(dependencies, enablers);
 const config = ['playwright.config.{js,ts}'];
 export const entry = ['**/*.@(spec|test).?(c|m)[jt]s?(x)'];
-const toEntryPatterns = (testMatch, cwd, configDir, localConfig) => {
+const toEntryPatterns = (testMatch, cwd, configDir, localConfig, rootConfig) => {
     if (!testMatch)
         return [];
-    const dir = relative(cwd, localConfig.testDir ? join(configDir, localConfig.testDir) : configDir);
+    const testDir = localConfig.testDir ?? rootConfig.testDir;
+    const dir = relative(cwd, testDir ? join(configDir, testDir) : configDir);
     const patterns = [testMatch].flat().filter((p) => typeof p === 'string');
     return patterns.map(pattern => toEntryPattern(join(dir, pattern)));
 };
@@ -17,7 +18,7 @@ const builtinReporters = ['dot', 'line', 'list', 'junit', 'html', 'blob', 'json'
 export const resolveEntryPaths = async (localConfig, options) => {
     const { cwd, configFileDir } = options;
     const projects = localConfig.projects ? [localConfig, ...localConfig.projects] : [localConfig];
-    return projects.flatMap(config => toEntryPatterns(config.testMatch, cwd, configFileDir, config));
+    return projects.flatMap(config => toEntryPatterns(config.testMatch, cwd, configFileDir, config, localConfig));
 };
 export const resolveConfig = async (config) => {
     const reporters = [config.reporter].flat().flatMap(reporter => {

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

@@ -26,3 +26,5 @@ export declare const isImportSpecifier: (node: ts.Node) => boolean;
 export declare const isReferencedInExportedType: (node: ts.Node) => boolean;
 export declare const getExportKeywordNode: (node: ts.Node) => ts.ExportKeyword | undefined;
 export declare const getDefaultKeywordNode: (node: ts.Node) => ts.DefaultKeyword | undefined;
+export declare const hasRequireCall: (node: ts.Node) => boolean;
+export declare const isModuleExportsAccess: (node: ts.PropertyAccessExpression) => boolean;

package/dist/typescript/ast-helpers.js

@@ -170,3 +170,9 @@ export const isReferencedInExportedType = (node) => {
 };
 export const getExportKeywordNode = (node) => node.modifiers?.find(mod => mod.kind === ts.SyntaxKind.ExportKeyword);
 export const getDefaultKeywordNode = (node) => node.modifiers?.find(mod => mod.kind === ts.SyntaxKind.DefaultKeyword);
+export const hasRequireCall = (node) => {
+    if (ts.isCallExpression(node) && ts.isIdentifier(node.expression) && node.expression.text === 'require')
+        return true;
+    return node.getChildren().some(child => hasRequireCall(child));
+};
+export const isModuleExportsAccess = (node) => ts.isIdentifier(node.expression) && node.expression.escapedText === 'module' && node.name.escapedText === 'exports';

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

@@ -102,9 +102,7 @@ const getImportsAndExports = (sourceFile, resolveModule, typeChecker, options) =
                 if (module.isExternalLibraryImport) {
                     if (skipTypeOnly && isTypeOnly)
                         return;
-                    const sanitizedSpecifier = isInNodeModules(specifier)
-                        ? getPackageNameFromFilePath(specifier)
-                        : sanitizeSpecifier(specifier);
+                    const sanitizedSpecifier = sanitizeSpecifier(isInNodeModules(specifier) || isInNodeModules(filePath) ? getPackageNameFromFilePath(specifier) : specifier);
                     if (!isStartsLikePackageName(sanitizedSpecifier)) {
                         return;
                     }

package/dist/typescript/visitors/dynamic-imports/requireCall.js

@@ -1,5 +1,6 @@
 import ts from 'typescript';
-import { findAncestor, findDescendants, isRequireCall, isTopLevel } from '../../ast-helpers.js';
+import { IMPORT_STAR } from '../../../constants.js';
+import { findAncestor, findDescendants, isModuleExportsAccess, isRequireCall, isTopLevel } from '../../ast-helpers.js';
 import { isNotJS } from '../helpers.js';
 import { importVisitor as visit } from '../index.js';
 export default visit(() => true, node => {
@@ -43,6 +44,11 @@ export default visit(() => true, node => {
                     }
                     return { identifier: 'default', specifier, pos: node.arguments[0].pos, resolve };
                 }
+                if (ts.isBinaryExpression(node.parent) &&
+                    ts.isPropertyAccessExpression(node.parent.left) &&
+                    isModuleExportsAccess(node.parent.left)) {
+                    return { identifier: IMPORT_STAR, specifier, isReExport: true, pos: node.arguments[0].pos };
+                }
                 return { identifier: 'default', specifier, pos: node.arguments[0].pos, resolve };
             }
         }

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

@@ -1,9 +1,8 @@
 import ts from 'typescript';
 import { SymbolType } from '../../../types/issues.js';
-import { stripQuotes } from '../../ast-helpers.js';
+import { hasRequireCall, isModuleExportsAccess, stripQuotes } from '../../ast-helpers.js';
 import { isJS } from '../helpers.js';
 import { exportVisitor as visit } from '../index.js';
-const isModuleExportsAccess = (node) => ts.isIdentifier(node.expression) && node.expression.escapedText === 'module' && node.name.escapedText === 'exports';
 export default visit(isJS, (node, { isFixExports }) => {
     if (ts.isExpressionStatement(node)) {
         if (ts.isBinaryExpression(node.expression)) {
@@ -29,6 +28,9 @@ export default visit(isJS, (node, { isFixExports }) => {
                             return { node, identifier: node.getText(), type: SymbolType.UNKNOWN, pos: node.getStart(), fix };
                         });
                     }
+                    if (ts.isCallExpression(node.expression.right) && hasRequireCall(node.expression.right)) {
+                        return;
+                    }
                     return { node, identifier: 'default', type: SymbolType.UNKNOWN, pos: expr.pos + 1, fix: undefined };
                 }
             }

package/dist/version.d.ts

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

package/dist/version.js

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

package/package.json

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