knip 5.24.2 → 5.24.4

package/dist/plugins/angular/index.js

@@ -23,6 +23,14 @@ const resolveConfig = async (config, options) => {
                 if ('main' in opts && typeof opts.main === 'string') {
                     dependencies.add(toProductionEntryPattern(join(cwd, opts.main)));
                 }
+                if ('browser' in opts && typeof opts.browser === 'string') {
+                    dependencies.add(toProductionEntryPattern(join(cwd, opts.browser)));
+                }
+                if ('ssr' in opts && opts.ssr && typeof opts.ssr === 'object') {
+                    if ('entry' in opts.ssr && typeof opts.ssr.entry === 'string') {
+                        dependencies.add(toProductionEntryPattern(join(cwd, opts.ssr.entry)));
+                    }
+                }
             }
         }
     }

package/dist/plugins/jest/index.js

@@ -1,5 +1,5 @@
 import { dirname, isInternal, join, toAbsolute } from '#p/util/path.js';
-import { hasDependency, load } from '#p/util/plugin.js';
+import { hasDependency, load, resolveEntry } from '#p/util/plugin.js';
 import { toEntryPattern } from '#p/util/protocols.js';
 const title = 'Jest';
 const enablers = ['jest'];
@@ -49,8 +49,6 @@ const resolveDependencies = async (config, options) => {
             .filter(reporter => !['default', 'github-actions', 'summary'].includes(reporter))
         : [];
     const watchPlugins = config.watchPlugins?.map(watchPlugin => (typeof watchPlugin === 'string' ? watchPlugin : watchPlugin[0])) ?? [];
-    const setupFiles = (config.setupFiles ?? []).map(toEntryPattern);
-    const setupFilesAfterEnv = (config.setupFilesAfterEnv ?? []).map(toEntryPattern);
     const transform = config.transform
         ? Object.values(config.transform).map(transform => (typeof transform === 'string' ? transform : transform[0]))
         : [];
@@ -60,8 +58,11 @@ const resolveDependencies = async (config, options) => {
     const testResultsProcessor = config.testResultsProcessor ? [config.testResultsProcessor] : [];
     const snapshotResolver = config.snapshotResolver ? [config.snapshotResolver] : [];
     const testSequencer = config.testSequencer ? [config.testSequencer] : [];
-    const globalSetup = (config.globalSetup ? [config.globalSetup] : []).map(toEntryPattern);
-    const globalTeardown = (config.globalTeardown ? [config.globalTeardown] : []).map(toEntryPattern);
+    const resolve = (specifier) => resolveEntry(options, specifier);
+    const setupFiles = (config.setupFiles ?? []).map(resolve);
+    const setupFilesAfterEnv = (config.setupFilesAfterEnv ?? []).map(resolve);
+    const globalSetup = (config.globalSetup ? [config.globalSetup] : []).map(resolve);
+    const globalTeardown = (config.globalTeardown ? [config.globalTeardown] : []).map(resolve);
     return [
         ...presets,
         ...projects,

package/dist/plugins/vitest/index.js

@@ -1,5 +1,5 @@
-import { isAbsolute, join, relative } from '#p/util/path.js';
-import { hasDependency, tryResolve } from '#p/util/plugin.js';
+import { join } from '#p/util/path.js';
+import { hasDependency, resolveEntry } from '#p/util/plugin.js';
 import { toEntryPattern } from '#p/util/protocols.js';
 import { getEnvPackageName, getExternalReporters } from './helpers.js';
 const title = 'Vitest';
@@ -7,15 +7,6 @@ const enablers = ['vitest'];
 const isEnabled = ({ dependencies }) => hasDependency(dependencies, enablers);
 const config = ['vitest*.config.{js,mjs,ts,cjs,mts,cts}', 'vitest.{workspace,projects}.{ts,js,json}'];
 const entry = ['**/*.{bench,test,test-d,spec}.?(c|m)[jt]s?(x)'];
-const resolveEntry = (options, rootDir, specifier) => {
-    const { configFileDir, configFileName } = options;
-    const resolvedPath = isAbsolute(specifier)
-        ? specifier
-        : tryResolve(join(configFileDir, rootDir, specifier), join(configFileDir, rootDir, configFileName));
-    if (resolvedPath)
-        return toEntryPattern(relative(configFileDir, resolvedPath));
-    return specifier;
-};
 const isVitestCoverageCommand = /vitest(.+)--coverage(?:\.enabled(?:=true)?)?/;
 const hasScriptWithCoverage = (scripts) => scripts
     ? Object.values(scripts).some(script => {
@@ -32,8 +23,8 @@ const findConfigDependencies = (localConfig, options) => {
     const hasCoverageEnabled = (testConfig.coverage && testConfig.coverage.enabled !== false) || hasScriptWithCoverage(manifest.scripts);
     const coverage = hasCoverageEnabled ? [`@vitest/coverage-${testConfig.coverage?.provider ?? 'v8'}`] : [];
     const rootDir = testConfig.root ?? '.';
-    const setupFiles = [testConfig.setupFiles ?? []].flat().map(v => resolveEntry(options, rootDir, v));
-    const globalSetup = [testConfig.globalSetup ?? []].flat().map(v => resolveEntry(options, rootDir, v));
+    const setupFiles = [testConfig.setupFiles ?? []].flat().map(specifier => resolveEntry(options, specifier, rootDir));
+    const globalSetup = [testConfig.globalSetup ?? []].flat().map(specifier => resolveEntry(options, specifier, rootDir));
     return [...environments, ...reporters, ...coverage, ...setupFiles, ...globalSetup];
 };
 const getConfigs = async (localConfig) => {
@@ -79,7 +70,7 @@ export const resolveConfig = async (localConfig, options) => {
             dependencies.add(dependency);
         const entry = cfg.build?.lib?.entry ?? [];
         const rootDir = cfg.test?.root ?? '.';
-        const deps = (typeof entry === 'string' ? [entry] : Object.values(entry)).map(specifier => resolveEntry(options, rootDir, specifier));
+        const deps = (typeof entry === 'string' ? [entry] : Object.values(entry)).map(specifier => resolveEntry(options, specifier, rootDir));
         for (const dependency of deps)
             dependencies.add(dependency);
     }

package/dist/typescript/visitors/dynamic-imports/importCall.js

@@ -30,7 +30,7 @@ export default visit(() => true, node => {
                                         });
                                     }
                                 }
-                                else if (ts.isObjectBindingPattern(arg.name)) {
+                                else if (arg && ts.isObjectBindingPattern(arg.name)) {
                                     return arg.name.elements.map(element => {
                                         const identifier = (element.propertyName ?? element.name).getText();
                                         const alias = element.propertyName ? element.name.getText() : undefined;

package/dist/util/plugin.d.ts

@@ -1,7 +1,6 @@
 export { _getDependenciesFromScripts as getDependenciesFromScripts } from '../binaries/index.js';
 export { _loadJSON as loadJSON } from './fs.js';
 export { _load as load } from './loader.js';
-export { _tryResolve as tryResolve } from './require.js';
 import type { RawPluginConfiguration } from '../types/config.js';
 import type { Plugin, PluginOptions } from '../types/plugins.js';
 export declare const toCamelCase: (name: string) => string;
@@ -37,3 +36,4 @@ export declare const toUnconfig: (moduleName: string, options?: {
     configFilesAllExtensions?: boolean;
     additionalExtensions?: string[];
 }) => string[];
+export declare const resolveEntry: (options: PluginOptions, specifier: string, rootDir?: string) => string;

package/dist/util/plugin.js

@@ -1,12 +1,12 @@
 export { _getDependenciesFromScripts as getDependenciesFromScripts } from '../binaries/index.js';
 export { _loadJSON as loadJSON } from './fs.js';
 export { _load as load } from './loader.js';
-export { _tryResolve as tryResolve } from './require.js';
 import { arrayify } from './array.js';
 import { _load as load } from './loader.js';
 import { get } from './object.js';
-import { basename } from './path.js';
+import { basename, isAbsolute, join, relative } from './path.js';
 import { toEntryPattern, toProductionEntryPattern } from './protocols.js';
+import { _tryResolve as tryResolve } from './require.js';
 export const toCamelCase = (name) => name.toLowerCase().replace(/(-[a-z])/g, group => group.toUpperCase().replace('-', ''));
 export const hasDependency = (dependencies, values) => values.some(value => {
     if (typeof value === 'string') {
@@ -89,3 +89,12 @@ export const toUnconfig = toConfigMap(['json', 'ts', 'mts', 'cts', 'js', 'mjs',
     rcSuffix: '',
     configFiles: false,
 });
+export const resolveEntry = (options, specifier, rootDir = '.') => {
+    const { configFileDir, configFileName } = options;
+    const resolvedPath = isAbsolute(specifier)
+        ? specifier
+        : tryResolve(join(configFileDir, rootDir, specifier), join(configFileDir, rootDir, configFileName));
+    if (resolvedPath)
+        return toEntryPattern(relative(configFileDir, resolvedPath));
+    return specifier;
+};

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "5.24.2";
+export declare const version = "5.24.4";

package/dist/version.js

@@ -1 +1 @@
-export const version = '5.24.2';
+export const version = '5.24.4';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knip",
-  "version": "5.24.2",
+  "version": "5.24.4",
   "description": "Find unused files, dependencies and exports in your TypeScript and JavaScript projects",
   "homepage": "https://knip.dev",
   "repository": {