xai-review 0.34.0__py3-none-any.whl → 0.36.0__py3-none-any.whl

ai_review/clients/claude/client.py

@@ -17,7 +17,7 @@ class ClaudeHTTPClientError(HTTPClientError):
 class ClaudeHTTPClient(HTTPClient, ClaudeHTTPClientProtocol):
     @handle_http_error(client="ClaudeHTTPClient", exception=ClaudeHTTPClientError)
     async def chat_api(self, request: ClaudeChatRequestSchema) -> Response:
-        return await self.post("/v1/messages", json=request.model_dump())
+        return await self.post("/v1/messages", json=request.model_dump(exclude_none=True))
 
     async def chat(self, request: ClaudeChatRequestSchema) -> ClaudeChatResponseSchema:
         response = await self.chat_api(request)

ai_review/clients/claude/schema.py

@@ -12,8 +12,8 @@ class ClaudeChatRequestSchema(BaseModel):
     model: str
     system: str | None = None
     messages: list[ClaudeMessageSchema]
-    max_tokens: int
-    temperature: float
+    max_tokens: int | None = None
+    temperature: float | None = None
 
 
 class ClaudeContentSchema(BaseModel):

ai_review/clients/gemini/client.py

@@ -19,7 +19,8 @@ class GeminiHTTPClient(HTTPClient, GeminiHTTPClientProtocol):
     async def chat_api(self, request: GeminiChatRequestSchema) -> Response:
         meta = settings.llm.meta
         return await self.post(
-            f"/v1beta/models/{meta.model}:generateContent", json=request.model_dump()
+            f"/v1beta/models/{meta.model}:generateContent",
+            json=request.model_dump(exclude_none=True)
         )
 
     async def chat(self, request: GeminiChatRequestSchema) -> GeminiChatResponseSchema:

ai_review/clients/gemini/schema.py

@@ -45,8 +45,8 @@ class GeminiCandidateSchema(BaseModel):
 class GeminiGenerationConfigSchema(BaseModel):
     model_config = ConfigDict(populate_by_name=True)
 
-    temperature: float
-    max_output_tokens: int = Field(alias="maxOutputTokens")
+    temperature: float | None = None
+    max_output_tokens: int | None = Field(alias="maxOutputTokens", default=None)
 
 
 class GeminiChatRequestSchema(BaseModel):

ai_review/clients/ollama/client.py

@@ -17,7 +17,7 @@ class OllamaHTTPClientError(HTTPClientError):
 class OllamaHTTPClient(HTTPClient, OllamaHTTPClientProtocol):
     @handle_http_error(client="OllamaHTTPClient", exception=OllamaHTTPClientError)
     async def chat_api(self, request: OllamaChatRequestSchema) -> Response:
-        return await self.post("/api/chat", json=request.model_dump())
+        return await self.post("/api/chat", json=request.model_dump(exclude_none=True))
 
     async def chat(self, request: OllamaChatRequestSchema) -> OllamaChatResponseSchema:
         response = await self.chat_api(request)

ai_review/clients/openai/{client.py → v1/client.py}

@@ -1,7 +1,7 @@
 from httpx import Response, AsyncHTTPTransport, AsyncClient
 
-from ai_review.clients.openai.schema import OpenAIChatRequestSchema, OpenAIChatResponseSchema
-from ai_review.clients.openai.types import OpenAIHTTPClientProtocol
+from ai_review.clients.openai.v1.schema import OpenAIChatRequestSchema, OpenAIChatResponseSchema
+from ai_review.clients.openai.v1.types import OpenAIV1HTTPClientProtocol
 from ai_review.config import settings
 from ai_review.libs.http.client import HTTPClient
 from ai_review.libs.http.event_hooks.logger import LoggerEventHook
@@ -10,22 +10,22 @@ from ai_review.libs.http.transports.retry import RetryTransport
 from ai_review.libs.logger import get_logger
 
 
-class OpenAIHTTPClientError(HTTPClientError):
+class OpenAIV1HTTPClientError(HTTPClientError):
     pass
 
 
-class OpenAIHTTPClient(HTTPClient, OpenAIHTTPClientProtocol):
-    @handle_http_error(client='OpenAIHTTPClient', exception=OpenAIHTTPClientError)
+class OpenAIV1HTTPClient(HTTPClient, OpenAIV1HTTPClientProtocol):
+    @handle_http_error(client='OpenAIV1HTTPClient', exception=OpenAIV1HTTPClientError)
     async def chat_api(self, request: OpenAIChatRequestSchema) -> Response:
-        return await self.post("/chat/completions", json=request.model_dump())
+        return await self.post("/chat/completions", json=request.model_dump(exclude_none=True))
 
     async def chat(self, request: OpenAIChatRequestSchema) -> OpenAIChatResponseSchema:
         response = await self.chat_api(request)
         return OpenAIChatResponseSchema.model_validate_json(response.text)
 
 
-def get_openai_http_client() -> OpenAIHTTPClient:
-    logger = get_logger("OPENAI_HTTP_CLIENT")
+def get_openai_v1_http_client() -> OpenAIV1HTTPClient:
+    logger = get_logger("OPENAI_V1_HTTP_CLIENT")
     logger_event_hook = LoggerEventHook(logger=logger)
     retry_transport = RetryTransport(logger=logger, transport=AsyncHTTPTransport())
 
@@ -40,4 +40,4 @@ def get_openai_http_client() -> OpenAIHTTPClient:
         }
     )
 
-    return OpenAIHTTPClient(client=client)
+    return OpenAIV1HTTPClient(client=client)

ai_review/clients/openai/{schema.py → v1/schema.py}

@@ -21,8 +21,8 @@ class OpenAIChoiceSchema(BaseModel):
 class OpenAIChatRequestSchema(BaseModel):
     model: str
     messages: list[OpenAIMessageSchema]
-    max_tokens: int
-    temperature: float
+    max_tokens: int | None = None
+    temperature: float | None = None
 
 
 class OpenAIChatResponseSchema(BaseModel):

ai_review/clients/openai/v1/types.py

@@ -0,0 +1,8 @@
+from typing import Protocol
+
+from ai_review.clients.openai.v1.schema import OpenAIChatRequestSchema, OpenAIChatResponseSchema
+
+
+class OpenAIV1HTTPClientProtocol(Protocol):
+    async def chat(self, request: OpenAIChatRequestSchema) -> OpenAIChatResponseSchema:
+        ...

ai_review/clients/openai/v2/client.py

