knip 0.9.1 → 0.11.0

package/README.md

@@ -83,9 +83,10 @@ Knip works by creating two sets of files:
       --ignore               Ignore files matching this glob pattern (can be set multiple times)
       --no-gitignore         Don't use .gitignore
       --dev                  Include `devDependencies` in report(s)
+      --include-entry-files  Report unused exports and types for entry files
       --no-progress          Don't show dynamic progress updates
       --max-issues           Maximum number of issues before non-zero exit code (default: 0)
-      --reporter             Select reporter: symbols, compact, codeowners (default: symbols)
+      --reporter             Select reporter: symbols, compact, codeowners, json (default: symbols)
       --reporter-options     Pass extra options to the reporter (as JSON string, see example)
       --jsdoc                Enable JSDoc parsing, with options: public
       --debug                Show debug output
@@ -125,10 +126,12 @@ following types of issues:
 - `nsTypes` - Unused types in namespaces: did not find direct references to this exported variable (2)
 - `duplicates` - Duplicate exports: the same thing is exported more than once with different names from the same file
 
-1.  This may also include dependencies that could not be resolved properly (such as non-relative `local/dir/file.ts` not
-    and `local` not being in `node_modules`).
+1.  This includes dependencies that could not be resolved. For instance, what does `unresolved/dir/module` mean?
+    - To target something in the (missing) `node_modules/unresolved` package?
+    - Target a local module that should have a relative path?
+    - It does not match any `paths` entry in `tsconfig.json#compilerOptions`.
 2.  The variable or type is not referenced directly, and has become a member of a namespace. That's why Knip is not sure
-    whether this export can be removed, so please look into it:
+    whether this export can be removed, so please look into it.
 
 You can `--include` or `--exclude` any of the types to slice & dice the report to your needs. Alternatively, they can be
 added to the configuration (e.g. `"exclude": ["dependencies"]`).
@@ -140,8 +143,8 @@ As always, make sure to backup files or use Git before deleting files or making
 - Unused files can be removed.
 - Unused dependencies can be removed from `package.json`.
 - Unlisted dependencies should be added to `package.json`.
-- Unused exports and types: remove the `export` keyword in front of unused exports. Then you (or tools such as the
-  TypeScript language server in VS Code and/or ESLint) can see whether the variable or type is used within the same
+- Unused exports and types: remove the `export` keyword in front of unused exports. Then you (or tools such as
+  TypeScript language services in VS Code and/or ESLint) can see whether the variable or type is used within the same
   file. If this is not the case, it can be removed.
 
 🔁 Repeat the process to reveal new unused files and exports. Sometimes it's so liberating to remove things!
