knip 5.44.5 → 5.46.0

package/dist/typescript/find-internal-references.js

@@ -1,6 +1,13 @@
 import ts from 'typescript';
 import { isIdChar } from '../util/regex.js';
 export const isType = (item) => item.type === 'type' || item.type === 'interface' || item.type === 'enum';
+const findInFlow = (flowNode, targetSymbol) => {
+    if (!flowNode?.node)
+        return false;
+    if (flowNode.node.symbol === targetSymbol)
+        return true;
+    return findInFlow(flowNode.antecedent, targetSymbol);
+};
 export const findInternalReferences = (item, sourceFile, typeChecker, referencedSymbolsInExport, isBindingElement) => {
     if (!item.symbol)
         return [0, false];
@@ -19,7 +26,7 @@ export const findInternalReferences = (item, sourceFile, typeChecker, referenced
             const isExportDeclaration = index === item.pos || index === item.pos + 1;
             if (!isExportDeclaration) {
                 const symbol = typeChecker.getSymbolAtLocation(ts.getTokenAtPosition(sourceFile, index));
-                if (symbol) {
+                if (symbol && id === symbol.escapedName) {
                     const isInExport = referencedSymbolsInExport.has(symbol);
                     if (isInExport)
                         isSymbolInExport = true;
@@ -30,7 +37,7 @@ export const findInternalReferences = (item, sourceFile, typeChecker, referenced
                     }
                     const declaration = symbol.declarations?.[0];
                     if (declaration) {
-                        if (item.symbol === declaration.name?.flowNode?.node?.symbol) {
+                        if (findInFlow(declaration.name?.flowNode, item.symbol)) {
                             return [++refCount, isSymbolInExport];
                         }
                         if (ts.isImportSpecifier(declaration) && symbols.has(symbol)) {

package/dist/util/cli-arguments.d.ts

@@ -1,4 +1,4 @@
-export declare const helpText = "\u2702\uFE0F  Find unused files, dependencies and exports in your JavaScript and TypeScript projects\n\nUsage: knip [options]\n\nOptions:\n  -c, --config [file]      Configuration file path (default: [.]knip.json[c], knip.js, knip.ts or package.json#knip)\n  -t, --tsConfig [file]    TypeScript configuration path (default: tsconfig.json)\n  --production             Analyze only production source files (e.g. no test files, devDependencies)\n  --strict                 Consider only direct dependencies of workspace (not devDependencies, not other workspaces)\n  -W, --workspace [dir]    Analyze a single workspace (default: analyze all configured workspaces)\n  --directory [dir]        Run process from a different directory (default: cwd)\n  --cache                  Enable caching\n  --cache-location         Change cache location (default: node_modules/.cache/knip)\n  --watch                  Watch mode\n  --no-gitignore           Don't use .gitignore\n  --include                Report only provided issue type(s), can be comma-separated or repeated (1)\n  --exclude                Exclude provided issue type(s) from report, can be comma-separated or repeated (1)\n  --dependencies           Shortcut for --include dependencies,unlisted,binaries,unresolved\n  --exports                Shortcut for --include exports,nsExports,classMembers,types,nsTypes,enumMembers,duplicates\n  --files                  Shortcut for --include files\n  --fix                    Fix issues\n  --fix-type               Fix only issues of type, can be comma-separated or repeated (2)\n  --allow-remove-files     Allow Knip to remove files (with --fix)\n  --include-libs           Include type definitions from external dependencies (default: false)\n  --include-entry-exports  Include entry files when reporting unused exports\n  --isolate-workspaces     Isolate workspaces into separate programs\n  -n, --no-progress        Don't show dynamic progress updates (automatically enabled in CI environments)\n  --preprocessor           Preprocess the results before providing it to the reporter(s), can be repeated\n  --preprocessor-options   Pass extra options to the preprocessor (as JSON string, see --reporter-options example)\n  --reporter               Select reporter: symbols, compact, codeowners, json, can be repeated (default: symbols)\n  --reporter-options       Pass extra options to the reporter (as JSON string, see example)\n  --tags                   Include or exclude tagged exports\n  --no-config-hints        Suppress configuration hints\n  --no-exit-code           Always exit with code zero (0)\n  --max-issues             Maximum number of issues before non-zero exit code (default: 0)\n  -d, --debug              Show debug output\n  --trace                  Show trace output\n  --trace-export [name]    Show trace output for named export(s)\n  --trace-file [file]      Show trace output for exports in file\n  --performance            Measure count and running time of expensive functions and display stats table\n  -h, --help               Print this help text\n  -V, --version            Print version\n\n(1) Issue types: files, dependencies, unlisted, unresolved, exports, nsExports, classMembers, types, nsTypes, enumMembers, duplicates\n(2) Fixable issue types: dependencies, exports, types\n\nExamples:\n\n$ knip\n$ knip --production\n$ knip --workspace packages/client --include files,dependencies\n$ knip -c ./config/knip.json --reporter compact\n$ knip --reporter codeowners --reporter-options '{\"path\":\".github/CODEOWNERS\"}'\n$ knip --tags=-lintignore\n\nWebsite: https://knip.dev";
+export declare const helpText = "\u2702\uFE0F  Find unused files, dependencies and exports in your JavaScript and TypeScript projects\n\nUsage: knip [options]\n\nOptions:\n  -c, --config [file]      Configuration file path (default: [.]knip.json[c], knip.js, knip.ts or package.json#knip)\n  -t, --tsConfig [file]    TypeScript configuration path (default: tsconfig.json)\n  --production             Analyze only production source files (e.g. no test files, devDependencies)\n  --strict                 Consider only direct dependencies of workspace (not devDependencies, not other workspaces)\n  -W, --workspace [dir]    Analyze a single workspace (default: analyze all configured workspaces)\n  --directory [dir]        Run process from a different directory (default: cwd)\n  --cache                  Enable caching\n  --cache-location         Change cache location (default: node_modules/.cache/knip)\n  --watch                  Watch mode\n  --no-gitignore           Don't use .gitignore\n  --include                Report only provided issue type(s), can be comma-separated or repeated (1)\n  --exclude                Exclude provided issue type(s) from report, can be comma-separated or repeated (1)\n  --dependencies           Shortcut for --include dependencies,unlisted,binaries,unresolved\n  --exports                Shortcut for --include exports,nsExports,classMembers,types,nsTypes,enumMembers,duplicates\n  --files                  Shortcut for --include files\n  --fix                    Fix issues\n  --fix-type               Fix only issues of type, can be comma-separated or repeated (2)\n  --allow-remove-files     Allow Knip to remove files (with --fix)\n  --include-libs           Include type definitions from external dependencies (default: false)\n  --include-entry-exports  Include entry files when reporting unused exports\n  --isolate-workspaces     Isolate workspaces into separate programs\n  -n, --no-progress        Don't show dynamic progress updates (automatically enabled in CI environments)\n  --preprocessor           Preprocess the results before providing it to the reporter(s), can be repeated\n  --preprocessor-options   Pass extra options to the preprocessor (as JSON string, see --reporter-options example)\n  --reporter               Select reporter: symbols, compact, codeowners, json, codeclimate, markdown, disclosure, can be repeated (default: symbols)\n  --reporter-options       Pass extra options to the reporter (as JSON string, see example)\n  --tags                   Include or exclude tagged exports\n  --no-config-hints        Suppress configuration hints\n  --treat-config-hints-as-errors    Exit with non-zero code (1) if there are any configuration hints\n  --no-exit-code           Always exit with code zero (0)\n  --max-issues             Maximum number of issues before non-zero exit code (default: 0)\n  -d, --debug              Show debug output\n  --trace                  Show trace output\n  --trace-export [name]    Show trace output for named export(s)\n  --trace-file [file]      Show trace output for exports in file\n  --performance            Measure count and running time of expensive functions and display stats table\n  -h, --help               Print this help text\n  -V, --version            Print version\n\n(1) Issue types: files, dependencies, unlisted, unresolved, exports, nsExports, classMembers, types, nsTypes, enumMembers, duplicates\n(2) Fixable issue types: dependencies, exports, types\n\nExamples:\n\n$ knip\n$ knip --production\n$ knip --workspace packages/client --include files,dependencies\n$ knip -c ./config/knip.json --reporter compact\n$ knip --reporter codeowners --reporter-options '{\"path\":\".github/CODEOWNERS\"}'\n$ knip --tags=-lintignore\n\nWebsite: https://knip.dev";
 declare const _default: {
     cache?: boolean | undefined;
     'cache-location'?: string | undefined;
@@ -35,6 +35,7 @@ declare const _default: {
     trace?: boolean | undefined;
     'trace-export'?: string | undefined;
     'trace-file'?: string | undefined;
+    'treat-config-hints-as-errors'?: boolean | undefined;
     tsConfig?: string | undefined;
     version?: boolean | undefined;
     watch?: boolean | undefined;

package/dist/util/cli-arguments.js

@@ -28,10 +28,11 @@ Options:
   -n, --no-progress        Don't show dynamic progress updates (automatically enabled in CI environments)
   --preprocessor           Preprocess the results before providing it to the reporter(s), can be repeated
   --preprocessor-options   Pass extra options to the preprocessor (as JSON string, see --reporter-options example)
-  --reporter               Select reporter: symbols, compact, codeowners, json, can be repeated (default: symbols)
+  --reporter               Select reporter: symbols, compact, codeowners, json, codeclimate, markdown, disclosure, can be repeated (default: symbols)
   --reporter-options       Pass extra options to the reporter (as JSON string, see example)
   --tags                   Include or exclude tagged exports
   --no-config-hints        Suppress configuration hints
+  --treat-config-hints-as-errors    Exit with non-zero code (1) if there are any configuration hints
   --no-exit-code           Always exit with code zero (0)
   --max-issues             Maximum number of issues before non-zero exit code (default: 0)
   -d, --debug              Show debug output
@@ -94,6 +95,7 @@ try {
             trace: { type: 'boolean' },
             'trace-export': { type: 'string' },
             'trace-file': { type: 'string' },
+            'treat-config-hints-as-errors': { type: 'boolean' },
             tsConfig: { type: 'string', short: 't' },
             version: { type: 'boolean', short: 'V' },
             watch: { type: 'boolean' },

package/dist/util/get-referenced-inputs.js

@@ -1,7 +1,7 @@
 import { IGNORED_RUNTIME_DEPENDENCIES } from '../constants.js';
 import { debugLog } from './debug.js';
 import { isDeferResolve, toDebugString } from './input.js';
-import { fromBinary, isBinary, isConfigPattern, isDeferResolveEntry, isDependency } from './input.js';
+import { fromBinary, isBinary, isConfig, isDeferResolveEntry, isDependency } from './input.js';
 import { getPackageNameFromSpecifier } from './modules.js';
 import { dirname, isAbsolute, isInternal, join } from './path.js';
 import { _resolveSync } from './resolve.js';
@@ -75,7 +75,7 @@ export const getReferencedInputsHandler = (collector, deputy, chief, isGitIgnore
         });
     }
     else if (!isGitIgnored(filePath)) {
-        if (!isDeferResolveEntry(input) && !isConfigPattern(input)) {
+        if (!isDeferResolveEntry(input) && !isConfig(input)) {
             collector.addIssue({
                 type: 'unresolved',
                 filePath: containingFilePath,

package/dist/util/input.d.ts

@@ -10,7 +10,7 @@ export interface Input {
 }
 export interface ConfigInput extends Input {
     type: 'config';
-    containingFilePath: string;
+    containingFilePath?: string;
     pluginName: PluginName;
 }
 type Options = {
@@ -25,8 +25,8 @@ export declare const toEntry: (specifier: string) => Input;
 export declare const isEntry: (input: Input) => boolean;
 export declare const toProductionEntry: (specifier: string, options?: Options) => Input;
 export declare const isProductionEntry: (input: Input) => boolean;
-export declare const toConfig: (pluginName: PluginName, specifier: string, containingFilePath: string) => ConfigInput;
-export declare const isConfigPattern: (input: Input) => input is ConfigInput;
+export declare const toConfig: (pluginName: PluginName, specifier: string, options?: Options) => ConfigInput;
+export declare const isConfig: (input: Input) => input is ConfigInput;
 export declare const toDependency: (specifier: string, options?: Options) => Input;
 export declare const isDependency: (input: Input) => boolean;
 export declare const toProductionDependency: (specifier: string) => Input;

package/dist/util/input.js

@@ -1,4 +1,4 @@
-import { toRelative } from './path.js';
+import { isAbsolute, toRelative } from './path.js';
 export const fromBinary = (input) => input.specifier;
 export const toBinary = (specifier, options = {}) => ({
     type: 'binary',
@@ -15,13 +15,13 @@ export const toProductionEntry = (specifier, options = {}) => ({
     ...options,
 });
 export const isProductionEntry = (input) => input.type === 'entry' && input.production === true;
-export const toConfig = (pluginName, specifier, containingFilePath) => ({
+export const toConfig = (pluginName, specifier, options = {}) => ({
     type: 'config',
     specifier,
     pluginName,
-    containingFilePath,
+    ...options,
 });
-export const isConfigPattern = (input) => input.type === 'config';
+export const isConfig = (input) => input.type === 'config';
 export const toDependency = (specifier, options = {}) => ({
     type: 'dependency',
     specifier,
@@ -44,4 +44,4 @@ export const toDeferResolveProductionEntry = (specifier) => ({
 export const isDeferResolveProductionEntry = (input) => input.type === 'deferResolveEntry' && input.production === true;
 export const toDeferResolveEntry = (specifier) => ({ type: 'deferResolveEntry', specifier });
 export const isDeferResolveEntry = (input) => input.type === 'deferResolveEntry';
-export const toDebugString = (input) => `${input.type}:${input.specifier}${input.containingFilePath ? ` (${toRelative(input.containingFilePath)})` : ''}`;
+export const toDebugString = (input) => `${input.type}:${isAbsolute(input.specifier) ? toRelative(input.specifier) : input.specifier}${input.containingFilePath ? ` (${toRelative(input.containingFilePath)})` : ''}`;

package/dist/util/issue-initializers.js

@@ -2,7 +2,7 @@ import { ISSUE_TYPES } from '../constants.js';
 export const initIssues = () => ({
     ...Object.fromEntries(ISSUE_TYPES.map(issueType => [issueType, {}])),
     files: new Set(),
-    _files: new Set(),
+    _files: {},
 });
 export const initCounters = () => ({
     ...Object.fromEntries(ISSUE_TYPES.map(issueType => [issueType, 0])),

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "5.44.5";
+export declare const version = "5.46.0";

package/dist/version.js

@@ -1 +1 @@
-export const version = '5.44.5';
+export const version = '5.46.0';

package/package.json

@@ -1,11 +1,11 @@
 {
   "name": "knip",
-  "version": "5.44.5",
+  "version": "5.46.0",
   "description": "Find and fix unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://knip.dev",
   "repository": {
     "type": "git",
-    "url": "https://github.com/webpro-nl/knip",
+    "url": "git+https://github.com/webpro-nl/knip.git",
     "directory": "packages/knip"
   },
   "bugs": "https://github.com/webpro-nl/knip/issues",
@@ -30,8 +30,8 @@
   ],
   "main": "./dist/index.js",
   "bin": {
-    "knip": "./bin/knip.js",
-    "knip-bun": "./bin/knip-bun.js"
+    "knip": "bin/knip.js",
+    "knip-bun": "bin/knip-bun.js"
   },
   "type": "module",
   "types": "./dist/index.d.ts",
@@ -49,7 +49,7 @@
     "qa": "bun lint && bun run build && bun knip && bun knip:production && bun run test",
     "release": "release-it",
     "create-plugin": "bun ./scripts/create-new-plugin.ts",
-    "postcreate-plugin": "bun run build && biome format --write schema.json schema-jsonc.json src/ConfigurationValidator.ts",
+    "postcreate-plugin": "bun run build && biome format --write schema.json schema-jsonc.json src/schema/plugins.ts",
     "generate-plugin-defs": "node ./scripts/generate-plugin-defs.js && biome check --write src/plugins/index.ts src/types/PluginNames.ts src/schema/plugins.ts"
   },
   "files": [
@@ -90,7 +90,7 @@
     "@types/webpack": "^5.28.5",
     "@wdio/types": "^9.5.0",
     "glob": "^10.4.2",
-    "release-it": "^18.1.2",
+    "release-it": "^19.0.0-next.1",
     "type-fest": "^4.31.0",
     "typescript": "^5.5.2"
   },

package/schema.json

@@ -102,6 +102,11 @@
       "title": "Include entry files when reporting unused exports",
       "type": "boolean"
     },
+    "tags": {
+      "title": "Exclude (-) or include (+) exports with the specified JSDoc/TSDoc tags",
+      "examples": ["+custom", "-lintignore", "-@internal"],
+      "$ref": "#/definitions/list"
+    },
     "workspaces": {
       "title": "Configuration for workspaces",
       "type": "object",