PyPI - confamnode - Versions diffs - 0.2.4__tar.gz → 0.2.5__tar.gz - Mend

confamnode 0.2.4tar.gz → 0.2.5tar.gz

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (28) hide show

{confamnode-0.2.4 → confamnode-0.2.5}/PKG-INFO RENAMED Viewed

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: confamnode
-Version: 0.2.4
+Version: 0.2.5
 Summary: The Nigerian AI inference gateway
 Project-URL: Repository, https://github.com/confamnodeai/confamnode
 Project-URL: Bug Tracker, https://github.com/confamnodeai/confamnode/issues

{confamnode-0.2.4 → confamnode-0.2.5}/confamnode/__init__.py RENAMED Viewed

@@ -8,7 +8,7 @@ from confamnode.exceptions import (
 from confamnode.ansa import Ansa, Usage, Cost
 from confamnode import models
-__version__ = "0.2.4"
+__version__ = "0.2.5"
 __all__ = [
     "ConfamNode",

confamnode-0.2.5/confamnode/utils.py ADDED Viewed

@@ -0,0 +1,48 @@
+"""
+Small internal helpers shared across the confamnode client.
+"""
+import re
+def extract_error(response) -> str:
+    """
+    Best-effort error detail from a non-2xx response, WITHOUT raising.
+    The server's normal error shape is {"detail": "..."}, but gateway-level
+    errors (e.g. a 429 throttle, a 502/504 from a proxy, a 524 origin timeout)
+    often return an empty or non-JSON body. Calling response.json() directly on
+    those bodies raises JSONDecodeError and swallows the status code -- the one
+    thing the caller actually needs. So we parse defensively and always fall
+    back to something readable.
+    HTML error pages (Cloudflare, nginx, ...) are summarised to their <title>
+    rather than dumped in full, so error messages and logs stay readable.
+    """
+    # 1. Normal JSON error shape.
+    try:
+        payload = response.json()
+        if isinstance(payload, dict):
+            detail = payload.get("detail") or payload.get("error") or payload.get("message")
+            if detail:
+                return str(detail)
+    except Exception:
+        pass
+    text = (getattr(response, "text", "") or "").strip()
+    if not text:
+        return  "no response body"
+    # 2. HTML error page: the full page is noise. Pull the <title>, which
+    #    carries the human-readable status (e.g. "524: A timeout occurred").
+    if "<html" in text[:200].lower() or "<!doctype html" in text[:200].lower():
+        m = re.search(r"<title[^>]*>(.*?)</title>", text, re.IGNORECASE | re.DOTALL)
+        if m:
+            title = re.sub(r"\s+", " ", m.group(1)).strip()
+            if title:
+                return title
+        return "HTML error page (no title)"
+    # 3. Plain-text body: return it, truncated so we never dump a wall.
+    return text if len(text) <= 300 else text[:297] + "..."

{confamnode-0.2.4 → confamnode-0.2.5}/pyproject.toml RENAMED Viewed

@@ -1,6 +1,6 @@
 [project]
 name = "confamnode"
-version = "0.2.4"
+version = "0.2.5"
 description = "The Nigerian AI inference gateway"
 readme = "README.md"
 requires-python = ">=3.10"

{confamnode-0.2.4 → confamnode-0.2.5}/tests/test_client_errors.py RENAMED Viewed

@@ -58,14 +58,20 @@ def test_json_detail_error_surfaces_detail(monkeypatch):
     assert "invalid model" in str(exc.value)
-def test_html_gateway_error_surfaces_text(monkeypatch):
-    _patch_post(monkeypatch, FakeResponse(502, json_raises=True,
-                                          text="<html>502 Bad Gateway</html>"))
+def test_html_gateway_error_surfaces_title(monkeypatch):
+    html = (
+        "<!DOCTYPE html><html><head>"
+        "<title>confamnode.com | 524: A timeout occurred</title>"
+        "</head><body>...</body></html>"
+    )
+    _patch_post(monkeypatch, FakeResponse(524, json_raises=True, text=html))
     client = ConfamNode(api_key=API_KEY)
     with pytest.raises(Exception) as exc:
         client.gist(model=MODEL, messages="hi")
-    assert "502" in str(exc.value)
-    assert "Bad Gateway" in str(exc.value)
+    msg = str(exc.value)
+    assert "524" in msg
+    assert "A timeout occurred" in msg
+    assert "<html" not in msg   # the page itself must NOT be dumped into the error
 def test_success_path_still_returns_ansa(monkeypatch):

{confamnode-0.2.4 → confamnode-0.2.5}/tests/test_utils.py RENAMED Viewed

@@ -29,6 +29,27 @@ def test_non_json_text_body_is_returned_stripped():
     assert extract_error(_Resp(json_raises=True, text="  upstream timeout  ")) == "upstream timeout"
+def test_html_error_page_summarised_to_title():
+    # A Cloudflare 524 page: don't dump the whole page, surface the <title>.
+    html = (
+        "<!DOCTYPE html><html><head>"
+        "<title>confamnode.com | 524: A timeout occurred</title>"
+        "</head><body>...lots of markup...</body></html>"
+    )
+    assert extract_error(_Resp(json_raises=True, text=html)) == "confamnode.com | 524: A timeout occurred"
+def test_html_without_title_falls_back():
+    html = "<html><body>502 Bad Gateway</body></html>"
+    assert extract_error(_Resp(json_raises=True, text=html)) == "HTML error page (no title)"
+def test_long_plain_text_is_truncated():
+    body = "x" * 500
+    out = extract_error(_Resp(json_raises=True, text=body))
+    assert len(out) == 300 and out.endswith("...")
 def test_detail_key_is_preferred():
     assert extract_error(_Resp(payload={"detail": "invalid model"})) == "invalid model"

{confamnode-0.2.4 → confamnode-0.2.5}/uv.lock RENAMED Viewed

@@ -40,7 +40,7 @@ wheels = [
 [[package]]
 name = "confamnode"
-version = "0.2.3"
+version = "0.2.4"
 source = { editable = "." }
 dependencies = [
     { name = "httpx" },

confamnode-0.2.4/confamnode/utils.py DELETED Viewed

@@ -1,26 +0,0 @@
-"""
-Small internal helpers shared across the confamnode client.
-"""
-def extract_error(response) -> str:
-    """
-    Best-effort error detail from a non-2xx response, WITHOUT raising.
-    The server's normal error shape is {"detail": "..."}, but gateway-level
-    errors (e.g. a 429 throttle, a 502/504 from a proxy) often return an
-    empty or non-JSON body. Calling response.json() directly on those bodies
-    raises JSONDecodeError and swallows the status code -- the one thing the
-    caller actually needs. So we parse defensively and always fall back to
-    the raw text, then to a status-only message.
-    """
-    try:
-        payload = response.json()
-        if isinstance(payload, dict):
-            detail = payload.get("detail") or payload.get("error") or payload.get("message")
-            if detail:
-                return str(detail)
-    except Exception:
-        pass
-    text = (getattr(response, "text", "") or "").strip()
-    return text or "no response body"