septum 0.1.4 → 0.1.6

package/src/mcp/tools/get-symbol-hotspots.ts

@@ -1,59 +1,54 @@
-import type { ValidatedSeptumConfig } from "../../core/config/schema.ts";
-import type { DatabaseRepository } from "../../core/database/repository.ts";
-import type { SymbolKind } from "../../types/index.ts";
-
-export interface GetSymbolHotspotsArgs {
-  min_lines?: number;
-  min_nesting?: number;
-  kind?: SymbolKind | string;
-  domain?: string;
-  limit?: number;
-  sort_by?: "lines" | "nesting" | "risk_score";
-}
-
-export function handleGetSymbolHotspots(
-  repo: DatabaseRepository,
-  _config: ValidatedSeptumConfig,
-  args: GetSymbolHotspotsArgs
-) {
-  const minLines = args.min_lines ?? (args.min_nesting ? 0 : 30);
-  const minNesting = args.min_nesting ?? 1;
-  const limit = args.limit ?? 30;
-  const sortBy = args.sort_by ?? "risk_score";
-
-  const hotspots = repo.getHotspotSymbols({
-    minLines,
-    minNesting,
-    kind: args.kind as SymbolKind | undefined,
-    domain: args.domain,
-    limit,
-    sortBy,
-  });
-
-  const payload = {
-    total_found: hotspots.length,
-    threshold_min_lines: minLines,
-    threshold_min_nesting: minNesting,
-    sort_by: sortBy,
-    hotspots: hotspots.map((h) => ({
-      name: h.name,
-      kind: h.kind,
-      line_count: h.line_count,
-      nesting_depth: (h as any).nesting_depth ?? 1,
-      risk_score: (h as any).risk_score ?? h.line_count,
-      line_span: `${h.line_start}-${h.line_end}`,
-      file_path: h.file_path,
-      domain: h.domain_name,
-      signature: h.signature,
-    })),
-  };
-
-  return {
-    content: [
-      {
-        type: "text" as const,
-        text: JSON.stringify(payload, null, 2),
-      },
-    ],
-  };
-}
+import type { ValidatedSeptumConfig } from "../../core/config/schema.ts";
+import type { SeptumRepository } from "../../core/database/repository.ts";
+import type { SymbolKind } from "../../types/index.ts";
+import type { GetSymbolHotspotsArgs } from "../schemas.ts";
+
+export type { GetSymbolHotspotsArgs } from "../schemas.ts";
+
+export function handleGetSymbolHotspots(
+  repo: SeptumRepository,
+  _config: ValidatedSeptumConfig,
+  args: GetSymbolHotspotsArgs,
+  _workspaceRoot?: string
+) {
+  const minLines = args.min_lines ?? (args.min_nesting ? 0 : 30);
+  const minNesting = args.min_nesting ?? 1;
+  const limit = args.limit ?? 30;
+  const sortBy = args.sort_by ?? "risk_score";
+
+  const hotspots = repo.getHotspotSymbols({
+    minLines,
+    minNesting,
+    kind: args.kind as SymbolKind | undefined,
+    domain: args.domain,
+    limit,
+    sortBy,
+  });
+
+  const payload = {
+    total_found: hotspots.length,
+    threshold_min_lines: minLines,
+    threshold_min_nesting: minNesting,
+    sort_by: sortBy,
+    hotspots: hotspots.map((h) => ({
+      name: h.name,
+      kind: h.kind,
+      line_count: h.line_count,
+      nesting_depth: (h as any).nesting_depth ?? 1,
+      risk_score: (h as any).risk_score ?? h.line_count,
+      line_span: `${h.line_start}-${h.line_end}`,
+      file_path: h.file_path,
+      domain: h.domain_name,
+      signature: h.signature,
+    })),
+  };
+
+  return {
+    content: [
+      {
+        type: "text" as const,
+        text: JSON.stringify(payload, null, 2),
+      },
+    ],
+  };
+}

package/src/mcp/tools/get-symbol-impact.ts

@@ -1,26 +1,28 @@
-import type { ValidatedSeptumConfig } from "../../core/config/schema.ts";
-import type { SeptumRepository } from "../../core/database/repository.ts";
-import type { GetSymbolImpactArgs } from "../../types/index.ts";
-
-export function handleGetSymbolImpact(
-  repo: SeptumRepository,
-  _config: ValidatedSeptumConfig,
-  args: GetSymbolImpactArgs
-) {
-  if (!args.symbol) {
-    throw new Error(
-      "Missing required argument: 'symbol' (e.g. 'OrderService::cancelOrder', 'OrderService', or 'cancelOrder')"
-    );
-  }
-
-  const impact = repo.getSymbolImpact(args.symbol);
-
-  return {
-    content: [
-      {
-        type: "text" as const,
-        text: JSON.stringify(impact, null, 2),
-      },
-    ],
-  };
-}
+import type { ValidatedSeptumConfig } from "../../core/config/schema.ts";
+import type { SeptumRepository } from "../../core/database/repository.ts";
+import type { GetSymbolImpactArgs } from "../../types/index.ts";
+
+export function handleGetSymbolImpact(
+  repo: SeptumRepository,
+  _config: ValidatedSeptumConfig,
+  args: GetSymbolImpactArgs,
+  _workspaceRoot?: string
+) {
+  const symbol = (args.symbol || (args as unknown as Record<string, unknown>).query || "").toString().trim();
+  if (!symbol) {
+    throw new Error(
+      "Missing required argument: 'symbol' (e.g. 'OrderService::cancelOrder', 'OrderService', or 'cancelOrder')"
+    );
+  }
+
+  const impact = repo.getSymbolImpact(symbol);
+
+  return {
+    content: [
+      {
+        type: "text" as const,
+        text: JSON.stringify(impact, null, 2),
+      },
+    ],
+  };
+}

