agent-dispatch 0.2.1__tar.gz → 0.2.2__tar.gz

{agent_dispatch-0.2.1 → agent_dispatch-0.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agent-dispatch
-Version: 0.2.1
+Version: 0.2.2
 Summary: MCP server that lets Claude Code agents delegate tasks to agents in other project directories
 Project-URL: Homepage, https://github.com/ginkida/agent-dispatch
 Project-URL: Repository, https://github.com/ginkida/agent-dispatch

{agent_dispatch-0.2.1 → agent_dispatch-0.2.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "agent-dispatch"
-version = "0.2.1"
+version = "0.2.2"
 description = "MCP server that lets Claude Code agents delegate tasks to agents in other project directories"
 readme = "README.md"
 license = "MIT"

{agent_dispatch-0.2.1 → agent_dispatch-0.2.2}/src/agent_dispatch/__init__.py

@@ -1,3 +1,3 @@
 """agent-dispatch: Delegate tasks between Claude Code agents across projects."""
 
-__version__ = "0.2.1"
+__version__ = "0.2.2"

{agent_dispatch-0.2.1 → agent_dispatch-0.2.2}/src/agent_dispatch/cli.py

@@ -202,10 +202,12 @@ def list_agents() -> None:
             click.echo(f"    config: {', '.join(extras)}")
         if agent.permission_mode:
             click.echo(f"    permission_mode: {agent.permission_mode}")
-        if agent.allowed_tools:
-            click.echo(f"    allowed_tools: {', '.join(agent.allowed_tools)}")
-        if agent.disallowed_tools:
-            click.echo(f"    disallowed_tools: {', '.join(agent.disallowed_tools)}")
+        if agent.allowed_tools is not None:
+            rendered = ", ".join(agent.allowed_tools) if agent.allowed_tools else "(none)"
+            click.echo(f"    allowed_tools: {rendered}")
+        if agent.disallowed_tools is not None:
+            rendered = ", ".join(agent.disallowed_tools) if agent.disallowed_tools else "(none)"
+            click.echo(f"    disallowed_tools: {rendered}")
         click.echo()
 
 

{agent_dispatch-0.2.1 → agent_dispatch-0.2.2}/src/agent_dispatch/server.py

@@ -113,9 +113,10 @@ async def list_agents(ctx: Context | None = None) -> str:
         }
         if agent.permission_mode:
             entry["permission_mode"] = agent.permission_mode
-        if agent.allowed_tools:
+        # Include when explicitly set (even []) to distinguish from inheriting defaults
+        if agent.allowed_tools is not None:
             entry["allowed_tools"] = agent.allowed_tools
-        if agent.disallowed_tools:
+        if agent.disallowed_tools is not None:
             entry["disallowed_tools"] = agent.disallowed_tools
         agents.append(entry)
     if ctx:

{agent_dispatch-0.2.1 → agent_dispatch-0.2.2}/tests/test_cli.py

@@ -145,6 +145,40 @@ class TestList:
         assert "permission_mode: bypassPermissions" in result.output
         assert "allowed_tools: Bash, Read" in result.output
 
+    def test_list_distinguishes_empty_from_inherit(self, tmp_path: Path):
+        """Explicit allowed_tools=[] should show '(none)' to distinguish from inherit (None)."""
+        agent_dir = tmp_path / "proj"
+        agent_dir.mkdir()
+        # Write config directly with explicit empty list
+        import yaml
+        from agent_dispatch.config import config_path
+        cfg_path = config_path()
+        cfg_path.parent.mkdir(parents=True, exist_ok=True)
+        yaml.safe_dump({
+            "agents": {
+                "proj": {
+                    "directory": str(agent_dir),
+                    "description": "Test",
+                    "allowed_tools": [],
+                    "disallowed_tools": [],
+                },
+            },
+        }, cfg_path.open("w"))
+
+        result = runner.invoke(cli, ["list"])
+        assert result.exit_code == 0
+        assert "allowed_tools: (none)" in result.output
+        assert "disallowed_tools: (none)" in result.output
+
+    def test_list_hides_tools_when_none(self, tmp_path: Path):
+        """allowed_tools=None (the default, meaning 'inherit') should not appear in list."""
+        agent_dir = tmp_path / "proj"
+        agent_dir.mkdir()
+        runner.invoke(cli, ["add", "proj", str(agent_dir), "-d", "Test"])
+        result = runner.invoke(cli, ["list"])
+        assert "allowed_tools" not in result.output
+        assert "disallowed_tools" not in result.output
+
 
 class TestUpdate:
     def test_update_permission_mode(self, tmp_path: Path):

{agent_dispatch-0.2.1 → agent_dispatch-0.2.2}/tests/test_server.py

@@ -593,6 +593,7 @@ class TestListAgentsPermissions:
 
     @pytest.mark.asyncio
     async def test_list_omits_empty_permissions(self, tmp_path: Path):
+        """allowed_tools=None (inherit) should NOT appear in response."""
         d = tmp_path / "proj"
         d.mkdir()
         config = DispatchConfig(
@@ -606,6 +607,26 @@ class TestListAgentsPermissions:
         assert "allowed_tools" not in agents[0]
         assert "disallowed_tools" not in agents[0]
 
+    @pytest.mark.asyncio
+    async def test_list_includes_explicit_empty_tools(self, tmp_path: Path):
+        """allowed_tools=[] (explicit empty) SHOULD appear as [] to signal override."""
+        d = tmp_path / "proj"
+        d.mkdir()
+        config = DispatchConfig(
+            agents={
+                "proj": AgentConfig(
+                    directory=d, description="test",
+                    allowed_tools=[], disallowed_tools=[],
+                ),
+            }
+        )
+        mock_ctx = AsyncMock()
+        with patch.object(server, "_get_config", return_value=config):
+            raw = await server.list_agents(ctx=mock_ctx)
+            agents = json.loads(raw)
+        assert agents[0]["allowed_tools"] == []
+        assert agents[0]["disallowed_tools"] == []
+
 
 class TestAddRemoveAgent:
     @pytest.mark.asyncio