knip 2.17.2 → 2.18.0

package/README.md

@@ -143,7 +143,7 @@ Using workspaces in a monorepo? Please see [workspaces][1] for more details abou
       --dependencies           Shortcut for --include dependencies,unlisted,unresolved
       --exports                Shortcut for --include exports,nsExports,classMembers,types,nsTypes,enumMembers,duplicates
       --include-entry-exports  Include entry files when reporting unused exports
-      -n, --no-progress        Don't show dynamic progress updates
+      -n, --no-progress        Don't show dynamic progress updates (this is automatically enabled in CI environments)
       --reporter               Select reporter: symbols, compact, codeowners, json (default: symbols)
       --reporter-options       Pass extra options to the reporter (as JSON string, see example)
       --no-config-hints        Suppress configuration hints

package/dist/cli.js

@@ -39,6 +39,7 @@ const run = async () => {
             noConfigHints,
             cwd,
             isProduction,
+            isShowProgress,
             options: reporterOptions,
         });
         const totalErrorCount = Object.keys(report)

package/dist/constants.js

@@ -48,6 +48,8 @@ export const IGNORED_FILE_EXTENSIONS = [
     '.webp',
     '.woff',
     '.woff2',
+    '.yaml',
+    '.yml',
 ];
 export const IGNORE_DEFINITELY_TYPED = ['node'];
 export const ISSUE_TYPES = [

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

@@ -9,4 +9,5 @@ type GetDependenciesDeep = (configFilePath: string, dependencies: Set<string>, o
     manifest: Manifest;
 }) => Promise<Set<string>>;
 export declare const getDependenciesDeep: GetDependenciesDeep;
+export declare const resolveExtendsSpecifier: (specifier: string) => string | undefined;
 export {};

package/dist/plugins/eslint/helpers.js

@@ -56,14 +56,15 @@ const resolvePackageName = (namespace, pluginName) => {
             : `${namespace}-${pluginName}`;
 };
 const resolvePluginPackageName = (pluginName) => resolvePackageName('eslint-plugin', pluginName);
