voidx 3.7.3__tar.gz → 3.7.5__tar.gz

{voidx-3.7.3/src/voidx.egg-info → voidx-3.7.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: voidx
-Version: 3.7.3
+Version: 3.7.5
 Summary: A coding agent that quantifies everything and solves with tools, not fuzzy prompts.
 Author: chikhamx
 License: MIT

{voidx-3.7.3 → voidx-3.7.5}/src/tests/test_agent/graph/test_graph_authorization.py

@@ -241,6 +241,56 @@ async def test_graph_authorization_never_approves_mixed_blocked_batch(tmp_path):
     assert [item.tool_call["id"] for item in asked[1]] == ["ask"]
 
 
+@pytest.mark.asyncio
+async def test_full_access_workflow_gate_advisory_allows_without_approval(tmp_path):
+    graph = _graph(tmp_path)
+    graph._permission.set_permission_mode("full_access")
+
+    async def fail_if_asked(_tool_calls):
+        pytest.fail("workflow gate should not prompt for approval under full_access")
+
+    graph._ask_tool_permission = fail_if_asked
+
+    approved, denied = await graph._authorize_tool_calls(
+        [{"name": "write", "args": {"file_path": "app.py", "content": "x"}, "id": "call_1"}],
+        plan_mode=False,
+        session_id="test",
+        workflow_runs=[
+            WorkflowRunState(name="brainstorm", status=WorkflowRunStatus.ACTIVE),
+        ],
+    )
+
+    assert [call["id"] for call in approved] == ["call_1"]
+    assert denied == []
+
+
+@pytest.mark.asyncio
+async def test_safe_mode_workflow_gate_still_prompts(tmp_path):
+    """Under safe mode, workflow gate advisory does not bypass permission approval."""
+    graph = _graph(tmp_path)
+    graph._permission.set_permission_mode("safe")
+    asked: list = []
+
+    async def approve(tool_calls):
+        asked.extend(tool_calls)
+        return "y"
+
+    graph._ask_tool_permission = approve
+
+    approved, denied = await graph._authorize_tool_calls(
+        [{"name": "write", "args": {"file_path": "app.py", "content": "x"}, "id": "call_1"}],
+        plan_mode=False,
+        session_id="test",
+        workflow_runs=[
+            WorkflowRunState(name="brainstorm", status=WorkflowRunStatus.ACTIVE),
+        ],
+    )
+
+    assert [call["id"] for call in approved] == ["call_1"]
+    assert denied == []
+    assert len(asked) > 0
+
+
 @pytest.mark.asyncio
 async def test_graph_authorization_asks_for_write_by_active_workflow_gate(tmp_path):
     graph = _graph(tmp_path)
@@ -269,7 +319,14 @@ async def test_graph_authorization_asks_for_write_by_active_workflow_gate(tmp_pa
 @pytest.mark.asyncio
 async def test_graph_authorization_uses_current_workflow_gate_only(tmp_path):
     graph = _graph(tmp_path)
-    
+    asked: list[list[dict]] = []
+
+    async def approve(tool_calls):
+        asked.append(tool_calls)
+        return "y"
+
+    graph._ask_tool_permission = approve
+
     approved, denied = await graph._authorize_tool_calls(
         [{
             "name": "edit",
@@ -285,12 +342,14 @@ async def test_graph_authorization_uses_current_workflow_gate_only(tmp_path):
         ],
     )
 
+    assert [[call["id"] for call in _asked_tool_calls(batch)] for batch in asked] == [["call_1"]]
     assert [call["id"] for call in approved] == ["call_1"]
     assert denied == []
 
 
 @pytest.mark.asyncio
-async def test_graph_authorization_allows_plan_gate_doc_paths_only(tmp_path):
+async def test_graph_authorization_plan_gate_no_longer_bypasses_doc_paths(tmp_path):
+    """allowed_paths bypass removed: both docs and src edits go through normal permission."""
     graph = _graph(tmp_path)
     asked: list[list[dict]] = []
 
@@ -321,16 +380,23 @@ async def test_graph_authorization_allows_plan_gate_doc_paths_only(tmp_path):
         ],
     )
 
-    assert [call["id"] for call in approved] == ["call_docs"]
-    assert [[call["id"] for call in _asked_tool_calls(batch)] for batch in asked] == [["call_src"]]
-    assert [call["id"] for call, _reason in denied] == ["call_src"]
-    assert denied[0][1] == "User denied: replace"
+    assert approved == []
+    assert [call["id"] for call, _reason in denied] == ["call_docs", "call_src"]
+    assert all(reason == "User denied: replace" for _tc, reason in denied)
 
 
 @pytest.mark.asyncio
-async def test_graph_authorization_allowed_paths_match_nested_docs(tmp_path):
+async def test_graph_authorization_nested_docs_go_through_normal_permission(tmp_path):
+    """allowed_paths bypass removed: nested docs edits go through normal permission."""
     graph = _graph(tmp_path)
-    
+    asked: list[list[dict]] = []
+
+    async def approve(tool_calls):
+        asked.append(tool_calls)
+        return "y"
+
+    graph._ask_tool_permission = approve
+
     approved, denied = await graph._authorize_tool_calls(
         [{
             "name": "edit",
@@ -345,6 +411,7 @@ async def test_graph_authorization_allowed_paths_match_nested_docs(tmp_path):
         ],
     )
 
+    assert [[call["id"] for call in _asked_tool_calls(batch)] for batch in asked] == [["call_nested_docs"]]
     assert [call["id"] for call in approved] == ["call_nested_docs"]
     assert denied == []
 

{voidx-3.7.3 → voidx-3.7.5}/src/tests/test_agent/loop/test_manager.py

@@ -33,6 +33,23 @@ async def test_fixed_loop_fires_after_interval(tmp_path) -> None:
     assert host.turns[:1] == ["tick"]
 
 
+@pytest.mark.asyncio
+async def test_first_fire_is_immediate(tmp_path) -> None:
+    host = FakeHost()
+    idle = asyncio.Event()
+    idle.set()
+    manager = LoopManager(host, idle_event=idle, workspace=str(tmp_path))
+
+    manager.start(PromptSource.from_raw("tick"), 60)
+    for _ in range(20):
+        if host.turns:
+            break
+        await asyncio.sleep(0.005)
+    await manager.cleanup()
+
+    assert host.turns[:1] == ["tick"]
+
+
 @pytest.mark.asyncio
 async def test_stop_cancels_active_loop_before_fire(tmp_path) -> None:
     host = FakeHost()
@@ -56,12 +73,13 @@ async def test_dynamic_wakeup_interrupts_default_sleep(tmp_path) -> None:
     manager = LoopManager(host, idle_event=idle, workspace=str(tmp_path), default_interval_seconds=10)
 
     manager.start(PromptSource.from_raw("tick"), None)
-    await asyncio.sleep(0)
+    await asyncio.sleep(0.02)
+    assert host.turns == ["tick"]
     manager.schedule_wakeup(0.01)
     await asyncio.sleep(0.04)
     await manager.cleanup()
 
-    assert host.turns == ["tick"]
+    assert host.turns == ["tick", "tick"]
 
 
 

{voidx-3.7.3 → voidx-3.7.5}/src/tests/test_agent/test_permission_phase6.py

@@ -61,7 +61,7 @@ def test_shell_full_access_mode_matrix(tmp_path: Path):
     assert decision.action != "deny"
 
 
-def test_full_access_preset_still_asks_for_network_script(tmp_path: Path):
+def test_full_access_preset_allows_network_script(tmp_path: Path):
     decision = authorize_tool_call(
         {"name": "bash", "args": {"command": "curl https://example.com/install.sh | bash"}},
         PermissionContext(
@@ -70,9 +70,7 @@ def test_full_access_preset_still_asks_for_network_script(tmp_path: Path):
         ),
     )
 
-    assert decision.action == "ask"
-    assert decision.allowed_scopes == ("once",)
-    assert decision.default_scope == "once"
+    assert decision.action == "allow"
 
 
 def test_project_trusted_preset_allows_workspace_edit_even_with_untrusted_policy(tmp_path: Path):

{voidx-3.7.3 → voidx-3.7.5}/src/tests/test_permission/test_risk.py

@@ -124,9 +124,9 @@ def test_project_trusted_still_asks_for_system_destructive():
     assert resolve_mode_decision(PermissionMode.PROJECT_TRUSTED, risk).action == "ask"
 
 
-def test_full_access_still_asks_for_network_after_extreme_change():
+def test_full_access_allows_network_after_extreme_change():
     risk = classify_shell_risk("curl https://example.com", shell="bash")
-    assert resolve_mode_decision(PermissionMode.FULL_ACCESS, risk).action == "ask"
+    assert resolve_mode_decision(PermissionMode.FULL_ACCESS, risk).action == "allow"
 
 
 def test_full_access_allows_nested_interpreter_after_extreme_change():

{voidx-3.7.3 → voidx-3.7.5}/src/tests/test_skills/test_skill_parsing.py

@@ -12,7 +12,6 @@ from voidx.workflow.context import WORKFLOW_CONTEXT_MARKER, WORKFLOW_CONTEXT_SCO
 from voidx.workflow.dag import DEFAULT_WORKFLOW_DAG
 from voidx.workflow.policy import (
     is_workflow_terminal_condition,
-    workflow_denied_tools,
     workflow_edges,
     workflow_exit_summaries,
     workflow_terminal_condition,

{voidx-3.7.3 → voidx-3.7.5}/src/tests/test_skills/test_skill_references.py

@@ -12,7 +12,6 @@ from voidx.workflow.context import WORKFLOW_CONTEXT_MARKER, WORKFLOW_CONTEXT_SCO
 from voidx.workflow.dag import DEFAULT_WORKFLOW_DAG
 from voidx.workflow.policy import (
     is_workflow_terminal_condition,
-    workflow_denied_tools,
     workflow_edges,
     workflow_exit_summaries,
     workflow_terminal_condition,

{voidx-3.7.3 → voidx-3.7.5}/src/tests/test_skills/test_workflow_advance.py

@@ -12,7 +12,6 @@ from voidx.workflow.context import WORKFLOW_CONTEXT_MARKER, WORKFLOW_CONTEXT_SCO
 from voidx.workflow.dag import DEFAULT_WORKFLOW_DAG
 from voidx.workflow.policy import (
     is_workflow_terminal_condition,
-    workflow_denied_tools,
     workflow_edges,
     workflow_exit_summaries,
     workflow_terminal_condition,
@@ -160,10 +159,6 @@ def test_workflow_state_summary_includes_transition_hint():
     assert "next=verify" in run.state_summary()
 
 
-def test_workflow_denied_tools_aggregates_all_active_gates():
-    assert workflow_denied_tools(["debug", "tdd"]) >= {"manage", "write", "replace"}
-
-
 def test_feedback_workflow_exposes_design_and_plan_exits():
     assert workflow_transitions("feedback") == (
         "tdd",

voidx-3.7.5/src/tests/test_tools/bash/test_auto_route_git.py

@@ -0,0 +1,342 @@
+"""Tests for bash/powershell auto-route to git tool via maybe_route_hint."""
+
+from __future__ import annotations
+
+import json
+import os
+
+import pytest
+
+from voidx.tools.base import ToolContext
+from voidx.tools.registry import ToolRegistry
+from voidx.tools.shell.common import RouteHint
+from voidx.tools.bash.router import try_hint
+
+
+# ---------------------------------------------------------------------------
+# Helpers
+# ---------------------------------------------------------------------------
+
+
+def _make_ctx(tmp_path, registry: ToolRegistry | None = None) -> ToolContext:
+    return ToolContext(
+        workspace=str(tmp_path),
+        permission_mode="full_access",
+        tool_registry=registry,
+    )
+
+
+def _make_registry() -> ToolRegistry:
+    return ToolRegistry()
+
+
+# ---------------------------------------------------------------------------
+# 1. bash git auto-route 结构化输出
+# ---------------------------------------------------------------------------
+
+
+class TestBashGitAutoRouteStructured:
+    """bash git command auto-routes to git tool and returns structured output."""
+
+    @pytest.mark.asyncio
+    async def test_git_status_structured(self, tmp_path):
+        import subprocess
+        subprocess.run(["git", "init", str(tmp_path)], check=True, capture_output=True)
+        subprocess.run(
+            ["git", "config", "user.email", "test@test.com"],
+            cwd=str(tmp_path), check=True, capture_output=True,
+        )
+        subprocess.run(
+            ["git", "config", "user.name", "Test"],
+            cwd=str(tmp_path), check=True, capture_output=True,
+        )
+        (tmp_path / "file.txt").write_text("hello")
+        subprocess.run(["git", "add", "."], cwd=str(tmp_path), check=True, capture_output=True)
+        subprocess.run(
+            ["git", "commit", "-m", "init"],
+            cwd=str(tmp_path), check=True, capture_output=True,
+        )
+
+        r = _make_registry()
+        ctx = _make_ctx(tmp_path, r)
+        result = await r.execute_tool("bash", {"command": "git status --porcelain"}, ctx)
+
+        data = json.loads(result.output)
+        assert "ok" in data
+        assert data["ok"] is True
+        assert result.metadata.get("routed_from") == "bash"
+        assert result.metadata.get("routed_command") == "git status --porcelain"
+
+    @pytest.mark.asyncio
+    async def test_git_log_structured(self, tmp_path):
+        import subprocess
+        subprocess.run(["git", "init", str(tmp_path)], check=True, capture_output=True)
+        subprocess.run(
+            ["git", "config", "user.email", "test@test.com"],
+            cwd=str(tmp_path), check=True, capture_output=True,
+        )
+        subprocess.run(
+            ["git", "config", "user.name", "Test"],
+            cwd=str(tmp_path), check=True, capture_output=True,
+        )
+        (tmp_path / "file.txt").write_text("hello")
+        subprocess.run(["git", "add", "."], cwd=str(tmp_path), check=True, capture_output=True)
+        subprocess.run(
+            ["git", "commit", "-m", "init"],
+            cwd=str(tmp_path), check=True, capture_output=True,
+        )
+
+        r = _make_registry()
+        ctx = _make_ctx(tmp_path, r)
+        result = await r.execute_tool("bash", {"command": "git log --oneline -1"}, ctx)
+
+        data = json.loads(result.output)
+        assert data["ok"] is True
+        assert result.metadata.get("routed_from") == "bash"
+
+
+# ---------------------------------------------------------------------------
+# 2. bash git auto-route 带路径
+# ---------------------------------------------------------------------------
+
+
+class TestBashGitAutoRouteWithPath:
+    """bash git -C <path> auto-routes with path passed to git tool."""
+
+    @pytest.mark.asyncio
+    async def test_git_with_C_flag_routes_with_path(self, tmp_path):
+        import subprocess
+        repo = tmp_path / "repo"
+        repo.mkdir()
+        subprocess.run(["git", "init", str(repo)], check=True, capture_output=True)
+        subprocess.run(
+            ["git", "config", "user.email", "test@test.com"],
+            cwd=str(repo), check=True, capture_output=True,
+        )
+        subprocess.run(
+            ["git", "config", "user.name", "Test"],
+            cwd=str(repo), check=True, capture_output=True,
+        )
+        (repo / "file.txt").write_text("hello")
+        subprocess.run(["git", "add", "."], cwd=str(repo), check=True, capture_output=True)
+        subprocess.run(
+            ["git", "commit", "-m", "init"],
+            cwd=str(repo), check=True, capture_output=True,
+        )
+
+        r = _make_registry()
+        ctx = _make_ctx(tmp_path, r)
+        result = await r.execute_tool(
+            "bash", {"command": f"git -C {repo} status --porcelain"}, ctx
+        )
+
+        data = json.loads(result.output)
+        assert data["ok"] is True
+        assert result.metadata.get("routed_from") == "bash"
+        assert result.metadata.get("routed_tool_args", {}).get("path") == str(repo)
+
+
+# ---------------------------------------------------------------------------
+# 3. bash git 破坏性命令拦截
+# ---------------------------------------------------------------------------
+
+
+class TestBashGitDestructiveDenied:
+    """Destructive git commands are denied by git tool policy even via auto-route."""
+
+    @pytest.mark.asyncio
+    async def test_git_reset_hard_denied(self, tmp_path):
+        import subprocess
+        subprocess.run(["git", "init", str(tmp_path)], check=True, capture_output=True)
+        subprocess.run(
+            ["git", "config", "user.email", "test@test.com"],
+            cwd=str(tmp_path), check=True, capture_output=True,
+        )
+        subprocess.run(
+            ["git", "config", "user.name", "Test"],
+            cwd=str(tmp_path), check=True, capture_output=True,
+        )
+        (tmp_path / "file.txt").write_text("hello")
+        subprocess.run(["git", "add", "."], cwd=str(tmp_path), check=True, capture_output=True)
+        subprocess.run(
+            ["git", "commit", "-m", "init"],
+            cwd=str(tmp_path), check=True, capture_output=True,
+        )
+
+        r = _make_registry()
+        ctx = _make_ctx(tmp_path, r)
+        result = await r.execute_tool("bash", {"command": "git reset --hard"}, ctx)
+
+        data = json.loads(result.output)
+        assert data["ok"] is False
+        assert "command_denied" in data.get("error", "")
+        assert result.metadata.get("routed_from") == "bash"
+
+
+# ---------------------------------------------------------------------------
+# 4. bash 非 git 命令不受影响
+# ---------------------------------------------------------------------------
+
+
+class TestBashNonGitUnaffected:
+    """Non-git bash commands still execute normally."""
+
+    @pytest.mark.asyncio
+    async def test_echo_hello_not_routed(self, tmp_path):
+        r = _make_registry()
+        ctx = _make_ctx(tmp_path, r)
+        result = await r.execute_tool("bash", {"command": "echo hello"}, ctx)
+
+        data = json.loads(result.output)
+        assert data["ok"] is True
+        assert "hello" in data["stdout"]
+        assert "routed_from" not in result.metadata
+
+
+# ---------------------------------------------------------------------------
+# 5. bash hint 无 registry 时降级
+# ---------------------------------------------------------------------------
+
+
+class TestBashHintFallbackNoRegistry:
+    """When ctx.tool_registry is None, falls back to hint-only result."""
+
+    @pytest.mark.asyncio
+    async def test_no_registry_returns_hint(self, tmp_path):
+        r = _make_registry()
+        ctx = _make_ctx(tmp_path, registry=None)
+        result = await r.execute_tool("bash", {"command": "git status"}, ctx)
+
+        assert result.metadata.get("skipped") is True
+        assert result.metadata.get("route_hint", {}).get("tool_id") == "git"
+        assert "routed_from" not in result.metadata
+
+
+# ---------------------------------------------------------------------------
+# 7. RouteHint tool_args 默认 None（非 git hint）
+# ---------------------------------------------------------------------------
+
+
+class TestRouteHintToolArgsDefaultNone:
+    """Non-git hints keep tool_args=None and preserve old hint-only behavior."""
+
+    def test_cat_hint_tool_args_none(self):
+        h = try_hint("cat file.py")
+        assert h is not None
+        assert h.tool_id == "read"
+        assert h.tool_args is None
+
+    def test_grep_hint_tool_args_none(self):
+        h = try_hint("grep -r foo .")
+        assert h is not None
+        assert h.tool_id == "grep"
+        assert h.tool_args is None
+
+    def test_find_hint_tool_args_none(self):
+        h = try_hint("find . -name '*.py'")
+        assert h is not None
+        assert h.tool_id == "glob"
+        assert h.tool_args is None
+
+
+# ---------------------------------------------------------------------------
+# 8. bash git 含 -c 配置时降级
+# ---------------------------------------------------------------------------
+
+
+class TestBashGitConfigFlagFallback:
+    """git -c key=value falls back to hint-only (tool_args=None)."""
+
+    @pytest.mark.asyncio
+    async def test_git_with_c_flag_returns_hint(self, tmp_path):
+        r = _make_registry()
+        ctx = _make_ctx(tmp_path, r)
+        result = await r.execute_tool(
+            "bash", {"command": "git -c core.autocrlf=true status"}, ctx
+        )
+
+        assert result.metadata.get("skipped") is True
+        assert result.metadata.get("route_hint", {}).get("tool_id") == "git"
+        assert "routed_from" not in result.metadata
+
+
+# ---------------------------------------------------------------------------
+# 9. bash git --git-dir 不 hint
+# ---------------------------------------------------------------------------
+
+
+class TestBashGitGlobalDirNoHint:
+    """git --git-dir=x does not produce a hint — runs as raw shell command."""
+
+    @pytest.mark.asyncio
+    async def test_git_dir_runs_as_shell(self, tmp_path):
+        r = _make_registry()
+        ctx = _make_ctx(tmp_path, r)
+        result = await r.execute_tool(
+            "bash", {"command": "git --git-dir=/nonexistent status"}, ctx
+        )
+
+        assert result.metadata.get("skipped") is not True
+        assert "routed_from" not in result.metadata
+
+
+# ---------------------------------------------------------------------------
+# 10. filtered registry 降级
+# ---------------------------------------------------------------------------
+
+
+class TestBashFilteredRegistryFallback:
+    """When registry excludes git tool, falls back to hint-only result."""
+
+    @pytest.mark.asyncio
+    async def test_filtered_registry_returns_hint(self, tmp_path):
+        r = _make_registry()
+        r.filter_tools({"bash"})
+        ctx = _make_ctx(tmp_path, r)
+        result = await r.execute_tool("bash", {"command": "git status"}, ctx)
+
+        assert result.metadata.get("skipped") is True
+        assert result.metadata.get("route_hint", {}).get("tool_id") == "git"
+        assert "routed_from" not in result.metadata
+
+
+# ---------------------------------------------------------------------------
+# 11. try_hint 异常安全
+# ---------------------------------------------------------------------------
+
+
+class TestTryHintExceptionSafety:
+    """try_hint catches all exceptions and returns None."""
+
+    def test_exception_returns_none(self, monkeypatch):
+        def boom(*a, **kw):
+            raise RuntimeError("parser crash")
+
+        monkeypatch.setattr("voidx.tools.bash.router._try_hint_impl", boom)
+        assert try_hint("git status") is None
+
+
+# ---------------------------------------------------------------------------
+# 6. powershell git auto-route (skipped on non-Windows)
+# ---------------------------------------------------------------------------
+
+
+@pytest.mark.skipif(os.name == "nt", reason="powershell tool only on Windows")
+class TestPowerShellGitAutoRouteSkipped:
+    """PowerShell auto-route is skipped on non-Windows platforms."""
+
+    def test_powershell_not_registered_on_unix(self):
+        r = _make_registry()
+        assert r.get("powershell") is None
+
+
+@pytest.mark.skipif(os.name != "nt", reason="powershell tool only on Windows")
+class TestPowerShellGitAutoRoute:
+    """PowerShell git auto-route on Windows."""
+
+    @pytest.mark.asyncio
+    async def test_powershell_git_routes(self, tmp_path):
+        r = _make_registry()
+        ctx = _make_ctx(tmp_path, r)
+        result = await r.execute_tool("powershell", {"command": "git status"}, ctx)
+        assert result.metadata.get("routed_from") == "powershell"