@superatomai/sdk-node 0.0.3-mds → 0.0.4-mds

package/dist/index.mjs

@@ -345,7 +345,8 @@ var UserQueryFiltersSchema = z3.object({
   username: z3.string().optional(),
   email: z3.string().optional(),
   role: z3.string().optional(),
-  fullname: z3.string().optional()
+  fullname: z3.string().optional(),
+  id: z3.number().optional()
 });
 var UsersRequestPayloadSchema = z3.object({
   operation: z3.enum(["create", "update", "delete", "getAll", "getOne", "query"]),
@@ -8316,9 +8317,9 @@ ${executedToolsText}`);
       const rawActions = result.actions || [];
       const actions = convertQuestionsToActions(rawActions);
       const finalComponents = matchedComponents.map((mc) => {
-        const originalComponent = components.find((c) => c.id === mc.componentId);
+        const originalComponent = components.find((c) => c.name === mc.componentName);
         if (!originalComponent) {
-          logger.warn(`[${this.getProviderName()}] Component ${mc.componentId} not found in available components`);
+          logger.warn(`[${this.getProviderName()}] Component "${mc.componentName}" not found in available components`);
           return null;
         }
         const cleanedProps = processComponentProps(
@@ -8778,7 +8779,7 @@ ${executedToolsText}`);
       let layoutDescription = "Multi-component dashboard";
       let actions = [];
       if (category === "general") {
-        logger.info(`[${this.getProviderName()}] Skipping component generation for general/conversational question`);
+        logger.info(`[${this.getProviderName()}] General category - wrapping text response in DynamicMarkdownBlock`);
         const nextQuestions = await this.generateNextQuestions(
           userPrompt,
           null,
@@ -8791,6 +8792,18 @@ ${executedToolsText}`);
           // pass text response as context
         );
         actions = convertQuestionsToActions(nextQuestions);
+        const markdownContent = textResponse.replace(/<DashboardComponents>[\s\S]*?<\/DashboardComponents>/g, "").trim();
+        matchedComponents = [{
+          id: "dynamic-markdown-block",
+          name: "DynamicMarkdownBlock",
+          type: "MarkdownBlock",
+          description: "Text response rendered as markdown",
+          props: {
+            content: markdownContent
+          }
+        }];
+        layoutTitle = "";
+        layoutDescription = "";
       } else if (components && components.length > 0) {
         const componentStreamCallback = streamBuffer.hasCallback() && category === "data_analysis" ? (component) => {
           const answerMarker = `__ANSWER_COMPONENT_START__${JSON.stringify(component)}__ANSWER_COMPONENT_END__`;
@@ -10715,12 +10728,13 @@ async function handleDashboardsRequest(data, collections, sendMessage) {
     const filters = requestData?.filters;
     const limit = requestData?.limit;
     const sort = requestData?.sort;
-    if (from.type !== "admin") {
+    const readOnlyOperations = ["query", "getAll", "getOne"];
+    if (!readOnlyOperations.includes(operation) && from.type !== "admin") {
       sendResponse4(id, {
         success: false,
         error: "Unauthorized: Only admin can manage dashboards"
       }, sendMessage, from.id);
-      logger.warn(`Unauthorized dashboard management attempt from: ${from.type}`);
+      logger.warn(`Unauthorized dashboard write attempt from: ${from.type}`);
       return;
     }
     const dashboardManager2 = getDashboardManager();
@@ -11085,12 +11099,13 @@ async function handleReportsRequest(data, collections, sendMessage) {
     const filters = requestData?.filters;
     const limit = requestData?.limit;
     const sort = requestData?.sort;
-    if (from.type !== "admin") {
+    const readOnlyOperations = ["query", "getAll", "getOne"];
+    if (!readOnlyOperations.includes(operation) && from.type !== "admin") {
       sendResponse5(id, {
         success: false,
         error: "Unauthorized: Only admin can manage reports"
       }, sendMessage, from.id);
-      logger.warn(`Unauthorized report management attempt from: ${from.type}`);
+      logger.warn(`Unauthorized report write attempt from: ${from.type}`);
       return;
     }
     const reportManager2 = getReportManager();
@@ -13075,7 +13090,8 @@ async function pickComponentWithLLM(prompt, components, anthropicApiKey, groqApi
         outputSchema: tool.outputSchema
       });
       logger.info(`[DASH_COMP_REQ] Tool ${tool.name} executed successfully`);
-      return JSON.stringify(result2, null, 2);
+      const resultJson = JSON.stringify(result2, null, 2);
+      return resultJson + "\n\n[REMINDER: The above is tool output for verification. You MUST still respond with ONLY the JSON component selection object. Do NOT summarize or describe these results.]";
     };
     const result = await LLM.streamWithTools(
       {
@@ -13093,18 +13109,59 @@ async function pickComponentWithLLM(prompt, components, anthropicApiKey, groqApi
       5
       // max iterations
     );
-    const jsonMatch = result.match(/\{[\s\S]*\}/);
-    const parsedResult = jsonMatch ? JSON.parse(jsonMatch[0]) : null;
+    let jsonMatch = result.match(/\{[\s\S]*\}/);
+    let parsedResult = jsonMatch ? (() => {
+      try {
+        return JSON.parse(jsonMatch[0]);
+      } catch {
+        return null;
+      }
+    })() : null;
+    const isValidComponent = parsedResult && parsedResult.componentId && parsedResult.props;
+    if (!isValidComponent && executedTools.length > 0) {
+      const toolDataSummary = executedTools.map(
+        (t) => `Tool "${t.name}" was called with params ${JSON.stringify(t.params)} and returned:
+${JSON.stringify(t.result, null, 2).substring(0, 5e3)}`
+      ).join("\n\n");
+      const retryUserPrompt = `Original user request: ${prompt}
+
+The following tool was already called and returned data:
+${toolDataSummary}
+
+Using this data, select the appropriate component and respond with ONLY the JSON component selection object. No explanation, no markdown, just JSON.`;
+      const retryResult = await LLM.text(
+        {
+          sys: prompts.system,
+          user: retryUserPrompt
+        },
+        {
+          model,
+          maxTokens: 4096,
+          temperature: 0.1,
+          apiKey
+        }
+      );
+      jsonMatch = retryResult.match(/\{[\s\S]*\}/);
+      parsedResult = jsonMatch ? (() => {
+        try {
+          return JSON.parse(jsonMatch[0]);
+        } catch {
+          return null;
+        }
+      })() : null;
+    }
     if (!parsedResult) {
       errors.push("Failed to parse LLM response as JSON");
       errors.push(`LLM Response: ${result}`);
+      logger.error(`[DASH_COMP_REQ] Failed to parse JSON from LLM response`);
       return { success: false, errors };
     }
-    logger.debug("[DASH_COMP_REQ] LLM response received");
+    logger.info(`[DASH_COMP_REQ] Parsed component: ${parsedResult.componentName} (${parsedResult.componentId})`);
     logger.file("[DASH_COMP_REQ] LLM response:", JSON.stringify(parsedResult, null, 2));
     if (!parsedResult.componentId || !parsedResult.props) {
       errors.push("Invalid LLM response: missing componentId or props");
       errors.push(`LLM Response: ${result}`);
+      logger.error(`[DASH_COMP_REQ] Invalid structure - missing componentId: ${!parsedResult.componentId}, missing props: ${!parsedResult.props}`);
       userPromptErrorLogger.logError("DASH_COMP_REQ", "Invalid LLM response structure", {
         prompt,
         result: parsedResult,