septum 0.1.4 → 0.1.5

package/src/mcp/tools/check-boundary.ts

@@ -1,152 +1,171 @@
-import type { BoundaryEvaluator } from "../../core/boundary/evaluator.ts";
-import type { ValidatedSeptumConfig } from "../../core/config/schema.ts";
-import type { SeptumRepository } from "../../core/database/repository.ts";
-
-export interface CheckBoundaryArgs {
-  file_path?: string;
-  filePath?: string;
-  file?: string;
-  path?: string;
-  file_paths?: string[];
-  filePaths?: string[];
-  files?: string[];
-  paths?: string[];
-  proposed_imports?: string[];
-  proposedImports?: string[];
-  imports?: string[];
-  feature_key?: string;
-  featureKey?: string;
-  feature?: string;
-}
-
-export function handleCheckBoundary(
-  evaluator: BoundaryEvaluator,
-  config: ValidatedSeptumConfig,
-  args: CheckBoundaryArgs,
-  repo?: SeptumRepository
-) {
-  const rawFilePaths = args.file_paths ?? args.filePaths ?? args.files ?? args.paths;
-  const rawFilePath = args.file_path ?? args.filePath ?? args.file ?? args.path;
-  const proposedImports = args.proposed_imports ?? args.proposedImports ?? args.imports ?? [];
-  const featureKey = args.feature_key ?? args.featureKey ?? args.feature;
-
-  const filesToCheck: string[] = [];
-  if (rawFilePaths && Array.isArray(rawFilePaths) && rawFilePaths.length > 0) {
-    filesToCheck.push(...rawFilePaths);
-  } else if (rawFilePath) {
-    if (Array.isArray(rawFilePath)) {
-      filesToCheck.push(...rawFilePath);
-    } else {
-      filesToCheck.push(rawFilePath);
-    }
-  } else {
-    throw new Error("Missing required argument: specify either 'file_path' or 'file_paths'");
-  }
-
-  // Multi-file batch evaluation
-  if (filesToCheck.length > 1) {
-    const results = filesToCheck.map((filePath) => {
-      const violations = evaluator.evaluate(filePath, proposedImports, featureKey, config);
-      const impact = repo?.getFileInboundImpact(filePath);
-      return {
-        file: filePath,
-        status: violations.length > 0 ? "rejected" : "approved",
-        violations,
-        impact_summary: impact
-          ? {
-              direct_dependents: impact.direct_dependents_count,
-              top_consumers: impact.top_consumers,
-              symbols_count: impact.symbols_count,
-              risk_level: impact.risk_level,
-            }
-          : undefined,
-      };
-    });
-
-    const totalViolations = results.reduce((acc, r) => acc + r.violations.length, 0);
-    const hasViolations = totalViolations > 0;
-
-    return {
-      content: [
-        {
-          type: "text" as const,
-          text: JSON.stringify(
-            {
-              status: hasViolations ? "rejected" : "approved",
-              total_checked: filesToCheck.length,
-              total_violations: totalViolations,
-              results,
-              message: hasViolations
-                ? `${totalViolations} boundary violation(s) detected across ${filesToCheck.length} file(s).`
-                : `All ${filesToCheck.length} file(s) approved. No boundary violations detected.`,
-            },
-            null,
-            2
-          ),
-        },
-      ],
-    };
-  }
-
-  // Single file evaluation (backward-compatible output)
-  const singleFile = filesToCheck[0];
-  const violations = evaluator.evaluate(singleFile, proposedImports, featureKey, config);
-  const impact = repo?.getFileInboundImpact(singleFile);
-
-  if (violations.length > 0) {
-    return {
-      content: [
-        {
-          type: "text" as const,
-          text: JSON.stringify(
-            {
-              status: "rejected",
-              file: singleFile,
-              violations_count: violations.length,
-              violations,
-              impact_summary: impact
-                ? {
-                    direct_dependents: impact.direct_dependents_count,
-                    top_consumers: impact.top_consumers,
-                    symbols_count: impact.symbols_count,
-                    risk_level: impact.risk_level,
-                  }
-                : undefined,
-              message: "Boundary violation detected. Proposed imports violate bounded context architecture.",
-            },
-            null,
-            2
-          ),
-        },
-      ],
-    };
-  }
-
-  return {
-    content: [
-      {
-        type: "text" as const,
-        text: JSON.stringify(
-          {
-            status: "approved",
-            file: singleFile,
-            impact_summary: impact
-              ? {
-                  direct_dependents: impact.direct_dependents_count,
-                  top_consumers: impact.top_consumers,
-                  symbols_count: impact.symbols_count,
-                  risk_level: impact.risk_level,
-                }
-              : undefined,
-            message:
-              impact && impact.direct_dependents_count > 0
-                ? `No boundary violations detected. Caution: ${impact.direct_dependents_count} dependent file(s) consume this file.`
-                : "No boundary violations detected. Proposed imports are permissible.",
-          },
-          null,
-          2
-        ),
-      },
-    ],
-  };
-}
+import type { BoundaryEvaluator } from "../../core/boundary/evaluator.ts";
+import type { ValidatedSeptumConfig } from "../../core/config/schema.ts";
+import type { SeptumRepository } from "../../core/database/repository.ts";
+
+export interface CheckBoundaryArgs {
+  file_path?: string;
+  filePath?: string;
+  file?: string;
+  path?: string;
+  from_file?: string;
+  fromFile?: string;
+  to_file?: string | string[];
+  toFile?: string | string[];
+  file_paths?: string[];
+  filePaths?: string[];
+  files?: string[];
+  paths?: string[];
+  proposed_imports?: string[];
+  proposedImports?: string[];
+  imports?: string[];
+  feature_key?: string;
+  featureKey?: string;
+  feature?: string;
+}
+
+export function handleCheckBoundary(
+  evaluator: BoundaryEvaluator,
+  config: ValidatedSeptumConfig,
+  args: CheckBoundaryArgs,
+  repo?: SeptumRepository,
+  _workspaceRoot?: string
+) {
+  const rawToFile = args.to_file ?? args.toFile;
+  const toFileImports = rawToFile ? (Array.isArray(rawToFile) ? rawToFile.map(String) : [String(rawToFile)]) : [];
+  const rawFilePaths = args.file_paths ?? args.filePaths ?? args.files ?? args.paths;
+  const rawFilePath = args.file_path ?? args.filePath ?? args.file ?? args.path ?? args.from_file ?? args.fromFile;
+  const proposedImports = [
+    ...(args.proposed_imports ?? args.proposedImports ?? args.imports ?? []),
+    ...toFileImports,
+  ];
+  const featureKey = args.feature_key ?? args.featureKey ?? args.feature;
+
+  const filesToCheck: string[] = [];
+  if (rawFilePaths && Array.isArray(rawFilePaths) && rawFilePaths.length > 0) {
+    filesToCheck.push(...rawFilePaths);
+  } else if (rawFilePath) {
+    if (Array.isArray(rawFilePath)) {
+      filesToCheck.push(...rawFilePath);
+    } else {
+      filesToCheck.push(rawFilePath);
+    }
+  } else {
+    throw new Error("Missing required argument: specify either 'file_path' or 'file_paths'");
+  }
+
+  // Multi-file batch evaluation
+  if (filesToCheck.length > 1) {
+    const results = filesToCheck.map((filePath) => {
+      const violations = evaluator.evaluate(filePath, proposedImports, featureKey, config);
+      const impact = repo?.getFileInboundImpact(filePath);
+      return {
+        file: filePath,
+        status: violations.length > 0 ? "rejected" : "approved",
+        violations,
+        impact_summary: impact
+          ? {
+              direct_dependents: impact.direct_dependents_count,
+              top_consumers: impact.top_consumers,
+              symbols_count: impact.symbols_count,
+              risk_level: impact.risk_level,
+            }
+          : undefined,
+      };
+    });
+
+    const totalViolations = results.reduce((acc, r) => acc + r.violations.length, 0);
+    const hasViolations = totalViolations > 0;
+
+    return {
+      content: [
+        {
+          type: "text" as const,
+          text: JSON.stringify(
+            {
+              status: hasViolations ? "rejected" : "approved",
+              total_checked: filesToCheck.length,
+              total_violations: totalViolations,
+              results,
+              message: hasViolations
+                ? `${totalViolations} boundary violation(s) detected across ${filesToCheck.length} file(s).`
+                : Object.keys(config?.domains || {}).length === 0
+                ? `All ${filesToCheck.length} file(s) approved. [Septum Notice: Domain catalog is currently unindexed. Run 'septum_register_domain' to initialize bounded contexts.]`
+                : `All ${filesToCheck.length} file(s) approved. No boundary violations detected.`,
+            },
+            null,
+            2
+          ),
+        },
+      ],
+    };
+  }
+
+  // Single file evaluation (backward-compatible output)
+  const singleFile = filesToCheck[0];
+  const violations = evaluator.evaluate(singleFile, proposedImports, featureKey, config);
+  const impact = repo?.getFileInboundImpact(singleFile);
+
+  if (violations.length > 0) {
+    return {
+      content: [
+        {
+          type: "text" as const,
+          text: JSON.stringify(
+            {
+              status: "rejected",
+              file: singleFile,
+              violations_count: violations.length,
+              violations,
+              impact_summary: impact
+                ? {
+                    direct_dependents: impact.direct_dependents_count,
+                    top_consumers: impact.top_consumers,
+                    symbols_count: impact.symbols_count,
+                    risk_level: impact.risk_level,
+                  }
+                : undefined,
+              message:
+                Object.keys(config?.domains || {}).length === 0
+                  ? "Boundary integrity check failed: Domain catalog is currently unindexed. Run 'septum_register_domain' to initialize bounded contexts."
+                  : violations.some((v) => v.rule === "unmapped_domain")
+                  ? "Boundary integrity check failed: File does not belong to any declared bounded context."
+                  : "Boundary violation detected. Proposed imports violate bounded context architecture.",
+            },
+            null,
+            2
+          ),
+        },
+      ],
+    };
+  }
+
+  return {
+    content: [
+      {
+        type: "text" as const,
+        text: JSON.stringify(
+          {
+            status: "approved",
+            file: singleFile,
+            impact_summary: impact
+              ? {
+                  direct_dependents: impact.direct_dependents_count,
+                  top_consumers: impact.top_consumers,
+                  symbols_count: impact.symbols_count,
+                  risk_level: impact.risk_level,
+                }
+              : undefined,
+            message:
+              Object.keys(config?.domains || {}).length === 0
+                ? "No boundary violations detected. [Septum Notice: Domain catalog is currently unindexed. Run 'septum_register_domain' to initialize bounded contexts.]"
+                : impact && impact.direct_dependents_count > 0
+                ? `No boundary violations detected. Caution: ${impact.direct_dependents_count} dependent file(s) consume this file.`
+                : "No boundary violations detected. Proposed imports are permissible.",
+          },
+          null,
+          2
+        ),
+      },
+    ],
+  };
+}

