code-review-ai-cli 2.0.0__tar.gz → 2.0.1__tar.gz

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: code-review-ai-cli
-Version: 2.0.0
+Version: 2.0.1
 Summary: Automated AI-powered code review CLI for Azure DevOps / TFS Pull Requests
 License: MIT
 Keywords: code-review,ai,azure-devops,tfs,pull-request,llm

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/code_review_ai_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: code-review-ai-cli
-Version: 2.0.0
+Version: 2.0.1
 Summary: Automated AI-powered code review CLI for Azure DevOps / TFS Pull Requests
 License: MIT
 Keywords: code-review,ai,azure-devops,tfs,pull-request,llm

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "code-review-ai-cli"
-version = "2.0.0"
+version = "2.0.1"
 description = "Automated AI-powered code review CLI for Azure DevOps / TFS Pull Requests"
 readme = "README.md"
 requires-python = ">=3.10"

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/src/ai_review.py

@@ -13,7 +13,6 @@ Usage:
     python ai_review.py list-prs         # List active Pull Requests
 
 Options:
-    --quick                              # Quick and concise review
     --detailed                           # Detailed review (default)
     --security                           # Security-focused review
     --review-scope <diff_only|full_code> # Review scope (default: diff_only)
@@ -216,10 +215,6 @@ def _add_global_options(parser: argparse.ArgumentParser) -> None:
     """Adds global options to a subparser."""
 
     group_review = parser.add_argument_group("Review Options")
-    group_review.add_argument(
-        "--quick", "-q", action="store_const", dest="verbosity",
-        const="quick", help="Quick and concise review"
-    )
     group_review.add_argument(
         "--detailed", "-d", action="store_const", dest="verbosity",
         const="detailed", help="Detailed review (default)"
@@ -834,17 +829,16 @@ def _ask_verbosity() -> str:
     """Asks for the verbosity mode."""
     c = Colors
     print(f"\n{c.BOLD}Review mode:{c.RESET}")
-    print(f"  {c.CYAN}1{c.RESET}) Quick")
-    print(f"  {c.CYAN}2{c.RESET}) Detailed")
-    print(f"  {c.CYAN}3{c.RESET}) Security")
+    print(f"  {c.CYAN}1{c.RESET}) Detailed")
+    print(f"  {c.CYAN}2{c.RESET}) Security")
 
     try:
-        mode_choice = input(f"{c.BOLD}Choose [1-3, default=2]: {c.RESET}").strip()
+        mode_choice = input(f"{c.BOLD}Choose [1-2, default=1]: {c.RESET}").strip()
     except (KeyboardInterrupt, EOFError):
         print("\n")
         return "detailed"
 
-    verbosity_map = {"1": "quick", "2": "detailed", "3": "security"}
+    verbosity_map = {"1": "detailed", "2": "security"}
     return verbosity_map.get(mode_choice, "detailed")
 
 

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/src/config.py

@@ -85,7 +85,7 @@ class ReviewConfig:
     tfs_repository: str = ""                # Default repository (empty = all)
 
     # --- Review -------------------------------------------------------
-    verbosity: str = "detailed"             # "quick" | "detailed" | "security"
+    verbosity: str = "detailed"             # "detailed" | "security"
     review_scope: str = "diff_only"         # "diff_only" | "full_code"
     max_diff_files: int = 50                 # Max diff files sent to LLM
     max_diff_lines: int = 2000              # Max diff lines
@@ -302,10 +302,10 @@ class ReviewConfig:
                     "  Configure bedrock.access_key_id when secret_access_key is set."
                 )
 
-        if self.verbosity not in ("quick", "detailed", "security"):
+        if self.verbosity not in ("detailed", "security"):
             issues.append(
                 f"Invalid verbosity: '{self.verbosity}'. "
-                "Use 'quick', 'detailed' or 'security'."
+                "Use 'detailed' or 'security'."
             )
 
         if self.review_scope not in ("diff_only", "full_code"):

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/src/git_utils.py

