nexus-agents 2.165.1 → 2.166.2

package/dist/{chunk-S6QSNP67.js → chunk-YIEMNZUD.js}

@@ -43,7 +43,7 @@ import {
 } from "./chunk-DHVMSIT5.js";
 
 // src/version.ts
-var VERSION = true ? "2.165.1" : "dev";
+var VERSION = true ? "2.166.2" : "dev";
 
 // src/config/schemas-core.ts
 import { z } from "zod";
@@ -2159,7 +2159,7 @@ async function runDoctorFix(result) {
   writeLine2("\u2500".repeat(40));
   let fixCount = 0;
   if (!result.dataDirectory.rootExists || result.dataDirectory.subdirectories.some((d) => !d.exists || !d.writable)) {
-    const { runSetup } = await import("./setup-command-3SVR247D.js");
+    const { runSetup } = await import("./setup-command-AVQWHV5L.js");
     const setupResult = runSetup({
       skipMcp: true,
       skipRules: true,
@@ -2272,4 +2272,4 @@ export {
   startStdioServer,
   closeServer
 };
-//# sourceMappingURL=chunk-S6QSNP67.js.map
+//# sourceMappingURL=chunk-YIEMNZUD.js.map

package/dist/cli.js

@@ -22,7 +22,7 @@ import "./chunk-SF2AK3JB.js";
 import {
   setupCommandAsync,
   verifyCommand
-} from "./chunk-FFPFN4W2.js";
+} from "./chunk-MLURL552.js";
 import "./chunk-2255SARF.js";
 import {
   AuthHandler,
@@ -142,11 +142,11 @@ import {
   validateCommand,
   validateWorkflow,
   wrapInMarkdownFence
-} from "./chunk-DPV7TCNP.js";
+} from "./chunk-RLRAUL43.js";
 import "./chunk-2UMMVKUW.js";
 import "./chunk-YOFRUVPJ.js";
 import "./chunk-HFOQKCD2.js";
-import "./chunk-3ACDP4E6.js";
+import "./chunk-AO4X3BMP.js";
 import {
   CATEGORY_DISPLAY_NAMES,
   DEFAULT_PR_REVIEW_CONFIG
@@ -170,7 +170,7 @@ import {
   loadConfig,
   runDoctor,
   validateNexusEnv
-} from "./chunk-S6QSNP67.js";
+} from "./chunk-YIEMNZUD.js";
 import "./chunk-FKOWIKR4.js";
 import {
   shutdownExpertBridge
@@ -10000,17 +10000,31 @@ function extractDependencies(sourceFile) {
 }
 
 // src/indexer/description-extraction.ts
-function classifyJsDocLine(trimmed, inComment) {
-  if (trimmed.startsWith("/**")) return "start";
+function isPlainBlockComment(trimmed) {
+  return trimmed.startsWith("/*") && trimmed.endsWith("*/");
+}
+function classifyOutsideComment(trimmed) {
+  if (isPlainBlockComment(trimmed)) return "skip";
+  if (trimmed.length > 0 && !trimmed.startsWith("//")) return "stop";
+  return "skip";
+}
+function classifyInsideComment(trimmed) {
   if (trimmed.endsWith("*/")) return "end";
-  if (!inComment && trimmed.length > 0 && !trimmed.startsWith("//")) return "stop";
-  if (!inComment) return "skip";
   if (trimmed.startsWith("* @") || trimmed.startsWith("@")) return "skip";
   return "content";
 }
+function classifyJsDocLine(trimmed, inComment) {
+  if (trimmed.startsWith("/**")) {
+    return trimmed.endsWith("*/") ? "startEnd" : "start";
+  }
+  return inComment ? classifyInsideComment(trimmed) : classifyOutsideComment(trimmed);
+}
 function extractJsDocLineContent(trimmed) {
   return trimmed.startsWith("*") ? trimmed.slice(1).trim() : trimmed;
 }
+function extractInlineJsDocContent(trimmed) {
+  return trimmed.slice(3, -2).trim();
+}
 function truncateDescription(full) {
   return truncateSentence(full, 150);
 }
@@ -10025,6 +10039,11 @@ function extractDescription(sourceFile) {
       inComment = true;
       continue;
     }
+    if (result === "startEnd") {
+      const inline = extractInlineJsDocContent(trimmed);
+      if (inline.length > 0) description.push(inline);
+      break;
+    }
     if (result === "end" || result === "stop") break;
     if (result === "skip") continue;
     const content = extractJsDocLineContent(trimmed);
@@ -10056,20 +10075,11 @@ function extractFileEntry(sourceFile, rootDir, extractDescriptions) {
   }
   return entry;
 }
-function shouldExcludeFile(filePath, excludePatterns, rootDir) {
-  return excludePatterns.some((pattern) => {
-    const patternBase = pattern.replace(/\*\*/g, "").replace(/\*/g, "");
-    return filePath.includes(patternBase.replace(rootDir, "").replace(/^\//, ""));
-  });
-}
-function processSourceFiles(project, rootDir, excludePatterns, extractDescriptions) {
+function processSourceFiles(project, rootDir, extractDescriptions) {
   const files = [];
   const errors = [];
   for (const sourceFile of project.getSourceFiles()) {
     const filePath = sourceFile.getFilePath();
-    if (shouldExcludeFile(filePath, excludePatterns, rootDir)) {
-      continue;
-    }
     try {
       files.push(extractFileEntry(sourceFile, rootDir, extractDescriptions));
     } catch (err2) {
@@ -10088,15 +10098,10 @@ function extractProject(options = {}) {
   });
   const rootDir = path9.resolve(process.cwd(), opts.rootDir);
   const includePatterns = opts.include.map((p) => path9.join(rootDir, p));
-  const excludePatterns = opts.exclude.map((p) => path9.join(rootDir, p));
+  const excludePatterns = opts.exclude.map((p) => `!${path9.join(rootDir, p)}`);
   try {
-    project.addSourceFilesAtPaths(includePatterns);
-    const { files, errors } = processSourceFiles(
-      project,
-      rootDir,
-      excludePatterns,
-      opts.extractDescriptions
-    );
+    project.addSourceFilesAtPaths([...includePatterns, ...excludePatterns]);
+    const { files, errors } = processSourceFiles(project, rootDir, opts.extractDescriptions);
     return { files, errors, durationMs: getTimeProvider().now() - startTime };
   } catch (err2) {
     const message = err2 instanceof Error ? err2.message : String(err2);
@@ -10778,12 +10783,12 @@ function extractMcpTools(project, packageRoot, mcpToolsPath, warnings) {
     const fileName = path13.basename(filePath);
     if (fileName.endsWith(".test.ts") || fileName === "index.ts") continue;
     const relativePath = path13.relative(process.cwd(), filePath);
-    const extractedTools = extractToolsFromFile(sourceFile, relativePath);
+    const extractedTools = extractToolsFromFile(sourceFile, relativePath, warnings);
     tools.push(...extractedTools);
   }
   return tools;
 }
-function extractToolsFromFile(sourceFile, relativePath) {
+function extractToolsFromFile(sourceFile, relativePath, warnings) {
   const tools = [];
   if (sourceFile === void 0) return tools;
   const callExpressions = sourceFile.getDescendantsOfKind(SyntaxKind2.CallExpression);
@@ -10799,71 +10804,93 @@ function extractToolsFromFile(sourceFile, relativePath) {
       continue;
     }
     const toolName = nameArg.getText().replace(/['"`]/g, "");
-    const { description, schemaArg } = extractToolMeta(callText, args, sourceFile);
+    const { description, schemaArg } = extractToolMeta(callText, args);
     let parameters = [];
     if (schemaArg !== void 0) {
-      parameters = extractParametersFromSchema(schemaArg, sourceFile);
+      parameters = extractParametersFromSchema(schemaArg);
     }
-    tools.push({
+    const tool = {
       name: toolName,
       description: description.replace(/\\n/g, " ").trim(),
       parameters,
       source_file: relativePath,
       source_line: callExpr.getStartLineNumber()
-    });
+    };
+    maybeWarnEmptyTool(tool, warnings);
+    tools.push(tool);
   }
   return tools;
 }
-function resolvePropertyValue(prop, sourceFile) {
+function maybeWarnEmptyTool(tool, warnings) {
+  if (warnings === void 0) return;
+  if (tool.description !== "" || tool.parameters.length > 0) return;
+  warnings.push(
+    `MCP tool "${tool.name}" (${tool.source_file}:${String(tool.source_line)}) resolved to an empty description AND empty parameters. Its description/inputSchema likely reference an identifier the extractor could not resolve statically (#2153).`
+  );
+}
+function resolveIdentifierInitializer(symbol) {
+  for (const decl of symbol?.getDeclarations() ?? []) {
+    const varDecl = decl.asKind(SyntaxKind2.VariableDeclaration);
+    const init = varDecl?.getInitializer();
+    if (init !== void 0) return init;
+  }
+  return void 0;
+}
+function resolvePropertyValue(prop) {
   const propAssign = prop.asKind(SyntaxKind2.PropertyAssignment);
-  if (propAssign !== void 0) return propAssign.getInitializer();
+  if (propAssign !== void 0) {
+    const init = propAssign.getInitializer();
+    if (init?.getKind() === SyntaxKind2.Identifier) {
+      return resolveIdentifierInitializer(init.getSymbol());
+    }
+    return init;
+  }
   const shorthand = prop.asKind(SyntaxKind2.ShorthandPropertyAssignment);
-  if (shorthand !== void 0 && sourceFile !== void 0) {
-    const varDecl = sourceFile.getVariableDeclaration(shorthand.getName());
-    return varDecl?.getInitializer();
+  if (shorthand !== void 0) {
+    return resolveIdentifierInitializer(shorthand.getValueSymbol());
   }
   return void 0;
 }
-function extractToolMeta(callText, args, _sourceFile) {
-  let description = "";
-  let schemaArg;
-  if (callText.endsWith(".registerTool")) {
-    const configArg = args[1];
-    if (configArg !== void 0) {
-      const configObj = configArg.asKind(SyntaxKind2.ObjectLiteralExpression);
-      if (configObj !== void 0) {
-        const descProp = configObj.getProperty("description");
-        if (descProp !== void 0) {
-          const init = resolvePropertyValue(descProp, _sourceFile);
-          description = init?.getText().replace(/^['"]|['"]$/g, "") ?? "";
-        }
-        const schemaProp = configObj.getProperty("inputSchema");
-        if (schemaProp !== void 0) {
-          schemaArg = resolvePropertyValue(schemaProp, _sourceFile);
-        }
-      }
-    }
-  } else {
-    const descArg = args[1];
-    if (descArg !== void 0) {
-      description = descArg.getText().replace(/^['"]|['"]$/g, "");
+function extractStringValue(node) {
+  if (node === void 0) return "";
+  const kind = node.getKind();
+  if (kind === SyntaxKind2.StringLiteral || kind === SyntaxKind2.NoSubstitutionTemplateLiteral) {
+    return node.asKind(kind)?.getLiteralText() ?? "";
+  }
+  if (kind === SyntaxKind2.BinaryExpression) {
+    const bin = node.asKind(SyntaxKind2.BinaryExpression);
+    if (bin?.getOperatorToken().getKind() === SyntaxKind2.PlusToken) {
+      return extractStringValue(bin.getLeft()) + extractStringValue(bin.getRight());
     }
-    schemaArg = args[2];
   }
+  return "";
+}
+function extractToolMeta(callText, args) {
+  if (callText.endsWith(".registerTool")) {
+    return extractRegisterToolMeta(args[1]);
+  }
+  return {
+    description: extractStringValue(args[1]),
+    schemaArg: args[2]
+  };
+}
+function extractRegisterToolMeta(configArg) {
+  const configObj = configArg?.asKind(SyntaxKind2.ObjectLiteralExpression);
+  if (configObj === void 0) return { description: "", schemaArg: void 0 };
+  const descProp = configObj.getProperty("description");
+  const description = descProp === void 0 ? "" : extractStringValue(resolvePropertyValue(descProp));
+  const schemaProp = configObj.getProperty("inputSchema");
+  const schemaArg = schemaProp === void 0 ? void 0 : resolvePropertyValue(schemaProp);
   return { description, schemaArg };
 }
-function extractParametersFromSchema(schemaArg, sourceFile) {
+function extractParametersFromSchema(schemaArg) {
   if (schemaArg.getKind() === SyntaxKind2.ObjectLiteralExpression) {
     return extractZodParameters(schemaArg);
   }
-  if (schemaArg.getKind() === SyntaxKind2.Identifier && sourceFile !== void 0) {
-    const varName = schemaArg.getText();
-    const varDecl = sourceFile.getVariableDeclaration(varName);
-    if (varDecl !== void 0) {
-      const init = varDecl.getInitializer();
-      if (init !== void 0) {
-        return extractZodParameters(init);
-      }
+  if (schemaArg.getKind() === SyntaxKind2.Identifier) {
+    const init = resolveIdentifierInitializer(schemaArg.getSymbol());
+    if (init !== void 0) {
+      return extractZodParameters(init);
     }
   }
   return [];