knip 2.16.1 → 2.17.0

package/dist/index.js

@@ -181,7 +181,7 @@ export const main = async (unresolvedConfiguration) => {
         const analyzeSourceFile = (filePath) => {
             const workspace = chief.findWorkspaceByFilePath(filePath);
             if (workspace) {
-                const { imports, exports, scripts } = principal.analyzeSourceFile(filePath, { skipTypeOnly: !isReportTypes });
+                const { imports, exports, scripts } = principal.analyzeSourceFile(filePath, { skipTypeOnly: isProduction });
                 const { internal, external, unresolved } = imports;
                 const { exported, duplicate } = exports;
                 if (exported.size > 0)
@@ -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/webpack/index.js

@@ -1,4 +1,5 @@
 import { compact } from '../../util/array.js';
+import { join } from '../../util/path.js';
 import { timerify } from '../../util/Performance.js';
 import { hasDependency, load } from '../../util/plugin.js';
 import { getDependenciesFromConfig } from '../babel/index.js';
@@ -53,13 +54,13 @@ const findWebpackDependencies = async (configFilePath, { manifest, isProduction
         const cfg = typeof config === 'function' ? config(env, argv) : config;
         return [cfg].flat().flatMap(config => {
             const dependencies = (config.module?.rules?.flatMap(resolveRuleSetDependencies) ?? []).map(loader => loader.replace(/\?.*/, ''));
-            const entries = cfg.entry
+            const entries = (cfg.entry
                 ? typeof cfg.entry === 'string'
                     ? [cfg.entry]
                     : Array.isArray(cfg.entry)
                         ? cfg.entry
                         : Object.values(cfg.entry).map(entry => (typeof entry === 'string' ? entry : entry.filename))
-                : [];
+                : []).map(entry => (config.context ? join(config.context, entry) : entry));
             return [...dependencies, ...entries];
         });
     });

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

@@ -7,7 +7,6 @@ interface ValidImportTypeNode extends ts.ImportTypeNode {
 export declare function isValidImportTypeNode(node: ts.Node): node is ValidImportTypeNode;
 export declare function isPrivateMember(node: ts.MethodDeclaration | ts.PropertyDeclaration): boolean;
 export declare function isDefaultImport(node: ts.ImportDeclaration | ts.ImportEqualsDeclaration | ts.ExportDeclaration): boolean;
-export declare function isVariableDeclarationList(node: ts.Node): node is ts.VariableDeclarationList;
 export declare function isAccessExpression(node: ts.Node): node is ts.PropertyAccessExpression | ts.ElementAccessExpression;
 export declare function isImportCall(node: ts.Node): node is ts.ImportCall;
 export declare function isRequireCall(callExpression: ts.Node): callExpression is ts.CallExpression;

package/dist/typescript/ast-helpers.js

@@ -8,9 +8,6 @@ export function isPrivateMember(node) {
 export function isDefaultImport(node) {
     return node.kind === ts.SyntaxKind.ImportDeclaration && !!node.importClause && !!node.importClause.name;
 }
-export function isVariableDeclarationList(node) {
-    return node.kind === ts.SyntaxKind.VariableDeclarationList;
-}
 export function isAccessExpression(node) {
     return ts.isPropertyAccessExpression(node) || ts.isElementAccessExpression(node);
 }

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/typescript/visitors/imports/importCall.js

@@ -1,40 +1,11 @@
 import ts from 'typescript';
-import { isImportCall, isAccessExpression, isVariableDeclarationList, findDescendants } from '../../ast-helpers.js';
+import { isImportCall } from '../../ast-helpers.js';
 import { importVisitor as visit } from '../index.js';
 export default visit(() => true, node => {
     if (isImportCall(node)) {
         if (node.arguments[0] && ts.isStringLiteralLike(node.arguments[0])) {
             const specifier = node.arguments[0].text;
-            if (isAccessExpression(node.parent)) {
-                return { specifier, isDynamic: true };
-            }
-            let _ancestor = node.parent?.parent?.parent;
-            if (_ancestor && isAccessExpression(_ancestor)) {
-                return { specifier, isDynamic: true };
-            }
-            while (_ancestor) {
-                if (_ancestor) {
-                    if (isVariableDeclarationList(_ancestor)) {
-                        return findDescendants(_ancestor, ts.isVariableDeclaration).flatMap(variableDeclaration => {
-                            if (ts.isIdentifier(variableDeclaration.name)) {
-                                return { identifier: 'default', specifier };
-                            }
-                            else {
-                                const binds = findDescendants(variableDeclaration, _node => ts.isBindingElement(_node) && ts.isIdentifier(_node.name));
-                                return binds.flatMap(element => {
-                                    const symbol = element.propertyName?.getText() || element.name.getText();
-                                    return { identifier: symbol, specifier };
-                                });
-                            }
-                        });
-                    }
-                    if (ts.isPropertyAssignment(_ancestor)) {
-                        return { identifier: 'default', specifier };
-                    }
-                }
-                _ancestor = _ancestor.parent;
-            }
-            return { specifier };
+            return { specifier, identifier: 'default' };
         }
     }
 });

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "2.16.1";
+export declare const version = "2.17.0";

package/dist/version.js

@@ -1 +1 @@
-export const version = '2.16.1';
+export const version = '2.17.0';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "2.16.1",
+  "version": "2.17.0",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://github.com/webpro/knip",
   "repository": "github:webpro/knip",
@@ -22,10 +22,11 @@
     "lint": "eslint scripts src tests",
     "lint:fix": "eslint scripts src tests --fix",
     "format": "prettier scripts src tests schema.json --with-node-modules --write --config .prettierrc",
-    "test": "glob -c \"node --no-warnings --loader tsx --test\" \"tests/**/*.test.ts\"",
-    "coverage": "c8 npm test",
+    "pretest": "node rmdir.js tmp && swc src -d tmp/src && swc tests -d tmp/tests",
+    "test": "node --no-warnings --test tmp",
+    "coverage": "c8 --reporter html node --no-warnings --loader tsx --test tests/*.test.ts tests/*/*.test.ts",
     "watch": "tsc --watch",
-    "prebuild": "node -e \"require('fs').rmSync('dist', { force: true, recursive: true })\"",
+    "prebuild": "node rmdir.js dist",
     "build": "tsc",
     "docs": "npm run docs:cli && npm run docs:plugins && npm run docs:format",
     "docs:cli": "tsx ./scripts/update-cli-usage-in-readme.ts",
@@ -63,6 +64,8 @@
     "@jest/types": "29.6.1",
     "@npmcli/package-json": "4.0.1",
     "@release-it/bumper": "5.0.0",
+    "@swc/cli": "0.1.62",
+    "@swc/core": "1.3.70",
     "@types/eslint": "8.44.0",
     "@types/js-yaml": "4.0.5",
     "@types/micromatch": "4.0.2",
@@ -77,7 +80,6 @@
     "eslint-import-resolver-typescript": "3.5.5",
     "eslint-plugin-import": "2.27.5",
     "eslint-plugin-n": "16.0.1",
-    "glob": "10.3.3",
     "prettier": "3.0.0",
     "release-it": "16.1.3",
     "remark-cli": "11.0.0",