@znt/mcp 2.0.2 → 2.0.3

package/README.md

@@ -207,6 +207,10 @@ Windows Credential Manager, macOS Keychain или Linux Secret Service. В YAML
 Проверяет Core, YAML и credential. С `check_provider: true` дополнительно
 проверяет endpoint и настроенные модели. Значение секрета не возвращается.
 
+Ключевые поля ответа:
+- `config_managed` (boolean): `true`, если конфигурация управляется MCP-сервером, или `false`, если выбран внешний файл конфигурации через `ZNT_CONFIG`.
+- `model`: имя генеративной LLM-модели. Если выбран `semantic_mode: "fast"` (локальные эвристики AST без вызовов LLM) и модель не задана, поле возвращает `"not_required (fast mode)"`.
+
 ### `search`
 
 Ищет компоненты в semantic index.
@@ -302,6 +306,14 @@ embedding не означает, что lexical symbol отсутствует.
 Допустимые форматы: `text`, `json`, `mermaid`. Для неоднозначного короткого
 имени core возвращает кандидатов; повторите запрос с квалифицированным `name`.
 
+Дополнительные параметры направления:
+- `direction` (`"down"` | `"up"` | `"both"`, по умолчанию `"down"`): в локальном режиме задает направление обхода вызовов:
+  - `"down"` (по умолчанию): исследует только исходящие вызовы (дерево вызовов / callees), исключая комбинаторный взрыв тестами и сторонними клиентами на `depth >= 2`;
+  - `"up"`: исследует иерархию вызывающих (callers / кто вызывает данный символ);
+  - `"both"`: двунаправленный обход (одновременно callers и callees).
+
+> 💡 **Чистый граф проекта**: MCP автоматически отсекает методы стандартных библиотек и вендоров (JDK, Go stdlib, Spring boilerplate: `HashMap`, `put`, `get`, `ok`), поэтому граф содержит только реальный код проекта.
+
 ### `outline`
 
 Возвращает полное top-level оглавление проиндексированного файла:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@znt/mcp",
-  "version": "2.0.2",
+  "version": "2.0.3",
   "description": "Model Context Protocol server for znt-core through @znt/sdk-nodejs",
   "type": "module",
   "main": "./index.js",

package/src/tool-definitions.js

