knip 6.9.0 → 6.10.0

package/dist/DependencyDeputy.d.ts

@@ -35,6 +35,7 @@ export declare class DependencyDeputy {
         optionalPeerDependencies: DependencySet;
         requiredPeerDependencies: DependencyArray;
         allDependencies: DependencySet;
+        engines: Record<string, string>;
         ignoreDependencies: (string | RegExp)[];
         ignoreBinaries: (string | RegExp)[];
         ignoreUnresolved: (string | RegExp)[];
@@ -57,7 +58,7 @@ export declare class DependencyDeputy {
         isPeerOptional: boolean;
     }[];
     getOptionalPeerDependencies(workspaceName: string): DependencySet;
-    maybeAddReferencedExternalDependency(workspace: Workspace, packageName: string, isDevOnly?: boolean): boolean;
+    maybeAddReferencedExternalDependency(workspace: Workspace, packageName: string, isDevOnly?: boolean, isTypeOnly?: boolean): boolean;
     maybeAddReferencedBinary(workspace: Workspace, binaryName: string): Set<string> | undefined;
     private isInDependencies;
     settleDependencyIssues(): {

package/dist/DependencyDeputy.js

@@ -73,6 +73,7 @@ export class DependencyDeputy {
             optionalPeerDependencies,
             requiredPeerDependencies,
             allDependencies: new Set(allDependencies),
+            engines: manifest.engines ?? {},
         });
     }
     getWorkspaceManifest(workspaceName) {
@@ -128,7 +129,7 @@ export class DependencyDeputy {
     getOptionalPeerDependencies(workspaceName) {
         return this._manifests.get(workspaceName)?.optionalPeerDependencies ?? new Set();
     }
-    maybeAddReferencedExternalDependency(workspace, packageName, isDevOnly) {
+    maybeAddReferencedExternalDependency(workspace, packageName, isDevOnly, isTypeOnly) {
         if (!this.isReportDependencies)
             return true;
         if (isBuiltin(packageName))
@@ -141,13 +142,16 @@ export class DependencyDeputy {
         const closestWorkspaceName = workspaceNames.find(name => this.isInDependencies(name, packageName, isDevOnly));
         const typesPackageName = !isDefinitelyTyped(packageName) && getDefinitelyTypedFor(packageName);
         const closestWorkspaceNameForTypes = typesPackageName && workspaceNames.find(name => this.isInDependencies(name, typesPackageName, isDevOnly));
-        if (closestWorkspaceName || closestWorkspaceNameForTypes) {
-            if (closestWorkspaceName)
-                this.addReferencedDependency(closestWorkspaceName, packageName);
-            if (closestWorkspaceNameForTypes && !this.hasTypesIncluded.get(closestWorkspaceNameForTypes)?.has(packageName))
-                this.addReferencedDependency(closestWorkspaceNameForTypes, typesPackageName);
+        if (closestWorkspaceNameForTypes && !this.hasTypesIncluded.get(closestWorkspaceNameForTypes)?.has(packageName))
+            this.addReferencedDependency(closestWorkspaceNameForTypes, typesPackageName);
+        if (closestWorkspaceName) {
+            this.addReferencedDependency(closestWorkspaceName, packageName);
             return true;
         }
+        if (closestWorkspaceNameForTypes && isTypeOnly)
+            return true;
+        if (this._manifests.get(workspace.name)?.engines[packageName])
+            return true;
         this.addReferencedDependency(workspace.name, packageName);
         return false;
     }

package/dist/graph/analyze.js

@@ -171,7 +171,7 @@ export const analyze = async ({ analyzedFiles, counselor, chief, collector, depu
                 if (file.imports?.external) {
                     for (const extImport of file.imports.external) {
                         const packageName = getPackageNameFromModuleSpecifier(extImport.specifier);
-                        const isHandled = packageName && deputy.maybeAddReferencedExternalDependency(ws, packageName);
+                        const isHandled = packageName && deputy.maybeAddReferencedExternalDependency(ws, packageName, undefined, extImport.isTypeOnly);
                         if (!isHandled)
                             collector.addIssue({
                                 type: 'unlisted',

package/dist/manifest/helpers.js

@@ -1,17 +1,48 @@
-import { join } from '../util/path.js';
+import { existsSync, readFileSync } from 'node:fs';
+import { dirname, join } from '../util/path.js';
 import { _require } from '../util/require.js';
+const monorepoRootCache = new Map();
+const findMonorepoRootAbove = (startDir) => {
+    if (monorepoRootCache.has(startDir))
+        return monorepoRootCache.get(startDir);
+    let current = dirname(startDir);
+    let result;
+    while (current !== dirname(current)) {
+        if (existsSync(join(current, 'pnpm-workspace.yaml'))) {
+            result = current;
+            break;
+        }
+        try {
+            const pkg = JSON.parse(readFileSync(join(current, 'package.json'), 'utf8'));
+            if (pkg.workspaces) {
+                result = current;
+                break;
+            }
+        }
+        catch { }
+        current = dirname(current);
+    }
+    monorepoRootCache.set(startDir, result);
+    return result;
+};
 export const loadPackageManifest = ({ dir, packageName, cwd }) => {
     try {
         return _require(join(dir, 'node_modules', packageName, 'package.json'));
     }
-    catch (_error) {
-        if (dir !== cwd) {
-            try {
-                return _require(join(cwd, 'node_modules', packageName, 'package.json'));
-            }
-            catch (_error) {
-            }
+    catch { }
+    if (dir !== cwd) {
+        try {
+            return _require(join(cwd, 'node_modules', packageName, 'package.json'));
+        }
+        catch { }
+        return;
+    }
+    const root = findMonorepoRootAbove(cwd);
+    if (root) {
+        try {
+            return _require(join(root, 'node_modules', packageName, 'package.json'));
         }
+        catch { }
     }
 };
 export const getFilteredScripts = (scripts) => {

package/dist/plugins/typescript/index.js

@@ -32,7 +32,8 @@ const resolveConfig = async (localConfig, options) => {
     return compact([
         ...extend,
         ...references,
-        ...[...types, ...plugins, ...importHelpers].map(id => toDeferResolve(id)),
+        ...types.map(id => toDeferResolve(id, { isTypeOnly: true })),
+        ...[...plugins, ...importHelpers].map(id => toDeferResolve(id)),
         ...jsx,
         ...aliases,
     ]);

package/dist/types/package-json.d.ts

@@ -53,6 +53,7 @@ export type PackageJson = {
     catalog?: Catalog;
     catalogs?: Catalogs;
     packageManager?: string;
+    engines?: Record<string, string>;
     pnpm?: {
         overrides?: Dependencies;
     };

package/dist/types/workspace.d.ts

@@ -10,6 +10,7 @@ type WorkspaceManifest = {
     optionalPeerDependencies: DependencySet;
     requiredPeerDependencies: DependencyArray;
     allDependencies: DependencySet;
+    engines: Record<string, string>;
     ignoreDependencies: (string | RegExp)[];
     ignoreBinaries: (string | RegExp)[];
     ignoreUnresolved: (string | RegExp)[];

package/dist/typescript/get-imports-and-exports.js

@@ -11,6 +11,7 @@ import { buildJSDocTagLookup } from './visitors/jsdoc.js';
 import { walkAST } from './visitors/walk.js';
 const getImportsAndExports = (filePath, sourceText, resolveModule, options, ignoreExportsUsedInFile, skipExportsForFile, visitor, pluginCtx, cachedParseResult) => {
     const skipExports = skipExportsForFile || !options.isReportExports;
+    const isDts = filePath.endsWith('.d.ts') || filePath.endsWith('.d.cts') || filePath.endsWith('.d.mts');
     const internal = new Map();
     const external = new Set();
     const unresolved = new Set();
@@ -53,7 +54,7 @@ const getImportsAndExports = (filePath, sourceText, resolveModule, options, igno
             pos: opts.pos,
             line: opts.line,
             col: opts.col,
-            isTypeOnly: !!(modifiers & IMPORT_FLAGS.TYPE_ONLY),
+            isTypeOnly: isDts || !!(modifiers & IMPORT_FLAGS.TYPE_ONLY),
         });
         const file = internal.get(importFilePath);
         const importMaps = file ?? createImports();
@@ -129,7 +130,7 @@ const getImportsAndExports = (filePath, sourceText, resolveModule, options, igno
                         pos: ePos,
                         line,
                         col,
-                        isTypeOnly: !!(modifiers & IMPORT_FLAGS.TYPE_ONLY),
+                        isTypeOnly: isDts || !!(modifiers & IMPORT_FLAGS.TYPE_ONLY),
                     });
                 }
             }
@@ -153,7 +154,7 @@ const getImportsAndExports = (filePath, sourceText, resolveModule, options, igno
                     pos: uPos,
                     line,
                     col,
-                    isTypeOnly: !!(modifiers & IMPORT_FLAGS.TYPE_ONLY),
+                    isTypeOnly: isDts || !!(modifiers & IMPORT_FLAGS.TYPE_ONLY),
                 });
             }
         }

package/dist/util/create-input-handler.js

@@ -53,7 +53,7 @@ export const createInputHandler = (deputy, chief, isGitIgnored, addIssue, extern
         const isWorkspace = chief.workspacesByPkgName.has(packageName);
         const inputWorkspace = getWorkspaceFor(input, chief, workspace);
         if (inputWorkspace) {
-            const isHandled = deputy.maybeAddReferencedExternalDependency(inputWorkspace, packageName, isConfig(input));
+            const isHandled = deputy.maybeAddReferencedExternalDependency(inputWorkspace, packageName, isConfig(input), input.isTypeOnly);
             if (externalRefs && !isWorkspace) {
                 addExternalRef(externalRefs, containingFilePath, { specifier: packageName, identifier: undefined });
             }

package/dist/util/input.d.ts

@@ -6,6 +6,7 @@ export interface Input {
     specifier: string;
     production?: boolean;
     optional?: boolean;
+    isTypeOnly?: boolean;
     dir?: string;
     containingFilePath?: string;
     allowIncludeExports?: boolean;
@@ -27,6 +28,7 @@ interface IgnoreInput extends Input {
 }
 type Options = {
     optional?: boolean;
+    isTypeOnly?: boolean;
     dir?: string;
     containingFilePath?: string;
     allowIncludeExports?: boolean;

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "6.9.0";
+export declare const version = "6.10.0";

package/dist/version.js

@@ -1 +1 @@
-export const version = '6.9.0';
+export const version = '6.10.0';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "6.9.0",
+  "version": "6.10.0",
   "description": "Find and fix unused dependencies, exports and files in your TypeScript and JavaScript projects",
   "keywords": [
     "analysis",