xai-review 0.28.0__tar.gz → 0.30.0__tar.gz

{xai_review-0.28.0/xai_review.egg-info → xai_review-0.30.0}/PKG-INFO

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

{xai_review-0.28.0 → xai_review-0.30.0}/README.md

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

{xai_review-0.28.0 → xai_review-0.30.0}/ai_review/clients/gitlab/mr/schema/discussions.py

@@ -1,21 +1,13 @@
 from pydantic import BaseModel, RootModel, Field
 
 from ai_review.clients.gitlab.mr.schema.notes import GitLabNoteSchema
-
-
-class GitLabDiscussionPositionSchema(BaseModel):
-    position_type: str = "text"
-    base_sha: str
-    head_sha: str
-    start_sha: str
-    new_path: str
-    new_line: int
+from ai_review.clients.gitlab.mr.schema.position import GitLabPositionSchema
 
 
 class GitLabDiscussionSchema(BaseModel):
     id: str
     notes: list[GitLabNoteSchema]
-    position: GitLabDiscussionPositionSchema | None = None
+    position: GitLabPositionSchema | None = None
 
 
 class GitLabGetMRDiscussionsQuerySchema(BaseModel):
@@ -29,7 +21,7 @@ class GitLabGetMRDiscussionsResponseSchema(RootModel[list[GitLabDiscussionSchema
 
 class GitLabCreateMRDiscussionRequestSchema(BaseModel):
     body: str
-    position: GitLabDiscussionPositionSchema
+    position: GitLabPositionSchema
 
 
 class GitLabCreateMRDiscussionResponseSchema(BaseModel):

{xai_review-0.28.0 → xai_review-0.30.0}/ai_review/clients/gitlab/mr/schema/notes.py

@@ -1,5 +1,6 @@
 from pydantic import BaseModel, RootModel
 
+from ai_review.clients.gitlab.mr.schema.position import GitLabPositionSchema
 from ai_review.clients.gitlab.mr.schema.user import GitLabUserSchema
 
 
@@ -7,6 +8,7 @@ class GitLabNoteSchema(BaseModel):
     id: int
     body: str
     author: GitLabUserSchema | None = None
+    position: GitLabPositionSchema | None = None
 
 
 class GitLabGetMRNotesQuerySchema(BaseModel):

xai_review-0.30.0/ai_review/clients/gitlab/mr/schema/position.py

@@ -0,0 +1,13 @@
+from pydantic import BaseModel
+
+
+class GitLabPositionSchema(BaseModel):
+    position_type: str | None = "text"
+    base_sha: str | None = None
+    head_sha: str | None = None
+    start_sha: str | None = None
+    old_path: str | None = None
+    new_path: str | None = None
+    old_line: int | None = None
+    new_line: int | None = None
+    line_range: dict | None = None

{xai_review-0.28.0 → xai_review-0.30.0}/ai_review/libs/asynchronous/gather.py

@@ -2,6 +2,9 @@ import asyncio
 from typing import Awaitable, Iterable, TypeVar
 
 from ai_review.config import settings
+from ai_review.libs.logger import get_logger
+
+logger = get_logger("GATHER")
 
 T = TypeVar("T")
 
@@ -11,7 +14,11 @@ async def bounded_gather(coroutines: Iterable[Awaitable[T]]) -> tuple[T, ...]:
 
     async def wrap(coro: Awaitable[T]) -> T:
         async with sem:
-            return await coro
+            try:
+                return await coro
+            except Exception as error:
+                logger.warning(f"Task failed: {type(error).__name__}: {error}")
+                return error
 
     results = await asyncio.gather(*(wrap(coroutine) for coroutine in coroutines), return_exceptions=True)
     return tuple(results)

{xai_review-0.28.0 → xai_review-0.30.0}/ai_review/libs/config/review.py

@@ -19,6 +19,7 @@ class ReviewMode(StrEnum):
 
 class ReviewConfig(BaseModel):
     mode: ReviewMode = ReviewMode.FULL_FILE_DIFF
+    dry_run: bool = False
     inline_tag: str = Field(default="#ai-review-inline")
     inline_reply_tag: str = Field(default="#ai-review-inline-reply")
     summary_tag: str = Field(default="#ai-review-summary")

xai_review-0.30.0/ai_review/services/git/service.py

@@ -0,0 +1,66 @@
+import subprocess
+from pathlib import Path
+
+from ai_review.libs.logger import get_logger
+from ai_review.services.git.types import GitServiceProtocol
+
+logger = get_logger("GIT_SERVICE")
+
+
+class GitService(GitServiceProtocol):
+    def __init__(self, repo_dir: str = "."):
+        self.repo_dir = Path(repo_dir)
+
+    def run_git(self, *args: str) -> str:
+        cmd = ["git", *args]
+        logger.debug(f"Running git command: {' '.join(cmd)} (cwd={self.repo_dir})")
+
+        try:
+            result = subprocess.run(
+                cmd,
+                cwd=self.repo_dir,
+                capture_output=True,
+                text=True,
+                check=True,
+            )
+            if result.stderr.strip():
+                logger.debug(f"Git stderr: {result.stderr.strip()}")
+            return result.stdout
+        except subprocess.CalledProcessError as error:
+            logger.warning(
+                f"Git command failed (exit={error.returncode}): {' '.join(cmd)}\n"
+                f"stderr: {error.stderr.strip()}"
+            )
+            raise
+
+    def get_diff(self, base_sha: str, head_sha: str, unified: int = 3) -> str:
+        return self.run_git("diff", f"--unified={unified}", base_sha, head_sha)
+
+    def get_diff_for_file(self, base_sha: str, head_sha: str, file: str, unified: int = 3) -> str:
+        if not file:
+            logger.warning(f"Skipping git diff for empty filename (base={base_sha}, head={head_sha})")
+            return ""
+
+        logger.debug(f"Generating diff for {file} between {base_sha}..{head_sha}")
+        output = self.run_git("diff", f"--unified={unified}", base_sha, head_sha, "--", file)
+        if not output.strip():
+            logger.info(f"No diff found for {file} (possibly deleted or not tracked)")
+
+        return output
+
+    def get_changed_files(self, base_sha: str, head_sha: str) -> list[str]:
+        output = self.run_git("diff", "--name-only", base_sha, head_sha)
+        files = [line.strip() for line in output.splitlines() if line.strip()]
+        logger.debug(f"Changed files between {base_sha}..{head_sha}: {files}")
+        return files
+
+    def get_file_at_commit(self, file_path: str, sha: str) -> str | None:
+        if not file_path:
+            logger.warning(f"Skipping git show for empty file_path at {sha}")
+            return None
+
+        try:
+            return self.run_git("show", f"{sha}:{file_path}")
+        except subprocess.CalledProcessError as e:
+            logger.warning(f"File '{file_path}' not found in commit {sha}: {e.stderr.strip()}")
+            return None

xai_review-0.30.0/ai_review/services/review/gateway/review_dry_run_comment_gateway.py

@@ -0,0 +1,42 @@
+from ai_review.libs.asynchronous.gather import bounded_gather
+from ai_review.libs.logger import get_logger
+from ai_review.services.hook import hook
+from ai_review.services.review.gateway.review_comment_gateway import ReviewCommentGateway
+from ai_review.services.review.internal.inline.schema import InlineCommentListSchema, InlineCommentSchema
+from ai_review.services.review.internal.inline_reply.schema import InlineCommentReplySchema
+from ai_review.services.review.internal.summary.schema import SummaryCommentSchema
+from ai_review.services.review.internal.summary_reply.schema import SummaryCommentReplySchema
+from ai_review.services.vcs.types import VCSClientProtocol
+
+logger = get_logger("REVIEW_DRY_RUN_COMMENT_GATEWAY")
+
+
+class ReviewDryRunCommentGateway(ReviewCommentGateway):
+    def __init__(self, vcs: VCSClientProtocol):
+        super().__init__(vcs)
+        logger.warning("Running in DRY RUN mode — no comments will be posted to VCS")
+
+    async def process_inline_reply(self, thread_id: str, reply: InlineCommentReplySchema) -> None:
+        await hook.emit_inline_comment_reply_start(reply)
+        logger.info(f"[dry-run] Would create inline reply for thread {thread_id}:\n{reply.body_with_tag}")
+        await hook.emit_inline_comment_reply_complete(reply)
+
+    async def process_summary_reply(self, thread_id: str, reply: SummaryCommentReplySchema) -> None:
+        await hook.emit_summary_comment_reply_start(reply)
+        logger.info(f"[dry-run] Would create summary reply for thread {thread_id}:\n{reply.body_with_tag}")
+        await hook.emit_summary_comment_reply_complete(reply)
+
+    async def process_inline_comment(self, comment: InlineCommentSchema) -> None:
+        await hook.emit_inline_comment_start(comment)
+        logger.info(
+            f"[dry-run] Would create inline comment for {comment.file}:{comment.line}:\n{comment.body_with_tag}"
+        )
+        await hook.emit_inline_comment_complete(comment)
+
+    async def process_summary_comment(self, comment: SummaryCommentSchema) -> None:
+        await hook.emit_summary_comment_start(comment)
+        logger.info(f"[dry-run] Would create summary comment:\n{comment.body_with_tag}")
+        await hook.emit_summary_comment_complete(comment)
+
+    async def process_inline_comments(self, comments: InlineCommentListSchema) -> None:
+        await bounded_gather([self.process_inline_comment(comment) for comment in comments.root])

{xai_review-0.28.0 → xai_review-0.30.0}/ai_review/services/review/service.py

@@ -1,3 +1,4 @@
+from ai_review.config import settings
 from ai_review.libs.logger import get_logger
 from ai_review.services.artifacts.service import ArtifactsService
 from ai_review.services.cost.service import CostService
@@ -5,8 +6,9 @@ from ai_review.services.diff.service import DiffService
 from ai_review.services.git.service import GitService
 from ai_review.services.llm.factory import get_llm_client
 from ai_review.services.prompt.service import PromptService
-from ai_review.services.review.gateway.comment import ReviewCommentGateway
-from ai_review.services.review.gateway.llm import ReviewLLMGateway
+from ai_review.services.review.gateway.review_comment_gateway import ReviewCommentGateway
+from ai_review.services.review.gateway.review_dry_run_comment_gateway import ReviewDryRunCommentGateway
+from ai_review.services.review.gateway.review_llm_gateway import ReviewLLMGateway
 from ai_review.services.review.internal.inline.service import InlineCommentService
 from ai_review.services.review.internal.inline_reply.service import InlineCommentReplyService
 from ai_review.services.review.internal.policy.service import ReviewPolicyService
@@ -42,7 +44,11 @@ class ReviewService:
             cost=self.cost,
             artifacts=self.artifacts
         )
-        self.review_comment_gateway = ReviewCommentGateway(vcs=self.vcs)
+        self.review_comment_gateway = (
+            ReviewDryRunCommentGateway(vcs=self.vcs)
+            if settings.review.dry_run
+            else ReviewCommentGateway(vcs=self.vcs)
+        )
 
         self.inline_review_runner = InlineReviewRunner(
             vcs=self.vcs,

{xai_review-0.28.0 → xai_review-0.30.0}/ai_review/services/vcs/bitbucket/adapter.py

@@ -13,11 +13,14 @@ def get_review_comment_from_bitbucket_pr_comment(comment: BitbucketPRCommentSche
         username=user.nickname if user else "",
     )
 
+    file = comment.inline.path if comment.inline and comment.inline.path else None
+    line = comment.inline.to_line if comment.inline else None
+
     return ReviewCommentSchema(
         id=comment.id,
         body=comment.content.raw or "",
-        file=comment.inline.path if comment.inline else None,
-        line=comment.inline.to_line if comment.inline else None,
+        file=file,
+        line=line,
         author=author,
         parent_id=parent_id,
         thread_id=thread_id,

{xai_review-0.28.0 → xai_review-0.30.0}/ai_review/services/vcs/bitbucket/client.py

@@ -207,22 +207,33 @@ class BitbucketVCSClient(VCSClientProtocol):
         try:
             comments = await self.get_inline_comments()
 
-            threads: dict[str | int, list[ReviewCommentSchema]] = defaultdict(list)
+            threads_by_id: dict[str | int, list[ReviewCommentSchema]] = defaultdict(list)
             for comment in comments:
-                threads[comment.thread_id].append(comment)
+                if not comment.file:
+                    continue
 
-            logger.info(f"Built {len(threads)} inline threads for {self.pull_request_ref}")
+                threads_by_id[comment.thread_id].append(comment)
 
-            return [
-                ReviewThreadSchema(
-                    id=thread_id,
-                    kind=ThreadKind.INLINE,
-                    file=thread[0].file,
-                    line=thread[0].line,
-                    comments=sorted(thread, key=lambda c: int(c.id)),
+            logger.info(f"Built {len(threads_by_id)} inline threads for {self.pull_request_ref}")
+
+            threads: list[ReviewThreadSchema] = []
+            for thread_id, thread in threads_by_id.items():
+                file = thread[0].file
+                line = thread[0].line
+                if not file:
+                    continue
+
+                threads.append(
+                    ReviewThreadSchema(
+                        id=thread_id,
+                        kind=ThreadKind.INLINE,
+                        file=file,
+                        line=line,
+                        comments=sorted(thread, key=lambda t: int(t.id)),
+                    )
                 )
-                for thread_id, thread in threads.items()
-            ]
+
+            return threads
         except Exception as error:
             logger.exception(f"Failed to fetch inline threads for {self.pull_request_ref}: {error}")
             return []

{xai_review-0.28.0 → xai_review-0.30.0}/ai_review/services/vcs/gitlab/adapter.py

@@ -16,11 +16,13 @@ def get_review_comment_from_gitlab_note(
         note: GitLabNoteSchema,
         discussion: GitLabDiscussionSchema
 ) -> ReviewCommentSchema:
+    position = note.position or discussion.position
+
     return ReviewCommentSchema(
         id=note.id,
         body=note.body or "",
-        file=discussion.position.new_path if discussion.position else None,
-        line=discussion.position.new_line if discussion.position else None,
+        file=position.new_path if position else None,
+        line=position.new_line if position else None,
         author=get_user_from_gitlab_user(note.author),
         thread_id=discussion.id,
     )

{xai_review-0.28.0 → xai_review-0.30.0}/ai_review/services/vcs/gitlab/client.py

@@ -1,8 +1,6 @@
 from ai_review.clients.gitlab.client import get_gitlab_http_client
-from ai_review.clients.gitlab.mr.schema.discussions import (
-    GitLabDiscussionPositionSchema,
-    GitLabCreateMRDiscussionRequestSchema,
-)
+from ai_review.clients.gitlab.mr.schema.discussions import GitLabCreateMRDiscussionRequestSchema
+from ai_review.clients.gitlab.mr.schema.position import GitLabPositionSchema
 from ai_review.config import settings
 from ai_review.libs.logger import get_logger
 from ai_review.services.vcs.gitlab.adapter import get_user_from_gitlab_user, get_review_comment_from_gitlab_note
@@ -135,7 +133,7 @@ class GitLabVCSClient(VCSClientProtocol):
 
             request = GitLabCreateMRDiscussionRequestSchema(
                 body=message,
-                position=GitLabDiscussionPositionSchema(
+                position=GitLabPositionSchema(
                     position_type="text",
                     base_sha=response.diff_refs.base_sha,
                     head_sha=response.diff_refs.head_sha,
@@ -191,19 +189,28 @@ class GitLabVCSClient(VCSClientProtocol):
             )
             logger.info(f"Fetched inline threads for MR {self.merge_request_ref}")
 
-            threads = [
-                ReviewThreadSchema(
-                    id=discussion.id,
-                    kind=ThreadKind.INLINE,
-                    file=discussion.position.new_path if discussion.position else None,
-                    line=discussion.position.new_line if discussion.position else None,
-                    comments=[
-                        get_review_comment_from_gitlab_note(note, discussion)
-                        for note in discussion.notes
-                    ]
+            threads: list[ReviewThreadSchema] = []
+            for discussion in response.root:
+                if not discussion.notes:
+                    continue
+
+                position = discussion.position or (
+                    discussion.notes[0].position if discussion.notes else None
                 )
-                for discussion in response.root if discussion.notes
-            ]
+
+                threads.append(
+                    ReviewThreadSchema(
+                        id=discussion.id,
+                        kind=ThreadKind.INLINE,
+                        file=position.new_path if position else None,
+                        line=position.new_line if position else None,
+                        comments=[
+                            get_review_comment_from_gitlab_note(note, discussion)
+                            for note in discussion.notes
+                        ],
+                    )
+                )
+
             logger.info(f"Built {len(threads)} inline threads for MR {self.merge_request_ref}")
             return threads
         except Exception as error:

{xai_review-0.28.0 → xai_review-0.30.0}/ai_review/tests/fixtures/clients/gitlab.py

@@ -12,13 +12,14 @@ from ai_review.clients.gitlab.mr.schema.discussions import (
     GitLabGetMRDiscussionsResponseSchema,
     GitLabCreateMRDiscussionRequestSchema,
     GitLabCreateMRDiscussionResponseSchema,
-    GitLabCreateMRDiscussionReplyResponseSchema, GitLabDiscussionPositionSchema,
+    GitLabCreateMRDiscussionReplyResponseSchema,
 )
 from ai_review.clients.gitlab.mr.schema.notes import (
     GitLabNoteSchema,
     GitLabGetMRNotesResponseSchema,
     GitLabCreateMRNoteResponseSchema,
 )
+from ai_review.clients.gitlab.mr.schema.position import GitLabPositionSchema
 from ai_review.clients.gitlab.mr.types import GitLabMergeRequestsHTTPClientProtocol
 from ai_review.config import settings
 from ai_review.libs.config.vcs.base import GitLabVCSConfig
@@ -81,17 +82,42 @@ class FakeGitLabMergeRequestsHTTPClient(GitLabMergeRequestsHTTPClientProtocol):
                 GitLabDiscussionSchema(
                     id="discussion-1",
                     notes=[
-                        GitLabNoteSchema(id=10, body="Inline comment A"),
-                        GitLabNoteSchema(id=11, body="Inline comment B"),
+                        GitLabNoteSchema(
+                            id=10,
+                            body="Inline comment A",
+                            position=GitLabPositionSchema(
+                                base_sha="abc123",
+                                head_sha="def456",
+                                start_sha="ghi789",
+                                new_path="src/app.py",
+                                new_line=12,
+                            ),
+                        ),
+                        GitLabNoteSchema(
+                            id=11,
+                            body="Inline comment B",
+                            position=GitLabPositionSchema(
+                                base_sha="abc123",
+                                head_sha="def456",
+                                start_sha="ghi789",
+                                new_path="src/app.py",
+                                new_line=14,
+                            ),
+                        ),
                     ],
-                    position=GitLabDiscussionPositionSchema(
+                    position=GitLabPositionSchema(
                         base_sha="abc123",
                         head_sha="def456",
                         start_sha="ghi789",
                         new_path="src/app.py",
                         new_line=12,
                     ),
-                )
+                ),
+                GitLabDiscussionSchema(
+                    id="discussion-2",
+                    notes=[GitLabNoteSchema(id=20, body="Outdated diff comment", position=None)],
+                    position=None,
+                ),
             ]
         )
 

xai_review-0.28.0/ai_review/tests/fixtures/services/review/gateway/comment.py → xai_review-0.30.0/ai_review/tests/fixtures/services/review/gateway/review_comment_gateway.py

@@ -2,7 +2,7 @@ from typing import Any
 
 import pytest
 
-from ai_review.services.review.gateway.comment import ReviewCommentGateway
+from ai_review.services.review.gateway.review_comment_gateway import ReviewCommentGateway
 from ai_review.services.review.gateway.types import ReviewCommentGatewayProtocol
 from ai_review.services.review.internal.inline.schema import InlineCommentSchema, InlineCommentListSchema
 from ai_review.services.review.internal.inline_reply.schema import InlineCommentReplySchema

xai_review-0.30.0/ai_review/tests/fixtures/services/review/gateway/review_dry_run_comment_gateway.py

@@ -0,0 +1,103 @@
+from typing import Any
+
+import pytest
+
+from ai_review.services.review.gateway.review_dry_run_comment_gateway import ReviewDryRunCommentGateway
+from ai_review.services.review.gateway.types import ReviewCommentGatewayProtocol
+from ai_review.services.review.internal.inline.schema import InlineCommentSchema, InlineCommentListSchema
+from ai_review.services.review.internal.inline_reply.schema import InlineCommentReplySchema
+from ai_review.services.review.internal.summary.schema import SummaryCommentSchema
+from ai_review.services.review.internal.summary_reply.schema import SummaryCommentReplySchema
+from ai_review.services.vcs.types import (
+    UserSchema,
+    ThreadKind,
+    ReviewThreadSchema,
+    ReviewCommentSchema,
+    VCSClientProtocol,
+)
+
+
+class FakeReviewDryRunCommentGateway(ReviewCommentGatewayProtocol):
+    def __init__(self, responses: dict[str, Any] | None = None):
+        self.calls: list[tuple[str, dict]] = []
+
+        fake_user = UserSchema(id="u1", username="tester", name="Tester")
+
+        fake_inline_thread = ReviewThreadSchema(
+            id="t1",
+            kind=ThreadKind.INLINE,
+            file="file.py",
+            line=10,
+            comments=[
+                ReviewCommentSchema(
+                    id="c1",
+                    body="#ai-review-inline some comment",
+                    file="file.py",
+                    line=10,
+                    author=fake_user,
+                ),
+            ],
+        )
+
+        fake_summary_thread = ReviewThreadSchema(
+            id="t2",
+            kind=ThreadKind.SUMMARY,
+            comments=[
+                ReviewCommentSchema(
+                    id="c2",
+                    body="#ai-review-summary summary comment",
+                    author=fake_user,
+                ),
+            ],
+        )
+
+        self.responses = responses or {
+            "get_inline_threads": [fake_inline_thread],
+            "get_summary_threads": [fake_summary_thread],
+            "has_existing_inline_comments": False,
+            "has_existing_summary_comments": False,
+        }
+
+    # --- Методы чтения ---
+    async def get_inline_threads(self) -> list[ReviewThreadSchema]:
+        self.calls.append(("get_inline_threads", {}))
+        return self.responses["get_inline_threads"]
+
+    async def get_summary_threads(self) -> list[ReviewThreadSchema]:
+        self.calls.append(("get_summary_threads", {}))
+        return self.responses["get_summary_threads"]
+
+    async def has_existing_inline_comments(self) -> bool:
+        self.calls.append(("has_existing_inline_comments", {}))
+        return self.responses["has_existing_inline_comments"]
+
+    async def has_existing_summary_comments(self) -> bool:
+        self.calls.append(("has_existing_summary_comments", {}))
+        return self.responses["has_existing_summary_comments"]
+
+    async def process_inline_reply(self, thread_id: str, reply: InlineCommentReplySchema) -> None:
+        self.calls.append(("process_inline_reply", {"thread_id": thread_id, "reply": reply}))
+
+    async def process_summary_reply(self, thread_id: str, reply: SummaryCommentReplySchema) -> None:
+        self.calls.append(("process_summary_reply", {"thread_id": thread_id, "reply": reply}))
+
+    async def process_inline_comment(self, comment: InlineCommentSchema) -> None:
+        self.calls.append(("process_inline_comment", {"comment": comment}))
+
+    async def process_summary_comment(self, comment: SummaryCommentSchema) -> None:
+        self.calls.append(("process_summary_comment", {"comment": comment}))
+
+    async def process_inline_comments(self, comments: InlineCommentListSchema) -> None:
+        self.calls.append(("process_inline_comments", {"comments": comments}))
+        for comment in comments.root:
+            await self.process_inline_comment(comment)
+
+
+@pytest.fixture
+def fake_review_dry_run_comment_gateway() -> FakeReviewDryRunCommentGateway:
+    return FakeReviewDryRunCommentGateway()
+
+
+@pytest.fixture
+def review_dry_run_comment_gateway(fake_vcs_client: VCSClientProtocol) -> ReviewDryRunCommentGateway:
+    return ReviewDryRunCommentGateway(vcs=fake_vcs_client)

xai_review-0.30.0/ai_review/tests/fixtures/services/review/gateway/review_llm_gateway.py

@@ -0,0 +1,34 @@
+import pytest
+
+from ai_review.services.review.gateway.review_llm_gateway import ReviewLLMGateway
+from ai_review.services.review.gateway.types import ReviewLLMGatewayProtocol
+from ai_review.tests.fixtures.services.artifacts import FakeArtifactsService
+from ai_review.tests.fixtures.services.cost import FakeCostService
+from ai_review.tests.fixtures.services.llm import FakeLLMClient
+
+
+class FakeReviewLLMGateway(ReviewLLMGatewayProtocol):
+    def __init__(self):
+        self.calls: list[tuple[str, dict]] = []
+
+    async def ask(self, prompt: str, prompt_system: str) -> str:
+        self.calls.append(("ask", {"prompt": prompt, "prompt_system": prompt_system}))
+        return "FAKE_LLM_RESPONSE"
+
+
+@pytest.fixture
+def fake_review_llm_gateway() -> FakeReviewLLMGateway:
+    return FakeReviewLLMGateway()
+
+
+@pytest.fixture
+def review_llm_gateway(
+        fake_llm_client: FakeLLMClient,
+        fake_cost_service: FakeCostService,
+        fake_artifacts_service: FakeArtifactsService,
+) -> ReviewLLMGateway:
+    return ReviewLLMGateway(
+        llm=fake_llm_client,
+        cost=fake_cost_service,
+        artifacts=fake_artifacts_service,
+    )

xai_review-0.28.0/ai_review/tests/suites/services/review/gateway/test_comment.py → xai_review-0.30.0/ai_review/tests/suites/services/review/gateway/test_review_comment_gateway.py

@@ -1,7 +1,7 @@
 import pytest
 
 from ai_review.config import settings
-from ai_review.services.review.gateway.comment import ReviewCommentGateway
+from ai_review.services.review.gateway.review_comment_gateway import ReviewCommentGateway
 from ai_review.services.review.internal.inline.schema import InlineCommentSchema, InlineCommentListSchema
 from ai_review.services.review.internal.inline_reply.schema import InlineCommentReplySchema
 from ai_review.services.review.internal.summary.schema import SummaryCommentSchema