codeant-ai-for-open-source[bot] commented on PR #38409:
URL: https://github.com/apache/superset/pull/38409#issuecomment-4030144331

   ## **Sequence Diagram**
   
   This PR updates the add_chart_to_existing_dashboard MCP tool so that the 
optional target_tab parameter is honored by resolving the correct tab (by ID or 
flexible name match from ROOT_ID and GRID_ID) before inserting the chart into 
the dashboard layout.
   
   ```mermaid
   sequenceDiagram
       participant Caller
       participant MCPTool as MCP tool
       participant Dashboard as Dashboard backend
   
       Caller->>MCPTool: add_chart_to_existing_dashboard with dashboard id, 
chart id, target_tab
       MCPTool->>Dashboard: Load dashboard layout and chart
       Dashboard-->>MCPTool: Return layout and chart
       MCPTool->>MCPTool: Resolve target tab from layout using target_tab (ID 
or normalized name, fallback to first tab)
       MCPTool->>Dashboard: Update layout to add chart under resolved tab
       Dashboard-->>MCPTool: Save and return updated dashboard
       MCPTool-->>Caller: Return success with new chart position
   ```
   
   ---
   *Generated by [CodeAnt AI](https://codeant.ai)*
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to