knip 2.20.0 → 2.21.0-op.0

package/dist/DependencyDeputy.d.ts

@@ -32,6 +32,7 @@ export declare class DependencyDeputy {
         scripts: string[];
         dependencies: string[];
         peerDependencies: string[];
+        optionalPeerDependencies: string[];
         optionalDependencies: string[];
         devDependencies: string[];
         allDependencies: string[];
@@ -46,12 +47,14 @@ export declare class DependencyDeputy {
     addReferencedBinary(workspaceName: string, binaryName: string): void;
     addPeerDependencies(workspaceName: string, peerDependencies: Map<string, Set<string>>): void;
     getPeerDependenciesOf(workspaceName: string, dependency: string): string[];
+    getOptionalPeerDependencies(workspaceName: string): string[];
     maybeAddReferencedExternalDependency(workspace: Workspace, packageName: string): boolean;
     maybeAddReferencedBinary(workspace: Workspace, binaryName: string): boolean;
     private isInDependencies;
     settleDependencyIssues(): {
         dependencyIssues: Issue[];
         devDependencyIssues: Issue[];
+        optionalPeerDependencyIssues: Issue[];
     };
     getConfigurationHints(): {
         configurationHints: ConfigurationHints;

package/dist/DependencyDeputy.js

@@ -22,6 +22,11 @@ export class DependencyDeputy {
         const dependencies = Object.keys(manifest.dependencies ?? {});
         const peerDependencies = Object.keys(manifest.peerDependencies ?? {});
         const optionalDependencies = Object.keys(manifest.optionalDependencies ?? {});
+        const optionalPeerDependencies = manifest.peerDependenciesMeta
+            ? peerDependencies.filter(peerDependency => manifest.peerDependenciesMeta &&
+                peerDependency in manifest.peerDependenciesMeta &&
+                manifest.peerDependenciesMeta[peerDependency].optional)
+            : [];
         const devDependencies = Object.keys(manifest.devDependencies ?? {});
         const allDependencies = [...dependencies, ...devDependencies, ...peerDependencies, ...optionalDependencies];
         this._manifests.set(name, {
@@ -32,6 +37,7 @@ export class DependencyDeputy {
             scripts,
             dependencies,
             peerDependencies,
+            optionalPeerDependencies,
             optionalDependencies,
             devDependencies,
             allDependencies,
@@ -79,6 +85,12 @@ export class DependencyDeputy {
     getPeerDependenciesOf(workspaceName, dependency) {
         return Array.from(this.peerDependencies.get(workspaceName)?.get(dependency) ?? []);
     }
+    getOptionalPeerDependencies(workspaceName) {
+        const manifest = this._manifests.get(workspaceName);
+        if (!manifest)
+            return [];
+        return manifest.optionalPeerDependencies;
+    }
     maybeAddReferencedExternalDependency(workspace, packageName) {
         if (isBuiltin(packageName))
             return true;
@@ -133,6 +145,7 @@ export class DependencyDeputy {
     settleDependencyIssues() {
         const dependencyIssues = [];
         const devDependencyIssues = [];
+        const optionalPeerDependencyIssues = [];
         for (const [workspaceName, { manifestPath, ignoreDependencies, ignoreBinaries }] of this._manifests.entries()) {
             const referencedDependencies = this.referencedDependencies.get(workspaceName);
             const installedBinaries = this.getInstalledBinaries(workspaceName);
@@ -175,6 +188,7 @@ export class DependencyDeputy {
             const isNotReferencedDependency = (dependency) => !isReferencedDependency(dependency);
             const pd = this.getProductionDependencies(workspaceName);
             const dd = this.getDevDependencies(workspaceName);
+            const od = this.getOptionalPeerDependencies(workspaceName);
             pd.filter(isNotIgnoredDependency)
                 .filter(isNotIgnoredBinary)
                 .filter(isNotReferencedDependency)
@@ -183,8 +197,12 @@ export class DependencyDeputy {
                 .filter(isNotIgnoredBinary)
                 .filter(isNotReferencedDependency)
                 .forEach(symbol => devDependencyIssues.push({ type: 'devDependencies', filePath: manifestPath, symbol }));
+            od.filter(isNotIgnoredDependency)
+                .filter(isNotIgnoredBinary)
+                .filter(p => isReferencedDependency(p))
+                .forEach(symbol => optionalPeerDependencyIssues.push({ type: 'optionalPeerDependencies', filePath: manifestPath, symbol }));
         }
-        return { dependencyIssues, devDependencyIssues };
+        return { dependencyIssues, devDependencyIssues, optionalPeerDependencyIssues };
     }
     getConfigurationHints() {
         const configurationHints = new Set();

package/dist/constants.js

@@ -59,6 +59,7 @@ export const ISSUE_TYPES = [
     'files',
     'dependencies',
     'devDependencies',
+    'optionalPeerDependencies',
     'unlisted',
     'binaries',
     'unresolved',
@@ -74,6 +75,7 @@ export const ISSUE_TYPE_TITLE = {
     files: 'Unused files',
     dependencies: 'Unused dependencies',
     devDependencies: 'Unused devDependencies',
+    optionalPeerDependencies: 'Referenced optional peerDependencies',
     unlisted: 'Unlisted dependencies',
     binaries: 'Unlisted binaries',
     unresolved: 'Unresolved imports',

package/dist/index.js

@@ -308,11 +308,12 @@ export const main = async (unresolvedConfiguration) => {
         }
     }
     if (isReportDependencies) {
-        const { dependencyIssues, devDependencyIssues } = deputy.settleDependencyIssues();
+        const { dependencyIssues, devDependencyIssues, optionalPeerDependencyIssues } = deputy.settleDependencyIssues();
         const { configurationHints } = deputy.getConfigurationHints();
         dependencyIssues.forEach(issue => collector.addIssue(issue));
         if (!isProduction)
             devDependencyIssues.forEach(issue => collector.addIssue(issue));
+        optionalPeerDependencyIssues.forEach(issue => collector.addIssue(issue));
         configurationHints.forEach(hint => collector.addConfigurationHint(hint));
     }
     const unusedIgnoredWorkspaces = chief.getUnusedIgnoredWorkspaces();

package/dist/reporters/json.js

@@ -22,6 +22,7 @@ export default async ({ report, issues, options }) => {
             ...(report.files && { files: false }),
             ...(report.dependencies && { dependencies: [] }),
             ...(report.devDependencies && { devDependencies: [] }),
+            ...(report.optionalPeerDependencies && { optionalPeerDependencies: [] }),
             ...(report.unlisted && { unlisted: [] }),
             ...(report.unresolved && { unresolved: [] }),
             ...((report.exports || report.nsExports) && { exports: [] }),

package/dist/types/issues.d.ts

@@ -22,6 +22,7 @@ export type Issues = {
     files: IssueSet;
     dependencies: IssueRecords;
     devDependencies: IssueRecords;
+    optionalPeerDependencies: IssueRecords;
     unlisted: IssueRecords;
     binaries: IssueRecords;
     unresolved: IssueRecords;

package/dist/types/workspace.d.ts

@@ -4,6 +4,7 @@ type WorkspaceManifest = {
     scripts: string[];
     dependencies: string[];
     peerDependencies: string[];
+    optionalPeerDependencies: string[];
     optionalDependencies: string[];
     devDependencies: string[];
     allDependencies: string[];

package/dist/util/get-included-issue-types.js

@@ -1,7 +1,14 @@
 import { ISSUE_TYPES } from '../constants.js';
 export const getIncludedIssueTypes = (cliArgs, { include = [], exclude = [], isProduction = false } = {}) => {
     if (cliArgs.dependencies) {
-        cliArgs.include = [...cliArgs.include, 'dependencies', 'unlisted', 'binaries', 'unresolved'];
+        cliArgs.include = [
+            ...cliArgs.include,
+            'dependencies',
+            'optionalPeerDependencies',
+            'unlisted',
+            'binaries',
+            'unresolved',
+        ];
     }
     if (cliArgs.exports) {
         const exports = ['exports', 'nsExports', 'classMembers', 'types', 'nsTypes', 'enumMembers', 'duplicates'];
@@ -21,9 +28,9 @@ export const getIncludedIssueTypes = (cliArgs, { include = [], exclude = [], isP
     }
     else {
         if (_include.includes('dependencies'))
-            _include.push('devDependencies');
+            _include.push('devDependencies', 'optionalPeerDependencies');
         if (_exclude.includes('dependencies'))
-            _exclude.push('devDependencies');
+            _exclude.push('devDependencies', 'optionalPeerDependencies');
     }
     const included = (_include.length > 0 ? _include : ISSUE_TYPES).filter(group => !_exclude.includes(group));
     return ISSUE_TYPES.reduce((types, group) => ((types[group] = included.includes(group)), types), {});

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "2.20.0";
+export declare const version = "2.21.0-op.0";

package/dist/version.js

@@ -1 +1 @@
-export const version = '2.20.0';
+export const version = '2.21.0-op.0';

package/package.json

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