knip 3.3.0 → 3.3.1

package/dist/DependencyDeputy.d.ts

@@ -48,7 +48,10 @@ export declare class DependencyDeputy {
     addReferencedDependency(workspaceName: string, packageName: string): void;
     addReferencedBinary(workspaceName: string, binaryName: string): void;
     addHostDependencies(workspaceName: string, hostDependencies: HostDependencies): void;
-    getHostDependenciesFor(workspaceName: string, dependency: string): string[];
+    getHostDependenciesFor(workspaceName: string, dependency: string): {
+        name: string;
+        isPeerOptional: boolean;
+    }[];
     getPeerDependencies(workspaceName: string): string[];
     getOptionalPeerDependencies(workspaceName: string): string[];
     maybeAddReferencedExternalDependency(workspace: Workspace, packageName: string): boolean;

package/dist/DependencyDeputy.js

@@ -195,16 +195,16 @@ export class DependencyDeputy {
                         ...this.getHostDependenciesFor(workspaceName, typedPackageName),
                     ];
                     if (hostDependencies.length)
-                        return !!hostDependencies.find(host => isReferencedDependency(host, true));
+                        return !!hostDependencies.find(host => isReferencedDependency(host.name, true));
                     if (!referencedDependencies)
                         return false;
                     return referencedDependencies.has(typedPackageName);
                 }
                 const hostDependencies = this.getHostDependenciesFor(workspaceName, dependency);
-                hostDependencies.forEach(dep => (!peerDepRecs[dep] ? (peerDepRecs[dep] = 1) : peerDepRecs[dep]++));
-                return hostDependencies.some(peerDependency => isReferencedDependency(peerDependency, true));
+                hostDependencies.forEach(({ name }) => (!peerDepRecs[name] ? (peerDepRecs[name] = 1) : peerDepRecs[name]++));
+                return hostDependencies.some(hostDependency => (isPeerDep === false || !hostDependency.isPeerOptional) && isReferencedDependency(hostDependency.name, true));
             };
-            const isNotReferencedDependency = (dependency) => !isReferencedDependency(dependency);
+            const isNotReferencedDependency = (dependency) => !isReferencedDependency(dependency, false);
             const pd = this.getProductionDependencies(workspaceName);
             const dd = this.getDevDependencies(workspaceName);
             const od = this.getOptionalPeerDependencies(workspaceName);

package/dist/manifest/index.js

@@ -39,14 +39,18 @@ const findManifestDependencies = async ({ manifest, isProduction, isStrict, dir,
                 }
             });
             const packagePeerDependencies = Object.keys(manifest.peerDependencies ?? {});
-            packagePeerDependencies.forEach(packagePeerDependency => {
+            for (const packagePeerDependency of packagePeerDependencies) {
+                const hostDependency = {
+                    name: packageName,
+                    isPeerOptional: manifest.peerDependenciesMeta?.[packagePeerDependency]?.optional ?? false,
+                };
                 if (hostDependencies.has(packagePeerDependency)) {
-                    hostDependencies.get(packagePeerDependency)?.add(packageName);
+                    hostDependencies.get(packagePeerDependency)?.add(hostDependency);
                 }
                 else {
-                    hostDependencies.set(packagePeerDependency, new Set([packageName]));
+                    hostDependencies.set(packagePeerDependency, new Set([hostDependency]));
                 }
-            });
+            }
             if (!isDefinitelyTyped(packageName) && (manifest.types || manifest.typings))
                 hasTypesIncluded.add(packageName);
         }

package/dist/types/workspace.d.ts

@@ -12,6 +12,9 @@ type WorkspaceManifest = {
     ignoreBinaries: string[];
 };
 export type WorkspaceManifests = Map<string, WorkspaceManifest>;
-export type HostDependencies = Map<string, Set<string>>;
+export type HostDependencies = Map<string, Set<{
+    name: string;
+    isPeerOptional: boolean;
+}>>;
 export type InstalledBinaries = Map<string, Set<string>>;
 export {};

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "3.3.0";
+export declare const version = "3.3.1";

package/dist/version.js

@@ -1 +1 @@
-export const version = '3.3.0';
+export const version = '3.3.1';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "3.3.0",
+  "version": "3.3.1",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://knip.dev",
   "repository": {