@knip/mcp 0.0.22 → 0.0.24

package/src/docs/docs/writing-a-plugin/index.md

@@ -251,20 +251,20 @@ This is why plugins can implement the `resolveFromAST` function.
 ### 8. resolveFromAST
 
 Let's take a look at the Astro plugin implementation. This example assumes some
-familiarity with Abstract Syntax Trees (AST) and the TypeScript compiler API.
-Knip will provide more and more AST helpers to make implementing plugins more
-fun and a little less tedious.
+familiarity with Abstract Syntax Trees (AST). Knip provides AST helpers to make
+implementing plugins more fun and a little less tedious.
 
 Anyway, let's dive in. Here's how we're adding the Starlight `components` paths
 to the default `production` file patterns:
 
 ```ts
-import ts from 'typescript';
+import type { Program } from 'oxc-parser';
 import {
+  findCallArg,
   getDefaultImportName,
   getImportMap,
   getPropertyValues,
-} from '../../typescript/ast-helpers.js';
+} from '../../typescript/ast-helpers.ts';
 
 const title = 'Astro';
 
@@ -275,44 +275,24 @@ const production = [
   'src/actions/index.{js,ts}',
 ];
 
-const getComponentPathsFromSourceFile = (sourceFile: ts.SourceFile) => {
-  const componentPaths: Set<string> = new Set();
-  const importMap = getImportMap(sourceFile);
+const getComponentPaths = (program: Program) => {
+  const importMap = getImportMap(program);
   const importName = getDefaultImportName(importMap, '@astrojs/starlight');
-
-  function visit(node: ts.Node) {
-    if (
-      ts.isCallExpression(node) &&
-      ts.isIdentifier(node.expression) &&
-      node.expression.text === importName // match the starlight() function call
-    ) {
-      const starlightConfig = node.arguments[0];
-      if (ts.isObjectLiteralExpression(starlightConfig)) {
-        const values = getPropertyValues(starlightConfig, 'components');
-        for (const value of values) componentPaths.add(value);
-      }
-    }
-
-    ts.forEachChild(node, visit);
-  }
-
-  visit(sourceFile);
-
-  return componentPaths;
+  if (!importName) return new Set<string>();
+  const starlightConfig = findCallArg(program, importName);
+  return getPropertyValues(starlightConfig, 'components');
 };
 
-const resolveFromAST: ResolveFromAST = (sourceFile: ts.SourceFile) => {
-  // Include './src/components/Head.astro' and './src/components/Footer.astro'
-  // as production entry files so they're also part of the analysis
-  const componentPaths = getComponentPathsFromSourceFile(sourceFile);
+const resolveFromAST: ResolveFromAST = (program: Program) => {
+  const componentPaths = getComponentPaths(program);
   return [...production, ...componentPaths].map(id => toProductionEntry(id));
 };
 
-const plugin: Plugin {
+const plugin: Plugin = {
   title,
   production,
   resolveFromAST,
-}
+};
 
 export default plugin;
 ```
@@ -386,6 +366,6 @@ plugin, this might be the right time to open a pull request!
 [2]: ../reference/plugins.md
 [3]: ../explanations/plugins.md#entry-files-from-config-files
 [4]: https://github.com/webpro-nl/knip/blob/6a6954386b33ee8a2919005230a4bc094e11bc03/knip.json#L12
-[5]: ./inputs.md
+[5]: ../writing-a-plugin/inputs.md
 [6]: ../features/script-parser.md
-[7]: ./argument-parsing.md
+[7]: ../writing-a-plugin/argument-parsing.md

package/src/texts.js

@@ -56,13 +56,16 @@ export const ERROR_HINT = `For unexpected errors (exit code 2) such as "error lo
 - If no config file exists, create knip.json in the project root
 - Run knip again`;
 
+export const UNCONFIGURED_HINT =
+  'Issues are suppressed because the project is not yet configured. Reported issues might be false positives. Address configuration hints first, then re-run to get the actual issues.';
+
 export const CONFIG_REVIEW_HINT = `Review the existing configuration for potential improvements:
 
-- Never use "ignore" patterns (hides real issues!), always prefer specific solutions, other ignore* options are allowed
-- Many unused exported types? Add: ignoreExportsUsedInFile: { interface: true, type: true } (prefer this over other ignore* options)
+- Never use "ignore" patterns (hides real issues!), always prefer specific solutions; other ignore* options are allowed
+- Many unused exported types → try ignoreExportsUsedInFile: { interface: true, type: true }
 - Remove ignore patterns that don't match any files
-- Remove redundant ignore patterns: Knip respects .gitignore (node_modules, dist, build, .git)
+- Remove redundant ignore patterns — Knip respects .gitignore (node_modules, dist, build, .git)
 - Remove entry patterns covered by config defaults and auto-detected plugins
-- Config files (e.g. vite.config.ts) showing as unused? Enable/disable the plugin explicitly
+- Config files (e.g. vite.config.ts) showing as unused → try enable or disable the plugin explicitly (vite: true)
 - Dependencies matching Node.js builtins: add to ignoreDependencies (e.g. buffer, process)
 - Unresolved imports from path aliases: add paths to Knip config (tsconfig.json semantics)`;

package/src/tools.js

@@ -3,7 +3,7 @@ import { dirname, join } from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { createOptions, createSession, finalizeConfigurationHints, KNIP_CONFIG_LOCATIONS } from 'knip/session';
 import { CURATED_RESOURCES } from './curated-resources.js';
-import { CONFIG_REVIEW_HINT } from './texts.js';
+import { CONFIG_REVIEW_HINT, UNCONFIGURED_HINT } from './texts.js';
 
 export { ERROR_HINT } from './texts.js';
 
@@ -30,15 +30,30 @@ const docsDir = join(__dirname, './docs');
  * @param {{ cwd: string, configFilePath: string | undefined }} options
  */
 export function buildResults(results, options) {
+  const configurationHints = finalizeConfigurationHints(results, options);
+
+  const isSuppressIssues =
+    results.counters.total >= 10 &&
+    configurationHints.some(hint => hint.type === 'top-level-unconfigured' || hint.type === 'workspace-unconfigured');
+
+  const configFile = options.configFilePath
+    ? { exists: true, filePath: options.configFilePath }
+    : { exists: false, locations: KNIP_CONFIG_LOCATIONS };
+
+  const reviewHint = isSuppressIssues ? UNCONFIGURED_HINT : options.configFilePath ? CONFIG_REVIEW_HINT : undefined;
+  const files = isSuppressIssues ? [] : Array.from(results.issues.files);
+  const issues = isSuppressIssues
+    ? []
+    : Object.fromEntries(Object.entries(results.issues).filter(([key]) => key !== 'files' && key !== '_files'));
+
   return {
-    configFile: options.configFilePath
-      ? { exists: true, filePath: options.configFilePath, reviewHint: CONFIG_REVIEW_HINT }
-      : { exists: false, locations: KNIP_CONFIG_LOCATIONS },
-    configurationHints: finalizeConfigurationHints(results, options),
+    reviewHint,
+    configFile,
+    configurationHints,
     counters: results.counters,
     enabledPlugins: results.enabledPlugins,
-    files: Array.from(results.issues.files),
-    issues: Object.fromEntries(Object.entries(results.issues).filter(([key]) => key !== 'files' && key !== '_files')),
+    files,
+    issues,
   };
 }