package/src/mcp/tools/get-domain-catalog.ts

@@ -1,121 +1,145 @@
-import * as fs from "node:fs";
-import * as path from "node:path";
-import type { ValidatedSeptumConfig } from "../../core/config/schema.ts";
-import type { SeptumRepository } from "../../core/database/repository.ts";
-
-export interface GetDomainCatalogArgs {
-  domain?: string;
-  archetype_filter?: string;
-  workspace_path?: string;
-}
-
-export function handleGetDomainCatalog(
-  repo: SeptumRepository,
-  config: ValidatedSeptumConfig,
-  args: GetDomainCatalogArgs,
-  workspaceRoot?: string
-) {
-  if (!args.domain) {
-    const allDomains = repo.getAllDomains();
-    if (allDomains.length === 0) {
-      const root = workspaceRoot || process.cwd();
-      const diagnostic = {
-        status: "NOT_INDEXED",
-        total_domains: 0,
-        message: `Workspace '${root}' belum memiliki domain terdaftar dalam Septum catalog.`,
-        suggested_action: {
-          tool: "septum_register_domain",
-          params: {
-            name: "core",
-            root: "src",
-            ingest_now: true,
-          },
-        },
-        agent_guidance:
-          "Panggil tool 'septum_register_domain(name: \"core\", root: \"src\", ingest_now: true)' untuk mendaftarkan dan mengindeks domain utama secara otomatis, atau jalankan 'septum init' di terminal workspace.",
-      };
-      return {
-        content: [
-          {
-            type: "text" as const,
-            text: JSON.stringify(diagnostic, null, 2),
-          },
-        ],
-        metadata: diagnostic,
-      };
-    }
-
-    const projectType = repo.getMeta("project_type") || "generic";
-    const framework = repo.getMeta("framework") || "unknown";
-    const archStyle = repo.getMeta("architecture_style") || "modular";
-    const totalSlices = repo.getAllVerticalSlices().length;
-
-    const macroMap = {
-      telescope_view: {
-        project_type: projectType,
-        framework,
-        architecture_style: archStyle,
-        total_domains: allDomains.length,
-        total_vertical_slices: totalSlices,
-      },
-      domains: allDomains.map((d) => ({
-        name: d.name,
-        root: d.root_path,
-        archetypes: Object.keys(JSON.parse(d.archetypes_json || "{}")),
-        allowed_dependencies: JSON.parse(d.allowed_deps_json || "[]"),
-        forbidden_dependencies: JSON.parse(d.forbidden_deps_json || "[]"),
-      })),
-      agent_instruction:
-        "Macro semantic map loaded from SQLite SSOT. Use 'septum_get_domain_catalog' with domain='<name>' for granular file/symbol catalogs, or 'septum_trace_vertical_slice' with query='<route|command|intent>' for end-to-end execution flow.",
-    };
-
-    return {
-      content: [
-        {
-          type: "text" as const,
-          text: JSON.stringify(macroMap, null, 2),
-        },
-      ],
-    };
-  }
-
-  const catalog = repo.getDomainCatalog(args.domain, args.archetype_filter);
-  if (!catalog) {
-    const available = Object.keys(config.domains).join(", ");
-    throw new Error(
-      `Domain '${args.domain}' not found. Available domains in Septum catalog: [${available}]. Run 'septum ingest' or 'septum_register_domain' if recently added.`
-    );
-  }
-
-  // Record access log for Gate 3 (Mandatory Catalog Consultation)
-  recordCatalogAccess(args.domain);
-
-  // Token-efficient serialization: keep payload dense and under 500 tokens
-  return {
-    content: [
-      {
-        type: "text" as const,
-        text: JSON.stringify(catalog, null, 2),
-      },
-    ],
-  };
-}
-
-function recordCatalogAccess(domain: string): void {
-  try {
-    const septumDir = path.resolve(process.cwd(), ".septum");
-    if (!fs.existsSync(septumDir)) {
-      fs.mkdirSync(septumDir, { recursive: true });
-    }
-    const logPath = path.join(septumDir, "catalog_access.json");
-    let log: { domains?: Record<string, string>; features?: Record<string, string> } = {};
-    if (fs.existsSync(logPath)) {
-      try {
-        log = JSON.parse(fs.readFileSync(logPath, "utf-8"));
-      } catch {}
-    }
-    log.domains = log.domains || {};
-    log.domains[domain] = new Date().toISOString();
-    fs.writeFileSync(logPath, JSON.stringify(log, null, 2), "utf-8");
-  } catch {}
-}
+import * as fs from "node:fs";
+import * as path from "node:path";
+import type { ValidatedSeptumConfig } from "../../core/config/schema.ts";
+import type { SeptumRepository } from "../../core/database/repository.ts";
+
+export interface GetDomainCatalogArgs {
+  domain?: string;
+  archetype_filter?: string;
+  workspace_path?: string;
+}
+
+export function handleGetDomainCatalog(
+  repo: SeptumRepository,
+  config: ValidatedSeptumConfig,
+  args: GetDomainCatalogArgs,
+  workspaceRoot?: string
+) {
+  const targetRoot = args.workspace_path || workspaceRoot || (repo as any)?.projectRoot || process.cwd();
+
+  if (!args.domain) {
+    const allDomains = repo.getAllDomains();
+    if (allDomains.length === 0) {
+      const root = targetRoot;
+      const diagnostic = {
+        status: "NOT_INDEXED",
+        total_domains: 0,
+        message: `Workspace '${root}' belum memiliki domain terdaftar dalam Septum catalog.`,
+        suggested_action: {
+          tool: "septum_register_domain",
+          params: {
+            name: "core",
+            root: "src",
+            ingest_now: true,
+          },
+        },
+        agent_guidance:
+          "Panggil tool 'septum_register_domain(name: \"core\", root: \"src\", ingest_now: true)' untuk mendaftarkan dan mengindeks domain utama secara otomatis, atau jalankan 'septum init' di terminal workspace.",
+      };
+      return {
+        content: [
+          {
+            type: "text" as const,
+            text: JSON.stringify(diagnostic, null, 2),
+          },
+        ],
+        metadata: diagnostic,
+      };
+    }
+
+    const projectType = repo.getMeta("project_type") || "generic";
+    const framework = repo.getMeta("framework") || "unknown";
+    const archStyle = repo.getMeta("architecture_style") || "modular";
+    const totalSlices = repo.getAllVerticalSlices().length;
+
+    const macroMap = {
+      telescope_view: {
+        project_type: projectType,
+        framework,
+        architecture_style: archStyle,
+        total_domains: allDomains.length,
+        total_vertical_slices: totalSlices,
+      },
+      domains: allDomains.map((d) => ({
+        name: d.name,
+        root: d.root_path,
+        archetypes: Object.keys(JSON.parse(d.archetypes_json || "{}")),
+        allowed_dependencies: JSON.parse(d.allowed_deps_json || "[]"),
+        forbidden_dependencies: JSON.parse(d.forbidden_deps_json || "[]"),
+      })),
+      agent_instruction:
+        "Macro semantic map loaded from SQLite SSOT. Use 'septum_get_domain_catalog' with domain='<name>' for granular file/symbol catalogs, or 'septum_trace_vertical_slice' with query='<route|command|intent>' for end-to-end execution flow.",
+    };
+
+    return {
+      content: [
+        {
+          type: "text" as const,
+          text: JSON.stringify(macroMap, null, 2),
+        },
+      ],
+    };
+  }
+
+  const catalog = repo.getDomainCatalog(args.domain, args.archetype_filter);
+  if (!catalog) {
+    const available = Object.keys(config.domains || {});
+    const suggestions = available.filter((d) =>
+      d.toLowerCase().includes(args.domain!.toLowerCase()) ||
+      args.domain!.toLowerCase().includes(d.toLowerCase())
+    );
+    const notFoundPayload = {
+      status: "not_found",
+      found: false,
+      domain: args.domain,
+      message: `Domain '${args.domain}' not found in Septum catalog.`,
+      available_domains: available,
+      suggestions: suggestions.length > 0 ? suggestions : available,
+      suggested_action: {
+        tool: "septum_register_domain",
+        params: { domain: args.domain, root: `src/domains/${args.domain}` },
+      },
+    };
+    return {
+      content: [
+        {
+          type: "text" as const,
+          text: JSON.stringify(notFoundPayload, null, 2),
+        },
+      ],
+      metadata: notFoundPayload,
+    };
+  }
+
+  // Record access log for Gate 3 (Mandatory Catalog Consultation)
+  recordCatalogAccess(args.domain, targetRoot);
+
+  // Token-efficient serialization: keep payload dense and under 500 tokens
+  return {
+    content: [
+      {
+        type: "text" as const,
+        text: JSON.stringify(catalog, null, 2),
+      },
+    ],
+  };
+}
+
+function recordCatalogAccess(domain: string, projectRoot: string = process.cwd()): void {
+  try {
+    const septumDir = path.resolve(projectRoot, ".septum");
+    if (!fs.existsSync(septumDir)) {
+      fs.mkdirSync(septumDir, { recursive: true });
+    }
+    const logPath = path.join(septumDir, "catalog_access.json");
+    let log: { domains?: Record<string, string>; features?: Record<string, string> } = {};
+    if (fs.existsSync(logPath)) {
+      try {
+        log = JSON.parse(fs.readFileSync(logPath, "utf-8"));
+      } catch {}
+    }
+    log.domains = log.domains || {};
+    log.domains[domain] = new Date().toISOString();
+    fs.writeFileSync(logPath, JSON.stringify(log, null, 2), "utf-8");
+  } catch {}
+}