@saptools/service-flow 0.1.34 → 0.1.36

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 0.1.35
+
+- Hardened OData path precedence so entity key, navigation, and media/property paths with placeholders remain terminal entity evidence instead of dynamic operation candidates.
+- Preserved separate evidence for service-routing placeholders, operation invocation argument placeholders, and entity key placeholders.
+- Render operation-resolved parser entity calls as operation calls in traces while retaining the original parser call type for auditability.
+- Added strict doctor coverage for dynamic remote-entity false positives without indexed operation evidence.
+
 ## 0.1.34
 
 - Prefer indexed CDS operation evidence over heuristic remote-entity classification for service-client operation invocations, while keeping true collection, entity, delete, navigation, and media paths terminal.

package/TECHNICAL-NOTE.md

@@ -137,3 +137,8 @@ Schema version 6 adds queryable external target metadata columns to `outbound_ca
 ### 0.1.17 parser ownership policy
 
 Outbound call extraction is AST-based and ignores comments, block comments, and string literals. CAP/service `.on(...)` registrations are indexed only when the receiver has CAP/service evidence, and top-level registrations receive `module:<relative-file>#event:<event-name>:<line>` synthetic owners. Generic event emitters such as desktop or window events are ignored by default rather than guessed as CAP async edges. Unsupported source shapes are surfaced through diagnostics and strict doctor ownerless categories instead of guessed graph edges.
+
+
+## 0.1.35 OData placeholder semantics
+
+Service-flow now records OData placeholders by semantic layer. Service-routing placeholders belong to service bindings and can make an operation edge dynamic until runtime variables are supplied. Operation invocation argument placeholders, such as action/function call arguments, remain operation evidence but are not service-routing variables. Entity key placeholders belong to entity addressing, so key, navigation, and media/property paths remain terminal remote entity/query edges unless indexed CDS operation evidence provides a credible operation match.

package/dist/{chunk-5SR4SFSU.js → chunk-2UOIY2NI.js}

