From dd0cca48b4a6ac363848f0ae4ba0a85b2e7b053a Mon Sep 17 00:00:00 2001 From: Zamil Majdy Date: Wed, 25 Mar 2026 22:58:59 +0700 Subject: [PATCH] Revert "fix(platform/blocks): filter sensitive fields and handle suffix collisions in tool dedup" This reverts commit 60d4dd8ff254c0261d7ede74186d1cd26802a74a. --- .../backend/backend/blocks/orchestrator.py | 27 ++++--------------- 1 file changed, 5 insertions(+), 22 deletions(-) diff --git a/autogpt_platform/backend/backend/blocks/orchestrator.py b/autogpt_platform/backend/backend/blocks/orchestrator.py index 27d940f348..ee89cd20d8 100644 --- a/autogpt_platform/backend/backend/blocks/orchestrator.py +++ b/autogpt_platform/backend/backend/blocks/orchestrator.py @@ -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}]" )