xai-review 0.28.0__py3-none-any.whl → 0.30.0__py3-none-any.whl

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

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

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

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)

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

ai_review/services/git/service.py

@@ -1,35 +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:
-        result = subprocess.run(
-            ["git", *args],
-            cwd=self.repo_dir,
-            capture_output=True,
-            text=True,
-            check=True,
-        )
-        return result.stdout
+        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:
-        return self.run_git("diff", f"--unified={unified}", base_sha, head_sha, "--", file)
+        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)
-        return [line.strip() for line in output.splitlines() if line.strip()]
+        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:
+        except subprocess.CalledProcessError as e:
+            logger.warning(f"File '{file_path}' not found in commit {sha}: {e.stderr.strip()}")
             return None

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])

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,

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,

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 []

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,
     )

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:

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,
+                ),
             ]
         )
 

ai_review/tests/fixtures/services/review/gateway/{comment.py → 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

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)

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,
+    )

ai_review/tests/suites/services/review/gateway/{test_comment.py → 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

ai_review/tests/suites/services/review/gateway/test_review_dry_run_comment_gateway.py

@@ -0,0 +1,93 @@
+import pytest
+
+from ai_review.services.review.gateway.review_dry_run_comment_gateway import ReviewDryRunCommentGateway
+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.tests.fixtures.services.vcs import FakeVCSClient
+
+
+@pytest.mark.asyncio
+async def test_process_inline_reply_dry_run_logs_and_no_vcs_calls(
+        capsys,
+        fake_vcs_client: FakeVCSClient,
+        review_dry_run_comment_gateway: ReviewDryRunCommentGateway
+):
+    """Dry-run: should log the inline reply but not call VCS."""
+    reply = InlineCommentReplySchema(message="AI reply dry-run")
+    await review_dry_run_comment_gateway.process_inline_reply("t1", reply)
+    output = capsys.readouterr().out
+
+    assert "[dry-run]" in output
+    assert "Would create inline reply" in output
+    assert not any(call[0].startswith("create_") for call in fake_vcs_client.calls)
+
+
+@pytest.mark.asyncio
+async def test_process_summary_reply_dry_run_logs_and_no_vcs_calls(
+        capsys,
+        fake_vcs_client: FakeVCSClient,
+        review_dry_run_comment_gateway: ReviewDryRunCommentGateway
+):
+    """Dry-run: should log the summary reply but not call VCS."""
+    reply = SummaryCommentReplySchema(text="Dry-run summary reply")
+    await review_dry_run_comment_gateway.process_summary_reply("t2", reply)
+    output = capsys.readouterr().out
+
+    assert "[dry-run]" in output
+    assert "Would create summary reply" in output
+    assert not any(call[0].startswith("create_") for call in fake_vcs_client.calls)
+
+
+@pytest.mark.asyncio
+async def test_process_inline_comment_dry_run_logs_and_no_vcs_calls(
+        capsys,
+        fake_vcs_client: FakeVCSClient,
+        review_dry_run_comment_gateway: ReviewDryRunCommentGateway
+):
+    """Dry-run: should log inline comment without creating one."""
+    comment = InlineCommentSchema(file="a.py", line=10, message="Test comment")
+    await review_dry_run_comment_gateway.process_inline_comment(comment)
+    output = capsys.readouterr().out
+
+    assert "[dry-run]" in output
+    assert "Would create inline comment" in output
+    assert "a.py" in output
+    assert not any(call[0].startswith("create_") for call in fake_vcs_client.calls)
+
+
+@pytest.mark.asyncio
+async def test_process_summary_comment_dry_run_logs_and_no_vcs_calls(
+        capsys,
+        fake_vcs_client: FakeVCSClient,
+        review_dry_run_comment_gateway: ReviewDryRunCommentGateway
+):
+    """Dry-run: should log summary comment but not send it."""
+    comment = SummaryCommentSchema(text="Dry-run summary comment")
+    await review_dry_run_comment_gateway.process_summary_comment(comment)
+    output = capsys.readouterr().out
+
+    assert "[dry-run]" in output
+    assert "Would create summary comment" in output
+    assert not any(call[0].startswith("create_") for call in fake_vcs_client.calls)
+
+
+@pytest.mark.asyncio
+async def test_process_inline_comments_iterates_all(
+        capsys,
+        fake_vcs_client: FakeVCSClient,
+        review_dry_run_comment_gateway: ReviewDryRunCommentGateway
+):
+    """Dry-run: should iterate through all inline comments and log each."""
+    comments = InlineCommentListSchema(root=[
+        InlineCommentSchema(file="a.py", line=1, message="C1"),
+        InlineCommentSchema(file="b.py", line=2, message="C2"),
+    ])
+    await review_dry_run_comment_gateway.process_inline_comments(comments)
+    output = capsys.readouterr().out
+
+    assert "[dry-run]" in output
+    assert "a.py" in output
+    assert "b.py" in output
+    assert not any(call[0].startswith("create_") for call in fake_vcs_client.calls)

ai_review/tests/suites/services/review/gateway/{test_llm.py → test_review_llm_gateway.py}

@@ -1,26 +1,12 @@
 import pytest
 
 from ai_review.services.llm.types import ChatResultSchema
-from ai_review.services.review.gateway.llm import ReviewLLMGateway
+from ai_review.services.review.gateway.review_llm_gateway import ReviewLLMGateway
 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
 
 
-@pytest.fixture
-def review_llm_gateway(
-        fake_llm_client: FakeLLMClient,
-        fake_cost_service: FakeCostService,
-        fake_artifacts_service: FakeArtifactsService,
-) -> ReviewLLMGateway:
-    """Fixture providing ReviewLLMGateway with fake dependencies."""
-    return ReviewLLMGateway(
-        llm=fake_llm_client,
-        cost=fake_cost_service,
-        artifacts=fake_artifacts_service,
-    )
-
-
 @pytest.mark.asyncio
 async def test_ask_happy_path(
         review_llm_gateway: ReviewLLMGateway,

ai_review/tests/suites/services/review/runner/test_context.py

@@ -4,8 +4,8 @@ from ai_review.services.review.runner.context import ContextReviewRunner
 from ai_review.tests.fixtures.services.cost import FakeCostService
 from ai_review.tests.fixtures.services.diff import FakeDiffService
 from ai_review.tests.fixtures.services.prompt import FakePromptService
-from ai_review.tests.fixtures.services.review.gateway.comment import FakeReviewCommentGateway
-from ai_review.tests.fixtures.services.review.gateway.llm import FakeReviewLLMGateway
+from ai_review.tests.fixtures.services.review.gateway.review_comment_gateway import FakeReviewCommentGateway
+from ai_review.tests.fixtures.services.review.gateway.review_llm_gateway import FakeReviewLLMGateway
 from ai_review.tests.fixtures.services.review.internal.inline import FakeInlineCommentService
 from ai_review.tests.fixtures.services.review.internal.policy import FakeReviewPolicyService
 from ai_review.tests.fixtures.services.vcs import FakeVCSClient

ai_review/tests/suites/services/review/runner/test_inline.py

@@ -6,8 +6,8 @@ from ai_review.tests.fixtures.services.cost import FakeCostService
 from ai_review.tests.fixtures.services.diff import FakeDiffService
 from ai_review.tests.fixtures.services.git import FakeGitService
 from ai_review.tests.fixtures.services.prompt import FakePromptService
-from ai_review.tests.fixtures.services.review.gateway.comment import FakeReviewCommentGateway
-from ai_review.tests.fixtures.services.review.gateway.llm import FakeReviewLLMGateway
+from ai_review.tests.fixtures.services.review.gateway.review_comment_gateway import FakeReviewCommentGateway
+from ai_review.tests.fixtures.services.review.gateway.review_llm_gateway import FakeReviewLLMGateway
 from ai_review.tests.fixtures.services.review.internal.inline import FakeInlineCommentService
 from ai_review.tests.fixtures.services.review.internal.policy import FakeReviewPolicyService
 from ai_review.tests.fixtures.services.vcs import FakeVCSClient

ai_review/tests/suites/services/review/runner/test_inline_reply.py

@@ -6,8 +6,8 @@ from ai_review.tests.fixtures.services.cost import FakeCostService
 from ai_review.tests.fixtures.services.diff import FakeDiffService
 from ai_review.tests.fixtures.services.git import FakeGitService
 from ai_review.tests.fixtures.services.prompt import FakePromptService
-from ai_review.tests.fixtures.services.review.gateway.comment import FakeReviewCommentGateway
-from ai_review.tests.fixtures.services.review.gateway.llm import FakeReviewLLMGateway
+from ai_review.tests.fixtures.services.review.gateway.review_comment_gateway import FakeReviewCommentGateway
+from ai_review.tests.fixtures.services.review.gateway.review_llm_gateway import FakeReviewLLMGateway
 from ai_review.tests.fixtures.services.review.internal.inline_reply import FakeInlineCommentReplyService
 from ai_review.tests.fixtures.services.vcs import FakeVCSClient
 

ai_review/tests/suites/services/review/runner/test_summary.py

@@ -5,8 +5,8 @@ from ai_review.services.review.runner.summary import SummaryReviewRunner
 from ai_review.tests.fixtures.services.cost import FakeCostService
 from ai_review.tests.fixtures.services.diff import FakeDiffService
 from ai_review.tests.fixtures.services.prompt import FakePromptService
-from ai_review.tests.fixtures.services.review.gateway.comment import FakeReviewCommentGateway
-from ai_review.tests.fixtures.services.review.gateway.llm import FakeReviewLLMGateway
+from ai_review.tests.fixtures.services.review.gateway.review_comment_gateway import FakeReviewCommentGateway
+from ai_review.tests.fixtures.services.review.gateway.review_llm_gateway import FakeReviewLLMGateway
 from ai_review.tests.fixtures.services.review.internal.policy import FakeReviewPolicyService
 from ai_review.tests.fixtures.services.review.internal.summary import FakeSummaryCommentService
 from ai_review.tests.fixtures.services.vcs import FakeVCSClient

ai_review/tests/suites/services/review/runner/test_summary_reply.py

@@ -5,8 +5,8 @@ from ai_review.services.vcs.types import ReviewInfoSchema, ReviewThreadSchema, R
 from ai_review.tests.fixtures.services.cost import FakeCostService
 from ai_review.tests.fixtures.services.diff import FakeDiffService
 from ai_review.tests.fixtures.services.prompt import FakePromptService
-from ai_review.tests.fixtures.services.review.gateway.comment import FakeReviewCommentGateway
-from ai_review.tests.fixtures.services.review.gateway.llm import FakeReviewLLMGateway
+from ai_review.tests.fixtures.services.review.gateway.review_comment_gateway import FakeReviewCommentGateway
+from ai_review.tests.fixtures.services.review.gateway.review_llm_gateway import FakeReviewLLMGateway
 from ai_review.tests.fixtures.services.review.internal.policy import FakeReviewPolicyService
 from ai_review.tests.fixtures.services.review.internal.summary_reply import FakeSummaryCommentReplyService
 from ai_review.tests.fixtures.services.vcs import FakeVCSClient

ai_review/tests/suites/services/review/test_service.py

@@ -1,6 +1,8 @@
 import pytest
 
 from ai_review.services.llm.types import ChatResultSchema
+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.service import ReviewService
 from ai_review.tests.fixtures.services.cost import FakeCostService
 from ai_review.tests.fixtures.services.review.runner.context import FakeContextReviewRunner
@@ -91,3 +93,19 @@ def test_report_total_cost_no_data(capsys, review_service: ReviewService):
     output = capsys.readouterr().out
 
     assert "No cost data collected" in output
+
+
+def test_review_service_uses_dry_run_comment_gateway(monkeypatch: pytest.MonkeyPatch):
+    """Should use ReviewDryRunCommentGateway when settings.review.dry_run=True."""
+    monkeypatch.setattr("ai_review.config.settings.review.dry_run", True)
+
+    service = ReviewService()
+    assert type(service.review_comment_gateway) is ReviewDryRunCommentGateway
+
+
+def test_review_service_uses_real_comment_gateway(monkeypatch: pytest.MonkeyPatch):
+    """Should use normal ReviewCommentGateway when dry_run=False."""
+    monkeypatch.setattr("ai_review.config.settings.review.dry_run", False)
+
+    service = ReviewService()
+    assert type(service.review_comment_gateway) is ReviewCommentGateway

ai_review/tests/suites/services/vcs/gitlab/test_adapter.py

@@ -1,8 +1,6 @@
-from ai_review.clients.gitlab.mr.schema.discussions import (
-    GitLabDiscussionSchema,
-    GitLabDiscussionPositionSchema,
-)
+from ai_review.clients.gitlab.mr.schema.discussions import GitLabDiscussionSchema
 from ai_review.clients.gitlab.mr.schema.notes import GitLabNoteSchema
+from ai_review.clients.gitlab.mr.schema.position import GitLabPositionSchema
 from ai_review.clients.gitlab.mr.schema.user import GitLabUserSchema
 from ai_review.services.vcs.gitlab.adapter import get_review_comment_from_gitlab_note
 from ai_review.services.vcs.types import ReviewCommentSchema, UserSchema
@@ -18,7 +16,7 @@ def test_maps_all_fields_correctly():
     discussion = GitLabDiscussionSchema(
         id="42",
         notes=[note],
-        position=GitLabDiscussionPositionSchema(
+        position=GitLabPositionSchema(
             base_sha="AAA000",
             head_sha="BBB111",
             start_sha="CCC222",
@@ -48,7 +46,7 @@ def test_maps_with_missing_author():
     discussion = GitLabDiscussionSchema(
         id="7",
         notes=[note],
-        position=GitLabDiscussionPositionSchema(
+        position=GitLabPositionSchema(
             base_sha="AAA000",
             head_sha="BBB111",
             start_sha="CCC222",
@@ -103,3 +101,34 @@ def test_maps_with_empty_body_and_defaults():
     assert result.line is None
     assert result.thread_id == "100"
     assert isinstance(result.author, UserSchema)
+
+
+def test_maps_with_note_position_fallback():
+    """Should use note.position when discussion.position is missing."""
+    note = GitLabNoteSchema(
+        id=77,
+        body="Inline note with its own position",
+        author=GitLabUserSchema(id=2, name="Carol", username="carol"),
+        position=GitLabPositionSchema(
+            base_sha="aaa",
+            head_sha="bbb",
+            start_sha="ccc",
+            new_path="module/utils.py",
+            new_line=42,
+        ),
+    )
+    discussion = GitLabDiscussionSchema(
+        id="200",
+        notes=[note],
+        position=None,
+    )
+
+    result = get_review_comment_from_gitlab_note(note, discussion)
+
+    assert isinstance(result, ReviewCommentSchema)
+    assert result.id == 77
+    assert result.file == "module/utils.py"
+    assert result.line == 42
+    assert result.thread_id == "200"
+    assert result.body == "Inline note with its own position"
+    assert result.author.name == "Carol"

ai_review/tests/suites/services/vcs/gitlab/test_client.py

@@ -71,14 +71,20 @@ async def test_get_inline_comments_returns_expected_list(
         gitlab_vcs_client: GitLabVCSClient,
         fake_gitlab_merge_requests_http_client: FakeGitLabMergeRequestsHTTPClient,
 ):
-    """Should return inline comments from MR discussions."""
+    """Should return inline comments from MR discussions (including ones without position)."""
     comments = await gitlab_vcs_client.get_inline_comments()
 
     assert all(isinstance(c, ReviewCommentSchema) for c in comments)
-    assert len(comments) == 2
+    assert len(comments) == 3
 
     first = comments[0]
     assert first.body == "Inline comment A"
+    assert first.file == "src/app.py"
+    assert first.line == 12
+
+    last = comments[-1]
+    assert last.file is None
+    assert last.line is None
 
     called_methods = [name for name, _ in fake_gitlab_merge_requests_http_client.calls]
     assert called_methods == ["get_discussions"]
@@ -183,19 +189,25 @@ async def test_get_inline_threads_returns_valid_schema(
         gitlab_vcs_client: GitLabVCSClient,
         fake_gitlab_merge_requests_http_client: FakeGitLabMergeRequestsHTTPClient,
 ):
-    """Should transform GitLab discussions into inline threads with proper fields."""
+    """Should transform GitLab discussions into inline threads, including those without position."""
     threads = await gitlab_vcs_client.get_inline_threads()
 
-    assert all(isinstance(t, ReviewThreadSchema) for t in threads)
-    assert len(threads) == 1
+    assert all(isinstance(thread, ReviewThreadSchema) for thread in threads)
+    assert len(threads) == 2
 
-    thread = threads[0]
-    assert thread.id == "discussion-1"
-    assert thread.kind == ThreadKind.INLINE
-    assert thread.file == "src/app.py"
-    assert thread.line == 12
-    assert len(thread.comments) == 2
-    assert isinstance(thread.comments[0], ReviewCommentSchema)
+    first_thread = threads[0]
+    assert first_thread.id == "discussion-1"
+    assert first_thread.kind == ThreadKind.INLINE
+    assert first_thread.file == "src/app.py"
+    assert first_thread.line == 12
+    assert len(first_thread.comments) == 2
+    assert isinstance(first_thread.comments[0], ReviewCommentSchema)
+
+    second_thread = threads[1]
+    assert second_thread.id == "discussion-2"
+    assert second_thread.file is None
+    assert second_thread.line is None
+    assert len(second_thread.comments) == 1
 
     called = [name for name, _ in fake_gitlab_merge_requests_http_client.calls]
     assert "get_discussions" in called

{xai_review-0.28.0.dist-info → xai_review-0.30.0.dist-info}/METADATA

@@ -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.dist-info → xai_review-0.30.0.dist-info}/RECORD

@@ -49,8 +49,9 @@ ai_review/clients/gitlab/mr/client.py,sha256=ydkecV9SlmXJvkoBVF0c-5C7RsadYTP9RXC
 ai_review/clients/gitlab/mr/types.py,sha256=3PpUjnek6AOAiIC0tYRuX4RwB2UeoIrSxigTvsGC1iI,1563
 ai_review/clients/gitlab/mr/schema/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/clients/gitlab/mr/schema/changes.py,sha256=qYvAih3zSvNuF4C8PtnuwmZVFk8VHM14miPxgC8hWG4,824
-ai_review/clients/gitlab/mr/schema/discussions.py,sha256=mbVc-n39MjJcWsSHQCFsLTJuvDZb3QrM-CjWYBNf8us,1083
-ai_review/clients/gitlab/mr/schema/notes.py,sha256=Z0NxDbgEo9CpKZbBjJNpoEuv8Q_u0oiH5GAjBeS2KMo,561
+ai_review/clients/gitlab/mr/schema/discussions.py,sha256=pZ6ajZaQQI7_jO83FFXQmR_xruzda_pVSv_V0Klvc2w,966
+ai_review/clients/gitlab/mr/schema/notes.py,sha256=9wmwULegmTO6ETSjYlMC6Fc_DIeT_Wa6hiOB722ykUA,687
+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
@@ -65,7 +66,7 @@ ai_review/libs/json.py,sha256=g-P5_pNUomQ-bGHCXASvPKj9Og0s9MaLFVEAkzqGp1A,350
 ai_review/libs/logger.py,sha256=LbXR2Zk1btJ-83I-vHee7cUETgT1mHToSsqEI_8uM0U,370
 ai_review/libs/resources.py,sha256=s9taAbL1Shl_GiGkPpkkivUcM1Yh6d_IQAG97gffsJU,748
 ai_review/libs/asynchronous/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/libs/asynchronous/gather.py,sha256=wH65sqBfrnwA1A9Juc5MSyLCJrcxzRqk2m0tSZ1tX8A,490
+ai_review/libs/asynchronous/gather.py,sha256=ns7eFWeV6ukRM13v78ZGu6WEpojH6QoPS-V0KHdn-sk,735
 ai_review/libs/config/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/libs/config/artifacts.py,sha256=8BzbQu5GxwV6i6qzrUKM1De1Ogb00Ph5WTqwZ3fVpGg,483
 ai_review/libs/config/base.py,sha256=sPf3OKeF1ID0ouOwiVaUtvpWuZXJXQvIw5kbnPUyN9o,686
@@ -73,7 +74,7 @@ ai_review/libs/config/core.py,sha256=ZQ2QtYr7vAF0tXbVLvVwk9QFE5h6JjAKAUQWcb9gHws
 ai_review/libs/config/http.py,sha256=dx5PwgnGbPocUwf9QRhFmXmjfFDoeerOM04yB3B6S8w,398
 ai_review/libs/config/logger.py,sha256=oPmjpjf6EZwW7CgOjT8mOQdGnT98CLwXepiGB_ajZvU,384
 ai_review/libs/config/prompt.py,sha256=jPBYvi75u6BUIHOZnXCg5CiL0XRONDfW5T4WaYlxPEE,6066
-ai_review/libs/config/review.py,sha256=q2EwLvUzsOp_WW1gtd_8fvUhAoStWiNGicxOYP4gm9E,1198
+ai_review/libs/config/review.py,sha256=jCfHfGfHs7Sc6H92hVUBd8bMRbLiIiThVmk6lFNlv40,1224
 ai_review/libs/config/llm/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/libs/config/llm/base.py,sha256=ovvULFhfwH66_705D1O87ZGMeaQOZO7ZQhRUzzfzguU,2089
 ai_review/libs/config/llm/claude.py,sha256=MoalXkBA6pEp01znS8ohTRopfea9RUcqhZX5lOIuek8,293
@@ -137,7 +138,7 @@ ai_review/services/diff/service.py,sha256=yRb4e0fZcgFTGkAZKm5q8Gw4rWxc3nyFtpBw7a
 ai_review/services/diff/tools.py,sha256=YHmH6Ult_rucCd563UhG0geMzqrPhqKFZKyug79xNuA,1963
 ai_review/services/diff/types.py,sha256=uaX0hK_wfRG7Lxs0DnHgjDdkKGSQ1PS2-kZVCVjUeR8,700
 ai_review/services/git/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/services/git/service.py,sha256=_RMwgcllDoSLUzl84JML38fWkr7swnkUr6MJ46hSkWw,1282
+ai_review/services/git/service.py,sha256=7kLCQqJfX--SsFcCWrNxblAEn_xLOYPUSZpr6NTlBUg,2570
 ai_review/services/git/types.py,sha256=QTOCTmR-Rt3sUjzZQHu2PGo_6un5gvNupifAa84wON4,413
 ai_review/services/hook/__init__.py,sha256=HDukG_ZosgGg4dT5GCGIzqZX7llbyYUofKVFeG7YR2A,98
 ai_review/services/hook/constants.py,sha256=GthpBviiqcb3Oj5uhiJhyYGmWuwKgLlToC8pyj8klRM,1585
@@ -161,10 +162,11 @@ ai_review/services/prompt/service.py,sha256=AvhLMfwmIxbnPWdViSSclwUUqnHGx2V-hSMs
 ai_review/services/prompt/tools.py,sha256=7yKQmHajlwFE37tvmMbbBYDmFObFVOb1ubHhNXBdAnE,1229
 ai_review/services/prompt/types.py,sha256=9p1RHuR5-5DNV0AtR_VtiT5OHo6eNqKzGggQoPWiSh8,1572
 ai_review/services/review/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/services/review/service.py,sha256=oqRJfWcww-ZuW5BToXqEymLCLlQbRSZ1ng5LxpUG_Y8,5286
+ai_review/services/review/service.py,sha256=aqt0dlA77_MPT4lcq1qbTjOZOKE-7y83I-7J5jwvro4,5579
 ai_review/services/review/gateway/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/services/review/gateway/comment.py,sha256=gOFoEFXIJ9SX7B05dTO605dUOnrchCtB-a2ExXCfUxE,5019
-ai_review/services/review/gateway/llm.py,sha256=5BQU8Vmo55pzWkVk6s_w5B3qX4tSkjjtj7wWWhLyc9s,1534
+ai_review/services/review/gateway/review_comment_gateway.py,sha256=gOFoEFXIJ9SX7B05dTO605dUOnrchCtB-a2ExXCfUxE,5019
+ai_review/services/review/gateway/review_dry_run_comment_gateway.py,sha256=3CLUBGt8ExX-r_d3PDUxhgaw-yUMqYYOvJbRzs-W63I,2420
+ai_review/services/review/gateway/review_llm_gateway.py,sha256=5BQU8Vmo55pzWkVk6s_w5B3qX4tSkjjtj7wWWhLyc9s,1534
 ai_review/services/review/gateway/types.py,sha256=C9zOUF0ZDnSa568T3AIWIxpNdnpcB8QUxU1rVqx7lZc,1795
 ai_review/services/review/internal/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/services/review/internal/inline/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -197,14 +199,14 @@ ai_review/services/vcs/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3
 ai_review/services/vcs/factory.py,sha256=AfhpZjQ257BkLjb_7zUyw_EUnfEiCUHgTph7GGm-MY4,753
 ai_review/services/vcs/types.py,sha256=LemhQ4LAGlOdwMSF-HlYIo7taSRu4494YQ0Rp2PBgcg,3169
 ai_review/services/vcs/bitbucket/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/services/vcs/bitbucket/adapter.py,sha256=kWR7QXeg9GXnGyMpn_v-2roVRRFEAvGiKF8R-xXgczI,871
-ai_review/services/vcs/bitbucket/client.py,sha256=dhsgJ3uYzWEwEX1XEi_TTQkfCc5xAc9B_c1F9hwPR7M,10401
+ai_review/services/vcs/bitbucket/adapter.py,sha256=b-8KT46C8WT-Sos-gUGFJsxIWY7mXfzTuJjIqYuzrBA,928
+ai_review/services/vcs/bitbucket/client.py,sha256=MhqkFDewutX7DFBCFBDhDopFznwf92dAdZsxy_oS_mc,10726
 ai_review/services/vcs/github/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/services/vcs/github/adapter.py,sha256=pxcbSLEXkOg1c1NtzB0hkZJVvyC4An9pCzNK5sPJKbA,1212
 ai_review/services/vcs/github/client.py,sha256=rSVvpyovT1wDLq0fIQPe5UJqmgIgIFh2P-CPqIQ_sf0,9371
 ai_review/services/vcs/gitlab/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/services/vcs/gitlab/adapter.py,sha256=RkBalrgY8EyxIB8QSH4r4PoTDhg46GROHh2zQyQ4zuY,999
-ai_review/services/vcs/gitlab/client.py,sha256=vPiIpTFEnLXkUV4tSqRJPab13kkLNlUfrIticzmTRGQ,9498
+ai_review/services/vcs/gitlab/adapter.py,sha256=s6EK4Un5rucEJUN61hyzinrTCyvudziNrhK1AGK_XC0,1008
+ai_review/services/vcs/gitlab/client.py,sha256=Dal_GeOAIdJzOQ2dRX7fm1E9p02no9_8sE5TnhIBo44,9779
 ai_review/tests/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/fixtures/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/fixtures/clients/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -212,7 +214,7 @@ ai_review/tests/fixtures/clients/bitbucket.py,sha256=K4Ez_hTOVKz5-KlXbgTOVDec6ie
 ai_review/tests/fixtures/clients/claude.py,sha256=6ldJlSSea0zsZV0hRDMi9mqWm0hWT3mp_ROwG_sVU1c,2203
 ai_review/tests/fixtures/clients/gemini.py,sha256=zhLJhm49keKEBCPOf_pLu8_zCatsKKAWM4-gXOhaXeM,2429
 ai_review/tests/fixtures/clients/github.py,sha256=kC1L-nWZMn9O_uRfuT_B8R4sn8FRvISlBJMkRKaioS0,7814
-ai_review/tests/fixtures/clients/gitlab.py,sha256=9jVvo5b_-SULDAh7Kij7ueZqraqBZgBig0HuDzn8_tg,6921
+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=UgfuRZWzl3X7ZVHMLKP4mZxNXVpcccitkc9tuUyffXE,2267
 ai_review/tests/fixtures/libs/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -230,8 +232,9 @@ ai_review/tests/fixtures/services/vcs.py,sha256=RXfbuW0Ffx7km7q2Zck__1qTZdZ-_vNr
 ai_review/tests/fixtures/services/review/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/fixtures/services/review/base.py,sha256=10LaT6xi5IP0CrR6LfatirVKnZ7D0n8Q1dV99nOBEuE,1501
 ai_review/tests/fixtures/services/review/gateway/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/tests/fixtures/services/review/gateway/comment.py,sha256=nm_dA3teFK5cZJltKS9UcDlUvxmbClii56CCv4vkRF8,4063
-ai_review/tests/fixtures/services/review/gateway/llm.py,sha256=OXYZtJbQ8VqvNlPxG6CLPT0mz-HrhfZA8hoEAgf8loM,515
+ai_review/tests/fixtures/services/review/gateway/review_comment_gateway.py,sha256=aclYEHKS7Kd0TK6_AGvkYsdMFhESHAGhk4Xpblxy1bQ,4078
+ai_review/tests/fixtures/services/review/gateway/review_dry_run_comment_gateway.py,sha256=3E-fDONV4j8GJlSaumqN90Rayoe8yKjnn6APwOrpDDs,4125
+ai_review/tests/fixtures/services/review/gateway/review_llm_gateway.py,sha256=tm3ktE5wFVIFFvHPSUO1wQG7LC3Ekta2PwWky9ugL98,1146
 ai_review/tests/fixtures/services/review/internal/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/fixtures/services/review/internal/inline.py,sha256=D-npuBMSsP_ng5RgplaUZnANl3q5f7GD4OkE8IvQv2M,1038
 ai_review/tests/fixtures/services/review/internal/inline_reply.py,sha256=NRfcVBHavyHvoxOiqMhYhcf6YWDLfT367ZoXeUZIieU,1047
@@ -311,10 +314,11 @@ ai_review/tests/suites/services/prompt/test_schema.py,sha256=rm2__LA2_4qQwSmNAZ_
 ai_review/tests/suites/services/prompt/test_service.py,sha256=jfpmM--eZH43NylwltX-ijmF58ZJ4WA83kmGshUbJfs,8312
 ai_review/tests/suites/services/prompt/test_tools.py,sha256=SmweFvrpxd-3RO5v18vCl7zonEqFE1n4eqHH7-6auYM,4511
 ai_review/tests/suites/services/review/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/tests/suites/services/review/test_service.py,sha256=1KSgYQ1SfO4dserZTMrkLfAzv0466k7LQG8m4TA_rGk,3324
+ai_review/tests/suites/services/review/test_service.py,sha256=9XBywDoVTzLXlfqMmrxwJbPoj8QoGFW5wDwirqu2UHI,4206
 ai_review/tests/suites/services/review/gateway/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/tests/suites/services/review/gateway/test_comment.py,sha256=mFraqWLAF2yTcy8qAwAE5YjuWsab7GixFT5II3Ca7zg,8816
-ai_review/tests/suites/services/review/gateway/test_llm.py,sha256=WO326PiZkF4fHPYl1CRDfpUkLsC2BrLPmJpu0rnWrwY,2903
+ai_review/tests/suites/services/review/gateway/test_review_comment_gateway.py,sha256=eueGnKY3YcLvLQ8hSdhKQuVfOlFwxP82cwgEHKyRPiQ,8831
+ai_review/tests/suites/services/review/gateway/test_review_dry_run_comment_gateway.py,sha256=YGSCZRyyV9BJaKpI66viHYZdM-GtuODJt4ZzEW1-29g,3862
+ai_review/tests/suites/services/review/gateway/test_review_llm_gateway.py,sha256=m4HKvqDQYNesY5F2wuOMeHqXwQe99K4K_depdyTs37Q,2508
 ai_review/tests/suites/services/review/internal/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/services/review/internal/inline/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/services/review/internal/inline/test_schema.py,sha256=2SGF8yNHsNv8lwG-MUlgO3RT4juTt0PQSl10qNhSt7o,2377
@@ -331,11 +335,11 @@ ai_review/tests/suites/services/review/internal/summary_reply/__init__.py,sha256
 ai_review/tests/suites/services/review/internal/summary_reply/test_schema.py,sha256=_2kfcHjB-thIFTIhdDBR7PsEl8beIAu4FtEkQjb0Ljk,859
 ai_review/tests/suites/services/review/internal/summary_reply/test_service.py,sha256=VlVr9wXXwmpp9qxy_4dMZnFHSZ12j2QSP_m58Vs_wXE,709
 ai_review/tests/suites/services/review/runner/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-ai_review/tests/suites/services/review/runner/test_context.py,sha256=57RO3lBR35RfLizJ71wdvJ6MAVz9h8M8i7i6ZKF8MNY,3948
-ai_review/tests/suites/services/review/runner/test_inline.py,sha256=ZehmjvoNe-0Zff55xBceBBg7tNn5AWRW_MdPNt-Uvrg,4651
-ai_review/tests/suites/services/review/runner/test_inline_reply.py,sha256=X_vu1lB83z5k8VG7mfBZaCtfFkq2D4VnGpQEsNJOrmo,4888
-ai_review/tests/suites/services/review/runner/test_summary.py,sha256=ePiORxLDtmsIqRA-eAZ8eDjVhVBjALrJ8jjIb6DlshE,3944
-ai_review/tests/suites/services/review/runner/test_summary_reply.py,sha256=L-G_Jpf3etnNqgPp9fhCy3JH_gmniyl9IUPGyEvkRLM,4546
+ai_review/tests/suites/services/review/runner/test_context.py,sha256=tuOntcwvgnH89mlFXAnwmERPbw3AMk4NgZ6rnRmWA98,3978
+ai_review/tests/suites/services/review/runner/test_inline.py,sha256=rnnfXqxeEYuzi7n3G9iscAfYst607LzovbtEwV6z0cQ,4681
+ai_review/tests/suites/services/review/runner/test_inline_reply.py,sha256=Q3gsOdWvA1fYKRn-8BII19XAylAKK3B1i20xeP_Z1u4,4918
+ai_review/tests/suites/services/review/runner/test_summary.py,sha256=VLIcKffScWSaxUztYHNLAsNUMGiJQWn7j_Le8Zcrizo,3974
+ai_review/tests/suites/services/review/runner/test_summary_reply.py,sha256=UExBEkWh-EG0akVchgLdnnpcd7HFqEnDyMAVbFY_rtU,4576
 ai_review/tests/suites/services/vcs/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 ai_review/tests/suites/services/vcs/test_factory.py,sha256=EergKSHW4b7RZg9vJJ5Cj0XfPsDTLEclV1kq2_9greA,1138
 ai_review/tests/suites/services/vcs/bitbucket/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -345,11 +349,11 @@ ai_review/tests/suites/services/vcs/github/__init__.py,sha256=47DEQpj8HBSa-_TImW
 ai_review/tests/suites/services/vcs/github/test_adapter.py,sha256=bK4k532v_8kDiud5RI9OlGetWMiLP8NaW1vEfHvcHfQ,4893
 ai_review/tests/suites/services/vcs/github/test_client.py,sha256=mNt1bA6aVU3REsJiU_tK1PokQxQTaCKun0tNBHuvIp8,8039
 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=VeORrVExX1rnrQluaBB7LUgmzl_FCwHgW-TAqG4qF1w,3368
-ai_review/tests/suites/services/vcs/gitlab/test_client.py,sha256=8qvf0qGZSEa3fTpMMz6fL-JSFw1rQlPda9bjfKXDC7Y,8028
-xai_review-0.28.0.dist-info/licenses/LICENSE,sha256=p-v8m7Kmz4KKc7PcvsGiGEmCw9AiSXY4_ylOPy_u--Y,11343
-xai_review-0.28.0.dist-info/METADATA,sha256=3iCvsSWPGbG6VXl9xM8z7DAbDzACRS0uAf7C7PI5p6A,11358
-xai_review-0.28.0.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-xai_review-0.28.0.dist-info/entry_points.txt,sha256=JyC5URanMi5io5P_PXQf7H_I1OGIpk5cZQhaPQ0g4Zs,53
-xai_review-0.28.0.dist-info/top_level.txt,sha256=sTsZbfzLoqvRZKdKa-BcxWvjlHdrpbeJ6DrGY0EuR0E,10
-xai_review-0.28.0.dist-info/RECORD,,
+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.30.0.dist-info/licenses/LICENSE,sha256=p-v8m7Kmz4KKc7PcvsGiGEmCw9AiSXY4_ylOPy_u--Y,11343
+xai_review-0.30.0.dist-info/METADATA,sha256=-Tmw1DyUEv-KEMfBUJCj59yzMM8STdCO-A-ZrCRet6Y,11358
+xai_review-0.30.0.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+xai_review-0.30.0.dist-info/entry_points.txt,sha256=JyC5URanMi5io5P_PXQf7H_I1OGIpk5cZQhaPQ0g4Zs,53
+xai_review-0.30.0.dist-info/top_level.txt,sha256=sTsZbfzLoqvRZKdKa-BcxWvjlHdrpbeJ6DrGY0EuR0E,10
+xai_review-0.30.0.dist-info/RECORD,,

ai_review/tests/fixtures/services/review/gateway/llm.py

@@ -1,17 +0,0 @@
-import pytest
-
-from ai_review.services.review.gateway.types import ReviewLLMGatewayProtocol
-
-
-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()