package/src/mcp/tools/get-symbol.ts

@@ -1,63 +1,65 @@
-import type { ValidatedSeptumConfig } from "../../core/config/schema.ts";
-import type { SeptumRepository } from "../../core/database/repository.ts";
-import { SymbolLocator } from "../../core/resolver/symbol-locator.ts";
-import type { GetSymbolArgs } from "../../types/index.ts";
-
-export function handleGetSymbol(
-  repo: SeptumRepository,
-  _config: ValidatedSeptumConfig,
-  args: GetSymbolArgs
-) {
-  if (!args.symbol) {
-    throw new Error(
-      "Missing required argument: 'symbol' (e.g. 'OrderController::cancelOrder', 'OrderService', or 'cancelOrder')"
-    );
-  }
-
-  const details = repo.getSymbolDetails(args.symbol, {
-    domain: args.domain,
-    includeDependencies: args.include_dependencies,
-  });
-
-  if (details) {
-    return {
-      content: [
-        {
-          type: "text" as const,
-          text: JSON.stringify(details, null, 2),
-        },
-      ],
-    };
-  }
-
-  // Fallback to fuzzy SymbolLocator to provide actionable hints
-  const locator = new SymbolLocator(repo);
-  const fallback = locator.locate(args.symbol, args.domain);
-
-  const payload = {
-    symbol: args.symbol,
-    found: false,
-    message: `Symbol '${args.symbol}' was not found in the deterministic catalog.`,
-    suggestions: fallback.suggestions.map((s) => ({
-      name: s.name,
-      similarity_score: Math.round(s.similarity_score * 100) / 100,
-      exact_range: {
-        start_line: s.line_start,
-        end_line: s.line_end,
-        total_lines: s.line_end - s.line_start + 1,
-      },
-      signature: s.signature,
-      file_path: s.file_path,
-    })),
-    sibling_methods: fallback.sibling_methods,
-  };
-
-  return {
-    content: [
-      {
-        type: "text" as const,
-        text: JSON.stringify(payload, null, 2),
-      },
-    ],
-  };
-}
+import type { ValidatedSeptumConfig } from "../../core/config/schema.ts";
+import type { SeptumRepository } from "../../core/database/repository.ts";
+import { SymbolLocator } from "../../core/resolver/symbol-locator.ts";
+import type { GetSymbolArgs } from "../../types/index.ts";
+
+export function handleGetSymbol(
+  repo: SeptumRepository,
+  _config: ValidatedSeptumConfig,
+  args: GetSymbolArgs,
+  _workspaceRoot?: string
+) {
+  const symbol = (args.symbol || (args as unknown as Record<string, unknown>).query || "").toString().trim();
+  if (!symbol) {
+    throw new Error(
+      "Missing required argument: 'symbol' (e.g. 'OrderController::cancelOrder', 'OrderService', or 'cancelOrder')"
+    );
+  }
+
+  const details = repo.getSymbolDetails(symbol, {
+    domain: args.domain,
+    includeDependencies: args.include_dependencies,
+  });
+
+  if (details) {
+    return {
+      content: [
+        {
+          type: "text" as const,
+          text: JSON.stringify(details, null, 2),
+        },
+      ],
+    };
+  }
+
+  // Fallback to fuzzy SymbolLocator to provide actionable hints
+  const locator = new SymbolLocator(repo);
+  const fallback = locator.locate(symbol, args.domain);
+
+  const payload = {
+    symbol,
+    found: false,
+    message: `Symbol '${symbol}' was not found in the deterministic catalog.`,
+    suggestions: fallback.suggestions.map((s) => ({
+      name: s.name,
+      similarity_score: Math.round(s.similarity_score * 100) / 100,
+      exact_range: {
+        start_line: s.line_start,
+        end_line: s.line_end,
+        total_lines: s.line_end - s.line_start + 1,
+      },
+      signature: s.signature,
+      file_path: s.file_path,
+    })),
+    sibling_methods: fallback.sibling_methods,
+  };
+
+  return {
+    content: [
+      {
+        type: "text" as const,
+        text: JSON.stringify(payload, null, 2),
+      },
+    ],
+  };
+}