@@ -0,0 +1,46 @@
+from httpx import Response, AsyncClient, AsyncHTTPTransport
+
+from ai_review.clients.openai.v2.schema import (
+    OpenAIResponsesRequestSchema,
+    OpenAIResponsesResponseSchema
+)
+from ai_review.clients.openai.v2.types import OpenAIV2HTTPClientProtocol
+from ai_review.config import settings
+from ai_review.libs.http.client import HTTPClient
+from ai_review.libs.http.event_hooks.logger import LoggerEventHook
+from ai_review.libs.http.handlers import HTTPClientError, handle_http_error
+from ai_review.libs.http.transports.retry import RetryTransport
+from ai_review.libs.logger import get_logger
+
+
+class OpenAIV2HTTPClientError(HTTPClientError):
+    pass
+
+
+class OpenAIV2HTTPClient(HTTPClient, OpenAIV2HTTPClientProtocol):
+    @handle_http_error(client='OpenAIV2HTTPClient', exception=OpenAIV2HTTPClientError)
+    async def chat_api(self, request: OpenAIResponsesRequestSchema) -> Response:
+        return await self.post("/responses", json=request.model_dump(exclude_none=True))
+
+    async def chat(self, request: OpenAIResponsesRequestSchema) -> OpenAIResponsesResponseSchema:
+        response = await self.chat_api(request)
+        return OpenAIResponsesResponseSchema.model_validate_json(response.text)
+
+
+def get_openai_v2_http_client() -> OpenAIV2HTTPClient:
+    logger = get_logger("OPENAI_V2_HTTP_CLIENT")
+    logger_event_hook = LoggerEventHook(logger=logger)
+    retry_transport = RetryTransport(logger=logger, transport=AsyncHTTPTransport())
+
+    client = AsyncClient(
+        timeout=settings.llm.http_client.timeout,
+        headers={"Authorization": f"Bearer {settings.llm.http_client.api_token_value}"},
+        base_url=settings.llm.http_client.api_url_value,
+        transport=retry_transport,
+        event_hooks={
+            'request': [logger_event_hook.request],
+            'response': [logger_event_hook.response]
+        }
+    )
+
+    return OpenAIV2HTTPClient(client=client)

ai_review/clients/openai/v2/schema.py

@@ -0,0 +1,47 @@
+from pydantic import BaseModel
+
+
+class OpenAIResponseUsageSchema(BaseModel):
+    total_tokens: int
+    input_tokens: int
+    output_tokens: int
+
+
+class OpenAIInputMessageSchema(BaseModel):
+    role: str
+    content: str
+
+
+class OpenAIResponseContentSchema(BaseModel):
+    type: str
+    text: str | None = None
+
+
+class OpenAIResponseOutputSchema(BaseModel):
+    type: str
+    role: str | None = None
+    content: list[OpenAIResponseContentSchema] | None = None
+
+
+class OpenAIResponsesRequestSchema(BaseModel):
+    model: str
+    input: list[OpenAIInputMessageSchema]
+    temperature: float | None = None
+    instructions: str | None = None
+    max_output_tokens: int | None = None
+
+
+class OpenAIResponsesResponseSchema(BaseModel):
+    usage: OpenAIResponseUsageSchema
+    output: list[OpenAIResponseOutputSchema]
+
+    @property
+    def first_text(self) -> str:
+        results: list[str] = []
+        for block in self.output:
+            if block.type == "message" and block.content:
+                for content in block.content:
+                    if content.type == "output_text" and content.text:
+                        results.append(content.text)
+
+        return "".join(results).strip()

ai_review/clients/openai/v2/types.py

@@ -0,0 +1,11 @@
+from typing import Protocol
+
+from ai_review.clients.openai.v2.schema import (
+    OpenAIResponsesRequestSchema,
+    OpenAIResponsesResponseSchema
+)
+
+
+class OpenAIV2HTTPClientProtocol(Protocol):
+    async def chat(self, request: OpenAIResponsesRequestSchema) -> OpenAIResponsesResponseSchema:
+        ...

ai_review/clients/openrouter/client.py

@@ -17,7 +17,7 @@ class OpenRouterHTTPClientError(HTTPClientError):
 class OpenRouterHTTPClient(HTTPClient, OpenRouterHTTPClientProtocol):
     @handle_http_error(client="OpenRouterHTTPClient", exception=OpenRouterHTTPClientError)
     async def chat_api(self, request: OpenRouterChatRequestSchema) -> Response:
-        return await self.post("/chat/completions", json=request.model_dump())
+        return await self.post("/chat/completions", json=request.model_dump(exclude_none=True))
 
     async def chat(self, request: OpenRouterChatRequestSchema) -> OpenRouterChatResponseSchema:
         response = await self.chat_api(request)

ai_review/clients/openrouter/schema.py

@@ -21,8 +21,8 @@ class OpenRouterChoiceSchema(BaseModel):
 class OpenRouterChatRequestSchema(BaseModel):
     model: str
     messages: list[OpenRouterMessageSchema]
-    max_tokens: int
-    temperature: float
+    max_tokens: int | None = None
+    temperature: float | None = None
 
 
 class OpenRouterChatResponseSchema(BaseModel):

ai_review/libs/config/llm/meta.py

@@ -3,5 +3,5 @@ from pydantic import BaseModel, Field
 
 class LLMMetaConfig(BaseModel):
     model: str
-    max_tokens: int = Field(default=5000, ge=1)
-    temperature: float = Field(default=0.3, ge=0.0, le=2.0)
+    max_tokens: int | None = Field(default=None, ge=1)
+    temperature: float | None = Field(default=None, ge=0.0, le=2.0)

ai_review/libs/config/llm/openai.py

@@ -5,6 +5,10 @@ from ai_review.libs.config.llm.meta import LLMMetaConfig
 class OpenAIMetaConfig(LLMMetaConfig):
     model: str = "gpt-4o-mini"
 
+    @property
+    def is_v2_model(self) -> bool:
+        return any(self.model.startswith(model) for model in ("gpt-5", "gpt-4.1"))
+
 
 class OpenAIHTTPClientConfig(HTTPClientWithTokenConfig):
     pass

ai_review/resources/pricing.yaml

@@ -1,3 +1,14 @@
+# ===============================
+# 📊 Pricing per 1 token (USD)
+# ===============================
+# NOTE:
+# - Prices are per 1 token and can change over time.
+# - For the latest OpenAI and Anthropic pricing, see:
+#   - https://openai.com/api/pricing
+#   - https://www.anthropic.com/pricing
+# - For OpenRouter models, prices vary by provider: https://openrouter.ai
+
+# --- OpenAI ---
 gpt-4o-mini:
   input: 0.15e-6
   output: 0.60e-6
