knip 2.16.2 → 2.17.1

package/README.md

@@ -152,8 +152,8 @@ Using workspaces in a monorepo? Please see [workspaces][1] for more details abou
       -d, --debug              Show debug output
       --debug-file-filter      Filter for files in debug output (regex as string)
       --performance            Measure count and running time of expensive functions and display stats table
-      --h, --help              Print this help text
-      --V, version             Print version
+      -h, --help               Print this help text
+      -V, --version            Print version
 
     (1) Issue types: files, dependencies, unlisted, unresolved, exports, nsExports, classMembers, types, nsTypes, enumMembers, duplicates
 

package/dist/binaries/resolvers/c8.js

@@ -1,9 +1,10 @@
 import parseArgs from 'minimist';
-import { toBinary } from '../util.js';
+import { toBinary, argsFrom } from '../util.js';
 export const resolve = (binary, args, { fromArgs }) => {
     const parsed = parseArgs(args, {
         boolean: ['all', 'check-coverage', 'clean', 'exclude-after-remap', 'per-file', 'skip-full'],
     });
     parsed._ = parsed._.filter(a => a !== 'check-coverage');
-    return [toBinary(binary), ...fromArgs(parsed._)];
+    const rest = argsFrom(args, parsed._[0]);
+    return [toBinary(binary), ...fromArgs(rest)];
 };

package/dist/binaries/resolvers/dotenv.js

@@ -1,6 +1,6 @@
 import parseArgs from 'minimist';
-import { toBinary } from '../util.js';
+import { toBinary, argsFrom } from '../util.js';
 export const resolve = (binary, args, { fromArgs }) => {
     const parsed = parseArgs(args);
-    return [toBinary(binary), ...fromArgs(parsed._)];
+    return [toBinary(binary), ...fromArgs(argsFrom(args, parsed._[0]))];
 };

package/dist/binaries/resolvers/node.js

@@ -2,6 +2,9 @@ import parseArgs from 'minimist';
 import { compact } from '../../util/array.js';
 import { tryResolveFilePath, tryResolveSpecifiers } from '../util.js';
 export const resolve = (binary, args, { cwd }) => {
-    const parsed = parseArgs(args, { string: ['r'], alias: { require: ['r', 'loader', 'experimental-loader'] } });
+    const parsed = parseArgs(args, {
+        string: ['r'],
+        alias: { require: ['r', 'loader', 'experimental-loader', 'test-reporter'] },
+    });
     return compact([tryResolveFilePath(cwd, parsed._[0]), ...tryResolveSpecifiers(cwd, [parsed.require].flat())]);
 };

package/dist/binaries/util.d.ts

@@ -4,3 +4,4 @@ export declare const toBinary: (specifier: string) => string;
 export declare const fromBinary: (specifier: string) => string;
 export declare const isBinary: (specifier: string) => boolean;
 export declare const stripBinaryPath: (command: string) => string;
+export declare const argsFrom: (args: string[], from: string) => string[];

package/dist/binaries/util.js

@@ -30,3 +30,4 @@ export const stripBinaryPath = (command) => stripNodeModulesFromPath(command)
     .replace(/^(\.bin\/)/, '')
     .replace(/\$\(npm bin\)\/(\w+)/, '$1')
     .replace(/(\S+)@.*/, '$1');
+export const argsFrom = (args, from) => args.slice(args.indexOf(from));

package/dist/index.js

@@ -202,8 +202,6 @@ export const main = async (unresolvedConfiguration) => {
                             importedModule.isReExport = importItems.isReExport;
                             importedModule.isReExportedBy.add(filePath);
                         }
-                        if (importItems.isDynamic)
-                            importedModule.isDynamic = importItems.isDynamic;
                         if (importItems.isStar)
                             importedModule.isStar = importItems.isStar;
                     }

package/dist/plugins/svelte/index.js

