knip 2.27.1 → 2.29.0

package/README.md

@@ -253,6 +253,7 @@ Knip contains a growing list of plugins:
 - [commitlint][plugin-commitlint]
 - [cspell][plugin-cspell]
 - [Cypress][plugin-cypress]
+- [Drizzle][plugin-drizzle]
 - [ESLint][plugin-eslint]
 - [Gatsby][plugin-gatsby]
 - [GitHub Actions][plugin-github-actions]
@@ -322,6 +323,21 @@ has them at `e2e-tests/*.spec.ts`. Here's how to configure this:
 }
 ```
 
+When overriding any plugin's configuration, the options object is fully replaced. Look at the plugin's page to find the
+default configuration and use that as a basis. Here's another example to override the test file pattern for Vitest:
+
+```json
+{
+  "vitest": {
+    "config": ["vitest.config.ts"],
+    "entry": ["**/*.vitest.ts"]
+  }
+}
+```
+
+This plugin configuration can be set on root and workspace level. If set on root level, it can be disabled on workspace
+level by setting it to `false` there.
+
 #### Multi-project repositories
 
 Some repositories have a single `package.json`, but consist of multiple projects with configuration files across the
@@ -875,6 +891,7 @@ Special thanks to the wonderful people who have contributed to this project:
 [plugin-commitlint]: ./src/plugins/commitlint
 [plugin-cspell]: ./src/plugins/cspell
 [plugin-cypress]: ./src/plugins/cypress
+[plugin-drizzle]: ./src/plugins/drizzle
 [plugin-eslint]: ./src/plugins/eslint
 [plugin-gatsby]: ./src/plugins/gatsby
 [plugin-github-actions]: ./src/plugins/github-actions

package/dist/ProjectPrincipal.js

@@ -6,7 +6,7 @@ import { getJSDocTags, isInModuleBlock } from './typescript/ast-helpers.js';
 import { createHosts } from './typescript/createHosts.js';
 import { getImportsAndExports } from './typescript/getImportsAndExports.js';
 import { SourceFileManager } from './typescript/SourceFileManager.js';
-import { isMaybePackageName } from './util/modules.js';
+import { isMaybePackageName, sanitizeSpecifier } from './util/modules.js';
 import { dirname, extname, isInNodeModules, join } from './util/path.js';
 import { timerify } from './util/Performance.js';
 const baseCompilerOptions = {
@@ -128,7 +128,7 @@ export class ProjectPrincipal {
                 }
             }
             else {
-                const sanitizedSpecifier = specifier.replace(/^([?!|-]+)?([^!?]+).*/, '$2');
+                const sanitizedSpecifier = sanitizeSpecifier(specifier);
                 if (isMaybePackageName(sanitizedSpecifier)) {
                     external.add(sanitizedSpecifier);
                 }

package/dist/WorkspaceWorker.d.ts

@@ -50,7 +50,7 @@ export declare class WorkspaceWorker {
         hostDependencies: HostDependencies;
         installedBinaries: InstalledBinaries;
         referencedDependencies: ReferencedDependencies;
-        enabledPlugins: ("angular" | "ava" | "babel" | "capacitor" | "changesets" | "commitizen" | "commitlint" | "cspell" | "cypress" | "eslint" | "gatsby" | "husky" | "jest" | "lefthook" | "markdownlint" | "mocha" | "next" | "nx" | "nyc" | "playwright" | "postcss" | "prettier" | "remark" | "remix" | "rollup" | "sentry" | "storybook" | "stryker" | "stylelint" | "tailwind" | "typedoc" | "typescript" | "vite" | "vitest" | "webpack" | "githubActions" | "lintStaged" | "npmPackageJsonLint" | "playwrightCt" | "releaseIt" | "semanticRelease" | "svelte")[];
+        enabledPlugins: ("angular" | "ava" | "babel" | "capacitor" | "changesets" | "commitizen" | "commitlint" | "cspell" | "cypress" | "eslint" | "gatsby" | "husky" | "jest" | "lefthook" | "markdownlint" | "mocha" | "next" | "nx" | "nyc" | "playwright" | "postcss" | "prettier" | "remark" | "remix" | "rollup" | "sentry" | "storybook" | "stryker" | "stylelint" | "tailwind" | "typedoc" | "typescript" | "vite" | "vitest" | "webpack" | "drizzle" | "githubActions" | "lintStaged" | "npmPackageJsonLint" | "playwrightCt" | "releaseIt" | "semanticRelease" | "svelte")[];
     }>;
 }
 export {};

package/dist/index.js

@@ -17,7 +17,7 @@ import { getEntryPathFromManifest, getPackageNameFromFilePath, getPackageNameFro
 import { dirname, isInNodeModules, join, isInternal, toAbsolute } from './util/path.js';
 import { _resolveSpecifier, _tryResolve } from './util/require.js';
 import { _require } from './util/require.js';
-import { loadTSConfig as loadCompilerOptions } from './util/tsconfig-loader.js';
+import { loadTSConfig } from './util/tsconfig-loader.js';
 import { WorkspaceWorker } from './WorkspaceWorker.js';
 export const main = async (unresolvedConfiguration) => {
     const { cwd, tsConfigFile, gitignore, isStrict, isProduction, isIgnoreInternal, isShowProgress, isIncludeEntryExports, } = unresolvedConfiguration;
@@ -92,7 +92,7 @@ export const main = async (unresolvedConfiguration) => {
         if (!manifestPath || !manifest)
             throw new LoaderError(`Unable to load package.json for ${name}`);
         deputy.addWorkspace({ name, dir, manifestPath, manifest, ignoreDependencies, ignoreBinaries });
-        const compilerOptions = await loadCompilerOptions(join(dir, tsConfigFile ?? 'tsconfig.json'));
+        const { compilerOptions, definitionPaths } = await loadTSConfig(join(dir, tsConfigFile ?? 'tsconfig.json'));
         const principal = factory.getPrincipal({ cwd: dir, paths, compilerOptions, compilers, pkgName });
         const worker = new WorkspaceWorker({
             name,
@@ -107,6 +107,8 @@ export const main = async (unresolvedConfiguration) => {
             enabledPluginsInAncestors: ancestors.flatMap(ancestor => enabledPluginsStore.get(ancestor) ?? []),
         });
         await worker.init();
+        principal.addEntryPaths(definitionPaths);
+        debugLogArray(`Found definition paths (${name})`, definitionPaths);
         const sharedGlobOptions = { cwd, workingDir: dir, gitignore, ignore: worker.getIgnorePatterns() };
         const entryPathsFromManifest = await getEntryPathFromManifest(cwd, dir, manifest);
         debugLogArray(`Found entry paths in package.json (${name})`, entryPathsFromManifest);

package/dist/plugins/drizzle/index.d.ts

@@ -0,0 +1,6 @@
+import type { GenericPluginCallback, IsPluginEnabledCallback } from '../../types/plugins.js';
+export declare const NAME = "Drizzle";
+export declare const ENABLERS: string[];
+export declare const isEnabled: IsPluginEnabledCallback;
+export declare const CONFIG_FILE_PATTERNS: string[];
+export declare const findDependencies: GenericPluginCallback;

package/dist/plugins/drizzle/index.js

@@ -0,0 +1,16 @@
+import { _glob } from '../../util/glob.js';
+import { timerify } from '../../util/Performance.js';
+import { hasDependency, load } from '../../util/plugin.js';
+export const NAME = 'Drizzle';
+export const ENABLERS = ['drizzle-kit'];
+export const isEnabled = ({ dependencies }) => hasDependency(dependencies, ENABLERS);
+export const CONFIG_FILE_PATTERNS = ['drizzle.config.{ts,js,json}'];
+const findDrizzleDependencies = async (configFilePath, { cwd }) => {
+    const config = await load(configFilePath);
+    if (!config || !config.schema)
+        return [];
+    const patterns = Array.isArray(config.schema) ? config.schema : [config.schema];
+    const paths = await _glob({ cwd, patterns });
+    return paths;
+};
+export const findDependencies = timerify(findDrizzleDependencies);

package/dist/plugins/drizzle/types.d.ts

@@ -0,0 +1,3 @@
+export interface DrizzleConfig {
+    schema?: string | string[];
+}

package/dist/plugins/drizzle/types.js

@@ -0,0 +1 @@
+export {};

package/dist/plugins/index.d.ts

@@ -7,6 +7,7 @@ export * as commitizen from './commitizen/index.js';
 export * as commitlint from './commitlint/index.js';
 export * as cspell from './cspell/index.js';
 export * as cypress from './cypress/index.js';
+export * as drizzle from './drizzle/index.js';
 export * as eslint from './eslint/index.js';
 export * as gatsby from './gatsby/index.js';
 export * as githubActions from './github-actions/index.js';

package/dist/plugins/index.js

@@ -7,6 +7,7 @@ export * as commitizen from './commitizen/index.js';
 export * as commitlint from './commitlint/index.js';
 export * as cspell from './cspell/index.js';
 export * as cypress from './cypress/index.js';
+export * as drizzle from './drizzle/index.js';
 export * as eslint from './eslint/index.js';
 export * as gatsby from './gatsby/index.js';
 export * as githubActions from './github-actions/index.js';

package/dist/plugins/jest/index.js

@@ -40,6 +40,7 @@ const resolveDependencies = (config) => {
     const moduleNameMapper = (config.moduleNameMapper
         ? Object.values(config.moduleNameMapper).map(mapper => (typeof mapper === 'string' ? mapper : mapper[0]))
         : []).filter(value => !/\$[0-9]/.test(value));
+    const testResultsProcessor = config.testResultsProcessor ? [config.testResultsProcessor] : [];
     return [
         ...presets,
         ...projects,
@@ -52,6 +53,7 @@ const resolveDependencies = (config) => {
         ...setupFilesAfterEnv,
         ...transform,
         ...moduleNameMapper,
+        ...testResultsProcessor,
     ];
 };
 const findJestDependencies = async (configFilePath, { cwd, manifest }) => {

package/dist/plugins/next/index.js

@@ -9,7 +9,7 @@ const productionEntryFilePatternsWithoutSrc = [
     'app/**/{error,layout,loading,not-found,page,template}.{js,jsx,ts,tsx}',
     'instrumentation.{js,ts}',
     'app/{manifest,sitemap,robots}.{js,ts}',
-    'app/**/{icon,apple-icon}-image.{js,ts,tsx}',
+    'app/**/{icon,apple-icon}.{js,ts,tsx}',
     'app/**/{opengraph,twitter}-image.{js,ts,tsx}',
     'pages/**/*.{js,jsx,ts,tsx}',
 ];

package/dist/plugins/typescript/index.js

@@ -22,7 +22,7 @@ const resolveExtensibleConfig = async (configFilePath) => {
     return config;
 };
 export const findTypeScriptDependencies = async (configFilePath) => {
-    const compilerOptions = await loadTSConfig(configFilePath);
+    const { compilerOptions } = await loadTSConfig(configFilePath);
     const config = await resolveExtensibleConfig(configFilePath);
     if (!compilerOptions || !config)
         return [];

package/dist/plugins/vitest/index.js

@@ -5,7 +5,7 @@ import { getEnvPackageName, getExternalReporters } from './helpers.js';
 export const NAME = 'Vitest';
 export const ENABLERS = ['vitest'];
 export const isEnabled = ({ dependencies }) => hasDependency(dependencies, ENABLERS);
-export const CONFIG_FILE_PATTERNS = ['vitest.config.ts'];
+export const CONFIG_FILE_PATTERNS = ['vitest.config.ts', 'vitest.{workspace,projects}.{ts,js,json}'];
 export const ENTRY_FILE_PATTERNS = [];
 export const findVitestDeps = (config) => {
     if (!config || !config.test)
@@ -18,8 +18,20 @@ export const findVitestDeps = (config) => {
     const globalSetup = cfg.globalSetup ? [cfg.globalSetup].flat() : [];
     return compact([...environments, ...reporters, ...coverage, ...setupFiles, ...globalSetup]);
 };
+const findVitestWorkspaceDeps = (config) => {
+    let deps = [];
+    for (const cfg of config) {
+        if (typeof cfg === 'string')
+            continue;
+        deps = [...deps, ...findVitestDeps(cfg)];
+    }
+    return compact(deps);
+};
 const findVitestDependencies = async (configFilePath) => {
     const config = await load(configFilePath);
+    if (Array.isArray(config)) {
+        return findVitestWorkspaceDeps(config);
+    }
     return findVitestDeps(config);
 };
 export const findDependencies = timerify(findVitestDependencies);

package/dist/plugins/vitest/types.d.ts

@@ -9,3 +9,4 @@ export interface VitestConfig {
         setupFiles?: string | string[];
     };
 }
+export type VitestWorkspaceConfig = (string | VitestConfig)[];

package/dist/typescript/getImportsAndExports.js

@@ -1,9 +1,7 @@
-import { existsSync } from 'node:fs';
 import { isBuiltin } from 'node:module';
-import { resolve, dirname } from 'node:path';
 import ts from 'typescript';
 import { getOrSet } from '../util/map.js';
-import { isMaybePackageName } from '../util/modules.js';
+import { isMaybePackageName, sanitizeSpecifier } from '../util/modules.js';
 import { isInNodeModules } from '../util/path.js';
 import { isDeclarationFileExtension, isAccessExpression, getAccessExpressionName, getJSDocTags, } from './ast-helpers.js';
 import getExportVisitors from './visitors/exports/index.js';
@@ -60,13 +58,14 @@ export const getImportsAndExports = (sourceFile, options) => {
                     if (!isInNodeModules(filePath)) {
                         addInternalImport({ identifier, specifier, symbol, filePath, isReExport });
                     }
-                    if (!isMaybePackageName(specifier))
+                    const sanitizedSpecifier = sanitizeSpecifier(specifier);
+                    if (!isMaybePackageName(sanitizedSpecifier))
                         return;
                     if (isDeclarationFileExtension(module.resolvedModule.extension)) {
-                        externalImports.add(specifier);
+                        externalImports.add(sanitizedSpecifier);
                     }
                     else {
-                        externalImports.add(module.resolvedModule.packageId?.name ?? specifier);
+                        externalImports.add(module.resolvedModule.packageId?.name ?? sanitizedSpecifier);
                     }
                 }
                 else {
@@ -75,13 +74,7 @@ export const getImportsAndExports = (sourceFile, options) => {
             }
         }
         else {
-            const filePath = resolve(dirname(sourceFile.fileName), specifier);
-            if (existsSync(filePath)) {
-                unresolvedImports.add(filePath);
-            }
-            else {
-                unresolvedImports.add(specifier);
-            }
+            unresolvedImports.add(specifier);
         }
     };
     const maybeAddNamespaceAccessAsImport = ({ namespace, member }) => {

package/dist/typescript/resolveModuleNames.js

@@ -1,11 +1,29 @@
+import { existsSync } from 'node:fs';
 import ts from 'typescript';
+import { sanitizeSpecifier } from '../util/modules.js';
+import { dirname, extname, isInternal, join } from '../util/path.js';
 import { ensureRealFilePath, isVirtualFilePath } from './utils.js';
 export function createCustomModuleResolver(customSys, compilerOptions, virtualFileExtensions) {
     function resolveModuleNames(moduleNames, containingFile) {
         return moduleNames.map(moduleName => resolveModuleName(moduleName, containingFile));
     }
     function resolveModuleName(name, containingFile) {
-        const tsResolvedModule = ts.resolveModuleName(name, containingFile, compilerOptions, ts.sys).resolvedModule;
+        const sanitizedSpecifier = sanitizeSpecifier(name);
+        const tsResolvedModule = ts.resolveModuleName(sanitizedSpecifier, containingFile, compilerOptions, ts.sys).resolvedModule;
+        if (!tsResolvedModule) {
+            const extension = extname(sanitizedSpecifier);
+            if (extension && isInternal(sanitizedSpecifier) && !virtualFileExtensions.includes(extension)) {
+                const resolvedFileName = join(dirname(containingFile), sanitizedSpecifier);
+                if (existsSync(resolvedFileName)) {
+                    return {
+                        resolvedFileName,
+                        extension,
+                        isExternalLibraryImport: false,
+                        resolvedUsingTsExtension: false,
+                    };
+                }
+            }
+        }
         if (virtualFileExtensions.length === 0)
             return tsResolvedModule;
         if (tsResolvedModule && !isVirtualFilePath(tsResolvedModule.resolvedFileName, virtualFileExtensions)) {

package/dist/util/modules.d.ts

@@ -6,3 +6,4 @@ export declare const isDefinitelyTyped: (packageName: string) => boolean;
 export declare const getDefinitelyTypedFor: (packageName: string) => string;
 export declare const getPackageFromDefinitelyTyped: (typedDependency: string) => string;
 export declare const getEntryPathFromManifest: (cwd: string, dir: string, manifest: PackageJson) => Promise<string[]>;
+export declare const sanitizeSpecifier: (specifier: string) => string;

package/dist/util/modules.js

@@ -50,3 +50,4 @@ export const getEntryPathFromManifest = (cwd, dir, manifest) => {
     }
     return _glob({ cwd, workingDir: dir, patterns: Array.from(entryPaths) });
 };
+export const sanitizeSpecifier = (specifier) => specifier.replace(/^([?!|-]+)?([^!?]+).*/, '$2');

package/dist/util/tsconfig-loader.d.ts

@@ -1,2 +1,5 @@
 import ts from 'typescript';
-export declare const loadTSConfig: (tsConfigFilePath: string) => Promise<ts.CompilerOptions>;
+export declare const loadTSConfig: (tsConfigFilePath: string) => Promise<{
+    compilerOptions: ts.CompilerOptions;
+    definitionPaths: string[];
+}>;

package/dist/util/tsconfig-loader.js

@@ -5,7 +5,9 @@ export const loadTSConfig = async (tsConfigFilePath) => {
     if (isFile(tsConfigFilePath)) {
         const config = ts.readConfigFile(tsConfigFilePath, ts.sys.readFile);
         const parsedConfig = ts.parseJsonConfigFileContent(config.config, ts.sys, dirname(tsConfigFilePath));
-        return parsedConfig.options ?? {};
+        const compilerOptions = parsedConfig.options ?? {};
+        const definitionPaths = parsedConfig.fileNames.filter(filePath => filePath.endsWith('.d.ts'));
+        return { compilerOptions, definitionPaths };
     }
-    return {};
+    return { compilerOptions: {}, definitionPaths: [] };
 };

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "2.27.1";
+export declare const version = "2.29.0";

package/dist/version.js

@@ -1 +1 @@
-export const version = '2.27.1';
+export const version = '2.29.0';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "2.27.1",
+  "version": "2.29.0",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://github.com/webpro/knip",
   "repository": "github:webpro/knip",
@@ -65,16 +65,16 @@
     "@npmcli/package-json": "5.0.0",
     "@release-it/bumper": "5.1.0",
     "@swc/cli": "0.1.62",
-    "@swc/core": "1.3.88",
+    "@swc/core": "1.3.89",
     "@types/eslint": "8.44.3",
     "@types/js-yaml": "4.0.6",
     "@types/micromatch": "4.0.2",
     "@types/minimist": "1.2.2",
-    "@types/node": "20.6.4",
+    "@types/node": "20.6.5",
     "@types/npmcli__map-workspaces": "3.0.2",
-    "@types/webpack": "5.28.2",
-    "@typescript-eslint/eslint-plugin": "6.7.2",
-    "@typescript-eslint/parser": "6.7.2",
+    "@types/webpack": "5.28.3",
+    "@typescript-eslint/eslint-plugin": "6.7.3",
+    "@typescript-eslint/parser": "6.7.3",
     "c8": "8.0.1",
     "eslint": "8.50.0",
     "eslint-import-resolver-typescript": "3.6.1",

package/schema.json

@@ -256,6 +256,10 @@
           "title": "Cypress plugin configuration (https://github.com/webpro/knip/blob/main/src/plugins/cypress/README.md)",
           "$ref": "#/definitions/plugin"
         },
+        "drizzle": {
+          "title": "Drizzle plugin configuration (https://github.com/webpro/knip/blob/main/src/plugins/drizzle/README.md)",
+          "$ref": "#/definitions/plugin"
+        },
         "eslint": {
           "title": "ESLint plugin configuration (https://github.com/webpro/knip/blob/main/src/plugins/eslint/README.md)",
           "$ref": "#/definitions/plugin"
@@ -312,6 +316,10 @@
           "title": "Playwright plugin configuration (https://github.com/webpro/knip/blob/main/src/plugins/playwright/README.md)",
           "$ref": "#/definitions/plugin"
         },
+        "playwright-ct": {
+          "title": "Playwright for components plugin configuration (https://github.com/webpro/knip/blob/main/src/plugins/playwright-ct/README.md)",
+          "$ref": "#/definitions/plugin"
+        },
         "postcss": {
           "title": "PostCSS plugin configuration (https://github.com/webpro/knip/blob/main/src/plugins/postcss/README.md)",
           "$ref": "#/definitions/plugin"