aminghadersohi commented on code in PR #33976: URL: https://github.com/apache/superset/pull/33976#discussion_r2208982836
########## superset/mcp_service/README_SCHEMAS.md: ########## @@ -0,0 +1,477 @@ +# Superset MCP Service: Tool Schemas Reference + +This document provides a reference for the input and output parameters of all MCP tools in the Superset MCP service. Each section lists the tool name, its input parameters (with type), and its output schema. + +## Dashboards + +### list_dashboards + +**Inputs:** +- `filters`: `Optional[List[DashboardFilter]]` — List of filter objects +- `columns`: `Optional[List[str]]` — Columns to include in the response Review Comment: The current approach we are going is the default response is limited to a set of columns that are set as default in the tool by us. Perhaps a prompt tool can help figure out to augment these columns if needed based on the user's need? One thing I ran into is that its not possible to get a pydantic schema used by an mcp server to only include the attributes that are non null - at least i am still working on figuring that out. https://github.com/pydantic/pydantic/discussions/5461 -- 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]