@@ -307,10 +310,23 @@ per file like this:
 
 ```json
 [
+  {
+    "file": "package.json",
+    "owners": ["@org/admin"],
+    "files": false,
+    "dependencies": ["jquery", "moment"],
+    "devDependencies": [],
+    "unlisted": [],
+    "exports": [],
+    "types": [],
+    "duplicates": []
+  },
   {
     "file": "src/Registration.tsx",
     "owners": ["@org/owner"],
     "files": true,
+    "dependencies": [],
+    "devDependencies": [],
     "unlisted": ["react"],
     "exports": ["lowercaseFirstLetter", "RegistrationBox"],
     "types": ["RegistrationServices", "RegistrationAction"],

package/dist/help.js

@@ -16,7 +16,7 @@ Options:
   --include-entry-files  Report unused exports and types for entry files
   --no-progress          Don't show dynamic progress updates
   --max-issues           Maximum number of issues before non-zero exit code (default: 0)
-  --reporter             Select reporter: symbols, compact, codeowners (default: symbols)
+  --reporter             Select reporter: symbols, compact, codeowners, json (default: symbols)
   --reporter-options     Pass extra options to the reporter (as JSON string, see example)
   --jsdoc                Enable JSDoc parsing, with options: public
   --debug                Show debug output

package/dist/index.d.ts

@@ -1,5 +1,5 @@
 import type { UnresolvedConfiguration } from './types';
-export declare const main: (options: UnresolvedConfiguration) => Promise<{
+export declare const main: (unresolvedConfiguration: UnresolvedConfiguration) => Promise<{
     report: import("./types").Report;
     issues: import("./types").Issues;
     counters: import("./types").Counters;

package/dist/index.js

@@ -8,18 +8,22 @@ const typescript_1 = __importDefault(require("typescript"));
 const config_1 = require("./util/config");
 const fs_1 = require("./util/fs");
 const path_1 = require("./util/path");
+const glob_1 = require("./util/glob");
 const project_1 = require("./util/project");
 const runner_1 = require("./runner");
 const errors_1 = require("./util/errors");
 const debug_1 = require("./util/debug");
 const progress_1 = require("./progress");
-const main = async (options) => {
-    const { cwd, workingDir, configFilePath: configFilePathArg, tsConfigFilePath: tsConfigFilePathArg, include, exclude, ignore, gitignore, isIncludeEntryFiles, isDev, isShowProgress, jsDoc, debug, } = options;
-    const updateMessage = (0, progress_1.getMessageUpdater)(options);
-    (0, debug_1.debugLogObject)(options, 1, 'Unresolved configuration', options);
+const main = async (unresolvedConfiguration) => {
+    const { cwd, workingDir, configFilePath: configFilePathArg, tsConfigFilePath: tsConfigFilePathArg, include, exclude, ignore, gitignore, isIncludeEntryFiles, isDev, isShowProgress, jsDoc, debug, } = unresolvedConfiguration;
+    const updateMessage = (0, progress_1.getMessageUpdater)(unresolvedConfiguration);
+    (0, debug_1.debugLogObject)(debug, 1, 'Unresolved configuration', unresolvedConfiguration);
     updateMessage('Reading configuration and manifest files...');
     const manifestPath = await (0, fs_1.findFile)(cwd, workingDir, 'package.json');
     const manifest = manifestPath && require(manifestPath);
+    if (!manifestPath || !manifest) {
+        throw new errors_1.ConfigurationError('Unable to find package.json');
+    }
     const configFilePath = configFilePathArg ?? 'knip.json';
     const resolvedConfigFilePath = await (0, fs_1.findFile)(cwd, workingDir, configFilePath);
     const localConfig = resolvedConfigFilePath && require(resolvedConfigFilePath);
@@ -31,10 +35,10 @@ const main = async (options) => {
     if (tsConfigFilePathArg && !resolvedTsConfigFilePath) {
         throw new errors_1.ConfigurationError(`Unable to find ${tsConfigFilePathArg}`);
     }
-    let tsConfigPaths = [];
+    let tsConfigPathGlobs = [];
     if (resolvedTsConfigFilePath) {
         const config = typescript_1.default.readConfigFile(resolvedTsConfigFilePath, typescript_1.default.sys.readFile);
-        tsConfigPaths = config.config.compilerOptions?.paths
+        tsConfigPathGlobs = config.config.compilerOptions?.paths
             ? Object.keys(config.config.compilerOptions.paths).map(p => p.replace(/\*/g, '**'))
             : [];
         if (config.error) {
@@ -43,7 +47,7 @@ const main = async (options) => {
     }
     const dir = (0, path_1.relative)(workingDir);
     const resolvedConfig = (0, config_1.resolveConfig)(manifest.knip ?? localConfig, { workingDir: dir, isDev });
-    (0, debug_1.debugLogObject)(options, 1, 'Resolved configuration', resolvedConfig);
+    (0, debug_1.debugLogObject)(debug, 1, 'Resolved configuration', resolvedConfig);
     if (!resolvedConfigFilePath && !manifest.knip && !resolvedTsConfigFilePath) {
         throw new errors_1.ConfigurationError(`Unable to find ${configFilePath} or package.json#knip or ${tsConfigFilePath}`);
     }
