gabotorresruiz commented on code in PR #45084:
URL: https://github.com/apache/superset/pull/45084#discussion_r4233151483
##########
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:
Small nit: this now reads `Unlike search results, native listings do not
deduct request instructions from the prose budget`, but after `796ff50f`
neither mode deducts, so the contrast is stale. Maybe drop the comparison and
just say the instructions ship in the unchanged input schema and are never
budgeted?
--
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]
