This is an automated email from the ASF dual-hosted git repository.
aminghadersohi pushed a change to branch aminghadersohi/mcp-semantic-layer-tools
in repository https://gitbox.apache.org/repos/asf/superset.git
from a7dcfd94f37 fix(mcp): add explicit type annotations flagged by
automated review
add f4bcbf31f3c fix(mcp): address 7 major findings from automated PR review
No new revisions were added by this update.
Summary of changes:
superset/daos/semantic_layer.py | 25 +++--
superset/mcp_service/chart/schemas.py | 16 ++-
.../semantic_layer/tool/get_compatible_metrics.py | 6 +-
.../mcp_service/semantic_layer/tool/get_table.py | 11 +-
.../semantic_layer/tool/list_metrics.py | 73 +++++++++---
superset/mcp_service/utils/response_utils.py | 13 ++-
.../tool/test_get_compatible_metrics.py | 26 ++++-
.../semantic_layer/tool/test_get_table.py | 34 ++++++
.../semantic_layer/tool/test_list_metrics.py | 123 ++++++++++++++++++++-
.../mcp_service/utils/test_response_utils.py | 109 ++++++++++++++++++
tests/unit_tests/semantic_layers/dao_test.py | 56 ++++++++++
11 files changed, 456 insertions(+), 36 deletions(-)
create mode 100644 tests/unit_tests/mcp_service/utils/test_response_utils.py