aminghadersohi commented on code in PR #45084:
URL: https://github.com/apache/superset/pull/45084#discussion_r4233312542


##########
superset/mcp_service/server.py:
##########
@@ -827,6 +823,51 @@ def _make_search_tool(self) -> Any:
     return _FixedBM25SearchTransform(**kwargs)
 
 
+def _apply_compact_tool_list_transform(
+    mcp_instance: Any, config: dict[str, Any]
+) -> None:
+    """Bound tool descriptions in the native ``tools/list`` when configured.
+
+    Opt-in via ``MCP_NATIVE_TOOL_LIST_CONFIG["compact"]``. Listed descriptions
+    are bounded with :func:`_truncate_description`. Unlike search results,
+    native listings do not deduct request instructions from the prose budget:

Review Comment:
   Fixed in a82b4e6854c29bb6d6ef964539ac6dd08f40174d.
   Removed the stale search/native contrast; request instructions ship in the 
unchanged input schema and never consume the prose budget.



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