aminghadersohi commented on PR #45818: URL: https://github.com/apache/superset/pull/45818#issuecomment-6474883062
Acceptance check at 0f5a55cbbc02419a9870339f420d9bf1a7b96235: ship. - Ran the new tests with test_tool_search_transform.py and test_mcp_e2e_smoke.py on Python 3.11 with requirements/development.txt (fastmcp 4.1.2, mcp 2.3.0): 149 passed. - To confirm the guard can fail, I made the result middleware always drop structuredContent while the schema stays advertised: 16 of the 18 new tests fail; the 2 that pass are the structured-output-off cases, as expected. - No findings. This PR changes tests only, so it guards the contract rather than changing behaviour. -- 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]