@@ -79,6 +79,7 @@ export const TOOL_DEFINITIONS = Object.freeze([
         depth: { type: "integer", minimum: 1, maximum: 100, default: 1 },
         format: { type: "string", enum: ["text", "json", "mermaid"], default: "text" },
         edge_types: { type: "string", description: "Comma-separated contains,call,inherits,implements values." },
+        direction: { type: "string", enum: ["down", "up", "both"], default: "down", description: "Call direction: 'down' for call tree (callees, default), 'up' for callers (call hierarchy), 'both' for bidirectional context." },
       },
       required: ["from"],
       additionalProperties: false,
@@ -123,12 +124,11 @@ export const TOOL_DEFINITIONS = Object.freeze([
   },
   {
     name: "scan",
-    description: "Scan a project only after setup is valid. If setup_required is returned, call setup before retrying.",
+    description: "Index the project workspace. Scans all supported source files, declarative schemas, and project resources.",
     inputSchema: {
       type: "object",
       properties: {
         project_path: { type: "string", minLength: 1, description: "Optional absolute project directory or a path relative to ZNT_PROJECT_ROOT. Defaults to ZNT_PROJECT_ROOT or cwd." },
-        language: { type: "string", minLength: 1, default: "auto", description: "Parser name from info.languages, or auto." },
       },
       additionalProperties: false,
     },

package/src/znt-tools.js

@@ -27,11 +27,13 @@ export class ZntTools {
     this.managedConfig = options.managedConfig ?? (explicitConfig === undefined);
     this.client = options.client ?? new ZntClient({
       configPath: this.configPath,
+      configManaged: this.managedConfig,
       ...(options.coreBinaryPath ? { coreBinaryPath: options.coreBinaryPath } : {}),
     });
     if (typeof this.client.setupManagedConfig !== "function") {
       this.client.setupManagedConfig = (opts) => setupManagedConfig(opts, {
         configPath: this.configPath,
+        configManaged: this.managedConfig,
         coreBinaryPath: this.customCoreBinary,
         client: this.client,
       });
@@ -39,6 +41,7 @@ export class ZntTools {
     if (typeof this.client.getSetupStatus !== "function") {
       this.client.getSetupStatus = (opts) => getSetupStatus(opts, {
         configPath: this.configPath,
+        configManaged: this.managedConfig,
         coreBinaryPath: this.customCoreBinary,
         client: this.client,
       });
@@ -145,10 +148,16 @@ export class ZntTools {
   }
 
   async setupStatus(checkProvider = false, signal) {
-    const status = await this.client.getSetupStatus({ configPath: this.configPath, checkProvider, signal });
+    const status = await this.client.getSetupStatus({
+      configPath: this.configPath,
+      configManaged: this.managedConfig,
+      checkProvider,
+      signal,
+    });
     return {
       ...status,
-      config_managed_by_mcp: this.managedConfig,
+      config_managed: this.managedConfig,
+      ...(status?.semantic_mode === "fast" && !status?.model ? { model: "not_required (fast mode)" } : {}),
     };
   }
 
@@ -179,7 +188,9 @@ export class ZntTools {
       "clear_api_key",
     ].some((key) => args[key] !== undefined);
 
-    if (!hasExplicitSettings || args.interactive === true) {
+    const isInteractive = args.interactive !== false && args.interactive !== "false";
+
+    if (isInteractive && (!hasExplicitSettings || args.interactive === true || args.interactive === "true")) {
       return this.openSetupWizard(args, signal);
     }
 
@@ -266,43 +277,55 @@ export class ZntTools {
       !args.clear_api_key
     );
 
+    const isInteractive = args.interactive !== false && args.interactive !== "false";
     const mode = args.mode ?? previousStatus?.mode ?? "openapi";
     const isInteractiveOpenapi = mode === "openapi" && !args.api_key && args.credential_source !== "env" && !hasExistingCredential;
 
     const status = await this.client.setupManagedConfig({
       ...args,
       config_path: this.configPath,
-      check_provider: isInteractiveOpenapi ? false : (args.check_provider !== false),
+      check_provider: (isInteractiveOpenapi && isInteractive) ? false : (args.check_provider !== false),
     });
 
     if (isInteractiveOpenapi || status.status === "credential_required") {
-      const providerUrl = args.provider_url ?? status.provider_url ?? "https://api.openai.com/v1";
-      const reference = credentialReference(providerUrl);
-      const setupUrl = await this.credentialSetup.open({
-        provider: providerUrl,
-        onCredential: async (credential) => {
-          if (typeof this.client.storeSecret === "function") {
-            await this.client.storeSecret(reference, credential);
-          }
-          if (typeof this.client.validateConfig === "function") {
-            await this.client.validateConfig(this.configPath, { checkProvider: true });
-          }
-          this.setupReady = true;
-        },
-      });
+      if (isInteractive) {
+        const providerUrl = args.provider_url ?? status.provider_url ?? "https://api.openai.com/v1";
+        const reference = credentialReference(providerUrl);
+        const setupUrl = await this.credentialSetup.open({
+          provider: providerUrl,
+          onCredential: async (credential) => {
+            if (typeof this.client.storeSecret === "function") {
+              await this.client.storeSecret(reference, credential);
+            }
+            if (typeof this.client.validateConfig === "function") {
+              await this.client.validateConfig(this.configPath, { checkProvider: true });
+            }
+            this.setupReady = true;
+          },
+        });
+        return {
+          status: "credential_required",
+          config_path: this.configPath,
+          credential_source: "keyring",
+          setup_url: setupUrl,
+          next: "Open setup_url, save the API key, then call setup_status with check_provider=true.",
+        };
+      }
+
       return {
+        ...status,
         status: "credential_required",
         config_path: this.configPath,
-        credential_source: "keyring",
-        setup_url: setupUrl,
-        next: "Open setup_url, save the API key, then call setup_status with check_provider=true.",
+        instruction: "API key is required. Pass api_key directly in setup arguments or set interactive: true to open the browser setup wizard.",
       };
     }
 
+
     this.setupReady = status.status === "ready";
     return {
       ...status,
-      config_managed_by_mcp: this.managedConfig,
+      config_managed: this.managedConfig,
+      ...(status?.semantic_mode === "fast" && !status?.model ? { model: "not_required (fast mode)" } : {}),
       scan_required: this.setupReady,
     };
   }
@@ -327,7 +350,7 @@ export class ZntTools {
       case "similar":
         return this.client.findSimilar(normalizeSimilarArgs(args), requestOptions);
       case "graph":
-        return this.client.getSubgraph(copyDefined(args, ["from", "to", "depth", "format", "edge_types"]), requestOptions);
+        return this.client.getSubgraph(normalizeGraphArgs(args), requestOptions);
       case "outline":
         return this.client.fileOutline({ file_path: args.path, include_code: args.include_code }, requestOptions);
       case "logs":
@@ -457,10 +480,31 @@ export class ZntTools {
   }
 }
 
+export const TYPE_BOOST_MAP = {
+  class: "declaration",
+  struct: "declaration",
+  interface: "declaration",
+  type: "declaration",
+  enum: "declaration",
+  declaration: "declaration",
+  method: "function",
+  function: "function",
+  constructor: "function",
+  field: "member",
+  property: "member",
+  member: "member",
+};
+
+export function normalizeAstType(type) {
+  if (!type || typeof type !== "string") return type;
+  const lower = type.trim().toLowerCase();
+  return TYPE_BOOST_MAP[lower] ?? type;
+}
+
 function normalizeSearchArgs(args) {
   return copyDefined({
     ...args,
-    type: args.type_boost,
+    type: normalizeAstType(args.type_boost),
     role: args.role_boost,
     file_path: args.file_pattern,
   }, ["query", "mode", "limit", "callers_level", "callees_level", "compact", "include_code", "max_code_lines", "type", "role", "file_path"]);
@@ -469,12 +513,20 @@ function normalizeSearchArgs(args) {
 function normalizeSimilarArgs(args) {
   return copyDefined({
     ...args,
-    type: args.type_boost,
+    type: normalizeAstType(args.type_boost),
     role: args.role_boost,
     file_path: args.file_pattern,
   }, ["target", "query", "limit", "include_code", "max_code_lines", "type", "role", "file_path", "edge_types"]);
 }
 
+function normalizeGraphArgs(args) {
+  return copyDefined({
+    ...args,
+    direction: args.direction ?? "down",
+    internal_only: true,
+  }, ["from", "to", "depth", "format", "edge_types", "direction", "internal_only"]);
+}
+
 function copyDefined(source, keys) {
   return Object.fromEntries(keys.filter((key) => source[key] !== undefined).map((key) => [key, source[key]]));
 }