@saptools/service-flow 0.1.59 → 0.1.60

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 0.1.60
+
+- Fixed RC-A by resolving remote `send({ query: ... })` entities from the original query initializer AST and its real lexical scope, removing the isolated text reparse that fabricated parameter, mutable-local, runtime-const, runtime-destructured, and shadowed entity names while preserving genuine static entities and top-level query aliases.
+- Fixed RC-B by assigning the existing query warning to query-present remote calls whose entity stays dynamic, matching local query diagnostics without changing the `remote_query` call type, confidence, schema, linker, trace, or output shape and completing the remote identifier-reparsing follow-up documented in 0.1.59.
+
 ## 0.1.59
 
 - Fixed RC-A by making local DB query entities lexical-binding-aware: parameters, simple mutable locals, runtime const aliases, destructuring from runtime or reassigned sources, and direct `this.<field>` targets now remain dynamic at the existing low-confidence warning path instead of becoming fabricated entities.

package/dist/{chunk-GLZSDBRG.js → chunk-BGD7UYJN.js}

@@ -3676,18 +3676,6 @@ function queryRunEvidence(source, argument) {
     } : {}
   };
 }
-function extractQueryEntity(expr) {
-  const source = ts10.createSourceFile("query.ts", `const __query = (${expr});`, ts10.ScriptTarget.Latest, true, ts10.ScriptKind.TS);
-  const initializers = variableInitializers(source);
-  let found;
-  const visit = (node) => {
-    if (found) return;
-    if (ts10.isParenthesizedExpression(node)) found = queryEntityFromAst(node.expression, initializers);
-    ts10.forEachChild(node, visit);
-  };
-  visit(source);
-  return found;
-}
 function queryWarning(expr) {
   if (/^\s*[`'"]/.test(expr)) return "raw_sql_or_cql_expression";
   if (/^\s*\w+\s*$/.test(expr)) return "query_variable_without_static_initializer";
@@ -3991,7 +3979,7 @@ function classifyOutboundCallsInSource(source, filePath) {
         const objectArg = node.arguments[0];
         if (objectArg && ts10.isObjectLiteralExpression(objectArg)) {
           const receiver = receiverName(expr.expression);
-          const query = objectPropertyText(objectArg, "query");
+          const queryExpression = propertyInitializer(objectArg, "query");
           const method = stripQuotes(resolveExpression(propertyInitializer(objectArg, "method"), node, "literal").value ?? objectPropertyText(objectArg, "method") ?? "POST");
           const pathExpr = propertyInitializer(objectArg, "path") ?? propertyInitializer(objectArg, "event");
           const pathAnalysis = analyzeOperationPath(pathExpr, node, method);
@@ -4002,8 +3990,9 @@ function classifyOutboundCallsInSource(source, filePath) {
           const entityCallTypes = { entity_mutation: "remote_entity_mutation", entity_delete: "remote_entity_delete", entity_media: "remote_entity_media", entity_candidate: "remote_entity_candidate" };
           const entityCallType = entityCallTypes[intent.kind];
           const isODataQueryRead = method.toUpperCase() === "GET" && ["entity_query", "entity_key_read", "entity_navigation_query"].includes(intent.kind);
-          const unresolvedReason = !query && pathExpr ? pathUnresolvedReason(pathAnalysis) : void 0;
-          add(node, { callType: query ? "remote_query" : entityCallType ?? (isODataQueryRead ? "remote_query" : "remote_action"), serviceVariableName: receiver, method, operationPathExpr, queryEntity: query ? extractQueryEntity(query) : isODataQueryRead ? intent.entitySegment : void 0, payloadSummary: summarizeExpression(objectArg.getText(source)), confidence: op || query ? 0.8 : 0.4, unresolvedReason }, { receiver, classifier: "service_client_send_object", operationPathExpression: shorthandPath ? op : void 0, rawPathExpression: pathAnalysis.rawExpression, literalPathSource: literalPathSource(pathAnalysis), odataPathIntent: operationPathExpr ? intent : void 0, pathAnalysis, staticPathCandidates: legacyPathCandidates(pathAnalysis), parserWarning: unresolvedReason });
+          const queryEntity = queryExpression ? queryEntityFromAst(queryExpression, initializers) : isODataQueryRead ? intent.entitySegment : void 0;
+          const unresolvedReason = queryExpression ? queryEntity ? void 0 : queryWarning(queryExpression.getText(source)) : pathExpr ? pathUnresolvedReason(pathAnalysis) : void 0;
+          add(node, { callType: queryExpression ? "remote_query" : entityCallType ?? (isODataQueryRead ? "remote_query" : "remote_action"), serviceVariableName: receiver, method, operationPathExpr, queryEntity, payloadSummary: summarizeExpression(objectArg.getText(source)), confidence: op || queryExpression ? 0.8 : 0.4, unresolvedReason }, { receiver, classifier: "service_client_send_object", operationPathExpression: shorthandPath ? op : void 0, rawPathExpression: pathAnalysis.rawExpression, literalPathSource: literalPathSource(pathAnalysis), odataPathIntent: operationPathExpr ? intent : void 0, pathAnalysis, staticPathCandidates: legacyPathCandidates(pathAnalysis), parserWarning: unresolvedReason });
         } else {
           const receiver = receiverName(expr.expression);
           const rootReceiver = rootReceiverName(expr.expression);
@@ -8970,4 +8959,4 @@ export {
   selectorRepoAmbiguousDiagnostic,
   trace
 };
-//# sourceMappingURL=chunk-GLZSDBRG.js.map
+//# sourceMappingURL=chunk-BGD7UYJN.js.map