@@ -14,17 +25,22 @@ gpt-4.1:
   input: 5.00e-6
   output: 15.00e-6
 
+gpt-5:
+  input: 10.00e-6
+  output: 30.00e-6
+
 gpt-3.5-turbo:
   input: 0.50e-6
   output: 1.50e-6
 
+# --- Google Gemini ---
 gemini-2.5-flash-lite:
   input: 0.10e-6
   output: 0.40e-6
 
 gemini-2.0-flash-lite:
   input: 0.019e-6
-  output: 0.1e-6
+  output: 0.10e-6
 
 gemini-2.0-pro:
   input: 0.125e-6
@@ -38,6 +54,7 @@ gemini-2.5-pro-long-context:
   input: 2.50e-6
   output: 15.00e-6
 
+# --- Anthropic Claude ---
 claude-3.5-sonnet:
   input: 3.00e-6
   output: 15.00e-6
@@ -53,3 +70,24 @@ claude-3-sonnet:
 claude-3-haiku:
   input: 0.25e-6
   output: 1.25e-6
+
+# --- OpenRouter ---
+o3-mini:
+  input: 1.10e-6
+  output: 4.40e-6
+
+openai/chatgpt-4o-latest:
+  input: 5.00e-6
+  output: 15.00e-6
+
+mistralai/mixtral-8x7b:
+  input: 0.24e-6
+  output: 0.48e-6
+
+meta-llama/llama-3-8b-instruct:
+  input: 0.20e-6
+  output: 0.40e-6
+
+meta-llama/llama-3-70b-instruct:
+  input: 0.80e-6
+  output: 1.60e-6

ai_review/services/llm/openai/client.py

@@ -1,28 +1,56 @@
-from ai_review.clients.openai.client import get_openai_http_client
-from ai_review.clients.openai.schema import OpenAIChatRequestSchema, OpenAIMessageSchema
+from ai_review.clients.openai.v1.client import get_openai_v1_http_client
+from ai_review.clients.openai.v1.schema import OpenAIChatRequestSchema, OpenAIMessageSchema
+from ai_review.clients.openai.v2.client import get_openai_v2_http_client
+from ai_review.clients.openai.v2.schema import OpenAIInputMessageSchema, OpenAIResponsesRequestSchema
 from ai_review.config import settings
 from ai_review.services.llm.types import LLMClientProtocol, ChatResultSchema
 
 
 class OpenAILLMClient(LLMClientProtocol):
     def __init__(self):
-        self.http_client = get_openai_http_client()
+        self.meta = settings.llm.meta
 
-    async def chat(self, prompt: str, prompt_system: str) -> ChatResultSchema:
-        meta = settings.llm.meta
+        self.http_client_v1 = get_openai_v1_http_client()
+        self.http_client_v2 = get_openai_v2_http_client()
+
+    async def chat_v1(self, prompt: str, prompt_system: str) -> ChatResultSchema:
         request = OpenAIChatRequestSchema(
-            model=meta.model,
+            model=self.meta.model,
             messages=[
                 OpenAIMessageSchema(role="system", content=prompt_system),
                 OpenAIMessageSchema(role="user", content=prompt),
             ],
-            max_tokens=meta.max_tokens,
-            temperature=meta.temperature,
+            max_tokens=self.meta.max_tokens,
+            temperature=self.meta.temperature,
         )
-        response = await self.http_client.chat(request)
+        response = await self.http_client_v1.chat(request)
         return ChatResultSchema(
             text=response.first_text,
             total_tokens=response.usage.total_tokens,
             prompt_tokens=response.usage.prompt_tokens,
             completion_tokens=response.usage.completion_tokens,
         )
+
+    async def chat_v2(self, prompt: str, prompt_system: str) -> ChatResultSchema:
+        request = OpenAIResponsesRequestSchema(
+            model=self.meta.model,
+            input=[
+                OpenAIInputMessageSchema(role="system", content=prompt_system),
+                OpenAIInputMessageSchema(role="user", content=prompt),
+            ],
+            temperature=self.meta.temperature,
+            max_output_tokens=self.meta.max_tokens,
+        )
+        response = await self.http_client_v2.chat(request)
+        return ChatResultSchema(
+            text=response.first_text,
+            total_tokens=response.usage.total_tokens,
+            prompt_tokens=response.usage.input_tokens,
+            completion_tokens=response.usage.output_tokens,
+        )
+
+    async def chat(self, prompt: str, prompt_system: str) -> ChatResultSchema:
+        if self.meta.is_v2_model:
+            return await self.chat_v2(prompt, prompt_system)
+
+        return await self.chat_v1(prompt, prompt_system)

ai_review/tests/fixtures/clients/openai.py

@@ -3,14 +3,22 @@ from typing import Any
 import pytest
 from pydantic import HttpUrl, SecretStr
 
