@saptools/service-flow 0.1.59 → 0.1.61

package/dist/index.js

@@ -17,7 +17,7 @@ import {
   stripQuotes,
   substituteVariables,
   trace
-} from "./chunk-GLZSDBRG.js";
+} from "./chunk-BGD7UYJN.js";
 
 // src/parsers/generated-constants-parser.ts
 import fs from "fs/promises";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@saptools/service-flow",
-  "version": "0.1.59",
+  "version": "0.1.61",
   "description": "Trace SAP CAP service-to-service flows across multi-repository workspaces with runtime-aware graph resolution",
   "type": "module",
   "publishConfig": {

package/src/parsers/outbound-call-parser.ts

@@ -57,18 +57,6 @@ function queryRunEvidence(
     } : {}),
   };
 }
-function extractQueryEntity(expr: string): string | undefined {
-  const source = ts.createSourceFile('query.ts', `const __query = (${expr});`, ts.ScriptTarget.Latest, true, ts.ScriptKind.TS);
-  const initializers = variableInitializers(source);
-  let found: string | undefined;
-  const visit = (node: ts.Node): void => {
-    if (found) return;
-    if (ts.isParenthesizedExpression(node)) found = queryEntityFromAst(node.expression, initializers);
-    ts.forEachChild(node, visit);
-  };
-  visit(source);
-  return found;
-}
 function queryWarning(expr: string): string {
   if (/^\s*[`'"]/.test(expr)) return 'raw_sql_or_cql_expression';
   if (/^\s*\w+\s*$/.test(expr)) return 'query_variable_without_static_initializer';
@@ -387,7 +375,7 @@ export function classifyOutboundCallsInSource(source: ts.SourceFile, filePath: s
         const objectArg = node.arguments[0];
         if (objectArg && ts.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);
@@ -398,8 +386,13 @@ export function classifyOutboundCallsInSource(source: ts.SourceFile, filePath: s
           const entityCallTypes: Record<string, OutboundCallFact['callType']> = { 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) : undefined;
-          add(node, { callType: query ? 'remote_query' : entityCallType ?? (isODataQueryRead ? 'remote_query' : 'remote_action'), serviceVariableName: receiver, method, operationPathExpr, queryEntity: query ? extractQueryEntity(query) : isODataQueryRead ? intent.entitySegment : undefined, payloadSummary: summarizeExpression(objectArg.getText(source)), confidence: op || query ? 0.8 : 0.4, unresolvedReason }, { receiver, classifier: 'service_client_send_object', operationPathExpression: shorthandPath ? op : undefined, rawPathExpression: pathAnalysis.rawExpression, literalPathSource: literalPathSource(pathAnalysis), odataPathIntent: operationPathExpr ? intent : undefined, pathAnalysis, staticPathCandidates: legacyPathCandidates(pathAnalysis), parserWarning: unresolvedReason });
+          const queryEntity = queryExpression
+            ? queryEntityFromAst(queryExpression, initializers)
+            : isODataQueryRead ? intent.entitySegment : undefined;
+          const unresolvedReason = queryExpression
+            ? queryEntity ? undefined : queryWarning(queryExpression.getText(source))
+            : pathExpr ? pathUnresolvedReason(pathAnalysis) : undefined;
+          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 : undefined, rawPathExpression: pathAnalysis.rawExpression, literalPathSource: literalPathSource(pathAnalysis), odataPathIntent: operationPathExpr ? intent : undefined, pathAnalysis, staticPathCandidates: legacyPathCandidates(pathAnalysis), parserWarning: unresolvedReason });
         } else {
           const receiver = receiverName(expr.expression);
           const rootReceiver = rootReceiverName(expr.expression);

package/src/parsers/symbol-parser.ts

@@ -242,7 +242,17 @@ export async function parseExecutableSymbols(
       if (localName) addSymbol('method', localName, node, parentClass);
     } else if (ts.isPropertyDeclaration(node) && parentClass && node.initializer && (ts.isArrowFunction(node.initializer) || ts.isFunctionExpression(node.initializer))) {
       const localName = nameOf(node.name);
-      if (localName) addSymbol('method', localName, node.initializer, parentClass, undefined, { source: 'class_property_function', memberKind: ts.isArrowFunction(node.initializer) ? 'arrow_function_property' : 'function_expression_property' });
+      if (localName) {
+        const flags = ts.getCombinedModifierFlags(node);
+        const staticPublicExported = exportedClasses.has(parentClass) && (flags & ts.ModifierFlags.Static) !== 0 && (flags & ts.ModifierFlags.Private) === 0 && (flags & ts.ModifierFlags.Protected) === 0;
+        const isArrow = ts.isArrowFunction(node.initializer);
+        const memberKind = isArrow
+          ? (staticPublicExported ? 'static_arrow_function' : 'arrow_function_property')
+          : (staticPublicExported ? 'static_function_expression' : 'function_expression_property');
+        addSymbol('method', localName, node.initializer, parentClass, staticPublicExported ? `${parentClass}.${localName}` : undefined, staticPublicExported
+          ? { source: 'exported_class_member', exportedClass: parentClass, memberKind }
+          : { source: 'class_property_function', memberKind });
+      }
     } else if (ts.isFunctionDeclaration(node) && node.name) addSymbol('function', node.name.text, node, undefined, exported(node) ? node.name.text : undefined);
     else if (ts.isVariableStatement(node)) {
       for (const d of node.declarationList.declarations) {