xai-review 0.32.0__tar.gz → 0.33.0__tar.gz

{xai_review-0.32.0/xai_review.egg-info → xai_review-0.33.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xai-review
-Version: 0.32.0
+Version: 0.33.0
 Summary: AI-powered code review tool
 Author-email: Nikita Filonov <nikita.filonov@example.com>
 Maintainer-email: Nikita Filonov <nikita.filonov@example.com>
@@ -220,7 +220,7 @@ jobs:
         with:
           fetch-depth: 0
 
-      - uses: Nikita-Filonov/ai-review@v0.32.0
+      - uses: Nikita-Filonov/ai-review@v0.33.0
         with:
           review-command: ${{ inputs.review-command }}
         env:

{xai_review-0.32.0 → xai_review-0.33.0}/README.md

@@ -186,7 +186,7 @@ jobs:
         with:
           fetch-depth: 0
 
-      - uses: Nikita-Filonov/ai-review@v0.32.0
+      - uses: Nikita-Filonov/ai-review@v0.33.0
         with:
           review-command: ${{ inputs.review-command }}
         env:

{xai_review-0.32.0 → xai_review-0.33.0}/ai_review/services/artifacts/tools.py

@@ -1,8 +1,8 @@
 import hashlib
-from datetime import datetime
+from datetime import datetime, timezone
 
 
 def make_artifact_id(text: str) -> str:
     sha = hashlib.sha1(text.encode()).hexdigest()[:8]
-    ts = datetime.utcnow().strftime("%Y.%m.%d_%H-%M-%S")
+    ts = datetime.now(timezone.utc).strftime("%Y.%m.%d_%H-%M-%S")
     return f"{ts}_{sha}"

{xai_review-0.32.0 → xai_review-0.33.0}/ai_review/tests/fixtures/services/artifacts.py

@@ -2,6 +2,7 @@ from pathlib import Path
 
 import pytest
 
+from ai_review.services.artifacts.service import ArtifactsService
 from ai_review.services.artifacts.types import ArtifactsServiceProtocol
 
 
@@ -49,3 +50,8 @@ class FakeArtifactsService(ArtifactsServiceProtocol):
 @pytest.fixture
 def fake_artifacts_service() -> FakeArtifactsService:
     return FakeArtifactsService()
+
+
+@pytest.fixture
+def artifacts_service() -> ArtifactsService:
+    return ArtifactsService()

xai_review-0.33.0/ai_review/tests/suites/services/artifacts/test_service.py

@@ -0,0 +1,92 @@
+import json
+from pathlib import Path
+
+import aiofiles
+import pytest
+
+from ai_review.config import settings
+from ai_review.libs.config.artifacts import ArtifactsConfig
+from ai_review.services.artifacts.service import ArtifactsService
+
+
+@pytest.mark.asyncio
+async def test_save_llm_interaction_creates_file(
+        tmp_path: Path,
+        monkeypatch: pytest.MonkeyPatch,
+        artifacts_service: ArtifactsService
+):
+    """Checks that a JSON file is created with the correct name and content when LLM saving is enabled."""
+    monkeypatch.setattr(settings, "artifacts", ArtifactsConfig(llm_dir=tmp_path, llm_enabled=True))
+
+    artifact_id = await artifacts_service.save_llm_interaction(
+        prompt="Hello world",
+        prompt_system="system prompt",
+        response="model answer"
+    )
+
+    assert artifact_id is not None
+    artifact_path = tmp_path / f"{artifact_id}.json"
+    assert artifact_path.exists(), "Artifact file was not created"
+
+    async with aiofiles.open(artifact_path, "r", encoding="utf-8") as file:
+        content = await file.read()
+    data = json.loads(content)
+    assert data["id"] == artifact_id
+    assert data["prompt"] == "Hello world"
+    assert data["response"] == "model answer"
+    assert data["prompt_system"] == "system prompt"
+
+
+@pytest.mark.asyncio
+async def test_save_llm_interaction_disabled(
+        monkeypatch: pytest.MonkeyPatch,
+        artifacts_service: ArtifactsService
+):
+    """Checks that the method returns None and does not create a file if LLM saving is disabled."""
+    monkeypatch.setattr(settings, "artifacts", ArtifactsConfig(llm_enabled=False))
+
+    artifact_id = await artifacts_service.save_llm_interaction(
+        prompt="ignored",
+        prompt_system="ignored",
+        response="ignored"
+    )
+
+    assert artifact_id is None
+
+
+@pytest.mark.asyncio
+async def test_save_artifact_writes_file(tmp_path: Path, artifacts_service: ArtifactsService):
+    """Checks that save_artifact writes content to the given file path."""
+    file_path = tmp_path / "test.json"
+    content = '{"key": "value"}'
+
+    result = await artifacts_service.save_artifact(file=file_path, content=content, kind="test")
+
+    assert result == file_path
+    assert file_path.exists()
+    async with aiofiles.open(file_path, "r", encoding="utf-8") as f:
+        saved_content = await f.read()
+    assert saved_content == content
+
+
+@pytest.mark.asyncio
+async def test_save_artifact_handles_exception(
+        monkeypatch: pytest.MonkeyPatch,
+        artifacts_service: ArtifactsService
+):
+    """Checks that save_artifact gracefully returns None on write error."""
+
+    class BrokenAsyncFile:
+        async def __aenter__(self):
+            raise OSError("disk full")
+
+        async def __aexit__(self, exc_type, exc_val, exc_tb):
+            return False
+
+    monkeypatch.setattr(
+        "ai_review.services.artifacts.service.aiofiles.open",
+        lambda *args, **kwargs: BrokenAsyncFile()
+    )
+
+    result = await artifacts_service.save_artifact(Path("/fake/path.json"), "data")
+    assert result is None

xai_review-0.33.0/ai_review/tests/suites/services/artifacts/test_tools.py

@@ -0,0 +1,28 @@
+import hashlib
+import re
+
+import pytest
+
+from ai_review.services.artifacts.tools import make_artifact_id
+
+
+@pytest.mark.parametrize("text", ["hello", "some longer text", ""])
+def test_make_artifact_id_format_and_sha(text: str):
+    """Checks that the function returns a valid artifact ID format and correct SHA prefix."""
+    artifact_id = make_artifact_id(text)
+
+    pattern = r"^\d{4}\.\d{2}\.\d{2}_\d{2}-\d{2}-\d{2}_[0-9a-f]{8}$"
+    assert re.match(pattern, artifact_id), f"Invalid format: {artifact_id}"
+
+    expected_sha = hashlib.sha1(text.encode()).hexdigest()[:8]
+    assert artifact_id.endswith(expected_sha)
+
+
+def test_make_artifact_id_is_deterministic_for_same_input():
+    """Checks that the SHA part is deterministic (identical for the same input)."""
+    sha1 = hashlib.sha1("repeatable".encode()).hexdigest()[:8]
+    artifact_id1 = make_artifact_id("repeatable")
+    artifact_id2 = make_artifact_id("repeatable")
+
+    assert artifact_id1.split("_")[2] == sha1
+    assert artifact_id2.split("_")[2] == sha1

{xai_review-0.32.0 → xai_review-0.33.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.32.0"
+version = "0.33.0"
 license = { text = "Apache-2.0" }
 authors = [
     { name = "Nikita Filonov", email = "nikita.filonov@example.com" }

{xai_review-0.32.0 → xai_review-0.33.0/xai_review.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xai-review
-Version: 0.32.0
+Version: 0.33.0
 Summary: AI-powered code review tool
 Author-email: Nikita Filonov <nikita.filonov@example.com>
 Maintainer-email: Nikita Filonov <nikita.filonov@example.com>
@@ -220,7 +220,7 @@ jobs:
         with:
           fetch-depth: 0
 
-      - uses: Nikita-Filonov/ai-review@v0.32.0
+      - uses: Nikita-Filonov/ai-review@v0.33.0
         with:
           review-command: ${{ inputs.review-command }}
         env:

{xai_review-0.32.0 → xai_review-0.33.0}/xai_review.egg-info/SOURCES.txt

@@ -311,6 +311,9 @@ ai_review/tests/suites/libs/llm/test_output_json_parser.py
 ai_review/tests/suites/libs/template/__init__.py
 ai_review/tests/suites/libs/template/test_render.py
 ai_review/tests/suites/services/__init__.py
+ai_review/tests/suites/services/artifacts/__init__.py
+ai_review/tests/suites/services/artifacts/test_service.py
+ai_review/tests/suites/services/artifacts/test_tools.py
 ai_review/tests/suites/services/cost/__init__.py
 ai_review/tests/suites/services/cost/test_schema.py
 ai_review/tests/suites/services/cost/test_service.py