Revert "fix(platform/blocks): filter sensitive fields and handle suffix collisions in tool dedup"

This reverts commit 60d4dd8ff2.
This commit is contained in:
Zamil Majdy
2026-03-25 22:58:59 +07:00
parent 60d4dd8ff2
commit dd0cca48b4

View File

@@ -258,11 +258,6 @@ def get_pending_tool_calls(conversation_history: list[Any] | None) -> dict[str,
return {call_id: count for call_id, count in pending_calls.items() if count > 0}
_SENSITIVE_FIELD_NAMES = frozenset(
{"credentials", "api_key", "password", "secret", "token", "auth"}
)
def _disambiguate_tool_names(tools: list[dict[str, Any]]) -> None:
"""Ensure all tool names are unique (Anthropic API requires this).
@@ -288,25 +283,13 @@ def _disambiguate_tool_names(tools: list[dict[str, Any]]) -> None:
if name not in duplicates:
continue
# Find next available suffix, skipping names already taken
counters[name] = counters.get(name, 0) + 1
while True:
suffix = f"_{counters[name]}"
candidate = f"{name[:64 - len(suffix)]}{suffix}"
if candidate not in taken:
break
counters[name] += 1
func["name"] = candidate
taken.add(candidate)
suffix = f"_{counters[name]}"
func["name"] = f"{name[:64 - len(suffix)]}{suffix}"
taken.add(func["name"])
# Filter out sensitive fields before building description
safe_defaults = {
k: v for k, v in defaults.items() if k.lower() not in _SENSITIVE_FIELD_NAMES
}
if safe_defaults:
summary = ", ".join(
f"{k}={json.dumps(v)}" for k, v in safe_defaults.items()
)
if defaults:
summary = ", ".join(f"{k}={json.dumps(v)}" for k, v in defaults.items())
func["description"] = (
f"{func.get('description', '')} [Pre-configured: {summary}]"
)