septum 0.1.3 → 0.1.5

package/src/core/database/repositories/domain.repository.ts

@@ -1,198 +1,218 @@
-import type { Database } from "bun:sqlite";
-import type {
-  DomainCatalogResponse,
-  DomainConfig,
-  DomainRecord,
-  FileRecord,
-  SymbolKind,
-  SymbolRecord,
-} from "../../../types/index.ts";
-
-export class DomainRepository {
-  constructor(private db: Database) {}
-
-  public upsertDomain(name: string, config: DomainConfig): number {
-    const normalizedName = name.trim();
-    const existing = this.getDomainByName(normalizedName);
-
-    // Merge dependencies and archetypes if existing and incoming is empty
-    let allowed = config.allowed_dependencies ?? [];
-    let forbidden = config.forbidden_dependencies ?? [];
-    let archetypes = config.archetypes ?? {};
-
-    if (existing) {
-      if (allowed.length === 0 && existing.allowed_deps_json && existing.allowed_deps_json !== "[]") {
-        try {
-          allowed = JSON.parse(existing.allowed_deps_json);
-        } catch {}
-      }
-      if (forbidden.length === 0 && existing.forbidden_deps_json && existing.forbidden_deps_json !== "[]") {
-        try {
-          forbidden = JSON.parse(existing.forbidden_deps_json);
-        } catch {}
-      }
-      if (Object.keys(archetypes).length === 0 && existing.archetypes_json && existing.archetypes_json !== "{}") {
-        try {
-          archetypes = JSON.parse(existing.archetypes_json);
-        } catch {}
-      }
-    }
-
-    const allowedJson = JSON.stringify(allowed);
-    const forbiddenJson = JSON.stringify(forbidden);
-    const archetypesJson = JSON.stringify(archetypes);
-
-    const query = `
-      INSERT INTO domains (name, root_path, allowed_deps_json, forbidden_deps_json, archetypes_json)
-      VALUES (?, ?, ?, ?, ?)
-      ON CONFLICT(name) DO UPDATE SET
-        root_path = excluded.root_path,
-        allowed_deps_json = excluded.allowed_deps_json,
-        forbidden_deps_json = excluded.forbidden_deps_json,
-        archetypes_json = excluded.archetypes_json,
-        updated_at = CURRENT_TIMESTAMP
-    `;
-    this.db.query(query).run(normalizedName, config.root, allowedJson, forbiddenJson, archetypesJson);
-
-    const record = this.getDomainByName(normalizedName);
-    return record?.id ?? 0;
-  }
-
-  public getDomainByName(name: string): DomainRecord | null {
-    return this.db
-      .query<DomainRecord, [string]>("SELECT * FROM domains WHERE LOWER(name) = LOWER(?)")
-      .get(name.trim());
-  }
-
-  public getAllDomains(): DomainRecord[] {
-    return this.db.query<DomainRecord, []>("SELECT * FROM domains ORDER BY name ASC").all();
-  }
-
-  public getDomainCatalog(
-    domainName: string,
-    archetypeFilter?: string
-  ): DomainCatalogResponse | null {
-    const domain = this.getDomainByName(domainName);
-    if (!domain) return null;
-
-    const allowedDeps: string[] = JSON.parse(domain.allowed_deps_json || "[]");
-    const forbiddenDeps: string[] = JSON.parse(domain.forbidden_deps_json || "[]");
-
-    let filesQuery = "SELECT * FROM files WHERE domain_id = ?";
-    const params: [number, string?] = [domain.id];
-
-    if (archetypeFilter) {
-      filesQuery += " AND archetype = ?";
-      params.push(archetypeFilter);
-    }
-    filesQuery += " ORDER BY path ASC";
-
-    const files = this.db.query<FileRecord, any>(filesQuery).all(...params);
-
-    const catalogFiles: DomainCatalogResponse["files"] = [];
-
-    for (const file of files) {
-      const symbols = this.db
-        .query<SymbolRecord, [number]>(
-          "SELECT * FROM symbols WHERE file_id = ? ORDER BY line_start ASC"
-        )
-        .all(file.id);
-
-      const symbolMap = new Map<
-        string,
-        {
-          name: string;
-          kind: SymbolKind;
-          properties: string[];
-          methods: string[];
-          signatures: string[];
-        }
-      >();
-
-      for (const sym of symbols) {
-        if (
-          sym.kind === "class" ||
-          sym.kind === "interface" ||
-          sym.kind === "trait" ||
-          sym.kind === "enum"
-        ) {
-          if (!symbolMap.has(sym.name)) {
-            symbolMap.set(sym.name, {
-              name: sym.name,
-              kind: sym.kind,
-              properties: [],
-              methods: [],
-              signatures: [sym.signature],
-            });
-          }
-        } else if (sym.kind === "method") {
-          let targetContainer = Array.from(symbolMap.values()).pop();
-          if (sym.name.includes("::")) {
-            const containerName = sym.name.split("::")[0];
-            if (symbolMap.has(containerName)) {
-              targetContainer = symbolMap.get(containerName);
-            }
-          }
-          if (targetContainer) {
-            targetContainer.methods.push(sym.signature);
-          } else {
-            symbolMap.set(sym.name, {
-              name: sym.name,
-              kind: sym.kind,
-              properties: [],
-              methods: [],
-              signatures: [sym.signature],
-            });
-          }
-        } else if (sym.kind === "property") {
-          let targetContainer = Array.from(symbolMap.values()).pop();
-          if (sym.name.includes("::")) {
-            const containerName = sym.name.split("::")[0];
-            if (symbolMap.has(containerName)) {
-              targetContainer = symbolMap.get(containerName);
-            }
-          }
-          if (targetContainer) {
-            targetContainer.properties.push(sym.signature);
-          } else {
-            symbolMap.set(sym.name, {
-              name: sym.name,
-              kind: sym.kind,
-              properties: [sym.signature],
-              methods: [],
-              signatures: [sym.signature],
-            });
-          }
-        } else {
-          symbolMap.set(sym.name, {
-            name: sym.name,
-            kind: sym.kind,
-            properties: [],
-            methods: [],
-            signatures: [sym.signature],
-          });
-        }
-      }
-
-      catalogFiles.push({
-        path: file.path,
-        archetype: file.archetype,
-        symbols: Array.from(symbolMap.values()).map((s) => ({
-          name: s.name,
-          kind: s.kind,
-          properties: s.properties.length > 0 ? s.properties : undefined,
-          methods: s.methods.length > 0 ? s.methods : undefined,
-          signatures: s.signatures.length > 0 ? s.signatures : undefined,
-        })),
-      });
-    }
-
-    return {
-      domain: domain.name,
-      root: domain.root_path,
-      allowed_dependencies: allowedDeps,
-      forbidden_dependencies: forbiddenDeps,
-      files: catalogFiles,
-    };
-  }
-}
+import type { Database } from "bun:sqlite";
+import type {
+  DomainCatalogResponse,
+  DomainConfig,
+  DomainRecord,
+  FileRecord,
+  SymbolKind,
+  SymbolRecord,
+} from "../../../types/index.ts";
+
+export class DomainRepository {
+  constructor(private db: Database) {}
+
+  public upsertDomain(name: string, config: DomainConfig): number {
+    const normalizedName = name.trim();
+    const existing = this.getDomainByName(normalizedName);
+
+    // Merge dependencies and archetypes if existing and incoming is empty
+    let allowed = config.allowed_dependencies ?? [];
+    let forbidden = config.forbidden_dependencies ?? [];
+    let archetypes = config.archetypes ?? {};
+
+    if (existing) {
+      if (allowed.length === 0 && existing.allowed_deps_json && existing.allowed_deps_json !== "[]") {
+        try {
+          allowed = JSON.parse(existing.allowed_deps_json);
+        } catch {}
+      }
+      if (forbidden.length === 0 && existing.forbidden_deps_json && existing.forbidden_deps_json !== "[]") {
+        try {
+          forbidden = JSON.parse(existing.forbidden_deps_json);
+        } catch {}
+      }
+      if (Object.keys(archetypes).length === 0 && existing.archetypes_json && existing.archetypes_json !== "{}") {
+        try {
+          archetypes = JSON.parse(existing.archetypes_json);
+        } catch {}
+      }
+    }
+
+    const allowedJson = JSON.stringify(allowed);
+    const forbiddenJson = JSON.stringify(forbidden);
+    const archetypesJson = JSON.stringify(archetypes);
+
+    const query = `
+      INSERT INTO domains (name, root_path, allowed_deps_json, forbidden_deps_json, archetypes_json)
+      VALUES (?, ?, ?, ?, ?)
+      ON CONFLICT(name) DO UPDATE SET
+        root_path = excluded.root_path,
+        allowed_deps_json = excluded.allowed_deps_json,
+        forbidden_deps_json = excluded.forbidden_deps_json,
+        archetypes_json = excluded.archetypes_json,
+        updated_at = CURRENT_TIMESTAMP
+    `;
+    this.db.query(query).run(normalizedName, config.root, allowedJson, forbiddenJson, archetypesJson);
+
+    const record = this.getDomainByName(normalizedName);
+    return record?.id ?? 0;
+  }
+
+  public getDomainByName(name: string): DomainRecord | null {
+    return this.db
+      .query<DomainRecord, [string]>("SELECT * FROM domains WHERE LOWER(name) = LOWER(?)")
+      .get(name.trim());
+  }
+
+  public getAllDomains(): DomainRecord[] {
+    return this.db.query<DomainRecord, []>("SELECT * FROM domains ORDER BY name ASC").all();
+  }
+
+  public getDomainCatalog(
+    domainName: string,
+    archetypeFilter?: string
+  ): DomainCatalogResponse | null {
+    const domain = this.getDomainByName(domainName);
+    if (!domain) return null;
+
+    const allowedDeps: string[] = JSON.parse(domain.allowed_deps_json || "[]");
+    const forbiddenDeps: string[] = JSON.parse(domain.forbidden_deps_json || "[]");
+
+    let filesQuery = "SELECT * FROM files WHERE domain_id = ?";
+    const params: [number, string?] = [domain.id];
+
+    if (archetypeFilter) {
+      filesQuery += " AND archetype = ?";
+      params.push(archetypeFilter);
+    }
+    filesQuery += " ORDER BY path ASC";
+
+    const files = this.db.query<FileRecord, any>(filesQuery).all(...params);
+
+    const catalogFiles: DomainCatalogResponse["files"] = [];
+
+    // Batch retrieve symbols for all files in one query to eliminate N+1 roundtrips
+    const symbolsByFileId = new Map<number, SymbolRecord[]>();
+    for (const f of files) {
+      symbolsByFileId.set(f.id, []);
+    }
+
+    if (files.length > 0) {
+      const allDomainSymbols = this.db
+        .query<SymbolRecord, [number]>(
+          `SELECT s.* FROM symbols s
+           JOIN files f ON s.file_id = f.id
+           WHERE f.domain_id = ?
+           ORDER BY s.file_id ASC, s.line_start ASC`
+        )
+        .all(domain.id);
+
+      for (const sym of allDomainSymbols) {
+        const list = symbolsByFileId.get(sym.file_id);
+        if (list) {
+          list.push(sym);
+        }
+      }
+    }
+
+    for (const file of files) {
+      const symbols = symbolsByFileId.get(file.id) ?? [];
+
+      const symbolMap = new Map<
+        string,
+        {
+          name: string;
+          kind: SymbolKind;
+          properties: string[];
+          methods: string[];
+          signatures: string[];
+        }
+      >();
+
+      for (const sym of symbols) {
+        if (
+          sym.kind === "class" ||
+          sym.kind === "interface" ||
+          sym.kind === "trait" ||
+          sym.kind === "enum"
+        ) {
+          if (!symbolMap.has(sym.name)) {
+            symbolMap.set(sym.name, {
+              name: sym.name,
+              kind: sym.kind,
+              properties: [],
+              methods: [],
+              signatures: [sym.signature],
+            });
+          }
+        } else if (sym.kind === "method") {
+          let targetContainer = Array.from(symbolMap.values()).pop();
+          if (sym.name.includes("::")) {
+            const containerName = sym.name.split("::")[0];
+            if (symbolMap.has(containerName)) {
+              targetContainer = symbolMap.get(containerName);
+            }
+          }
+          if (targetContainer) {
+            targetContainer.methods.push(sym.signature);
+          } else {
+            symbolMap.set(sym.name, {
+              name: sym.name,
+              kind: sym.kind,
+              properties: [],
+              methods: [],
+              signatures: [sym.signature],
+            });
+          }
+        } else if (sym.kind === "property") {
+          let targetContainer = Array.from(symbolMap.values()).pop();
+          if (sym.name.includes("::")) {
+            const containerName = sym.name.split("::")[0];
+            if (symbolMap.has(containerName)) {
+              targetContainer = symbolMap.get(containerName);
+            }
+          }
+          if (targetContainer) {
+            targetContainer.properties.push(sym.signature);
+          } else {
+            symbolMap.set(sym.name, {
+              name: sym.name,
+              kind: sym.kind,
+              properties: [sym.signature],
+              methods: [],
+              signatures: [sym.signature],
+            });
+          }
+        } else {
+          symbolMap.set(sym.name, {
+            name: sym.name,
+            kind: sym.kind,
+            properties: [],
+            methods: [],
+            signatures: [sym.signature],
+          });
+        }
+      }
+
+      catalogFiles.push({
+        path: file.path,
+        archetype: file.archetype,
+        symbols: Array.from(symbolMap.values()).map((s) => ({
+          name: s.name,
+          kind: s.kind,
+          properties: s.properties.length > 0 ? s.properties : undefined,
+          methods: s.methods.length > 0 ? s.methods : undefined,
+          signatures: s.signatures.length > 0 ? s.signatures : undefined,
+        })),
+      });
+    }
+
+    return {
+      domain: domain.name,
+      root: domain.root_path,
+      allowed_dependencies: allowedDeps,
+      forbidden_dependencies: forbiddenDeps,
+      files: catalogFiles,
+    };
+  }
+}

