knip 5.30.3 → 5.30.5

package/dist/DependencyDeputy.js

@@ -175,11 +175,11 @@ export class DependencyDeputy {
                     return false;
                 const [scope, typedDependency] = dependency.split('/');
                 if (scope === DT_SCOPE) {
-                    if (hasTypesIncluded?.has(typedDependency))
-                        return false;
                     const typedPackageName = getPackageFromDefinitelyTyped(typedDependency);
-                    if (IGNORE_DEFINITELY_TYPED.includes(typedPackageName))
+                    if (IGNORE_DEFINITELY_TYPED.has(typedPackageName))
                         return true;
+                    if (hasTypesIncluded?.has(typedDependency))
+                        return false;
                     const hostDependencies = [
                         ...this.getHostDependenciesFor(workspace, dependency),
                         ...this.getHostDependenciesFor(workspace, typedPackageName),

package/dist/WorkspaceWorker.d.ts

@@ -34,7 +34,7 @@ export declare class WorkspaceWorker {
     isStrict: boolean;
     rootIgnore: Configuration['ignore'];
     negatedWorkspacePatterns: string[];
-    enabledPluginsMap: Record<"astro" | "angular" | "ava" | "babel" | "capacitor" | "changesets" | "commitizen" | "commitlint" | "cspell" | "cucumber" | "cypress" | "eleventy" | "eslint" | "gatsby" | "husky" | "jest" | "ladle" | "lefthook" | "linthtml" | "markdownlint" | "mocha" | "moonrepo" | "msw" | "nest" | "netlify" | "next" | "nuxt" | "nx" | "nyc" | "oclif" | "playwright" | "postcss" | "preconstruct" | "prettier" | "remark" | "remix" | "rollup" | "rsbuild" | "rspack" | "sentry" | "storybook" | "stryker" | "stylelint" | "svelte" | "syncpack" | "tailwind" | "tsup" | "typedoc" | "typescript" | "unbuild" | "unocss" | "vue" | "vike" | "vite" | "vitest" | "webpack" | "wireit" | "wrangler" | "xo" | "yorkie" | "drizzle" | "githubActions" | "graphqlCodegen" | "lintStaged" | "lockfileLint" | "lostPixel" | "nodeTestRunner" | "npmPackageJsonLint" | "playwrightCt" | "reactCosmos" | "releaseIt" | "semanticRelease" | "simpleGitHooks" | "sizeLimit" | "vercelOg" | "webdriverIo", boolean>;
+    enabledPluginsMap: Record<"angular" | "astro" | "ava" | "babel" | "capacitor" | "changesets" | "commitizen" | "commitlint" | "cspell" | "cucumber" | "cypress" | "drizzle" | "eleventy" | "eslint" | "gatsby" | "githubActions" | "graphqlCodegen" | "husky" | "jest" | "ladle" | "lefthook" | "linthtml" | "lintStaged" | "lockfileLint" | "lostPixel" | "markdownlint" | "mocha" | "moonrepo" | "msw" | "nest" | "netlify" | "next" | "nodeTestRunner" | "npmPackageJsonLint" | "nuxt" | "nx" | "nyc" | "oclif" | "playwright" | "playwrightCt" | "postcss" | "preconstruct" | "prettier" | "reactCosmos" | "releaseIt" | "remark" | "remix" | "rollup" | "rsbuild" | "rspack" | "semanticRelease" | "sentry" | "simpleGitHooks" | "sizeLimit" | "storybook" | "stryker" | "stylelint" | "svelte" | "syncpack" | "tailwind" | "tsup" | "typedoc" | "typescript" | "unbuild" | "unocss" | "vercelOg" | "vike" | "vite" | "vitest" | "vue" | "webdriverIo" | "webpack" | "wireit" | "wrangler" | "xo" | "yorkie", boolean>;
     enabledPlugins: PluginName[];
     enabledPluginsInAncestors: string[];
     cache: CacheConsultant<CacheItem>;
@@ -55,7 +55,7 @@ export declare class WorkspaceWorker {
         entryFilePatterns: Set<string>;
         productionEntryFilePatterns: Set<string>;
         referencedDependencies: ReferencedDependencies;
-        enabledPlugins: ("astro" | "angular" | "ava" | "babel" | "capacitor" | "changesets" | "commitizen" | "commitlint" | "cspell" | "cucumber" | "cypress" | "eleventy" | "eslint" | "gatsby" | "husky" | "jest" | "ladle" | "lefthook" | "linthtml" | "markdownlint" | "mocha" | "moonrepo" | "msw" | "nest" | "netlify" | "next" | "nuxt" | "nx" | "nyc" | "oclif" | "playwright" | "postcss" | "preconstruct" | "prettier" | "remark" | "remix" | "rollup" | "rsbuild" | "rspack" | "sentry" | "storybook" | "stryker" | "stylelint" | "svelte" | "syncpack" | "tailwind" | "tsup" | "typedoc" | "typescript" | "unbuild" | "unocss" | "vue" | "vike" | "vite" | "vitest" | "webpack" | "wireit" | "wrangler" | "xo" | "yorkie" | "drizzle" | "githubActions" | "graphqlCodegen" | "lintStaged" | "lockfileLint" | "lostPixel" | "nodeTestRunner" | "npmPackageJsonLint" | "playwrightCt" | "reactCosmos" | "releaseIt" | "semanticRelease" | "simpleGitHooks" | "sizeLimit" | "vercelOg" | "webdriverIo")[];
+        enabledPlugins: ("angular" | "astro" | "ava" | "babel" | "capacitor" | "changesets" | "commitizen" | "commitlint" | "cspell" | "cucumber" | "cypress" | "drizzle" | "eleventy" | "eslint" | "gatsby" | "githubActions" | "graphqlCodegen" | "husky" | "jest" | "ladle" | "lefthook" | "linthtml" | "lintStaged" | "lockfileLint" | "lostPixel" | "markdownlint" | "mocha" | "moonrepo" | "msw" | "nest" | "netlify" | "next" | "nodeTestRunner" | "npmPackageJsonLint" | "nuxt" | "nx" | "nyc" | "oclif" | "playwright" | "playwrightCt" | "postcss" | "preconstruct" | "prettier" | "reactCosmos" | "releaseIt" | "remark" | "remix" | "rollup" | "rsbuild" | "rspack" | "semanticRelease" | "sentry" | "simpleGitHooks" | "sizeLimit" | "storybook" | "stryker" | "stylelint" | "svelte" | "syncpack" | "tailwind" | "tsup" | "typedoc" | "typescript" | "unbuild" | "unocss" | "vercelOg" | "vike" | "vite" | "vitest" | "vue" | "webdriverIo" | "webpack" | "wireit" | "wrangler" | "xo" | "yorkie")[];
     }>;
     onDispose(): void;
 }

package/dist/constants.d.ts

@@ -14,6 +14,6 @@ export declare const IGNORED_GLOBAL_BINARIES: Set<string>;
 export declare const IGNORED_DEPENDENCIES: Set<string>;
 export declare const IGNORED_RUNTIME_DEPENDENCIES: Set<string>;
 export declare const FOREIGN_FILE_EXTENSIONS: Set<string>;
-export declare const IGNORE_DEFINITELY_TYPED: string[];
+export declare const IGNORE_DEFINITELY_TYPED: Set<string>;
 export declare const ISSUE_TYPES: IssueType[];
 export declare const ISSUE_TYPE_TITLE: Record<IssueType, string>;

package/dist/constants.js

@@ -142,7 +142,11 @@ export const FOREIGN_FILE_EXTENSIONS = new Set([
     '.yaml',
     '.yml',
 ]);
-export const IGNORE_DEFINITELY_TYPED = ['node', 'bun'];
+export const IGNORE_DEFINITELY_TYPED = new Set([
+    'node',
+    'bun',
+    'jest',
+]);
 export const ISSUE_TYPES = [
     'files',
     'dependencies',

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.3";
+export declare const version = "5.30.5";

package/dist/version.js

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

package/package.json

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