knip 1.0.0-beta.6 → 1.0.0-beta.7

package/README.md

@@ -314,6 +314,14 @@ use the `--production` flag. Here's an example:
 }
 ```
 
+Here's what's included in production mode analysis:
+
+- Only `entry` and `project` patterns suffixed with `!`.
+- Only `entry` patterns from plugins exported as `PRODUCTION_ENTRY_FILE_PATTERNS` (such as Next.js and Gatsby).
+- Only the `npm start` script (e.g. not the `test` or other npm scripts in `package.json`).
+- Only `exports`, `nsExports` and `classMembers` are included in the report (`types`, `nsTypes`, `enumMembers` are
+  ignored).
+
 #### Strict
 
 Additionally, the `--strict` flag can be used to:

package/dist/cli.js

@@ -7,13 +7,13 @@ import { ConfigurationError } from './util/errors.js';
 import { measure } from './util/performance.js';
 import { main } from './index.js';
 register();
-const { values: { debug: isDebug = false, help, 'include-entry-exports': isIncludeEntryExports = false, 'max-issues': maxIssues = '0', 'no-exit-code': noExitCode = false, 'no-gitignore': isNoGitIgnore = false, 'no-progress': noProgress = false, production: isProduction = false, reporter = 'symbols', 'reporter-options': reporterOptions = '', strict: isStrict = false, tsConfig, }, } = parsedArgs;
+const { values: { debug: isDebug = false, help, 'include-entry-exports': isIncludeEntryExports = false, 'max-issues': maxIssues = '0', 'no-exit-code': noExitCode = false, 'no-gitignore': isNoGitIgnore = false, 'no-progress': isNoProgress = false, production: isProduction = false, reporter = 'symbols', 'reporter-options': reporterOptions = '', strict: isStrict = false, tsConfig, }, } = parsedArgs;
 if (help) {
     console.log(helpText);
     process.exit(0);
 }
 const cwd = process.cwd();
-const isShowProgress = !isDebug && noProgress === false && process.stdout.isTTY && typeof process.stdout.cursorTo === 'function';
+const isShowProgress = !isDebug && isNoProgress === false && process.stdout.isTTY && typeof process.stdout.cursorTo === 'function';
 const printReport = reporter in reporters ? reporters[reporter] : await import(path.join(cwd, reporter));
 const run = async () => {
     try {

package/dist/configuration-chief.d.ts

@@ -4,7 +4,7 @@ import { ConfigurationValidator } from './configuration-validator.js';
 import type { Configuration, WorkspaceConfiguration } from './types/config.js';
 import type { PackageJson } from '@npmcli/package-json';
 type ConfigurationManagerOptions = {
-    cwd?: string;
+    cwd: string;
     isProduction: boolean;
 };
 export default class ConfigurationChief {

package/dist/configuration-chief.js

@@ -30,14 +30,14 @@ const defaultConfig = {
 };
 const PLUGIN_NAMES = Object.keys(plugins);
 export default class ConfigurationChief {
-    cwd = process.cwd();
+    cwd;
     isProduction = false;
     config;
     manifestPath;
     manifest;
     manifestWorkspaces;
     constructor({ cwd, isProduction }) {
-        this.cwd = cwd ?? this.cwd;
+        this.cwd = cwd;
         this.isProduction = isProduction;
         this.config = defaultConfig;
     }

package/dist/index.js

@@ -260,7 +260,7 @@ export const main = async (unresolvedConfiguration) => {
     if (report.dependencies) {
         const { dependencyIssues, devDependencyIssues } = deputy.settleDependencyIssues();
         dependencyIssues.forEach(issue => collector.addIssue(issue));
-        if (!isProduction)
+        if (!isStrict)
             devDependencyIssues.forEach(issue => collector.addIssue(issue));
     }
     const { issues, counters } = collector.getIssues();

package/dist/{npm-scripts → manifest}/index.js

@@ -16,8 +16,8 @@ const findManifestDependencies = async ({ rootConfig, manifest, isRoot, isProduc
         .filter(binaryName => !IGNORED_GLOBAL_BINARIES.includes(binaryName))
         .filter(binaryName => !ignoreBinaries.includes(binaryName));
     const installedBinaries = new Map();
-    const deps = [...Object.keys(manifest.dependencies ?? {}), ...Object.keys(manifest.devDependencies ?? {})];
-    for (const packageName of deps) {
+    const packageNames = [...Object.keys(manifest.dependencies ?? {}), ...Object.keys(manifest.devDependencies ?? {})];
+    for (const packageName of packageNames) {
         const manifest = await getPackageManifest(dir, packageName, isRoot, cwd);
         if (manifest) {
             const binaries = typeof manifest.bin === 'string' ? [packageName] : Object.keys(manifest.bin ?? {});

package/dist/plugins/sentry/index.d.ts

@@ -1,6 +1,5 @@
-import type { IsPluginEnabledCallback, GenericPluginCallback } from '../../types/plugins.js';
+import type { IsPluginEnabledCallback } from '../../types/plugins.js';
 export declare const NAME = "Sentry";
 export declare const ENABLERS: string[];
 export declare const isEnabled: IsPluginEnabledCallback;
 export declare const ENTRY_FILE_PATTERNS: string[];
-export declare const findDependencies: GenericPluginCallback;

package/dist/plugins/sentry/index.js

@@ -1,11 +1,13 @@
-import { _load } from '../../util/loader.js';
-import { timerify } from '../../util/performance.js';
 export const NAME = 'Sentry';
-export const ENABLERS = ['@sentry/replay'];
+export const ENABLERS = [
+    '@sentry/browser',
+    '@sentry/electron',
+    '@sentry/ember',
+    '@sentry/gatsby',
+    '@sentry/nextjs',
+    '@sentry/remix',
+    '@sentry/replay',
+    '@sentry/tracing',
+];
 export const isEnabled = ({ dependencies }) => ENABLERS.some(enabler => dependencies.has(enabler));
 export const ENTRY_FILE_PATTERNS = ['sentry.{client,server}.config.{js,ts}'];
-const findNycDependencies = async (configFilePath) => {
-    const config = await _load(configFilePath);
-    return [config.extends];
-};
-export const findDependencies = timerify(findNycDependencies);

package/dist/plugins/webpack/index.js

@@ -4,7 +4,7 @@ import { timerify } from '../../util/performance.js';
 export const NAME = 'Webpack';
 export const ENABLERS = ['webpack'];
 export const isEnabled = ({ dependencies }) => ENABLERS.some(enabler => dependencies.has(enabler));
-export const CONFIG_FILE_PATTERNS = ['webpack.config*.js'];
+export const CONFIG_FILE_PATTERNS = ['webpack.config*.{js,ts}'];
 const resolveRuleSetLoaders = (rule) => {
     if (!rule || typeof rule === 'string')
         return [];
@@ -24,7 +24,10 @@ const resolveUseItemLoader = (use) => {
     return [];
 };
 const findWebpackDependencies = async (configFilePath, { manifest }) => {
-    const config = await _load(configFilePath);
+    let config = await _load(configFilePath);
+    if (typeof config === 'function') {
+        config = config();
+    }
     const loaders = (config.module?.rules?.flatMap(resolveRuleSetLoaders) ?? [])
         .map(loader => loader.replace(/\?.*/, ''))
         .filter(loader => !loader.startsWith('/'));

package/dist/util/glob.js

@@ -16,7 +16,7 @@ export const negate = (pattern) => pattern.replace(/^!?/, '!');
 export const hasProductionSuffix = (pattern) => pattern.endsWith('!');
 export const hasNoProductionSuffix = (pattern) => !pattern.endsWith('!');
 const removeProductionSuffix = (pattern) => pattern.replace(/!$/, '');
-const negatedLast = (a) => (a.startsWith('!') ? 1 : -1);
+const negatedLast = (pattern) => (pattern.startsWith('!') ? 1 : -1);
 const glob = async ({ cwd, workingDir = cwd, patterns, ignore = [], gitignore = true }) => {
     const cwdPosix = ensurePosixPath(cwd);
     const workingDirPosix = ensurePosixPath(workingDir);

package/dist/util/loader.js

@@ -7,7 +7,7 @@ import parsedArgs from './cli-arguments.js';
 import { loadJSON } from './fs.js';
 import { timerify } from './performance.js';
 const require = createRequire(process.cwd());
-const { values: { 'no-progress': noProgress = false }, } = parsedArgs;
+const { values: { 'no-progress': isNoProgress = false, debug: isDebug = false }, } = parsedArgs;
 const load = async (filePath) => {
     try {
         if (path.extname(filePath) === '.json' || /rc$/.test(filePath)) {
@@ -20,7 +20,7 @@ const load = async (filePath) => {
         return imported.default ?? imported;
     }
     catch (error) {
-        if (noProgress) {
+        if (isNoProgress || isDebug) {
             console.log('Failed to load ' + filePath);
             console.log(error?.toString());
         }

package/dist/workspace-worker.js

@@ -1,6 +1,6 @@
 import path from 'node:path';
 import { ROOT_WORKSPACE_NAME, TEST_FILE_PATTERNS } from './constants.js';
-import * as npm from './npm-scripts/index.js';
+import * as npm from './manifest/index.js';
 import * as plugins from './plugins/index.js';
 import { debugLogFiles, debugLogIssues, debugLogObject } from './util/debug.js';
 import { _pureGlob, negate, hasProductionSuffix, hasNoProductionSuffix } from './util/glob.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "1.0.0-beta.6",
+  "version": "1.0.0-beta.7",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript project",
   "keywords": [
     "find",
@@ -62,7 +62,7 @@
     "@snyk/github-codeowners": "1.1.0",
     "chalk": "5.2.0",
     "easy-table": "1.2.0",
-    "esbuild": "0.16.12",
+    "esbuild": "0.16.13",
     "esbuild-register": "3.4.2",
     "eslint": "8.31.0",
     "fast-glob": "3.2.12",
@@ -87,8 +87,8 @@
     "@types/node": "18.11.18",
     "@types/npmcli__map-workspaces": "3.0.0",
     "@types/webpack": "5.28.0",
-    "@typescript-eslint/eslint-plugin": "5.47.1",
-    "@typescript-eslint/parser": "5.47.1",
+    "@typescript-eslint/eslint-plugin": "5.48.0",
+    "@typescript-eslint/parser": "5.48.0",
     "eslint-import-resolver-typescript": "3.5.2",
     "eslint-plugin-import": "2.26.0",
     "globstar": "1.0.0",