nocfo-cli 1.3.0__tar.gz → 1.4.0__tar.gz

{nocfo_cli-1.3.0 → nocfo_cli-1.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nocfo-cli
-Version: 1.3.0
+Version: 1.4.0
 Summary: NoCFO CLI, MCP server, and Cursor skill toolkit.
 License: MIT
 License-File: LICENSE

{nocfo_cli-1.3.0 → nocfo_cli-1.4.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
 name = "nocfo-cli"
-version = "1.3.0"
+version = "1.4.0"
 description = "NoCFO CLI, MCP server, and Cursor skill toolkit."
 authors = ["NoCFO"]
 readme = "README.md"

{nocfo_cli-1.3.0 → nocfo_cli-1.4.0}/src/nocfo_toolkit/mcp/http_error_capture.py

@@ -21,21 +21,31 @@ def get_last_http_error() -> dict[str, Any] | None:
 
 
 async def capture_http_error_response(response: httpx.Response) -> None:
-    """Store status/payload for the latest non-success HTTP response."""
+    """Store status/payload for the latest HTTP response.
 
-    if response.status_code < 400:
-        _LAST_HTTP_ERROR.set(None)
-        return
+    FastMCP can issue downstream requests with streaming enabled. In that mode,
+    accessing ``response.content``/``response.json`` without ``aread()`` raises
+    ``ResponseNotRead``. We therefore always drain the body first, even for
+    success responses, so tool handlers can safely decode the payload.
+    """
 
     try:
         await response.aread()
-    except httpx.HTTPError:
-        pass
+    except (httpx.HTTPError, httpx.StreamError):
+        # Keep best-effort behavior: failures here should not shadow the original
+        # tool error path. The middleware still clears stale captured state below.
+        _LAST_HTTP_ERROR.set(None)
+        return
+
+    if response.status_code < 400:
+        _LAST_HTTP_ERROR.set(None)
+        return
 
     try:
         payload: Any = response.json() if response.content else None
-    except (ValueError, httpx.ResponseNotRead):
-        payload = (response.text or "").strip()[:1000] or None
+    except ValueError:
+        text = response.text.strip()
+        payload = text[:1000] if text else None
 
     _LAST_HTTP_ERROR.set(
         {

{nocfo_cli-1.3.0 → nocfo_cli-1.4.0}/src/nocfo_toolkit/mcp/server.py

@@ -200,6 +200,7 @@ def create_server(
     from fastmcp import FastMCP
 
     opts = options or MCPServerOptions()
+
     if opts.auth_mode == "oauth":
         client = _create_oauth_client(
             config,