gdmcode 0.1.10__tar.gz → 0.1.11__tar.gz

{gdmcode-0.1.10 → gdmcode-0.1.11}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: gdmcode
-Version: 0.1.10
+Version: 0.1.11
 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.11/gdmcode/__init__.py

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

{gdmcode-0.1.10 → gdmcode-0.1.11}/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.10") -> None:
+    def __init__(self, name: str = "gdmcode", version: str = "0.1.11") -> None:
         self._name = name
         self._version = version
         self._tools: dict[str, MCPTool] = {}

{gdmcode-0.1.10 → gdmcode-0.1.11}/gdmcode/permissions.py

@@ -14,6 +14,7 @@ Decision hierarchy (first match wins):
 from __future__ import annotations
 
 import logging
+import sys
 from dataclasses import dataclass, field
 from enum import StrEnum
 from typing import TYPE_CHECKING
@@ -61,6 +62,40 @@ log = logging.getLogger(__name__)
 _active_bridge: "PermissionBridge | None" = None
 
 
+def _read_permission_choice() -> str:
+    """Read a permission choice using plain stdin/stdout.
+
+    Rich/Prompt.ask can interact poorly with active terminal renderers on some
+    Windows shells. A simple flushed line prompt keeps the approval input
+    editable and visible.
+    """
+    valid = {
+        "": "y",
+        "y": "y",
+        "yes": "y",
+        "n": "n",
+        "no": "n",
+        "s": "session",
+        "session": "session",
+        "a": "always",
+        "always": "always",
+        "v": "never",
+        "never": "never",
+    }
+    prompt = "Allow? type y/n/session/always/never and press Enter [y]: "
+    while True:
+        sys.stdout.write(prompt)
+        sys.stdout.flush()
+        line = sys.stdin.readline()
+        if line == "":
+            return "n"
+        choice = valid.get(line.strip().lower())
+        if choice is not None:
+            return choice
+        sys.stdout.write("Please enter y, n, session, always, or never.\n")
+        sys.stdout.flush()
+
+
 def set_permission_bridge(bridge: "PermissionBridge") -> None:
     """Activate a PermissionBridge so _prompt_user() delegates to it."""
     global _active_bridge
@@ -250,9 +285,9 @@ class PermissionContext:
             self._session_allow = self._session_allow | {tool}
             return Decision.ALLOW_SESSION
 
-        # Rich prompt — imported lazily to avoid circular dep at module init
+        # Rich output only; use plain stdin for the actual answer so terminal
+        # input remains editable even after prompt_toolkit/status rendering.
         from rich.console import Console
-        from rich.prompt import Prompt
 
         console = Console()
         console.print(f"\n[bold yellow]⚠  Permission request[/bold yellow]")
@@ -261,11 +296,7 @@ class PermissionContext:
             import json
             console.print(f"Args: [dim]{json.dumps(args, indent=2)[:400]}[/dim]")
 
-        choice = Prompt.ask(
-            "Allow?",
-            choices=["y", "n", "always", "session", "never"],
-            default="y",
-        )
+        choice = _read_permission_choice()
 
         decision_map = {
             "y": Decision.ALLOW_SESSION,

{gdmcode-0.1.10 → gdmcode-0.1.11}/pyproject.toml

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

{gdmcode-0.1.10 → gdmcode-0.1.11}/tests/test_permissions.py

@@ -104,3 +104,25 @@ def test_non_interactive_denies_unknown(mock_db):
     with pytest.raises(ToolPermissionError) as exc_info:
         ctx_ni.check("some_tool", {})
     assert "non-interactive" in str(exc_info.value).lower()
+
+
+def test_plain_permission_prompt_accepts_yes(ctx, monkeypatch):
+    monkeypatch.setattr("sys.stdin.readline", lambda: "y\n")
+
+    assert ctx.check("bash", {"command": "ls -F"}) == Decision.ALLOW_SESSION
+
+
+def test_plain_permission_prompt_denies_no(ctx, monkeypatch):
+    monkeypatch.setattr("sys.stdin.readline", lambda: "n\n")
+
+    with pytest.raises(ToolPermissionError) as exc_info:
+        ctx.check("bash", {"command": "ls -F"})
+
+    assert "denied by user" in str(exc_info.value).lower()
+
+
+def test_plain_permission_prompt_reprompts_invalid_choice(ctx, monkeypatch):
+    choices = iter(["maybe\n", "always\n"])
+    monkeypatch.setattr("sys.stdin.readline", lambda: next(choices))
+
+    assert ctx.check("bash", {"command": "ls -F"}) == Decision.ALLOW_ALWAYS

gdmcode-0.1.10/gdmcode/__init__.py

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