xai-review 0.22.0__tar.gz → 0.24.0__tar.gz

{xai_review-0.22.0 → xai_review-0.24.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xai-review
-Version: 0.22.0
+Version: 0.24.0
 Summary: AI-powered code review tool
 Author-email: Nikita Filonov <nikita.filonov@example.com>
 Maintainer-email: Nikita Filonov <nikita.filonov@example.com>
@@ -209,7 +209,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - uses: Nikita-Filonov/ai-review@v0.22.0
+      - uses: Nikita-Filonov/ai-review@v0.24.0
         with:
           review-command: ${{ inputs.review-command }}
         env:

{xai_review-0.22.0 → xai_review-0.24.0}/README.md

@@ -175,7 +175,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - uses: Nikita-Filonov/ai-review@v0.22.0
+      - uses: Nikita-Filonov/ai-review@v0.24.0
         with:
           review-command: ${{ inputs.review-command }}
         env:

{xai_review-0.22.0 → xai_review-0.24.0}/ai_review/config.py

@@ -13,11 +13,11 @@ from ai_review.libs.config.base import (
     get_json_config_file_or_default
 )
 from ai_review.libs.config.core import CoreConfig
-from ai_review.libs.config.llm import LLMConfig
+from ai_review.libs.config.llm.base import LLMConfig
 from ai_review.libs.config.logger import LoggerConfig
 from ai_review.libs.config.prompt import PromptConfig
 from ai_review.libs.config.review import ReviewConfig
-from ai_review.libs.config.vcs import VCSConfig
+from ai_review.libs.config.vcs.base import VCSConfig
 
 
 class Settings(BaseSettings):

xai_review-0.22.0/ai_review/libs/config/llm.py → xai_review-0.24.0/ai_review/libs/config/llm/base.py

@@ -4,9 +4,9 @@ from typing import Annotated, Literal
 import yaml
 from pydantic import BaseModel, Field, FilePath
 
-from ai_review.libs.config.claude import ClaudeHTTPClientConfig, ClaudeMetaConfig
-from ai_review.libs.config.gemini import GeminiHTTPClientConfig, GeminiMetaConfig
-from ai_review.libs.config.openai import OpenAIHTTPClientConfig, OpenAIMetaConfig
+from ai_review.libs.config.llm.claude import ClaudeHTTPClientConfig, ClaudeMetaConfig
+from ai_review.libs.config.llm.gemini import GeminiHTTPClientConfig, GeminiMetaConfig
+from ai_review.libs.config.llm.openai import OpenAIHTTPClientConfig, OpenAIMetaConfig
 from ai_review.libs.constants.llm_provider import LLMProvider
 from ai_review.libs.resources import load_resource
 

xai_review-0.22.0/ai_review/libs/config/vcs.py → xai_review-0.24.0/ai_review/libs/config/vcs/base.py

@@ -2,8 +2,8 @@ from typing import Annotated, Literal
 
 from pydantic import BaseModel, Field
 
-from ai_review.libs.config.github import GitHubPipelineConfig, GitHubHTTPClientConfig
-from ai_review.libs.config.gitlab import GitLabPipelineConfig, GitLabHTTPClientConfig
+from ai_review.libs.config.vcs.github import GitHubPipelineConfig, GitHubHTTPClientConfig
+from ai_review.libs.config.vcs.gitlab import GitLabPipelineConfig, GitLabHTTPClientConfig
 from ai_review.libs.constants.vcs_provider import VCSProvider
 
 

{xai_review-0.22.0 → xai_review-0.24.0}/ai_review/libs/http/transports/retry.py

@@ -36,7 +36,7 @@ class RetryTransport(AsyncBaseTransport):
                 return last_response
 
             self.logger.warning(
-                f"Attempt {attempt}/{self.max_retries} failed "
+                f"Attempt {attempt + 1}/{self.max_retries} failed "
                 f"with status={last_response.status_code} for {request.method} {request.url}. "
                 f"Retrying in {self.retry_delay:.1f}s..."
             )

{xai_review-0.22.0 → xai_review-0.24.0}/ai_review/services/diff/renderers.py

@@ -116,14 +116,14 @@ def render_unified(
     added_new_positions = file.added_line_numbers()
     removed_old_positions = file.removed_line_numbers()
 
-    def in_context(old_no: int | None, new_no: int | None) -> bool:
+    def in_context(inner_old_no: int | None, inner_new_no: int | None) -> bool:
         """Check if an unchanged line falls within context radius."""
         if context <= 0:
             return False
-        if include_added and new_no is not None:
+        if include_added and inner_new_no is not None:
             if any(abs(new_no - a) <= context for a in added_new_positions):
                 return True
-        if include_removed and old_no is not None:
+        if include_removed and inner_old_no is not None:
             if any(abs(old_no - r) <= context for r in removed_old_positions):
                 return True
         return False

xai_review-0.24.0/ai_review/services/prompt/schema.py

@@ -0,0 +1,41 @@
+from pydantic import BaseModel, Field, field_serializer
+
+from ai_review.config import settings
+from ai_review.libs.template.render import render_template
+
+
+class PromptContextSchema(BaseModel):
+    review_title: str = ""
+    review_description: str = ""
+
+    review_author_name: str = ""
+    review_author_username: str = ""
+
+    review_reviewer: str = ""
+    review_reviewers: list[str] = Field(default_factory=list)
+    review_reviewers_usernames: list[str] = Field(default_factory=list)
+
+    review_assignees: list[str] = Field(default_factory=list)
+    review_assignees_usernames: list[str] = Field(default_factory=list)
+
+    source_branch: str = ""
+    target_branch: str = ""
+
+    labels: list[str] = Field(default_factory=list)
+    changed_files: list[str] = Field(default_factory=list)
+
+    @field_serializer(
+        "review_reviewers",
+        "review_reviewers_usernames",
+        "review_assignees",
+        "review_assignees_usernames",
+        "labels",
+        "changed_files",
+        when_used="always"
+    )
+    def list_of_strings_serializer(self, value: list[str]) -> str:
+        return ", ".join(value)
+
+    def apply_format(self, prompt: str) -> str:
+        values = {**self.model_dump(), **settings.prompt.context}
+        return render_template(prompt, values, settings.prompt.context_placeholder)

{xai_review-0.22.0 → xai_review-0.24.0}/ai_review/tests/fixtures/clients/claude.py

@@ -2,8 +2,8 @@ import pytest
 from pydantic import HttpUrl, SecretStr
 
 from ai_review.config import settings
-from ai_review.libs.config.claude import ClaudeMetaConfig, ClaudeHTTPClientConfig
-from ai_review.libs.config.llm import ClaudeLLMConfig
+from ai_review.libs.config.llm.base import ClaudeLLMConfig
+from ai_review.libs.config.llm.claude import ClaudeMetaConfig, ClaudeHTTPClientConfig
 from ai_review.libs.constants.llm_provider import LLMProvider
 
 

{xai_review-0.22.0 → xai_review-0.24.0}/ai_review/tests/fixtures/clients/gemini.py

@@ -2,8 +2,8 @@ import pytest
 from pydantic import HttpUrl, SecretStr
 
 from ai_review.config import settings
-from ai_review.libs.config.gemini import GeminiMetaConfig, GeminiHTTPClientConfig
-from ai_review.libs.config.llm import GeminiLLMConfig
+from ai_review.libs.config.llm.base import GeminiLLMConfig
+from ai_review.libs.config.llm.gemini import GeminiMetaConfig, GeminiHTTPClientConfig
 from ai_review.libs.constants.llm_provider import LLMProvider
 
 

{xai_review-0.22.0 → xai_review-0.24.0}/ai_review/tests/fixtures/clients/github.py

@@ -18,8 +18,8 @@ from ai_review.clients.github.pr.schema.pull_request import (
 from ai_review.clients.github.pr.schema.reviews import GitHubGetPRReviewsResponseSchema, GitHubPRReviewSchema
 from ai_review.clients.github.pr.types import GitHubPullRequestsHTTPClientProtocol
 from ai_review.config import settings
-from ai_review.libs.config.github import GitHubPipelineConfig, GitHubHTTPClientConfig
-from ai_review.libs.config.vcs import GitHubVCSConfig
+from ai_review.libs.config.vcs.base import GitHubVCSConfig
+from ai_review.libs.config.vcs.github import GitHubPipelineConfig, GitHubHTTPClientConfig
 from ai_review.libs.constants.vcs_provider import VCSProvider
 from ai_review.services.vcs.github.client import GitHubVCSClient
 

{xai_review-0.22.0 → xai_review-0.24.0}/ai_review/tests/fixtures/clients/gitlab.py

@@ -20,8 +20,8 @@ from ai_review.clients.gitlab.mr.schema.notes import (
 )
 from ai_review.clients.gitlab.mr.types import GitLabMergeRequestsHTTPClientProtocol
 from ai_review.config import settings
-from ai_review.libs.config.gitlab import GitLabPipelineConfig, GitLabHTTPClientConfig
-from ai_review.libs.config.vcs import GitLabVCSConfig
+from ai_review.libs.config.vcs.base import GitLabVCSConfig
+from ai_review.libs.config.vcs.gitlab import GitLabPipelineConfig, GitLabHTTPClientConfig
 from ai_review.libs.constants.vcs_provider import VCSProvider
 from ai_review.services.vcs.gitlab.client import GitLabVCSClient
 

{xai_review-0.22.0 → xai_review-0.24.0}/ai_review/tests/fixtures/clients/openai.py

@@ -2,8 +2,8 @@ import pytest
 from pydantic import HttpUrl, SecretStr
 
 from ai_review.config import settings
-from ai_review.libs.config.llm import OpenAILLMConfig
-from ai_review.libs.config.openai import OpenAIMetaConfig, OpenAIHTTPClientConfig
+from ai_review.libs.config.llm.base import OpenAILLMConfig
+from ai_review.libs.config.llm.openai import OpenAIMetaConfig, OpenAIHTTPClientConfig
 from ai_review.libs.constants.llm_provider import LLMProvider
 
 

{xai_review-0.22.0 → xai_review-0.24.0}/ai_review/tests/suites/services/cost/test_service.py

@@ -1,6 +1,6 @@
 import pytest
 
-from ai_review.libs.config.llm import LLMPricingConfig, LLMConfigBase
+from ai_review.libs.config.llm.base import LLMPricingConfig, LLMConfigBase
 from ai_review.services.cost.schema import CostReportSchema
 from ai_review.services.cost.service import CostService
 from ai_review.services.llm.types import ChatResultSchema

xai_review-0.24.0/ai_review/tests/suites/services/prompt/test_schema.py

@@ -0,0 +1,136 @@
+import pytest
+
+from ai_review.config import settings
+from ai_review.services.prompt.schema import PromptContextSchema
+
+
+def test_apply_format_inserts_variables() -> None:
+    """Ensures simple string fields are correctly substituted into the template."""
+    context = PromptContextSchema(
+        review_title="My Review",
+        review_author_username="nikita"
+    )
+    template = "Title: <<review_title>>, Author: @<<review_author_username>>"
+    result = context.apply_format(template)
+    assert result == "Title: My Review, Author: @nikita"
+
+
+def test_apply_format_with_lists() -> None:
+    """Ensures list fields are serialized as CSV strings and substituted into the template."""
+    context = PromptContextSchema(
+        review_reviewers=["Alice", "Bob"],
+        review_reviewers_usernames=["alice", "bob"],
+        labels=["bug", "feature"],
+        changed_files=["a.py", "b.py"],
+    )
+    template = (
+        "Reviewers: <<review_reviewers>>\n"
+        "Usernames: <<review_reviewers_usernames>>\n"
+        "Labels: <<labels>>\n"
+        "Files: <<changed_files>>"
+    )
+    result = context.apply_format(template)
+    assert "Alice, Bob" in result
+    assert "alice, bob" in result
+    assert "bug, feature" in result
+    assert "a.py, b.py" in result
+
+
+def test_apply_format_handles_missing_fields() -> None:
+    """Ensures missing fields are replaced with empty strings."""
+    context = PromptContextSchema()
+    template = "Title: <<review_title>>, Reviewer: <<review_reviewer>>"
+    result = context.apply_format(template)
+    assert result == "Title: , Reviewer: "
+
+
+def test_apply_format_unknown_placeholder_kept() -> None:
+    """Ensures unknown placeholders remain unchanged in the template."""
+    context = PromptContextSchema()
+    template = "Unknown: <<does_not_exist>>"
+    result = context.apply_format(template)
+    assert result == "Unknown: <<does_not_exist>>"
+
+
+def test_apply_format_multiple_occurrences() -> None:
+    """Ensures multiple occurrences of the same placeholder are all replaced."""
+    context = PromptContextSchema(review_title="My Review")
+    template = "<<review_title>> again <<review_title>>"
+    result = context.apply_format(template)
+    assert result == "My Review again My Review"
+
+
+def test_apply_format_override_from_settings(monkeypatch: pytest.MonkeyPatch) -> None:
+    """Ensures values from settings.prompt.context override local model values."""
+    monkeypatch.setitem(settings.prompt.context, "review_title", "Overridden")
+    context = PromptContextSchema(review_title="Local Value")
+    template = "Title: <<review_title>>"
+    result = context.apply_format(template)
+    assert result == "Title: Overridden"
+
+
+def test_apply_format_prefers_override_even_if_empty(monkeypatch: pytest.MonkeyPatch) -> None:
+    """Ensures overrides take precedence even if the override value is empty."""
+    monkeypatch.setitem(settings.prompt.context, "review_title", "")
+    context = PromptContextSchema(review_title="Local Value")
+    template = "Title: <<review_title>>"
+    result = context.apply_format(template)
+    assert result == "Title: "
+
+
+def test_apply_format_empty_list_serializes_to_empty_string() -> None:
+    """Ensures empty lists are serialized to empty strings."""
+    context = PromptContextSchema(labels=[])
+    template = "Labels: <<labels>>"
+    result = context.apply_format(template)
+    assert result == "Labels: "
+
+
+def test_apply_format_single_element_list() -> None:
+    """Ensures lists with a single element are serialized without extra separators."""
+    context = PromptContextSchema(labels=["bug"])
+    template = "Labels: <<labels>>"
+    result = context.apply_format(template)
+    assert result == "Labels: bug"
+
+
+def test_apply_format_list_with_spaces() -> None:
+    """Ensures list items containing spaces are preserved in serialization."""
+    context = PromptContextSchema(labels=[" bug ", " feature "])
+    template = "Labels: <<labels>>"
+    result = context.apply_format(template)
+    assert result == "Labels:  bug ,  feature "
+
+
+def test_apply_format_placeholder_case_sensitive() -> None:
+    """Ensures placeholder matching is case-sensitive."""
+    context = PromptContextSchema(review_title="My Review")
+    template = "Title: <<Review_Title>>"
+    result = context.apply_format(template)
+    assert result == "Title: <<Review_Title>>"
+
+
+def test_apply_format_override_with_none(monkeypatch: pytest.MonkeyPatch) -> None:
+    """Ensures None in overrides is treated as an empty string."""
+    monkeypatch.setitem(settings.prompt.context, "review_title", None)
+    context = PromptContextSchema(review_title="Local Value")
+    template = "Title: <<review_title>>"
+    result = context.apply_format(template)
+    assert result == "Title: "
+
+
+def test_apply_format_placeholder_inside_word() -> None:
+    """Ensures placeholders inside words are still replaced correctly."""
+    context = PromptContextSchema(review_title="REV")
+    template = "prefix-<<review_title>>-suffix"
+    result = context.apply_format(template)
+    assert result == "prefix-REV-suffix"
+
+
+def test_apply_format_large_list() -> None:
+    """Ensures large lists are serialized correctly without truncation."""
+    context = PromptContextSchema(labels=[str(index) for index in range(100)])
+    template = "Labels: <<labels>>"
+    result = context.apply_format(template)
+    assert result.startswith("Labels: 0, 1, 2")
+    assert "99" in result

{xai_review-0.22.0 → xai_review-0.24.0}/pyproject.toml

@@ -5,7 +5,7 @@ build-backend = "setuptools.build_meta"
 [project]
 name = "xai-review"
 readme = { file = "README.md", content-type = "text/markdown" }
-version = "0.22.0"
+version = "0.24.0"
 license = { text = "Apache-2.0" }
 authors = [
     { name = "Nikita Filonov", email = "nikita.filonov@example.com" }

{xai_review-0.22.0 → xai_review-0.24.0}/xai_review.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xai-review
-Version: 0.22.0
+Version: 0.24.0
 Summary: AI-powered code review tool
 Author-email: Nikita Filonov <nikita.filonov@example.com>
 Maintainer-email: Nikita Filonov <nikita.filonov@example.com>
@@ -209,7 +209,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-      - uses: Nikita-Filonov/ai-review@v0.22.0
+      - uses: Nikita-Filonov/ai-review@v0.24.0
         with:
           review-command: ${{ inputs.review-command }}
         env:

{xai_review-0.22.0 → xai_review-0.24.0}/xai_review.egg-info/SOURCES.txt

@@ -48,18 +48,20 @@ ai_review/libs/asynchronous/gather.py
 ai_review/libs/config/__init__.py
 ai_review/libs/config/artifacts.py
 ai_review/libs/config/base.py
-ai_review/libs/config/claude.py
 ai_review/libs/config/core.py
-ai_review/libs/config/gemini.py
-ai_review/libs/config/github.py
-ai_review/libs/config/gitlab.py
 ai_review/libs/config/http.py
-ai_review/libs/config/llm.py
 ai_review/libs/config/logger.py
-ai_review/libs/config/openai.py
 ai_review/libs/config/prompt.py
 ai_review/libs/config/review.py
-ai_review/libs/config/vcs.py
+ai_review/libs/config/llm/__init__.py
+ai_review/libs/config/llm/base.py
+ai_review/libs/config/llm/claude.py
+ai_review/libs/config/llm/gemini.py
+ai_review/libs/config/llm/openai.py
+ai_review/libs/config/vcs/__init__.py
+ai_review/libs/config/vcs/base.py
+ai_review/libs/config/vcs/github.py
+ai_review/libs/config/vcs/gitlab.py
 ai_review/libs/constants/__init__.py
 ai_review/libs/constants/llm_provider.py
 ai_review/libs/constants/vcs_provider.py

xai_review-0.22.0/ai_review/services/prompt/schema.py

@@ -1,52 +0,0 @@
-from pydantic import BaseModel, Field
-
-from ai_review.config import settings
-from ai_review.libs.template.render import render_template
-
-
-class PromptContextSchema(BaseModel):
-    review_title: str = ""
-    review_description: str = ""
-
-    review_author_name: str = ""
-    review_author_username: str = ""
-
-    review_reviewer: str = ""
-    review_reviewers: list[str] = Field(default_factory=list)
-    review_reviewers_usernames: list[str] = Field(default_factory=list)
-
-    review_assignees: list[str] = Field(default_factory=list)
-    review_assignees_usernames: list[str] = Field(default_factory=list)
-
-    source_branch: str = ""
-    target_branch: str = ""
-
-    labels: list[str] = Field(default_factory=list)
-    changed_files: list[str] = Field(default_factory=list)
-
-    @property
-    def render_values(self) -> dict[str, str]:
-        return {
-            "review_title": self.review_title,
-            "review_description": self.review_description,
-
-            "review_author_name": self.review_author_name,
-            "review_author_username": self.review_author_username,
-
-            "review_reviewer": self.review_reviewer,
-            "review_reviewers": ", ".join(self.review_reviewers),
-            "review_reviewers_usernames": ", ".join(self.review_reviewers_usernames),
-
-            "review_assignees": ", ".join(self.review_assignees),
-            "review_assignees_usernames": ", ".join(self.review_assignees_usernames),
-
-            "source_branch": self.source_branch,
-            "target_branch": self.target_branch,
-
-            "labels": ", ".join(self.labels),
-            "changed_files": ", ".join(self.changed_files),
-        }
-
-    def apply_format(self, prompt: str) -> str:
-        values = {**self.render_values, **settings.prompt.context}
-        return render_template(prompt, values, settings.prompt.context_placeholder)

xai_review-0.22.0/ai_review/tests/suites/services/prompt/test_schema.py

@@ -1,71 +0,0 @@
-import pytest
-
-from ai_review.config import settings
-from ai_review.services.prompt.schema import PromptContextSchema
-
-
-def test_apply_format_inserts_variables() -> None:
-    context = PromptContextSchema(
-        review_title="My Review",
-        review_author_username="nikita"
-    )
-    template = "Title: <<review_title>>, Author: @<<review_author_username>>"
-    result = context.apply_format(template)
-    assert result == "Title: My Review, Author: @nikita"
-
-
-def test_apply_format_with_lists() -> None:
-    context = PromptContextSchema(
-        review_reviewers=["Alice", "Bob"],
-        review_reviewers_usernames=["alice", "bob"],
-        labels=["bug", "feature"],
-        changed_files=["a.py", "b.py"],
-    )
-    template = (
-        "Reviewers: <<review_reviewers>>\n"
-        "Usernames: <<review_reviewers_usernames>>\n"
-        "Labels: <<labels>>\n"
-        "Files: <<changed_files>>"
-    )
-    result = context.apply_format(template)
-    assert "Alice, Bob" in result
-    assert "alice, bob" in result
-    assert "bug, feature" in result
-    assert "a.py, b.py" in result
-
-
-def test_apply_format_handles_missing_fields() -> None:
-    context = PromptContextSchema()
-    template = "Title: <<review_title>>, Reviewer: <<review_reviewer>>"
-    result = context.apply_format(template)
-    assert result == "Title: , Reviewer: "
-
-
-def test_apply_format_unknown_placeholder_kept() -> None:
-    context = PromptContextSchema()
-    template = "Unknown: <<does_not_exist>>"
-    result = context.apply_format(template)
-    assert result == "Unknown: <<does_not_exist>>"
-
-
-def test_apply_format_multiple_occurrences() -> None:
-    context = PromptContextSchema(review_title="My Review")
-    template = "<<review_title>> again <<review_title>>"
-    result = context.apply_format(template)
-    assert result == "My Review again My Review"
-
-
-def test_apply_format_override_from_settings(monkeypatch: pytest.MonkeyPatch) -> None:
-    monkeypatch.setitem(settings.prompt.context, "review_title", "Overridden")
-    context = PromptContextSchema(review_title="Local Value")
-    template = "Title: <<review_title>>"
-    result = context.apply_format(template)
-    assert result == "Title: Overridden"
-
-
-def test_apply_format_prefers_override_even_if_empty(monkeypatch: pytest.MonkeyPatch) -> None:
-    monkeypatch.setitem(settings.prompt.context, "review_title", "")
-    context = PromptContextSchema(review_title="Local Value")
-    template = "Title: <<review_title>>"
-    result = context.apply_format(template)
-    assert result == "Title: "