knip 1.12.2 → 1.12.4

package/dist/configuration-validator.d.ts

@@ -1,22 +1,19 @@
 import { z } from 'zod';
-export declare const ConfigurationValidator: z.ZodObject<z.extendShape<z.extendShape<z.extendShape<{
+export declare const ConfigurationValidator: z.ZodObject<{
+    exclude: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
+    ignore: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
+    include: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
     entry: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
     project: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
     paths: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodArray<z.ZodString, "many">>>;
-    ignore: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
     ignoreBinaries: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
     ignoreDependencies: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
     ignoreWorkspaces: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
-}, {
-    include: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
-    exclude: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
-}>, {
-    workspaces: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodObject<z.extendShape<{
+    workspaces: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodObject<{
+        ignore: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
         entry: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
         project: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
         paths: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodArray<z.ZodString, "many">>>;
-        ignore: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
-    }, {
         babel: z.ZodOptional<z.ZodUnion<[z.ZodLiteral<false>, z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>, z.ZodObject<{
             config: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
             entry: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
@@ -407,7 +404,7 @@ export declare const ConfigurationValidator: z.ZodObject<z.extendShape<z.extendS
             entry?: string | string[] | undefined;
             project?: string | string[] | undefined;
         }>]>>;
-    }>, "strip", z.ZodTypeAny, {
+    }, "strip", z.ZodTypeAny, {
         ignore?: string | string[] | undefined;
         entry?: string | string[] | undefined;
         project?: string | string[] | undefined;
@@ -718,7 +715,6 @@ export declare const ConfigurationValidator: z.ZodObject<z.extendShape<z.extendS
             project?: string | string[] | undefined;
         } | undefined;
     }>>>;
-}>, {
     babel: z.ZodOptional<z.ZodUnion<[z.ZodLiteral<false>, z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>, z.ZodObject<{
         config: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
         entry: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
@@ -1109,7 +1105,7 @@ export declare const ConfigurationValidator: z.ZodObject<z.extendShape<z.extendS
         entry?: string | string[] | undefined;
         project?: string | string[] | undefined;
     }>]>>;
-}>, "strip", z.ZodTypeAny, {
+}, "strip", z.ZodTypeAny, {
     exclude?: string[] | undefined;
     ignore?: string | string[] | undefined;
     include?: string[] | undefined;

package/dist/index.js

@@ -186,16 +186,16 @@ export const main = async (unresolvedConfiguration) => {
             if (report.dependencies || report.unlisted || report.files) {
                 await worker.findDependenciesByPlugins();
                 const { referencedDependencyIssues, peerDependencies, installedBinaries, entryFiles, enabledPlugins } = worker.getFinalDependencies();
+                deputy.addPeerDependencies(name, peerDependencies);
+                deputy.setInstalledBinaries(name, installedBinaries);
+                principal.addEntryPaths(entryFiles);
+                enabledPluginsStore.set(name, enabledPlugins);
                 referencedDependencyIssues.forEach(issue => {
                     const workspace = { name, dir, config, ancestors };
                     const unlistedDependency = deputy.maybeAddListedReferencedDependency(workspace, issue.symbol);
                     if (unlistedDependency)
                         collector.addIssue(issue);
                 });
-                deputy.addPeerDependencies(name, peerDependencies);
-                deputy.setInstalledBinaries(name, installedBinaries);
-                principal.addEntryPaths(entryFiles);
-                enabledPluginsStore.set(name, enabledPlugins);
             }
         }
     }

package/dist/plugins/eslint/fallback.d.ts

@@ -0,0 +1,5 @@
+type Options = {
+    cwd: string;
+};
+export declare const fallback: (configFilePath: string, { cwd }: Options) => Promise<string[]>;
+export {};

package/dist/plugins/eslint/fallback.js

@@ -0,0 +1,20 @@
+import { ESLint } from 'eslint';
+import { compact } from '../../util/array.js';
+import { getPackageName } from '../../util/modules.js';
+import { resolvePluginPackageName, getDependenciesFromSettings } from './helpers.js';
+export const fallback = async (configFilePath, { cwd }) => {
+    const engine = new ESLint({ cwd, overrideConfigFile: configFilePath, useEslintrc: false });
+    const jsConfig = await engine.calculateConfigForFile('__placeholder__.js');
+    const tsConfig = await engine.calculateConfigForFile('__placeholder__.ts');
+    const tsxConfig = await engine.calculateConfigForFile('__placeholder__.tsx');
+    const dependencies = [jsConfig, tsConfig, tsxConfig].map(config => {
+        if (!config)
+            return [];
+        const plugins = config.plugins?.map(resolvePluginPackageName) ?? [];
+        const parsers = config.parser ? [config.parser] : [];
+        const extraParsers = config.parserOptions?.babelOptions?.presets ?? [];
+        const settings = config.settings ? getDependenciesFromSettings(config.settings) : [];
+        return [...parsers, ...extraParsers, ...plugins, ...settings].map(getPackageName);
+    });
+    return compact(dependencies.flat());
+};

package/dist/plugins/eslint/helpers.d.ts

@@ -1,4 +1,4 @@
 import type { ESLintConfig } from './types.js';
+export declare const getDependenciesDeep: (configFilePath: string, dependencies?: Set<string>) => Promise<Set<string>>;
 export declare const resolvePluginPackageName: (pluginName: string) => string;
-export declare const customResolvePluginPackageNames: (extend: string) => string | undefined;
 export declare const getDependenciesFromSettings: (settings?: ESLintConfig['settings']) => string[];

package/dist/plugins/eslint/helpers.js

@@ -1,5 +1,35 @@
+import path from 'node:path';
 import { compact } from '../../util/array.js';
+import { _load } from '../../util/loader.js';
 import { getPackageName } from '../../util/modules.js';
+import { require } from '../../util/require.js';
+const getDependencies = (config) => {
+    const extend = config.extends ? [config.extends].flat().map(customResolvePluginPackageNames) : [];
+    if (extend.includes('eslint-plugin-prettier'))
+        extend.push('eslint-config-prettier');
+    const plugins = config.plugins ? config.plugins.map(resolvePluginPackageName) : [];
+    const parser = config.parser;
+    const extraParsers = config.parserOptions?.babelOptions?.presets ?? [];
+    const settings = config.settings ? getDependenciesFromSettings(config.settings) : [];
+    return compact([...extend, ...plugins, parser, ...extraParsers, ...settings]).map(getPackageName);
+};
+export const getDependenciesDeep = async (configFilePath, dependencies = new Set()) => {
+    const addAll = (deps) => deps.forEach(dependency => dependencies.add(dependency));
+    const config = await _load(configFilePath);
+    if (config.extends) {
+        for (const extend of [config.extends].flat()) {
+            if (extend.startsWith('.') || (extend.startsWith('/') && !extend.includes('/node_modules/'))) {
+                const extendConfigFilePath = require.resolve(path.join(path.dirname(configFilePath), extend));
+                addAll(await getDependenciesDeep(extendConfigFilePath));
+            }
+        }
+    }
+    if (config.overrides)
+        for (const override of config.overrides)
+            addAll(getDependencies(override));
+    addAll(getDependencies(config));
+    return dependencies;
+};
 const resolvePackageName = (namespace, pluginName) => {
     return pluginName.startsWith('@')
         ? pluginName.includes('/')
@@ -8,7 +38,7 @@ const resolvePackageName = (namespace, pluginName) => {
         : `${namespace}-${pluginName}`;
 };
 export const resolvePluginPackageName = (pluginName) => resolvePackageName('eslint-plugin', pluginName);
-export const customResolvePluginPackageNames = (extend) => {
+const customResolvePluginPackageNames = (extend) => {
     if (extend.includes('/node_modules/'))
         return getPackageName(extend);
     if (extend.startsWith('/') || extend.startsWith('.'))

package/dist/plugins/eslint/index.js

@@ -1,51 +1,29 @@
-import { ESLint } from 'eslint';
-import { compact } from '../../util/array.js';
-import { debugLogFiles, debugLogObject } from '../../util/debug.js';
-import { _firstGlob } from '../../util/glob.js';
 import { _load } from '../../util/loader.js';
-import { getPackageName } from '../../util/modules.js';
 import { timerify } from '../../util/performance.js';
 import { hasDependency } from '../../util/plugin.js';
-import { resolvePluginPackageName, customResolvePluginPackageNames, getDependenciesFromSettings } from './helpers.js';
+import { fallback } from './fallback.js';
+import { getDependenciesDeep } from './helpers.js';
 export const NAME = 'ESLint';
 export const ENABLERS = ['eslint'];
 export const isEnabled = ({ dependencies }) => hasDependency(dependencies, ENABLERS);
 export const CONFIG_FILE_PATTERNS = ['.eslintrc', '.eslintrc.{js,json,cjs}', '.eslintrc.{yml,yaml}', 'package.json'];
 export const ENTRY_FILE_PATTERNS = ['eslint.config.js'];
-const findESLintDependencies = async (configFilePath, { cwd, manifest, workspaceConfig }) => {
-    if (configFilePath.endsWith('package.json') && !manifest.eslintConfig)
-        return [];
+const findESLintDependencies = async (configFilePath, { cwd, manifest }) => {
     let config = undefined;
     try {
         config = configFilePath.endsWith('package.json') ? manifest.eslintConfig : await _load(configFilePath);
     }
     catch (error) {
-        if (error instanceof Error && error.cause instanceof Error && !/Failed to patch ESLint/.test(error.cause.message)) {
+        if (error instanceof Error && error.cause instanceof Error && /Failed to patch ESLint/.test(error.cause.message)) {
+            return fallback(configFilePath, { cwd });
+        }
+        else {
             throw error;
         }
     }
-    const rootExtends = config?.extends ? [config.extends].flat().map(customResolvePluginPackageNames) : [];
-    if (rootExtends.includes('eslint-plugin-prettier'))
-        rootExtends.push('eslint-config-prettier');
-    const patterns = compact([
-        workspaceConfig.entry,
-        ...(config?.overrides?.map(overrides => [overrides.files].flat()) ?? []),
-    ]);
-    debugLogObject('ESLint overrides file patterns', patterns);
-    const samples = await Promise.all(patterns.map(patterns => _firstGlob({ patterns, cwd })));
-    const sampleFilePaths = compact(samples.filter((filePath) => typeof filePath !== 'undefined').map(String));
-    debugLogFiles('ESLint overrides sample files', sampleFilePaths);
-    const engine = new ESLint({ cwd, overrideConfigFile: configFilePath, useEslintrc: false });
-    const calculateConfigForFile = async (sampleFile) => await engine.calculateConfigForFile(sampleFile);
-    const dependencies = await Promise.all(sampleFilePaths.map(calculateConfigForFile)).then(configs => configs.flatMap(config => {
-        if (!config)
-            return [];
-        const plugins = config.plugins?.map(resolvePluginPackageName) ?? [];
-        const parsers = config.parser ? [config.parser] : [];
-        const extraParsers = config.parserOptions?.babelOptions?.presets ?? [];
-        const settings = config.settings ? getDependenciesFromSettings(config.settings) : [];
-        return [...parsers, ...extraParsers, ...plugins, ...settings].map(getPackageName);
-    }));
-    return compact([...rootExtends, ...dependencies.flat()]);
+    if (!config)
+        return [];
+    const dependencies = await getDependenciesDeep(configFilePath);
+    return Array.from(dependencies);
 };
 export const findDependencies = timerify(findESLintDependencies);

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

@@ -1,14 +1,25 @@
-export type ESLintConfig = {
+type ParserOptions = {
+    project?: string;
+    babelOptions?: {
+        presets: string[];
+    };
+};
+type Settings = Record<string, Record<string, unknown>>;
+type Rules = Record<string, string | number>;
+type BaseConfig = {
     extends?: string | string[];
     parser?: string;
-    parserOptions?: {
-        babelOptions?: {
-            presets: string[];
-        };
-    };
+    parserOptions?: ParserOptions;
+    processor?: string;
     plugins?: string[];
-    settings?: Record<string, Record<string, unknown>>;
-    overrides?: {
-        files: string[];
-    }[];
+    settings?: Settings;
+    rules?: Rules;
+};
+type OverrideConfig = BaseConfig & {
+    files: string[];
+};
+export type ESLintConfig = BaseConfig & {
+    env?: Record<string, boolean>;
+    overrides?: OverrideConfig[];
 };
+export {};

package/dist/plugins/stryker/index.js

@@ -4,7 +4,7 @@ import { hasDependency } from '../../util/plugin.js';
 export const NAME = 'Stryker';
 export const ENABLERS = ['@stryker-mutator/core'];
 export const isEnabled = ({ dependencies }) => hasDependency(dependencies, ENABLERS);
-export const CONFIG_FILE_PATTERNS = ['?(.)stryker.{conf,config}.{js,mjs,json}'];
+export const CONFIG_FILE_PATTERNS = ['?(.)stryker.{conf,config}.{js,mjs,cjs,json}'];
 const findStrykerDependencies = async (configFilePath) => {
     const config = await _load(configFilePath);
     if (config) {

package/dist/types/plugins.d.ts

@@ -1,4 +1,4 @@
-import { Configuration, PluginConfiguration, WorkspaceConfiguration } from './config.js';
+import type { Configuration, PluginConfiguration, WorkspaceConfiguration } from './config.js';
 import type { PackageJson } from 'type-fest';
 type IsPluginEnabledCallbackOptions = {
     cwd: string;

package/dist/util/binaries/index.d.ts

@@ -1,2 +1,2 @@
-import type { GetBinariesFromScripts } from './types.js';
-export declare const _getReferencesFromScripts: GetBinariesFromScripts;
+import type { GetReferencesFromScripts } from './types.js';
+export declare const _getReferencesFromScripts: GetReferencesFromScripts;

package/dist/util/binaries/types.d.ts

@@ -5,7 +5,7 @@ type Options = {
     ignore?: string[];
     knownGlobalsOnly?: boolean;
 };
-export type GetBinariesFromScripts = (npmScripts: string | string[], options?: Options) => {
+export type GetReferencesFromScripts = (npmScripts: string | string[], options?: Options) => {
     entryFiles: string[];
     binaries: string[];
 };

package/dist/util/loader.js

@@ -1,5 +1,6 @@
 import fs from 'node:fs/promises';
 import path from 'node:path';
+import { pathToFileURL } from 'node:url';
 import { load as esmLoad } from '@esbuild-kit/esm-loader';
 import yaml from 'js-yaml';
 import { require } from '../util/require.js';
@@ -8,12 +9,18 @@ import { loadJSON } from './fs.js';
 import { timerify } from './performance.js';
 const load = async (filePath) => {
     try {
-        if (/\.jsonc?$/.test(filePath) || /rc$/.test(filePath)) {
+        const ext = path.extname(filePath);
+        if (ext === '.json' || ext === '.jsonc' || /rc$/.test(filePath)) {
             return loadJSON(filePath);
         }
-        if (path.extname(filePath) === '.yaml' || path.extname(filePath) === '.yml') {
+        if (ext === '.yaml' || ext === '.yml') {
             return yaml.load((await fs.readFile(filePath)).toString());
         }
+        if (ext === '.mjs') {
+            const fileUrl = pathToFileURL(filePath);
+            const imported = await import(fileUrl.href);
+            return imported.default ?? imported;
+        }
         const imported = await esmLoad(filePath, {}, require);
         return imported.default ?? imported;
     }

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "1.12.2";
+export declare const version = "1.12.4";

package/dist/version.js

@@ -1 +1 @@
-export const version = '1.12.2';
+export const version = '1.12.4';

package/dist/workspace-worker.js

@@ -43,11 +43,10 @@ export default class WorkspaceWorker {
         const dependencies = new Set([Object.keys(manifest?.dependencies ?? {}), Object.keys(manifest?.devDependencies ?? {})].flat());
         const pluginEntries = Object.entries(plugins);
         for (const [pluginName, plugin] of pluginEntries) {
-            const hasIsEnabled = typeof plugin.isEnabled === 'function';
+            const isEnabled = this.config[pluginName] !== false;
+            const isEnabledInAncestor = enabledPluginsInAncestors.includes(pluginName);
             this.enabled[pluginName] =
-                this.config[pluginName] !== false &&
-                    (enabledPluginsInAncestors.includes(pluginName) ||
-                        (hasIsEnabled && (await plugin.isEnabled({ cwd: this.dir, manifest: this.manifest, dependencies }))));
+                isEnabled && (isEnabledInAncestor || (await plugin.isEnabled({ cwd: this.dir, manifest, dependencies })));
         }
         this.enabledPlugins = pluginEntries.filter(([name]) => this.enabled[name]).map(([name]) => name);
         const enabledPluginNames = this.enabledPlugins.map(name => plugins[name].NAME);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "1.12.2",
+  "version": "1.12.4",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://github.com/webpro/knip",
   "repository": "github:webpro/knip",
@@ -39,21 +39,21 @@
     "schema.json"
   ],
   "dependencies": {
-    "@esbuild-kit/esm-loader": "^2.5.4",
-    "@npmcli/map-workspaces": "^3.0.1",
+    "@esbuild-kit/esm-loader": "^2.5.5",
+    "@npmcli/map-workspaces": "^3.0.2",
     "@snyk/github-codeowners": "^1.1.0",
     "bash-parser": "^0.5.0",
     "chalk": "^5.2.0",
     "easy-table": "^1.2.0",
-    "esbuild": "^0.17.4",
+    "esbuild": "^0.17.10",
     "esbuild-register": "3.4.2",
-    "eslint": "^8.32.0",
+    "eslint": "^8.34.0",
     "fast-glob": "^3.2.12",
-    "get-tsconfig": "^4.3.0",
+    "get-tsconfig": "^4.4.0",
     "globby": "^13.1.3",
     "js-yaml": "^4.1.0",
     "micromatch": "^4.0.5",
-    "minimist": "^1.2.7",
+    "minimist": "^1.2.8",
     "nano-memoize": "^2.0.0",
     "patch-package": "^6.5.1",
     "pretty-ms": "^8.0.0",
@@ -61,31 +61,31 @@
     "summary": "^2.1.0",
     "ts-morph": "^17.0.1",
     "ts-morph-helpers": "^0.6.3",
-    "zod": "^3.20.2"
+    "zod": "^3.20.6"
   },
   "devDependencies": {
-    "@jest/types": "^29.4.0",
+    "@jest/types": "29.4.3",
     "@npmcli/package-json": "3.0.0",
-    "@release-it/bumper": "^4.0.2",
-    "@types/eslint": "8.4.10",
+    "@release-it/bumper": "4.0.2",
+    "@types/eslint": "8.21.1",
     "@types/js-yaml": "4.0.5",
     "@types/micromatch": "4.0.2",
-    "@types/minimist": "^1.2.2",
-    "@types/node": "18.11.18",
+    "@types/minimist": "1.2.2",
+    "@types/node": "18.14.0",
     "@types/npmcli__map-workspaces": "3.0.0",
     "@types/webpack": "5.28.0",
-    "@typescript-eslint/eslint-plugin": "^5.49.0",
-    "@typescript-eslint/parser": "^5.49.0",
+    "@typescript-eslint/eslint-plugin": "5.53.0",
+    "@typescript-eslint/parser": "5.53.0",
     "eslint-import-resolver-typescript": "3.5.3",
     "eslint-plugin-import": "2.27.5",
     "globstar": "1.0.0",
-    "prettier": "2.8.3",
+    "prettier": "2.8.4",
     "release-it": "15.6.0",
     "remark-cli": "11.0.0",
     "remark-preset-webpro": "0.0.1",
-    "tsx": "3.12.2",
-    "type-fest": "^3.5.3",
-    "typescript": "4.9.4"
+    "tsx": "3.12.3",
+    "type-fest": "3.6.0",
+    "typescript": "4.9.5"
   },
   "engines": {
     "node": ">=16.17.0 <17 || >=18.6.0"