xai-review 0.5.0__tar.gz → 0.6.0__tar.gz

{xai_review-0.5.0 → xai_review-0.6.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xai-review
-Version: 0.5.0
+Version: 0.6.0
 Summary: AI-powered code review tool
 Author-email: Nikita Filonov <nikita.filonov@example.com>
 Maintainer-email: Nikita Filonov <nikita.filonov@example.com>
@@ -211,19 +211,38 @@ ai-review:
     # Custom context variables
     # ===============================
     # You can inject custom variables into prompts via PROMPT__CONTEXT__*.
-    # These will be available as placeholders {var} in all templates.
+    # These will be available in all templates through placeholders.
+    #
+    # Placeholder syntax is defined separately in PROMPT__CONTEXT_PLACEHOLDER.
+    # Default: <<{value}>>
     #
     # Example usage in prompt templates:
-    #   Project: {company_name}
-    #   Env: {environment}
-    #   Pipeline: {ci_pipeline_url}
+    #   Project: <<company_name>>
+    #   Env: <<environment>>
+    #   Pipeline: <<ci_pipeline_url>>
     #
     # Values override built-in variables if names collide.
-    # To avoid clashes, prefer namespaced keys (ci_pipeline_url, org_notify_handle, env_name).
+    # To avoid clashes, prefer namespaced keys
+    # (ci_pipeline_url, org_notify_handle, env_name).
     #
     # PROMPT__CONTEXT__ENVIRONMENT: "staging"
     # PROMPT__CONTEXT__COMPANY_NAME: "ACME Corp"
     # PROMPT__CONTEXT__CI_PIPELINE_URL: "https://gitlab.com/pipelines/123"
+    #
+    # ===============================
+    # Context placeholder
+    # ===============================
+    # Defines how placeholders are written in prompt templates.
+    # Must contain "{value}" which will be replaced by the variable name.
+    #
+    # Default: <<{value}>>
+    #
+    # Example:
+    #   PROMPT__CONTEXT_PLACEHOLDER: "<<{value}>>"
+    #   Template: "Env: <<environment>>"
+    #   Result:   "Env: staging"
+    #
+    # PROMPT__CONTEXT_PLACEHOLDER: "<<{value}>>"
 
     # ===============================
     # Review options

{xai_review-0.5.0 → xai_review-0.6.0}/README.md

@@ -187,19 +187,38 @@ ai-review:
     # Custom context variables
     # ===============================
     # You can inject custom variables into prompts via PROMPT__CONTEXT__*.
-    # These will be available as placeholders {var} in all templates.
+    # These will be available in all templates through placeholders.
+    #
+    # Placeholder syntax is defined separately in PROMPT__CONTEXT_PLACEHOLDER.
+    # Default: <<{value}>>
     #
     # Example usage in prompt templates:
-    #   Project: {company_name}
-    #   Env: {environment}
-    #   Pipeline: {ci_pipeline_url}
+    #   Project: <<company_name>>
+    #   Env: <<environment>>
+    #   Pipeline: <<ci_pipeline_url>>
     #
     # Values override built-in variables if names collide.
-    # To avoid clashes, prefer namespaced keys (ci_pipeline_url, org_notify_handle, env_name).
+    # To avoid clashes, prefer namespaced keys
+    # (ci_pipeline_url, org_notify_handle, env_name).
     #
     # PROMPT__CONTEXT__ENVIRONMENT: "staging"
     # PROMPT__CONTEXT__COMPANY_NAME: "ACME Corp"
     # PROMPT__CONTEXT__CI_PIPELINE_URL: "https://gitlab.com/pipelines/123"
+    #
+    # ===============================
+    # Context placeholder
+    # ===============================
+    # Defines how placeholders are written in prompt templates.
+    # Must contain "{value}" which will be replaced by the variable name.
+    #
+    # Default: <<{value}>>
+    #
+    # Example:
+    #   PROMPT__CONTEXT_PLACEHOLDER: "<<{value}>>"
+    #   Template: "Env: <<environment>>"
+    #   Result:   "Env: staging"
+    #
+    # PROMPT__CONTEXT_PLACEHOLDER: "<<{value}>>"
 
     # ===============================
     # Review options

{xai_review-0.5.0 → xai_review-0.6.0}/ai_review/libs/config/prompt.py

@@ -8,6 +8,7 @@ from ai_review.libs.resources import load_resource
 
 class PromptConfig(BaseModel):
     context: dict[str, str] = Field(default_factory=dict)
+    context_placeholder: str = "<<{value}>>"
     inline_prompt_files: list[FilePath] | None = None
     context_prompt_files: list[FilePath] | None = None
     summary_prompt_files: list[FilePath] | None = None

xai_review-0.6.0/ai_review/libs/template/render.py

@@ -0,0 +1,13 @@
+import re
+from typing import Mapping
+
+
+def render_template(text: str, values: Mapping[str, str], placeholder: str = "<<{value}>>") -> str:
+    regex_pattern = re.escape(placeholder).replace(r"\{value\}", r"([\w\.-]+)")
+    regex = re.compile(regex_pattern)
+
+    def replacer(match: re.Match) -> str:
+        key = match.group(1)
+        return values.get(key, match.group(0))
+
+    return regex.sub(replacer, text)

xai_review-0.6.0/ai_review/services/prompt/schema.py

@@ -0,0 +1,52 @@
+from pydantic import BaseModel, Field
+
+from ai_review.config import settings
+from ai_review.libs.template.render import render_template
+
+
+class PromptContextSchema(BaseModel):
+    merge_request_title: str = ""
+    merge_request_description: str = ""
+
+    merge_request_author_name: str = ""
+    merge_request_author_username: str = ""
+
+    merge_request_reviewer: str = ""
+    merge_request_reviewers: list[str] = Field(default_factory=list)
+    merge_request_reviewers_usernames: list[str] = Field(default_factory=list)
+
+    merge_request_assignees: list[str] = Field(default_factory=list)
+    merge_request_assignees_usernames: list[str] = Field(default_factory=list)
+
+    source_branch: str = ""
+    target_branch: str = ""
+
+    labels: list[str] = Field(default_factory=list)
+    changed_files: list[str] = Field(default_factory=list)
+
+    @property
+    def render_values(self) -> dict[str, str]:
+        return {
+            "merge_request_title": self.merge_request_title,
+            "merge_request_description": self.merge_request_description,
+
+            "merge_request_author_name": self.merge_request_author_name,
+            "merge_request_author_username": self.merge_request_author_username,
+
+            "merge_request_reviewer": self.merge_request_reviewer,
+            "merge_request_reviewers": ", ".join(self.merge_request_reviewers),
+            "merge_request_reviewers_usernames": ", ".join(self.merge_request_reviewers_usernames),
+
+            "merge_request_assignees": ", ".join(self.merge_request_assignees),
+            "merge_request_assignees_usernames": ", ".join(self.merge_request_assignees_usernames),
+
+            "source_branch": self.source_branch,
+            "target_branch": self.target_branch,
+
+            "labels": ", ".join(self.labels),
+            "changed_files": ", ".join(self.changed_files),
+        }
+
+    def apply_format(self, prompt: str) -> str:
+        values = {**self.render_values, **settings.prompt.context}
+        return render_template(prompt, values, settings.prompt.context_placeholder)

xai_review-0.6.0/ai_review/tests/suites/clients/gitlab/test_client.py

@@ -0,0 +1,35 @@
+import pytest
+from httpx import AsyncClient
+from pydantic import HttpUrl, SecretStr
+
+from ai_review.clients.gitlab.client import get_gitlab_http_client, GitLabHTTPClient
+from ai_review.clients.gitlab.mr.client import GitLabMergeRequestsHTTPClient
+from ai_review.config import settings
+from ai_review.libs.config.gitlab import GitLabPipelineConfig, GitLabHTTPClientConfig
+from ai_review.libs.config.vcs import GitLabVCSConfig
+from ai_review.libs.constants.vcs_provider import VCSProvider
+
+
+@pytest.fixture(autouse=True)
+def gitlab_http_client_config(monkeypatch: pytest.MonkeyPatch):
+    fake_config = GitLabVCSConfig(
+        provider=VCSProvider.GITLAB,
+        pipeline=GitLabPipelineConfig(
+            project_id="project-id",
+            merge_request_id="merge-request-id"
+        ),
+        http_client=GitLabHTTPClientConfig(
+            timeout=10,
+            api_url=HttpUrl("https://gitlab.com"),
+            api_token=SecretStr("fake-token"),
+        )
+    )
+    monkeypatch.setattr(settings, "vcs", fake_config)
+
+
+def test_get_gitlab_http_client_builds_ok():
+    gitlab_http_client = get_gitlab_http_client()
+
+    assert isinstance(gitlab_http_client, GitLabHTTPClient)
+    assert isinstance(gitlab_http_client.mr, GitLabMergeRequestsHTTPClient)
+    assert isinstance(gitlab_http_client.mr.client, AsyncClient)

xai_review-0.6.0/ai_review/tests/suites/libs/config/test_prompt.py

@@ -0,0 +1,57 @@
+from pathlib import Path
+
+import pytest
+
+from ai_review.libs.config.prompt import PromptConfig
+
+
+def test_inline_prompt_files_or_default_uses_defaults(tmp_path: Path, monkeypatch: pytest.MonkeyPatch):
+    dummy_file = tmp_path / "dummy.md"
+    dummy_file.write_text("DUMMY")
+    monkeypatch.setattr("ai_review.libs.config.prompt.load_resource", lambda **_: dummy_file)
+
+    config = PromptConfig()
+    result = config.inline_prompt_files_or_default
+
+    assert result == [dummy_file]
+    assert config.load_inline() == ["DUMMY"]
+
+
+def test_system_inline_prompts_none_returns_global(tmp_path: Path, monkeypatch: pytest.MonkeyPatch):
+    dummy_file = tmp_path / "global.md"
+    dummy_file.write_text("GLOBAL")
+    monkeypatch.setattr("ai_review.libs.config.prompt.load_resource", lambda **_: dummy_file)
+
+    config = PromptConfig(system_inline_prompt_files=None)
+    result = config.system_inline_prompt_files_or_default
+
+    assert result == [dummy_file]
+    assert config.load_system_inline() == ["GLOBAL"]
+
+
+def test_system_inline_prompts_include_true(tmp_path: Path, monkeypatch: pytest.MonkeyPatch):
+    global_file = tmp_path / "global.md"
+    global_file.write_text("GLOBAL")
+    custom_file = tmp_path / "custom.md"
+    custom_file.write_text("CUSTOM")
+    monkeypatch.setattr("ai_review.libs.config.prompt.load_resource", lambda **_: global_file)
+
+    config = PromptConfig(system_inline_prompt_files=[custom_file], include_inline_system_prompts=True)
+    result = config.system_inline_prompt_files_or_default
+
+    assert global_file in result and custom_file in result
+    assert config.load_system_inline() == ["GLOBAL", "CUSTOM"]
+
+
+def test_system_inline_prompts_include_false(tmp_path: Path, monkeypatch: pytest.MonkeyPatch):
+    global_file = tmp_path / "global.md"
+    global_file.write_text("GLOBAL")
+    custom_file = tmp_path / "custom.md"
+    custom_file.write_text("CUSTOM")
+    monkeypatch.setattr("ai_review.libs.config.prompt.load_resource", lambda **_: global_file)
+
+    config = PromptConfig(system_inline_prompt_files=[custom_file], include_inline_system_prompts=False)
+    result = config.system_inline_prompt_files_or_default
+
+    assert result == [custom_file]
+    assert config.load_system_inline() == ["CUSTOM"]

xai_review-0.6.0/ai_review/tests/suites/libs/template/test_render.py

@@ -0,0 +1,64 @@
+from ai_review.libs.template.render import render_template
+
+
+def test_replaces_single_variable():
+    text = "Hello, <<name>>!"
+    values = {"name": "Alice"}
+    result = render_template(text, values)
+    assert result == "Hello, Alice!"
+
+
+def test_replaces_multiple_variables():
+    text = "User: <<user>>, Branch: <<branch>>"
+    values = {"user": "nikita", "branch": "main"}
+    result = render_template(text, values)
+    assert result == "User: nikita, Branch: main"
+
+
+def test_missing_variable_keeps_placeholder():
+    text = "Hello, <<name>>, you are <<role>>"
+    values = {"name": "Bob"}
+    result = render_template(text, values)
+    assert result == "Hello, Bob, you are <<role>>"
+
+
+def test_custom_placeholder_format():
+    text = "Hello, [[name]]!"
+    values = {"name": "Alice"}
+    result = render_template(text, values, placeholder="[[{value}]]")
+    assert result == "Hello, Alice!"
+
+
+def test_placeholder_with_digits_and_underscores():
+    text = "Key: <<var_123>>"
+    values = {"var_123": "ok"}
+    result = render_template(text, values)
+    assert result == "Key: ok"
+
+
+def test_no_placeholders_in_text():
+    text = "Nothing to replace"
+    values = {"name": "Alice"}
+    result = render_template(text, values)
+    assert result == "Nothing to replace"
+
+
+def test_partial_overlap_does_not_replace():
+    text = "This is <<var>> and <<var_extra>>"
+    values = {"var": "A", "var_extra": "B"}
+    result = render_template(text, values)
+    assert result == "This is A and B"
+
+
+def test_placeholder_with_dot_and_dash():
+    text = "Branch: <<feature-1.2>>"
+    values = {"feature-1.2": "ok"}
+    result = render_template(text, values)
+    assert result == "Branch: ok"
+
+
+def test_multiple_same_placeholders():
+    text = "<<name>>, <<name>>, <<name>>!"
+    values = {"name": "Alice"}
+    result = render_template(text, values)
+    assert result == "Alice, Alice, Alice!"

xai_review-0.6.0/ai_review/tests/suites/services/prompt/test_schema.py

@@ -0,0 +1,71 @@
+import pytest
+
+from ai_review.config import settings
+from ai_review.services.prompt.schema import PromptContextSchema
+
+
+def test_apply_format_inserts_variables() -> None:
+    context = PromptContextSchema(
+        merge_request_title="My MR",
+        merge_request_author_username="nikita"
+    )
+    template = "Title: <<merge_request_title>>, Author: @<<merge_request_author_username>>"
+    result = context.apply_format(template)
+    assert result == "Title: My MR, Author: @nikita"
+
+
+def test_apply_format_with_lists() -> None:
+    context = PromptContextSchema(
+        merge_request_reviewers=["Alice", "Bob"],
+        merge_request_reviewers_usernames=["alice", "bob"],
+        labels=["bug", "feature"],
+        changed_files=["a.py", "b.py"],
+    )
+    template = (
+        "Reviewers: <<merge_request_reviewers>>\n"
+        "Usernames: <<merge_request_reviewers_usernames>>\n"
+        "Labels: <<labels>>\n"
+        "Files: <<changed_files>>"
+    )
+    result = context.apply_format(template)
+    assert "Alice, Bob" in result
+    assert "alice, bob" in result
+    assert "bug, feature" in result
+    assert "a.py, b.py" in result
+
+
+def test_apply_format_handles_missing_fields() -> None:
+    context = PromptContextSchema()
+    template = "Title: <<merge_request_title>>, Reviewer: <<merge_request_reviewer>>"
+    result = context.apply_format(template)
+    assert result == "Title: , Reviewer: "
+
+
+def test_apply_format_unknown_placeholder_kept() -> None:
+    context = PromptContextSchema()
+    template = "Unknown: <<does_not_exist>>"
+    result = context.apply_format(template)
+    assert result == "Unknown: <<does_not_exist>>"
+
+
+def test_apply_format_multiple_occurrences() -> None:
+    context = PromptContextSchema(merge_request_title="My MR")
+    template = "<<merge_request_title>> again <<merge_request_title>>"
+    result = context.apply_format(template)
+    assert result == "My MR again My MR"
+
+
+def test_apply_format_override_from_settings(monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.setitem(settings.prompt.context, "merge_request_title", "Overridden")
+    context = PromptContextSchema(merge_request_title="Local Value")
+    template = "Title: <<merge_request_title>>"
+    result = context.apply_format(template)
+    assert result == "Title: Overridden"
+
+
+def test_apply_format_prefers_override_even_if_empty(monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.setitem(settings.prompt.context, "merge_request_title", "")
+    context = PromptContextSchema(merge_request_title="Local Value")
+    template = "Title: <<merge_request_title>>"
+    result = context.apply_format(template)
+    assert result == "Title: "

{xai_review-0.5.0 → xai_review-0.6.0}/pyproject.toml

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

{xai_review-0.5.0 → xai_review-0.6.0}/xai_review.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: xai-review
-Version: 0.5.0
+Version: 0.6.0
 Summary: AI-powered code review tool
 Author-email: Nikita Filonov <nikita.filonov@example.com>
 Maintainer-email: Nikita Filonov <nikita.filonov@example.com>
@@ -211,19 +211,38 @@ ai-review:
     # Custom context variables
     # ===============================
     # You can inject custom variables into prompts via PROMPT__CONTEXT__*.
-    # These will be available as placeholders {var} in all templates.
+    # These will be available in all templates through placeholders.
+    #
+    # Placeholder syntax is defined separately in PROMPT__CONTEXT_PLACEHOLDER.
+    # Default: <<{value}>>
     #
     # Example usage in prompt templates:
-    #   Project: {company_name}
-    #   Env: {environment}
-    #   Pipeline: {ci_pipeline_url}
+    #   Project: <<company_name>>
+    #   Env: <<environment>>
+    #   Pipeline: <<ci_pipeline_url>>
     #
     # Values override built-in variables if names collide.
-    # To avoid clashes, prefer namespaced keys (ci_pipeline_url, org_notify_handle, env_name).
+    # To avoid clashes, prefer namespaced keys
+    # (ci_pipeline_url, org_notify_handle, env_name).
     #
     # PROMPT__CONTEXT__ENVIRONMENT: "staging"
     # PROMPT__CONTEXT__COMPANY_NAME: "ACME Corp"
     # PROMPT__CONTEXT__CI_PIPELINE_URL: "https://gitlab.com/pipelines/123"
+    #
+    # ===============================
+    # Context placeholder
+    # ===============================
+    # Defines how placeholders are written in prompt templates.
+    # Must contain "{value}" which will be replaced by the variable name.
+    #
+    # Default: <<{value}>>
+    #
+    # Example:
+    #   PROMPT__CONTEXT_PLACEHOLDER: "<<{value}>>"
+    #   Template: "Env: <<environment>>"
+    #   Result:   "Env: staging"
+    #
+    # PROMPT__CONTEXT_PLACEHOLDER: "<<{value}>>"
 
     # ===============================
     # Review options

{xai_review-0.5.0 → xai_review-0.6.0}/xai_review.egg-info/SOURCES.txt

@@ -61,6 +61,8 @@ ai_review/libs/http/event_hooks/base.py
 ai_review/libs/http/event_hooks/logger.py
 ai_review/libs/http/transports/__init__.py
 ai_review/libs/http/transports/retry.py
+ai_review/libs/template/__init__.py
+ai_review/libs/template/render.py
 ai_review/prompts/__init__.py
 ai_review/prompts/default_context.md
 ai_review/prompts/default_inline.md
@@ -125,14 +127,20 @@ ai_review/tests/suites/clients/claude/test_schema.py
 ai_review/tests/suites/clients/gemini/__init__.py
 ai_review/tests/suites/clients/gemini/test_client.py
 ai_review/tests/suites/clients/gemini/test_schema.py
+ai_review/tests/suites/clients/gitlab/__init__.py
+ai_review/tests/suites/clients/gitlab/test_client.py
 ai_review/tests/suites/clients/openai/__init__.py
 ai_review/tests/suites/clients/openai/test_client.py
 ai_review/tests/suites/clients/openai/test_schema.py
 ai_review/tests/suites/libs/__init__.py
+ai_review/tests/suites/libs/config/__init__.py
+ai_review/tests/suites/libs/config/test_prompt.py
 ai_review/tests/suites/libs/diff/__init__.py
 ai_review/tests/suites/libs/diff/test_models.py
 ai_review/tests/suites/libs/diff/test_parser.py
 ai_review/tests/suites/libs/diff/test_tools.py
+ai_review/tests/suites/libs/template/__init__.py
+ai_review/tests/suites/libs/template/test_render.py
 ai_review/tests/suites/services/__init__.py
 ai_review/tests/suites/services/diff/__init__.py
 ai_review/tests/suites/services/diff/test_renderers.py

xai_review-0.5.0/ai_review/services/prompt/schema.py

@@ -1,71 +0,0 @@
-from pydantic import BaseModel, Field
-
-from ai_review.config import settings
-
-
-class PromptContextSchema(BaseModel):
-    merge_request_title: str | None = None
-    merge_request_description: str | None = None
-
-    merge_request_author_name: str | None = None
-    merge_request_author_username: str | None = None
-
-    merge_request_reviewer: str | None = None
-    merge_request_reviewers: list[str] = Field(default_factory=list)
-    merge_request_reviewers_usernames: list[str] = Field(default_factory=list)
-
-    merge_request_assignees: list[str] = Field(default_factory=list)
-    merge_request_assignees_usernames: list[str] = Field(default_factory=list)
-
-    source_branch: str | None = None
-    target_branch: str | None = None
-
-    labels: list[str] = Field(default_factory=list)
-    changed_files: list[str] = Field(default_factory=list)
-
-    @property
-    def reviewers_format(self) -> str:
-        return ", ".join(self.merge_request_reviewers)
-
-    @property
-    def reviewers_usernames_format(self) -> str:
-        return ", ".join(self.merge_request_reviewers_usernames)
-
-    @property
-    def assignees_format(self) -> str:
-        return ", ".join(self.merge_request_assignees)
-
-    @property
-    def assignees_usernames_format(self) -> str:
-        return ", ".join(self.merge_request_assignees_usernames)
-
-    @property
-    def labels_format(self) -> str:
-        return ", ".join(self.labels)
-
-    @property
-    def changed_files_format(self) -> str:
-        return ", ".join(self.changed_files)
-
-    def apply_format(self, prompt: str) -> str:
-        return prompt.format(
-            merge_request_title=self.merge_request_title or "",
-            merge_request_description=self.merge_request_description or "",
-
-            merge_request_author_name=self.merge_request_author_name or "",
-            merge_request_author_username=self.merge_request_author_username or "",
-
-            merge_request_reviewer=self.merge_request_reviewer or "",
-            merge_request_reviewers=self.reviewers_format,
-            merge_request_reviewers_usernames=self.reviewers_usernames_format,
-
-            merge_request_assignees=self.assignees_format,
-            merge_request_assignees_usernames=self.assignees_usernames_format,
-
-            source_branch=self.source_branch or "",
-            target_branch=self.target_branch or "",
-
-            labels=self.labels_format,
-            changed_files=self.changed_files_format,
-            **settings.prompt.context
-        )

xai_review-0.5.0/ai_review/tests/suites/services/prompt/test_schema.py

@@ -1,38 +0,0 @@
-from ai_review.services.prompt.schema import PromptContextSchema
-
-
-def test_apply_format_inserts_variables() -> None:
-    context = PromptContextSchema(
-        merge_request_title="My MR",
-        merge_request_author_username="nikita"
-    )
-    template = "Title: {merge_request_title}, Author: @{merge_request_author_username}"
-    result = context.apply_format(template)
-    assert result == "Title: My MR, Author: @nikita"
-
-
-def test_apply_format_with_lists() -> None:
-    context = PromptContextSchema(
-        merge_request_reviewers=["Alice", "Bob"],
-        merge_request_reviewers_usernames=["alice", "bob"],
-        labels=["bug", "feature"],
-        changed_files=["a.py", "b.py"],
-    )
-    template = (
-        "Reviewers: {merge_request_reviewers}\n"
-        "Usernames: {merge_request_reviewers_usernames}\n"
-        "Labels: {labels}\n"
-        "Files: {changed_files}"
-    )
-    result = context.apply_format(template)
-    assert "Alice, Bob" in result
-    assert "alice, bob" in result
-    assert "bug, feature" in result
-    assert "a.py, b.py" in result
-
-
-def test_apply_format_handles_missing_fields() -> None:
-    context = PromptContextSchema()
-    template = "Title: {merge_request_title}, Reviewer: {merge_request_reviewer}"
-    result = context.apply_format(template)
-    assert result == "Title: , Reviewer: "