knip 5.68.0 → 5.69.1

package/dist/graph/analyze.js

@@ -4,7 +4,7 @@ import { getPackageNameFromModuleSpecifier } from '../util/modules.js';
 import { findMatch } from '../util/regex.js';
 import { getShouldIgnoreHandler, getShouldIgnoreTagHandler } from '../util/tag.js';
 import { createAndPrintTrace, printTrace } from '../util/trace.js';
-export const analyze = async ({ analyzedFiles, chief, collector, deputy, entryPaths, factory, fixer, graph, streamer, unreferencedFiles, options, }) => {
+export const analyze = async ({ analyzedFiles, counselor, chief, collector, deputy, entryPaths, factory, graph, streamer, unreferencedFiles, options, }) => {
     const shouldIgnore = getShouldIgnoreHandler(options.isProduction);
     const shouldIgnoreTags = getShouldIgnoreTagHandler(options.tags);
     const isIdentifierReferenced = getIsIdentifierReferencedHandler(graph, entryPaths, options.isTrace);
@@ -75,7 +75,7 @@ export const analyze = async ({ analyzedFiles, chief, collector, deputy, entryPa
                                             if (!isReferenced) {
                                                 if (isIgnored)
                                                     continue;
-                                                const isIssueAdded = collector.addIssue({
+                                                collector.addIssue({
                                                     type: 'enumMembers',
                                                     filePath,
                                                     workspace: workspace.name,
@@ -84,9 +84,8 @@ export const analyze = async ({ analyzedFiles, chief, collector, deputy, entryPa
                                                     pos: member.pos,
                                                     line: member.line,
                                                     col: member.col,
+                                                    fixes: member.fix ? [member.fix] : [],
                                                 });
-                                                if (options.isFix && isIssueAdded && member.fix)
-                                                    fixer.addUnusedTypeNode(filePath, [member.fix]);
                                             }
                                             else if (isIgnored) {
                                                 for (const tagName of exportedItem.jsDocTags) {
@@ -110,7 +109,7 @@ export const analyze = async ({ analyzedFiles, chief, collector, deputy, entryPa
                                             }
                                             continue;
                                         }
-                                        const isIssueAdded = collector.addIssue({
+                                        collector.addIssue({
                                             type: 'classMembers',
                                             filePath,
                                             workspace: workspace.name,
@@ -119,9 +118,8 @@ export const analyze = async ({ analyzedFiles, chief, collector, deputy, entryPa
                                             pos: member.pos,
                                             line: member.line,
                                             col: member.col,
+                                            fixes: member.fix ? [member.fix] : [],
                                         });
-                                        if (options.isFix && isIssueAdded && member.fix)
-                                            fixer.addUnusedTypeNode(filePath, [member.fix]);
                                     }
                                 }
                                 continue;
@@ -138,7 +136,7 @@ export const analyze = async ({ analyzedFiles, chief, collector, deputy, entryPa
                             if (!options.isSkipLibs && principal?.hasExternalReferences(filePath, exportedItem))
                                 continue;
                             const type = getType(hasStrictlyNsRefs, isType);
-                            const isIssueAdded = collector.addIssue({
+                            collector.addIssue({
                                 type,
                                 filePath,
                                 workspace: workspace.name,
@@ -148,13 +146,8 @@ export const analyze = async ({ analyzedFiles, chief, collector, deputy, entryPa
                                 pos: exportedItem.pos,
                                 line: exportedItem.line,
                                 col: exportedItem.col,
+                                fixes: exportedItem.fixes,
                             });
-                            if (options.isFix && isIssueAdded) {
-                                if (isType)
-                                    fixer.addUnusedTypeNode(filePath, exportedItem.fixes);
-                                else
-                                    fixer.addUnusedExportNode(filePath, exportedItem.fixes);
-                            }
                         }
                     }
                 }
@@ -167,7 +160,7 @@ export const analyze = async ({ analyzedFiles, chief, collector, deputy, entryPa
                     for (const symbols of file.duplicates) {
                         if (symbols.length > 1) {
                             const symbol = symbols.map(s => s.symbol).join('|');
-                            collector.addIssue({ type: 'duplicates', filePath, workspace: ws.name, symbol, symbols });
+                            collector.addIssue({ type: 'duplicates', filePath, workspace: ws.name, symbol, symbols, fixes: [] });
                         }
                     }
                 }
@@ -185,13 +178,23 @@ export const analyze = async ({ analyzedFiles, chief, collector, deputy, entryPa
                                 pos: extImport.pos,
                                 line: extImport.line,
                                 col: extImport.col,
+                                fixes: [],
                             });
                     }
                 }
                 if (file.imports?.unresolved) {
                     for (const unresolvedImport of file.imports.unresolved) {
                         const { specifier, pos, line, col } = unresolvedImport;
-                        collector.addIssue({ type: 'unresolved', filePath, workspace: ws.name, symbol: specifier, pos, line, col });
+                        collector.addIssue({
+                            type: 'unresolved',
+                            filePath,
+                            workspace: ws.name,
+                            symbol: specifier,
+                            pos,
+                            line,
+                            col,
+                            fixes: [],
+                        });
                     }
                 }
             }
@@ -213,6 +216,9 @@ export const analyze = async ({ analyzedFiles, chief, collector, deputy, entryPa
             for (const hint of configurationHints)
                 collector.addConfigurationHint(hint);
         }
+        const catalogIssues = await counselor.settleCatalogIssues(options);
+        for (const issue of catalogIssues)
+            collector.addIssue(issue);
         const unusedIgnoredWorkspaces = chief.getUnusedIgnoredWorkspaces();
         for (const identifier of unusedIgnoredWorkspaces) {
             collector.addConfigurationHint({ type: 'ignoreWorkspaces', identifier });

package/dist/graph/build.d.ts

@@ -1,3 +1,4 @@
+import type { CatalogCounselor } from '../CatalogCounselor.js';
 import type { ConfigurationChief, Workspace } from '../ConfigurationChief.js';
 import type { ConsoleStreamer } from '../ConsoleStreamer.js';
 import type { DependencyDeputy } from '../DependencyDeputy.js';
@@ -9,6 +10,7 @@ import type { MainOptions } from '../util/create-options.js';
 interface BuildOptions {
     chief: ConfigurationChief;
     collector: IssueCollector;
+    counselor: CatalogCounselor;
     deputy: DependencyDeputy;
     factory: PrincipalFactory;
     isGitIgnored: (path: string) => boolean;
@@ -16,7 +18,7 @@ interface BuildOptions {
     workspaces: Workspace[];
     options: MainOptions;
 }
-export declare function build({ chief, collector, deputy, factory, isGitIgnored, streamer, workspaces, options, }: BuildOptions): Promise<{
+export declare function build({ chief, collector, counselor, deputy, factory, isGitIgnored, streamer, workspaces, options, }: BuildOptions): Promise<{
     graph: ModuleGraph;
     entryPaths: Set<string>;
     analyzedFiles: Set<string>;

package/dist/graph/build.js

@@ -13,7 +13,7 @@ import { getEntrySpecifiersFromManifest, getManifestImportDependencies } from '.
 import { dirname, extname, isAbsolute, join, relative, toRelative } from '../util/path.js';
 import { augmentWorkspace, getToSourcePathHandler, getToSourcePathsHandler } from '../util/to-source-path.js';
 import { WorkspaceWorker } from '../WorkspaceWorker.js';
-export async function build({ chief, collector, deputy, factory, isGitIgnored, streamer, workspaces, options, }) {
+export async function build({ chief, collector, counselor, deputy, factory, isGitIgnored, streamer, workspaces, options, }) {
     const configFilesMap = new Map();
     const enabledPluginsStore = new Map();
     const toSourceFilePath = getToSourcePathHandler(chief);
@@ -34,6 +34,7 @@ export async function build({ chief, collector, deputy, factory, isGitIgnored, s
             manifest,
             ...chief.getIgnores(name),
         });
+        counselor.addWorkspace(manifest);
     }
     collector.addIgnorePatterns(chief.config.ignore.map(p => join(options.cwd, p)));
     collector.addIgnoreFilesPatterns(chief.config.ignoreFiles.map(p => join(options.cwd, p)));

package/dist/index.js

@@ -1,5 +1,6 @@
 import { watch } from 'node:fs';
 import { formatly } from 'formatly';
+import { CatalogCounselor } from './CatalogCounselor.js';
 import { ConfigurationChief } from './ConfigurationChief.js';
 import { ConsoleStreamer } from './ConsoleStreamer.js';
 import { DependencyDeputy } from './DependencyDeputy.js';
@@ -13,7 +14,6 @@ import { debugLogArray, debugLogObject } from './util/debug.js';
 import { getGitIgnoredHandler } from './util/glob-core.js';
 import { getWatchHandler } from './util/watch.js';
 export const main = async (options) => {
-    const { cwd } = options;
     debugLogObject('*', 'Unresolved configuration', options);
     debugLogObject('*', 'Included issue types', options.includedIssueTypes);
     const chief = new ConfigurationChief(options);
@@ -22,6 +22,7 @@ export const main = async (options) => {
     const streamer = new ConsoleStreamer(options);
     const fixer = new IssueFixer(options);
     const collector = new IssueCollector(options);
+    const counselor = new CatalogCounselor(options);
     streamer.cast('Reading workspace configuration');
     const workspaces = await chief.getWorkspaces();
     const isGitIgnored = await getGitIgnoredHandler(options);
@@ -31,6 +32,7 @@ export const main = async (options) => {
     const { graph, entryPaths, analyzedFiles, unreferencedFiles, analyzeSourceFile } = await build({
         chief,
         collector,
+        counselor,
         deputy,
         factory,
         isGitIgnored,
@@ -40,6 +42,7 @@ export const main = async (options) => {
     });
     const reAnalyze = await analyze({
         analyzedFiles,
+        counselor,
         chief,
         collector,
         deputy,
@@ -74,7 +77,7 @@ export const main = async (options) => {
     if (options.isFix) {
         const touchedFiles = await fixer.fixIssues(issues);
         if (options.isFormat) {
-            const report = await formatly(Array.from(touchedFiles), { cwd });
+            const report = await formatly(Array.from(touchedFiles));
             if (report.ran && report.result && (report.result.runner === 'virtual' || report.result.code === 0)) {
                 debugLogArray('*', `Formatted files using ${report.formatter.name} (${report.formatter.runner})`, touchedFiles);
             }

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

@@ -1,12 +1,3 @@
-import type { IsPluginEnabled, ResolveConfig } from '../../types/config.js';
-import type { PluginConfig } from './types.js';
-declare const _default: {
-    title: string;
-    enablers: string[];
-    isEnabled: IsPluginEnabled;
-    config: string[];
-    entry: string[];
-    production: string[];
-    resolveConfig: ResolveConfig<PluginConfig>;
-};
-export default _default;
+import type { Plugin } from '../../types/config.js';
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/_template/index.js

@@ -10,7 +10,7 @@ const resolveConfig = async (config) => {
     const inputs = config?.plugins ?? [];
     return [...inputs].map(id => toDeferResolve(id));
 };
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
@@ -19,3 +19,4 @@ export default {
     production,
     resolveConfig,
 };
+export default plugin;

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

@@ -1,10 +1,3 @@
-import type { IsPluginEnabled, ResolveConfig } from '../../types/config.js';
-import type { AngularCLIWorkspaceConfiguration } from './types.js';
-declare const _default: {
-    title: string;
-    enablers: string[];
-    isEnabled: IsPluginEnabled;
-    config: string[];
-    resolveConfig: ResolveConfig<AngularCLIWorkspaceConfiguration>;
-};
-export default _default;
+import type { Plugin } from '../../types/config.js';
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/angular/index.js

@@ -108,10 +108,11 @@ const isAngularBuilderRefWithName = ({ builderRef, name }) => {
 };
 const PRODUCTION_CONFIG_NAME = 'production';
 const BUILD_TARGET_NAME = 'build';
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
     config,
     resolveConfig,
 };
+export default plugin;

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

@@ -1,14 +1,4 @@
-import type { IsPluginEnabled, Resolve, ResolveFromAST } from '../../types/config.js';
+import type { Plugin } from '../../types/config.js';
 export declare const config: string[];
-declare const _default: {
-    title: string;
-    enablers: string[];
-    isEnabled: IsPluginEnabled;
-    config: string[];
-    entry: string[];
-    production: string[];
-    resolveFromAST: ResolveFromAST;
-    resolve: Resolve;
-    project: string[];
-};
-export default _default;
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/astro/index.js

@@ -33,7 +33,7 @@ const resolve = options => {
     }
     return inputs;
 };
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
@@ -44,3 +44,4 @@ export default {
     resolve,
     project,
 };
+export default plugin;

package/dist/plugins/astro-db/index.d.ts

@@ -1,8 +1,3 @@
-import type { IsPluginEnabled } from '../../types/config.js';
-declare const _default: {
-    title: string;
-    enablers: string[];
-    isEnabled: IsPluginEnabled;
-    entry: string[];
-};
-export default _default;
+import type { Plugin } from '../../types/config.js';
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/astro-db/index.js

@@ -3,9 +3,10 @@ const title = 'Astro DB';
 const enablers = ['@astrojs/db'];
 const isEnabled = ({ dependencies }) => hasDependency(dependencies, enablers);
 const entry = ['db/config.{js,ts}', 'db/seed.{js,ts}'];
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
     entry,
 };
+export default plugin;

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

@@ -1,11 +1,3 @@
-import type { IsPluginEnabled, ResolveConfig } from '../../types/config.js';
-import type { AvaConfig } from './types.js';
-declare const _default: {
-    title: string;
-    enablers: string[];
-    isEnabled: IsPluginEnabled;
-    config: string[];
-    entry: string[];
-    resolveConfig: ResolveConfig<AvaConfig>;
-};
-export default _default;
+import type { Plugin } from '../../types/config.js';
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/ava/index.js

@@ -25,7 +25,7 @@ const resolveConfig = async (localConfig, options) => {
     const fakeCommand = `node ${nodeArgs.join(' ')} ${requireArgs.join(' ')}`;
     return files.concat(options.getInputsFromScripts(fakeCommand, { knownBinsOnly: true }));
 };
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
@@ -33,3 +33,4 @@ export default {
     entry,
     resolveConfig,
 };
+export default plugin;

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

@@ -1,12 +1,6 @@
-import type { IsPluginEnabled, ResolveConfig } from '../../types/config.js';
+import type { Plugin } from '../../types/config.js';
 import { type Input } from '../../util/input.js';
-import type { BabelConfig, BabelConfigObj } from './types.js';
+import type { BabelConfigObj } from './types.js';
 export declare const getDependenciesFromConfig: (config: BabelConfigObj) => Input[];
-declare const _default: {
-    title: string;
-    enablers: RegExp[];
-    isEnabled: IsPluginEnabled;
-    config: string[];
-    resolveConfig: ResolveConfig<BabelConfig>;
-};
-export default _default;
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/babel/index.js

@@ -26,10 +26,11 @@ const resolveConfig = async (config) => {
         return [];
     return getDependenciesFromConfig(config);
 };
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
     config,
     resolveConfig,
 };
+export default plugin;

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

@@ -1,10 +1,3 @@
-import type { IsPluginEnabled, ResolveConfig } from '../../types/config.js';
-import type { BiomeConfig } from './types.js';
-declare const _default: {
-    title: string;
-    enablers: string[];
-    isEnabled: IsPluginEnabled;
-    config: string[];
-    resolveConfig: ResolveConfig<BiomeConfig>;
-};
-export default _default;
+import type { Plugin } from '../../types/config.js';
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/biome/index.js

@@ -18,10 +18,11 @@ const resolveExtends = (extendsArray, options) => {
 const resolveConfig = (config, options) => {
     return [...resolveExtends(arrayify(config.extends), options)];
 };
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
     config,
     resolveConfig,
 };
+export default plugin;

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

@@ -1,8 +1,3 @@
-import type { IsPluginEnabled } from '../../types/config.js';
-declare const _default: {
-    title: string;
-    enablers: string[];
-    isEnabled: IsPluginEnabled;
-    entry: string[];
-};
-export default _default;
+import type { Plugin } from '../../types/config.js';
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/bumpp/index.js

@@ -4,9 +4,10 @@ const title = 'bumpp';
 const enablers = ['bumpp'];
 const isEnabled = ({ dependencies }) => hasDependency(dependencies, enablers);
 const entry = ['package.json', ...toC12config('bump')];
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
     entry,
 };
+export default plugin;

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

@@ -1,11 +1,3 @@
-import type { ResolveConfig } from '../../types/config.js';
-import type { PackageJson } from '../../types/package-json.js';
-declare const _default: {
-    title: string;
-    enablers: string[];
-    isEnabled: () => boolean;
-    config: string[];
-    packageJsonPath: (id: PackageJson) => PackageJson;
-    resolveConfig: ResolveConfig<PackageJson>;
-};
-export default _default;
+import type { Plugin } from '../../types/config.js';
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/bun/index.js

@@ -18,7 +18,7 @@ const resolveConfig = localConfig => {
     }
     return [];
 };
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
@@ -26,3 +26,4 @@ export default {
     packageJsonPath,
     resolveConfig,
 };
+export default plugin;

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

@@ -1,10 +1,3 @@
-import type { ParsedArgs } from 'minimist';
-declare const _default: {
-    title: string;
-    args: {
-        args: (args: string[]) => string[];
-        boolean: string[];
-        fromArgs: (parsed: ParsedArgs, args: string[]) => string[];
-    };
-};
-export default _default;
+import type { Plugin } from '../../types/config.js';
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/c8/index.js

@@ -5,7 +5,8 @@ const args = {
     boolean: ['all', 'check-coverage', 'clean', 'exclude-after-remap', 'per-file', 'skip-full'],
     fromArgs: (parsed, args) => argsFrom(args, parsed._[0]),
 };
-export default {
+const plugin = {
     title,
     args,
 };
+export default plugin;

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

@@ -1,10 +1,3 @@
-import type { IsPluginEnabled, ResolveConfig } from '../../types/config.js';
-import type { CapacitorConfig } from './types.js';
-declare const _default: {
-    title: string;
-    enablers: RegExp[];
-    isEnabled: IsPluginEnabled;
-    config: string[];
-    resolveConfig: ResolveConfig<CapacitorConfig>;
-};
-export default _default;
+import type { Plugin } from '../../types/config.js';
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/capacitor/index.js

@@ -13,10 +13,11 @@ const resolveConfig = async (config, { configFileDir }) => {
     const ios = (await exists('ios/App/Podfile')) ? ['@capacitor/ios'] : [];
     return [...plugins, ...android, ...ios].map(id => toDependency(id));
 };
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
     config,
     resolveConfig,
 };
+export default plugin;

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

@@ -1,9 +1,3 @@
-import type { IsPluginEnabled } from '../../types/config.js';
-declare const _default: {
-    title: string;
-    enablers: string[];
-    isEnabled: IsPluginEnabled;
-    isRootOnly: true;
-    entry: string[];
-};
-export default _default;
+import type { Plugin } from '../../types/config.js';
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/changelogen/index.js

@@ -5,10 +5,11 @@ const enablers = ['changelogen'];
 const isEnabled = ({ dependencies }) => hasDependency(dependencies, enablers);
 const entry = ['package.json', ...toC12config('changelog')];
 const isRootOnly = true;
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
     isRootOnly,
     entry,
 };
+export default plugin;

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

@@ -1,9 +1,3 @@
-import type { IsPluginEnabled } from '../../types/config.js';
-declare const _default: {
-    title: string;
-    enablers: string[];
-    isEnabled: IsPluginEnabled;
-    isRootOnly: true;
-    entry: string[];
-};
-export default _default;
+import type { Plugin } from '../../types/config.js';
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/changelogithub/index.js

@@ -5,10 +5,11 @@ const enablers = ['changelogithub'];
 const isEnabled = ({ dependencies }) => hasDependency(dependencies, enablers);
 const entry = ['package.json', ...toC12config('changelogithub')];
 const isRootOnly = true;
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
     isRootOnly,
     entry,
 };
+export default plugin;

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

@@ -1,11 +1,3 @@
-import type { IsPluginEnabled, ResolveConfig } from '../../types/config.js';
-import type { ChangesetsConfig } from './types.js';
-declare const _default: {
-    title: string;
-    enablers: string[];
-    isEnabled: IsPluginEnabled;
-    isRootOnly: true;
-    config: string[];
-    resolveConfig: ResolveConfig<ChangesetsConfig>;
-};
-export default _default;
+import type { Plugin } from '../../types/config.js';
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/changesets/index.js

@@ -12,7 +12,7 @@ const resolveConfig = config => {
             ? [config.changelog]
             : []).map(id => toDependency(id));
 };
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
@@ -20,3 +20,4 @@ export default {
     config,
     resolveConfig,
 };
+export default plugin;

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

@@ -1,12 +1,3 @@
-import type { IsPluginEnabled, ResolveConfig } from '../../types/config.js';
-import type { CommitizenConfig } from './types.js';
-declare const _default: {
-    title: string;
-    enablers: string[];
-    isEnabled: IsPluginEnabled;
-    isRootOnly: true;
-    packageJsonPath: string;
-    config: string[];
-    resolveConfig: ResolveConfig<CommitizenConfig>;
-};
-export default _default;
+import type { Plugin } from '../../types/config.js';
+declare const plugin: Plugin;
+export default plugin;

package/dist/plugins/commitizen/index.js

@@ -9,7 +9,7 @@ const config = ['.czrc', '.cz.json', 'package.json'];
 const resolveConfig = config => {
     return config.path ? [toDependency(config.path)] : [];
 };
-export default {
+const plugin = {
     title,
     enablers,
     isEnabled,
@@ -18,3 +18,4 @@ export default {
     config,
     resolveConfig,
 };
+export default plugin;