knip 3.3.0 → 3.3.2

package/dist/ConfigurationValidator.d.ts

@@ -35,6 +35,7 @@ export declare const ConfigurationValidator: z.ZodObject<{
         ignore: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
         ignoreBinaries: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
         ignoreDependencies: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
+        includeEntryExports: z.ZodOptional<z.ZodBoolean>;
         astro: z.ZodOptional<z.ZodUnion<[z.ZodBoolean, z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>, z.ZodObject<{
             config: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
             entry: z.ZodOptional<z.ZodUnion<[z.ZodString, z.ZodArray<z.ZodString, "many">]>>;
@@ -614,6 +615,7 @@ export declare const ConfigurationValidator: z.ZodObject<{
         ignore?: string | string[] | undefined;
         ignoreBinaries?: string[] | undefined;
         ignoreDependencies?: string[] | undefined;
+        includeEntryExports?: boolean | undefined;
         astro?: string | boolean | string[] | {
             config?: string | string[] | undefined;
             entry?: string | string[] | undefined;
@@ -841,6 +843,7 @@ export declare const ConfigurationValidator: z.ZodObject<{
         ignore?: string | string[] | undefined;
         ignoreBinaries?: string[] | undefined;
         ignoreDependencies?: string[] | undefined;
+        includeEntryExports?: boolean | undefined;
         astro?: string | boolean | string[] | {
             config?: string | string[] | undefined;
             entry?: string | string[] | undefined;
@@ -1657,6 +1660,7 @@ export declare const ConfigurationValidator: z.ZodObject<{
         ignore?: string | string[] | undefined;
         ignoreBinaries?: string[] | undefined;
         ignoreDependencies?: string[] | undefined;
+        includeEntryExports?: boolean | undefined;
         astro?: string | boolean | string[] | {
             config?: string | string[] | undefined;
             entry?: string | string[] | undefined;
@@ -2121,6 +2125,7 @@ export declare const ConfigurationValidator: z.ZodObject<{
         ignore?: string | string[] | undefined;
         ignoreBinaries?: string[] | undefined;
         ignoreDependencies?: string[] | undefined;
+        includeEntryExports?: boolean | undefined;
         astro?: string | boolean | string[] | {
             config?: string | string[] | undefined;
             entry?: string | string[] | undefined;

package/dist/ConfigurationValidator.js

@@ -113,6 +113,7 @@ const baseWorkspaceConfigurationSchema = z.object({
     ignore: globSchema.optional(),
     ignoreBinaries: z.array(z.string()).optional(),
     ignoreDependencies: z.array(z.string()).optional(),
+    includeEntryExports: z.boolean().optional(),
 });
 const workspaceConfigurationSchema = baseWorkspaceConfigurationSchema.merge(pluginsSchema.partial());
 const workspacesConfigurationSchema = z.object({

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/util/compilers.d.ts

@@ -23,6 +23,7 @@ export declare const partitionCompilers: (rawLocalConfig: RawConfiguration) => {
         ignore?: string | string[] | undefined;
         ignoreBinaries?: string[] | undefined;
         ignoreDependencies?: string[] | undefined;
+        includeEntryExports?: boolean | undefined;
         astro?: string | boolean | string[] | {
             config?: string | string[] | undefined;
             entry?: string | string[] | undefined;

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

@@ -1,30 +1,24 @@
 import { ISSUE_TYPES } from '../constants.js';
 import { ConfigurationError } from './errors.js';
+const normalize = (values) => values.map(value => value.split(',')).flat();
 export const getIncludedIssueTypes = (cliArgs, { include = [], exclude = [], isProduction = false } = {}) => {
-    [...cliArgs.include, ...cliArgs.exclude, ...include, ...exclude].forEach(type => {
+    let incl = normalize(cliArgs.include);
+    let excl = normalize(cliArgs.exclude);
+    [...incl, ...excl, ...include, ...exclude].forEach(type => {
         if (!ISSUE_TYPES.includes(type))
             throw new ConfigurationError(`Invalid issue type: ${type}`);
     });
+    const excludes = exclude.filter(exclude => !incl.includes(exclude));
+    const includes = include.filter(include => !excl.includes(include));
     if (cliArgs.dependencies) {
-        cliArgs.include = [
-            ...cliArgs.include,
-            'dependencies',
-            'optionalPeerDependencies',
-            'unlisted',
-            'binaries',
-            'unresolved',
-        ];
+        incl = [...incl, 'dependencies', 'optionalPeerDependencies', 'unlisted', 'binaries', 'unresolved'];
     }
     if (cliArgs.exports) {
         const exports = ['exports', 'nsExports', 'classMembers', 'types', 'nsTypes', 'enumMembers', 'duplicates'];
-        cliArgs.include = [...cliArgs.include, ...exports];
+        incl = [...incl, ...exports];
     }
-    const normalizedIncludesArg = cliArgs.include.map(value => value.split(',')).flat();
-    const normalizedExcludesArg = cliArgs.exclude.map(value => value.split(',')).flat();
-    const excludes = exclude.filter(exclude => !normalizedIncludesArg.includes(exclude));
-    const includes = include.filter(include => !normalizedExcludesArg.includes(include));
-    const _include = [normalizedIncludesArg, includes].flat();
-    const _exclude = [normalizedExcludesArg, excludes].flat();
+    const _include = [...incl, ...includes];
+    const _exclude = [...excl, ...excludes];
     if (isProduction) {
         _exclude.push('devDependencies');
     }

package/dist/version.d.ts

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

package/dist/version.js

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

package/package.json

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

package/schema.json

@@ -75,6 +75,7 @@
       ]
     },
     "includeEntryExports": {
+      "title": "Include entry files when reporting unused exports",
       "type": "boolean"
     },
     "workspaces": {
@@ -187,6 +188,10 @@
           "title": "Dependencies from package.json to ignore",
           "examples": ["husky", "lint-staged"],
           "$ref": "#/definitions/list"
+        },
+        "includeEntryExports": {
+          "title": "Include entry files when reporting unused exports",
+          "type": "boolean"
         }
       }
     },