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

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/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/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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xai-review
-Version: 0.29.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.29.0
+      - uses: Nikita-Filonov/ai-review@v0.30.0
         with:
           review-command: ${{ inputs.review-command }}
         env:

{xai_review-0.29.0.dist-info → xai_review-0.30.0.dist-info}/RECORD

@@ -74,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
@@ -162,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
@@ -231,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
@@ -312,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
@@ -332,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
@@ -348,9 +351,9 @@ ai_review/tests/suites/services/vcs/github/test_client.py,sha256=mNt1bA6aVU3REsJ
 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=BYBP2g1AKF_jCSJYJj16pW7M_6PprwD9reYEpdw3StU,4340
 ai_review/tests/suites/services/vcs/gitlab/test_client.py,sha256=dnI-YxYADmVF2GS9rp6-JPkcqsn4sN8Fjbe4MkeYMaE,8476
-xai_review-0.29.0.dist-info/licenses/LICENSE,sha256=p-v8m7Kmz4KKc7PcvsGiGEmCw9AiSXY4_ylOPy_u--Y,11343
-xai_review-0.29.0.dist-info/METADATA,sha256=gcWmlsiSWriTYeM_noFaGBajgOpdJtl4xCWRbJyzMSU,11358
-xai_review-0.29.0.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-xai_review-0.29.0.dist-info/entry_points.txt,sha256=JyC5URanMi5io5P_PXQf7H_I1OGIpk5cZQhaPQ0g4Zs,53
-xai_review-0.29.0.dist-info/top_level.txt,sha256=sTsZbfzLoqvRZKdKa-BcxWvjlHdrpbeJ6DrGY0EuR0E,10
-xai_review-0.29.0.dist-info/RECORD,,
+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()