@superatomai/sdk-node 0.0.48 → 0.0.50

package/dist/index.js

@@ -5720,24 +5720,22 @@ ${JSON.stringify(tool.requiredFields || [], null, 2)}`;
       if (executedTools && executedTools.length > 0) {
         logger.info(`[${this.getProviderName()}] Passing ${executedTools.length} executed tools to component matching`);
         executedToolsText = "The following external tools were executed to fetch data.\n**IMPORTANT: For components displaying this data, use externalTool prop instead of query.**\n**IMPORTANT: Use the outputSchema fields to set correct config keys (xAxisKey, yAxisKey, valueKey, nameKey, etc.)**\n**IMPORTANT: Use the result data to populate deferred tool parameters when applicable.**\n\n" + executedTools.map((tool, idx) => {
-          let resultPreview = "No result data";
-          if (tool.result) {
-            const resultStr = typeof tool.result === "string" ? tool.result : JSON.stringify(tool.result, null, 2);
-            resultPreview = resultStr.length > 2e3 ? resultStr.substring(0, 2e3) + "\n... (truncated)" : resultStr;
-          }
           let outputSchemaText = "Not available";
+          let recordCount = "unknown";
           if (tool.outputSchema) {
             const fields = tool.outputSchema.fields || [];
+            recordCount = tool.result?._recordCount || (Array.isArray(tool.result) ? tool.result.length : "unknown");
             outputSchemaText = `${tool.outputSchema.description}
-   Fields (use these exact names for component config):
+   Fields in EACH ROW (per-row values, NOT aggregates):
 ${fields.map((f) => `     - ${f.name} (${f.type}): ${f.description}`).join("\n")}`;
           }
           return `${idx + 1}. **${tool.name}**
    toolId: "${tool.id}" (USE THIS EXACT VALUE for externalTool.toolId)
    toolName: "${tool.name}" (USE THIS EXACT VALUE for externalTool.toolName)
    parameters: ${JSON.stringify(tool.params || {})} (USE THESE for externalTool.parameters)
+   recordCount: ${recordCount} rows returned
    outputSchema: ${outputSchemaText}
-   result preview: ${resultPreview}`;
+   \u26A0\uFE0F DO NOT embed this tool's data in SQL - use externalTool prop OR query database tables`;
         }).join("\n\n");
       }
       const schemaDoc = schema.generateSchemaDocumentation();
@@ -5850,23 +5848,9 @@ ${fields.map((f) => `     - ${f.name} (${f.type}): ${f.description}`).join("\n")
         true
         // Parse as JSON
       );
-      logger.debug(`[${this.getProviderName()}] Component matching response parsed successfully`);
-      const componentSuggestionPattern = /c\d+:(\w+)\s*:\s*(.+)/g;
-      const suggestedComponents = [];
-      let match;
-      while ((match = componentSuggestionPattern.exec(analysisContent)) !== null) {
-        suggestedComponents.push({
-          type: match[1],
-          reasoning: match[2].trim()
-        });
-      }
       const matchedComponents = result.matchedComponents || [];
       const layoutTitle = result.layoutTitle || "Dashboard";
       const layoutDescription = result.layoutDescription || "Multi-component dashboard";