@@ -557,15 +557,28 @@ function classifyODataPathIntent(path9, method) {
   const hasNavigationSegments = segments.length > 1;
   const entitySegment = entitySegmentFromPath(pathWithoutQuery);
   const placeholderKeys = [...new Set(extractTemplatePlaceholders(rawPath))];
-  const base = { rawPath, method: normalizedMethod, pathWithoutQuery, queryString, hasQueryString: queryIndex >= 0, entitySegment, placeholderKeys };
+  const firstOpen = firstSegment.indexOf("(");
+  const firstClose = firstOpen >= 0 ? matchingClose(firstSegment, firstOpen) : void 0;
+  const keyPredicateText = firstOpen >= 0 && firstClose !== void 0 ? firstSegment.slice(firstOpen + 1, firstClose) : "";
+  const rawKeyPredicatePlaceholderKeys = [...new Set(extractTemplatePlaceholders(keyPredicateText))];
+  const navigationSuffix = hasNavigationSegments ? segments.slice(1).join("/") : void 0;
+  const lastSegment = segments.at(-1) ?? "";
+  const mediaOrPropertySuffix = hasNavigationSegments ? lastSegment : void 0;
+  const invocation = normalizeODataOperationInvocationPath(pathWithoutQuery);
+  const operationNameFromInvocation = invocation?.wasInvocation ? invocation.normalizedOperationPath.replace(/^\//, "").split(".").at(-1) : void 0;
+  const topLevelName = firstSegment.split("(")[0]?.replace(/^\//, "");
+  const topLevelOperationName = operationNameFromInvocation ?? (!hasNavigationSegments && !firstSegment.includes("(") ? topLevelName : void 0);
+  const topLevelOperationInvocation = Boolean(invocation?.wasInvocation && looksLikeLowerCamelInvocation(firstSegment));
+  const keyPredicatePlaceholderKeys = topLevelOperationInvocation ? [] : rawKeyPredicatePlaceholderKeys;
+  const topLevelOperationNameCandidate = Boolean(topLevelOperationName && !hasNavigationSegments && !firstSegment.includes("("));
+  const base = { rawPath, method: normalizedMethod, pathWithoutQuery, queryString, hasQueryString: queryIndex >= 0, entitySegment, placeholderKeys, keyPredicatePlaceholderKeys, invocationArgumentPlaceholderKeys: invocation?.invocationArgumentPlaceholderKeys ?? [], navigationSuffix, mediaOrPropertySuffix, hasEntityKeyPredicate: firstOpen >= 0 && firstClose !== void 0, hasNavigationSuffix: hasNavigationSegments, hasMediaOrPropertySuffix: Boolean(mediaOrPropertySuffix && isMediaOrPropertySuffix(mediaOrPropertySuffix)), topLevelOperationName, topLevelOperationNameCandidate, topLevelOperationInvocation };
   if (!rawPath || !rawPath.startsWith("/")) return { ...base, kind: "unknown", reason: "path_missing_or_not_absolute" };
   const upperEntityLike = /^[A-Z][A-Za-z0-9_]*$/.test(entitySegment ?? firstSegment);
-  const mediaLike = ["content", "$value"].includes((segments.at(-1) ?? "").toLowerCase());
-  const invocation = normalizeODataOperationInvocationPath(pathWithoutQuery);
+  const mediaLike = isMediaOrPropertySuffix(segments.at(-1) ?? "");
   if (normalizedMethod !== "GET") {
     if (invocation?.wasInvocation && looksLikeLowerCamelInvocation(firstSegment)) return { ...base, kind: "operation_invocation", reason: "non_get_balanced_top_level_operation_invocation" };
     if (mediaLike) return { ...base, kind: "entity_media", reason: "non_get_entity_media_stream_path" };
-    if (hasNavigationSegments || firstSegment.includes("(")) return { ...base, kind: normalizedMethod === "DELETE" ? "entity_delete" : "entity_mutation", reason: "non_get_entity_path_shape" };
+    if (hasNavigationSegments || firstSegment.includes("(")) return { ...base, kind: normalizedMethod === "DELETE" ? "entity_delete" : "entity_mutation", reason: firstSegment.includes("(") ? "non_get_entity_key_or_navigation_path_shape" : "non_get_entity_navigation_path_shape" };
     if (upperEntityLike) return { ...base, kind: normalizedMethod === "DELETE" ? "entity_delete" : "entity_mutation", reason: "non_get_entity_path_shape" };
     return { ...base, kind: "operation_invocation", reason: "non_get_lowercase_path_may_be_operation" };
   }
@@ -589,6 +602,9 @@ function entitySegmentFromPath(path9) {
   const entity = (open >= 0 ? first.slice(0, open) : first).trim();
   return entity || void 0;
 }
+function isMediaOrPropertySuffix(segment) {
+  return ["file", "content", "$value", "metadata", "items"].includes(segment.toLowerCase());
+}
 function looksLikeLowerCamelInvocation(segment) {
   const open = segment.indexOf("(");
   if (open <= 0) return false;
@@ -1319,16 +1335,32 @@ async function importsFor(repoPath, filePath, sf) {
   }
   return imports;
 }
-function collectReturnedObjectBindings(fn) {
+function collectLocalBindingFacts(fn) {
   const bindings = /* @__PURE__ */ new Map();
-  const returns = /* @__PURE__ */ new Map();
   function visit(node) {
     if (node !== fn && (ts5.isFunctionDeclaration(node) || ts5.isArrowFunction(node) || ts5.isFunctionExpression(node)))
       return;
     if (ts5.isVariableDeclaration(node) && ts5.isIdentifier(node.name) && node.initializer) {
       const fact = findConnectInExpression(node.initializer);
       if (fact) bindings.set(node.name.text, fact);
+      const call = unwrapCall(node.initializer);
+      if (call && ts5.isPropertyAccessExpression(call.expression) && call.expression.name.text === "tx" && ts5.isIdentifier(call.expression.expression)) {
+        const sourceName = call.expression.expression.text;
+        const source = bindings.get(sourceName);
+        if (source) bindings.set(node.name.text, { ...source, helperChain: [...source.helperChain ?? [], { aliasOf: sourceName, callerVariable: node.name.text, aliasKind: "transaction", transactionAliasSource: sourceName }] });
+      }
     }
+    ts5.forEachChild(node, visit);
+  }
+  visit(fn);
+  return bindings;
+}
+function collectReturnedObjectBindings(fn) {
+  const bindings = collectLocalBindingFacts(fn);
+  const returns = /* @__PURE__ */ new Map();
+  function visit(node) {
+    if (node !== fn && (ts5.isFunctionDeclaration(node) || ts5.isArrowFunction(node) || ts5.isFunctionExpression(node)))
+      return;
     if (ts5.isReturnStatement(node) && node.expression && ts5.isObjectLiteralExpression(node.expression)) {
       for (const prop of node.expression.properties) {
         if (ts5.isShorthandPropertyAssignment(prop)) returns.set(prop.name.text, prop.name.text);
@@ -1354,15 +1386,11 @@ function functionLikeInitializer(expr) {
   return void 0;
 }
 function directReturnConnectFact(fn) {
-  const localBindings = /* @__PURE__ */ new Map();
+  const localBindings = collectLocalBindingFacts(fn);
   let returned;
   function visit(node) {
     if (node !== fn && (ts5.isFunctionDeclaration(node) || ts5.isArrowFunction(node) || ts5.isFunctionExpression(node)))
       return;
-    if (ts5.isVariableDeclaration(node) && ts5.isIdentifier(node.name) && node.initializer) {
-      const fact = findConnectInExpression(node.initializer);
-      if (fact) localBindings.set(node.name.text, fact);
-    }
     if (!returned && ts5.isReturnStatement(node) && node.expression)
       returned = node.expression;
     if (!returned) ts5.forEachChild(node, visit);
@@ -1466,8 +1494,11 @@ async function parseServiceBindings(repoPath, filePath) {
   const helperCache = /* @__PURE__ */ new Map();
   const classHelpers = collectClassHelpers(sourceFileAst);
   const localObjectHelpers = /* @__PURE__ */ new Map();
+  const localDirectHelpers = /* @__PURE__ */ new Map();
   for (const stmt of sourceFileAst.statements) {
     if (ts5.isFunctionDeclaration(stmt) && stmt.name) {
+      const directFact = directConnectFactFromFunctionLike(stmt);
+      if (directFact) localDirectHelpers.set(stmt.name.text, { ...directFact, exportedName: stmt.name.text, sourceFile: normalizePath(filePath), sourceLine: lineOf4(sourceFileAst, stmt) });
       const rows2 = [];
       for (const [returnedProperty, fact] of collectReturnedObjectBindings(stmt))
         rows2.push({ ...fact, exportedName: stmt.name.text, returnedProperty, sourceFile: normalizePath(filePath), sourceLine: lineOf4(sourceFileAst, stmt) });
@@ -1478,6 +1509,8 @@ async function parseServiceBindings(repoPath, filePath) {
         if (!ts5.isIdentifier(decl.name)) continue;
         const helper = functionLikeInitializer(decl.initializer);
         if (!helper) continue;
+        const directFact = directConnectFactFromFunctionLike(helper);
+        if (directFact) localDirectHelpers.set(decl.name.text, { ...directFact, exportedName: decl.name.text, sourceFile: normalizePath(filePath), sourceLine: lineOf4(sourceFileAst, decl) });
         const rows2 = [];
         for (const [returnedProperty, fact] of collectReturnedObjectBindings(helper))
           rows2.push({ ...fact, exportedName: decl.name.text, returnedProperty, sourceFile: normalizePath(filePath), sourceLine: lineOf4(sourceFileAst, decl) });
@@ -1539,7 +1572,8 @@ async function parseServiceBindings(repoPath, filePath) {
         helperChain: aliasKind === "assignment" ? [{ callerVariable: targetName, assignedFrom: call.expression.getText(sourceFileAst), aliasKind, scopeRule: "same-file-source-order" }] : void 0
       });
     else if (ts5.isIdentifier(call.expression)) {
-      const resolved2 = await importedHelper(call.expression.text);
+      const localDirect = localDirectHelpers.get(call.expression.text);
+      const resolved2 = localDirect ? { helper: localDirect, imp: void 0 } : await importedHelper(call.expression.text);
       if (resolved2)
         out.push({
           variableName: targetName,
@@ -1552,12 +1586,13 @@ async function parseServiceBindings(repoPath, filePath) {
           sourceFile: normalizePath(filePath),
           sourceLine: lineOf4(sourceFileAst, node),
           helperChain: [
+            ...resolved2.helper.helperChain ?? [],
             {
               callerVariable: targetName,
               ...aliasKind === "assignment" ? { assignedFrom: call.expression.text, aliasKind, scopeRule: "same-file-source-order" } : {},
               importedHelper: call.expression.text,
-              importSource: resolved2.imp.sourceFile,
-              exportedSymbol: resolved2.imp.exportedName,
+              importSource: resolved2.imp?.sourceFile,
+              exportedSymbol: resolved2.imp?.exportedName ?? resolved2.helper.exportedName,
               helperSourceFile: resolved2.helper.sourceFile,
               helperSourceLine: resolved2.helper.sourceLine
             }
@@ -1597,7 +1632,7 @@ async function parseServiceBindings(repoPath, filePath) {
         placeholders: resolved2.helper.placeholders,
         sourceFile: normalizePath(filePath),
         sourceLine: lineOf4(sourceFileAst, decl),
-        helperChain: [{ callerVariable: el.name.text, helperFunction: call.expression.getText(sourceFileAst), returnedProperty: propertyName, importSource: resolved2.imp?.sourceFile, exportedSymbol: resolved2.imp?.exportedName, helperSourceFile: resolved2.helper.sourceFile, helperSourceLine: resolved2.helper.sourceLine }]
+        helperChain: [...resolved2.helper.helperChain ?? [], { callerVariable: el.name.text, helperFunction: call.expression.getText(sourceFileAst), returnedProperty: propertyName, importSource: resolved2.imp?.sourceFile, exportedSymbol: resolved2.imp?.exportedName, helperSourceFile: resolved2.helper.sourceFile, helperSourceLine: resolved2.helper.sourceLine }]
       });
     }
   }
@@ -1630,7 +1665,7 @@ async function parseServiceBindings(repoPath, filePath) {
         placeholders: resolved2.helper.placeholders,
         sourceFile: normalizePath(filePath),
         sourceLine: lineOf4(sourceFileAst, node),
-        helperChain: [{ callerVariable: targetName, assignedFrom: call.expression.getText(sourceFileAst), aliasKind: "assignment", scopeRule: "same-file-source-order", returnedProperty: propertyName, importSource: resolved2.imp?.sourceFile, exportedSymbol: resolved2.imp?.exportedName, helperSourceFile: resolved2.helper.sourceFile, helperSourceLine: resolved2.helper.sourceLine }]
+        helperChain: [...resolved2.helper.helperChain ?? [], { callerVariable: targetName, assignedFrom: call.expression.getText(sourceFileAst), aliasKind: "assignment", scopeRule: "same-file-source-order", returnedProperty: propertyName, importSource: resolved2.imp?.sourceFile, exportedSymbol: resolved2.imp?.exportedName, helperSourceFile: resolved2.helper.sourceFile, helperSourceLine: resolved2.helper.sourceLine }]
       });
     }
   }
@@ -1666,6 +1701,68 @@ async function parseServiceBindings(repoPath, filePath) {
       });
     }
   }
+  function arrayElementsFromExpression(expr) {
+    const unwrapped = unwrapIdentityExpression(expr);
+    if (ts5.isArrayLiteralExpression(unwrapped)) return { elements: unwrapped.elements, promiseAll: false };
+    const call = unwrapCall(expr);
+    if (!call) return void 0;
+    if (!ts5.isPropertyAccessExpression(call.expression) || call.expression.name.text !== "all" || call.expression.expression.getText(sourceFileAst) !== "Promise") return void 0;
+    const first = call.arguments[0];
+    if (!first) return void 0;
+    const container = unwrapIdentityExpression(first);
+    if (!ts5.isArrayLiteralExpression(container)) return void 0;
+    return { elements: container.elements, promiseAll: true };
+  }
+  async function recordArrayElementBinding(targetName, expr, node, arrayIndex, promiseAll) {
+    const before = out.length;
+    await recordBindingFromExpression(targetName, expr, node, "declaration");
+    if (out.length > before) {
+      const row = out[out.length - 1];
+      row.helperChain = [
+        ...row.helperChain ?? [],
+        { callerVariable: targetName, targetVariable: targetName, arrayIndex, promiseAll, arrayContainer: promiseAll ? "Promise.all" : "array_literal" }
+      ];
+      return;
+    }
+    const unwrapped = unwrapIdentityExpression(expr);
+    if (ts5.isIdentifier(unwrapped)) {
+      const existing = bindingForVariable(unwrapped.text);
+      if (!existing) return;
+      out.push({
+        ...existing,
+        variableName: targetName,
+        sourceLine: lineOf4(sourceFileAst, node),
+        helperChain: [
+          ...existing.helperChain ?? [],
+          { callerVariable: targetName, targetVariable: targetName, sourceVariable: unwrapped.text, aliasKind: "array-destructuring", arrayIndex, promiseAll, arrayContainer: promiseAll ? "Promise.all" : "array_literal" }
+        ]
+      });
+    }
+  }
+  async function recordArrayDestructuredVariable(decl) {
+    if (!ts5.isArrayBindingPattern(decl.name) || !decl.initializer) return;
+    const container = arrayElementsFromExpression(decl.initializer);
+    if (!container) return;
+    for (let index = 0; index < decl.name.elements.length; index += 1) {
+      const el = decl.name.elements[index];
+      if (!el || ts5.isOmittedExpression(el) || ts5.isBindingElement(el) && el.dotDotDotToken) continue;
+      if (!ts5.isBindingElement(el) || !ts5.isIdentifier(el.name)) continue;
+      const source = container.elements[index];
+      if (!source || ts5.isOmittedExpression(source)) continue;
+      await recordArrayElementBinding(el.name.text, source, decl, index, container.promiseAll);
+    }
+  }
+  async function recordArrayDestructuredAssignment(pattern, expr, node) {
+    const container = arrayElementsFromExpression(expr);
+    if (!container) return;
+    for (let index = 0; index < pattern.elements.length; index += 1) {
+      const el = pattern.elements[index];
+      if (!el || ts5.isOmittedExpression(el) || ts5.isSpreadElement(el) || !ts5.isIdentifier(el)) continue;
+      const source = container.elements[index];
+      if (!source || ts5.isOmittedExpression(source)) continue;
+      await recordArrayElementBinding(el.text, source, node, index, container.promiseAll);
+    }
+  }
   const events = [];
   function collectEvents(node) {
     if (ts5.isVariableDeclaration(node)) events.push({ pos: node.getStart(sourceFileAst), node });
@@ -1679,6 +1776,7 @@ async function parseServiceBindings(repoPath, filePath) {
     if (ts5.isVariableDeclaration(event.node)) {
       const decl = event.node;
       await recordDestructuredHelper(decl);
+      await recordArrayDestructuredVariable(decl);
       recordDestructuredClassHelper(decl);
       await recordVariable(decl);
       recordIdentityAlias(decl);
@@ -1700,6 +1798,8 @@ async function parseServiceBindings(repoPath, filePath) {
     const left = ts5.isParenthesizedExpression(assignment.left) ? assignment.left.expression : assignment.left;
     if (ts5.isObjectLiteralExpression(left))
       await recordDestructuredAssignment(left, assignment.right, assignment);
+    if (ts5.isArrayLiteralExpression(left))
+      await recordArrayDestructuredAssignment(left, assignment.right, assignment);
   }
   return out;
 }
@@ -2106,17 +2206,20 @@ function insertCallEdge(db, workspaceId, call, vars, generation) {
   const destination = call.destinationExpr ?? call.requireDestination;
   const isDynamic = Boolean(Number(call.isDynamic ?? 0));
   const isRemoteEntityCall = callType.startsWith("remote_entity_");
-  const operationLikeRemoteEntity = isRemoteEntityCall && Boolean(op) && !["entity_media", "entity_delete", "entity_key_read", "entity_navigation_query"].includes(intent.kind);
+  const indexedOperationCandidateCount = operationCandidateCount(db, workspaceId, op, intent.topLevelOperationName);
+  const credibleOperationSignal = Boolean(normalized?.wasInvocation) || Boolean(intent.topLevelOperationNameCandidate) && indexedOperationCandidateCount > 0;
+  const strongEntitySignal = ["entity_media", "entity_delete", "entity_key_read", "entity_navigation_query"].includes(intent.kind) || intent.kind === "entity_mutation" && (intent.hasEntityKeyPredicate || intent.hasNavigationSuffix);
+  const operationLikeRemoteEntity = isRemoteEntityCall && Boolean(op) && credibleOperationSignal && (!strongEntitySignal || indexedOperationCandidateCount > 0);
   const isOperationCall = operationLikeRemoteEntity || (callType === "remote_action" || callType === "local_service_call" || callType === "remote_query" && Boolean(op));
   const resolution = isOperationCall ? resolveOperation(db, { servicePath, operationPath: op, serviceName: call.local_service_name, repoId: callType === "local_service_call" ? Number(call.repo_id) : void 0, alias: applyVariables(call.aliasExpr ?? call.alias, vars), destination: destination ? applyVariables(destination, vars) : void 0, isDynamic, hasExplicitOverride: Object.keys(vars).length > 0 || callType === "local_service_call" }, workspaceId) : { status: "unresolved", candidates: [], reasons: [] };
-  const evidence = callEvidence(call, resolution, servicePath, op, destination ? applyVariables(destination, vars) : void 0, normalized, intent);
+  const evidence = { ...callEvidence(call, resolution, servicePath, op, destination ? applyVariables(destination, vars) : void 0, normalized, intent), indexedOperationCandidateCount, parserCallType: callType };
   if (isRemoteEntityCall && (resolution.target || resolution.candidates.length > 0 || resolution.status === "dynamic")) {
     if (resolution.target) {
       db.prepare("INSERT INTO graph_edges(workspace_id,edge_type,status,from_kind,from_id,to_kind,to_id,confidence,evidence_json,is_dynamic,generation) VALUES(?,?,?,?,?,?,?,?,?,?,?)").run(workspaceId, "REMOTE_CALL_RESOLVES_TO_OPERATION", "resolved", "call", String(call.id), "operation", String(resolution.target.operationId), resolution.target.score, JSON.stringify({ ...evidence, operationEntityPrecedence: "indexed_operation_over_parser_entity" }), 0, generation);
       return { status: "resolved", callType };
     }
     const status2 = resolution.status === "dynamic" ? "dynamic" : resolution.status === "ambiguous" ? "ambiguous" : "unresolved";
-    db.prepare("INSERT INTO graph_edges(workspace_id,edge_type,status,from_kind,from_id,to_kind,to_id,confidence,evidence_json,is_dynamic,unresolved_reason,generation) VALUES(?,?,?,?,?,?,?,?,?,?,?,?)").run(workspaceId, status2 === "dynamic" ? "DYNAMIC_EDGE_CANDIDATE" : "UNRESOLVED_EDGE", status2, "call", String(call.id), "operation_candidate", op ? `Remote action: ${op}` : "Remote action: unknown path", Number(call.confidence ?? 0.2), JSON.stringify({ ...evidence, operationEntityPrecedence: "parser_entity_with_indexed_operation_candidates" }), status2 === "dynamic" ? 1 : 0, unresolvedOperationReason(resolution), generation);
+    db.prepare("INSERT INTO graph_edges(workspace_id,edge_type,status,from_kind,from_id,to_kind,to_id,confidence,evidence_json,is_dynamic,unresolved_reason,generation) VALUES(?,?,?,?,?,?,?,?,?,?,?,?)").run(workspaceId, status2 === "dynamic" ? "DYNAMIC_EDGE_CANDIDATE" : "UNRESOLVED_EDGE", status2, "call", String(call.id), "operation_candidate", op ? `Remote action: ${op}` : "Remote action: unknown path", Number(call.confidence ?? 0.2), JSON.stringify({ ...evidence, operationEntityPrecedence: resolution.candidates.length > 0 ? "parser_entity_with_indexed_operation_candidates" : "parser_entity_operation_candidate_without_indexed_match" }), status2 === "dynamic" ? 1 : 0, unresolvedOperationReason(resolution), generation);
     return { status: status2, callType };
   }
   if (isRemoteEntityCall) {
@@ -2149,6 +2252,12 @@ function insertCallEdge(db, workspaceId, call, vars, generation) {
   db.prepare("INSERT INTO graph_edges(workspace_id,edge_type,status,from_kind,from_id,to_kind,to_id,confidence,evidence_json,is_dynamic,unresolved_reason,generation) VALUES(?,?,?,?,?,?,?,?,?,?,?,?)").run(workspaceId, edgeType, status, "call", String(call.id), targetKind, targetId, Number(call.confidence ?? 0.2), JSON.stringify(finalEvidence), graphLevelDynamic ? 1 : 0, unresolvedReason, generation);
   return { status, callType };
 }
+function operationCandidateCount(db, workspaceId, operationPath, operationName) {
+  if (!operationPath && !operationName) return 0;
+  const normalizedName = operationName ?? operationPath?.replace(/^\//, "").split(".").at(-1);
+  const row = db.prepare(`SELECT COUNT(*) count FROM cds_operations o JOIN cds_services s ON s.id=o.service_id JOIN repositories r ON r.id=s.repo_id WHERE r.workspace_id=? AND (o.operation_path=? OR o.operation_path=? OR o.operation_name=?)`).get(workspaceId, operationPath, normalizedName ? `/${normalizedName}` : operationPath, normalizedName);
+  return Number(row?.count ?? 0);
+}
 function unresolvedOperationReason(resolution) {
   if (resolution.status === "dynamic") return `Dynamic target requires runtime variable overrides: ${(resolution.reasons.length ? resolution.reasons : ["missing runtime variables"]).join(", ")}`;
   if (resolution.candidates.length === 0) return "No indexed target operation matched";
@@ -2569,6 +2678,11 @@ function handlerScope(db, methodId) {
   if (!row) return void 0;
   return { repoId: row.repoId, files: new Set(row.sourceFile ? [row.sourceFile] : []), symbolId: row.symbolId };
 }
+function traceEdgeType(call, row) {
+  if (row.to_kind === "operation" && row.edge_type === "REMOTE_CALL_RESOLVES_TO_OPERATION") return "remote_action";
+  if (row.to_kind === "operation" && row.edge_type === "LOCAL_CALL_RESOLVES_TO_OPERATION") return "local_service_call";
+  return String(call.call_type);
+}
 function includeCall(type, options) {
   if (!options.includeDb && type === "local_db_query") return false;
   if (!options.includeExternal && type === "external_http") return false;
@@ -2862,7 +2976,7 @@ function trace(db, start, options) {
         const to = edgeTarget(effectiveRow, evidence);
         edges.push({
           step: current.depth,
-          type: String(call.call_type),
+          type: traceEdgeType(call, effectiveRow),
           from: `${call.repoName}:${call.source_file}:${call.source_line}`,
           to,
           evidence,
@@ -2934,6 +3048,7 @@ export {
   redactText,
   redactValue,
   normalizeODataOperationInvocationPath,
+  classifyODataPathIntent,
   containsSupportedOutboundCall,
   parseOutboundCalls,
   parseServiceBindings,
@@ -2943,4 +3058,4 @@ export {
   linkWorkspace,
   trace
 };
-//# sourceMappingURL=chunk-5SR4SFSU.js.map
+//# sourceMappingURL=chunk-2UOIY2NI.js.map