@@ -4,6 +4,6 @@ export const ENABLERS = ['svelte'];
 export const isEnabled = ({ dependencies }) => hasDependency(dependencies, ENABLERS);
 export const ENTRY_FILE_PATTERNS = ['svelte.config.js', 'vite.config.ts'];
 export const PRODUCTION_ENTRY_FILE_PATTERNS = [
-    'src/routes/**/+{page,page.server,error,layout,layout.server}{,@*}.{js,ts,svelte}',
+    'src/routes/**/+{page,server,page.server,error,layout,layout.server}{,@*}.{js,ts,svelte}',
 ];
 export const PROJECT_FILE_PATTERNS = ['src/**/*.{js,ts,svelte}'];

package/dist/types/compilers.d.ts

@@ -1,6 +1,6 @@
 type FileExtension = string;
-export type SyncCompilerFn = (source: string) => string;
-export type AsyncCompilerFn = (source: string) => Promise<string>;
+export type SyncCompilerFn = (source: string, path: string) => string;
+export type AsyncCompilerFn = (source: string, path: string) => Promise<string>;
 export type SyncCompilers = Map<FileExtension, SyncCompilerFn>;
 export type AsyncCompilers = Map<FileExtension, AsyncCompilerFn>;
 export {};

package/dist/types/imports.d.ts

@@ -8,7 +8,6 @@ export type ImportedModule = {
     isStar: boolean;
     isReExport: boolean;
     isReExportedBy: Set<string>;
-    isDynamic: boolean;
 };
 export type Imports = Map<FilePath, ImportedModule>;
 export {};

package/dist/typescript/SourceFileManager.js

