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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xai-review
-Version: 0.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 → xai_review-0.30.0}/README.md

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

xai_review-0.30.0/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)

xai_review-0.29.0/ai_review/tests/suites/services/review/gateway/test_llm.py → xai_review-0.30.0/ai_review/tests/suites/services/review/gateway/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,

{xai_review-0.29.0 → xai_review-0.30.0}/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

{xai_review-0.29.0 → xai_review-0.30.0}/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

{xai_review-0.29.0 → xai_review-0.30.0}/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
 

{xai_review-0.29.0 → xai_review-0.30.0}/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

{xai_review-0.29.0 → xai_review-0.30.0}/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

{xai_review-0.29.0 → xai_review-0.30.0}/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 → xai_review-0.30.0}/pyproject.toml

@@ -5,7 +5,7 @@ build-backend = "setuptools.build_meta"
 [project]
 name = "xai-review"
 readme = { file = "README.md", content-type = "text/markdown" }
-version = "0.29.0"
+version = "0.30.0"
 license = { text = "Apache-2.0" }
 authors = [
     { name = "Nikita Filonov", email = "nikita.filonov@example.com" }

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

@@ -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 → xai_review-0.30.0}/xai_review.egg-info/SOURCES.txt

@@ -167,8 +167,9 @@ ai_review/services/prompt/types.py
 ai_review/services/review/__init__.py
 ai_review/services/review/service.py
 ai_review/services/review/gateway/__init__.py
-ai_review/services/review/gateway/comment.py
-ai_review/services/review/gateway/llm.py
+ai_review/services/review/gateway/review_comment_gateway.py
+ai_review/services/review/gateway/review_dry_run_comment_gateway.py
+ai_review/services/review/gateway/review_llm_gateway.py
 ai_review/services/review/gateway/types.py
 ai_review/services/review/internal/__init__.py
 ai_review/services/review/internal/inline/__init__.py
@@ -234,8 +235,9 @@ ai_review/tests/fixtures/services/vcs.py
 ai_review/tests/fixtures/services/review/__init__.py
 ai_review/tests/fixtures/services/review/base.py
 ai_review/tests/fixtures/services/review/gateway/__init__.py
-ai_review/tests/fixtures/services/review/gateway/comment.py
-ai_review/tests/fixtures/services/review/gateway/llm.py
+ai_review/tests/fixtures/services/review/gateway/review_comment_gateway.py
+ai_review/tests/fixtures/services/review/gateway/review_dry_run_comment_gateway.py
+ai_review/tests/fixtures/services/review/gateway/review_llm_gateway.py
 ai_review/tests/fixtures/services/review/internal/__init__.py
 ai_review/tests/fixtures/services/review/internal/inline.py
 ai_review/tests/fixtures/services/review/internal/inline_reply.py
@@ -317,8 +319,9 @@ ai_review/tests/suites/services/prompt/test_tools.py
 ai_review/tests/suites/services/review/__init__.py
 ai_review/tests/suites/services/review/test_service.py
 ai_review/tests/suites/services/review/gateway/__init__.py
-ai_review/tests/suites/services/review/gateway/test_comment.py
-ai_review/tests/suites/services/review/gateway/test_llm.py
+ai_review/tests/suites/services/review/gateway/test_review_comment_gateway.py
+ai_review/tests/suites/services/review/gateway/test_review_dry_run_comment_gateway.py
+ai_review/tests/suites/services/review/gateway/test_review_llm_gateway.py
 ai_review/tests/suites/services/review/internal/__init__.py
 ai_review/tests/suites/services/review/internal/inline/__init__.py
 ai_review/tests/suites/services/review/internal/inline/test_schema.py

xai_review-0.29.0/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()