knip 2.38.3 → 2.38.5

package/README.md

@@ -277,7 +277,7 @@ Knip contains a growing list of plugins:
 - [ESLint][plugin-eslint]
 - [Gatsby][plugin-gatsby]
 - [GitHub Actions][plugin-github-actions]
-- [Graphql Codegen][plugin-graphql-codegen]
+- [GraphQL Codegen][plugin-graphql-codegen]
 - [husky][plugin-husky]
 - [Jest][plugin-jest]
 - [Lefthook][plugin-lefthook]

package/dist/ProjectPrincipal.d.ts

@@ -1,5 +1,6 @@
 import { isGitIgnoredSync } from 'globby';
 import ts from 'typescript';
+import { createCustomModuleResolver } from './typescript/resolveModuleNames.js';
 import { SourceFileManager } from './typescript/SourceFileManager.js';
 import type { SyncCompilers, AsyncCompilers } from './types/compilers.js';
 import type { ExportItem, ExportItemMember } from './types/exports.js';
@@ -21,7 +22,7 @@ export declare class ProjectPrincipal {
     backend: {
         fileManager: SourceFileManager;
         compilerHost: ts.CompilerHost;
-        languageServiceHost: ts.LanguageServiceHost;
+        resolveModuleNames: ReturnType<typeof createCustomModuleResolver>;
         lsFindReferences: ts.LanguageService['findReferences'];
         program?: ts.Program;
     };
@@ -53,7 +54,7 @@ export declare class ProjectPrincipal {
         };
         scripts: Set<string>;
     };
