knip 2.19.9 → 2.19.10

package/dist/DependencyDeputy.d.ts

@@ -45,7 +45,7 @@ export declare class DependencyDeputy {
     addReferencedDependency(workspaceName: string, packageName: string): void;
     addReferencedBinary(workspaceName: string, binaryName: string): void;
     addPeerDependencies(workspaceName: string, peerDependencies: Map<string, Set<string>>): void;
-    getPeerDependencies(workspaceName: string, dependency: string): string[];
+    getPeerDependenciesOf(workspaceName: string, dependency: string): string[];
     maybeAddReferencedExternalDependency(workspace: Workspace, packageName: string): boolean;
     maybeAddReferencedBinary(workspace: Workspace, binaryName: string): boolean;
     private isInDependencies;

package/dist/DependencyDeputy.js

@@ -76,7 +76,7 @@ export class DependencyDeputy {
     addPeerDependencies(workspaceName, peerDependencies) {
         this.peerDependencies.set(workspaceName, peerDependencies);
     }
-    getPeerDependencies(workspaceName, dependency) {
+    getPeerDependenciesOf(workspaceName, dependency) {
         return Array.from(this.peerDependencies.get(workspaceName)?.get(dependency) ?? []);
     }
     maybeAddReferencedExternalDependency(workspace, packageName) {
@@ -150,35 +150,38 @@ export class DependencyDeputy {
                 return true;
             };
             const peerDepRecs = {};
-            const isNotReferencedDependency = (dependency, isPeerDep) => {
+            const isReferencedDependency = (dependency, isPeerDep) => {
                 if (referencedDependencies?.has(dependency))
-                    return false;
-                if (isPeerDep && peerDepRecs[dependency])
                     return true;
+                if (isPeerDep && peerDepRecs[dependency])
+                    return false;
                 const [scope, typedDependency] = dependency.split('/');
                 if (scope === '@types') {
                     const typedPackageName = getPackageFromDefinitelyTyped(typedDependency);
                     if (IGNORE_DEFINITELY_TYPED.includes(typedPackageName))
-                        return false;
-                    const peerDependencies = this.getPeerDependencies(workspaceName, typedPackageName);
+                        return true;
+                    const peerDependencies = this.getPeerDependenciesOf(workspaceName, typedPackageName);
                     if (peerDependencies.length) {
-                        return !peerDependencies.find(peerDependency => !isNotReferencedDependency(peerDependency, true));
+                        return !!peerDependencies.find(peerDependency => isReferencedDependency(peerDependency, true));
                     }
-                    return !referencedDependencies?.has(typedPackageName);
+                    if (!referencedDependencies)
+                        return false;
+                    return referencedDependencies.has(typedPackageName);
                 }
-                const peerDependencies = this.getPeerDependencies(workspaceName, dependency);
+                const peerDependencies = this.getPeerDependenciesOf(workspaceName, dependency);
                 peerDependencies.forEach(dep => (!peerDepRecs[dep] ? (peerDepRecs[dep] = 1) : peerDepRecs[dep]++));
-                return !peerDependencies.find(peerDependency => !isNotReferencedDependency(peerDependency, true));
+                return !!peerDependencies.find(peerDependency => isReferencedDependency(peerDependency, true));
             };
+            const isNotReferencedDependency = (dependency) => !isReferencedDependency(dependency);
             const pd = this.getProductionDependencies(workspaceName);
             const dd = this.getDevDependencies(workspaceName);
             pd.filter(isNotIgnoredDependency)
                 .filter(isNotIgnoredBinary)
-                .filter(d => isNotReferencedDependency(d))
+                .filter(isNotReferencedDependency)
                 .forEach(symbol => dependencyIssues.push({ type: 'dependencies', filePath: manifestPath, symbol }));
             dd.filter(isNotIgnoredDependency)
                 .filter(isNotIgnoredBinary)
-                .filter(d => isNotReferencedDependency(d))
+                .filter(isNotReferencedDependency)
                 .forEach(symbol => devDependencyIssues.push({ type: 'devDependencies', filePath: manifestPath, symbol }));
         }
         return { dependencyIssues, devDependencyIssues };

package/dist/ProjectPrincipal.js

@@ -158,8 +158,7 @@ export class ProjectPrincipal {
     }
     getHasReferences(filePath, exportedItem) {
         const hasReferences = { external: false, internal: false };
-        const node = ts.isExportAssignment(exportedItem.node) ? exportedItem.node.getChildAt(1) : exportedItem.node;
-        const symbolReferences = this.findReferences(filePath, node).flatMap(f => f.references);
+        const symbolReferences = this.findReferences(filePath, exportedItem.pos).flatMap(f => f.references);
         for (const reference of symbolReferences) {
             if (reference.fileName === filePath) {
                 if (!reference.isDefinition) {
@@ -180,7 +179,7 @@ export class ProjectPrincipal {
             .filter(member => {
             if (this.isPublicExport(member))
                 return false;
-            const referencedSymbols = this.findReferences(filePath, member.node);
+            const referencedSymbols = this.findReferences(filePath, member.pos);
             const files = referencedSymbols
                 .flatMap(refs => refs.references)
                 .filter(ref => !ref.isDefinition)
@@ -191,8 +190,8 @@ export class ProjectPrincipal {
         })
             .map(member => member.identifier);
     }
-    findReferences(filePath, node) {
-        return this.backend.lsFindReferences(filePath, node.getStart()) ?? [];
+    findReferences(filePath, pos) {
+        return this.backend.lsFindReferences(filePath, pos) ?? [];
     }
     isPublicExport(exportedItem) {
         const tags = this.getJSDocTags(exportedItem);

package/dist/typescript/visitors/exports/exportAssignment.js

@@ -3,6 +3,7 @@ import { SymbolType } from '../../../types/issues.js';
 import { exportVisitor as visit } from '../index.js';
 export default visit(() => true, node => {
     if (ts.isExportAssignment(node)) {
-        return { node, identifier: 'default', type: SymbolType.UNKNOWN, pos: node.expression.getStart() };
+        const pos = node.getChildAt(1).getStart();
+        return { node, identifier: 'default', type: SymbolType.UNKNOWN, pos };
     }
 });

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "2.19.9";
+export declare const version = "2.19.10";

package/dist/version.js

@@ -1 +1 @@
-export const version = '2.19.9';
+export const version = '2.19.10';

package/package.json

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