-from ai_review.clients.openai.schema import (
+from ai_review.clients.openai.v1.schema import (
     OpenAIUsageSchema,
     OpenAIChoiceSchema,
     OpenAIMessageSchema,
     OpenAIChatRequestSchema,
     OpenAIChatResponseSchema,
 )
-from ai_review.clients.openai.types import OpenAIHTTPClientProtocol
+from ai_review.clients.openai.v1.types import OpenAIV1HTTPClientProtocol
+from ai_review.clients.openai.v2.schema import (
+    OpenAIResponsesRequestSchema,
+    OpenAIResponsesResponseSchema,
+    OpenAIResponseUsageSchema,
+    OpenAIResponseOutputSchema,
+    OpenAIResponseContentSchema,
+)
+from ai_review.clients.openai.v2.types import OpenAIV2HTTPClientProtocol
 from ai_review.config import settings
 from ai_review.libs.config.llm.base import OpenAILLMConfig
 from ai_review.libs.config.llm.openai import OpenAIMetaConfig, OpenAIHTTPClientConfig
@@ -18,7 +26,7 @@ from ai_review.libs.constants.llm_provider import LLMProvider
 from ai_review.services.llm.openai.client import OpenAILLMClient
 
 
-class FakeOpenAIHTTPClient(OpenAIHTTPClientProtocol):
+class FakeOpenAIV1HTTPClient(OpenAIV1HTTPClientProtocol):
     def __init__(self, responses: dict[str, Any] | None = None) -> None:
         self.calls: list[tuple[str, dict]] = []
         self.responses = responses or {}
@@ -31,7 +39,39 @@ class FakeOpenAIHTTPClient(OpenAIHTTPClientProtocol):
                 usage=OpenAIUsageSchema(total_tokens=12, prompt_tokens=5, completion_tokens=7),
                 choices=[
                     OpenAIChoiceSchema(
-                        message=OpenAIMessageSchema(role="assistant", content="FAKE_OPENAI_RESPONSE")
+                        message=OpenAIMessageSchema(
+                            role="assistant",
+                            content="FAKE_OPENAI_V1_RESPONSE"
+                        )
+                    )
+                ],
+            ),
+        )
+
+
+class FakeOpenAIV2HTTPClient(OpenAIV2HTTPClientProtocol):
+    def __init__(self, responses: dict[str, Any] | None = None) -> None:
+        self.calls: list[tuple[str, dict]] = []
+        self.responses = responses or {}
+
+    async def chat(self, request: OpenAIResponsesRequestSchema) -> OpenAIResponsesResponseSchema:
+        self.calls.append(("chat", {"request": request}))
+        return self.responses.get(
+            "chat",
+            OpenAIResponsesResponseSchema(
+                usage=OpenAIResponseUsageSchema(
+                    total_tokens=20, input_tokens=10, output_tokens=10
+                ),
+                output=[
+                    OpenAIResponseOutputSchema(
+                        type="message",
+                        role="assistant",
+                        content=[
+                            OpenAIResponseContentSchema(
+                                type="output_text",
+                                text="FAKE_OPENAI_V2_RESPONSE"
+                            )
+                        ],
                     )
                 ],
             ),
@@ -39,31 +79,63 @@ class FakeOpenAIHTTPClient(OpenAIHTTPClientProtocol):
 
 
 @pytest.fixture
-def fake_openai_http_client() -> FakeOpenAIHTTPClient:
-    return FakeOpenAIHTTPClient()
+def fake_openai_v1_http_client() -> FakeOpenAIV1HTTPClient:
+    return FakeOpenAIV1HTTPClient()
+
+
+@pytest.fixture
+def fake_openai_v2_http_client() -> FakeOpenAIV2HTTPClient:
+    return FakeOpenAIV2HTTPClient()
 
 
 @pytest.fixture
 def openai_llm_client(
         monkeypatch: pytest.MonkeyPatch,
-        fake_openai_http_client: FakeOpenAIHTTPClient
+        fake_openai_v1_http_client: FakeOpenAIV1HTTPClient,
+        fake_openai_v2_http_client: FakeOpenAIV2HTTPClient,
 ) -> OpenAILLMClient:
     monkeypatch.setattr(
-        "ai_review.services.llm.openai.client.get_openai_http_client",
-        lambda: fake_openai_http_client,
+        "ai_review.services.llm.openai.client.get_openai_v1_http_client",
+        lambda: fake_openai_v1_http_client,
+    )
+    monkeypatch.setattr(
+        "ai_review.services.llm.openai.client.get_openai_v2_http_client",
+        lambda: fake_openai_v2_http_client,
     )
     return OpenAILLMClient()
 
 
 @pytest.fixture
-def openai_http_client_config(monkeypatch: pytest.MonkeyPatch):
+def openai_v1_http_client_config(monkeypatch: pytest.MonkeyPatch):
     fake_config = OpenAILLMConfig(
-        meta=OpenAIMetaConfig(),
+        meta=OpenAIMetaConfig(
+            model="gpt-4o-mini",
+            max_tokens=1200,
+            temperature=0.3
+        ),
         provider=LLMProvider.OPENAI,
         http_client=OpenAIHTTPClientConfig(
             timeout=10,
             api_url=HttpUrl("https://api.openai.com/v1"),
             api_token=SecretStr("fake-token"),
-        )
+        ),
+    )
+    monkeypatch.setattr(settings, "llm", fake_config)
+
+
+@pytest.fixture
+def openai_v2_http_client_config(monkeypatch: pytest.MonkeyPatch):
+    fake_config = OpenAILLMConfig(
+        meta=OpenAIMetaConfig(
+            model="gpt-5",
+            max_tokens=2000,
+            temperature=0.2
+        ),
+        provider=LLMProvider.OPENAI,
+        http_client=OpenAIHTTPClientConfig(
+            timeout=10,
+            api_url=HttpUrl("https://api.openai.com/v1"),
+            api_token=SecretStr("fake-token"),
+        ),
     )
     monkeypatch.setattr(settings, "llm", fake_config)

ai_review/tests/suites/clients/openai/v1/test_client.py

@@ -0,0 +1,12 @@
+import pytest
+from httpx import AsyncClient
+
+from ai_review.clients.openai.v1.client import get_openai_v1_http_client, OpenAIV1HTTPClient
+
+
+@pytest.mark.usefixtures('openai_v1_http_client_config')
+def test_get_openai_v1_http_client_builds_ok():
+    openai_http_client = get_openai_v1_http_client()
+
+    assert isinstance(openai_http_client, OpenAIV1HTTPClient)
+    assert isinstance(openai_http_client.client, AsyncClient)

ai_review/tests/suites/clients/openai/{test_schema.py → v1/test_schema.py}

