codex-agent-framework 0.1.64__tar.gz → 0.1.66__tar.gz

{codex_agent_framework-0.1.64/codex_agent_framework.egg-info → codex_agent_framework-0.1.66}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codex-agent-framework
-Version: 0.1.64
+Version: 0.1.66
 Summary: A lightweight event-driven Codex agent runtime.
 Author: Baptiste
 License-Expression: MIT

{codex_agent_framework-0.1.64 → codex_agent_framework-0.1.66}/codex_agent/agent.py

@@ -7,7 +7,6 @@ from .utils import (
     upload_file_to_folder,
 )
 import os
-from modict import modict
 from .message import (
     DeveloperMessage,
     ServerToolResponseMessage,
@@ -323,7 +322,11 @@ class Agent:
         )
 
     def add_message(self, msg, pending=False):
-        """Queue a message for the next pending flush, except tool results."""
+        """Queue a message for the next flush, except tool results.
+
+        The ``pending`` argument is kept for compatibility; pending routing is
+        controlled by the current mainloop state.
+        """
         if isinstance(msg, ToolResultMessage):
             return self.sessions_manager.append_message(msg)
         if self.mainloop_manager.dumping_pending:
@@ -334,7 +337,8 @@ class Agent:
         """Helper to create and add a message based on provided keyword arguments.
 
         Args:
-            pending: If True, stores message in pending list instead of main messages.
+            pending: Backward-compatible hint; pending routing is controlled by
+                the current mainloop state.
             **kwargs: Message fields. Must include an explicit ``type``.
 
         Returns:

{codex_agent_framework-0.1.64 → codex_agent_framework-0.1.66}/codex_agent/ai.py

@@ -327,11 +327,11 @@ class AIClient:
 
     def is_bad_request_error(self, exc):
         response = getattr(exc, "response", None)
-        status_code = getattr(response, "status_code", None)
-        if status_code == 400:
-            return True
-        status_code = getattr(exc, "status_code", None)
-        if status_code == 400:
+        status_codes = (
+            getattr(response, "status_code", None),
+            getattr(exc, "status_code", None),
+        )
+        if 400 in status_codes:
             return True
         return "400" in str(exc) and "Bad Request" in str(exc)
 

{codex_agent_framework-0.1.64 → codex_agent_framework-0.1.66}/codex_agent/client.py

@@ -281,7 +281,7 @@ class AgentClient:
             payload.status_code = exc.response.status_code
             try:
                 detail = exc.response.json().get("detail")
-            except Exception:
+            except ValueError:
                 detail = exc.response.text
             if detail:
                 payload.detail = detail

{codex_agent_framework-0.1.64 → codex_agent_framework-0.1.66}/codex_agent/hooks.py

@@ -24,6 +24,8 @@ class HookContext(modict):
 
 
 class HookManager:
+    """Registry and dispatcher for named extension hooks."""
+
     def __init__(self, agent=None):
         self.agent = agent
         self.handlers = modict()

{codex_agent_framework-0.1.64 → codex_agent_framework-0.1.66}/codex_agent/provider.py

@@ -20,6 +20,7 @@ def provider(func=None, **options):
 
 
 def provider_options(func):
+    """Return provider decorator metadata from a function or bound method."""
     options = getattr(func, PROVIDER_MARKER, None)
     if options is not None:
         return options
@@ -27,6 +28,8 @@ def provider_options(func):
 
 
 class ProvidersManager:
+    """Registry for context providers contributed by the agent and plugins."""
+
     def __init__(self, agent):
         self.agent = agent
         self.providers = modict()

{codex_agent_framework-0.1.64 → codex_agent_framework-0.1.66}/codex_agent/stream_utils.py

@@ -7,15 +7,18 @@ from typing import Union
 import json
 
 class END:
-    #sentinel value for end of stream
-    pass
+    """Sentinel value placed in queues to mark the end of a stream."""
 
 
 class StreamError:
+    """Wrapper used to propagate producer exceptions through queue readers."""
+
     def __init__(self, error):
         self.error = error
 
+
 class Task:
+    """Tiny thread-backed task helper used by legacy stream utilities."""
 
     def __init__(self,target=None,args=None,kwargs=None):
         self.target_task=target or self.target
@@ -34,6 +37,7 @@ class Task:
         self.thread.join()
 
 class Tee:
+    """Fan out one input stream into independent reader streams."""
 
     def __init__(self):
         self.queues={}
@@ -59,6 +63,7 @@ class Tee:
         return tuple(reader(queue) for queue in self.queues.values())
     
 class Splitter:
+    """Split a stream into true/false readers according to a predicate."""
 
     def __init__(self,condition=None):
         self.queue_true=None
@@ -91,6 +96,7 @@ class Splitter:
         return reader(self.queue_true), reader(self.queue_false)
   
 class Streamer:
+    """Base helper that optionally runs stream processing in a worker thread."""
 
     def __init__(self,stream_processor=None,threaded=True):
         self.queue=None

codex_agent_framework-0.1.66/codex_agent/version.py

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

{codex_agent_framework-0.1.64 → codex_agent_framework-0.1.66/codex_agent_framework.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: codex-agent-framework
-Version: 0.1.64
+Version: 0.1.66
 Summary: A lightweight event-driven Codex agent runtime.
 Author: Baptiste
 License-Expression: MIT

{codex_agent_framework-0.1.64 → codex_agent_framework-0.1.66}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "codex-agent-framework"
-version = "0.1.64"
+version = "0.1.66"
 description = "A lightweight event-driven Codex agent runtime."
 readme = "README.md"
 requires-python = ">=3.11"

{codex_agent_framework-0.1.64 → codex_agent_framework-0.1.66}/tests/test_ai.py

@@ -267,6 +267,27 @@ def test_codex_backend_params_move_leading_system_messages_to_instructions():
     }]
 
 
+def test_is_bad_request_error_uses_response_status_code():
+    ai = AIClient(agent=None)
+    exc = RuntimeError("request failed")
+    exc.response = SimpleNamespace(status_code=400)
+
+    assert ai.is_bad_request_error(exc) is True
+
+
+def test_is_bad_request_error_uses_exception_status_code():
+    ai = AIClient(agent=None)
+    exc = SimpleNamespace(status_code=400)
+
+    assert ai.is_bad_request_error(exc) is True
+
+
+def test_is_bad_request_error_keeps_message_fallback():
+    ai = AIClient(agent=None)
+
+    assert ai.is_bad_request_error(RuntimeError("400 Bad Request: nope")) is True
+
+
 def test_codex_stream_idle_timeout_aborts_silent_backend_stream():
     agent = FakeAgent(codex_stream_idle_timeout=0.01)
     ai = AIClient(agent=agent)

{codex_agent_framework-0.1.64 → codex_agent_framework-0.1.66}/tests/test_client.py

@@ -151,6 +151,26 @@ def test_agent_client_http_error_payload_includes_status_and_detail():
     assert payload.error == "an event client is already connected"
 
 
+def test_agent_client_http_error_payload_falls_back_to_text_for_non_json_body():
+    client = AgentClient()
+
+    class Response:
+        status_code = 502
+        text = "gateway unavailable"
+
+        def json(self):
+            raise ValueError("not json")
+
+    error = requests.HTTPError("502 Bad Gateway")
+    error.response = Response()
+
+    payload = client.event_error_payload(error)
+
+    assert payload.status_code == 502
+    assert payload.detail == "gateway unavailable"
+    assert payload.error == "gateway unavailable"
+
+
 def test_agent_client_context_status_delegates_to_get_status(monkeypatch):
     client = AgentClient(base_url="http://agent.local")
     calls = []

codex_agent_framework-0.1.64/codex_agent/version.py

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