knip 2.12.1 → 2.12.3

package/README.md

@@ -181,21 +181,21 @@ This example shows more output related to unused and unlisted dependencies:
 
 The report contains the following types of issues:
 
-| Key            | Title                               | Description                                           |
-| :------------- | :---------------------------------- | :---------------------------------------------------- |
-| `files`        | Unused files                        | unable to find references to this file                |
-| `dependencies` | Unused dependencies                 | unable to find references to this dependency          |
-| `dependencies` | Unused devDependencies              | unable to find references to this dependency          |
-| `unlisted`     | Unlisted dependencies               | used dependencies not listed in package.json          |
-| `binaries`     | Unlisted binaries                   | binaries from dependencies not in package.json        |
-| `unresolved`   | Unresolved imports                  | unable to resolve this (import) specifier             |
-| `exports`      | Unused exports                      | unable to find references to this export              |
-| `nsExports`    | Unused exports in namespaces        | unable to find direct references to this export _(1)_ |
-| `types`        | Unused exported types               | unable to find references to this exported type       |
-| `nsTypes`      | Unused exported types in namespaces | unable to find direct references to this export _(1)_ |
-| `enumMembers`  | Unused exported enum members        | unable to find references to this enum member         |
-| `classMembers` | Unused exported class members       | unable to find references to this class member        |
-| `duplicates`   | Duplicate exports                   | the same thing is exported more than once             |
+| Key               | Title                               | Description                                           |
+| :---------------- | :---------------------------------- | :---------------------------------------------------- |
+| `files`           | Unused files                        | unable to find references to this file                |
+| `dependencies`    | Unused dependencies                 | unable to find references to this dependency          |
+| `devDependencies` | Unused devDependencies              | unable to find references to this devDependency       |
+| `unlisted`        | Unlisted dependencies               | used dependencies not listed in package.json          |
+| `binaries`        | Unlisted binaries                   | binaries from dependencies not in package.json        |
+| `unresolved`      | Unresolved imports                  | unable to resolve this (import) specifier             |
+| `exports`         | Unused exports                      | unable to find references to this export              |
+| `nsExports`       | Unused exports in namespaces        | unable to find direct references to this export _(1)_ |
+| `types`           | Unused exported types               | unable to find references to this exported type       |
+| `nsTypes`         | Unused exported types in namespaces | unable to find direct references to this export _(1)_ |
+| `enumMembers`     | Unused exported enum members        | unable to find references to this enum member         |
+| `classMembers`    | Unused exported class members       | unable to find references to this class member        |
+| `duplicates`      | Duplicate exports                   | the same thing is exported more than once             |
 
 When an issue type has zero issues, it is not shown.
 

package/dist/ConfigurationChief.d.ts