package/src/core/database/repositories/environment.repository.ts

@@ -0,0 +1,207 @@
+import type { Database } from "bun:sqlite";
+import type { CanonicalPlatformTag, EnvironmentLayer, EnvironmentLayerInfo, TopologyNode } from "../../topology/types.ts";
+
+export class EnvironmentRepository {
+  constructor(private db: Database) {}
+
+  public getLayer(layer: EnvironmentLayer): EnvironmentLayerInfo | null {
+    const row = this.db
+      .query<{
+        layer: string;
+        platform: string | null;
+        status: string;
+        detected_from_json: string;
+        constraints_json: string;
+        unresolved_inquiry: string | null;
+      }, [string]>("SELECT * FROM environment_topology WHERE layer = ?")
+      .get(layer);
+
+    if (!row) return null;
+    return {
+      layer: row.layer as EnvironmentLayer,
+      platform: row.platform ?? undefined,
+      status: row.status as any,
+      detected_from: JSON.parse(row.detected_from_json || "[]"),
+      constraints: JSON.parse(row.constraints_json || "[]"),
+      unresolved_inquiry: row.unresolved_inquiry ?? undefined,
+    };
+  }
+
+  public getAllLayers(): Record<EnvironmentLayer, EnvironmentLayerInfo> {
+    const rows = this.db
+      .query<{
+        layer: string;
+        platform: string | null;
+        status: string;
+        detected_from_json: string;
+        constraints_json: string;
+        unresolved_inquiry: string | null;
+      }, []>("SELECT * FROM environment_topology")
+      .all();
+
+    const result: Partial<Record<EnvironmentLayer, EnvironmentLayerInfo>> = {};
+    for (const row of rows) {
+      result[row.layer as EnvironmentLayer] = {
+        layer: row.layer as EnvironmentLayer,
+        platform: row.platform ?? undefined,
+        status: row.status as any,
+        detected_from: JSON.parse(row.detected_from_json || "[]"),
+        constraints: JSON.parse(row.constraints_json || "[]"),
+        unresolved_inquiry: row.unresolved_inquiry ?? undefined,
+      };
+    }
+    return result as Record<EnvironmentLayer, EnvironmentLayerInfo>;
+  }
+
+  public saveLayer(info: EnvironmentLayerInfo): void {
+    this.db
+      .query(
+        `INSERT INTO environment_topology (
+          layer, platform, status, detected_from_json, constraints_json, unresolved_inquiry, updated_at
+        ) VALUES (?, ?, ?, ?, ?, ?, CURRENT_TIMESTAMP)
+        ON CONFLICT(layer) DO UPDATE SET
+          platform = excluded.platform,
+          status = excluded.status,
+          detected_from_json = excluded.detected_from_json,
+          constraints_json = excluded.constraints_json,
+          unresolved_inquiry = excluded.unresolved_inquiry,
+          updated_at = CURRENT_TIMESTAMP`
+      )
+      .run(
+        info.layer,
+        info.platform ?? null,
+        info.status,
+        JSON.stringify(info.detected_from ?? []),
+        JSON.stringify(info.constraints ?? []),
+        info.unresolved_inquiry ?? null
+      );
+  }
+
+  public clearAll(): void {
+    this.db.run("DELETE FROM environment_topology");
+    this.db.run("DELETE FROM environment_nodes");
+  }
+
+  // Node CRUD operations for Multi-Node Topology
+  public getNode(id: string): TopologyNode | null {
+    const row = this.db
+      .query<{
+        id: string;
+        layer: string;
+        name: string;
+        platform: string;
+        canonical_tag: string;
+        domain_or_ip: string | null;
+        role: string | null;
+        status: string;
+        detected_from_json: string;
+        constraints_json: string;
+        connected_to_json: string;
+        unresolved_inquiry: string | null;
+      }, [string]>("SELECT * FROM environment_nodes WHERE id = ?")
+      .get(id);
+
+    if (!row) return null;
+    return this.mapNodeRow(row);
+  }
+
+  public getAllNodes(): TopologyNode[] {
+    const rows = this.db
+      .query<{
+        id: string;
+        layer: string;
+        name: string;
+        platform: string;
+        canonical_tag: string;
+        domain_or_ip: string | null;
+        role: string | null;
+        status: string;
+        detected_from_json: string;
+        constraints_json: string;
+        connected_to_json: string;
+        unresolved_inquiry: string | null;
+      }, []>("SELECT * FROM environment_nodes ORDER BY layer, id")
+      .all();
+
+    return rows.map((r) => this.mapNodeRow(r));
+  }
+
+  public getNodesByLayer(layer: EnvironmentLayer): TopologyNode[] {
+    const rows = this.db
+      .query<{
+        id: string;
+        layer: string;
+        name: string;
+        platform: string;
+        canonical_tag: string;
+        domain_or_ip: string | null;
+        role: string | null;
+        status: string;
+        detected_from_json: string;
+        constraints_json: string;
+        connected_to_json: string;
+        unresolved_inquiry: string | null;
+      }, [string]>("SELECT * FROM environment_nodes WHERE layer = ? ORDER BY id")
+      .all(layer);
+
+    return rows.map((r) => this.mapNodeRow(r));
+  }
+
+  public saveNode(node: TopologyNode): void {
+    this.db
+      .query(
+        `INSERT INTO environment_nodes (
+          id, layer, name, platform, canonical_tag, domain_or_ip, role, status,
+          detected_from_json, constraints_json, connected_to_json, unresolved_inquiry, updated_at
+        ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, CURRENT_TIMESTAMP)
+        ON CONFLICT(id) DO UPDATE SET
+          layer = excluded.layer,
+          name = excluded.name,
+          platform = excluded.platform,
+          canonical_tag = excluded.canonical_tag,
+          domain_or_ip = excluded.domain_or_ip,
+          role = excluded.role,
+          status = excluded.status,
+          detected_from_json = excluded.detected_from_json,
+          constraints_json = excluded.constraints_json,
+          connected_to_json = excluded.connected_to_json,
+          unresolved_inquiry = excluded.unresolved_inquiry,
+          updated_at = CURRENT_TIMESTAMP`
+      )
+      .run(
+        node.id,
+        node.layer,
+        node.name,
+        node.platform,
+        node.canonical_tag || "unspecified",
+        node.domain_or_ip ?? null,
+        node.role ?? null,
+        node.status,
+        JSON.stringify(node.detected_from ?? []),
+        JSON.stringify(node.constraints ?? []),
+        JSON.stringify(node.connected_to ?? []),
+        node.unresolved_inquiry ?? null
+      );
+  }
+
+  public deleteNode(id: string): void {
+    this.db.query("DELETE FROM environment_nodes WHERE id = ?").run(id);
+  }
+
+  private mapNodeRow(row: any): TopologyNode {
+    return {
+      id: row.id,
+      layer: row.layer as EnvironmentLayer,
+      name: row.name,
+      platform: row.platform,
+      canonical_tag: (row.canonical_tag as CanonicalPlatformTag) || "unspecified",
+      domain_or_ip: row.domain_or_ip ?? undefined,
+      role: row.role ?? undefined,
+      status: row.status as any,
+      detected_from: JSON.parse(row.detected_from_json || "[]"),
+      constraints: JSON.parse(row.constraints_json || "[]"),
+      connected_to: JSON.parse(row.connected_to_json || "[]"),
+      unresolved_inquiry: row.unresolved_inquiry ?? undefined,
+    };
+  }
+}