knip 5.66.1 → 5.66.3

package/dist/graph/analyze.js

@@ -163,7 +163,7 @@ export const analyze = async ({ analyzedFiles, chief, collector, deputy, entryPa
         for (const [filePath, file] of graph.entries()) {
             const ws = chief.findWorkspaceByFilePath(filePath);
             if (ws) {
-                if (file.duplicates) {
+                if (file.duplicates && options.includedIssueTypes.duplicates) {
                     for (const symbols of file.duplicates) {
                         if (symbols.length > 1) {
                             const symbol = symbols.map(s => s.symbol).join('|');

package/dist/graph/build.js

@@ -78,7 +78,8 @@ export async function build({ chief, collector, deputy, factory, isGitIgnored, s
         collector.addIgnorePatterns(config.ignore.map(p => join(options.cwd, prependDirToPattern(name, p))));
         collector.addIgnoreFilesPatterns(config.ignoreFiles.map(p => join(options.cwd, prependDirToPattern(name, p))));
         const entrySpecifiersFromManifest = getEntrySpecifiersFromManifest(manifest);
-        for (const filePath of await toSourceFilePaths(entrySpecifiersFromManifest, dir, extensionGlobStr)) {
+        const label = 'entry paths from package.json';
+        for (const filePath of await toSourceFilePaths(entrySpecifiersFromManifest, dir, extensionGlobStr, label)) {
             inputs.add(toProductionEntry(filePath));
         }
         for (const identifier of entrySpecifiersFromManifest) {

package/dist/index.d.ts

@@ -1,6 +1,4 @@
 import type { MainOptions } from './util/create-options.js';
-export type { RawConfiguration as KnipConfig } from './types/config.js';
-export type { Preprocessor, Reporter, ReporterOptions } from './types/issues.js';
 export declare const main: (options: MainOptions) => Promise<{
     issues: import("./types/issues.js").Issues;
     counters: import("./types/issues.js").Counters;

package/dist/plugins/next/index.js

@@ -7,7 +7,7 @@ const isEnabled = ({ dependencies }) => hasDependency(dependencies, enablers);
 const config = ['next.config.{js,ts,cjs,mjs}'];
 const defaultPageExtensions = ['{js,jsx,ts,tsx}'];
 const productionEntryFilePatterns = [
-    '{instrumentation,instrumentation-client,middleware}.{js,ts}',
+    '{instrumentation,instrumentation-client,middleware,proxy}.{js,ts}',
     'app/global-error.{js,jsx,ts,tsx}',
     'app/**/{error,layout,loading,not-found,page,template,default}.{js,jsx,ts,tsx}',
     'app/**/route.{js,jsx,ts,tsx}',

package/dist/plugins/rsbuild/index.js

@@ -1,25 +1,25 @@
-import { toEntry } from '../../util/input.js';
+import { toProductionEntry } from '../../util/input.js';
 import { hasDependency } from '../../util/plugin.js';
 const title = 'Rsbuild';
 const enablers = ['@rsbuild/core'];
 const isEnabled = ({ dependencies }) => hasDependency(dependencies, enablers);
 const config = ['rsbuild*.config.{mjs,ts,js,cjs,mts,cts}'];
 const resolveConfig = async (config) => {
-    const inputs = new Set();
+    const entries = new Set();
     const checkSource = (source) => {
         if (source?.entry) {
             for (const entry of Object.values(source.entry)) {
                 if (typeof entry === 'string')
-                    inputs.add(toEntry(entry));
+                    entries.add(entry);
                 else if (Array.isArray(entry))
                     for (const e of entry)
-                        inputs.add(toEntry(e));
+                        entries.add(e);
                 else {
                     if (typeof entry.import === 'string')
-                        inputs.add(toEntry(entry.import));
+                        entries.add(entry.import);
                     else if (Array.isArray(entry.import))
                         for (const e of entry.import)
-                            inputs.add(toEntry(e));
+                            entries.add(e);
                 }
             }
         }
@@ -30,7 +30,7 @@ const resolveConfig = async (config) => {
             checkSource(environment.source);
         }
     }
-    return Array.from(inputs);
+    return Array.from(entries).map(input => toProductionEntry(input));
 };
 export default {
     title,

package/dist/plugins/vitest/helpers.js

@@ -16,6 +16,7 @@ export const getEnvPackageName = (env) => {
 };
 const builtInReporters = [
     'basic',
+    'blob',
     'default',
     'dot',
     'github-actions',
@@ -25,6 +26,7 @@ const builtInReporters = [
     'junit',
     'tap',
     'tap-flat',
+    'tree',
     'verbose',
 ];
 export const getExternalReporters = (reporters) => reporters

package/dist/reporters/index.d.ts

@@ -1,11 +1,11 @@
 declare const _default: {
-    symbols: (options: import("../index.js").ReporterOptions) => void;
-    compact: ({ report, issues, isShowProgress, cwd }: import("../index.js").ReporterOptions) => void;
-    codeowners: ({ report, issues, isShowProgress, options, cwd }: import("../index.js").ReporterOptions) => void;
-    disclosure: ({ report, issues, cwd }: import("../index.js").ReporterOptions) => void;
-    codeclimate: ({ report, issues, cwd }: import("../index.js").ReporterOptions) => Promise<void>;
-    json: ({ report, issues, options, cwd }: import("../index.js").ReporterOptions) => Promise<void>;
-    markdown: ({ report, issues, cwd }: import("../index.js").ReporterOptions) => void;
-    'github-actions': ({ report, issues, cwd, configurationHints, isDisableConfigHints, isTreatConfigHintsAsErrors, configFilePath, }: import("../index.js").ReporterOptions) => void;
+    symbols: (options: import("../types.js").ReporterOptions) => void;
+    compact: ({ report, issues, isShowProgress, cwd }: import("../types.js").ReporterOptions) => void;
+    codeowners: ({ report, issues, isShowProgress, options, cwd }: import("../types.js").ReporterOptions) => void;
+    disclosure: ({ report, issues, cwd }: import("../types.js").ReporterOptions) => void;
+    codeclimate: ({ report, issues, cwd }: import("../types.js").ReporterOptions) => Promise<void>;
+    json: ({ report, issues, options, cwd }: import("../types.js").ReporterOptions) => Promise<void>;
+    markdown: ({ report, issues, cwd }: import("../types.js").ReporterOptions) => void;
+    'github-actions': ({ report, issues, cwd, configurationHints, isDisableConfigHints, isTreatConfigHintsAsErrors, configFilePath, }: import("../types.js").ReporterOptions) => void;
 };
 export default _default;

package/dist/types.d.ts

@@ -0,0 +1,2 @@
+export type { RawConfiguration as KnipConfig } from './types/config.js';
+export type { Preprocessor, Reporter, ReporterOptions } from './types/issues.js';

package/dist/types.js

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

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

@@ -33,7 +33,7 @@ export default visit(isModule, (node, { isFixExports, isFixTypes, isReportClassM
                             return {
                                 node: element,
                                 symbol: element.symbol,
-                                identifier: element.getText(),
+                                identifier: element.name.getText(),
                                 type: SYMBOL_TYPE.UNKNOWN,
                                 pos: element.getStart(),
                                 fix,

package/dist/typescript/visitors/imports/importDeclaration.js

@@ -37,7 +37,7 @@ export default visit(() => true, node => {
                         specifier,
                         symbol: element.symbol,
                         isTypeOnly: node.importClause?.isTypeOnly,
-                        pos: element.pos,
+                        pos: element.getStart(),
                     });
                 }
             }

package/dist/util/create-options.js

@@ -58,6 +58,10 @@ export const createOptions = async (options) => {
             ...(parsedCLIArgs.files ? shorthandFiles : []),
         ],
     });
+    for (const [key, value] of Object.entries(includedIssueTypes)) {
+        if (!value)
+            rules[key] = 'off';
+    }
     const fixTypes = rest.fixTypes ?? parsedCLIArgs['fix-type'] ?? [];
     const isFixFiles = parsedCLIArgs['allow-remove-files'] && (fixTypes.length === 0 || fixTypes.includes('files'));
     const isIncludeLibs = parsedCLIArgs['include-libs'] ?? rest.isIncludeLibs ?? false;

package/dist/util/module-graph.js

@@ -24,9 +24,8 @@ export const updateImportMap = (file, importMap, graph) => {
             updateImportDetails(importedFileImports, importDetails);
         const importedFile = getOrCreateFileNode(graph, importedFilePath);
         if (!importedFile.imported)
-            importedFile.imported = importDetails;
-        else
-            updateImportDetails(importedFile.imported, importDetails);
+            importedFile.imported = createImports();
+        updateImportDetails(importedFile.imported, importDetails);
         graph.set(importedFilePath, importedFile);
     }
 };

package/dist/util/to-source-path.d.ts

@@ -2,5 +2,5 @@ import type { CompilerOptions } from 'typescript';
 import type { ConfigurationChief, Workspace } from '../ConfigurationChief.js';
 export declare const augmentWorkspace: (workspace: Workspace, dir: string, compilerOptions: CompilerOptions) => void;
 export declare const getToSourcePathHandler: (chief: ConfigurationChief) => (filePath: string) => string | undefined;
-export declare const getToSourcePathsHandler: (chief: ConfigurationChief) => (specifiers: Set<string>, dir: string, extensions?: string) => Promise<string[]>;
+export declare const getToSourcePathsHandler: (chief: ConfigurationChief) => (specifiers: Set<string>, dir: string, extensions: string | undefined, label: string) => Promise<string[]>;
 export type ToSourceFilePath = ReturnType<typeof getToSourcePathHandler>;

package/dist/util/to-source-path.js

@@ -1,7 +1,7 @@
 import { DEFAULT_EXTENSIONS } from '../constants.js';
 import { debugLog, debugLogArray } from './debug.js';
 import { isDirectory } from './fs.js';
-import { _glob, _syncGlob } from './glob.js';
+import { _glob, _syncGlob, prependDirToPattern } from './glob.js';
 import { isAbsolute, isInternal, join, toRelative } from './path.js';
 const defaultExtensions = `.{${DEFAULT_EXTENSIONS.map(ext => ext.slice(1)).join(',')}}`;
 const hasTSExt = /(?<!\.d)\.(m|c)?tsx?$/;
@@ -35,10 +35,10 @@ export const getToSourcePathHandler = (chief) => {
     };
 };
 export const getToSourcePathsHandler = (chief) => {
-    return async (specifiers, dir, extensions = defaultExtensions) => {
+    return async (specifiers, dir, extensions = defaultExtensions, label) => {
         const patterns = new Set();
         for (const specifier of specifiers) {
-            const absSpecifier = isAbsolute(specifier) ? specifier : join(dir, specifier);
+            const absSpecifier = isAbsolute(specifier) ? specifier : prependDirToPattern(dir, specifier);
             const ws = chief.findWorkspaceByFilePath(absSpecifier);
             if (ws?.srcDir && ws.outDir && !absSpecifier.startsWith(ws.srcDir) && absSpecifier.startsWith(ws.outDir)) {
                 const pattern = absSpecifier.replace(ws.outDir, ws.srcDir).replace(matchExt, extensions);
@@ -48,7 +48,7 @@ export const getToSourcePathsHandler = (chief) => {
                 patterns.add(absSpecifier);
             }
         }
-        const filePaths = await _glob({ patterns: Array.from(patterns), cwd: dir });
+        const filePaths = await _glob({ patterns: Array.from(patterns), cwd: dir, label });
         debugLogArray(toRelative(dir, chief.cwd), 'Source mapping (package.json)', filePaths);
         return filePaths;
     };

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "5.66.1";
+export declare const version = "5.66.3";

package/dist/version.js

@@ -1 +1 @@
-export const version = '5.66.1';
+export const version = '5.66.3';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "5.66.1",
+  "version": "5.66.3",
   "description": "Find and fix unused dependencies, exports and files in your TypeScript and JavaScript projects",
   "homepage": "https://knip.dev",
   "repository": {
@@ -30,7 +30,7 @@
     "knip-bun": "bin/knip-bun.js"
   },
   "type": "module",
-  "types": "./dist/index.d.ts",
+  "types": "./dist/types.d.ts",
   "scripts": {
     "knip": "node ./dist/cli.js --directory ../..",
     "knip:production": "node ./dist/cli.js --directory ../.. --production --strict",