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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xai-review
-Version: 0.22.0
+Version: 0.23.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.23.0
         with:
           review-command: ${{ inputs.review-command }}
         env:

{xai_review-0.22.0 → xai_review-0.23.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.23.0
         with:
           review-command: ${{ inputs.review-command }}
         env:

{xai_review-0.22.0 → xai_review-0.23.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.23.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.23.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.23.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.23.0"
 license = { text = "Apache-2.0" }
 authors = [
     { name = "Nikita Filonov", email = "nikita.filonov@example.com" }

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xai-review
-Version: 0.22.0
+Version: 0.23.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.23.0
         with:
           review-command: ${{ inputs.review-command }}
         env:

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: "