@@ -23,7 +23,7 @@ export class SourceFileManager {
             throw new Error(`Unable to read ${filePath}`);
         const ext = extname(filePath);
         const compiler = this.syncCompilers?.get(ext);
-        const compiled = compiler ? compiler(contents) : contents;
+        const compiled = compiler ? compiler(contents, filePath) : contents;
         if (compiler)
             debugLog(`Compiled ${filePath}`);
         return this.createSourceFile(filePath, compiled);
@@ -45,7 +45,7 @@ export class SourceFileManager {
         const ext = extname(filePath);
         const compiler = this.asyncCompilers?.get(ext);
         if (compiler) {
-            const compiled = await compiler(contents);
+            const compiled = await compiler(contents, filePath);
             debugLog(`Compiled ${filePath}`);
             this.createSourceFile(filePath, compiled);
         }

package/dist/typescript/getImportsAndExports.d.ts

@@ -10,7 +10,6 @@ export type AddImportOptions = {
     specifier: string;
     symbol?: ts.Symbol;
     identifier?: string;
-    isDynamic?: boolean;
     isReExport?: boolean;
 };
 export type AddExportOptions = ExportItem & {

package/dist/typescript/getImportsAndExports.js

@@ -22,7 +22,7 @@ export const getImportsAndExports = (sourceFile, options) => {
     const importedInternalSymbols = new Map();
     const visitors = getVisitors(sourceFile);
     const addInternalImport = (options) => {
-        const { identifier, specifier, symbol, filePath, isDynamic, isReExport } = options;
+        const { identifier, specifier, symbol, filePath, isReExport } = options;
         const isStar = identifier === '*';
         const internalImport = getOrSet(internalImports, filePath, {
             specifier,
@@ -30,7 +30,6 @@ export const getImportsAndExports = (sourceFile, options) => {
             isReExport,
             isReExportedBy: new Set(),
             symbols: new Set(),
-            isDynamic,
         });
         if (isReExport) {
             internalImport.isReExport = isReExport;
@@ -42,11 +41,9 @@ export const getImportsAndExports = (sourceFile, options) => {
             internalImport.symbols.add(identifier);
         if (isStar && symbol)
             importedInternalSymbols.set(symbol, filePath);
-        if (isDynamic)
-            internalImport.isDynamic = isDynamic;
     };
     const addImport = (options) => {
-        const { specifier, symbol, identifier = '__anonymous', isDynamic = false, isReExport = false } = options;
+        const { specifier, symbol, identifier = '__anonymous', isReExport = false } = options;
         if (isBuiltin(specifier))
             return;
         const module = sourceFile.resolvedModules?.get(specifier, undefined);
@@ -55,7 +52,7 @@ export const getImportsAndExports = (sourceFile, options) => {
             if (filePath) {
                 if (module.resolvedModule.isExternalLibraryImport) {
                     if (!isInNodeModules(filePath)) {
-                        addInternalImport({ identifier, specifier, symbol, filePath, isDynamic, isReExport });
+                        addInternalImport({ identifier, specifier, symbol, filePath, isReExport });
                     }
                     if (!isMaybePackageName(specifier))
                         return;
@@ -67,7 +64,7 @@ export const getImportsAndExports = (sourceFile, options) => {
                     }
                 }
                 else {
-                    addInternalImport({ identifier, specifier, symbol, filePath, isDynamic, isReExport });
+                    addInternalImport({ identifier, specifier, symbol, filePath, isReExport });
                 }
             }
         }

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 tests, devDependencies, exported types)\n  --strict                 Consider only direct dependencies of workspace (not devDependencies, not other workspaces)\n  --workspace              Analyze a single workspace (default: analyze all configured workspaces)\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,unresolved\n  --exports                Shortcut for --include exports,nsExports,classMembers,types,nsTypes,enumMembers,duplicates\n  --include-entry-exports  Include entry files when reporting unused exports\n  -n, --no-progress        Don't show dynamic progress updates\n  --reporter               Select reporter: symbols, compact, codeowners, json (default: symbols)\n  --reporter-options       Pass extra options to the reporter (as JSON string, see example)\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  --debug-file-filter      Filter for files in debug output (regex as string)\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\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 --debug --debug-file-filter '(specific|particular)-module'\n\nMore documentation and bug reports: https://github.com/webpro/knip";
+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 tests, devDependencies, exported types)\n  --strict                 Consider only direct dependencies of workspace (not devDependencies, not other workspaces)\n  --workspace              Analyze a single workspace (default: analyze all configured workspaces)\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,unresolved\n  --exports                Shortcut for --include exports,nsExports,classMembers,types,nsTypes,enumMembers,duplicates\n  --include-entry-exports  Include entry files when reporting unused exports\n  -n, --no-progress        Don't show dynamic progress updates\n  --reporter               Select reporter: symbols, compact, codeowners, json (default: symbols)\n  --reporter-options       Pass extra options to the reporter (as JSON string, see example)\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  --debug-file-filter      Filter for files in debug output (regex as string)\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\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 --debug --debug-file-filter '(specific|particular)-module'\n\nMore documentation and bug reports: https://github.com/webpro/knip";
 declare const _default: {
     config: string | undefined;
     debug: boolean | undefined;

package/dist/util/cli-arguments.js

@@ -24,8 +24,8 @@ Options:
   -d, --debug              Show debug output
   --debug-file-filter      Filter for files in debug output (regex as string)
   --performance            Measure count and running time of expensive functions and display stats table
-  --h, --help              Print this help text
-  --V, version             Print version
+  -h, --help               Print this help text
+  -V, --version            Print version
 
 (1) Issue types: files, dependencies, unlisted, unresolved, exports, nsExports, classMembers, types, nsTypes, enumMembers, duplicates
 

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "2.16.2";
+export declare const version = "2.17.1";

package/dist/version.js

@@ -1 +1 @@
-export const version = '2.16.2';
+export const version = '2.17.1';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "2.16.2",
+  "version": "2.17.1",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://github.com/webpro/knip",
   "repository": "github:webpro/knip",
@@ -24,7 +24,7 @@
     "format": "prettier scripts src tests schema.json --with-node-modules --write --config .prettierrc",
     "pretest": "node rmdir.js tmp && swc src -d tmp/src && swc tests -d tmp/tests",
     "test": "node --no-warnings --test tmp",
-    "coverage": "c8 npm test",
+    "coverage": "c8 --reporter html node --no-warnings --loader tsx --test tests/*.test.ts tests/*/*.test.ts",
     "watch": "tsc --watch",
     "prebuild": "node rmdir.js dist",
     "build": "tsc",