sadpandajoe opened a new issue, #45510: URL: https://github.com/apache/superset/issues/45510
## Problem `tests/unit_tests/mcp_service/test_native_tool_surface.py::test_native_inventory_report_matches_registered_tools` fails on master, turning the required `unit-tests-required` check red. First failing run: https://github.com/apache/superset/actions/runs/39367174764 `AssertionError: Native tool definitions grew beyond the recorded inventory`: output schemas of `generate_chart`, `update_chart` and `get_chart_preview` measure 496 bytes over the recorded values (e.g. 20358 vs 19862). ## Root cause #44558 added `PNGPreview` (`superset/mcp_service/chart/schemas.py:6843`), a sixth `ChartPreviewContent` union member (`schemas.py:6915-6922`) and new `format`/`width`/`height` fields (`schemas.py:6807-6825`). That exceeds the budget headroom of 2% or 200 bytes (`test_native_tool_surface.py:114-116`), and `native_tool_inventory.json` wasn't regenerated. The PR's last CI ran before the inventory test landed (#45081/#45084). ## Steps to reproduce On master @ 1e18d01d76: `pytest tests/unit_tests/mcp_service/test_native_tool_surface.py::test_native_inventory_report_matches_registered_tools -q` (Found via automated investigation — [Agor](https://agor.live/).) -- 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]
