remove namespaced name

Signed-off-by: Prashant <prashant.rakheja@sap.com>

Author: prashantrakheja

src/sap_cloud_sdk/agentgateway/__init__.py

@@ -15,7 +15,7 @@
     # Discover tools
     tools = await agw_client.list_mcp_tools()
     for tool in tools:
-        print(f"{tool.namespaced_name}: {tool.description}")
+        print(f"{tool.name}: {tool.description}")
 
     # Invoke a tool
     # Note: kwargs like "order_id" are tool-specific input parameters.

src/sap_cloud_sdk/agentgateway/_models.py

@@ -1,7 +1,5 @@
 """Data models for Agent Gateway MCP tools."""
 
-import hashlib
-import re
 from dataclasses import dataclass
 from typing import Any
 
@@ -30,37 +28,6 @@ class MCPTool:
     url: str
     fragment_name: str | None = None
 
-    @property
-    def namespaced_name(self) -> str:
-        """Unique tool name safe for LLM APIs: sanitized, namespaced, max 64 chars.
-
-        LLM tool-calling APIs (Anthropic, OpenAI) require names matching
-        ^[a-zA-Z0-9-_]+$ with a max length of 64 characters.
-
-        This property combines server_name and tool name to avoid collisions
-        across multiple MCP servers, then sanitizes and enforces the limit.
-
-        Examples:
-            Short names pass through unchanged:
-                "myserver__list_orders" (21 chars) → "myserver__list_orders"
-
-            Invalid chars are replaced with underscores:
-                "my.server:v1__get/data" → "my_server_v1__get_data"
-
-            Names over 64 chars are truncated with a hash suffix for uniqueness:
-                "sales_order_mcp_demo__get_supplier_operational_eval_scores_by_region" (70 chars)
-                → "sales_order_mcp_demo__get_supplier_operational_eval_s_a3b7c9d1" (64 chars)
-
-            Two servers with the same tool name remain distinct:
-                "server_a__get_metadata" vs "server_b__get_metadata"
-        """
-        raw = f"{self.server_name}__{self.name}"
-        sanitized = re.sub(r"[^a-zA-Z0-9\-_]", "_", raw)
-        if len(sanitized) <= 64:
-            return sanitized
-        suffix = hashlib.sha256(sanitized.encode()).hexdigest()[:8]
-        return f"{sanitized[:55]}_{suffix}"
-
 
 @dataclass
 class IntegrationDependency:

src/sap_cloud_sdk/agentgateway/agw_client.py