@@ -36,6 +36,7 @@ export declare class ConfigurationChief {
             dependencies: import("./types/issues.js").IssueSeverity;
             devDependencies: import("./types/issues.js").IssueSeverity;
             unlisted: import("./types/issues.js").IssueSeverity;
+            binaries: import("./types/issues.js").IssueSeverity;
             unresolved: import("./types/issues.js").IssueSeverity;
             exports: import("./types/issues.js").IssueSeverity;
             types: import("./types/issues.js").IssueSeverity;
@@ -44,7 +45,6 @@ export declare class ConfigurationChief {
             duplicates: import("./types/issues.js").IssueSeverity;
             enumMembers: import("./types/issues.js").IssueSeverity;
             classMembers: import("./types/issues.js").IssueSeverity;
-            binaries: import("./types/issues.js").IssueSeverity;
         };
         include: string[];
         exclude: string[];

package/dist/ConfigurationValidator.d.ts

@@ -1,7 +1,7 @@
 import { z } from 'zod';
 export declare const ConfigurationValidator: z.ZodObject<{
     exclude: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
-    rules: z.ZodOptional<z.ZodRecord<z.ZodUnion<[z.ZodLiteral<"files">, z.ZodLiteral<"dependencies">, z.ZodLiteral<"devDependencies">, z.ZodLiteral<"unlisted">, z.ZodLiteral<"unresolved">, z.ZodLiteral<"exports">, z.ZodLiteral<"types">, z.ZodLiteral<"nsExports">, z.ZodLiteral<"nsTypes">, z.ZodLiteral<"duplicates">, z.ZodLiteral<"enumMembers">, z.ZodLiteral<"classMembers">]>, z.ZodEnum<["error", "warn", "off"]>>>;
+    rules: z.ZodOptional<z.ZodRecord<z.ZodUnion<[z.ZodLiteral<"files">, z.ZodLiteral<"dependencies">, z.ZodLiteral<"devDependencies">, z.ZodLiteral<"unlisted">, z.ZodLiteral<"binaries">, z.ZodLiteral<"unresolved">, z.ZodLiteral<"exports">, z.ZodLiteral<"types">, z.ZodLiteral<"nsExports">, z.ZodLiteral<"nsTypes">, z.ZodLiteral<"duplicates">, z.ZodLiteral<"enumMembers">, z.ZodLiteral<"classMembers">]>, z.ZodEnum<["error", "warn", "off"]>>>;
     entry: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
     project: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
     paths: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodArray<z.ZodString, "many">>>;
@@ -1405,7 +1405,7 @@ export declare const ConfigurationValidator: z.ZodObject<{
     }>]>>;
 }, "strip", z.ZodTypeAny, {
     exclude?: string[] | undefined;
-    rules?: Partial<Record<"files" | "dependencies" | "devDependencies" | "unlisted" | "unresolved" | "exports" | "types" | "nsExports" | "nsTypes" | "duplicates" | "enumMembers" | "classMembers", "error" | "warn" | "off">> | undefined;
+    rules?: Partial<Record<"files" | "dependencies" | "devDependencies" | "unlisted" | "binaries" | "unresolved" | "exports" | "types" | "nsExports" | "nsTypes" | "duplicates" | "enumMembers" | "classMembers", "error" | "warn" | "off">> | undefined;
     entry?: string | string[] | undefined;
     project?: string | string[] | undefined;
     paths?: Record<string, string[]> | undefined;
@@ -1807,7 +1807,7 @@ export declare const ConfigurationValidator: z.ZodObject<{
     } | undefined;
 }, {
     exclude?: string[] | undefined;
-    rules?: Partial<Record<"files" | "dependencies" | "devDependencies" | "unlisted" | "unresolved" | "exports" | "types" | "nsExports" | "nsTypes" | "duplicates" | "enumMembers" | "classMembers", "error" | "warn" | "off">> | undefined;
+    rules?: Partial<Record<"files" | "dependencies" | "devDependencies" | "unlisted" | "binaries" | "unresolved" | "exports" | "types" | "nsExports" | "nsTypes" | "duplicates" | "enumMembers" | "classMembers", "error" | "warn" | "off">> | undefined;
     entry?: string | string[] | undefined;
     project?: string | string[] | undefined;
     paths?: Record<string, string[]> | undefined;

package/dist/ConfigurationValidator.js

@@ -10,6 +10,7 @@ const issueTypeSchema = z.union([
     z.literal('dependencies'),
     z.literal('devDependencies'),
     z.literal('unlisted'),
+    z.literal('binaries'),
     z.literal('unresolved'),
     z.literal('exports'),
     z.literal('types'),

package/dist/binaries/index.js

@@ -12,9 +12,7 @@ const getDependenciesFromScripts = (npmScripts, options = {}) => {
     return compact(results.map(identifier => {
         if (isBinary(identifier) || isInternal(identifier))
             return identifier;
-        const packageName = getPackageNameFromModuleSpecifier(identifier);
-        if (!packageName.startsWith('.'))
-            return packageName;
+        return getPackageNameFromModuleSpecifier(identifier);
     }));
 };
 export const _getDependenciesFromScripts = timerify(getDependenciesFromScripts);

package/dist/binaries/util.js

@@ -6,10 +6,13 @@ export const tryResolveFilePath = (cwd, specifier, acceptModuleSpecifier) => {
         const filePath = join(cwd, specifier);
         if (!isInNodeModules(filePath)) {
             const resolvedFilePath = _tryResolve(filePath, cwd);
-            if (resolvedFilePath)
+            if (resolvedFilePath) {
                 return [resolvedFilePath];
-            else if (acceptModuleSpecifier)
-                return [getPackageNameFromModuleSpecifier(specifier)];
+            }
+            else if (acceptModuleSpecifier) {
+                const packageName = getPackageNameFromModuleSpecifier(specifier);
+                return packageName ? [packageName] : [];
+            }
         }
         else {
             return [getPackageNameFromFilePath(specifier)];

package/dist/index.js

@@ -63,10 +63,10 @@ export const main = async (unresolvedConfiguration) => {
                 const packageName = isInNodeModules(specifier)
                     ? getPackageNameFromFilePath(specifier)
                     : getPackageNameFromModuleSpecifier(specifier);
-                const isHandled = deputy.maybeAddReferencedExternalDependency(workspace, packageName);
+                const isHandled = packageName && deputy.maybeAddReferencedExternalDependency(workspace, packageName);
                 if (!isHandled)
                     collector.addIssue({ type: 'unlisted', filePath: containingFilePath, symbol: specifier });
-                if (specifier !== packageName) {
+                if (packageName && specifier !== packageName) {
                     const otherWorkspace = chief.findWorkspaceByPackageName(packageName);
                     if (otherWorkspace) {
                         const filePath = _resolveSpecifier(otherWorkspace.dir, specifier);
@@ -194,7 +194,7 @@ export const main = async (unresolvedConfiguration) => {
                     exportedSymbols.set(filePath, exported);
                 for (const [specifierFilePath, importItems] of internal.entries()) {
                     const packageName = getPackageNameFromModuleSpecifier(importItems.specifier);
-                    if (chief.localWorkspaces.has(packageName))
+                    if (packageName && chief.localWorkspaces.has(packageName))
                         external.add(packageName);
                     if (!importedSymbols.has(specifierFilePath)) {
                         importedSymbols.set(specifierFilePath, importItems);
@@ -220,7 +220,7 @@ export const main = async (unresolvedConfiguration) => {
                 });
                 external.forEach(specifier => {
                     const packageName = getPackageNameFromModuleSpecifier(specifier);
-                    const isHandled = deputy.maybeAddReferencedExternalDependency(workspace, packageName);
+                    const isHandled = packageName && deputy.maybeAddReferencedExternalDependency(workspace, packageName);
                     if (!isHandled)
                         collector.addIssue({ type: 'unlisted', filePath, symbol: specifier });
                 });

package/dist/plugins/eslint/helpers.js

@@ -57,10 +57,12 @@ const resolveExtendsSpecifier = (specifier) => {
         .replace(/\/(eslint-)?(recommended.*|strict|all)$/, '');
     if (/eslint-(config|plugin)-/.test(strippedSpecifier))
         return strippedSpecifier;
-    const pluginName = getPackageNameFromModuleSpecifier(strippedSpecifier);
-    if (pluginName === 'eslint')
+    const packageName = getPackageNameFromModuleSpecifier(strippedSpecifier);
+    if (!packageName)
         return;
-    return resolvePackageName(specifier.startsWith('plugin:') ? 'eslint-plugin' : 'eslint-config', pluginName);
+    if (packageName === 'eslint')
+        return;
+    return resolvePackageName(specifier.startsWith('plugin:') ? 'eslint-plugin' : 'eslint-config', packageName);
 };
 const getDependenciesFromSettings = (settings = {}) => {
     return compact(Object.entries(settings).flatMap(([settingKey, settings]) => {

package/dist/typescript/getImportsAndExports.js

@@ -1,6 +1,7 @@
 import { isBuiltin } from 'node:module';
 import ts from 'typescript';
 import { getOrSet } from '../util/map.js';
+import { isMaybePackageName } from '../util/modules.js';
 import { isInNodeModules } from '../util/path.js';
 import { isDeclarationFileExtension, isAccessExpression, getAccessExpressionName } from './ast-helpers.js';
 import getExportVisitors from './visitors/exports/index.js';
@@ -56,7 +57,9 @@ export const getImportsAndExports = (sourceFile, options) => {
                     if (!isInNodeModules(filePath)) {
                         addInternalImport({ identifier, specifier, symbol, filePath, isDynamic, isReExport });
                     }
-                    else if (isDeclarationFileExtension(module.resolvedModule.extension)) {
+                    if (!isMaybePackageName(specifier))
+                        return;
+                    if (isDeclarationFileExtension(module.resolvedModule.extension)) {
                         externalImports.add(specifier);
                     }
                     else {

package/dist/util/compilers.d.ts

@@ -4,7 +4,7 @@ export declare const partitionCompilers: (rawLocalConfig: RawConfiguration) => {
     syncCompilers: Record<string, SyncCompilerFn>;
     asyncCompilers: Record<string, AsyncCompilerFn>;
     exclude?: string[] | undefined;
-    rules?: Partial<Record<"files" | "dependencies" | "devDependencies" | "unlisted" | "unresolved" | "exports" | "types" | "nsExports" | "nsTypes" | "duplicates" | "enumMembers" | "classMembers", "error" | "warn" | "off">> | undefined;
+    rules?: Partial<Record<"files" | "dependencies" | "devDependencies" | "unlisted" | "binaries" | "unresolved" | "exports" | "types" | "nsExports" | "nsTypes" | "duplicates" | "enumMembers" | "classMembers", "error" | "warn" | "off">> | undefined;
     entry?: string | string[] | undefined;
     project?: string | string[] | undefined;
     paths?: Record<string, string[]> | undefined;

package/dist/util/modules.d.ts

@@ -1,6 +1,7 @@
 import type { PackageJson } from '@npmcli/package-json';
-export declare const getPackageNameFromModuleSpecifier: (moduleSpecifier: string) => string;
+export declare const getPackageNameFromModuleSpecifier: (moduleSpecifier: string) => string | undefined;
 export declare const getPackageNameFromFilePath: (value: string) => string;
+export declare const isMaybePackageName: (specifier: string) => boolean;
 export declare const isDefinitelyTyped: (packageName: string) => boolean;
 export declare const getDefinitelyTypedFor: (packageName: string) => string;
 export declare const getPackageFromDefinitelyTyped: (typedDependency: string) => string;

package/dist/util/modules.js

@@ -2,6 +2,8 @@ import { _glob } from './glob.js';
 import { getStringValues } from './object.js';
 import { toPosix } from './path.js';
 export const getPackageNameFromModuleSpecifier = (moduleSpecifier) => {
+    if (!isMaybePackageName(moduleSpecifier))
+        return;
     const parts = moduleSpecifier.split('/').slice(0, 2);
     return moduleSpecifier.startsWith('@') ? parts.join('/') : parts[0];
 };
@@ -11,6 +13,7 @@ export const getPackageNameFromFilePath = (value) => {
         return match[match.length - 1];
     return value;
 };
+export const isMaybePackageName = (specifier) => /^@?[a-z0-9]/.test(specifier);
 export const isDefinitelyTyped = (packageName) => packageName.startsWith('@types/');
 export const getDefinitelyTypedFor = (packageName) => {
     if (isDefinitelyTyped(packageName))

package/dist/util/require.js

@@ -21,8 +21,11 @@ const resolveSpecifier = (dir, specifier) => {
         return toPosix(require.resolve(specifier));
     }
     catch {
-        const relativeSpecifier = specifier.replace(getPackageNameFromModuleSpecifier(specifier), '.');
-        return tryResolve(join(dir, relativeSpecifier), dir);
+        const packageName = getPackageNameFromModuleSpecifier(specifier);
+        if (packageName) {
+            const relativeSpecifier = specifier.replace(packageName, '.');
+            return tryResolve(join(dir, relativeSpecifier), dir);
+        }
     }
 };
 export const _require = timerify(require);

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "2.12.1";
+export declare const version = "2.12.3";

package/dist/version.js

@@ -1 +1 @@
-export const version = '2.12.1';
+export const version = '2.12.3';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "2.12.1",
+  "version": "2.12.3",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://github.com/webpro/knip",
   "repository": "github:webpro/knip",