code-puppy 0.0.892__tar.gz → 0.0.894__tar.gz

{code_puppy-0.0.892 → code_puppy-0.0.894}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: code-puppy
-Version: 0.0.892
+Version: 0.0.894
 Summary: Code generation agent
 Project-URL: repository, https://github.com/mpfaffenberger/code_puppy
 Project-URL: HomePage, https://github.com/mpfaffenberger/code_puppy

{code_puppy-0.0.892 → code_puppy-0.0.894}/code_puppy/mcp_/http_auth.py

@@ -37,4 +37,13 @@ def http_auth(config: dict, url: str, headers: dict | None) -> OAuth | None:
         raise ValueError(t("mcp.oauth.https_required"))
     if not parsed.hostname or parsed.username or parsed.password:
         raise ValueError(t("mcp.oauth.https_required"))
-    return OAuth(client_name="Code Puppy", callback_host="127.0.0.1")
+    # Register as a public client (RFC 8252 §8.4): a CLI cannot keep a secret,
+    # and PKCE already protects the code exchange. Left unset, servers default
+    # to client_secret_basic, and the MCP SDK then sends the Basic header *and*
+    # client_id in the body; strict servers (mcp.cloudflare.com) reject that
+    # token request as "multiple authentication methods".
+    return OAuth(
+        client_name="Code Puppy",
+        callback_host="127.0.0.1",
+        additional_client_metadata={"token_endpoint_auth_method": "none"},
+    )

{code_puppy-0.0.892 → code_puppy-0.0.894}/code_puppy/models_dev_parser.py

@@ -554,9 +554,8 @@ def get_registry() -> Optional[ModelsDevRegistry]:
 
     This serves the model-resolution path (``config.get_model_max_output_tokens``),
     which runs at startup and on every model switch, so it must never touch the
-    network: a plain ``hi`` would otherwise phone models.dev, and the CI egress
-    guard (``tests/integration/test_network_traffic_monitoring.py``) blocks
-    every release over it. The interactive ``/add_model`` and
+    network: a plain ``hi`` would otherwise phone models.dev (pinned by
+    ``tests/test_models_dev_registry_cache.py``). The interactive ``/add_model`` and
     ``/refresh_models`` commands build their own live ``ModelsDevRegistry()``
     on demand, so the snapshot only ever lags until the user asks for fresh
     data. The first caller pays the parse cost once; everyone after reuses

code_puppy-0.0.894/code_puppy/tools/approval_feedback.py

@@ -0,0 +1,29 @@
+"""Keep blocking feedback input off the loop without abandoning stdin."""
+
+import asyncio
+from collections.abc import Callable
+
+
+async def read_feedback(prompt: Callable[[], str]) -> str:
+    """Read feedback in a worker, retaining ownership through cancellation.
+
+    A blocking terminal read cannot be interrupted by cancelling its asyncio
+    waiter. Defer cancellation until the user submits or closes the input;
+    callers must retain their stdin suspension and approval lock throughout.
+    """
+    reader = asyncio.create_task(asyncio.to_thread(prompt))
+    try:
+        return await asyncio.shield(reader)
+    except asyncio.CancelledError:
+        # Further cancel requests must not release stdin while it is still
+        # being read. Retrieve the result/exception before propagating cancel.
+        while not reader.done():
+            try:
+                await asyncio.shield(reader)
+            except asyncio.CancelledError:
+                continue
+            except Exception:
+                break
+        if not reader.cancelled():
+            reader.exception()
+        raise

{code_puppy-0.0.892 → code_puppy-0.0.894}/code_puppy/tools/common.py

@@ -1136,14 +1136,18 @@ async def _get_user_approval_async_impl(
             confirmed = False
             emit_info("")
             emit_info(t("tools.common.approval.tell", puppy_name=puppy_name))
-            # Prompt.ask reads stdin — suspend the key listener or it eats
-            # roughly half the keystrokes (feedback box looks "broken").
+            # Reading feedback must not block the agent's event loop.
+            # Keep stdin ownership suspended until the worker finishes.
             from code_puppy.agents._key_listeners import suspended_key_listener
+            from code_puppy.tools.approval_feedback import read_feedback
 
             with suspended_key_listener():
-                user_feedback = Prompt.ask(
-                    "[bold green]➤[/bold green]",
-                    default="",
+                user_feedback = (
+                    await read_feedback(
+                        lambda: Prompt.ask(
+                            "[bold green]\u27a4[/bold green]", default=""
+                        )
+                    )
                 ).strip()
 
             if not user_feedback:

{code_puppy-0.0.892 → code_puppy-0.0.894}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "code-puppy"
-version = "0.0.892"
+version = "0.0.894"
 description = "Code generation agent"
 readme = "README.md"
 requires-python = ">=3.11,<3.15"