@@ -142,7 +142,7 @@ async def list_mcp_tools(
             ```python
             tools = await agw_client.list_mcp_tools()
             for tool in tools:
-                print(f"{tool.namespaced_name}: {tool.description}")
+                print(f"{tool.name}: {tool.description}")
             ```
         """
         try:

src/sap_cloud_sdk/agentgateway/converters.py

@@ -1,7 +1,8 @@
-"""Converters for MCPTool to framework-specific tools.
+"""Reference converter for MCPTool to LangChain StructuredTool.
 
-This module provides converters to transform MCPTool objects into
-tools compatible with popular agent frameworks.
+This module provides a sample converter as a starting point.
+End-users have full flexibility to write their own converters with
+custom tool naming, argument schemas, or framework integrations.
 """
 
 from __future__ import annotations
@@ -23,6 +24,9 @@ def mcp_tool_to_langchain(
 ) -> StructuredTool:
     """Convert MCPTool to LangChain StructuredTool.
 
+    This is a reference implementation. End-users can write their own
+    converter with custom naming, argument handling, or framework bindings.
+
     Args:
         mcp_tool: MCPTool object from list_mcp_tools().
         call_tool: Callable to invoke the MCP tool (e.g., agw_client.call_mcp_tool).
@@ -72,12 +76,12 @@ async def run(**kwargs) -> str:
     properties = mcp_tool.input_schema.get("properties", {})
     fields: dict[str, Any] = {k: (str, ...) for k in properties}
     args_schema = (
-        create_model(f"{mcp_tool.namespaced_name}_args", **fields) if fields else None
+        create_model(f"{mcp_tool.name}_args", **fields) if fields else None
     )
 
     return StructuredTool.from_function(
         coroutine=run,
-        name=mcp_tool.namespaced_name,
+        name=mcp_tool.name,
         description=mcp_tool.description,
         args_schema=args_schema,
     )

src/sap_cloud_sdk/agentgateway/user-guide.md

@@ -27,7 +27,7 @@ agw_client = create_client()
 tools = await agw_client.list_mcp_tools()
 
 for tool in tools:
-    print(f"{tool.namespaced_name}: {tool.description}")
+    print(f"{tool.name}: {tool.description}")
 
 # Invoke a tool with user principal propagation
 result = await agw_client.call_mcp_tool(

tests/agentgateway/unit/test_converters.py

@@ -27,7 +27,7 @@ def test_creates_structured_tool(self):
 
         result = mcp_tool_to_langchain(tool, call_tool, get_user_token)
 
-        assert result.name == "s4hana__create_order"
+        assert result.name == "create_order"
         assert result.description == "Create a purchase order"
         assert result.coroutine is not None
 
@@ -75,7 +75,7 @@ def test_handles_empty_input_schema(self):
 
         result = mcp_tool_to_langchain(tool, call_tool, lambda: "token")
 
-        assert result.name == "server__simple_tool"
+        assert result.name == "simple_tool"
         assert result.args_schema is not None
 
     def test_handles_input_schema_without_properties(self):

tests/agentgateway/unit/test_models.py

@@ -6,93 +6,6 @@
 class TestMCPTool:
     """Tests for MCPTool dataclass."""
 
-    def test_namespaced_name(self):
-        """Test namespaced_name property."""
-        tool = MCPTool(
-            name="create_order",
-            server_name="s4hana_procurement",
-            description="Create purchase order",
-            input_schema={"type": "object"},
-            url="https://example.com/mcp",
-        )
-
-        assert tool.namespaced_name == "s4hana_procurement__create_order"
-
-    def test_namespaced_name_with_special_chars(self):
-        """Test namespaced_name sanitizes invalid characters to underscores."""
-        tool = MCPTool(
-            name="get-item",
-            server_name="my-server",
-            description="Get item",
-            input_schema={},
-            url="https://example.com/mcp",
-        )
-
-        assert tool.namespaced_name == "my-server__get-item"
-
-    def test_namespaced_name_sanitizes_dots_and_colons(self):
-        """Test that dots, colons, slashes are replaced with underscores."""
-        tool = MCPTool(
-            name="get/data",
-            server_name="my.server:v1",
-            description="Get data",
-            input_schema={},
-            url="https://example.com/mcp",
-        )
-
-        assert tool.namespaced_name == "my_server_v1__get_data"
-
-    def test_namespaced_name_truncates_long_names(self):
-        """Test that names over 64 chars are truncated to 55 + hash suffix."""
-        tool = MCPTool(
-            name="get_supplier_operational_eval_scores_by_region",
-            server_name="sales_order_mcp_demo",
-            description="Long name tool",
-            input_schema={},
-            url="https://example.com/mcp",
-        )
-
-        result = tool.namespaced_name
-        assert len(result) == 64
-        # First 55 chars are preserved from the sanitized name
-        assert result[:55] == "sales_order_mcp_demo__get_supplier_operational_eval_sco"
-        # Followed by underscore and 8-char hash
-        assert result[55] == "_"
-        assert len(result[56:]) == 8
-
-    def test_namespaced_name_short_names_unchanged(self):
-        """Test that short valid names pass through without modification."""
-        tool = MCPTool(
-            name="list_orders",
-            server_name="myserver",
-            description="List orders",
-            input_schema={},
-            url="https://example.com/mcp",
-        )
-
-        assert tool.namespaced_name == "myserver__list_orders"
-
-    def test_namespaced_name_uniqueness_on_truncation(self):
-        """Test that two different long names produce different truncated results."""
-        tool_a = MCPTool(
-            name="get_supplier_operational_eval_scores_by_region_east",
-            server_name="sales_order_mcp_demo",
-            description="Tool A",
-            input_schema={},
-            url="https://example.com/mcp",
-        )
-        tool_b = MCPTool(
-            name="get_supplier_operational_eval_scores_by_region_west",
-            server_name="sales_order_mcp_demo",
-            description="Tool B",
-            input_schema={},
-            url="https://example.com/mcp",
-        )
-
-        assert tool_a.namespaced_name != tool_b.namespaced_name
-        assert len(tool_a.namespaced_name) == 64
-        assert len(tool_b.namespaced_name) == 64
-
     def test_create_tool_with_all_fields(self):
         """Test MCPTool creation with all fields."""
         tool = MCPTool(