gdmcode 0.1.4__tar.gz → 0.1.5__tar.gz

{gdmcode-0.1.4 → gdmcode-0.1.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gdmcode
-Version: 0.1.4
+Version: 0.1.5
 Summary: gdm: AI coding agent for professional developers
 Project-URL: Homepage, https://github.com/guidegdm/gdmcode
 Project-URL: Repository, https://github.com/guidegdm/gdmcode

gdmcode-0.1.5/gdmcode/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.1.5"

{gdmcode-0.1.4 → gdmcode-0.1.5}/gdmcode/agent/tool_orchestrator.py

@@ -25,7 +25,7 @@ from typing import Any
 from gdmcode._internal.constants import _ALWAYS_DENY_TOOLS, _WRITE_TOOLS
 from gdmcode.exceptions import ToolPermissionError
 from gdmcode.security import AuditLogger
-from gdmcode.tools import REGISTRY, ToolResult
+from gdmcode.tools import REGISTRY, ToolResult, ensure_default_tools_registered
 
 __all__ = ["ToolOrchestrator"]
 
@@ -144,15 +144,7 @@ _TOOL_CACHE: _ToolResultCache = _ToolResultCache()
 
 def _ensure_tools_registered() -> None:
     """Import all tool modules so they register themselves with REGISTRY."""
-    import gdmcode.tools.bash_tool      # noqa: F401
-    import gdmcode.tools.read_tools     # noqa: F401
-    import gdmcode.tools.search_tools   # noqa: F401
-    import gdmcode.tools.shell_tools    # noqa: F401
-    import gdmcode.tools.ask_user_tool  # noqa: F401
-    import gdmcode.tools.write_tools    # noqa: F401
-    import gdmcode.tools.dep_tools      # noqa: F401
-    import gdmcode.tools.impact_tools   # noqa: F401
-    import gdmcode.tools.browser_tool   # noqa: F401
+    ensure_default_tools_registered()
 
 
 _ensure_tools_registered()
@@ -399,4 +391,4 @@ class ToolOrchestrator:
         """Dispatch a registered plugin tool by its namespaced tool_name."""
         if tool_name not in self._plugin_channels:
             raise KeyError(f"Unknown plugin tool: {tool_name}")
-        return self._dispatch_plugin(tool_name, args)
+        return self._dispatch_plugin(tool_name, args)

{gdmcode-0.1.4 → gdmcode-0.1.5}/gdmcode/cli.py

@@ -287,7 +287,8 @@ def cmd_health(
         GdmDatabase = _GdmDatabase
     from gdmcode.models.definitions import PROVIDER_BASE_URLS, ModelTier, get_model
     if REGISTRY is None:
-        from gdmcode.tools import REGISTRY as _REGISTRY
+        from gdmcode.tools import REGISTRY as _REGISTRY, ensure_default_tools_registered
+        ensure_default_tools_registered()
         REGISTRY = _REGISTRY
 
     @dataclass
@@ -352,8 +353,28 @@ def cmd_health(
         try:
             tools = REGISTRY.all_tools()
             count = len(tools)
-            status = "ok" if count > 0 else "fail"
-            return CheckResult("tools", status, latency_ms=int((time.monotonic() - t0) * 1000), detail=f"{count} tools")
+            if count == 0:
+                return CheckResult("tools", "fail", latency_ms=int((time.monotonic() - t0) * 1000), detail="0 tools registered")
+            probe_name = "powershell" if sys.platform == "win32" else "bash"
+            probe_args = (
+                {"command": "$PWD.Path", "timeout": 5}
+                if probe_name == "powershell"
+                else {"command": "pwd", "timeout": 5, "read_only": True}
+            )
+            probe = REGISTRY.call(probe_name, probe_args)
+            if not probe.ok:
+                return CheckResult(
+                    "tools",
+                    "fail",
+                    latency_ms=int((time.monotonic() - t0) * 1000),
+                    detail=f"{count} registered; {probe_name} probe failed: {probe.error}",
+                )
+            return CheckResult(
+                "tools",
+                "ok",
+                latency_ms=int((time.monotonic() - t0) * 1000),
+                detail=f"{count} registered; {probe_name} probe ok",
+            )
         except Exception as exc:  # noqa: BLE001
             return CheckResult("tools", "fail", detail=str(exc))
 
@@ -398,7 +419,8 @@ def cmd_health(
     ]
 
     overall_fail = any(
-        r.status in ("fail", "missing") and r.priority in ("P0", "P1")
+        r.status == "fail" and r.priority in ("P0", "P1")
+        or r.status == "missing" and r.priority == "P0"
         for r in all_checks
     )
 
@@ -1179,6 +1201,7 @@ def _run_one_shot(cfg: object, prompt: str, *, yes: bool, model_override: str |
     """
     from rich.status import Status
     from gdmcode.agent.loop import EventType
+    from gdmcode.repl import _format_llm_error
 
     if not _has_model_connection(cfg):
         console.print("[red]No model connection configured.[/red]")
@@ -1248,10 +1271,21 @@ def _run_one_shot(cfg: object, prompt: str, *, yes: bool, model_override: str |
                 console.print(event.content)
             elif event.type == EventType.ERROR:
                 status.stop()
-                console.print(f"[red]Error:[/red] {event.content}")
+                console.print(_format_llm_error(event.content))
                 had_error = True
             elif event.type == EventType.TOOL_CALL:
+                status.stop()
                 console.print(f"[yellow]  [tool] {event.tool_name}[/yellow]")
+            elif event.type == EventType.TOOL_RESULT:
+                status.stop()
+                result = event.result
+                if result is not None and getattr(result, "error", None):
+                    console.print(f"[red]  [tool error] {event.tool_name}:[/red] {getattr(result, 'error')}")
+                    had_error = True
+                elif result is not None:
+                    output = (getattr(result, "output", "") or "").strip()
+                    if output:
+                        console.print(f"[dim]{output[:2000]}[/dim]")
             elif event.type == EventType.DONE:
                 status.stop()
     except Exception as exc:  # noqa: BLE001

{gdmcode-0.1.4 → gdmcode-0.1.5}/gdmcode/integrations/mcp_server.py

@@ -51,7 +51,7 @@ _INTERNAL_ERROR   = -32603
 class MCPServer:
     """Minimal MCP server with stdio transport."""
 
-    def __init__(self, name: str = "gdmcode", version: str = "0.1.4") -> None:
+    def __init__(self, name: str = "gdmcode", version: str = "0.1.5") -> None:
         self._name = name
         self._version = version
         self._tools: dict[str, MCPTool] = {}

{gdmcode-0.1.4 → gdmcode-0.1.5}/gdmcode/main.py

@@ -6,6 +6,7 @@ pyproject.toml's `gdm = "gdmcode.main:main"` script works.
 from __future__ import annotations
 
 from importlib.metadata import version as _pkg_version, PackageNotFoundError
+import sys
 
 from gdmcode.cli import app
 
@@ -20,8 +21,21 @@ def _get_version() -> str:
 
 def main() -> None:
     """Entry point for the `gdm` CLI command."""
+    _configure_stdio()
     app()
 
 
+def _configure_stdio() -> None:
+    """Avoid Windows cp1252 crashes when Rich output is redirected."""
+    for stream_name in ("stdout", "stderr"):
+        stream = getattr(sys, stream_name, None)
+        if stream is None or not hasattr(stream, "reconfigure"):
+            continue
+        try:
+            stream.reconfigure(encoding="utf-8", errors="replace")
+        except (OSError, ValueError):
+            pass
+
+
 if __name__ == "__main__":
     main()

{gdmcode-0.1.4 → gdmcode-0.1.5}/gdmcode/repl.py

@@ -162,6 +162,35 @@ def _format_llm_error(error: object) -> str:
     return f"[red]{title}{code}.[/red]\n[dim]{escape(raw[:800])}[/dim]\n[yellow]{hint}[/yellow]"
 
 
+def _is_connection_probe(text: str) -> bool:
+    """Return True for tiny greetings/checks that should not enter tool mode."""
+    import re
+
+    normalized = re.sub(r"[^a-z0-9\s']", " ", text.lower()).strip()
+    normalized = re.sub(r"\s+", " ", normalized)
+    if not normalized:
+        return False
+    direct = {
+        "hey",
+        "hi",
+        "hello",
+        "yo",
+        "ping",
+        "test",
+        "hello there",
+        "hey there",
+        "hi there",
+        "are you there",
+        "you there",
+    }
+    if normalized in direct:
+        return True
+    words = normalized.split()
+    return len(words) <= 4 and words[0] in {"hey", "hi", "hello"} and all(
+        word in {"hey", "hi", "hello", "there", "gdm", "agent", "please"} for word in words
+    )
+
+
 def _render_event(event: object, status: Status, console: Console) -> bool:
     """Render one AgentEvent to the terminal while the spinner is live."""
     from gdmcode.agent.loop import EventType  # lazy
@@ -173,6 +202,7 @@ def _render_event(event: object, status: Status, console: Console) -> bool:
             label = f"[dim cyan]Thinking... {snippet}[/dim cyan]"
             status.update(label)
         case EventType.TOOL_CALL:
+            status.stop()
             console.print(f"[yellow][tool] {ev.tool_name}({_fmt_args(ev.args)})[/yellow]")  # type: ignore[union-attr]
         case EventType.TOOL_RESULT:
             result_str = str(ev.result or "")[:80]  # type: ignore[union-attr]
@@ -384,6 +414,51 @@ def start_repl(cfg: "GdmConfig", db: "GdmDatabase", *, yes: bool = False, model_
             console.print("[yellow]Agent loop not available in this installation.[/yellow]")
             return False
 
+    def _run_connection_probe(text: str) -> None:
+        """Send a tiny no-tools request to confirm the active LLM route."""
+        nonlocal cfg
+        if not _has_model_route():
+            try:
+                cfg = _reload_cfg(require_credentials=True)  # type: ignore[assignment]
+                dispatcher._cfg = cfg  # type: ignore[attr-defined]
+                dispatcher._provider = getattr(cfg, "provider", dispatcher._provider)  # type: ignore[attr-defined]
+            except Exception as exc:  # noqa: BLE001
+                _print_onboarding(console, error=str(exc))
+                return
+
+        status = Status("[cyan]Checking LLM connection...[/cyan]", console=console, spinner="dots")
+        status.start()
+        try:
+            from gdmcode.models.client import GdmClient
+            from gdmcode.models.definitions import ModelTier, get_model
+
+            tier = model_override or ModelTier.CODER
+            model_def = get_model(tier, cfg.provider)
+            proxy_active = bool(
+                getattr(cfg, "proxy_enabled", False)
+                and getattr(cfg, "proxy_url", "")
+                and getattr(cfg, "proxy_token", "")
+            )
+            client = (
+                GdmClient.for_proxy(getattr(cfg, "proxy_url"), getattr(cfg, "proxy_token"))
+                if proxy_active
+                else GdmClient(cfg)
+            )
+            response = client.complete_text(
+                text,
+                model=model_def.id,
+                max_tokens=80,
+                system=(
+                    "You are gdm code's CLI connection check. Reply in one short, friendly "
+                    "sentence. Confirm the LLM is connected. Do not inspect files or call tools."
+                ),
+            ).strip()
+            status.stop()
+            console.print(response or "[green]LLM connected.[/green]")
+        except Exception as exc:  # noqa: BLE001
+            status.stop()
+            console.print(_format_llm_error(exc))
+
     if not getattr(cfg, "gdm_quiet", False):
         if _has_model_route():
             console.print("[dim]Agent will initialize on first prompt.[/dim]")
@@ -587,6 +662,10 @@ def start_repl(cfg: "GdmConfig", db: "GdmDatabase", *, yes: bool = False, model_
                     console.print("[yellow]Restore not supported by current agent.[/yellow]")
             continue
 
+        if _is_connection_probe(text):
+            _run_connection_probe(text)
+            continue
+
         pending = _show_pending_btw(db, session_id, console)
         full_message = text
         if pending:

{gdmcode-0.1.4 → gdmcode-0.1.5}/gdmcode/tools/__init__.py

@@ -20,6 +20,7 @@ __all__ = [
     "ToolResult",
     "ToolRegistry",
     "REGISTRY",
+    "ensure_default_tools_registered",
     "BrowserTools",
     "BrowserToolResult",
 ]
@@ -155,6 +156,20 @@ class ToolRegistry:
 # Module-level singleton — all concrete tool modules import and extend this.
 REGISTRY: ToolRegistry = ToolRegistry()
 
+
+def ensure_default_tools_registered() -> None:
+    """Import built-in tool modules so they register with REGISTRY."""
+    import gdmcode.tools.bash_tool      # noqa: F401
+    import gdmcode.tools.read_tools     # noqa: F401
+    import gdmcode.tools.search_tools   # noqa: F401
+    import gdmcode.tools.shell_tools    # noqa: F401
+    import gdmcode.tools.ask_user_tool  # noqa: F401
+    import gdmcode.tools.write_tools    # noqa: F401
+    import gdmcode.tools.dep_tools      # noqa: F401
+    import gdmcode.tools.impact_tools   # noqa: F401
+    import gdmcode.tools.browser_tool   # noqa: F401
+
+
 # Re-export browser tools so callers can do `from gdmcode.tools import BrowserTools`.
 # Import after REGISTRY is defined to avoid circular imports.
 from gdmcode.tools.browser_tools import BrowserToolResult, BrowserTools, BrowserToolsSync  # noqa: E402, F401

{gdmcode-0.1.4 → gdmcode-0.1.5}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "gdmcode"
-version = "0.1.4"
+version = "0.1.5"
 description = "gdm: AI coding agent for professional developers"
 readme = "README.md"
 license = { text = "MIT" }

{gdmcode-0.1.4 → gdmcode-0.1.5}/tests/test_cli_smoke.py

@@ -1,6 +1,7 @@
 """Smoke tests for gdm CLI entrypoints."""
 from __future__ import annotations
 
+from types import SimpleNamespace
 from unittest.mock import MagicMock, patch
 
 import pytest
@@ -57,3 +58,39 @@ class TestCliSmoke:
     def test_version_shows_version_string(self) -> None:
         result = runner.invoke(app, ["version"])
         assert "gdm" in result.output.lower() or "0." in result.output
+
+    def test_health_bootstraps_and_executes_tools(self, tmp_path, monkeypatch) -> None:
+        import gdmcode.cli as cli
+
+        class FakeDb:
+            def __init__(self, project_root):
+                self.project_root = project_root
+
+            def execute(self, *_args, **_kwargs):
+                return None
+
+        class FakeDaemon:
+            is_running = True
+
+            def __init__(self, db):
+                self.db = db
+
+            def pending_count(self):
+                return 0
+
+        monkeypatch.setattr(cli, "GdmDatabase", FakeDb)
+        monkeypatch.setattr(cli, "BackgroundDaemon", FakeDaemon)
+        monkeypatch.setattr(cli, "REGISTRY", None)
+        monkeypatch.setattr(
+            cli,
+            "load_config",
+            lambda: SimpleNamespace(provider="gemini", api_key="key", project_root=tmp_path),
+        )
+
+        response = SimpleNamespace(status_code=200)
+        with patch("gdmcode.cli.httpx.get", return_value=response):
+            result = runner.invoke(app, ["health"])
+
+        assert "tools" in result.output
+        assert "registered" in result.output
+        assert "probe ok" in result.output

{gdmcode-0.1.4 → gdmcode-0.1.5}/tests/test_repl_smoke.py

@@ -39,6 +39,13 @@ class TestReplHelpers:
         from gdmcode.repl import _fmt_args
         assert _fmt_args({}) == ""
 
+    def test_connection_probe_detection(self) -> None:
+        from gdmcode.repl import _is_connection_probe
+
+        assert _is_connection_probe("hey")
+        assert _is_connection_probe("hello there")
+        assert not _is_connection_probe("fix the failing tests")
+
 
 class TestReplStartRepl:
     """Test start_repl runs and exits cleanly when given /exit input."""
@@ -157,6 +164,24 @@ class TestReplStartRepl:
         assert "/login grok" in output
         assert "/proxy token" in output
 
+    def test_plain_hey_with_credentials_uses_no_tool_probe(self, tmp_path, capsys) -> None:
+        from gdmcode.repl import start_repl
+
+        cfg, db = self._make_mocks(tmp_path)
+        cfg.api_key = "test-key"
+
+        with patch("gdmcode.repl._build_input_fn") as mock_build:
+            mock_build.return_value = MagicMock(side_effect=["hey", EOFError])
+            with patch("gdmcode.models.client.GdmClient.complete_text", return_value="Hey, LLM connected."):
+                with patch("gdmcode.repl._run_agent_turn") as run_agent:
+                    with patch("gdmcode.cost_tracker.CostTracker"):
+                        with patch("gdmcode.repl._ensure_session", return_value="session-123"):
+                            start_repl(cfg, db)
+
+        output = capsys.readouterr().out
+        assert "LLM connected" in output
+        run_agent.assert_not_called()
+
 
 class TestReplErrorFormatting:
     def test_format_llm_error_rate_limit(self) -> None:
@@ -193,6 +218,17 @@ class TestReplErrorFormatting:
         output = capsys.readouterr().out
         assert "credit" in output.lower() or "quota" in output.lower()
 
+    def test_render_tool_call_stops_spinner_before_permission_work(self, capsys) -> None:
+        from gdmcode.agent.loop import EventType
+        from gdmcode.repl import _render_event
+
+        status = MagicMock()
+        console = MagicMock()
+        event = SimpleNamespace(type=EventType.TOOL_CALL, tool_name="bash", args={"command": "ls -F"})
+
+        assert _render_event(event, status, console) is False
+        status.stop.assert_called_once()
+
 
 class TestCommandDispatcher:
     """Smoke tests for CommandDispatcher (called by start_repl)."""

gdmcode-0.1.4/gdmcode/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.1.4"