-    resolveModule(specifier: string, filePath?: string): ts.ResolvedModuleFull | undefined;
+    resolveModule(specifier: string, filePath?: string): ts.ResolvedModule | undefined;
     getHasReferences(filePath: string, exportedItem: ExportItem): {
         external: boolean;
         internal: boolean;

package/dist/ProjectPrincipal.js

@@ -5,6 +5,7 @@ import { IGNORED_FILE_EXTENSIONS } from './constants.js';
 import { getJSDocTags, getLineAndCharacterOfPosition, isInModuleBlock } from './typescript/ast-helpers.js';
 import { createHosts } from './typescript/createHosts.js';
 import { getImportsAndExports } from './typescript/getImportsAndExports.js';
+import { createCustomModuleResolver } from './typescript/resolveModuleNames.js';
 import { SourceFileManager } from './typescript/SourceFileManager.js';
 import { isMaybePackageName, sanitizeSpecifier } from './util/modules.js';
 import { dirname, extname, isInNodeModules, join } from './util/path.js';
@@ -46,7 +47,7 @@ export class ProjectPrincipal {
         this.extensions = new Set([...DEFAULT_EXTENSIONS, ...syncCompilers.keys(), ...asyncCompilers.keys()]);
         this.syncCompilers = syncCompilers;
         this.asyncCompilers = asyncCompilers;
-        const { fileManager, compilerHost, languageServiceHost } = createHosts({
+        const { fileManager, compilerHost, languageServiceHost, resolveModuleNames } = createHosts({
             cwd: this.cwd,
             compilerOptions: this.compilerOptions,
             entryPaths: this.entryPaths,
@@ -57,7 +58,7 @@ export class ProjectPrincipal {
         this.backend = {
             fileManager,
             compilerHost,
-            languageServiceHost,
+            resolveModuleNames,
             lsFindReferences,
         };
     }
@@ -129,15 +130,14 @@ export class ProjectPrincipal {
             }
             else {
                 const sanitizedSpecifier = sanitizeSpecifier(specifier);
-                const ext = extname(sanitizedSpecifier);
-                const hasIgnoredExtension = IGNORED_FILE_EXTENSIONS.includes(ext);
-                if (!hasIgnoredExtension && isMaybePackageName(sanitizedSpecifier)) {
+                if (isMaybePackageName(sanitizedSpecifier)) {
                     external.add(sanitizedSpecifier);
                 }
                 else {
                     const isIgnored = this.isGitIgnored(join(dirname(filePath), sanitizedSpecifier));
                     if (!isIgnored) {
                         const ext = extname(sanitizedSpecifier);
+                        const hasIgnoredExtension = IGNORED_FILE_EXTENSIONS.includes(ext);
                         if (!ext || (ext !== '.json' && !hasIgnoredExtension)) {
                             unresolvedImports.add(specifier);
                         }
@@ -156,8 +156,7 @@ export class ProjectPrincipal {
         };
     }
     resolveModule(specifier, filePath = specifier) {
-        const module = ts.resolveModuleName(specifier, filePath, this.compilerOptions, this.backend.languageServiceHost);
-        return module?.resolvedModule;
+        return this.backend.resolveModuleNames([specifier], filePath)[0];
     }
     getHasReferences(filePath, exportedItem) {
         const hasReferences = { external: false, internal: false };

package/dist/binaries/resolvers/fallback.js

@@ -4,7 +4,9 @@ import { toBinary } from '../../util/protocols.js';
 import { tryResolveFilePath, tryResolveSpecifiers } from '../util.js';
 const withPositional = parsed => [parsed._[0], parsed.require].flat();
 const withoutPositional = parsed => [parsed.require].flat();
+const withoutRequire = () => [];
 const argFilters = {
+    adb: withoutRequire,
     'babel-node': withPositional,
     esbuild: withPositional,
     execa: withPositional,

package/dist/constants.d.ts

@@ -5,6 +5,7 @@ export declare const DEFAULT_EXTENSIONS: string[];
 export declare const GLOBAL_IGNORE_PATTERNS: string[];
 export declare const IGNORED_GLOBAL_BINARIES: string[];
 export declare const IGNORED_DEPENDENCIES: string[];
+export declare const VIRTUAL_FILE_EXTENSIONS: string[];
 export declare const IGNORED_FILE_EXTENSIONS: string[];
 export declare const IGNORE_DEFINITELY_TYPED: string[];
 export declare const ISSUE_TYPES: IssueType[];

package/dist/constants.js

@@ -32,9 +32,10 @@ export const IGNORED_GLOBAL_BINARIES = [
     'yarn',
 ];
 export const IGNORED_DEPENDENCIES = ['knip', 'typescript'];
+export const VIRTUAL_FILE_EXTENSIONS = ['.css', '.html', '.svg'];
 export const IGNORED_FILE_EXTENSIONS = [
+    ...VIRTUAL_FILE_EXTENSIONS,
     '.avif',
-    '.css',
     '.eot',
     '.gif',
     '.ico',
@@ -44,7 +45,6 @@ export const IGNORED_FILE_EXTENSIONS = [
     '.png',
     '.sass',
     '.scss',
-    '.svg',
     '.sh',
     '.ttf',
     '.webp',

package/dist/plugins/graphql-codegen/index.d.ts

@@ -1,5 +1,5 @@
 import type { IsPluginEnabledCallback, GenericPluginCallback } from '../../types/plugins.js';
-export declare const NAME = "Graphql Codegen";
+export declare const NAME = "GraphQL Codegen";
 export declare const ENABLERS: RegExp[];
 export declare const PACKAGE_JSON_PATH = "codegen";
 export declare const isEnabled: IsPluginEnabledCallback;

package/dist/plugins/graphql-codegen/index.js

@@ -1,7 +1,7 @@
 import { timerify } from '../../util/Performance.js';
 import { hasDependency, load } from '../../util/plugin.js';
 import { isConfigurationOutput } from './types.js';
-export const NAME = 'Graphql Codegen';
+export const NAME = 'GraphQL Codegen';
 export const ENABLERS = [/^@graphql-codegen\//];
 export const PACKAGE_JSON_PATH = 'codegen';
 export const isEnabled = ({ dependencies }) => hasDependency(dependencies, ENABLERS);
@@ -23,12 +23,10 @@ const findPluginDependencies = async (configFilePath, options) => {
         .map(presetName => `@graphql-codegen/${presetName}${presetName.endsWith('-preset') ? '' : '-preset'}`);
     const flatPlugins = generateSet
         .filter((config) => !isConfigurationOutput(config))
-        .map(item => Object.keys(item))
-        .flat()
+        .flatMap(item => Object.keys(item))
         .map(plugin => `@graphql-codegen/${plugin}`);
     const nestedPlugins = configurationOutput
-        .map(configOutput => (configOutput.plugins ? configOutput.plugins : []))
-        .flat()
+        .flatMap(configOutput => (configOutput.plugins ? configOutput.plugins : []))
         .map(plugin => `@graphql-codegen/${plugin}`);
     return [...presets, ...flatPlugins, ...nestedPlugins];
 };

package/dist/plugins/husky/index.js

@@ -1,9 +1,8 @@
-import { readFileSync } from 'fs';
 import { _getDependenciesFromScripts } from '../../binaries/index.js';
 import { getGitHookPaths } from '../../util/git.js';
 import { FAKE_PATH } from '../../util/loader.js';
 import { timerify } from '../../util/Performance.js';
-import { hasDependency } from '../../util/plugin.js';
+import { hasDependency, loadFile } from '../../util/plugin.js';
 export const NAME = 'husky';
 export const ENABLERS = ['husky'];
 export const isEnabled = ({ dependencies }) => hasDependency(dependencies, ENABLERS);
@@ -13,7 +12,9 @@ const findHuskyDependencies = async (configFilePath, options) => {
     const { cwd, manifest, isProduction } = options;
     if (isProduction || configFilePath === FAKE_PATH)
         return [];
-    const script = readFileSync(configFilePath);
+    const script = await loadFile(configFilePath);
+    if (!script)
+        return [];
     return _getDependenciesFromScripts(String(script), {
         cwd,
         manifest,

package/dist/plugins/lefthook/index.js

@@ -1,9 +1,8 @@
-import { readFileSync } from 'fs';
 import { _getDependenciesFromScripts } from '../../binaries/index.js';
 import { getGitHookPaths } from '../../util/git.js';
 import { getValuesByKeyDeep } from '../../util/object.js';
 import { timerify } from '../../util/Performance.js';
-import { hasDependency, load } from '../../util/plugin.js';
+import { hasDependency, load, loadFile } from '../../util/plugin.js';
 import { fromBinary } from '../../util/protocols.js';
 export const NAME = 'Lefthook';
 export const ENABLERS = ['lefthook', '@arkweid/lefthook', '@evilmartians/lefthook'];
@@ -23,7 +22,9 @@ const findLefthookDependencies = async (configFilePath, options) => {
         const lefthook = process.env.CI ? ENABLERS.filter(dependency => dependencies.includes(dependency)) : [];
         return [...lefthook, ..._getDependenciesFromScripts(scripts, { cwd, manifest, knownGlobalsOnly: true })];
     }
-    const script = readFileSync(configFilePath, 'utf8');
+    const script = await loadFile(configFilePath);
+    if (!script)
+        return [];
     const scriptDependencies = _getDependenciesFromScripts([script], { cwd, manifest, knownGlobalsOnly: false });
     const matches = scriptDependencies.find(dep => dependencies.includes(fromBinary(dep)));
     return matches ? [matches] : [];

package/dist/typescript/createHosts.d.ts

@@ -11,5 +11,6 @@ export declare const createHosts: ({ cwd, compilerOptions, entryPaths, compilers
     fileManager: SourceFileManager;
     languageServiceHost: ts.LanguageServiceHost;
     compilerHost: ts.CompilerHost;
+    resolveModuleNames: (moduleNames: string[], containingFile: string) => (ts.ResolvedModule | undefined)[];
 };
 export {};

package/dist/typescript/createHosts.js

@@ -1,6 +1,7 @@
 import { EOL } from 'node:os';
 import path from 'node:path';
 import ts from 'typescript';
+import { VIRTUAL_FILE_EXTENSIONS } from '../constants.js';
 import { createCustomModuleResolver } from './resolveModuleNames.js';
 import { SourceFileManager } from './SourceFileManager.js';
 import { createCustomSys } from './sys.js';
@@ -8,7 +9,7 @@ const libLocation = path.dirname(ts.getDefaultLibFilePath({}));
 const fileManager = new SourceFileManager();
 export const createHosts = ({ cwd, compilerOptions, entryPaths, compilers }) => {
     fileManager.installCompilers(compilers);
-    const virtualFileExtensions = [...compilers[0].keys(), ...compilers[1].keys()];
+    const virtualFileExtensions = [...VIRTUAL_FILE_EXTENSIONS, ...compilers[0].keys(), ...compilers[1].keys()];
     const sys = createCustomSys(cwd, virtualFileExtensions);
     const resolveModuleNames = createCustomModuleResolver(sys, compilerOptions, virtualFileExtensions);
     const languageServiceHost = {
@@ -35,5 +36,5 @@ export const createHosts = ({ cwd, compilerOptions, entryPaths, compilers }) =>
         fileExists: languageServiceHost.fileExists,
         resolveModuleNames: languageServiceHost.resolveModuleNames,
     };
-    return { fileManager, languageServiceHost, compilerHost };
+    return { fileManager, languageServiceHost, compilerHost, resolveModuleNames };
 };

package/dist/typescript/resolveModuleNames.js

@@ -1,11 +1,11 @@
 import { existsSync } from 'node:fs';
 import ts from 'typescript';
 import { sanitizeSpecifier } from '../util/modules.js';
-import { dirname, extname, isInternal, join } from '../util/path.js';
+import { dirname, extname, isAbsolute, isInternal, join } from '../util/path.js';
 import { isDeclarationFileExtension } from './ast-helpers.js';
 import { ensureRealFilePath, isVirtualFilePath } from './utils.js';
 const simpleResolver = (name, containingFile) => {
-    const resolvedFileName = join(dirname(containingFile), name);
+    const resolvedFileName = isAbsolute(name) ? name : join(dirname(containingFile), name);
     if (existsSync(resolvedFileName)) {
         return {
             resolvedFileName,

package/dist/util/loader.d.ts

@@ -1,2 +1,3 @@
 export declare const FAKE_PATH = "__FAKE__";
 export declare const _load: (filePath: string) => Promise<any>;
+export declare const _loadFile: (filePath: string) => Promise<string | undefined>;

package/dist/util/loader.js

@@ -25,10 +25,21 @@ const load = async (filePath) => {
             const imported = await import(fileUrl.href);
             return imported.default ?? imported;
         }
-        return jiti(filePath);
+        return await jiti(filePath);
+    }
+    catch (error) {
+        throw new LoaderError(`Error loading ${filePath}`, { cause: error });
+    }
+};
+const loadFileAsync = async (filePath) => {
+    if (filePath === FAKE_PATH)
+        return;
+    try {
+        return await loadFile(filePath);
     }
     catch (error) {
         throw new LoaderError(`Error loading ${filePath}`, { cause: error });
     }
 };
 export const _load = timerify(load);
+export const _loadFile = timerify(loadFileAsync);

package/dist/util/path.d.ts

@@ -1,3 +1,4 @@
+export declare const isAbsolute: (path: string) => boolean;
 export declare const dirname: (path: string) => string;
 export declare const extname: (path: string) => string;
 export declare const join: (...paths: string[]) => string;

package/dist/util/path.js

@@ -1,7 +1,7 @@
 import path from 'node:path';
 import parsedArgValues from './cli-arguments.js';
 const { directory } = parsedArgValues;
-const isAbsolute = path.isAbsolute;
+export const isAbsolute = path.isAbsolute;
 export const dirname = path.posix.dirname;
 export const extname = path.posix.extname;
 export const join = path.posix.join;

package/dist/util/plugin.d.ts

@@ -1,4 +1,4 @@
-export { _load as load } from './loader.js';
+export { _load as load, _loadFile as loadFile } from './loader.js';
 import type { RawPluginConfiguration } from '../types/config.js';
 export declare const toCamelCase: (name: string) => string;
 export declare const hasDependency: (dependencies: Set<string>, values: (string | RegExp)[]) => boolean;

package/dist/util/plugin.js

@@ -1,4 +1,4 @@
-export { _load as load } from './loader.js';
+export { _load as load, _loadFile as loadFile } from './loader.js';
 import { arrayify } from './array.js';
 export const toCamelCase = (name) => name.toLowerCase().replace(/(-[a-z])/g, group => group.toUpperCase().replace('-', ''));
 export const hasDependency = (dependencies, values) => values.some(value => {

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "2.38.3";
+export declare const version = "2.38.5";

package/dist/version.js

@@ -1 +1 @@
-export const version = '2.38.3';
+export const version = '2.38.5';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "2.38.3",
+  "version": "2.38.5",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://github.com/webpro/knip",
   "repository": "github:webpro/knip",
@@ -46,13 +46,13 @@
     "@npmcli/map-workspaces": "^3.0.4",
     "@pkgjs/parseargs": "0.11.0",
     "@pnpm/logger": "5.0.0",
-    "@pnpm/workspace.pkgs-graph": "2.0.9",
+    "@pnpm/workspace.pkgs-graph": "2.0.10",
     "@snyk/github-codeowners": "^1.1.0",
     "chalk": "^5.2.0",
     "easy-table": "^1.2.0",
     "fast-glob": "^3.2.12",
     "globby": "^13.1.3",
-    "jiti": "^1.19.3",
+    "jiti": "1.21.0",
     "js-yaml": "^4.1.0",
     "micromatch": "^4.0.5",
     "minimist": "^1.2.8",
@@ -73,12 +73,12 @@
     "@types/js-yaml": "4.0.8",
     "@types/micromatch": "4.0.4",
     "@types/minimist": "1.2.4",
-    "@types/node": "20.8.9",
+    "@types/node": "20.8.10",
     "@types/npmcli__map-workspaces": "3.0.3",
     "@types/pkgjs__parseargs": "0.10.2",
     "@types/webpack": "5.28.4",
-    "@typescript-eslint/eslint-plugin": "6.9.0",
-    "@typescript-eslint/parser": "6.9.0",
+    "@typescript-eslint/eslint-plugin": "6.9.1",
+    "@typescript-eslint/parser": "6.9.1",
     "c8": "8.0.1",
     "eslint": "8.52.0",
     "eslint-import-resolver-typescript": "3.6.1",