knip 3.3.1 → 3.3.3

package/dist/ConfigurationChief.js

@@ -10,7 +10,7 @@ import { arrayify } from './util/array.js';
 import parsedArgValues from './util/cli-arguments.js';
 import { partitionCompilers } from './util/compilers.js';
 import { ConfigurationError, LoaderError } from './util/errors.js';
-import { findFile, loadJSON } from './util/fs.js';
+import { findFile, isDirectory, isFile, loadJSON } from './util/fs.js';
 import { getIncludedIssueTypes } from './util/get-included-issue-types.js';
 import { _dirGlob } from './util/glob.js';
 import { _load } from './util/loader.js';
@@ -363,6 +363,11 @@ export class ConfigurationChief {
     getUnusedIgnoredWorkspaces() {
         const ignoredWorkspaceNames = this.config.ignoreWorkspaces;
         const workspaceNames = [...this.manifestWorkspaces.keys(), ...this.additionalWorkspaceNames];
-        return ignoredWorkspaceNames.filter(ignoredWorkspaceName => !workspaceNames.some(name => micromatch.isMatch(name, ignoredWorkspaceName)));
+        return ignoredWorkspaceNames
+            .filter(ignoredWorkspaceName => !workspaceNames.some(name => micromatch.isMatch(name, ignoredWorkspaceName)))
+            .filter(ignoredWorkspaceName => {
+            const dir = join(this.cwd, ignoredWorkspaceName);
+            return !isDirectory(dir) || isFile(join(dir, 'package.json'));
+        });
     }
 }

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

@@ -1,3 +1,4 @@
+export declare const isDirectory: (filePath: string) => boolean;
 export declare const isFile: (filePath: string) => boolean;
 export declare const findFile: (workingDir: string, fileName: string) => string | undefined;
 export declare const loadFile: (filePath: string) => Promise<string>;

package/dist/util/fs.js

@@ -4,6 +4,10 @@ import yaml from 'js-yaml';
 import stripJsonComments from 'strip-json-comments';
 import { LoaderError } from './errors.js';
 import { dirname, join } from './path.js';
+export const isDirectory = (filePath) => {
+    const stat = statSync(filePath, { throwIfNoEntry: false });
+    return stat !== undefined && stat.isDirectory();
+};
 export const isFile = (filePath) => {
     const stat = statSync(filePath, { throwIfNoEntry: false });
     return stat !== undefined && stat.isFile();

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/util/modules.js

@@ -57,5 +57,7 @@ export const sanitizeSpecifier = (specifier) => {
         return specifier;
     if (isAbsolute(specifier))
         return specifier;
+    if (specifier.startsWith('virtual:'))
+        return specifier;
     return specifier.replace(/^([?!|-]+)?([^!?:]+).*/, '$2');
 };

package/dist/version.d.ts

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

package/dist/version.js

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "3.3.1",
+  "version": "3.3.3",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://knip.dev",
   "repository": {
@@ -102,6 +102,9 @@
         "npm run knip",
         "npm run knip:production",
         "npm test"
+      ],
+      "after:bump": [
+        "git add ../../package-lock.json"
       ]
     },
     "github": {

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"
         }
       }
     },