dispatch_agents 0.14.2__tar.gz → 0.14.3__tar.gz

{dispatch_agents-0.14.2 → dispatch_agents-0.14.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: dispatch_agents
-Version: 0.14.2
+Version: 0.14.3
 Summary: Dispatch Agents SDK
 License-File: LICENSE
 License-File: LICENSE-3rdparty.csv

dispatch_agents-0.14.3/RELEASE_NOTES.md

@@ -0,0 +1,2 @@
+## Bug Fixes
+- MCP servers managed by Dispatch are now correctly skipped during local development, preventing unintended interactions with remotely managed server configurations.

{dispatch_agents-0.14.2 → dispatch_agents-0.14.3}/dispatch_agents/mcp.py

@@ -26,6 +26,7 @@ from dispatch_agents._internal.dispatch import (
     get_current_trace_id,
     get_invocation_id_for_trace,
 )
+from dispatch_agents.logging import get_logger
 
 if TYPE_CHECKING:
     from dispatch_agents.models import (
@@ -41,6 +42,8 @@ if TYPE_CHECKING:
 
 __all__ = ["MCPClient", "get_mcp_client", "get_mcp_servers_config"]
 
+_logger = get_logger(__name__)
+
 MCP_CONFIG_PATH = os.environ.get("MCP_CONFIG_PATH", ".mcp.json")
 
 
@@ -282,14 +285,29 @@ def _get_dispatch_mcp_servers() -> dict[str, Any]:
 
     Constructs URLs and auth headers from runtime environment variables
     (``DISPATCH_MCP_GATEWAY_URL``, ``DISPATCH_NAMESPACE``, ``DISPATCH_API_KEY``).
-    No file I/O.
+    No file I/O. Returns ``{}`` in local dev, where the gateway is unavailable.
     """
+    from dispatch_agents._internal.transport import is_local_dev_mode
     from dispatch_agents.config import _load_runtime_config
 
     cfg = _load_runtime_config()
     if not cfg.mcp_servers:
         return {}
 
+    # The MCP gateway is a deployed/production concern; it is not available under
+    # `dispatch agent dev` (the local router does not proxy /api/v1/mcp/...). Skip
+    # dispatch-managed servers in local dev rather than crashing on the missing
+    # gateway URL. Authors who want to exercise MCP locally supply a .mcp.json.
+    if is_local_dev_mode():
+        names = ", ".join(s.server for s in cfg.mcp_servers)
+        _logger.warning(
+            "Skipping dispatch-managed MCP server(s) in local dev mode: %s. "
+            "The MCP gateway is not available under `dispatch agent dev`. "
+            "To test MCP locally, define the server(s) in a .mcp.json file.",
+            names,
+        )
+        return {}
+
     gateway = _require_env("DISPATCH_MCP_GATEWAY_URL")
     namespace = os.environ.get("DISPATCH_NAMESPACE") or cfg.namespace
     if not namespace:
@@ -346,13 +364,25 @@ def _get_merged_mcp_servers() -> dict[str, Any]:
 
 def _get_server_config(server_name: str) -> dict[str, Any]:
     """Get configuration for a specific MCP server."""
+    from dispatch_agents._internal.transport import is_local_dev_mode
+
     servers = _get_merged_mcp_servers()
     if server_name not in servers:
         available = list(servers.keys())
-        raise ValueError(
+        message = (
             f"MCP server '{server_name}' not found in config. "
             f"Available servers: {available}"
         )
+        # Dispatch-managed servers are skipped under `dispatch agent dev` (the
+        # local router does not proxy the MCP gateway), so a server declared in
+        # dispatch.yaml will be absent here. Point the author at the local escape
+        # hatch rather than leaving them with a bare "not found".
+        if is_local_dev_mode():
+            message += (
+                " Dispatch-managed MCP servers are skipped under `dispatch agent "
+                "dev`; define the server in a .mcp.json file to use it locally."
+            )
+        raise ValueError(message)
     return servers[server_name]
 
 
@@ -365,6 +395,10 @@ def get_mcp_servers_config() -> dict[str, McpHttpServerConfig]:
     is merged additively; duplicate names with dispatch-managed servers are
     rejected.
 
+    In local dev (``dispatch agent dev``) the MCP gateway is unavailable, so
+    dispatch-managed servers are skipped and only ``.mcp.json`` servers are
+    returned.
+
     Returns:
         Mapping of server name to HTTP URL and headers.
 

{dispatch_agents-0.14.2 → dispatch_agents-0.14.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "dispatch_agents"
-version = "0.14.2"
+version = "0.14.3"
 description = "Dispatch Agents SDK"
 requires-python = ">=3.11"
 dependencies = [

{dispatch_agents-0.14.2 → dispatch_agents-0.14.3}/tests/test_mcp.py

@@ -5,6 +5,7 @@ config builders.
 """
 
 import json
+import logging
 from unittest.mock import AsyncMock, MagicMock, patch
 
 import pytest
@@ -272,6 +273,7 @@ class TestGetDispatchMcpServers:
 
     def test_missing_required_env_raises(self, monkeypatch):
         monkeypatch.delenv("DISPATCH_MCP_GATEWAY_URL", raising=False)
+        monkeypatch.delenv("DISPATCH_LOCAL_DEV", raising=False)
         monkeypatch.setenv("DISPATCH_NAMESPACE", "ns")
         monkeypatch.setenv("DISPATCH_API_KEY", "key")
 
@@ -285,6 +287,33 @@ class TestGetDispatchMcpServers:
         ):
             _get_dispatch_mcp_servers()
 
+    def test_local_dev_skips_dispatch_servers(self, monkeypatch, caplog):
+        # In local dev the MCP gateway is unavailable, so dispatch-managed
+        # servers are skipped (with a warning) instead of crashing.
+        monkeypatch.setenv("DISPATCH_LOCAL_DEV", "true")
+        monkeypatch.delenv("DISPATCH_MCP_GATEWAY_URL", raising=False)
+
+        config = DispatchConfig(
+            mcp_servers=[MCPServerConfig(server="my-server")],
+        )
+
+        # The SDK logger sets propagate=False, so caplog's root handler never
+        # sees the record. Attach caplog's handler to the SDK logger directly.
+        sdk_logger = logging.getLogger("dispatch_agents")
+        sdk_logger.addHandler(caplog.handler)
+        try:
+            with patch(
+                "dispatch_agents.config._load_runtime_config", return_value=config
+            ):
+                with caplog.at_level("WARNING", logger="dispatch_agents"):
+                    result = _get_dispatch_mcp_servers()
+        finally:
+            sdk_logger.removeHandler(caplog.handler)
+
+        assert result == {}
+        assert "local dev mode" in caplog.text
+        assert "my-server" in caplog.text
+
 
 class TestGetMergedMcpServers:
     def test_merges_dispatch_and_user_servers(self, tmp_path, monkeypatch):
@@ -326,6 +355,28 @@ class TestGetMergedMcpServers:
 
         assert list(result) == ["dispatch-server"]
 
+    def test_local_dev_uses_only_user_servers(self, tmp_path, monkeypatch):
+        # In local dev, dispatch-managed servers are skipped, so the merged set
+        # comes entirely from the user's .mcp.json -- no gateway URL required.
+        monkeypatch.setenv("DISPATCH_LOCAL_DEV", "true")
+        monkeypatch.delenv("DISPATCH_MCP_GATEWAY_URL", raising=False)
+
+        config = DispatchConfig(
+            mcp_servers=[MCPServerConfig(server="dispatch-server")],
+        )
+
+        user_config = {"mcpServers": {"user-server": {"url": "http://user:3000"}}}
+        config_file = tmp_path / ".mcp.json"
+        config_file.write_text(json.dumps(user_config))
+
+        with (
+            patch("dispatch_agents.config._load_runtime_config", return_value=config),
+            patch("dispatch_agents.mcp.MCP_CONFIG_PATH", str(config_file)),
+        ):
+            result = _get_merged_mcp_servers()
+
+        assert list(result) == ["user-server"]
+
     def test_duplicate_user_server_raises(self, tmp_path, monkeypatch):
         monkeypatch.setenv("DISPATCH_MCP_GATEWAY_URL", "https://gw.example.com")
         monkeypatch.setenv("DISPATCH_NAMESPACE", "ns")
@@ -350,13 +401,28 @@ class TestGetMergedMcpServers:
 
 
 class TestGetServerConfig:
-    def test_not_found_raises(self, tmp_path):
+    def test_not_found_raises(self, tmp_path, monkeypatch):
+        monkeypatch.delenv("DISPATCH_LOCAL_DEV", raising=False)
         config = {"mcpServers": {"other-server": {"url": "http://localhost:3000"}}}
         config_file = tmp_path / ".mcp.json"
         config_file.write_text(json.dumps(config))
 
         with patch("dispatch_agents.mcp.MCP_CONFIG_PATH", str(config_file)):
-            with pytest.raises(ValueError, match="not found in config"):
+            with pytest.raises(ValueError, match="not found in config") as exc_info:
+                _get_server_config("missing-server")
+        # Outside local dev, no dev-mode hint is appended.
+        assert "dispatch agent dev" not in str(exc_info.value)
+
+    def test_not_found_in_local_dev_hints_at_mcp_json(self, tmp_path, monkeypatch):
+        # In local dev a dispatch-managed server is skipped, so it is absent here.
+        # The error should point the author at the .mcp.json escape hatch instead
+        # of leaving them with a bare "not found".
+        monkeypatch.setenv("DISPATCH_LOCAL_DEV", "true")
+        config_file = tmp_path / ".mcp.json"
+        config_file.write_text(json.dumps({"mcpServers": {}}))
+
+        with patch("dispatch_agents.mcp.MCP_CONFIG_PATH", str(config_file)):
+            with pytest.raises(ValueError, match="dispatch agent dev"):
                 _get_server_config("missing-server")
 
     def test_returns_server_config(self, tmp_path):

{dispatch_agents-0.14.2 → dispatch_agents-0.14.3}/uv.lock

@@ -567,7 +567,7 @@ wheels = [
 
 [[package]]
 name = "dispatch-agents"
-version = "0.14.2"
+version = "0.14.3"
 source = { editable = "." }
 dependencies = [
     { name = "aiohttp" },

dispatch_agents-0.14.2/RELEASE_NOTES.md

@@ -1,2 +0,0 @@
-## Features
-- The logging configuration module is now publicly accessible as `dispatch_agents.logging`, allowing users to directly import and use logging utilities from the top-level package.