aminghadersohi opened a new pull request, #36032:
URL: https://github.com/apache/superset/pull/36032
### SUMMARY
Renames the MCP tool from `get_superset_instance_info` to
`get_instance_info` to reduce verbosity while maintaining clarity.
**Rationale**: The `superset` prefix in the tool name is redundant since all
MCP tools operate within the Superset context. Shorter tool names improve the
developer experience and reduce token usage when AI assistants interact with
the MCP service.
**Changes**:
- Renamed tool file: `get_superset_instance_info.py` → `get_instance_info.py`
- Updated function name: `get_superset_instance_info` → `get_instance_info`
- Updated all imports and references across:
- `superset/mcp_service/app.py` (tool imports and documentation)
- `superset/mcp_service/system/tool/__init__.py` (module exports)
- `superset/mcp_service/system/schemas.py` (docstring)
- `superset/mcp_service/system/resources/instance_metadata.py` (import and
usage tip)
- `superset/mcp_service/system/prompts/quickstart.py` (documentation)
**Design Decision**: Only the tool name changes - the schema class name
`GetSupersetInstanceInfoRequest` remains unchanged to avoid breaking changes to
existing integrations that might reference the schema type.
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N/A - This is a backend refactor with no UI changes.
### TESTING INSTRUCTIONS
1. **Start the MCP service**:
```bash
superset mcp run --port 5008
```
2. **Test tool listing** (verify new name appears):
```bash
# Using MCP inspector or Claude Desktop
# Verify tool name shows as "get_instance_info"
```
3. **Test tool execution** (verify functionality unchanged):
```bash
# Call get_instance_info via MCP client
# Verify it returns instance statistics as before
```
4. **Verify documentation updates**:
- Check `DEFAULT_INSTRUCTIONS` in `app.py` mentions `get_instance_info`
- Check quickstart prompt references `get_instance_info`
- Check instance metadata resource tips mention `get_instance_info`
### ADDITIONAL INFORMATION
- [ ] Has associated issue: N/A
- [ ] Required feature flags: None
- [ ] Changes UI: No
- [ ] Includes DB Migration: No
- [ ] Introduces new feature or API: No
- [ ] Removes existing feature or API: No (renamed only)
--
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]
