@saptools/service-flow 0.1.42 → 0.1.43

package/dist/cli.js

@@ -13,7 +13,7 @@ import {
   parsePackageJson,
   parseServiceBindings,
   trace
-} from "./chunk-RP3BJ64F.js";
+} from "./chunk-KHQK7CFH.js";
 
 // src/cli.ts
 import { Command } from "commander";
@@ -209,7 +209,7 @@ function migrate(db) {
 // package.json
 var package_default = {
   name: "@saptools/service-flow",
-  version: "0.1.42",
+  version: "0.1.43",
   description: "Trace SAP CAP service-to-service flows across multi-repository workspaces with runtime-aware graph resolution",
   type: "module",
   publishConfig: {
@@ -1151,36 +1151,66 @@ async function repositoryFingerprint(root, files, facts) {
 function materializeCdsExtensionOperations(db, workspaceId) {
   const extensions = db.prepare(`SELECT s.id,r.id repoId,s.service_name serviceName,s.qualified_name qualifiedName,s.source_file sourceFile,s.extension_module_specifier moduleSpecifier,s.extension_imported_symbol importedSymbol,s.extension_import_kind importKind
     FROM cds_services s JOIN repositories r ON r.id=s.repo_id WHERE r.workspace_id=? AND s.is_extend=1`).all(workspaceId);
-  for (const extension of extensions) {
-    const bases = resolveBase(db, workspaceId, extension);
-    const status = bases.length === 1 ? "resolved" : bases.length > 1 ? "ambiguous" : "unresolved";
-    db.prepare("UPDATE cds_services SET extension_base_status=?, extension_base_service_id=? WHERE id=?").run(status, status === "resolved" ? bases[0]?.id : null, extension.id);
-    if (bases.length !== 1) {
-      db.prepare("DELETE FROM cds_operations WHERE service_id=? AND provenance='inherited'").run(extension.id);
-      db.prepare("DELETE FROM search_index WHERE repo=? AND kind='operation' AND name NOT IN (SELECT operation_name FROM cds_operations o JOIN cds_services s ON s.id=o.service_id WHERE s.repo_id=?)").run(String(extension.repoId), extension.repoId);
-      continue;
+  db.transaction(() => {
+    const changedRepos = /* @__PURE__ */ new Set();
+    for (const extension of extensions) {
+      if (reconcileExtension(db, workspaceId, extension)) changedRepos.add(extension.repoId);
     }
-    reconcileInheritedOperations(db, extension, bases[0]);
-  }
+    for (const repoId of changedRepos) markRepositoryDerivedFactsChanged(db, repoId);
+  });
 }
-function reconcileInheritedOperations(db, extension, base) {
-  const existing = db.prepare("SELECT id,operation_name operationName,operation_path operationPath,base_operation_id baseOperationId FROM cds_operations WHERE service_id=? AND provenance='inherited'").all(extension.id);
-  const desired = db.prepare("SELECT id,operation_type operationType,operation_name operationName,operation_path operationPath,params_json paramsJson,return_type returnType,source_file sourceFile,source_line sourceLine FROM cds_operations WHERE service_id=? AND provenance='direct' AND NOT EXISTS (SELECT 1 FROM cds_operations direct WHERE direct.service_id=? AND direct.provenance='direct' AND direct.operation_name=cds_operations.operation_name AND direct.operation_path=cds_operations.operation_path)").all(base.id, extension.id);
-  const desiredKeys = new Set(desired.map((row) => `${String(row.operationName)}\0${String(row.operationPath)}`));
+function reconcileExtension(db, workspaceId, extension) {
+  const bases = resolveBase(db, workspaceId, extension);
+  const status = bases.length === 1 ? "resolved" : bases.length > 1 ? "ambiguous" : "unresolved";
+  const baseId = status === "resolved" ? bases[0]?.id ?? null : null;
+  const desired = baseId === null ? [] : desiredInheritedOperations(db, extension, baseId);
+  const statusChanged = updateExtensionStatus(db, extension.id, status, baseId);
+  const operationChanged = reconcileInheritedOperations(db, extension, desired);
+  reconcileOperationSearchRows(db, extension.repoId);
+  return statusChanged || operationChanged;
+}
+function desiredInheritedOperations(db, extension, baseId) {
+  return db.prepare("SELECT id,operation_type operationType,operation_name operationName,operation_path operationPath,params_json paramsJson,return_type returnType,source_file sourceFile,source_line sourceLine FROM cds_operations WHERE service_id=? AND provenance='direct' AND NOT EXISTS (SELECT 1 FROM cds_operations direct WHERE direct.service_id=? AND direct.provenance='direct' AND direct.operation_name=cds_operations.operation_name AND direct.operation_path=cds_operations.operation_path) ORDER BY operation_name,operation_path,id").all(baseId, extension.id);
+}
+function updateExtensionStatus(db, serviceId, status, baseId) {
+  const row = db.prepare("SELECT extension_base_status status,extension_base_service_id baseId FROM cds_services WHERE id=?").get(serviceId);
+  if (row?.status === status && (row.baseId ?? null) === baseId) return false;
+  db.prepare("UPDATE cds_services SET extension_base_status=?, extension_base_service_id=? WHERE id=?").run(status, baseId, serviceId);
+  return true;
+}
+function reconcileInheritedOperations(db, extension, desired) {
+  const existing = db.prepare("SELECT id,operation_type operationType,operation_name operationName,operation_path operationPath,params_json paramsJson,return_type returnType,source_file sourceFile,source_line sourceLine,base_operation_id baseOperationId FROM cds_operations WHERE service_id=? AND provenance='inherited'").all(extension.id);
+  const desiredKeys = new Set(desired.map((row) => `${row.operationName}\0${row.operationPath}`));
   const byKey = new Map(existing.map((row) => [`${row.operationName}\0${row.operationPath}`, row]));
+  let changed = false;
   for (const row of existing) {
-    if (!desiredKeys.has(`${row.operationName}\0${row.operationPath}`)) db.prepare("DELETE FROM cds_operations WHERE id=?").run(row.id);
+    if (desiredKeys.has(`${row.operationName}\0${row.operationPath}`)) continue;
+    db.prepare("DELETE FROM cds_operations WHERE id=?").run(row.id);
+    changed = true;
   }
   const update = db.prepare("UPDATE cds_operations SET operation_type=?,params_json=?,return_type=?,source_file=?,source_line=?,base_operation_id=? WHERE id=?");
   const add = db.prepare("INSERT INTO cds_operations(service_id,operation_type,operation_name,operation_path,params_json,return_type,source_file,source_line,provenance,base_operation_id) VALUES(?,?,?,?,?,?,?,?,'inherited',?)");
-  const search = db.prepare("INSERT INTO search_index(kind,name,path,repo) SELECT ?,?,?,? WHERE NOT EXISTS (SELECT 1 FROM search_index WHERE kind=? AND name=? AND path=? AND repo=?)");
   for (const row of desired) {
-    const key = `${String(row.operationName)}\0${String(row.operationPath)}`;
-    const current = byKey.get(key);
+    const current = byKey.get(`${row.operationName}\0${row.operationPath}`);
+    if (current && operationMatches(current, row)) continue;
     if (current) update.run(row.operationType, row.paramsJson, row.returnType, row.sourceFile, row.sourceLine, row.id, current.id);
     else add.run(extension.id, row.operationType, row.operationName, row.operationPath, row.paramsJson, row.returnType, row.sourceFile, row.sourceLine, row.id);
-    search.run("operation", row.operationName, row.operationPath, String(extension.repoId), "operation", row.operationName, row.operationPath, String(extension.repoId));
+    changed = true;
   }
+  return changed;
+}
+function operationMatches(current, desired) {
+  return current.operationType === desired.operationType && current.paramsJson === desired.paramsJson && current.returnType === desired.returnType && current.sourceFile === desired.sourceFile && current.sourceLine === desired.sourceLine && current.baseOperationId === desired.id;
+}
+function reconcileOperationSearchRows(db, repoId) {
+  db.prepare("DELETE FROM search_index WHERE repo=? AND kind='operation'").run(String(repoId));
+  db.prepare(`INSERT INTO search_index(kind,name,path,repo)
+    SELECT 'operation',o.operation_name,o.operation_path,?
+    FROM cds_operations o JOIN cds_services s ON s.id=o.service_id
+    WHERE s.repo_id=? ORDER BY o.operation_name,o.operation_path,o.id`).run(String(repoId), repoId);
+}
+function markRepositoryDerivedFactsChanged(db, repoId) {
+  db.prepare("UPDATE repositories SET fact_generation=COALESCE(fact_generation,0)+1, graph_stale_reason='derived_facts_changed', graph_stale_at=? WHERE id=?").run((/* @__PURE__ */ new Date()).toISOString(), repoId);
 }
 function resolveBase(db, workspaceId, extension) {
   const symbol = extension.importedSymbol ?? extension.serviceName;