@@ -54,30 +58,32 @@ const main = async (options) => {
                 ? { tsConfigFilePath: resolvedTsConfigFilePath }
                 : { compilerOptions: { allowJs: true } };
             updateMessage('Resolving entry files...');
-            const entryPaths = await (0, path_1.resolvePaths)({
+            const entryPaths = await (0, glob_1.glob)({
+                cwd,
                 workingDir,
                 patterns: resolvedConfig.entryFiles,
                 ignore,
                 gitignore,
             });
-            (0, debug_1.debugLogFiles)(options, 1, 'Globbed entry paths', entryPaths);
+            (0, debug_1.debugLogFiles)(debug, 1, 'Globbed entry paths', entryPaths);
             const production = (0, project_1.createProject)({ ...projectOptions, ...skipAddFiles }, entryPaths);
             const entryFiles = production.getSourceFiles();
-            (0, debug_1.debugLogSourceFiles)(options, 1, 'Included entry source files', entryFiles);
+            (0, debug_1.debugLogSourceFiles)(debug, 1, 'Resolved entry source files', entryFiles);
             production.resolveSourceFileDependencies();
-            const productionFiles = production.getSourceFiles();
-            (0, debug_1.debugLogSourceFiles)(options, 1, 'Included production source files', productionFiles);
+            const productionFiles = (0, project_1.removeExternalSourceFiles)(production);
+            (0, debug_1.debugLogSourceFiles)(debug, 1, 'Resolved production source files', productionFiles);
             updateMessage('Resolving project files...');
-            const projectPaths = await (0, path_1.resolvePaths)({
+            const projectPaths = await (0, glob_1.glob)({
+                cwd,
                 workingDir,
                 patterns: resolvedConfig.projectFiles,
                 ignore,
                 gitignore,
             });
-            (0, debug_1.debugLogFiles)(options, 1, 'Globbed project paths', projectPaths);
+            (0, debug_1.debugLogFiles)(debug, 1, 'Globbed project paths', projectPaths);
             const project = (0, project_1.createProject)({ ...projectOptions, ...skipAddFiles }, projectPaths);
             const projectFiles = project.getSourceFiles();
-            (0, debug_1.debugLogSourceFiles)(options, 1, 'Included project source files', projectFiles);
+            (0, debug_1.debugLogSourceFiles)(debug, 1, 'Resolved project source files', projectFiles);
             return { entryFiles, productionFiles, projectFiles };
         }
         else {
@@ -95,12 +101,13 @@ const main = async (options) => {
         productionFiles,
         projectFiles,
         isIncludeEntryFiles: !resolvedConfig || isIncludeEntryFiles,
+        manifestPath,
         dependencies: Object.keys(manifest.dependencies ?? {}),
         peerDependencies: Object.keys(manifest.peerDependencies ?? {}),
         optionalDependencies: Object.keys(manifest.optionalDependencies ?? {}),
         devDependencies: Object.keys(manifest.devDependencies ?? {}),
         isDev: Boolean(resolvedConfig?.dev),
-        tsConfigPaths,
+        tsConfigPathGlobs: tsConfigPathGlobs,
         isShowProgress,
         jsDocOptions: {
             isReadPublicTag: jsDoc.includes('public'),
@@ -108,7 +115,7 @@ const main = async (options) => {
         debug,
     };
     const { issues, counters } = await (0, runner_1.findIssues)(config);
-    (0, debug_1.debugLogObject)(options, 2, 'Issues', issues);
+    (0, debug_1.debugLogObject)(debug, 2, 'Issues', issues);
     return { report, issues, counters };
 };
 exports.main = main;

package/dist/reporters/json.js

@@ -26,6 +26,8 @@ exports.default = async ({ report, issues, options }) => {
             file,
             ...(codeownersEngine && { owners: codeownersEngine.calcFileOwnership(file) }),
             ...(report.files && { files: false }),
+            ...(report.dependencies && { dependencies: [] }),
+            ...(report.devDependencies && { devDependencies: [] }),
             ...(report.unlisted && { unlisted: [] }),
             ...((report.exports || report.nsExports) && { exports: [] }),
             ...((report.types || report.nsTypes) && { types: [] }),

package/dist/runner.js

@@ -13,20 +13,21 @@ const dependencies_1 = require("./util/dependencies");
 const debug_1 = require("./util/debug");
 const progress_1 = require("./progress");
 async function findIssues(configuration) {
-    const { workingDir, report, isDev, jsDocOptions } = configuration;
+    const { workingDir, report, isDev, jsDocOptions, debug } = configuration;
     const { entryFiles, productionFiles, projectFiles, isIncludeEntryFiles } = configuration;
+    const { manifestPath } = configuration;
     const updateMessage = (0, progress_1.getMessageUpdater)(configuration);
     const { getUnresolvedDependencies, getUnusedDependencies, getUnusedDevDependencies } = (0, dependencies_1.getDependencyAnalyzer)(configuration);
     const [usedProductionFiles, unreferencedProductionFiles] = (0, project_1.partitionSourceFiles)(projectFiles, productionFiles);
     const [usedEntryFiles, usedNonEntryFiles] = (0, project_1.partitionSourceFiles)(usedProductionFiles, entryFiles);
-    (0, debug_1.debugLogSourceFiles)(configuration, 1, 'Used production files', usedProductionFiles);
-    (0, debug_1.debugLogSourceFiles)(configuration, 1, 'Unreferenced production files', unreferencedProductionFiles);
-    (0, debug_1.debugLogSourceFiles)(configuration, 1, 'Used entry files', usedEntryFiles);
-    (0, debug_1.debugLogSourceFiles)(configuration, 1, 'Used non-entry files', usedNonEntryFiles);
+    (0, debug_1.debugLogSourceFiles)(debug, 1, 'Used production files', usedProductionFiles);
+    (0, debug_1.debugLogSourceFiles)(debug, 1, 'Unreferenced production files', unreferencedProductionFiles);
+    (0, debug_1.debugLogSourceFiles)(debug, 1, 'Used entry files', usedEntryFiles);
+    (0, debug_1.debugLogSourceFiles)(debug, 1, 'Used non-entry files', usedNonEntryFiles);
     const issues = {
         files: new Set(unreferencedProductionFiles.map(file => file.getFilePath())),
-        dependencies: new Set(),
-        devDependencies: new Set(),
+        dependencies: {},
+        devDependencies: {},
         unlisted: {},
         exports: {},
         types: {},
@@ -36,8 +37,8 @@ async function findIssues(configuration) {
     };
     const counters = {
         files: issues.files.size,
-        dependencies: issues.dependencies.size,
-        devDependencies: issues.dependencies.size,
+        dependencies: 0,
+        devDependencies: 0,
         unlisted: 0,
         exports: 0,
         types: 0,
@@ -50,19 +51,12 @@ async function findIssues(configuration) {
     const updateCounters = (0, progress_1.getCountersUpdater)(configuration, counters);
     const addSymbolIssue = (issueType, issue) => {
         const { filePath, symbol } = issue;
-        const key = node_path_1.default.relative(workingDir, filePath);
+        const key = node_path_1.default.relative(workingDir, filePath).replace(/\\/g, '/');
         issues[issueType][key] = issues[issueType][key] ?? {};
         issues[issueType][key][symbol] = issue;
         counters[issueType]++;
         updateCounters(issue);
     };
-    const addProjectIssue = (issueType, issue) => {
-        if (!issues[issueType].has(issue.symbol)) {
-            issues[issueType].add(issue.symbol);
-            counters[issueType]++;
-        }
-        updateCounters(issue);
-    };
     updateMessage('Connecting the dots...');
     if (report.dependencies ||
         report.unlisted ||
@@ -174,10 +168,10 @@ async function findIssues(configuration) {
     }
     if (report.dependencies) {
         const unusedDependencies = getUnusedDependencies();
-        unusedDependencies.forEach(symbol => addProjectIssue('dependencies', { filePath: '', symbol }));
+        unusedDependencies.forEach(symbol => addSymbolIssue('dependencies', { filePath: manifestPath, symbol }));
         if (isDev) {
             const unusedDevDependencies = getUnusedDevDependencies();
-            unusedDevDependencies.forEach(symbol => addProjectIssue('devDependencies', { filePath: '', symbol }));
+            unusedDevDependencies.forEach(symbol => addSymbolIssue('devDependencies', { filePath: manifestPath, symbol }));
         }
     }
     updateCounters();

package/dist/types.d.ts

@@ -10,8 +10,8 @@ export declare type IssueSet = Set<string>;
 export declare type IssueRecords = Record<string, Record<string, Issue>>;
 export declare type Issues = {
     files: IssueSet;
-    dependencies: IssueSet;
-    devDependencies: IssueSet;
+    dependencies: IssueRecords;
+    devDependencies: IssueRecords;
     unlisted: IssueRecords;
     exports: IssueRecords;
     types: IssueRecords;
@@ -20,8 +20,7 @@ export declare type Issues = {
     duplicates: IssueRecords;
 };
 export declare type IssueType = keyof Issues;
-export declare type ProjectIssueType = Extract<IssueType, 'files' | 'dependencies' | 'devDependencies'>;
-export declare type SymbolIssueType = Exclude<IssueType, ProjectIssueType>;
+export declare type SymbolIssueType = Exclude<IssueType, 'files'>;
 export declare type Report = {
     [key in keyof Issues]: boolean;
 };
@@ -60,12 +59,13 @@ export declare type Configuration = {
     productionFiles: SourceFile[];
     entryFiles: SourceFile[];
     isIncludeEntryFiles: boolean;
+    manifestPath: string;
     dependencies: string[];
     peerDependencies: string[];
     optionalDependencies: string[];
     devDependencies: string[];
     isDev: boolean;
-    tsConfigPaths: string[];
+    tsConfigPathGlobs: string[];
     isShowProgress: boolean;
     jsDocOptions: {
         isReadPublicTag: boolean;

package/dist/util/config.d.ts

@@ -1,6 +1,6 @@
-import type { ImportedConfiguration, LocalConfiguration } from '../types';
+import type { ImportedConfiguration, LocalConfiguration, Report } from '../types';
 export declare const resolveConfig: (importedConfiguration: ImportedConfiguration, options?: {
     workingDir?: string;
     isDev?: boolean;
 }) => LocalConfiguration | undefined;
-export declare const resolveIncludedIssueTypes: (includeArg: string[], excludeArg: string[], resolvedConfig?: LocalConfiguration) => import("../types").Report;
+export declare const resolveIncludedIssueTypes: (includeArg: string[], excludeArg: string[], resolvedConfig?: LocalConfiguration) => Report;

package/dist/util/config.js

@@ -31,15 +31,13 @@ exports.resolveConfig = resolveConfig;
 const resolveIncludedIssueTypes = (includeArg, excludeArg, resolvedConfig) => {
     const deps = resolvedConfig?.dev ? ['dependencies', 'devDependencies'] : ['dependencies'];
     const groups = ['files', ...deps, 'unlisted', 'exports', 'types', 'nsExports', 'nsTypes', 'duplicates'];
-    const include = [includeArg, resolvedConfig?.include ?? []]
-        .flat()
-        .map(value => value.split(','))
-        .flat();
-    const exclude = [excludeArg, resolvedConfig?.exclude ?? []]
-        .flat()
-        .map(value => value.split(','))
-        .flat();
-    const includes = (include.length > 0 ? include : groups).filter(group => !exclude.includes(group));
-    return groups.reduce((r, group) => ((r[group] = includes.includes(group)), r), {});
+    const normalizedIncludesArg = includeArg.map(value => value.split(',')).flat();
+    const normalizedExcludesArg = excludeArg.map(value => value.split(',')).flat();
+    const excludes = (resolvedConfig?.exclude ?? []).filter(exclude => !normalizedIncludesArg.includes(exclude));
+    const includes = (resolvedConfig?.include ?? []).filter(include => !normalizedExcludesArg.includes(include));
+    const include = [normalizedIncludesArg, includes].flat();
+    const exclude = [normalizedExcludesArg, excludes].flat();
+    const included = (include.length > 0 ? include : groups).filter(group => !exclude.includes(group));
+    return groups.reduce((types, group) => ((types[group] = included.includes(group)), types), {});
 };
 exports.resolveIncludedIssueTypes = resolveIncludedIssueTypes;

package/dist/util/debug.d.ts

@@ -1,11 +1,9 @@
 import type { SourceFile } from 'ts-morph';
-declare type Config = {
-    debug: {
-        isEnabled: boolean;
-        level: number;
-    };
+declare type Debug = {
+    isEnabled: boolean;
+    level: number;
 };
-export declare const debugLogObject: (config: Config, minimumLevel: number, name: string, obj: unknown) => void;
-export declare const debugLogFiles: (config: Config, minimumLevel: number, name: string, filePaths: string[]) => void;
-export declare const debugLogSourceFiles: (config: Config, minimumLevel: number, name: string, sourceFiles: SourceFile[]) => void;
+export declare const debugLogObject: (debug: Debug, minimumLevel: number, name: string, obj: unknown) => void;
+export declare const debugLogFiles: (debug: Debug, minimumLevel: number, name: string, filePaths: string[]) => void;
+export declare const debugLogSourceFiles: (debug: Debug, minimumLevel: number, name: string, sourceFiles: SourceFile[]) => void;
 export {};

package/dist/util/debug.js

@@ -6,41 +6,34 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.debugLogSourceFiles = exports.debugLogFiles = exports.debugLogObject = void 0;
 const node_util_1 = __importDefault(require("node:util"));
 const logArray = (collection) => console.log(node_util_1.default.inspect(collection, { maxArrayLength: null }));
-const debugLogObject = (config, minimumLevel, name, obj) => {
-    if (minimumLevel > config.debug.level)
+const debugLogObject = (debug, minimumLevel, name, obj) => {
+    if (minimumLevel > debug.level)
         return;
     console.log(`[knip] ${name}:`);
     console.log(node_util_1.default.inspect(obj, { depth: null, colors: true }));
 };
 exports.debugLogObject = debugLogObject;
-const debugLogFiles = (config, minimumLevel, name, filePaths) => {
-    if (minimumLevel > config.debug.level)
+const debugLogFiles = (debug, minimumLevel, name, filePaths) => {
+    if (minimumLevel > debug.level)
         return;
-    const { debug } = config;
+    console.debug(`[knip] ${name} (${filePaths.length}):`);
     if (debug.level > 1) {
-        console.debug(`[knip] ${name} (${filePaths.length}):`);
         logArray(filePaths);
     }
-    else {
-        console.debug(`[knip] ${name} (${filePaths.length})`);
-    }
 };
 exports.debugLogFiles = debugLogFiles;
-const debugLogSourceFiles = (config, minimumLevel, name, sourceFiles) => {
-    if (minimumLevel > config.debug.level)
+const debugLogSourceFiles = (debug, minimumLevel, name, sourceFiles) => {
+    if (minimumLevel > debug.level)
         return;
-    const { debug } = config;
+    console.debug(`[knip] ${name} (${sourceFiles.length})`);
     if (debug.level > 1) {
-        console.debug(`[knip] ${name} (${sourceFiles.length}):`);
-        logArray(sourceFiles.map(sourceFile => sourceFile.getFilePath()));
-    }
-    else {
-        console.debug(`[knip] ${name} (${sourceFiles.length})`);
+        const files = sourceFiles.map(sourceFile => sourceFile.getFilePath());
+        logArray(files);
     }
 };
 exports.debugLogSourceFiles = debugLogSourceFiles;
-const debugLogDiff = (config, minimumLevel, name, arrA, arrB) => {
-    if (minimumLevel > config.debug.level)
+const debugLogDiff = (debug, minimumLevel, name, arrA, arrB) => {
+    if (minimumLevel > debug.level)
         return;
     const onlyInA = arrA.filter(itemA => !arrB.includes(itemA)).sort();
     const onlyInB = arrB.filter(itemB => !arrA.includes(itemB)).sort();

package/dist/util/dependencies.js

@@ -7,29 +7,35 @@ exports.getDependencyAnalyzer = void 0;
 const ts_morph_1 = require("ts-morph");
 const is_builtin_module_1 = __importDefault(require("is-builtin-module"));
 const micromatch_1 = __importDefault(require("micromatch"));
+const ts_morph_helpers_1 = require("ts-morph-helpers");
 const compact = (collection) => Array.from(new Set(collection)).filter((value) => Boolean(value));
+const findRequireModuleSpecifiers = (sourceFile) => (0, ts_morph_helpers_1.findCallExpressionsByName)(sourceFile, 'require').map(expression => expression.getFirstDescendantByKind(ts_morph_1.ts.SyntaxKind.StringLiteral));
+const isExternalDependency = (moduleSpecifier, tsConfigPathGlobs) => {
+    if (moduleSpecifier.startsWith('.'))
+        return false;
+    if ((0, is_builtin_module_1.default)(moduleSpecifier))
+        return false;
+    if (tsConfigPathGlobs.length > 0 && micromatch_1.default.isMatch(moduleSpecifier, tsConfigPathGlobs))
+        return false;
+    return true;
+};
+const resolvePackageName = (moduleSpecifier) => {
+    const parts = moduleSpecifier.split('/').slice(0, 2);
+    return moduleSpecifier.startsWith('@') ? parts.join('/') : parts[0];
+};
 const getDependencyAnalyzer = (configuration) => {
-    const { dependencies, devDependencies, peerDependencies, optionalDependencies, tsConfigPaths } = configuration;
+    const { dependencies, devDependencies, peerDependencies, optionalDependencies, tsConfigPathGlobs } = configuration;
     const productionDependencies = [...dependencies, ...peerDependencies, ...optionalDependencies];
     const referencedDependencies = new Set();
     const getUnresolvedDependencies = (sourceFile) => {
         const unresolvedDependencies = new Set();
         const importLiterals = sourceFile.getImportStringLiterals();
-        const requires = sourceFile
-            .getDescendantsOfKind(ts_morph_1.ts.SyntaxKind.CallExpression)
-            .filter(callExpression => callExpression.getExpression().getText() === 'require')
-            .map(expression => expression.getFirstDescendantByKind(ts_morph_1.ts.SyntaxKind.StringLiteral));
-        const literals = compact([importLiterals, requires].flat());
-        literals.forEach(importLiteral => {
-            const moduleSpecifier = importLiteral.getLiteralText();
-            if (moduleSpecifier.startsWith('.'))
-                return;
-            if ((0, is_builtin_module_1.default)(moduleSpecifier))
-                return;
-            if (tsConfigPaths.length > 0 && micromatch_1.default.isMatch(moduleSpecifier, tsConfigPaths))
+        const requireCallExpressions = findRequireModuleSpecifiers(sourceFile);
+        const moduleSpecifiers = compact([importLiterals, requireCallExpressions].flat()).map(i => i.getLiteralText());
+        moduleSpecifiers.forEach(moduleSpecifier => {
+            if (!isExternalDependency(moduleSpecifier, tsConfigPathGlobs))
                 return;
-            const parts = moduleSpecifier.split('/').slice(0, 2);
-            const packageName = moduleSpecifier.startsWith('@') ? parts.join('/') : parts[0];
+            const packageName = resolvePackageName(moduleSpecifier);
             if (!productionDependencies.includes(packageName) && !devDependencies.includes(packageName)) {
                 unresolvedDependencies.add({ filePath: sourceFile.getFilePath(), symbol: moduleSpecifier });
             }

package/dist/util/glob.d.ts

@@ -0,0 +1,7 @@
+export declare const glob: ({ cwd, workingDir, patterns, ignore, gitignore, }: {
+    cwd: string;
+    workingDir: string;
+    patterns: string[];
+    ignore: string[];
+    gitignore: boolean;
+}) => Promise<string[]>;

package/dist/util/glob.js

@@ -0,0 +1,29 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.glob = void 0;
+const node_path_1 = __importDefault(require("node:path"));
+let _globby;
+const globProxy = async function (patterns, options) {
+    if (!_globby) {
+        const { globby } = await eval('import("globby")');
+        _globby = globby;
+    }
+    return _globby(patterns, options);
+};
+const prependDirToPattern = (workingDir, pattern) => {
+    if (pattern.startsWith('!'))
+        return '!' + node_path_1.default.posix.join(workingDir, pattern.slice(1));
+    return node_path_1.default.posix.join(workingDir, pattern);
+};
+const glob = async ({ cwd, workingDir, patterns, ignore, gitignore, }) => {
+    return globProxy(patterns.map(pattern => prependDirToPattern(node_path_1.default.posix.relative(cwd, workingDir), pattern)), {
+        cwd,
+        ignore: [...ignore, '**/node_modules/**'],
+        gitignore,
+        absolute: true,
+    });
+};
+exports.glob = glob;

package/dist/util/path.d.ts

@@ -1,7 +1 @@
 export declare const relative: (to: string) => string;
-export declare const resolvePaths: ({ workingDir, patterns, ignore, gitignore, }: {
-    workingDir: string;
-    patterns: string[];
-    ignore: string[];
-    gitignore: boolean;
-}) => Promise<string[]>;

package/dist/util/path.js

@@ -3,28 +3,8 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.resolvePaths = exports.relative = void 0;
+exports.relative = void 0;
 const node_path_1 = __importDefault(require("node:path"));
 const cwd = process.cwd();
 const relative = (to) => node_path_1.default.relative(cwd, to);
 exports.relative = relative;
-let _globby;
-const glob = async function (patterns, options) {
-    if (!_globby) {
-        const { globby } = await eval('import("globby")');
-        _globby = globby;
-    }
-    return _globby(patterns, options);
-};
-const prependDirToPattern = (workingDir, pattern) => {
-    if (pattern.startsWith('!'))
-        return '!' + node_path_1.default.join(workingDir, pattern.slice(1));
-    return node_path_1.default.join(workingDir, pattern);
-};
-const resolvePaths = async ({ workingDir, patterns, ignore, gitignore, }) => glob(patterns.map(pattern => prependDirToPattern((0, exports.relative)(workingDir), pattern)), {
-    cwd,
-    ignore,
-    gitignore,
-    absolute: true,
-});
-exports.resolvePaths = resolvePaths;

package/dist/util/project.d.ts

@@ -1,4 +1,5 @@
 import { Project } from 'ts-morph';
 import type { ProjectOptions, SourceFile } from 'ts-morph';
 export declare const createProject: (projectOptions: ProjectOptions, paths?: string[]) => Project;
+export declare const removeExternalSourceFiles: (project: Project) => SourceFile[];
 export declare const partitionSourceFiles: (projectFiles: SourceFile[], productionFiles: SourceFile[]) => SourceFile[][];

package/dist/util/project.js

@@ -1,14 +1,22 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.partitionSourceFiles = exports.createProject = void 0;
+exports.partitionSourceFiles = exports.removeExternalSourceFiles = exports.createProject = void 0;
 const ts_morph_1 = require("ts-morph");
 const createProject = (projectOptions, paths) => {
-    const workspace = new ts_morph_1.Project(projectOptions);
+    const project = new ts_morph_1.Project(projectOptions);
     if (paths)
-        workspace.addSourceFilesAtPaths(paths);
-    return workspace;
+        project.addSourceFilesAtPaths(paths);
+    return project;
 };
 exports.createProject = createProject;
+const removeExternalSourceFiles = (project) => project.getSourceFiles().filter(sourceFile => {
+    if (/\/node_modules\//.test(sourceFile.getFilePath())) {
+        project.removeSourceFile(sourceFile);
+        return false;
+    }
+    return true;
+});
+exports.removeExternalSourceFiles = removeExternalSourceFiles;
 const partitionSourceFiles = (projectFiles, productionFiles) => {
     const productionFilePaths = productionFiles.map(file => file.getFilePath());
     const usedFiles = [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "0.9.1",
+  "version": "0.11.0",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript project",
   "keywords": [
     "find",
@@ -25,7 +25,7 @@
   },
   "scripts": {
     "knip": "node ./dist/cli.js --jsdoc public",
-    "test": "node --loader tsx --test test/*.spec.ts",
+    "test": "globstar -- node --loader tsx --test \"test/*.spec.ts\"",
     "watch": "tsc --watch",
     "build": "rm -rf dist && tsc",
     "prepublishOnly": "npm test && npm run build && npm run knip",
@@ -50,6 +50,7 @@
   "devDependencies": {
     "@types/micromatch": "4.0.2",
     "@types/node": "18.11.2",
+    "globstar": "1.0.0",
     "prettier": "2.7.1",
     "release-it": "15.5.0",
     "tsx": "3.10.3",