-const resolveExtendsSpecifier = (specifier) => {
+export const resolveExtendsSpecifier = (specifier) => {
     if (isInternal(specifier))
         return;
     if (/\/eslint-(config|plugin)/.test(specifier))
         return specifier;
-    const strippedSpecifier = specifier
-        .replace(/(^plugin:|:.+$)/, '')
-        .replace(/\/(eslint-)?(recommended.*|stylistic.*|strict.*|all)$/, '');
+    const noProtocolSpecifier = specifier.replace(/(^plugin:|:.+$)/, '');
+    if (noProtocolSpecifier.startsWith('@typescript-eslint/'))
+        return '@typescript-eslint/eslint-plugin';
+    const strippedSpecifier = noProtocolSpecifier.replace(/\/(eslint-)?(recommended|all)$/, '');
     if (/eslint-(config|plugin)-/.test(strippedSpecifier))
         return strippedSpecifier;
     const packageName = getPackageNameFromModuleSpecifier(strippedSpecifier);

package/dist/plugins/jest/index.js

@@ -17,16 +17,12 @@ const resolveExtensibleConfig = async (configFilePath) => {
     }
     return config;
 };
-const findJestDependencies = async (configFilePath, { cwd, manifest }) => {
-    let config = configFilePath.endsWith('package.json')
-        ? manifest.jest
-        : await resolveExtensibleConfig(configFilePath);
-    if (typeof config === 'function')
-        config = await config();
-    if (!config)
-        return [];
-    const replaceRootDir = (name) => name.includes('<rootDir>') ? join(cwd, name.replace(/^.*<rootDir>/, '')) : name;
+const resolveDependencies = (config) => {
     const presets = (config.preset ? [config.preset] : []).map(preset => isInternal(preset) ? preset : join(preset, 'jest-preset'));
+    const projects = Array.isArray(config.projects)
+        ? config.projects.map(config => (typeof config === 'string' ? config : resolveDependencies(config))).flat()
+        : [];
+    const runner = config.runner ? [config.runner] : [];
     const environments = config.testEnvironment === 'jsdom' ? ['jest-environment-jsdom'] : [];
     const resolvers = config.resolver ? [config.resolver] : [];
     const reporters = config.reporters
@@ -45,6 +41,8 @@ const findJestDependencies = async (configFilePath, { cwd, manifest }) => {
         : []).filter(value => !/\$[0-9]/.test(value));
     return [
         ...presets,
+        ...projects,
+        ...runner,
         ...environments,
         ...resolvers,
         ...reporters,
@@ -53,6 +51,17 @@ const findJestDependencies = async (configFilePath, { cwd, manifest }) => {
         ...setupFilesAfterEnv,
         ...transform,
         ...moduleNameMapper,
-    ].map(replaceRootDir);
+    ];
+};
+const findJestDependencies = async (configFilePath, { cwd, manifest }) => {
+    let config = configFilePath.endsWith('package.json')
+        ? manifest.jest
+        : await resolveExtensibleConfig(configFilePath);
+    if (typeof config === 'function')
+        config = await config();
+    if (!config)
+        return [];
+    const replaceRootDir = (name) => name.includes('<rootDir>') ? join(cwd, name.replace(/^.*<rootDir>/, '')) : name;
+    return resolveDependencies(config).map(replaceRootDir);
 };
 export const findDependencies = timerify(findJestDependencies);

package/dist/plugins/typescript/index.js

@@ -1,18 +1,34 @@
 import { compact } from '../../util/array.js';
-import { isInternal } from '../../util/path.js';
+import { dirname, isInternal, toAbsolute } from '../../util/path.js';
 import { timerify } from '../../util/Performance.js';
 import { hasDependency, load } from '../../util/plugin.js';
 export const NAME = 'TypeScript';
 export const ENABLERS = ['typescript'];
 export const isEnabled = ({ dependencies }) => hasDependency(dependencies, ENABLERS);
 export const CONFIG_FILE_PATTERNS = ['tsconfig.json'];
-const findTypeScriptDependencies = async (configFilePath) => {
+const resolveExtensibleConfig = async (configFilePath) => {
     const config = await load(configFilePath);
+    if (config?.extends) {
+        if (isInternal(config.extends)) {
+            const presetConfigPath = toAbsolute(config.extends, dirname(configFilePath));
+            const presetConfig = await resolveExtensibleConfig(presetConfigPath);
+            Object.assign(config, presetConfig);
+        }
+    }
+    return config;
+};
+const findTypeScriptDependencies = async (configFilePath) => {
+    const config = await resolveExtensibleConfig(configFilePath);
     if (!config)
         return [];
     const extend = config.extends ? [config.extends].flat().filter(extend => !isInternal(extend)) : [];
     const plugins = compact(config.compilerOptions?.plugins?.map(plugin => plugin.name) ?? []);
     const importHelpers = config.compilerOptions?.importHelpers ? ['tslib'] : [];
-    return [...extend, ...plugins, ...importHelpers];
+    const jsx = config.compilerOptions?.jsxImportSource
+        ? [config.compilerOptions.jsxImportSource]
+        : config.compilerOptions?.jsx
+            ? ['react']
+            : [];
+    return [...extend, ...plugins, ...importHelpers, ...jsx];
 };
 export const findDependencies = timerify(findTypeScriptDependencies);

package/dist/reporters/codeowners.d.ts

@@ -1,3 +1,3 @@
 import type { ReporterOptions } from '../types/issues.js';
-declare const _default: ({ report, issues, options }: ReporterOptions) => void;
+declare const _default: ({ report, issues, isShowProgress, options }: ReporterOptions) => void;
 export default _default;

package/dist/reporters/codeowners.js

@@ -11,7 +11,7 @@ const logIssueRecord = (issues) => {
     const sortedByFilePath = issues.sort((a, b) => (a.owner < b.owner ? -1 : 1));
     sortedByFilePath.forEach(({ filePath, symbols, owner, parentSymbol }) => logIssueLine({ owner, filePath, symbols, parentSymbol }));
 };
-export default ({ report, issues, options }) => {
+export default ({ report, issues, isShowProgress, options }) => {
     let opts = {};
     try {
         opts = options ? JSON.parse(options) : opts;
@@ -56,7 +56,7 @@ export default ({ report, issues, options }) => {
             totalIssues = totalIssues + issuesForType.length;
         }
     }
-    if (totalIssues === 0) {
+    if (totalIssues === 0 && isShowProgress) {
         console.log('✂️  Excellent, Knip found no issues.');
     }
 };

package/dist/reporters/compact.d.ts

@@ -1,3 +1,3 @@
 import type { ReporterOptions } from '../types/issues.js';
-declare const _default: ({ report, issues }: ReporterOptions) => void;
+declare const _default: ({ report, issues, isShowProgress }: ReporterOptions) => void;
 export default _default;

package/dist/reporters/compact.js

@@ -3,7 +3,7 @@ const logIssueRecord = (issues) => {
     const sortedByFilePath = issues.sort((a, b) => (a.filePath > b.filePath ? 1 : -1));
     sortedByFilePath.forEach(logIssueLine);
 };
-export default ({ report, issues }) => {
+export default ({ report, issues, isShowProgress }) => {
     const reportMultipleGroups = Object.values(report).filter(Boolean).length > 1;
     let totalIssues = 0;
     for (const [reportType, isReportType] of Object.entries(report)) {
@@ -30,7 +30,7 @@ export default ({ report, issues }) => {
             totalIssues = totalIssues + issuesForType.length;
         }
     }
-    if (totalIssues === 0) {
+    if (totalIssues === 0 && isShowProgress) {
         console.log('✂️  Excellent, Knip found no issues.');
     }
 };

package/dist/reporters/index.d.ts

@@ -1,7 +1,7 @@
 declare const _default: {
-    symbols: ({ report, issues, configurationHints, noConfigHints }: import("../index.js").ReporterOptions) => void;
-    compact: ({ report, issues }: import("../index.js").ReporterOptions) => void;
-    codeowners: ({ report, issues, options }: import("../index.js").ReporterOptions) => void;
+    symbols: ({ report, issues, configurationHints, noConfigHints, isShowProgress }: import("../index.js").ReporterOptions) => void;
+    compact: ({ report, issues, isShowProgress }: import("../index.js").ReporterOptions) => void;
+    codeowners: ({ report, issues, isShowProgress, options }: import("../index.js").ReporterOptions) => void;
     json: ({ report, issues, options }: import("../index.js").ReporterOptions) => Promise<void>;
 };
 export default _default;

package/dist/reporters/symbols.d.ts

@@ -1,3 +1,3 @@
 import type { ReporterOptions } from '../types/issues.js';
-declare const _default: ({ report, issues, configurationHints, noConfigHints }: ReporterOptions) => void;
+declare const _default: ({ report, issues, configurationHints, noConfigHints, isShowProgress }: ReporterOptions) => void;
 export default _default;

package/dist/reporters/symbols.js

@@ -17,7 +17,7 @@ const logIssueRecord = (issues) => {
     });
     console.log(table.sort(['filePath', 'parentSymbol', 'symbol']).print().trim());
 };
-export default ({ report, issues, configurationHints, noConfigHints }) => {
+export default ({ report, issues, configurationHints, noConfigHints, isShowProgress }) => {
     const reportMultipleGroups = Object.values(report).filter(Boolean).length > 1;
     let totalIssues = 0;
     for (const [reportType, isReportType] of Object.entries(report)) {
@@ -48,7 +48,7 @@ export default ({ report, issues, configurationHints, noConfigHints }) => {
             console.warn(chalk.grey(`${message}${workspace}:`), identifier);
         });
     }
-    if (totalIssues === 0) {
+    if (totalIssues === 0 && isShowProgress) {
         console.log('✂️  Excellent, Knip found no issues.');
     }
 };

package/dist/types/issues.d.ts

@@ -47,6 +47,7 @@ export type ReporterOptions = {
     cwd: string;
     workingDir: string;
     isProduction: boolean;
+    isShowProgress: boolean;
     options: string;
 };
 export type Reporter = (options: ReporterOptions) => void;

package/dist/typescript/SourceFile.d.ts

@@ -3,10 +3,16 @@ type SymbolTable = Map<string, ts.Symbol>;
 type SymbolWithExports = ts.Symbol & {
     exports?: SymbolTable;
 };
+type PragmaMap = {
+    arguments: {
+        factory: string;
+    };
+};
 export interface BoundSourceFile extends ts.SourceFile {
     symbol?: SymbolWithExports;
     resolvedModules?: ts.ModeAwareCache<ts.ResolvedModuleWithFailedLookupLocations>;
     locals?: SymbolTable;
     scriptKind?: ts.ScriptKind;
+    pragmas?: Map<string, PragmaMap | PragmaMap[]>;
 }
 export {};

package/dist/typescript/SourceFileManager.js

@@ -1,6 +1,6 @@
 import ts from 'typescript';
 import { debugLog } from '../util/debug.js';
-import { extname } from '../util/path.js';
+import { extname, isInternal } from '../util/path.js';
 export class SourceFileManager {
     sourceFileCache = new Map();
     snapshotCache = new Map();
@@ -11,7 +11,8 @@ export class SourceFileManager {
         this.asyncCompilers = compilers[1];
     }
     createSourceFile(filePath, contents) {
-        const sourceFile = ts.createSourceFile(filePath, contents, ts.ScriptTarget.Latest, true);
+        const setParentNodes = isInternal(filePath);
+        const sourceFile = ts.createSourceFile(filePath, contents, ts.ScriptTarget.Latest, setParentNodes);
         this.sourceFileCache.set(filePath, sourceFile);
         return sourceFile;
     }

package/dist/typescript/getImportsAndExports.js

@@ -5,6 +5,7 @@ import { isMaybePackageName } from '../util/modules.js';
 import { isInNodeModules } from '../util/path.js';
 import { isDeclarationFileExtension, isAccessExpression, getAccessExpressionName } from './ast-helpers.js';
 import getExportVisitors from './visitors/exports/index.js';
+import { getJSXImplicitImportBase } from './visitors/helpers.js';
 import getImportVisitors from './visitors/imports/index.js';
 import getScriptVisitors from './visitors/scripts/index.js';
 const getVisitors = (sourceFile) => ({
@@ -20,6 +21,9 @@ export const getImportsAndExports = (sourceFile, options) => {
     const aliasedExports = {};
     const scripts = new Set();
     const importedInternalSymbols = new Map();
+    const jsxImport = getJSXImplicitImportBase(sourceFile);
+    if (jsxImport)
+        externalImports.add(jsxImport);
     const visitors = getVisitors(sourceFile);
     const addInternalImport = (options) => {
         const { identifier, specifier, symbol, filePath, isReExport } = options;

package/dist/typescript/visitors/helpers.d.ts

@@ -1,3 +1,4 @@
 import { BoundSourceFile } from '../SourceFile.js';
 export declare const isNotJS: (sourceFile: BoundSourceFile) => boolean;
 export declare const isJS: (sourceFile: BoundSourceFile) => boolean;
+export declare function getJSXImplicitImportBase(sourceFile: BoundSourceFile): string | undefined;

package/dist/typescript/visitors/helpers.js

@@ -1,3 +1,10 @@
 import ts from 'typescript';
 export const isNotJS = (sourceFile) => sourceFile.scriptKind !== ts.ScriptKind.JS && sourceFile.scriptKind !== ts.ScriptKind.JSX;
 export const isJS = (sourceFile) => sourceFile.scriptKind === ts.ScriptKind.JS || sourceFile.scriptKind === ts.ScriptKind.JSX;
+export function getJSXImplicitImportBase(sourceFile) {
+    const jsxImportSourcePragmas = sourceFile.pragmas?.get('jsximportsource');
+    const jsxImportSourcePragma = Array.isArray(jsxImportSourcePragmas)
+        ? jsxImportSourcePragmas[jsxImportSourcePragmas.length - 1]
+        : jsxImportSourcePragmas;
+    return jsxImportSourcePragma?.arguments.factory;
+}

package/dist/util/cli-arguments.d.ts

@@ -1,4 +1,4 @@
-export declare const helpText = "\u2702\uFE0F  Find unused files, dependencies and exports in your JavaScript and TypeScript projects\n\nUsage: knip [options]\n\nOptions:\n  -c, --config [file]      Configuration file path (default: [.]knip.json[c], knip.js, knip.ts or package.json#knip)\n  -t, --tsConfig [file]    TypeScript configuration path (default: tsconfig.json)\n  --production             Analyze only production source files (e.g. no tests, devDependencies, exported types)\n  --strict                 Consider only direct dependencies of workspace (not devDependencies, not other workspaces)\n  --workspace              Analyze a single workspace (default: analyze all configured workspaces)\n  --no-gitignore           Don't use .gitignore\n  --include                Report only provided issue type(s), can be comma-separated or repeated (1)\n  --exclude                Exclude provided issue type(s) from report, can be comma-separated or repeated (1)\n  --dependencies           Shortcut for --include dependencies,unlisted,unresolved\n  --exports                Shortcut for --include exports,nsExports,classMembers,types,nsTypes,enumMembers,duplicates\n  --include-entry-exports  Include entry files when reporting unused exports\n  -n, --no-progress        Don't show dynamic progress updates\n  --reporter               Select reporter: symbols, compact, codeowners, json (default: symbols)\n  --reporter-options       Pass extra options to the reporter (as JSON string, see example)\n  --no-config-hints        Suppress configuration hints\n  --no-exit-code           Always exit with code zero (0)\n  --max-issues             Maximum number of issues before non-zero exit code (default: 0)\n  -d, --debug              Show debug output\n  --debug-file-filter      Filter for files in debug output (regex as string)\n  --performance            Measure count and running time of expensive functions and display stats table\n  -h, --help               Print this help text\n  -V, --version            Print version\n\n(1) Issue types: files, dependencies, unlisted, unresolved, exports, nsExports, classMembers, types, nsTypes, enumMembers, duplicates\n\nExamples:\n\n$ knip\n$ knip --production\n$ knip --workspace packages/client --include files,dependencies\n$ knip -c ./config/knip.json --reporter compact\n$ knip --reporter codeowners --reporter-options '{\"path\":\".github/CODEOWNERS\"}'\n$ knip --debug --debug-file-filter '(specific|particular)-module'\n\nMore documentation and bug reports: https://github.com/webpro/knip";
+export declare const helpText = "\u2702\uFE0F  Find unused files, dependencies and exports in your JavaScript and TypeScript projects\n\nUsage: knip [options]\n\nOptions:\n  -c, --config [file]      Configuration file path (default: [.]knip.json[c], knip.js, knip.ts or package.json#knip)\n  -t, --tsConfig [file]    TypeScript configuration path (default: tsconfig.json)\n  --production             Analyze only production source files (e.g. no tests, devDependencies, exported types)\n  --strict                 Consider only direct dependencies of workspace (not devDependencies, not other workspaces)\n  --workspace              Analyze a single workspace (default: analyze all configured workspaces)\n  --no-gitignore           Don't use .gitignore\n  --include                Report only provided issue type(s), can be comma-separated or repeated (1)\n  --exclude                Exclude provided issue type(s) from report, can be comma-separated or repeated (1)\n  --dependencies           Shortcut for --include dependencies,unlisted,unresolved\n  --exports                Shortcut for --include exports,nsExports,classMembers,types,nsTypes,enumMembers,duplicates\n  --include-entry-exports  Include entry files when reporting unused exports\n  -n, --no-progress        Don't show dynamic progress updates (this is automatically enabled in CI environments)\n  --reporter               Select reporter: symbols, compact, codeowners, json (default: symbols)\n  --reporter-options       Pass extra options to the reporter (as JSON string, see example)\n  --no-config-hints        Suppress configuration hints\n  --no-exit-code           Always exit with code zero (0)\n  --max-issues             Maximum number of issues before non-zero exit code (default: 0)\n  -d, --debug              Show debug output\n  --debug-file-filter      Filter for files in debug output (regex as string)\n  --performance            Measure count and running time of expensive functions and display stats table\n  -h, --help               Print this help text\n  -V, --version            Print version\n\n(1) Issue types: files, dependencies, unlisted, unresolved, exports, nsExports, classMembers, types, nsTypes, enumMembers, duplicates\n\nExamples:\n\n$ knip\n$ knip --production\n$ knip --workspace packages/client --include files,dependencies\n$ knip -c ./config/knip.json --reporter compact\n$ knip --reporter codeowners --reporter-options '{\"path\":\".github/CODEOWNERS\"}'\n$ knip --debug --debug-file-filter '(specific|particular)-module'\n\nMore documentation and bug reports: https://github.com/webpro/knip";
 declare const _default: {
     config: string | undefined;
     debug: boolean | undefined;

package/dist/util/cli-arguments.js

@@ -15,7 +15,7 @@ Options:
   --dependencies           Shortcut for --include dependencies,unlisted,unresolved
   --exports                Shortcut for --include exports,nsExports,classMembers,types,nsTypes,enumMembers,duplicates
   --include-entry-exports  Include entry files when reporting unused exports
-  -n, --no-progress        Don't show dynamic progress updates
+  -n, --no-progress        Don't show dynamic progress updates (this is automatically enabled in CI environments)
   --reporter               Select reporter: symbols, compact, codeowners, json (default: symbols)
   --reporter-options       Pass extra options to the reporter (as JSON string, see example)
   --no-config-hints        Suppress configuration hints

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "2.17.2";
+export declare const version = "2.18.0";

package/dist/version.js

@@ -1 +1 @@
-export const version = '2.17.2';
+export const version = '2.18.0';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "2.17.2",
+  "version": "2.18.0",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://github.com/webpro/knip",
   "repository": "github:webpro/knip",
@@ -63,29 +63,29 @@
   "devDependencies": {
     "@jest/types": "29.6.1",
     "@npmcli/package-json": "4.0.1",
-    "@release-it/bumper": "5.0.0",
+    "@release-it/bumper": "5.1.0",
     "@swc/cli": "0.1.62",
-    "@swc/core": "1.3.70",
-    "@types/eslint": "8.44.0",
+    "@swc/core": "1.3.73",
+    "@types/eslint": "8.44.1",
     "@types/js-yaml": "4.0.5",
     "@types/micromatch": "4.0.2",
     "@types/minimist": "1.2.2",
-    "@types/node": "20.4.4",
+    "@types/node": "20.4.5",
     "@types/npmcli__map-workspaces": "3.0.1",
     "@types/webpack": "5.28.1",
-    "@typescript-eslint/eslint-plugin": "6.1.0",
-    "@typescript-eslint/parser": "6.1.0",
-    "c8": "8.0.0",
-    "eslint": "8.45.0",
+    "@typescript-eslint/eslint-plugin": "6.2.1",
+    "@typescript-eslint/parser": "6.2.1",
+    "c8": "8.0.1",
+    "eslint": "8.46.0",
     "eslint-import-resolver-typescript": "3.5.5",
-    "eslint-plugin-import": "2.27.5",
+    "eslint-plugin-import": "2.28.0",
     "eslint-plugin-n": "16.0.1",
     "prettier": "3.0.0",
     "release-it": "16.1.3",
     "remark-cli": "11.0.0",
     "remark-preset-webpro": "0.0.3",
     "tsx": "3.12.7",
-    "type-fest": "4.0.0"
+    "type-fest": "4.1.0"
   },
   "engines": {
     "node": ">=16.17.0 <17 || >=18.6.0"