@@ -284,32 +284,37 @@ class GitUtils:
     # ------------------------------------------------------------------
     def filter_diff_additions_only(self, diff: str) -> str:
         """
-        Removes context lines and deleted lines (-) from the diff.
-        Keeps only added lines (+) and structural headers needed for the LLM.
+        Removes deleted lines (-) from the diff, keeping context lines and added lines.
+        Structural headers are always preserved.
 
         Lines kept:
             - ``diff --git ...``
             - ``--- a/...``
             - ``+++ b/...``
             - ``@@ ... @@``
-            - ``+ <content>``
+            - ``+ <content>`` (added lines)
+            - `` <content>`` (context lines — unchanged surrounding code)
+
+        Lines removed:
+            - ``- <content>`` (deleted lines)
+            - ``\\ No newline ...`` markers
 
         FULL_FILE_CONTEXT blocks — delimited by
         ``### FULL_FILE_CONTEXT_START: <path> ###`` and
         ``### FULL_FILE_CONTEXT_END ###`` — are preserved in their entirety.
         These blocks are appended by
         :py:meth:`TFSClient._build_unified_diff_part` and carry the complete
-        new-version file content as read-only background for the LLM.  Every
-        line between the sentinel markers is kept, regardless of whether it
-        starts with ``+``, ``-``, or a space.
+        new-version file content (with line numbers) as read-only background
+        for the LLM.  Every line between the sentinel markers is kept,
+        regardless of whether it starts with ``+``, ``-``, or a space.
 
         Args:
             diff: Raw unified diff string, possibly containing
                 FULL_FILE_CONTEXT blocks.
 
         Returns:
-            Filtered diff string with context/deleted lines stripped but
-            FULL_FILE_CONTEXT blocks intact.
+            Filtered diff string with deleted lines stripped but context lines
+            and FULL_FILE_CONTEXT blocks intact.
         """
         result = []
         in_context_block = False
@@ -331,9 +336,10 @@ class GitUtils:
                 or line.startswith("+++ ")
                 or line.startswith("@@")
                 or (line.startswith("+") and not line.startswith("+++"))
+                or line.startswith(" ")  # context lines — unchanged surrounding code
             ):
                 result.append(line)
-            # Context lines, deleted lines and '\ No newline' markers are discarded.
+            # Deleted lines (-) and '\\ No newline' markers are discarded.
         return "\n".join(result)
 
     def _split_diff_sections(self, diff: str) -> tuple[list[list[str]], bool]:

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/src/llm_client.py

@@ -27,7 +27,7 @@ class LLMError(Exception):
 # ---------------------------------------------------------------------------
 # System Prompts
 # ---------------------------------------------------------------------------
-# Maps each verbosity level ("quick", "detailed", "security") to its
+# Maps each verbosity level ("detailed", "security") to its
 # English-only system prompt.
 #
 # Type:
