chore: remove mcp and exposed param decorators (#80)

Author: rolanbadrislamov

src/sap_cloud_sdk/agent_decorators/__init__.py

@@ -2,15 +2,13 @@
 
 Decorator-based configuration-as-code for SAP AI agents. Developers
 annotate functions with decorators to expose configuration fields
-(prompts, models, settings, MCP servers) to a low-code UI.
+(prompts, models, settings) to a low-code UI.
 
 Usage:
     from sap_cloud_sdk.agent_decorators import (
         prompt_section,
         agent_config,
         agent_model,
-        mcp_server,
-        exposed_param,
     )
 
     @prompt_section(
@@ -25,8 +23,6 @@ def system_prompt() -> str:
 from sap_cloud_sdk.agent_decorators.decorators import (
     agent_config,
     agent_model,
-    exposed_param,
-    mcp_server,
     prompt_section,
 )
 from sap_cloud_sdk.agent_decorators.exceptions import (
@@ -38,8 +34,6 @@ def system_prompt() -> str:
     "prompt_section",
     "agent_config",
     "agent_model",
-    "mcp_server",
-    "exposed_param",
     # Exceptions
     "AgentDecoratorError",
 ]

src/sap_cloud_sdk/agent_decorators/decorators.py

@@ -136,75 +136,3 @@ def decorator(fn: Callable) -> Callable:
         return fn
 
     return decorator
-
-
-def mcp_server(
-    key: str,
-    label: str,
-    description: str,
-) -> Callable:
-    """Expose an MCP server configuration.
-
-    Args:
-        key: Unique identifier (e.g. ``"mcp.filesystem"``).
-        label: Human-readable label.
-        description: Help text.
-
-    Returns:
-        A decorator that validates the key and returns the function unchanged.
-
-    Raises:
-        AgentDecoratorError: If the key is empty or whitespace-only.
-
-    Example::
-
-        @mcp_server(
-            key="mcp.filesystem",
-            label="Filesystem Server",
-            description="MCP server for file operations",
-        )
-        def get_fs_server() -> dict:
-            return {"command": "npx", "args": ["-y", "@anthropic/mcp-fs"]}
-    """
-    _validate_key(key)
-
-    def decorator(fn: Callable) -> Callable:
-        return fn
-
-    return decorator
-
-
-def exposed_param(
-    key: str,
-    label: str,
-    description: str,
-) -> Callable:
-    """Expose a runtime parameter for editing.
-
-    Args:
-        key: Unique identifier (e.g. ``"params.max_retries"``).
-        label: Human-readable label.
-        description: Help text.
-
-    Returns:
-        A decorator that validates the key and returns the function unchanged.
-
-    Raises:
-        AgentDecoratorError: If the key is empty or whitespace-only.
-
-    Example::
-
-        @exposed_param(
-            key="params.max_retries",
-            label="Max Retries",
-            description="Number of retry attempts for failed requests",
-        )
-        def get_max_retries() -> int:
-            return 3
-    """
-    _validate_key(key)
-
-    def decorator(fn: Callable) -> Callable:
-        return fn
-
-    return decorator

src/sap_cloud_sdk/agent_decorators/user-guide.md

@@ -1,6 +1,6 @@
 # Agent Decorators User Guide
 
-This module provides a decorator-based configuration-as-code system for SAP AI agents. Developers annotate functions with decorators to expose configuration fields — prompts, model selections, agent settings, and MCP servers — to a low-code UI. External tooling discovers these markers in source text and extracts their arguments and return values.
+This module provides a decorator-based configuration-as-code system for SAP AI agents. Developers annotate functions with decorators to expose configuration fields — prompts, model selections, and agent settings — to a low-code UI. External tooling discovers these markers in source text and extracts their arguments and return values.
 
 ## Installation
 
@@ -13,8 +13,6 @@ from sap_cloud_sdk.agent_decorators import (
     prompt_section,
     agent_config,
     agent_model,
-    mcp_server,
-    exposed_param,
 )
 ```
 
@@ -89,41 +87,6 @@ def default_model() -> str:
     return "gpt-4"
 ```
 
-### @mcp_server
-
-Expose an MCP (Model Context Protocol) server configuration.
-
-```python
-from sap_cloud_sdk.agent_decorators import mcp_server
-
-@mcp_server(
-    key="mcp.filesystem",
-    label="Filesystem Server",
-    description="MCP server for filesystem access",
-)
-def filesystem_server() -> dict:
-    return {
-        "command": "npx",
-        "args": ["-y", "@modelcontextprotocol/server-filesystem", "/data"],
-    }
-```
-
-### @exposed_param
-
-Expose a runtime parameter for editing.
-
-```python
-from sap_cloud_sdk.agent_decorators import exposed_param
-
-@exposed_param(
-    key="params.max_retries",
-    label="Max Retries",
-    description="Number of retry attempts for failed requests",
-)
-def max_retries() -> int:
-    return 3
-```
-
 ## Error Handling
 
 ```python

tests/agent_decorators/unit/test_decorators.py

@@ -5,8 +5,6 @@
 from sap_cloud_sdk.agent_decorators.decorators import (
     agent_config,
     agent_model,
-    exposed_param,
-    mcp_server,
     prompt_section,
 )
 from sap_cloud_sdk.agent_decorators.exceptions import AgentDecoratorError
@@ -71,37 +69,6 @@ def model():
 
         assert model() == "claude-3"
 
-
-class TestMcpServer:
-    """Tests for @mcp_server decorator."""
-
-    def test_function_still_callable(self):
-        @mcp_server(
-            key="mcp.filesystem",
-            label="Filesystem Server",
-            description="MCP server for filesystem access",
-        )
-        def fs_server():
-            return {"command": "python", "args": ["-m", "server"]}
-
-        assert fs_server() == {"command": "python", "args": ["-m", "server"]}
-
-
-class TestExposedParam:
-    """Tests for @exposed_param decorator."""
-
-    def test_function_still_callable(self):
-        @exposed_param(
-            key="params.max_retries",
-            label="Max Retries",
-            description="Number of retry attempts for failed requests",
-        )
-        def max_retries():
-            return 3
-
-        assert max_retries() == 3
-
-
 class TestKeyValidation:
     """Tests for decorator key validation."""
 
@@ -122,7 +89,7 @@ def bad():
     def test_none_key_raises_error(self):
         with pytest.raises(AgentDecoratorError, match="non-empty string"):
 
-            @mcp_server(key=None, label="L", description="D")  # ty: ignore[invalid-argument-type]
+            @agent_model(key=None, label="L")  # ty: ignore[invalid-argument-type]
             def bad():
                 return {}
 
@@ -138,8 +105,6 @@ def test_all_decorators_validate_key(self):
             lambda fn: prompt_section(key="", label="L", description="D")(fn),
             lambda fn: agent_config(key="", label="L", description="D")(fn),
             lambda fn: agent_model(key="", label="L")(fn),
-            lambda fn: mcp_server(key="", label="L", description="D")(fn),
-            lambda fn: exposed_param(key="", label="L", description="D")(fn),
         ]
 
         for dec in decorators:

tests/core/unit/auditlog_ng/unit/test_client.py

@@ -34,7 +34,7 @@ def _make_config(**overrides: Unpack[ConfigKwargs]) -> AuditLogNGConfig:
         "namespace": "namespace-123",
         "insecure": True,
     }
-    defaults.update(overrides)  # ty: ignore[invalid-argument-type]
+    defaults.update(overrides)
     return AuditLogNGConfig(**defaults)