Minor housekeeping 🔍

webpro-nl · Jan 8, 2025 · 4a9ff56 · 4a9ff56
1 parent 0c49689
commit 4a9ff56
Show file tree

Hide file tree

Showing 8 changed files with 9 additions and 27 deletions.
diff --git a/packages/knip/src/plugins/expo/index.ts b/packages/knip/src/plugins/expo/index.ts
@@ -13,7 +13,7 @@ const enablers = ['expo'];
 
 const isEnabled: IsPluginEnabled = ({ dependencies }) => hasDependency(dependencies, enablers);
 
-const config: string[] = ['app.json', 'app.config.{ts,js}'];
+const config = ['app.json', 'app.config.{ts,js}'];
 
 const resolveEntryPaths: ResolveEntryPaths<ExpoConfig> = async (expoConfig, { manifest }) => {
   const config = 'expo' in expoConfig ? expoConfig.expo : expoConfig;

diff --git a/packages/knip/src/plugins/linthtml/index.ts b/packages/knip/src/plugins/linthtml/index.ts
@@ -2,7 +2,7 @@ import type { IsPluginEnabled, Plugin, ResolveConfig } from '../../types/config.
 import { toDeferResolve } from '../../util/input.js';
 import { toCosmiconfig } from '../../util/plugin-config.js';
 import { hasDependency } from '../../util/plugin.js';
-import type { PluginConfig } from './types.js';
+import type { LintHTMLConfig } from './types.js';
 
 // https://linthtml.vercel.app/
 
@@ -16,7 +16,7 @@ const isEnabled: IsPluginEnabled = ({ dependencies }) => hasDependency(dependenc
 
 const config = ['package.json', ...toCosmiconfig('linthtml')];
 
-const resolveConfig: ResolveConfig<PluginConfig> = config => {
+const resolveConfig: ResolveConfig<LintHTMLConfig> = config => {
   const extensions = config.extends ?? [];
   const plugins = config.plugins ?? [];
   return [extensions, plugins].flat().map(toDeferResolve);

diff --git a/packages/knip/src/plugins/linthtml/types.ts b/packages/knip/src/plugins/linthtml/types.ts
@@ -1,4 +1,4 @@
-export type PluginConfig = {
+export type LintHTMLConfig = {
   extends?: string | string[];
   plugins?: string[];
 };
diff --git a/packages/knip/src/plugins/metro/index.ts b/packages/knip/src/plugins/metro/index.ts
@@ -16,9 +16,7 @@ const enablers = ['metro', 'react-native'];
 
 const isEnabled: IsPluginEnabled = options => hasDependency(options.dependencies, enablers);
 
-const packageJsonPath = 'metro';
-
-const config: string[] = ['metro.config.{js,cjs,json}', 'package.json'];
+const config = ['metro.config.{js,cjs,json}', 'package.json'];
 
 const DEFAULT_PLATFORMS = ['ios', 'android', 'windows', 'web'];
 const PLATFORMS = [...DEFAULT_PLATFORMS, 'native', 'default'];
@@ -57,7 +55,6 @@ export default {
   note,
   enablers,
   isEnabled,
-  packageJsonPath,
   config,
   production,
   resolveEntryPaths,

diff --git a/packages/knip/src/plugins/release-it/index.ts b/packages/knip/src/plugins/release-it/index.ts
@@ -13,8 +13,6 @@ const enablers = ['release-it'];
 
 const isEnabled: IsPluginEnabled = ({ dependencies }) => hasDependency(dependencies, enablers);
 
-const packageJsonPath = 'release-it';
-
 const config = ['.release-it.{json,js,cjs,ts,yml,yaml,toml}', 'package.json'];
 
 const resolveConfig: ResolveConfig<ReleaseItConfig> = (config, options) => {
@@ -35,7 +33,6 @@ export default {
   title,
   enablers,
   isEnabled,
-  packageJsonPath,
   config,
   resolveConfig,
 } satisfies Plugin;
diff --git a/packages/knip/src/plugins/simple-git-hooks/index.ts b/packages/knip/src/plugins/simple-git-hooks/index.ts
@@ -1,7 +1,7 @@
 import type { IsPluginEnabled, Plugin, ResolveConfig } from '../../types/config.js';
 import type { Input } from '../../util/input.js';
 import { hasDependency } from '../../util/plugin.js';
-import type { PluginConfig } from './types.js';
+import type { SimpleGitHooksConfig } from './types.js';
 
 // https://github.com/toplenboren/simple-git-hooks
 
@@ -11,17 +11,9 @@ const enablers = ['simple-git-hooks'];
 
 const isEnabled: IsPluginEnabled = ({ dependencies }) => hasDependency(dependencies, enablers);
 
-const packageJsonPath = 'simple-git-hooks';
+const config = ['.simple-git-hooks.{js,cjs,json}', 'simple-git-hooks.{js,cjs,json}', 'package.json'];
 
-const config = [
-  '.simple-git-hooks.{js,cjs}',
-  'simple-git-hooks.{js,cjs}',
-  '.simple-git-hooks.json',
-  'simple-git-hooks.json',
-  'package.json',
-];
-
-const resolveConfig: ResolveConfig<PluginConfig> = async (config, options) => {
+const resolveConfig: ResolveConfig<SimpleGitHooksConfig> = async (config, options) => {
   if (typeof config === 'function') config = config();
 
   if (!config) return [];
@@ -39,7 +31,6 @@ export default {
   title,
   enablers,
   isEnabled,
-  packageJsonPath,
   config,
   resolveConfig,
 } satisfies Plugin;
diff --git a/packages/knip/src/plugins/simple-git-hooks/types.ts b/packages/knip/src/plugins/simple-git-hooks/types.ts
@@ -5,4 +5,4 @@
 // }
 type Config = Record<string, string>;
 
-export type PluginConfig = Config | (() => Config);
+export type SimpleGitHooksConfig = Config | (() => Config);
diff --git a/packages/knip/src/plugins/wireit/index.ts b/packages/knip/src/plugins/wireit/index.ts
@@ -10,8 +10,6 @@ const enablers = ['wireit'];
 
 const isEnabled: IsPluginEnabled = ({ dependencies }) => hasDependency(dependencies, enablers);
 
-const packageJsonPath = 'wireit';
-
 const config = ['package.json'];
 
 const resolveConfig: ResolveConfig<WireitConfig> = (localConfig, options) => {
@@ -26,7 +24,6 @@ export default {
   title,
   enablers,
   isEnabled,
-  packageJsonPath,
   config,
   resolveConfig,
 } satisfies Plugin;