aminghadersohi opened a new pull request, #40690:
URL: https://github.com/apache/superset/pull/40690

   ### SUMMARY
   
   Removes 8 MCP tools that were identified as low-value under our agent-native 
policy: resources that are purely admin/infrastructure CRUD with no user-facing 
workflow value.
   
   **Removed tools:**
   - `action_log`: `list_action_logs`, `get_action_log_info`
   - `css_template`: `list_css_templates`, `get_css_template_info`
   - `plugin`: `list_plugins`, `get_plugin_info`
   - `theme`: `list_themes`, `get_theme_info`
   
   **Policy rationale:** These tools expose low-level admin/infrastructure 
resources. Their primary user is an IT admin acting through the API, not an LLM 
helping an end user with data work.
   
   **Changes:**
   - Deleted 8 tool implementation files and their unit tests
   - Emptied the `tool/__init__.py` for each affected module (directories 
remain)
   - Removed the 4 import blocks from `app.py`
   - Removed the action_log runtime config guard from `_apply_config_guards` 
(no longer needed)
   - Updated the system prompt to remove references to all 8 tools
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   
   N/A — no UI changes.
   
   ### TESTING INSTRUCTIONS
   
   1. Start the MCP service
   2. Verify that `list_action_logs`, `get_action_log_info`, 
`list_css_templates`, `get_css_template_info`, `list_plugins`, 
`get_plugin_info`, `list_themes`, and `get_theme_info` are not listed when 
calling `list_tools`
   3. Run `pytest tests/unit_tests/mcp_service/ -x` to confirm remaining tests 
pass
   
   ### ADDITIONAL INFORMATION
   
   - [ ] Has associated issue:
   - [ ] Required feature flags:
   - [ ] Changes UI
   - [ ] Includes DB Migration
   - [ ] Introduces new feature or API
   - [x] Removes existing feature or API


-- 
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]

Reply via email to