ccs-llmconnector 1.0.2__tar.gz → 1.0.3__tar.gz

{ccs_llmconnector-1.0.2/src/ccs_llmconnector.egg-info → ccs_llmconnector-1.0.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ccs-llmconnector
-Version: 1.0.2
+Version: 1.0.3
 Summary: Lightweight wrapper around different LLM provider Python SDK Responses APIs.
 Author: CCS
 License: MIT

{ccs_llmconnector-1.0.2 → ccs_llmconnector-1.0.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ccs-llmconnector"
-version = "1.0.2"
+version = "1.0.3"
 description = "Lightweight wrapper around different LLM provider Python SDK Responses APIs."
 readme = "README.md"
 requires-python = ">=3.8"

{ccs_llmconnector-1.0.2 → ccs_llmconnector-1.0.3/src/ccs_llmconnector.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ccs-llmconnector
-Version: 1.0.2
+Version: 1.0.3
 Summary: Lightweight wrapper around different LLM provider Python SDK Responses APIs.
 Author: CCS
 License: MIT

{ccs_llmconnector-1.0.2 → ccs_llmconnector-1.0.3}/src/llmconnector/anthropic_client.py

@@ -5,6 +5,7 @@ from __future__ import annotations
 import base64
 import mimetypes
 from pathlib import Path
+import logging
 from typing import Optional, Sequence, Union
 from urllib.error import URLError
 from urllib.request import urlopen
@@ -12,6 +13,7 @@ from urllib.request import urlopen
 from anthropic import APIError, Anthropic
 
 ImageInput = Union[str, Path]
+logger = logging.getLogger(__name__)
 
 
 class AnthropicClient:
@@ -65,11 +67,15 @@ class AnthropicClient:
 
         client = Anthropic(api_key=api_key)
 
+        try:
         response = client.messages.create(
             model=model,
             max_tokens=max_tokens,
             messages=[{"role": "user", "content": content_blocks}],
         )
+        except Exception as exc:
+            logger.exception("Anthropic messages.create failed: %s", exc)
+            raise
 
         text_blocks: list[str] = []
         for block in getattr(response, "content", []) or []:
@@ -79,9 +85,23 @@ class AnthropicClient:
                     text_blocks.append(text)
 
         if text_blocks:
-            return "".join(text_blocks)
-
-        raise RuntimeError("Anthropic response did not include any text output.")
+            result_text = "".join(text_blocks)
+            logger.info(
+                "Anthropic messages.create succeeded: model=%s images=%d text_len=%d",
+                model,
+                len(images or []),
+                len(result_text or ""),
+            )
+            return result_text
+
+        # Treat successful calls without textual content as a successful, empty response
+        # rather than raising. This aligns with callers that handle empty outputs gracefully.
+        logger.info(
+            "Anthropic messages.create succeeded with no text: model=%s images=%d",
+            model,
+            len(images or []),
+        )
+        return ""
 
     def list_models(self, *, api_key: str) -> list[dict[str, Optional[str]]]:
         """Return the models available to the authenticated Anthropic account."""
@@ -91,7 +111,13 @@ class AnthropicClient:
         client = Anthropic(api_key=api_key)
         models: list[dict[str, Optional[str]]] = []
 
-        for model in client.models.list():
+        try:
+            iterator = client.models.list()
+        except Exception as exc:
+            logger.exception("Anthropic list models failed: %s", exc)
+            raise
+
+        for model in iterator:
             model_id = getattr(model, "id", None)
             if model_id is None and isinstance(model, dict):
                 model_id = model.get("id")
@@ -104,6 +130,7 @@ class AnthropicClient:
 
             models.append({"id": model_id, "display_name": display_name})
 
+        logger.info("Anthropic list_models succeeded: count=%d", len(models))
         return models
 
     @staticmethod

{ccs_llmconnector-1.0.2 → ccs_llmconnector-1.0.3}/src/llmconnector/gemini_client.py

@@ -5,6 +5,7 @@ from __future__ import annotations
 import base64
 import mimetypes
 from pathlib import Path
+import logging
 from typing import Optional, Sequence, Union
 from urllib.error import URLError
 from urllib.request import urlopen
@@ -13,6 +14,7 @@ from google import genai
 from google.genai import types
 
 ImageInput = Union[str, Path]
+logger = logging.getLogger(__name__)
 
 
 class GeminiClient:
@@ -72,11 +74,15 @@ class GeminiClient:
 
         client = genai.Client(api_key=api_key)
         try:
-            response = client.models.generate_content(
-                model=model,
-                contents=[content],
-                config=config,
-            )
+            try:
+                response = client.models.generate_content(
+                    model=model,
+                    contents=[content],
+                    config=config,
+                )
+            except Exception as exc:
+                logger.exception("Gemini generate_content failed: %s", exc)
+                raise
         finally:
             closer = getattr(client, "close", None)
             if callable(closer):
@@ -86,7 +92,14 @@ class GeminiClient:
                     pass
 
         if response.text:
-            return response.text
+            result_text = response.text
+            logger.info(
+                "Gemini generate_content succeeded: model=%s images=%d text_len=%d",
+                model,
+                len(images or []),
+                len(result_text or ""),
+            )
+            return result_text
 
         candidate_texts: list[str] = []
         for candidate in getattr(response, "candidates", []) or []:
@@ -99,9 +112,23 @@ class GeminiClient:
                     candidate_texts.append(text)
 
         if candidate_texts:
-            return "\n".join(candidate_texts)
+            result_text = "\n".join(candidate_texts)
+            logger.info(
+                "Gemini generate_content succeeded (candidates): model=%s images=%d text_len=%d",
+                model,
+                len(images or []),
+                len(result_text or ""),
+            )
+            return result_text
 
-        raise RuntimeError("Gemini response did not include any text output.")
+        # Treat successful calls without textual content as a successful, empty response
+        # rather than raising. This aligns with callers that handle empty outputs gracefully.
+        logger.info(
+            "Gemini generate_content succeeded with no text: model=%s images=%d",
+            model,
+            len(images or []),
+        )
+        return ""
 
     def list_models(self, *, api_key: str) -> list[dict[str, Optional[str]]]:
         """Return the models available to the authenticated Gemini account."""
@@ -111,7 +138,12 @@ class GeminiClient:
         models: list[dict[str, Optional[str]]] = []
         client = genai.Client(api_key=api_key)
         try:
-            for model in client.models.list():
+            try:
+                iterator = client.models.list()
+            except Exception as exc:
+                logger.exception("Gemini list models failed: %s", exc)
+                raise
+            for model in iterator:
                 model_id = getattr(model, "name", None)
                 if model_id is None and isinstance(model, dict):
                     model_id = model.get("name")
@@ -135,6 +167,7 @@ class GeminiClient:
                 except Exception:
                     pass
 
+        logger.info("Gemini list_models succeeded: count=%d", len(models))
         return models
 
     @staticmethod

{ccs_llmconnector-1.0.2 → ccs_llmconnector-1.0.3}/src/llmconnector/grok_client.py

@@ -5,6 +5,7 @@ from __future__ import annotations
 import base64
 import mimetypes
 from pathlib import Path
+import logging
 from typing import Optional, Sequence, Union
 
 from xai_sdk import Client
@@ -12,6 +13,7 @@ from xai_sdk.chat import image as chat_image
 from xai_sdk.chat import user
 
 ImageInput = Union[str, Path]
+logger = logging.getLogger(__name__)
 
 
 class GrokClient:
@@ -74,18 +76,41 @@ class GrokClient:
         if normalized_effort in {"low", "high"}:
             create_kwargs["reasoning_effort"] = normalized_effort
 
-        chat = grok_client.chat.create(**create_kwargs)
-        response = chat.sample()
+        try:
+            chat = grok_client.chat.create(**create_kwargs)
+            response = chat.sample()
+        except Exception as exc:
+            logger.exception("xAI Grok chat request failed: %s", exc)
+            raise
 
         content = getattr(response, "content", None)
         if content:
+            logger.info(
+                "xAI chat succeeded: model=%s images=%d text_len=%d",
+                model,
+                len(images or []),
+                len(content or ""),
+            )
             return content
 
         reasoning_content = getattr(response, "reasoning_content", None)
         if reasoning_content:
+            logger.info(
+                "xAI chat succeeded (reasoning): model=%s images=%d text_len=%d",
+                model,
+                len(images or []),
+                len(reasoning_content or ""),
+            )
             return reasoning_content
 
-        raise RuntimeError("xAI response did not include any text output.")
+        # Treat successful calls without textual content as a successful, empty response
+        # rather than raising. This aligns with callers that handle empty outputs gracefully.
+        logger.info(
+            "xAI chat succeeded with no text: model=%s images=%d",
+            model,
+            len(images or []),
+        )
+        return ""
 
     def list_models(self, *, api_key: str) -> list[dict[str, Optional[str]]]:
         """Return the Grok language models available to the authenticated account."""
@@ -95,7 +120,13 @@ class GrokClient:
         grok_client = Client(api_key=api_key)
         models: list[dict[str, Optional[str]]] = []
 
-        for model in grok_client.models.list_language_models():
+        try:
+            iterator = grok_client.models.list_language_models()
+        except Exception as exc:
+            logger.exception("xAI list language models failed: %s", exc)
+            raise
+
+        for model in iterator:
             model_id = getattr(model, "name", None)
             if not model_id:
                 continue
@@ -115,6 +146,7 @@ class GrokClient:
                 }
             )
 
+        logger.info("xAI list_language_models succeeded: count=%d", len(models))
         return models
 
     @staticmethod
@@ -136,4 +168,3 @@ def _encode_image_path(path: Path) -> str:
     mime_type = mimetypes.guess_type(expanded.name)[0] or "application/octet-stream"
     encoded = base64.b64encode(data).decode("utf-8")
     return f"data:{mime_type};base64,{encoded}"
-

{ccs_llmconnector-1.0.2 → ccs_llmconnector-1.0.3}/src/llmconnector/openai_client.py

@@ -4,7 +4,8 @@ from __future__ import annotations
 
 import base64
 import mimetypes
-from pathlib import Path
+from pathlib import Path
+import logging
 from typing import Optional, Sequence, Union
 
 from openai import OpenAI
@@ -14,7 +15,8 @@ try:
 except ImportError:  # pragma: no cover - fallback for older SDKs
     from openai.error import OpenAIError  # type: ignore
 
-ImageInput = Union[str, Path]
+ImageInput = Union[str, Path]
+logger = logging.getLogger(__name__)
 
 
 class OpenAIResponsesClient:
@@ -81,9 +83,20 @@ class OpenAIResponsesClient:
         if reasoning_effort:
             request_payload["reasoning"] = {"effort": reasoning_effort}
 
-        response = client.responses.create(**request_payload)
-
-        return response.output_text
+        try:
+            response = client.responses.create(**request_payload)
+        except Exception as exc:  # Log and re-raise to preserve default behavior
+            logger.exception("OpenAI Responses API request failed: %s", exc)
+            raise
+
+        output_text = response.output_text
+        logger.info(
+            "OpenAI generate_response succeeded: model=%s images=%d text_len=%d",
+            model,
+            len(images or []),
+            len(output_text or ""),
+        )
+        return output_text
 
     @staticmethod
     def _to_image_block(image: ImageInput) -> dict:
@@ -111,7 +124,11 @@ class OpenAIResponsesClient:
             raise ValueError("api_key must be provided.")
 
         client = OpenAI(api_key=api_key)
-        response = client.models.list()
+        try:
+            response = client.models.list()
+        except Exception as exc:
+            logger.exception("OpenAI list models failed: %s", exc)
+            raise
         data = getattr(response, "data", []) or []
 
         models: list[dict[str, Optional[str]]] = []
@@ -128,6 +145,7 @@ class OpenAIResponsesClient:
 
             models.append({"id": model_id, "display_name": display_name})
 
+        logger.info("OpenAI list_models succeeded: count=%d", len(models))
         return models