knip 2.12.2 → 2.12.3

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/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.2";
+export declare const version = "2.12.3";

package/dist/version.js

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "2.12.2",
+  "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",