@@ -38,14 +38,6 @@ class LLMError(Exception):
 #     get_system_prompt: Resolves a prompt for a given verbosity, with a
 #     fallback to "detailed" when the key is not recognised.
 SYSTEM_PROMPTS = {
-    "quick": (
-        "You are an experienced Senior Code Reviewer. Analyze the provided code diff "
-        "and give a CONCISE review. Focus on critical issues:\n"
-        "- Bugs and logic errors\n"
-        "- Security issues\n"
-        "- Major performance problems\n\n"
-        "Format: Bullet points with file and line when possible. "
-    ),
     "detailed": (
         "You are an experienced Senior Code Reviewer. Analyze the provided code and return only inline comments.\n\n"
         "Output format — for each issue found, output exactly:\n"
@@ -134,7 +126,7 @@ def get_system_prompt(verbosity: str) -> str:
     When no issues are found the expected output is ``No issues found.``
 
     Args:
-        verbosity: Review depth key — one of ``"quick"``, ``"detailed"``,
+        verbosity: Review depth key — one of ``"detailed"``
             or ``"security"``. Any unrecognised value falls back to
             ``"detailed"``.
 
@@ -149,14 +141,15 @@ def get_scope_guidance(review_scope: str, structured: bool = False) -> str:
 
     For ``diff_only`` scope, the instructions inform the LLM that:
 
-    * Only added lines (``+``) appear in the diff section.
+    * Added lines are prefixed with ``+`` and context lines (unchanged code)
+      have no prefix. Deleted lines are not included.
     * A ``### FULL_FILE_CONTEXT_START: <path> ###`` /
       ``### FULL_FILE_CONTEXT_END ###`` block is embedded in the payload for
       each changed file, containing the complete new-version (after changes)
-      file content as **read-only** background.
+      file content with **1-based line numbers** as **read-only** background.
     * The review must focus **exclusively** on the changed lines (``+``); the
-      full-file section exists only to prevent the model from hallucinating
-      about the surrounding code.
+      full-file section exists only to provide a global view and authoritative
+      line numbers for the LLM.
 
     For ``full_code`` scope, the instructions inform the LLM that the diff
     represents the entire new file content (every line prefixed with ``+``)
@@ -182,21 +175,26 @@ def get_scope_guidance(review_scope: str, structured: bool = False) -> str:
 
     if structured:
         return (
-            "Review scope: diff_only. The diff contains only added lines (+) — context and deletions were removed. "
-            "The complete new-version file content, after the changes (between ### FULL_FILE_CONTEXT_START and ### FULL_FILE_CONTEXT_END markers) "
-            "is provided for each file as read-only context. "
-            "Use it to understand the surrounding code, but focus your review EXCLUSIVELY on the changed lines (marked + in the diff). "
-            "Do NOT report issues in unchanged lines unless they directly affect the correctness of the changes. "
+            "Review scope: diff_only. The diff contains added lines (marked +) and surrounding "
+            "context lines (unchanged code, no prefix). Deleted lines were removed. "
+            "The complete new-version file content with 1-based line numbers is provided for each file "
+            "between ### FULL_FILE_CONTEXT_START and ### FULL_FILE_CONTEXT_END markers as read-only background. "
+            "Use the line numbers in FULL_FILE_CONTEXT as the authoritative reference when reporting issues. "
+            "Focus your review EXCLUSIVELY on the changed lines (marked + in the diff). "
+            "Do NOT report issues in context or unchanged lines unless they directly affect the correctness of the changes. "
             "For every problem, you MUST provide a valid file and line (>0) to allow inline comments. "
             "Do not emit general problem comments without file/line."
         )
 
     return (
-        "Review scope: diff_only. The diff contains only added lines (+). "
-        "The new-version (post-change) full file content (between ### FULL_FILE_CONTEXT_START and ### FULL_FILE_CONTEXT_END markers) "
-        "is provided for each file as read-only context. "
-        "Use it to understand the surrounding code, but focus your review EXCLUSIVELY on the changed lines. "
-        "Do NOT report issues in unchanged lines unless they directly affect the correctness of the changes."
+        "Review scope: diff_only. The diff contains added lines (marked +) and surrounding "
+        "context lines (unchanged code, no prefix). Deleted lines were removed. "
+        "The complete new-version file content with 1-based line numbers "
+        "(between ### FULL_FILE_CONTEXT_START and ### FULL_FILE_CONTEXT_END markers) "
+        "is provided for each file as read-only background. "
+        "Use the line numbers in FULL_FILE_CONTEXT as the authoritative reference when reporting issues. "
+        "Focus your review EXCLUSIVELY on the changed lines. "
+        "Do NOT report issues in context or unchanged lines unless they directly affect the correctness of the changes."
     )
 
 

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/src/prompts/config.yaml.template

@@ -72,7 +72,7 @@ tfs:
 # Review Preferences
 # ---------------------------------------------------------------------------
 review:
-  # Default verbosity: "quick", "detailed" or "security"
+  # Default verbosity: "detailed" or "security"
   verbosity: detailed
 
   # Review scope:

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/src/tfs_client.py

@@ -526,10 +526,13 @@ class TFSClient:
         # complete resulting file so it can correctly understand line numbers
         # and surrounding code for the added lines, avoiding confusion from
         # showing stale pre-change content.
+        # Lines are numbered (1-based) so the LLM can use them as the
+        # authoritative reference when reporting issue positions.
         # delete is excluded automatically because new_lines is empty for deleted files.
         if new_lines:
             result.append(f"### FULL_FILE_CONTEXT_START: {file_path} ###")
-            result.extend(new_lines)
+            for i, line in enumerate(new_lines, start=1):
+                result.append(f"{i:4d}: {line}")
             result.append("### FULL_FILE_CONTEXT_END ###")
 
         return result

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/tests/test_ai_review.py

@@ -88,7 +88,7 @@ def test_build_parser_parses_pr_review_options() -> None:
         "pr-review",
         "42",
         "--dry-run",
-        "--quick",
+        "--security",
         "--review-scope",
         "full_code",
         "--max-diff-files",
@@ -100,7 +100,7 @@ def test_build_parser_parses_pr_review_options() -> None:
     assert args.command == "pr-review"
     assert args.pr_id == 42
     assert args.dry_run is True
-    assert args.verbosity == "quick"
+    assert args.verbosity == "security"
     assert args.review_scope == "full_code"
     assert args.max_diff_files == 3
     assert args.output == "review.md"
@@ -333,7 +333,7 @@ def test_select_comments_to_post_respects_individual_answers(mocker) -> None:
 
 @pytest.mark.parametrize(
     ("choice", "expected"),
-    [("1", "quick"), ("2", "detailed"), ("3", "security"), ("", "detailed"), ("99", "detailed")],
+    [("1", "detailed"), ("2", "security"), ("", "detailed"), ("99", "detailed")],
 )
 def test_ask_verbosity(choice: str, expected: str, mocker) -> None:
     """It should map menu choices to supported verbosity values."""

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/tests/test_config.py

@@ -179,7 +179,7 @@ tfs:
   ca_bundle: ~/ca.pem
   repository: repo-a
 review:
-  verbosity: quick
+  verbosity: security
   scope: full_code
   max_diff_files: 12
   max_diff_lines: 456
@@ -222,7 +222,7 @@ output:
     assert config.tfs_verify_ssl is False
     assert config.tfs_ca_bundle == "~/ca.pem"
     assert config.tfs_repository == "repo-a"
-    assert config.verbosity == "quick"
+    assert config.verbosity == "security"
     assert config.review_scope == "full_code"
     assert config.max_diff_files == 12
     assert config.max_diff_lines == 456

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/tests/test_git_utils.py

@@ -249,7 +249,7 @@ def test_filter_diff_additions_only_preserves_full_file_context_block() -> None:
         "--- a/src/app.py",
         "+++ b/src/app.py",
         "@@ -1,2 +1,2 @@",
-        " this_context_line_outside",      # context — must be stripped
+        " this_context_line_outside",      # context — must now be KEPT
         "-deleted_line_outside",            # deletion — must be stripped
         "+added_line_outside",              # addition — must be kept
         "### FULL_FILE_CONTEXT_START: /src/app.py ###",
@@ -262,8 +262,8 @@ def test_filter_diff_additions_only_preserves_full_file_context_block() -> None:
 
     filtered = instance.filter_diff_additions_only(diff)
 
-    assert "this_context_line_outside" not in filtered
-    assert "deleted_line_outside" not in filtered
+    assert "this_context_line_outside" in filtered       # context lines are now kept
+    assert "deleted_line_outside" not in filtered        # deletions are still stripped
     assert "+added_line_outside" in filtered
     assert "### FULL_FILE_CONTEXT_START: /src/app.py ###" in filtered
     assert "raw_context_inside" in filtered
@@ -297,8 +297,8 @@ def test_filter_diff_additions_only_handles_multiple_full_file_context_blocks()
     assert "+line_b" in filtered
 
 
-def test_filter_diff_additions_only_without_context_block_strips_context_and_deletions() -> None:
-    """Without FULL_FILE_CONTEXT blocks the method must still strip context and deletion lines."""
+def test_filter_diff_additions_only_without_context_block_keeps_context_strips_deletions() -> None:
+    """Without FULL_FILE_CONTEXT blocks the method must keep context lines and strip deletion lines."""
     instance = make_git_utils()
     diff = "\n".join([
         "diff --git a/src/app.py b/src/app.py",
@@ -310,18 +310,18 @@ def test_filter_diff_additions_only_without_context_block_strips_context_and_del
     filtered = instance.filter_diff_additions_only(diff)
 
     assert "+added_line" in filtered
-    assert "context_line" not in filtered
-    assert "removed_line" not in filtered
+    assert "context_line" in filtered       # context lines are now kept
+    assert "removed_line" not in filtered   # deletions are still stripped
 
 
 def test_filter_diff_additions_only_context_block_ends_before_next_diff_section() -> None:
-    """The section after ### FULL_FILE_CONTEXT_END ### must revert to normal filtering."""
+    """The section after ### FULL_FILE_CONTEXT_END ### must revert to normal filtering (keep context, strip deletions)."""
     instance = make_git_utils()
     diff = "\n".join([
         "### FULL_FILE_CONTEXT_START: /src/a.py ###",
         "inside_block",
         "### FULL_FILE_CONTEXT_END ###",
-        " outside_context_line",   # context — must be stripped after block ends
+        " outside_context_line",   # context — must be KEPT after block ends
         "-outside_deleted",         # deletion — must be stripped after block ends
         "+outside_added",           # addition — must be kept
     ])
@@ -329,6 +329,6 @@ def test_filter_diff_additions_only_context_block_ends_before_next_diff_section(
     filtered = instance.filter_diff_additions_only(diff)
 
     assert "inside_block" in filtered
-    assert "outside_context_line" not in filtered
+    assert "outside_context_line" in filtered   # context is now kept
     assert "outside_deleted" not in filtered
     assert "+outside_added" in filtered

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/tests/test_llm_client.py

@@ -89,7 +89,7 @@ def make_llm_config(**changes: object) -> ReviewConfig:
 
 def test_prompt_helpers_select_expected_language_and_scope() -> None:
     """It should select prompts and scope guidance consistently."""
-    assert "code reviewer" in get_system_prompt("quick").lower()
+    assert "code reviewer" in get_system_prompt("detailed").lower()
     assert "JSON" in PR_COMMENT_PROMPT
     assert "full_code" in get_scope_guidance("full_code")
     assert "file and line" in get_scope_guidance("diff_only", structured=True).lower()
@@ -104,7 +104,7 @@ def test_get_pr_comment_prompt_function_removed() -> None:
 def test_get_system_prompt_and_get_scope_guidance_reject_language_kwarg() -> None:
     """Regression: language parameter was removed from both prompt helpers."""
     with pytest.raises(TypeError):
-        get_system_prompt("quick", language="pt")  # type: ignore[call-arg]
+        get_system_prompt("detailed", language="pt")  # type: ignore[call-arg]
 
     with pytest.raises(TypeError):
         get_scope_guidance("diff_only", structured=False, language="pt")  # type: ignore[call-arg]
@@ -115,9 +115,9 @@ def test_get_system_prompt_and_get_scope_guidance_reject_language_kwarg() -> Non
 # SYSTEM_PROMPTS / get_system_prompt — Phase 1 change validation
 # ---------------------------------------------------------------------------
 
-def test_get_system_prompt_quick_returns_non_empty_string() -> None:
-    """quick prompt must return a non-empty string."""
-    prompt = get_system_prompt("quick")
+def test_get_system_prompt_detailed_returns_non_empty_string() -> None:
+    """detailed prompt must return a non-empty string."""
+    prompt = get_system_prompt("detailed")
     assert isinstance(prompt, str)
     assert len(prompt.strip()) > 0
 

{code_review_ai_cli-2.0.0 → code_review_ai_cli-2.0.1}/tests/test_tfs_client.py

@@ -354,7 +354,7 @@ def test_build_diff_parts_and_file_content(mocker) -> None:
 # ---------------------------------------------------------------------------
 
 def test_build_unified_diff_part_appends_full_file_context_block(mocker) -> None:
-    """_build_unified_diff_part must append FULL_FILE_CONTEXT markers with the new (source branch, post-change) file content."""
+    """_build_unified_diff_part must append FULL_FILE_CONTEXT markers with numbered new-version file content."""
     client = TFSClient(make_tfs_config())
     mocker.patch(
         "src.tfs_client.TFSClient._get_file_content",
@@ -373,6 +373,9 @@ def test_build_unified_diff_part_appends_full_file_context_block(mocker) -> None
     assert "new line two" in context_block
     assert "old line" not in context_block
     assert "### FULL_FILE_CONTEXT_END ###" in joined
+    # Lines must be prefixed with 1-based line numbers
+    assert "   1: new line one" in context_block
+    assert "   2: new line two" in context_block
     # Context block must appear after the diff headers
     assert joined.index("### FULL_FILE_CONTEXT_START") > joined.index("diff --git")
 
@@ -416,7 +419,7 @@ def test_build_unified_diff_part_no_context_block_on_delete(mocker) -> None:
 
 def test_build_unified_diff_part_context_block_uses_new_file_path(mocker) -> None:
     """The FULL_FILE_CONTEXT_START marker must reference the new file path (file_path), not original_path,
-    and the block content must be the new (renamed) file's content."""
+    and the block content must be the new (renamed) file's content with 1-based line numbers."""
     client = TFSClient(make_tfs_config())
     mocker.patch(
         "src.tfs_client.TFSClient._get_file_content",
@@ -434,6 +437,7 @@ def test_build_unified_diff_part_context_block_uses_new_file_path(mocker) -> Non
     assert "### FULL_FILE_CONTEXT_START: /src/original.py ###" not in joined
     assert "new content" in context_block
     assert "old content" not in context_block
+    assert "   1: new content" in context_block
 
 
 def test_build_unified_diff_part_no_context_block_when_content_unavailable(mocker) -> None: