knip 5.30.6 → 5.31.0

package/dist/ConfigurationValidator.d.ts

@@ -2775,9 +2775,9 @@ export declare const ConfigurationValidator: z.ZodObject<z.objectUtil.extendShap
     ignoreExportsUsedInFile?: boolean | Partial<Record<"function" | "type" | "enum" | "class" | "interface" | "member", boolean>> | undefined;
     ignoreWorkspaces?: string[] | undefined;
     includeEntryExports?: boolean | undefined;
-    compilers?: Record<string, ((args_0: string, args_1: string, ...args: unknown[]) => string) | ((args_0: string, args_1: string, ...args: unknown[]) => Promise<string>)> | undefined;
-    syncCompilers?: Record<string, (args_0: string, args_1: string, ...args: unknown[]) => string> | undefined;
-    asyncCompilers?: Record<string, (args_0: string, args_1: string, ...args: unknown[]) => Promise<string>> | undefined;
+    compilers?: Record<string, ((args_0: string, args_1: string, ...args_2: unknown[]) => string) | ((args_0: string, args_1: string, ...args_2: unknown[]) => Promise<string>)> | undefined;
+    syncCompilers?: Record<string, (args_0: string, args_1: string, ...args_2: unknown[]) => string> | undefined;
+    asyncCompilers?: Record<string, (args_0: string, args_1: string, ...args_2: unknown[]) => Promise<string>> | undefined;
     astro?: string | boolean | string[] | {
         config?: string | string[] | undefined;
         entry?: string | string[] | undefined;
@@ -3552,9 +3552,9 @@ export declare const ConfigurationValidator: z.ZodObject<z.objectUtil.extendShap
     ignoreExportsUsedInFile?: boolean | Partial<Record<"function" | "type" | "enum" | "class" | "interface" | "member", boolean>> | undefined;
     ignoreWorkspaces?: string[] | undefined;
     includeEntryExports?: boolean | undefined;
-    compilers?: Record<string, ((args_0: string, args_1: string, ...args: unknown[]) => string) | ((args_0: string, args_1: string, ...args: unknown[]) => Promise<string>)> | undefined;
-    syncCompilers?: Record<string, (args_0: string, args_1: string, ...args: unknown[]) => string> | undefined;
-    asyncCompilers?: Record<string, (args_0: string, args_1: string, ...args: unknown[]) => Promise<string>> | undefined;
+    compilers?: Record<string, ((args_0: string, args_1: string, ...args_2: unknown[]) => string) | ((args_0: string, args_1: string, ...args_2: unknown[]) => Promise<string>)> | undefined;
+    syncCompilers?: Record<string, (args_0: string, args_1: string, ...args_2: unknown[]) => string> | undefined;
+    asyncCompilers?: Record<string, (args_0: string, args_1: string, ...args_2: unknown[]) => Promise<string>> | undefined;
     astro?: string | boolean | string[] | {
         config?: string | string[] | undefined;
         entry?: string | string[] | undefined;

package/dist/IssueFixer.d.ts

@@ -1,4 +1,4 @@
-import type { Fixes } from './types/exports.js';
+import type { Fix, Fixes } from './types/exports.js';
 import type { Issues } from './types/issues.js';
 interface Fixer {
     isEnabled: boolean;
@@ -13,15 +13,15 @@ export declare class IssueFixer {
     isFixDependencies: boolean;
     isFixUnusedTypes: boolean;
     isFixUnusedExports: boolean;
-    unusedTypeNodes: Map<string, Set<[number, number]>>;
-    unusedExportNodes: Map<string, Set<[number, number]>>;
+    unusedTypeNodes: Map<string, Set<Fix>>;
+    unusedExportNodes: Map<string, Set<Fix>>;
     constructor({ isEnabled, cwd, fixTypes, isRemoveFiles }: Fixer);
     addUnusedTypeNode(filePath: string, fixes: Fixes | undefined): void;
     addUnusedExportNode(filePath: string, fixes: Fixes | undefined): void;
     fixIssues(issues: Issues): Promise<void>;
     private markExportFixed;
     private removeUnusedFiles;
-    private removeUnusedExportKeywords;
+    private removeUnusedExports;
     private removeUnusedDependencies;
 }
 export {};

package/dist/IssueFixer.js

@@ -1,6 +1,7 @@
 import { readFile, rm, writeFile } from 'node:fs/promises';
 import { load, save } from './util/package-json.js';
 import { join, relative } from './util/path.js';
+import { removeExport } from './util/remove-export.js';
 export class IssueFixer {
     isEnabled = false;
     cwd = process.cwd();
@@ -38,13 +39,13 @@ export class IssueFixer {
     }
     async fixIssues(issues) {
         await this.removeUnusedFiles(issues);
-        await this.removeUnusedExportKeywords(issues);
+        await this.removeUnusedExports(issues);
         await this.removeUnusedDependencies(issues);
     }
     markExportFixed(issues, filePath) {
         const relPath = relative(filePath);
         const types = [
-            ...(this.isFixUnusedTypes ? ['types', 'nsTypes'] : []),
+            ...(this.isFixUnusedTypes ? ['types', 'nsTypes', 'classMembers', 'enumMembers'] : []),
             ...(this.isFixUnusedExports ? ['exports', 'nsExports'] : []),
         ];
         for (const type of types) {
@@ -61,19 +62,15 @@ export class IssueFixer {
             issue.isFixed = true;
         }
     }
-    async removeUnusedExportKeywords(issues) {
+    async removeUnusedExports(issues) {
         const filePaths = new Set([...this.unusedTypeNodes.keys(), ...this.unusedExportNodes.keys()]);
         for (const filePath of filePaths) {
-            const exportPositions = [
-                ...(this.isFixUnusedTypes ? (this.unusedTypeNodes.get(filePath) ?? []) : []),
-                ...(this.isFixUnusedExports ? (this.unusedExportNodes.get(filePath) ?? []) : []),
-            ].sort((a, b) => b[0] - a[0]);
+            const types = (this.isFixUnusedTypes && this.unusedTypeNodes.get(filePath)) || [];
+            const exports = (this.isFixUnusedExports && this.unusedExportNodes.get(filePath)) || [];
+            const exportPositions = [...types, ...exports].filter(fix => fix !== undefined).sort((a, b) => b[0] - a[0]);
             if (exportPositions.length > 0) {
-                const sourceFileText = exportPositions.reduce((text, [start, end]) => text.substring(0, start) + text.substring(end), await readFile(filePath, 'utf-8'));
-                const withoutEmptyReExports = sourceFileText
-                    .replaceAll(/export \{[ ,]+\} from ('|")[^'"]+('|");?\r?\n?/g, '')
-                    .replaceAll(/export \{[ ,]+\};?\r?\n?/g, '');
-                await writeFile(filePath, withoutEmptyReExports);
+                const sourceFileText = exportPositions.reduce((text, [start, end, flags]) => removeExport({ text, start, end, flags }), await readFile(filePath, 'utf-8'));
+                await writeFile(filePath, sourceFileText);
                 this.markExportFixed(issues, filePath);
             }
         }

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/compilers/index.d.ts

@@ -17,7 +17,7 @@ export declare const partitionCompilers: (rawLocalConfig: RawConfiguration) => {
     ignoreExportsUsedInFile?: boolean | Partial<Record<"function" | "type" | "enum" | "class" | "interface" | "member", boolean>> | undefined;
     ignoreWorkspaces?: string[] | undefined;
     includeEntryExports?: boolean | undefined;
-    compilers?: Record<string, ((args_0: string, args_1: string, ...args: unknown[]) => string) | ((args_0: string, args_1: string, ...args: unknown[]) => Promise<string>)> | undefined;
+    compilers?: Record<string, ((args_0: string, args_1: string, ...args_2: unknown[]) => string) | ((args_0: string, args_1: string, ...args_2: unknown[]) => Promise<string>)> | undefined;
     astro?: string | boolean | string[] | {
         config?: string | string[] | undefined;
         entry?: string | string[] | undefined;

package/dist/constants.d.ts

@@ -17,3 +17,8 @@ export declare const FOREIGN_FILE_EXTENSIONS: Set<string>;
 export declare const IGNORE_DEFINITELY_TYPED: Set<string>;
 export declare const ISSUE_TYPES: IssueType[];
 export declare const ISSUE_TYPE_TITLE: Record<IssueType, string>;
+export declare const FIX_FLAGS: {
+    readonly NONE: 0;
+    readonly OBJECT_BINDING: number;
+    readonly EMPTY_DECLARATION: number;
+};

package/dist/constants.js

@@ -180,3 +180,8 @@ export const ISSUE_TYPE_TITLE = {
     classMembers: 'Unused exported class members',
     duplicates: 'Duplicate exports',
 };
+export const FIX_FLAGS = {
+    NONE: 0,
+    OBJECT_BINDING: 1 << 0,
+    EMPTY_DECLARATION: 1 << 1,
+};

package/dist/index.js

@@ -290,7 +290,7 @@ export const main = async (unresolvedConfiguration) => {
                     }
                     const importsForExport = file.imported;
                     for (const [identifier, exportedItem] of exportItems.entries()) {
-                        if (exportedItem.isReExport)
+                        if (!isFix && exportedItem.isReExport)
                             continue;
                         if (shouldIgnore(exportedItem.jsDocTags))
                             continue;
@@ -331,7 +331,7 @@ export const main = async (unresolvedConfiguration) => {
                                             if (!isReferenced) {
                                                 if (isIgnored)
                                                     continue;
-                                                collector.addIssue({
+                                                const isIssueAdded = collector.addIssue({
                                                     type: 'enumMembers',
                                                     filePath,
                                                     workspace: workspace.name,
@@ -341,6 +341,8 @@ export const main = async (unresolvedConfiguration) => {
                                                     line: member.line,
                                                     col: member.col,
                                                 });
+                                                if (isFix && isIssueAdded && member.fix)
+                                                    fixer.addUnusedTypeNode(filePath, [member.fix]);
                                             }
                                             else if (isIgnored) {
                                                 for (const tagName of exportedItem.jsDocTags) {
@@ -364,7 +366,7 @@ export const main = async (unresolvedConfiguration) => {
                                             }
                                             continue;
                                         }
-                                        collector.addIssue({
+                                        const isIssueAdded = collector.addIssue({
                                             type: 'classMembers',
                                             filePath,
                                             workspace: workspace.name,
@@ -374,6 +376,8 @@ export const main = async (unresolvedConfiguration) => {
                                             line: member.line,
                                             col: member.col,
                                         });
+                                        if (isFix && isIssueAdded && member.fix)
+                                            fixer.addUnusedTypeNode(filePath, [member.fix]);
                                     }
                                 }
                                 continue;

package/dist/reporters/symbols.js

@@ -15,7 +15,7 @@ const logIssueRecord = (issues) => {
         const pos = issue.line === undefined ? '' : `:${issue.line}${issue.col === undefined ? '' : `:${issue.col}`}`;
         const cell = `${relative(issue.filePath)}${pos}`;
         table.cell('filePath', print(cell));
-        issue.isFixed && table.cell('fixed', print('(fixed)'));
+        issue.isFixed && table.cell('fixed', print('(removed)'));
         table.newRow();
     }
     console.log(table.sort(['filePath', 'parentSymbol', 'symbol']).print().trim());
@@ -35,7 +35,7 @@ export default ({ report, issues, tagHints, configurationHints, noConfigHints, i
                     for (const issue of issuesForType) {
                         const relPath = toRelative(issue.filePath);
                         if (issue.isFixed)
-                            console.log(picocolors.gray(`${relPath} (deleted)`));
+                            console.log(picocolors.gray(`${relPath} (removed)`));
                         else if (issue.severity === 'warn')
                             console.log(picocolors.gray(relPath));
                         else

package/dist/types/exports.d.ts

@@ -1,7 +1,7 @@
 import type ts from 'typescript';
 import type { SymbolType } from './issues.js';
 type Identifier = string;
-type ExportPosTuple = [number, number];
+type ExportPosTuple = [number, number, number];
 export type Fix = ExportPosTuple | undefined;
 export type Fixes = Array<ExportPosTuple>;
 export type ExportNode = {

package/dist/typescript/visitors/dynamic-imports/importCall.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[];
 export default _default;

package/dist/typescript/visitors/dynamic-imports/importType.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[];
 export default _default;

package/dist/typescript/visitors/dynamic-imports/index.d.ts

@@ -1,3 +1,3 @@
 import type ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => ((node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[] | undefined)[];
+declare const _default: (sourceFile: ts.SourceFile) => ((node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[])[];
 export default _default;

package/dist/typescript/visitors/dynamic-imports/jsDocType.d.ts

@@ -1,4 +1,4 @@
 import ts from 'typescript';
 import type { ImportNode } from '../../../types/imports.js';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => ImportNode | ImportNode[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | ImportNode | ImportNode[];
 export default _default;

package/dist/typescript/visitors/dynamic-imports/propertyAccessCall.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[];
 export default _default;

package/dist/typescript/visitors/dynamic-imports/requireCall.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[];
 export default _default;

package/dist/typescript/visitors/exports/exportAssignment.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/exports.js").ExportNode | import("../../../types/exports.js").ExportNode[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/exports.js").ExportNode | import("../../../types/exports.js").ExportNode[];
 export default _default;

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

@@ -1,10 +1,11 @@
 import ts from 'typescript';
+import { FIX_FLAGS } from '../../../constants.js';
 import { SymbolType } from '../../../types/issues.js';
 import { exportVisitor as visit } from '../index.js';
 export default visit(() => true, (node, { isFixExports }) => {
     if (ts.isExportAssignment(node)) {
         const pos = node.getChildAt(1).getStart();
-        const fix = isFixExports ? [node.getStart(), node.getEnd() + 1] : undefined;
+        const fix = isFixExports ? [node.getStart(), node.getEnd() + 1, FIX_FLAGS.NONE] : undefined;
         const symbol = node.getSourceFile().locals?.get(node.expression.escapedText);
         return { node, symbol, identifier: 'default', type: SymbolType.UNKNOWN, pos, fix };
     }

package/dist/typescript/visitors/exports/exportDeclaration.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/exports.js").ExportNode | import("../../../types/exports.js").ExportNode[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/exports.js").ExportNode | import("../../../types/exports.js").ExportNode[];
 export default _default;

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

@@ -1,10 +1,11 @@
 import ts from 'typescript';
+import { FIX_FLAGS } from '../../../constants.js';
 import { SymbolType } from '../../../types/issues.js';
 import { exportVisitor as visit } from '../index.js';
 export default visit(() => true, (node, { isFixExports, isFixTypes }) => {
     if (ts.isExportDeclaration(node)) {
         if (node.exportClause && ts.isNamedExports(node.exportClause)) {
-            const type = node.isTypeOnly ? SymbolType.TYPE : SymbolType.UNKNOWN;
+            const nodeType = node.isTypeOnly ? SymbolType.TYPE : SymbolType.UNKNOWN;
             const sourceFile = node.getSourceFile();
             const declarations = sourceFile.getNamedDeclarations?.();
             return node.exportClause.elements.map(element => {
@@ -12,7 +13,10 @@ export default visit(() => true, (node, { isFixExports, isFixTypes }) => {
                 const propName = element.propertyName?.text;
                 const symbol = declarations?.get(propName ?? identifier)?.[0]?.symbol;
                 const pos = element.name.pos;
-                const fix = isFixExports || isFixTypes ? [element.getStart(), element.getEnd()] : undefined;
+                const type = element.isTypeOnly ? SymbolType.TYPE : nodeType;
+                const fix = (isFixExports && type !== SymbolType.TYPE) || (isFixTypes && type === SymbolType.TYPE)
+                    ? [element.getStart(), element.getEnd(), FIX_FLAGS.OBJECT_BINDING | FIX_FLAGS.EMPTY_DECLARATION]
+                    : undefined;
                 return { node: element, symbol, identifier, type, pos, fix };
             });
         }

package/dist/typescript/visitors/exports/exportKeyword.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/exports.js").ExportNode | import("../../../types/exports.js").ExportNode[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/exports.js").ExportNode | import("../../../types/exports.js").ExportNode[];
 export default _default;

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

@@ -1,4 +1,5 @@
 import ts from 'typescript';
+import { FIX_FLAGS } from '../../../constants.js';
 import { SymbolType } from '../../../types/issues.js';
 import { compact } from '../../../util/array.js';
 import { getDefaultKeywordNode, getExportKeywordNode, isGetOrSetAccessorDeclaration, isPrivateMember, stripQuotes, } from '../../ast-helpers.js';
@@ -6,15 +7,16 @@ import { exportVisitor as visit } from '../index.js';
 export default visit(() => true, (node, { isFixExports, isFixTypes, isReportClassMembers }) => {
     const exportKeyword = getExportKeywordNode(node);
     if (exportKeyword) {
-        const getFix = (node, defaultKeyword) => isFixExports ? [node.getStart(), (defaultKeyword ?? node).getEnd() + 1] : undefined;
-        const getElementFix = (node) => (isFixExports ? [node.getStart(), node.getEnd()] : undefined);
-        const getTypeFix = (node) => (isFixTypes ? [node.getStart(), node.getEnd() + 1] : undefined);
+        const getFix = (node, defaultKeyword) => isFixExports ? [node.getStart(), (defaultKeyword ?? node).getEnd() + 1, FIX_FLAGS.NONE] : undefined;
+        const getTypeFix = (node) => isFixTypes ? [node.getStart(), node.getEnd() + 1, FIX_FLAGS.NONE] : undefined;
         if (ts.isVariableStatement(node)) {
             return node.declarationList.declarations.flatMap(declaration => {
                 if (ts.isObjectBindingPattern(declaration.name)) {
                     return compact(declaration.name.elements.map(element => {
                         if (ts.isIdentifier(element.name)) {
-                            const fix = getElementFix(element);
+                            const fix = isFixExports
+                                ? [element.getStart(), element.getEnd(), FIX_FLAGS.OBJECT_BINDING]
+                                : undefined;
                             return {
                                 node: element,
                                 identifier: element.name.escapedText.toString(),
@@ -28,7 +30,7 @@ export default visit(() => true, (node, { isFixExports, isFixTypes, isReportClas
                 if (ts.isArrayBindingPattern(declaration.name)) {
                     return compact(declaration.name.elements.map(element => {
                         if (ts.isBindingElement(element)) {
-                            const fix = getElementFix(element);
+                            const fix = isFixExports ? [element.getStart(), element.getEnd(), FIX_FLAGS.NONE] : undefined;
                             return {
                                 node: element,
                                 identifier: element.getText(),
@@ -67,7 +69,7 @@ export default visit(() => true, (node, { isFixExports, isFixTypes, isReportClas
                     identifier: member.name.getText(),
                     pos: member.name.getStart() + (ts.isComputedPropertyName(member.name) ? 1 : 0),
                     type: SymbolType.MEMBER,
-                    fix: undefined,
+                    fix: isFixTypes ? [member.getStart(), member.getEnd(), FIX_FLAGS.NONE] : undefined,
                 }))
                 : [];
             return { node, identifier, type: SymbolType.CLASS, pos, members, fix };
@@ -93,7 +95,7 @@ export default visit(() => true, (node, { isFixExports, isFixTypes, isReportClas
                 identifier: stripQuotes(member.name.getText()),
                 pos: member.name.getStart(),
                 type: SymbolType.MEMBER,
-                fix: undefined,
+                fix: isFixTypes ? [member.getStart(), member.getEnd(), FIX_FLAGS.OBJECT_BINDING] : undefined,
             }));
             return { node, identifier, type: SymbolType.ENUM, pos, members, fix };
         }

package/dist/typescript/visitors/exports/exportsAccessExpression.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/exports.js").ExportNode | import("../../../types/exports.js").ExportNode[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/exports.js").ExportNode | import("../../../types/exports.js").ExportNode[];
 export default _default;

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

@@ -1,4 +1,5 @@
 import ts from 'typescript';
+import { FIX_FLAGS } from '../../../constants.js';
 import { SymbolType } from '../../../types/issues.js';
 import { isJS } from '../helpers.js';
 import { exportVisitor as visit } from '../index.js';
@@ -7,7 +8,7 @@ export default visit(isJS, (node, { isFixExports }) => {
         if (ts.isIdentifier(node.left.expression) && node.left.expression.escapedText === 'exports') {
             const identifier = node.left.name.getText();
             const pos = node.left.name.pos;
-            const fix = isFixExports ? [node.getStart(), node.getEnd()] : undefined;
+            const fix = isFixExports ? [node.getStart(), node.getEnd(), FIX_FLAGS.NONE] : undefined;
             return {
                 node: node.left.name,
                 identifier,

package/dist/typescript/visitors/exports/index.d.ts

@@ -1,3 +1,3 @@
 import type ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => ((node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/exports.js").ExportNode | import("../../../types/exports.js").ExportNode[] | undefined)[];
+declare const _default: (sourceFile: ts.SourceFile) => ((node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/exports.js").ExportNode | import("../../../types/exports.js").ExportNode[])[];
 export default _default;

package/dist/typescript/visitors/exports/moduleExportsAccessExpression.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/exports.js").ExportNode | import("../../../types/exports.js").ExportNode[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/exports.js").ExportNode | import("../../../types/exports.js").ExportNode[];
 export default _default;

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

@@ -1,4 +1,5 @@
 import ts from 'typescript';
+import { FIX_FLAGS } from '../../../constants.js';
 import { SymbolType } from '../../../types/issues.js';
 import { hasRequireCall, isModuleExportsAccess, stripQuotes } from '../../ast-helpers.js';
 import { isJS } from '../helpers.js';
@@ -11,7 +12,7 @@ export default visit(isJS, (node, { isFixExports }) => {
                     isModuleExportsAccess(node.expression.left.expression)) {
                     const identifier = node.expression.left.name.getText();
                     const pos = node.expression.left.name.pos;
-                    const fix = isFixExports ? [node.getStart(), node.getEnd()] : undefined;
+                    const fix = isFixExports ? [node.getStart(), node.getEnd(), FIX_FLAGS.NONE] : undefined;
                     return {
                         node: node.expression.left.name,
                         identifier,
@@ -24,7 +25,7 @@ export default visit(isJS, (node, { isFixExports }) => {
                     const expr = node.expression.right;
                     if (ts.isObjectLiteralExpression(expr) && expr.properties.every(ts.isShorthandPropertyAssignment)) {
                         return expr.properties.map(node => {
-                            const fix = isFixExports ? [node.getStart(), node.getEnd()] : undefined;
+                            const fix = isFixExports ? [node.getStart(), node.getEnd(), FIX_FLAGS.NONE] : undefined;
                             return { node, identifier: node.getText(), type: SymbolType.UNKNOWN, pos: node.getStart(), fix };
                         });
                     }
@@ -40,7 +41,7 @@ export default visit(isJS, (node, { isFixExports }) => {
                 isModuleExportsAccess(node.expression.left.expression)) {
                 const identifier = stripQuotes(node.expression.left.argumentExpression.getText());
                 const pos = node.expression.left.argumentExpression.pos;
-                const fix = isFixExports ? [node.getStart(), node.getEnd()] : undefined;
+                const fix = isFixExports ? [node.getStart(), node.getEnd(), FIX_FLAGS.NONE] : undefined;
                 return {
                     node: node.expression.left.argumentExpression,
                     identifier,

package/dist/typescript/visitors/imports/importDeclaration.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[];
 export default _default;

package/dist/typescript/visitors/imports/importEqualsDeclaration.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[];
 export default _default;

package/dist/typescript/visitors/imports/index.d.ts

@@ -1,3 +1,3 @@
 import type ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => ((node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[] | undefined)[];
+declare const _default: (sourceFile: ts.SourceFile) => ((node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[])[];
 export default _default;

package/dist/typescript/visitors/imports/reExportDeclaration.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | import("../../../types/imports.js").ImportNode | import("../../../types/imports.js").ImportNode[];
 export default _default;

package/dist/typescript/visitors/scripts/bun.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => string | string[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | string | string[];
 export default _default;

package/dist/typescript/visitors/scripts/execa.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => string | string[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | string | string[];
 export default _default;

package/dist/typescript/visitors/scripts/index.d.ts

@@ -1,3 +1,3 @@
 import type ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => ((node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => string | string[] | undefined)[];
+declare const _default: (sourceFile: ts.SourceFile) => ((node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | string | string[])[];
 export default _default;

package/dist/typescript/visitors/scripts/zx.d.ts

@@ -1,3 +1,3 @@
 import ts from 'typescript';
-declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => string | string[] | undefined;
+declare const _default: (sourceFile: ts.SourceFile) => (node: ts.Node, options: import("../../get-imports-and-exports.js").GetImportsAndExportsOptions) => undefined | string | string[];
 export default _default;

package/dist/util/fs.d.ts

@@ -7,5 +7,4 @@ export declare const loadYAML: (filePath: string) => Promise<unknown>;
 export declare const loadTOML: (filePath: string) => Promise<Record<string, import("smol-toml").TomlPrimitive>>;
 export declare const parseJSON: (filePath: string, contents: string) => Promise<any>;
 export declare const parseYAML: (contents: string) => unknown;
-export declare function isTypeModule(path: string): boolean;
 export declare const _loadJSON: (filePath: string) => Promise<any>;

package/dist/util/fs.js

@@ -1,11 +1,11 @@
-import { readFileSync, statSync } from 'node:fs';
+import { statSync } from 'node:fs';
 import { readFile } from 'node:fs/promises';
 import yaml from 'js-yaml';
 import { parse as parseTOML } from 'smol-toml';
 import stripJsonComments from 'strip-json-comments';
 import { timerify } from './Performance.js';
 import { LoaderError } from './errors.js';
-import { dirname, join } from './path.js';
+import { join } from './path.js';
 export const isDirectory = (filePath) => {
     try {
         return statSync(filePath).isDirectory();
@@ -58,19 +58,4 @@ export const parseJSON = async (filePath, contents) => {
 export const parseYAML = (contents) => {
     return yaml.load(contents);
 };
-export function isTypeModule(path) {
-    while (path && path !== '.' && path !== '/') {
-        path = dirname(path);
-        try {
-            const pkg = readFileSync(join(path, 'package.json'), 'utf-8');
-            try {
-                return JSON.parse(pkg).type === 'module';
-            }
-            catch { }
-            break;
-        }
-        catch { }
-    }
-    return false;
-}
 export const _loadJSON = timerify(loadJSON);

package/dist/util/glob-core.js

@@ -36,7 +36,7 @@ export const parseAndConvertGitignorePatterns = (patterns, ancestor) => {
         .split(/\r?\n/)
         .filter(line => line.trim() && !line.startsWith('#'))
         .flatMap(line => {
-        const pattern = line.replace(/(?<!\\)#.*/, '').trim();
+        const pattern = line.replace(/^\\(?=#)/, '').trim();
         if (ancestor && matchFrom) {
             if (pattern.match(matchFrom))
                 return [pattern.replace(matchFrom, '$1')];

package/dist/util/jiti.d.ts

@@ -1,2 +1 @@
-export declare const jitiCJS: any;
-export declare const jitiESM: any;
+export declare const jiti: import("jiti").Jiti;

package/dist/util/jiti.js

@@ -1,17 +1,12 @@
 import { fileURLToPath } from 'node:url';
-import createJITI, {} from 'jiti';
-import { DEFAULT_EXTENSIONS } from '../constants.js';
+import { createJiti } from 'jiti';
 import { join } from './path.js';
 const empty = join(fileURLToPath(import.meta.url), '../empty.js');
 const options = {
-    interopDefault: true,
-    extensions: DEFAULT_EXTENSIONS,
-    esmResolve: false,
     alias: {
         '@rushstack/eslint-config/patch/modern-module-resolution': empty,
         '@rushstack/eslint-patch/modern-module-resolution': empty,
     },
 };
-const createLoader = (options) => createJITI(process.cwd(), options);
-export const jitiCJS = createLoader(options);
-export const jitiESM = createLoader({ ...options, esmResolve: true });
+const createLoader = (options) => createJiti(process.cwd(), options);
+export const jiti = createLoader(options);

package/dist/util/loader.js

@@ -1,9 +1,7 @@
-import { pathToFileURL } from 'node:url';
 import { timerify } from './Performance.js';
 import { LoaderError } from './errors.js';
 import { loadFile, loadJSON, loadTOML, loadYAML, parseJSON, parseYAML } from './fs.js';
-import { isTypeModule } from './fs.js';
-import { jitiCJS, jitiESM } from './jiti.js';
+import { jiti } from './jiti.js';
 import { extname, isInternal } from './path.js';
 const load = async (filePath) => {
     try {
@@ -33,15 +31,7 @@ const load = async (filePath) => {
         if (ext === '.toml') {
             return await loadTOML(filePath);
         }
-        if (ext === '.mjs' || (ext === '.js' && isTypeModule(filePath))) {
-            const fileUrl = pathToFileURL(filePath);
-            const imported = await import(fileUrl.href);
-            return imported.default ?? imported;
-        }
-        if (ext === '.mts' || ((ext === '.ts' || ext === '.tsx') && isTypeModule(filePath))) {
-            return await jitiESM(filePath);
-        }
-        return await jitiCJS(filePath);
+        return await jiti.import(filePath);
     }
     catch (error) {
         throw new LoaderError(`Error loading ${filePath}`, { cause: error });

package/dist/util/remove-export.d.ts

@@ -0,0 +1,8 @@
+interface FixerOptions {
+    text: string;
+    start: number;
+    end: number;
+    flags: number;
+}
+export declare const removeExport: ({ text, start, end, flags }: FixerOptions) => string;
+export {};

package/dist/util/remove-export.js

@@ -0,0 +1,68 @@
+import { FIX_FLAGS } from '../constants.js';
+const getOpeningBracketIndex = (text) => {
+    let bracketOpenIndex = -1;
+    let j = text.length - 1;
+    while (j >= 0) {
+        const char = text[j];
+        if (char === '{') {
+            bracketOpenIndex = j;
+            break;
+        }
+        if (!/\s/.test(char) && char !== ',') {
+            if (text.substring(j - 3, j + 1) === 'type') {
+                j = j - 4;
+                continue;
+            }
+            break;
+        }
+        j--;
+    }
+    return bracketOpenIndex;
+};
+export const removeExport = ({ text, start, end, flags }) => {
+    const beforeStart = text.substring(0, start);
+    const afterEnd = text.substring(end);
+    if (flags % FIX_FLAGS.NONE)
+        return beforeStart + afterEnd;
+    const subject = text.substring(start, end).trim();
+    if (subject === 'export' || subject === 'export default')
+        return beforeStart + afterEnd;
+    let closingBracketOffset = -1;
+    let commaOffset = -1;
+    if (flags & FIX_FLAGS.OBJECT_BINDING) {
+        let i = 0;
+        while (i <= afterEnd.length) {
+            const char = afterEnd[i];
+            if (char === ',') {
+                commaOffset = i + 1;
+            }
+            else if (char === '}') {
+                closingBracketOffset = i + 1;
+                break;
+            }
+            else if (!/\s/.test(char))
+                break;
+            i++;
+        }
+    }
+    if (flags & FIX_FLAGS.EMPTY_DECLARATION && closingBracketOffset !== -1) {
+        const openingBracketIndex = getOpeningBracketIndex(beforeStart);
+        if (closingBracketOffset !== -1 && openingBracketIndex !== -1) {
+            const beforeBracket = beforeStart.substring(0, openingBracketIndex).trim();
+            const exportLength = beforeBracket.endsWith('export') ? 6 : beforeBracket.endsWith('export type') ? 12 : 0;
+            const exportKeywordOffset = beforeBracket.length - exportLength;
+            if (exportLength) {
+                const fromBracket = afterEnd.substring(closingBracketOffset).trim();
+                if (fromBracket.startsWith('from')) {
+                    const specifierQuoteMatch = afterEnd.match(/['"][^'"]+['"]/);
+                    if (specifierQuoteMatch?.index) {
+                        const fromSpecifierLength = specifierQuoteMatch.index + specifierQuoteMatch[0].length;
+                        return beforeBracket.substring(0, exportKeywordOffset) + afterEnd.substring(fromSpecifierLength);
+                    }
+                }
+                return beforeBracket.substring(0, exportKeywordOffset) + afterEnd.substring(closingBracketOffset);
+            }
+        }
+    }
+    return beforeStart + (commaOffset === -1 ? afterEnd : afterEnd.substring(commaOffset));
+};

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "5.30.6";
+export declare const version = "5.31.0";

package/dist/version.js

@@ -1 +1 @@
-export const version = '5.30.6';
+export const version = '5.31.0';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "5.30.6",
+  "version": "5.31.0",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://knip.dev",
   "repository": {
@@ -66,7 +66,7 @@
     "easy-table": "1.2.0",
     "enhanced-resolve": "^5.17.1",
     "fast-glob": "^3.3.2",
-    "jiti": "^1.21.6",
+    "jiti": "^2.1.0",
     "js-yaml": "^4.1.0",
     "minimist": "^1.2.8",
     "picocolors": "^1.0.0",