-      logger.info(`[${this.getProviderName()}] \u{1F4CA} Component Suggestions from Text Analysis: ${suggestedComponents.length}`);
-      suggestedComponents.forEach((comp, idx) => {
-        logger.info(`[${this.getProviderName()}]   c${idx + 1}: ${comp.type} - ${comp.reasoning}`);
-      });
       logger.info(`[${this.getProviderName()}] \u{1F4E6} Matched Components from LLM: ${matchedComponents.length}`);
       matchedComponents.forEach((comp, idx) => {
         logger.info(`[${this.getProviderName()}]   ${idx + 1}. ${comp.componentType} (${comp.componentName}) - ${comp.originalSuggestion || "N/A"}`);
@@ -5874,22 +5858,7 @@ ${fields.map((f) => `     - ${f.name} (${f.type}): ${f.description}`).join("\n")
       logger.file("\n=============================\nFull LLM response:", JSON.stringify(result, null, 2));
       const rawActions = result.actions || [];
       const actions = convertQuestionsToActions(rawActions);
-      logger.info(`[${this.getProviderName()}] Matched ${matchedComponents.length} components from text response`);
-      logger.info(`[${this.getProviderName()}] Layout title: "${layoutTitle}"`);
-      logger.info(`[${this.getProviderName()}] Layout description: "${layoutDescription}"`);
-      logger.info(`[${this.getProviderName()}] Generated ${actions.length} follow-up actions`);
-      if (suggestedComponents.length > 0) {
-        logCollector?.info(`\u{1F4DD} Text Analysis suggested ${suggestedComponents.length} dashboard components:`);
-        suggestedComponents.forEach((comp, idx) => {
-          logCollector?.info(`  c${idx + 1}: ${comp.type} - ${comp.reasoning}`);
-        });
-      }
       if (matchedComponents.length > 0) {
-        logCollector?.info(`\u{1F4E6} Matched ${matchedComponents.length} components for dashboard`);
-        if (suggestedComponents.length !== matchedComponents.length) {
-          logCollector?.warn(`\u26A0\uFE0F Component count mismatch: Suggested ${suggestedComponents.length}, but matched ${matchedComponents.length}`);
-        }
-        logCollector?.info(`Dashboard: "${layoutTitle}"`);
         matchedComponents.forEach((comp, idx) => {
           logCollector?.info(`  ${idx + 1}. ${comp.componentName} (${comp.componentType}): ${comp.reasoning}`);
           if (comp.props?.query) {
@@ -5901,23 +5870,37 @@ ${fields.map((f) => `     - ${f.name} (${f.type}): ${f.description}`).join("\n")
           }
         });
       }
-      if (actions.length > 0) {
-        logCollector?.info(`Generated ${actions.length} follow-up questions`);
-        actions.forEach((action, idx) => {
-          logCollector?.info(`  ${idx + 1}. ${action.name}`);
-        });
-      }
       const finalComponents = matchedComponents.map((mc) => {
         const originalComponent = components.find((c) => c.id === mc.componentId);
         if (!originalComponent) {
           logger.warn(`[${this.getProviderName()}] Component ${mc.componentId} not found in available components`);
           return null;
         }
+        let cleanedProps = { ...mc.props };
+        if (cleanedProps.externalTool) {
+          const toolId = cleanedProps.externalTool.toolId;
+          const validToolIds = (executedTools || []).map((t) => t.id);
+          const isValidTool = toolId && typeof toolId === "string" && validToolIds.includes(toolId);
+          if (!isValidTool) {
+            logger.warn(`[${this.getProviderName()}] externalTool.toolId "${toolId}" not found in executed tools [${validToolIds.join(", ")}], setting to null`);
+            cleanedProps.externalTool = null;
+          }
+        }
+        if (cleanedProps.query) {
+          const queryStr = typeof cleanedProps.query === "string" ? cleanedProps.query : cleanedProps.query?.sql || "";
+          if (queryStr.includes("OPENJSON") || queryStr.includes("JSON_VALUE")) {
+            logger.warn(`[${this.getProviderName()}] Query contains OPENJSON/JSON_VALUE (invalid - cannot parse tool result), setting query to null`);
+            cleanedProps.query = null;
+          }
+        }
+        if (cleanedProps.query && cleanedProps.externalTool) {
+          logger.info(`[${this.getProviderName()}] Both query and externalTool exist, keeping both - frontend will decide`);
+        }
         return {
           ...originalComponent,
           props: {
             ...originalComponent.props,
-            ...mc.props
+            ...cleanedProps
           }
         };
       }).filter(Boolean);