@@ -1,4 +1,4 @@
-from ai_review.clients.openai.schema import (
+from ai_review.clients.openai.v1.schema import (
     OpenAIUsageSchema,
     OpenAIMessageSchema,
     OpenAIChoiceSchema,

ai_review/tests/suites/clients/openai/v2/test_client.py

@@ -0,0 +1,12 @@
+import pytest
+from httpx import AsyncClient
+
+from ai_review.clients.openai.v2.client import get_openai_v2_http_client, OpenAIV2HTTPClient
+
+
+@pytest.mark.usefixtures('openai_v2_http_client_config')
+def test_get_openai_v2_http_client_builds_ok():
+    openai_http_client = get_openai_v2_http_client()
+
+    assert isinstance(openai_http_client, OpenAIV2HTTPClient)
+    assert isinstance(openai_http_client.client, AsyncClient)

ai_review/tests/suites/clients/openai/v2/test_schema.py

@@ -0,0 +1,80 @@
+from ai_review.clients.openai.v2.schema import (
+    OpenAIResponseUsageSchema,
+    OpenAIInputMessageSchema,
+    OpenAIResponseContentSchema,
+    OpenAIResponseOutputSchema,
+    OpenAIResponsesRequestSchema,
+    OpenAIResponsesResponseSchema,
+)
+
+
+# ---------- OpenAIResponsesResponseSchema ----------
+
+def test_first_text_returns_combined_text():
+    resp = OpenAIResponsesResponseSchema(
+        usage=OpenAIResponseUsageSchema(total_tokens=42, input_tokens=21, output_tokens=21),
+        output=[
+            OpenAIResponseOutputSchema(
+                type="message",
+                role="assistant",
+                content=[
+                    OpenAIResponseContentSchema(type="output_text", text="Hello"),
+                    OpenAIResponseContentSchema(type="output_text", text=" World"),
+                ],
+            )
+        ],
+    )
+
+    assert resp.first_text == "Hello World"
+
+
+def test_first_text_empty_if_no_output():
+    resp = OpenAIResponsesResponseSchema(
+        usage=OpenAIResponseUsageSchema(total_tokens=0, input_tokens=0, output_tokens=0),
+        output=[],
+    )
+    assert resp.first_text == ""
+
+
+def test_first_text_ignores_non_message_blocks():
+    resp = OpenAIResponsesResponseSchema(
+        usage=OpenAIResponseUsageSchema(total_tokens=5, input_tokens=2, output_tokens=3),
+        output=[
+            OpenAIResponseOutputSchema(
+                type="reasoning",  # игнорируется
+                role=None,
+                content=None,
+            )
+        ],
+    )
+    assert resp.first_text == ""
+
+
+# ---------- OpenAIResponsesRequestSchema ----------
+
+def test_responses_request_schema_builds_ok():
+    msg = OpenAIInputMessageSchema(role="user", content="hello")
+    req = OpenAIResponsesRequestSchema(
+        model="gpt-5",
+        input=[msg],
+        temperature=0.2,
+        max_output_tokens=512,
+        instructions="You are a helpful assistant.",
+    )
+
+    assert req.model == "gpt-5"
+    assert req.input[0].role == "user"
+    assert req.input[0].content == "hello"
+    assert req.temperature == 0.2
+    assert req.max_output_tokens == 512
+    assert req.instructions == "You are a helpful assistant."
+
+
+def test_responses_request_schema_allows_none_tokens():
+    req = OpenAIResponsesRequestSchema(
+        model="gpt-5",
+        input=[OpenAIInputMessageSchema(role="user", content="test")],
+    )
+
+    dumped = req.model_dump(exclude_none=True)
+    assert "max_output_tokens" not in dumped

ai_review/tests/suites/libs/config/llm/test_openai.py

@@ -0,0 +1,28 @@
+import pytest
+
+from ai_review.libs.config.llm.openai import OpenAIMetaConfig
+
+
+@pytest.mark.parametrize(
+    "model, expected",
+    [
+        ("gpt-5", True),
+        ("gpt-5-preview", True),
+        ("gpt-4.1", True),
+        ("gpt-4.1-mini", True),
+        ("gpt-4o", False),
+        ("gpt-4o-mini", False),
+        ("gpt-3.5-turbo", False),
+        ("text-davinci-003", False),
+    ],
+)
+def test_is_v2_model_detection(model: str, expected: bool):
+    meta = OpenAIMetaConfig(model=model)
+    assert meta.is_v2_model is expected, f"Model {model} expected {expected} but got {meta.is_v2_model}"
+
+
+def test_is_v2_model_default_false():
+    meta = OpenAIMetaConfig()
+    assert meta.model == "gpt-4o-mini"
+    assert meta.is_v2_model is False
+    assert meta.max_tokens is None

ai_review/tests/suites/services/llm/openai/test_client.py

@@ -2,21 +2,38 @@ import pytest
 
 from ai_review.services.llm.openai.client import OpenAILLMClient
 from ai_review.services.llm.types import ChatResultSchema
-from ai_review.tests.fixtures.clients.openai import FakeOpenAIHTTPClient
+from ai_review.tests.fixtures.clients.openai import FakeOpenAIV1HTTPClient, FakeOpenAIV2HTTPClient
 
 
 @pytest.mark.asyncio
-@pytest.mark.usefixtures("openai_http_client_config")
-async def test_openai_llm_chat(
+@pytest.mark.usefixtures("openai_v1_http_client_config")
+async def test_openai_llm_chat_v1(
         openai_llm_client: OpenAILLMClient,
-        fake_openai_http_client: FakeOpenAIHTTPClient
+        fake_openai_v1_http_client: FakeOpenAIV1HTTPClient
 ):
     result = await openai_llm_client.chat("prompt", "prompt_system")
 
     assert isinstance(result, ChatResultSchema)
-    assert result.text == "FAKE_OPENAI_RESPONSE"
+    assert result.text == "FAKE_OPENAI_V1_RESPONSE"
     assert result.total_tokens == 12
     assert result.prompt_tokens == 5
     assert result.completion_tokens == 7
 
-    assert fake_openai_http_client.calls[0][0] == "chat"
+    assert fake_openai_v1_http_client.calls[0][0] == "chat"
+
+
+@pytest.mark.asyncio
+@pytest.mark.usefixtures("openai_v2_http_client_config")
+async def test_openai_llm_chat_v2(
+        openai_llm_client: OpenAILLMClient,
+        fake_openai_v2_http_client: FakeOpenAIV2HTTPClient
+):
+    result = await openai_llm_client.chat("prompt", "prompt_system")
+
+    assert isinstance(result, ChatResultSchema)
+    assert result.text == "FAKE_OPENAI_V2_RESPONSE"
+    assert result.total_tokens == 20
+    assert result.prompt_tokens == 10
+    assert result.completion_tokens == 10
+
+    assert fake_openai_v2_http_client.calls[0][0] == "chat"

ai_review/tests/suites/services/llm/test_factory.py

@@ -8,7 +8,7 @@ from ai_review.services.llm.openai.client import OpenAILLMClient
 from ai_review.services.llm.openrouter.client import OpenRouterLLMClient
 
 
-@pytest.mark.usefixtures("openai_http_client_config")
+@pytest.mark.usefixtures("openai_v1_http_client_config")
 def test_get_llm_client_returns_openai(monkeypatch: pytest.MonkeyPatch):
     client = get_llm_client()
     assert isinstance(client, OpenAILLMClient)

{xai_review-0.34.0.dist-info → xai_review-0.36.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xai-review
-Version: 0.34.0
+Version: 0.36.0
 Summary: AI-powered code review tool for GitHub, GitLab, Bitbucket and Gitea — built with LLMs like OpenAI, Claude, Gemini, Ollama, and OpenRouter
 Author-email: Nikita Filonov <nikita.filonov@example.com>
 Maintainer-email: Nikita Filonov <nikita.filonov@example.com>
@@ -221,7 +221,7 @@ jobs:
         with:
           fetch-depth: 0
 
-      - uses: Nikita-Filonov/ai-review@v0.34.0
+      - uses: Nikita-Filonov/ai-review@v0.36.0
         with:
           review-command: ${{ inputs.review-command }}
         env:

{xai_review-0.34.0.dist-info → xai_review-0.36.0.dist-info}/RECORD

@@ -22,12 +22,12 @@ ai_review/clients/bitbucket/pr/schema/files.py,sha256=qBDfIv370TDMKM8yoLlm1c-BTH
 ai_review/clients/bitbucket/pr/schema/pull_request.py,sha256=Eq8huJmZ9smy2JlsPuMF-Vv8EmHYR3rFQ_5EFVc2QRc,845
 ai_review/clients/bitbucket/pr/schema/user.py,sha256=-TsvtifDNZSK82fqNXv6N3li2hnui1_9g2ugSBx4Ibs,125
 ai_review/clients/claude/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/clients/claude/client.py,sha256=uEadbBNBJnzjHDczbxXiiw1V1H1PdUWKu-Gn-eIDEmw,1890
-ai_review/clients/claude/schema.py,sha256=LE6KCjJKDXqBGU2Cno5XL5R8vUfScgskE9MqvE0Pt2A,887
+ai_review/clients/claude/client.py,sha256=I1UVlUHmLygFYw3pJIP39zYenULmEx_h65JH3XW7jG4,1907
+ai_review/clients/claude/schema.py,sha256=FJy5Wh5A3IJWPNn4K6KIFn4OTC713pxO9KCqdfUKwNA,915
 ai_review/clients/claude/types.py,sha256=y_-yF7zQrTvyiowS2b9xjIlAzkF8i6OfOjqo9eB8Xo4,267
 ai_review/clients/gemini/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/clients/gemini/client.py,sha256=4G1LBcpiFcrITOysQbMwhY1db4hHcSGgyI-0XazZMV0,1889
-ai_review/clients/gemini/schema.py,sha256=5oVvbI-h_sw8bFreS4JUmMj-aXa_frvxK3H8sg4iJIA,2264
+ai_review/clients/gemini/client.py,sha256=rZsvGQG62Wu2LpBlAFr8w6_S_0oHH1fVZ8lK41995bk,1918
+ai_review/clients/gemini/schema.py,sha256=mBESW6CupZaRRSzFv5K0WTEPfzk6brAFpZG_EiloKRI,2299
 ai_review/clients/gemini/types.py,sha256=D-P0THorrQ8yq5P-NKAC65zzhEYRa9HkiXTORG9QoIk,267
 ai_review/clients/gitea/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/clients/gitea/client.py,sha256=cMxp1Ic44JFvSkuqLdWkAhsgMC6CwM3qmf-1CptqnV8,1152
@@ -65,16 +65,21 @@ ai_review/clients/gitlab/mr/schema/notes.py,sha256=9wmwULegmTO6ETSjYlMC6Fc_DIeT_
 ai_review/clients/gitlab/mr/schema/position.py,sha256=oYml4x6rlrqGahEEbSB1c1ko70geL_0_otbwP0JqV6k,371
 ai_review/clients/gitlab/mr/schema/user.py,sha256=RxgCM8oryPBNPDaFxcVqe11MogchMGaO1gALkZiscrU,112
 ai_review/clients/ollama/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/clients/ollama/client.py,sha256=KoJ9J5_Vfpv5XNJREshE_gA46uo9J0Z3qVC7wJPEcX8,1720
+ai_review/clients/ollama/client.py,sha256=lo_yOmB6nSdiFdIeSG1t1-AVHUZZEzKMpD6y3w2UJco,1737
 ai_review/clients/ollama/schema.py,sha256=A6oKwkkEVrduyzMR_lhLnaLyvKXqlfsXjkMIF2eXaYw,1310
 ai_review/clients/ollama/types.py,sha256=9ES8K-EClKYU7UsaMKgXvZ3sUOF9o6reEvfL6wFOJ4M,267
 ai_review/clients/openai/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/clients/openai/client.py,sha256=jY1XG_5GtNboNjkXu3UtuXFx5V9rD6UskK7VT0lOzP8,1816
-ai_review/clients/openai/schema.py,sha256=glxwMtBrDA6W0BQgH-ruKe0bKH3Ps1P-Y1-2jGdqaUM,764
-ai_review/clients/openai/types.py,sha256=4VRY45ihKjii8w0d5XLnUGnHuBSh9wRsOP6lmkseC0Q,267
+ai_review/clients/openai/v1/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
+ai_review/clients/openai/v1/client.py,sha256=zThyVUFWOF2wXi47PPrejK9ApXSc7I-2yaBHlDdiMaM,1861
+ai_review/clients/openai/v1/schema.py,sha256=WnYzQP4e5rHfX40kVHCekxgXXufX7pz6FBw4qiSKUKA,792
+ai_review/clients/openai/v1/types.py,sha256=giLb2LKyoHNgIq4wO__asPl4dUi1PEMiOAQIbsPpuNk,272
+ai_review/clients/openai/v2/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
+ai_review/clients/openai/v2/client.py,sha256=MN61bCVkNPuOxaYZEh5duaZgpUmqnnSL2ZVA_9HWTHs,1896
+ai_review/clients/openai/v2/schema.py,sha256=AU9ewqgLSozSgAgVh15ql18VMMchQeTO7MW9gUT9jd0,1195
+ai_review/clients/openai/v2/types.py,sha256=BE4iFUSW1OHsah3YxTwMt1bWhZCZMssBEkTlLcR0POs,304
 ai_review/clients/openrouter/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/clients/openrouter/client.py,sha256=HfcfOuUs08w7Na344i197I_WrNNjyB1xSBzirxkPGO0,2088
-ai_review/clients/openrouter/schema.py,sha256=U4c1wNhhAVLQ85J69IjR5g5tLly8KfrwGP7T5tsJPwI,799
+ai_review/clients/openrouter/client.py,sha256=llfdisXWXyA7XufqeToGaTKnGHkNdV0y84TrPsaa41A,2105
+ai_review/clients/openrouter/schema.py,sha256=hTPLIMIUHIKD4wDJsVPidn0LTQz58SsI1fId6JVU8KQ,827
 ai_review/clients/openrouter/types.py,sha256=9CFUy52GnfjjLRufz7SwY_fnzhQnn8czLl-XLWBSKGc,303
 ai_review/libs/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/libs/json.py,sha256=g-P5_pNUomQ-bGHCXASvPKj9Og0s9MaLFVEAkzqGp1A,350
@@ -94,9 +99,9 @@ ai_review/libs/config/llm/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJW
 ai_review/libs/config/llm/base.py,sha256=yhRxizyRwwYH1swKeVzZaGAVm3I3nNME_y-h0iW5WTw,2374
 ai_review/libs/config/llm/claude.py,sha256=MoalXkBA6pEp01znS8ohTRopfea9RUcqhZX5lOIuek8,293
 ai_review/libs/config/llm/gemini.py,sha256=SKtlzsRuNWOlM9m3SFvcqOIjnml8lpPidp7FiGmIEz4,265
-ai_review/libs/config/llm/meta.py,sha256=cEcAHOwy-mQBKo9_KJrQe0I7qppq6h99lSmoWX4ElJI,195
+ai_review/libs/config/llm/meta.py,sha256=MtWboDCqoj_jZrrGoClVJNcHyskgyqJLcPMOztfxkiM,210
 ai_review/libs/config/llm/ollama.py,sha256=M6aiPb5GvYvkiGcgHTsh9bOw5JsBLqmfSKoIbHCejrU,372
-ai_review/libs/config/llm/openai.py,sha256=jGVL4gJ2wIacoKeK9Zc9LCgY95TxdeYOThdglVPErFU,262
+ai_review/libs/config/llm/openai.py,sha256=g2EF0h2JGD5iTOtNHg-G9bdqgVtqGjg4KLf-p9V6flY,395
 ai_review/libs/config/llm/openrouter.py,sha256=6G5fApCOv0fKRHCUpsuiPOcEdyUpDe5qiUUbHjA6TbE,337
 ai_review/libs/config/vcs/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/libs/config/vcs/base.py,sha256=RJZhKysD-d8oYZQ2v1H74jyqdqtOCc8zZ0n9S4ovfHk,1471
@@ -137,7 +142,7 @@ ai_review/prompts/default_system_inline_reply.md,sha256=OhWQ5PZ-QFFgVVfmrsXEmvNA
 ai_review/prompts/default_system_summary.md,sha256=unEJ09G925TKqvjkTFKgl3g2AXT9GICe8kxTO50QhRg,224
 ai_review/prompts/default_system_summary_reply.md,sha256=qGc_qQtVUpmj_gOw-6dhL2gz6saSA__9UOcS0jAjBD0,687
 ai_review/resources/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/resources/pricing.yaml,sha256=jZHCGF78GTlZsXC_IGZT8JutKqpUyKikYXwtxIFEAaE,746
+ai_review/resources/pricing.yaml,sha256=wxwF9sH6uUkQyaDok-cBcbXReGxg7gdsXgC2pRrQWn0,1553
 ai_review/services/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/services/artifacts/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/services/artifacts/schema.py,sha256=o4dqG5LFCdAQY3wjRF5rImANe-X20g4zX_bCIKiHLSk,291
@@ -171,7 +176,7 @@ ai_review/services/llm/gemini/client.py,sha256=TR4HshVxtDV8_luQKCM3aFNH9tjAjpzNe
 ai_review/services/llm/ollama/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/services/llm/ollama/client.py,sha256=817nOQRsnaVqoY6LdO95l5JkRHkGvvS8TX7hezT2gqk,1479
 ai_review/services/llm/openai/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/services/llm/openai/client.py,sha256=c3DWwLnwTheERdSGnMiQIbg5SaICouUAGClcQZSh1fE,1159
+ai_review/services/llm/openai/client.py,sha256=edOBi6pVY_jquUj8BCRj-Gn_q-JxcK_ohBmElGClXlw,2437
 ai_review/services/llm/openrouter/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/services/llm/openrouter/client.py,sha256=wrUEETrer3XmNRtu3YzLIRNa_3ODTzsR_LU9kDkl-7I,1212
 ai_review/services/prompt/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -239,7 +244,7 @@ ai_review/tests/fixtures/clients/gitea.py,sha256=WQLbOyFTqqtVQGHuLFgk9qANYS03eeC
 ai_review/tests/fixtures/clients/github.py,sha256=kC1L-nWZMn9O_uRfuT_B8R4sn8FRvISlBJMkRKaioS0,7814
 ai_review/tests/fixtures/clients/gitlab.py,sha256=AD6NJOJSw76hjAEiWewQ6Vu5g-cfQn0GTtdchuDBH9o,8042
 ai_review/tests/fixtures/clients/ollama.py,sha256=UUHDDPUraQAG8gBC-0UvftaK0BDYir5cJDlRKJymSQg,2109
-ai_review/tests/fixtures/clients/openai.py,sha256=PfnaYdrKwGiuAx8fnSJBZamAZEYJR1y8I4oHBj2SmU4,2291
+ai_review/tests/fixtures/clients/openai.py,sha256=sgRt7DxKqbPUv0GPFE94XUDtFwa5tIgi1DQYWPqPQHA,4749
 ai_review/tests/fixtures/clients/openrouter.py,sha256=TWCojwXP0y0_dlzFMzJra4uXSQ3Dv5wZQnm_Hbvxodg,2532
 ai_review/tests/fixtures/libs/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/fixtures/libs/llm/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -297,8 +302,12 @@ ai_review/tests/suites/clients/ollama/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCe
 ai_review/tests/suites/clients/ollama/test_client.py,sha256=XZ8NAd1bS_ltTuYZPgqlutPRA6kbvH3_3SKTCbNBTgA,404
 ai_review/tests/suites/clients/ollama/test_schema.py,sha256=A93wCmxwGdvudfbA97VCPYP3gT6u6EYMetAg5fgURRA,1836
 ai_review/tests/suites/clients/openai/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/tests/suites/clients/openai/test_client.py,sha256=6Wsxw6-6Uk0uPYFkzpWSwsxfCYUZhT3UYznayo-xlPI,404
-ai_review/tests/suites/clients/openai/test_schema.py,sha256=x1tamS4GC9pOTpjieKDbK2D73CVV4BkATppytwMevLo,1599
+ai_review/tests/suites/clients/openai/v1/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
+ai_review/tests/suites/clients/openai/v1/test_client.py,sha256=raN0AvAQW46FEDOY8TfTmI3vzD4F2G7mrFdeMF2YSxQ,423
+ai_review/tests/suites/clients/openai/v1/test_schema.py,sha256=vLoo4nzcmVKA1hwuuZI354yYRV2N8ujSoF23iNkkipQ,1602
+ai_review/tests/suites/clients/openai/v2/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
+ai_review/tests/suites/clients/openai/v2/test_client.py,sha256=pe72sdUCt916BLZ6vC6xsLPzZqEI-uO-3KvCJqgGf2M,423
+ai_review/tests/suites/clients/openai/v2/test_schema.py,sha256=yzwLILjI4j_MKfNlFeQ1QWISCsQTNNI87WvdWbrRlO4,2461
 ai_review/tests/suites/clients/openrouter/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/clients/openrouter/test_client.py,sha256=WNH0p1Bl5J6zLKB1gSQ9smQMRTOo5-U-A60iJ0n4_DI,444
 ai_review/tests/suites/clients/openrouter/test_schema.py,sha256=9wt8-lR1u2KGvd6Iget_Yy-r33BllYLA-3AKe-S2E-c,1731
@@ -308,6 +317,8 @@ ai_review/tests/suites/libs/asynchronous/__init__.py,sha256=47DEQpj8HBSa-_TImW-5
 ai_review/tests/suites/libs/asynchronous/test_gather.py,sha256=Uzo0ctGFlllJU6TqA6hVWu_mcycJBAX5am8_bDO2eXk,1282
 ai_review/tests/suites/libs/config/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/libs/config/test_prompt.py,sha256=H3QC6LvYAwebnLed-NeT86TY9kPaMPy7Eppk5Bfi2SM,5601
+ai_review/tests/suites/libs/config/llm/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
+ai_review/tests/suites/libs/config/llm/test_openai.py,sha256=H9sPccXuayoYrLVHxWc2WMD2rK2zlGP1xlAaa3G03Gs,777
 ai_review/tests/suites/libs/diff/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/libs/diff/test_models.py,sha256=RBFQ97LWhU8TlupxXkJ97ryAvJrSuOHLtT9biUBUMXg,3321
 ai_review/tests/suites/libs/diff/test_parser.py,sha256=rvWEVGIdaLBlDAnSevjRY7I1Zikj12d5GOgMk9QyHQQ,3013
@@ -332,7 +343,7 @@ ai_review/tests/suites/services/diff/test_tools.py,sha256=vsOSSIDZKkuD8dMCoBBEBt
 ai_review/tests/suites/services/hook/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/services/hook/test_service.py,sha256=TjNU2xiOQfUZZa8M3L2eHbtTwxse_B7QNn2h4118z1U,6637
 ai_review/tests/suites/services/llm/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/tests/suites/services/llm/test_factory.py,sha256=e3pND1Rq4lIXpm_R8J1mRiEc7moEqJ9cGU3rK5tk4Bo,1685
+ai_review/tests/suites/services/llm/test_factory.py,sha256=HEE65FBfIrSQbwpIYRq3SyVv1NljELPTGUHpKCahiQ0,1688
 ai_review/tests/suites/services/llm/claude/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/services/llm/claude/test_client.py,sha256=ymIeuIax0Bp_CuXBSApK1RDl1JmbGc97uzXZToQOZO8,761
 ai_review/tests/suites/services/llm/gemini/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -340,7 +351,7 @@ ai_review/tests/suites/services/llm/gemini/test_client.py,sha256=RjYViMZTgTdbzmD
 ai_review/tests/suites/services/llm/ollama/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/services/llm/ollama/test_client.py,sha256=Eu4OERB00SJwCKznyOCyqSFTDBp9J2Lw-BcW7sPJQM4,760
 ai_review/tests/suites/services/llm/openai/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/tests/suites/services/llm/openai/test_client.py,sha256=yzIL8GYHyX9iLKIlaF__87aue9w0cr66feoMaCv5gms,761
+ai_review/tests/suites/services/llm/openai/test_client.py,sha256=bF0MFaTq_45ob4f8eZXF3Fn0aUMy848dBaQXiAiqIPg,1373
 ai_review/tests/suites/services/llm/openrouter/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/services/llm/openrouter/test_client.py,sha256=9YRwhkgeRDdRi_EMFh_T0u4wgEFj2AMgAiusrYWzeEc,813
 ai_review/tests/suites/services/prompt/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -389,9 +400,9 @@ ai_review/tests/suites/services/vcs/github/test_client.py,sha256=mNt1bA6aVU3REsJ
 ai_review/tests/suites/services/vcs/gitlab/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/services/vcs/gitlab/test_adapter.py,sha256=BYBP2g1AKF_jCSJYJj16pW7M_6PprwD9reYEpdw3StU,4340
 ai_review/tests/suites/services/vcs/gitlab/test_client.py,sha256=dnI-YxYADmVF2GS9rp6-JPkcqsn4sN8Fjbe4MkeYMaE,8476
-xai_review-0.34.0.dist-info/licenses/LICENSE,sha256=p-v8m7Kmz4KKc7PcvsGiGEmCw9AiSXY4_ylOPy_u--Y,11343
-xai_review-0.34.0.dist-info/METADATA,sha256=Zpuk0sHxSQKsA84Xdv4Ba1V6Seghvxd_XCh48f5BKDw,12911
-xai_review-0.34.0.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-xai_review-0.34.0.dist-info/entry_points.txt,sha256=JyC5URanMi5io5P_PXQf7H_I1OGIpk5cZQhaPQ0g4Zs,53
-xai_review-0.34.0.dist-info/top_level.txt,sha256=sTsZbfzLoqvRZKdKa-BcxWvjlHdrpbeJ6DrGY0EuR0E,10
-xai_review-0.34.0.dist-info/RECORD,,
+xai_review-0.36.0.dist-info/licenses/LICENSE,sha256=p-v8m7Kmz4KKc7PcvsGiGEmCw9AiSXY4_ylOPy_u--Y,11343
+xai_review-0.36.0.dist-info/METADATA,sha256=kLVjP_H0HJzK7_3tWvhrBbnuYaKPssSx2hzldptp0HI,12911
+xai_review-0.36.0.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+xai_review-0.36.0.dist-info/entry_points.txt,sha256=JyC5URanMi5io5P_PXQf7H_I1OGIpk5cZQhaPQ0g4Zs,53
+xai_review-0.36.0.dist-info/top_level.txt,sha256=sTsZbfzLoqvRZKdKa-BcxWvjlHdrpbeJ6DrGY0EuR0E,10
+xai_review-0.36.0.dist-info/RECORD,,

ai_review/clients/openai/types.py

@@ -1,8 +0,0 @@
-from typing import Protocol
-
-from ai_review.clients.openai.schema import OpenAIChatRequestSchema, OpenAIChatResponseSchema
-
-
-class OpenAIHTTPClientProtocol(Protocol):
-    async def chat(self, request: OpenAIChatRequestSchema) -> OpenAIChatResponseSchema:
-        ...

ai_review/tests/suites/clients/openai/test_client.py

@@ -1,12 +0,0 @@
-import pytest
-from httpx import AsyncClient
-
-from ai_review.clients.openai.client import get_openai_http_client, OpenAIHTTPClient
-
-
-@pytest.mark.usefixtures('openai_http_client_config')
-def test_get_openai_http_client_builds_ok():
-    openai_http_client = get_openai_http_client()
-
-    assert isinstance(openai_http_client, OpenAIHTTPClient)
-    assert isinstance(openai_http_client.client, AsyncClient)