github-code-review 3.2.0__tar.gz → 3.4.1__tar.gz

{github_code_review-3.2.0 → github_code_review-3.4.1}/PKG-INFO

@@ -1,8 +1,9 @@
-Metadata-Version: 2.3
+Metadata-Version: 2.4
 Name: github-code-review
-Version: 3.2.0
+Version: 3.4.1
 Summary: AI code review tool that works with any language model provider. It detects issues in GitHub pull requests or local changes—instantly, reliably, and without vendor lock-in.
 License: MIT
+License-File: LICENSE
 Keywords: static code analysis,code review,code quality,ai,coding,assistant,llm,github,automation,devops,developer tools,github actions,workflows,git
 Author: Nayjest
 Author-email: mail@vitaliy.in
@@ -14,14 +15,16 @@ Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
 Classifier: Topic :: Software Development
 Requires-Dist: GitPython (>=3.1.44,<4.0.0)
-Requires-Dist: ai-microcore (==4.3.0)
-Requires-Dist: anthropic (>=0.57.1,<0.58.0)
+Requires-Dist: ai-microcore (>=4.5,<4.6)
+Requires-Dist: anthropic (>=0.57.1,<1)
 Requires-Dist: ghapi (>=1.0.6,<1.1.0)
 Requires-Dist: google-generativeai (>=0.8.5,<0.9.0)
 Requires-Dist: jira (>=3.8.0,<4.0.0)
-Requires-Dist: typer (>=0.16.0,<0.17.0)
+Requires-Dist: pydantic (>=2.12.3,<3.0.0)
+Requires-Dist: typer (>=0.16.0,<0.21)
 Requires-Dist: unidiff (>=0.7.5,<0.8.0)
 Project-URL: Homepage, https://github.com/Nayjest/Gito
 Project-URL: Repository, https://github.com/Nayjest/Gito
@@ -99,7 +102,7 @@ jobs:
       uses: actions/setup-python@v5
       with: { python-version: "3.13" }
     - name: Install AI Code Review tool
-      run: pip install gito.bot~=3.0
+      run: pip install gito.bot~=3.4
     - name: Run AI code analysis
       env:
         LLM_API_KEY: ${{ secrets.LLM_API_KEY }}
@@ -130,15 +133,15 @@ See [GitHub Setup Guide](https://github.com/Nayjest/Gito/blob/main/documentation
 
 **Prerequisites:** [Python](https://www.python.org/downloads/) 3.11 / 3.12 / 3.13  
 
-**Step1:** Install [gito.bot](https://github.com/Nayjest/Gito) using [pip](https://en.wikipedia.org/wiki/Pip_(package_manager)).
+**Step 1:** Install [gito.bot](https://github.com/Nayjest/Gito) using [pip](https://en.wikipedia.org/wiki/Pip_(package_manager)).
 ```bash
 pip install gito.bot
 ```
 
 > **Troubleshooting:**  
-> pip may be also available via cli as `pip3` depending on your Python installation.
+> pip may also be available via cli as `pip3` depending on your Python installation.
 
-**Step2:** Perform initial setup
+**Step 2:** Perform initial setup
 
 The following command will perform one-time setup using an interactive wizard.
 You will be prompted to enter LLM configuration details (API type, API key, etc).
@@ -149,15 +152,15 @@ gito setup
 ```
 
 > **Troubleshooting:**  
-> On some systems, `gito` command may not became available immediately after installation.  
+> On some systems, `gito` command may not become available immediately after installation.  
 > Try restarting your terminal or running `python -m gito` instead.
 
 
 #### Perform your first AI code review locally
 
-**Step1:** Navigate to your repository root directory.  
-**Step2:** Switch to the branch you want to review.  
-**Step3:** Run following command
+**Step 1:** Navigate to your repository root directory.  
+**Step 2:** Switch to the branch you want to review.  
+**Step 3:** Run following command
 ```bash
 gito review
 ```
@@ -231,5 +234,5 @@ See [CONTRIBUTING.md](https://github.com/Nayjest/Gito/blob/main/CONTRIBUTING.md)
 
 Licensed under the [MIT License](https://github.com/Nayjest/Gito/blob/main/LICENSE).
 
-© 2025 [Vitalii Stepanenko](mailto:mail@vitaliy.in)
+© 2025–2026 [Vitalii Stepanenko](mailto:mail@vitaliy.in)
 

{github_code_review-3.2.0 → github_code_review-3.4.1}/README.md

@@ -70,7 +70,7 @@ jobs:
       uses: actions/setup-python@v5
       with: { python-version: "3.13" }
     - name: Install AI Code Review tool
-      run: pip install gito.bot~=3.0
+      run: pip install gito.bot~=3.4
     - name: Run AI code analysis
       env:
         LLM_API_KEY: ${{ secrets.LLM_API_KEY }}
@@ -101,15 +101,15 @@ See [GitHub Setup Guide](https://github.com/Nayjest/Gito/blob/main/documentation
 
 **Prerequisites:** [Python](https://www.python.org/downloads/) 3.11 / 3.12 / 3.13  
 
-**Step1:** Install [gito.bot](https://github.com/Nayjest/Gito) using [pip](https://en.wikipedia.org/wiki/Pip_(package_manager)).
+**Step 1:** Install [gito.bot](https://github.com/Nayjest/Gito) using [pip](https://en.wikipedia.org/wiki/Pip_(package_manager)).
 ```bash
 pip install gito.bot
 ```
 
 > **Troubleshooting:**  
-> pip may be also available via cli as `pip3` depending on your Python installation.
+> pip may also be available via cli as `pip3` depending on your Python installation.
 
-**Step2:** Perform initial setup
+**Step 2:** Perform initial setup
 
 The following command will perform one-time setup using an interactive wizard.
 You will be prompted to enter LLM configuration details (API type, API key, etc).
@@ -120,15 +120,15 @@ gito setup
 ```
 
 > **Troubleshooting:**  
-> On some systems, `gito` command may not became available immediately after installation.  
+> On some systems, `gito` command may not become available immediately after installation.  
 > Try restarting your terminal or running `python -m gito` instead.
 
 
 #### Perform your first AI code review locally
 
-**Step1:** Navigate to your repository root directory.  
-**Step2:** Switch to the branch you want to review.  
-**Step3:** Run following command
+**Step 1:** Navigate to your repository root directory.  
+**Step 2:** Switch to the branch you want to review.  
+**Step 3:** Run following command
 ```bash
 gito review
 ```
@@ -202,4 +202,4 @@ See [CONTRIBUTING.md](https://github.com/Nayjest/Gito/blob/main/CONTRIBUTING.md)
 
 Licensed under the [MIT License](https://github.com/Nayjest/Gito/blob/main/LICENSE).
 
-© 2025 [Vitalii Stepanenko](mailto:mail@vitaliy.in)
+© 2025–2026 [Vitalii Stepanenko](mailto:mail@vitaliy.in)

{github_code_review-3.2.0 → github_code_review-3.4.1}/gito/bootstrap.py

@@ -13,6 +13,7 @@ from .env import Env
 
 
 def setup_logging(log_level: int = logging.INFO):
+    """Setup custom CLI logging format with colored output."""
     class CustomFormatter(logging.Formatter):
         def format(self, record):
             dt = datetime.fromtimestamp(record.created).strftime("%Y-%m-%d %H:%M:%S")
@@ -23,7 +24,11 @@ def setup_logging(log_level: int = logging.INFO):
             if record.levelno >= logging.ERROR:
                 message = mc.ui.red(message)
                 level_name = mc.ui.red(level_name)
-            return f"{dt} {level_name}: {message}"
+
+            formatted_message = f"{dt} {level_name}: {message}"
+            if record.exc_info:
+                formatted_message += "\n" + self.formatException(record.exc_info)
+            return formatted_message
 
     handler = logging.StreamHandler()
     handler.setFormatter(CustomFormatter())

{github_code_review-3.2.0 → github_code_review-3.4.1}/gito/cli.py

@@ -7,6 +7,7 @@ import textwrap
 import microcore as mc
 import typer
 from git import Repo
+from gito.constants import REFS_VALUE_ALL
 
 from .core import review, get_diff, filter_diff, answer
 from .cli_base import (
@@ -24,6 +25,7 @@ from .constants import HOME_ENV_PATH, GITHUB_MD_REPORT_FILE_NAME
 from .bootstrap import bootstrap
 from .utils import no_subcommand, extract_gh_owner_repo, remove_html_comments
 from .gh_api import resolve_gh_token
+from .project_config import ProjectConfig
 
 # Import fix command to register it
 from .commands import fix, gh_react_to_comment, repl, deploy, version  # noqa
@@ -97,6 +99,7 @@ def cmd_review(
     filters: str = arg_filters(),
     merge_base: bool = typer.Option(default=True, help="Use merge base for comparison"),
     url: str = typer.Option("", "--url", help="Git repository URL"),
+    path: str = typer.Option("", "--path", help="Git repository path"),
     post_comment: bool = typer.Option(default=False, help="Post review comment to GitHub"),
     pr: int = typer.Option(
         default=None,
@@ -106,8 +109,17 @@ def cmd_review(
         in the github actions PR is resolved from the environment)
         """)
     ),
-    out: str = arg_out()
+    out: str = arg_out(),
+    all: bool = typer.Option(default=False, help="Review all codebase"),
 ):
+    if all:
+        if refs and refs != REFS_VALUE_ALL:
+            raise typer.BadParameter(
+                "The --all option overrides the refs argument. "
+                "Please remove the refs argument if you want to review all codebase."
+            )
+        refs = REFS_VALUE_ALL
+        merge_base = False
     _what, _against = args_to_target(refs, what, against)
     pr = pr or os.getenv("PR_NUMBER_FROM_WORKFLOW_DISPATCH")
     with get_repo_context(url, _what) as (repo, out_folder):
@@ -160,6 +172,11 @@ def cmd_answer(
     aux_files: list[str] = typer.Option(
         default=None,
         help="Auxiliary files that might be helpful"
+    ),
+    save_to: str = typer.Option(
+        help="Save answer to file",
+        default=None,
+        show_default=False
     )
 ):
     _what, _against = args_to_target(refs, what, against)
@@ -183,6 +200,11 @@ def cmd_answer(
     if post_to == 'linear':
         logging.info("Posting answer to Linear...")
         linear_comment(remove_html_comments(out))
+    if save_to:
+        with open(save_to, "w", encoding="utf-8") as f:
+            f.write(out)
+        logging.info(f"Answer saved to {mc.utils.file_link(save_to)}")
+
     return out
 
 
@@ -191,8 +213,8 @@ def setup():
     mc.interactive_setup(HOME_ENV_PATH)
 
 
-@app.command(name="render", help="Render and display code review report.")
-@app.command(name="report", hidden=True)
+@app.command(name="report", help="Render and display code review report.")
+@app.command(name="render", hidden=True)
 def render(
     format: str = typer.Argument(default=Report.Format.CLI),
     source: str = typer.Option(
@@ -223,11 +245,15 @@ def files(
     try:
         patch_set = get_diff(repo=repo, what=_what, against=_against, use_merge_base=merge_base)
         patch_set = filter_diff(patch_set, filters)
+        cfg = ProjectConfig.load_for_repo(repo)
+        if cfg.exclude_files:
+            patch_set = filter_diff(patch_set, cfg.exclude_files, exclude=True)
         print(
             f"Changed files: "
             f"{mc.ui.green(_what or 'INDEX')} vs "
             f"{mc.ui.yellow(_against or repo.remotes.origin.refs.HEAD.reference.name)}"
-            f"{' filtered by ' + mc.ui.cyan(filters) if filters else ''}"
+            f"{' filtered by ' + mc.ui.cyan(filters) if filters else ''} --> "
+            f"{mc.ui.cyan(len(patch_set or []))} file(s)."
         )
 
         for patch in patch_set:

{github_code_review-3.2.0 → github_code_review-3.4.1}/gito/cli_base.py

@@ -5,10 +5,15 @@ import tempfile
 import microcore as mc
 import typer
 from git import Repo
-from gito.utils import parse_refs_pair
+from gito.constants import REFS_VALUE_ALL
+
+from .utils import parse_refs_pair
+from .env import Env
 
 
 def args_to_target(refs, what, against) -> tuple[str | None, str | None]:
+    if refs == REFS_VALUE_ALL:
+        return REFS_VALUE_ALL, None
     _what, _against = parse_refs_pair(refs)
     if _what:
         if what:
@@ -73,6 +78,8 @@ app = typer.Typer(pretty_exceptions_show_locals=False)
 
 @contextlib.contextmanager
 def get_repo_context(url: str, branch: str):
+    if branch == REFS_VALUE_ALL:
+        branch = None
     """Context manager for handling both local and remote repositories."""
     if url:
         with tempfile.TemporaryDirectory() as temp_dir:
@@ -81,10 +88,13 @@ def get_repo_context(url: str, branch: str):
                 f"Cloning [{mc.ui.green(url)}] to {mc.utils.file_link(temp_dir)} ..."
             )
             repo = Repo.clone_from(url, branch=branch, to_path=temp_dir)
+            prev_folder = Env.working_folder
+            Env.working_folder = temp_dir
             try:
                 yield repo, temp_dir
             finally:
                 repo.close()
+                Env.working_folder = prev_folder
     else:
         logging.info("get_repo_context: Using local repo...")
         repo = Repo(".")

{github_code_review-3.2.0 → github_code_review-3.4.1}/gito/config.toml

@@ -37,6 +37,12 @@ report_template_md = """
     {%- endfor -%}
     {{ "\n" }}
 {%- endfor -%}
+{%- if report.processing_warnings -%}
+    {{- "\n\n" }}## Processing Warnings
+    {%- for warning in report.processing_warnings -%}
+        {{- "\n" }}  - {{ warning.message -}}
+    {%- endfor -%}
+{%- endif -%}
 {{- HTML_CR_COMMENT_MARKER -}}
 """
 report_template_cli = """
@@ -109,6 +115,13 @@ report_template_cli = """
     {%- endfor -%}
     {{ "\n" }}
 {%- endfor -%}
+{%- if report.processing_warnings -%}
+    {{- "\n" }}
+    {{- "\n" }}{{ Style.BRIGHT }}⚠️ PROCESSING WARNINGS{{ Style.RESET_ALL -}}
+    {%- for warning in report.processing_warnings -%}
+        {{- "\n" }}  {{ Fore.YELLOW }}- {{ warning.message }}{{ Style.RESET_ALL -}}
+    {%- endfor -%}
+{%- endif -%}
 """
 retries = 3
 prompt = """
@@ -184,7 +197,7 @@ Confidence scale:
 - 1 — Highest, 100% confidence that code requires changes in any context
 - 2 — Very High
 - 3 — High
-- 4 — Medium - Should not be reported
+- 4 — Medium, Should not be reported
 
 (!) - If no issues found according to the criteria, respond with empty list: []
 """
@@ -204,12 +217,14 @@ Summarize the code review in one sentence.
 --Issues Detected by You--
 {{ issues | tojson(indent=2) }}
 ---
+{% if awards -%}
 If the code changes include exceptional achievements, you may also present an award to the author in the summary text.
  - (!) Only give awards to initial codebase authors, NOT to reviewers.
  - (!) If you give an award, place the hidden <!-- award --> HTML comment on its own line immediately before the award text.
 --Available Awards--
 {{ awards }}
 ---
+{%- endif %}
 {% if pipeline_out.associated_issue and pipeline_out.associated_issue.title %}
 ----SUBTASK----
 Include one sentence about how the code changes address the requirements of the associated issue listed below.

{github_code_review-3.2.0 → github_code_review-3.4.1}/gito/constants.py

@@ -12,3 +12,4 @@ EXECUTABLE = "gito"
 TEXT_ICON_URL = 'https://raw.githubusercontent.com/Nayjest/Gito/main/press-kit/logo/gito-bot-1_64top.png'  # noqa: E501
 HTML_TEXT_ICON = f'<a href="https://github.com/Nayjest/Gito"><img src="{TEXT_ICON_URL}" align="left" width=64 height=50 title="Gito v{Env.gito_version}"/></a>'  # noqa: E501
 HTML_CR_COMMENT_MARKER = '<!-- GITO_COMMENT:CODE_REVIEW_REPORT -->'
+REFS_VALUE_ALL = '!all'

{github_code_review-3.2.0 → github_code_review-3.4.1}/gito/core.py

@@ -1,3 +1,6 @@
+"""
+Gito core business logic.
+"""
 import os
 import fnmatch
 import logging
@@ -6,7 +9,6 @@ from pathlib import Path
 from functools import partial
 
 import microcore as mc
-from gito.gh_api import gh_api
 from microcore import ui
 from git import Repo, Commit
 from git.exc import GitCommandError
@@ -15,11 +17,12 @@ from unidiff.constants import DEV_NULL
 
 from .context import Context
 from .project_config import ProjectConfig
-from .report_struct import Report
-from .constants import JSON_REPORT_FILE_NAME
+from .report_struct import ProcessingWarning, Report, RawIssue
+from .constants import JSON_REPORT_FILE_NAME, REFS_VALUE_ALL
 from .utils import make_streaming_function
 from .pipeline import Pipeline
 from .env import Env
+from .gh_api import gh_api
 
 
 def review_subject_is_index(what):
@@ -94,7 +97,7 @@ def get_base_branch(repo: Repo, pr: int | str = None):
     except AttributeError:
         try:
             logging.info(
-                "Checking if repo has 'main' or 'master' branchs to use as --against branch..."
+                "Checking if repo has 'main' or 'master' branches to use as --against branch..."
             )
             remote_refs = repo.remotes.origin.refs
             for branch_name in ['main', 'master']:
@@ -115,91 +118,99 @@ def get_diff(
     pr: str | int = None
 ) -> PatchSet | list[PatchedFile]:
     repo = repo or Repo(".")
+    if what == REFS_VALUE_ALL:
+        what = get_base_branch(repo, pr=pr)
+        # Git's canonical empty tree hash
+        against = "4b825dc642cb6eb9a060e54bf8d69288fbee4904"
+        use_merge_base = False
     if not against:
         # 'origin/main', 'origin/master', etc
         against = get_base_branch(repo, pr=pr)
     if review_subject_is_index(what):
         what = None  # working copy
     if use_merge_base:
-        if review_subject_is_index(what):
-            try:
-                current_ref = repo.active_branch.name
-            except TypeError:
-                # In detached HEAD state, use HEAD directly
-                current_ref = "HEAD"
-                logging.info(
-                    "Detected detached HEAD state, using HEAD as current reference"
-                )
-        else:
-            current_ref = what
-        merge_base = repo.merge_base(current_ref or repo.active_branch.name, against)[0]
-        logging.info(
-            f"Merge base({ui.green(current_ref)},{ui.yellow(against)})"
-            f" --> {ui.cyan(merge_base.hexsha)}"
-        )
-        # if branch is already an ancestor of "against", merge_base == branch ⇒ it’s been merged
-        if merge_base.hexsha == repo.commit(current_ref or repo.active_branch.name).hexsha:
-            # @todo: check case: reviewing working copy index in main branch #103
-            logging.info(
-                f"Branch is already merged. ({ui.green(current_ref)} vs {ui.yellow(against)})"
-            )
-            merge_sha = repo.git.log(
-                '--merges',
-                '--ancestry-path',
-                f'{current_ref}..{against}',
-                '-n',
-                '1',
-                '--pretty=format:%H'
-            ).strip()
-            if merge_sha:
-                logging.info(f"Merge commit is {ui.cyan(merge_sha)}")
-                merge_commit = repo.commit(merge_sha)
-
-                other_merge_parent = None
-                for parent in merge_commit.parents:
-                    logging.info(f"Checking merge parent: {parent.hexsha[:8]}")
-                    if parent.hexsha == merge_base.hexsha:
-                        logging.info(f"merge parent is {ui.cyan(parent.hexsha[:8])}, skipping")
-                        continue
-                    if not commit_in_branch(repo, parent, against):
-                        logging.warning(f"merge parent is not in {against}, skipping")
-                        continue
-                    logging.info(f"Found other merge parent: {ui.cyan(parent.hexsha[:8])}")
-                    other_merge_parent = parent
-                    break
-                if other_merge_parent:
-                    first_common_ancestor = repo.merge_base(other_merge_parent, merge_base)[0]
-                    # for gito remote (feature_branch vs origin/main)
-                    # the same merge base appears in first_common_ancestor again
-                    if first_common_ancestor.hexsha == merge_base.hexsha:
-                        if merge_base.parents:
-                            first_common_ancestor = repo.merge_base(
-                                other_merge_parent, merge_base.parents[0]
-                            )[0]
-                        else:
-                            logging.error(
-                                "merge_base has no parents, "
-                                "using merge_base as first_common_ancestor"
-                            )
+        try:
+            if review_subject_is_index(what):
+                try:
+                    current_ref = repo.active_branch.name
+                except TypeError:
+                    # In detached HEAD state, use HEAD directly
+                    current_ref = "HEAD"
                     logging.info(
-                        f"{what} will be compared to "
-                        f"first common ancestor of {what} and {against}: "
-                        f"{ui.cyan(first_common_ancestor.hexsha[:8])}"
+                        "Detected detached HEAD state, using HEAD as current reference"
                     )
-                    against = first_common_ancestor.hexsha
-                else:
-                    logging.error(f"Can't find other merge parent for {merge_sha}")
             else:
-                logging.warning(
-                    f"No merge‐commit found for {current_ref!r}→{against!r}; "
-                    "falling back to merge‐base diff"
-                )
-        else:
-            # normal case: branch not yet merged
-            against = merge_base.hexsha
+                current_ref = what
+            merge_base = repo.merge_base(current_ref or repo.active_branch.name, against)[0]
             logging.info(
-                f"Using merge base: {ui.cyan(merge_base.hexsha[:8])} ({merge_base.summary})"
+                f"Merge base({ui.green(current_ref)},{ui.yellow(against)})"
+                f" --> {ui.cyan(merge_base.hexsha)}"
             )
+            # if branch is already an ancestor of "against", merge_base == branch ⇒ it’s been merged
+            if merge_base.hexsha == repo.commit(current_ref or repo.active_branch.name).hexsha:
+                # @todo: check case: reviewing working copy index in main branch #103
+                logging.info(
+                    f"Branch is already merged. ({ui.green(current_ref)} vs {ui.yellow(against)})"
+                )
+                merge_sha = repo.git.log(
+                    '--merges',
+                    '--ancestry-path',
+                    f'{current_ref}..{against}',
+                    '-n',
+                    '1',
+                    '--pretty=format:%H'
+                ).strip()
+                if merge_sha:
+                    logging.info(f"Merge commit is {ui.cyan(merge_sha)}")
+                    merge_commit = repo.commit(merge_sha)
+
+                    other_merge_parent = None
+                    for parent in merge_commit.parents:
+                        logging.info(f"Checking merge parent: {parent.hexsha[:8]}")
+                        if parent.hexsha == merge_base.hexsha:
+                            logging.info(f"merge parent is {ui.cyan(parent.hexsha[:8])}, skipping")
+                            continue
+                        if not commit_in_branch(repo, parent, against):
+                            logging.warning(f"merge parent is not in {against}, skipping")
+                            continue
+                        logging.info(f"Found other merge parent: {ui.cyan(parent.hexsha[:8])}")
+                        other_merge_parent = parent
+                        break
+                    if other_merge_parent:
+                        first_common_ancestor = repo.merge_base(other_merge_parent, merge_base)[0]
+                        # for gito remote (feature_branch vs origin/main)
+                        # the same merge base appears in first_common_ancestor again
+                        if first_common_ancestor.hexsha == merge_base.hexsha:
+                            if merge_base.parents:
+                                first_common_ancestor = repo.merge_base(
+                                    other_merge_parent, merge_base.parents[0]
+                                )[0]
+                            else:
+                                logging.error(
+                                    "merge_base has no parents, "
+                                    "using merge_base as first_common_ancestor"
+                                )
+                        logging.info(
+                            f"{what} will be compared to "
+                            f"first common ancestor of {what} and {against}: "
+                            f"{ui.cyan(first_common_ancestor.hexsha[:8])}"
+                        )
+                        against = first_common_ancestor.hexsha
+                    else:
+                        logging.error(f"Can't find other merge parent for {merge_sha}")
+                else:
+                    logging.warning(
+                        f"No merge‐commit found for {current_ref!r}→{against!r}; "
+                        "falling back to merge‐base diff"
+                    )
+            else:
+                # normal case: branch not yet merged
+                against = merge_base.hexsha
+                logging.info(
+                    f"Using merge base: {ui.cyan(merge_base.hexsha[:8])} ({merge_base.summary})"
+                )
+        except Exception as e:
+            logging.error(f"Error finding merge base: {e}")
     logging.info(
         f"Making diff: {ui.green(what or 'INDEX')} vs {ui.yellow(against)}"
     )
@@ -225,12 +236,21 @@ def get_diff(
 
 
 def filter_diff(
-    patch_set: PatchSet | Iterable[PatchedFile], filters: str | list[str]
+    patch_set: PatchSet | Iterable[PatchedFile],
+    filters: str | list[str],
+    exclude: bool = False,
 ) -> PatchSet | Iterable[PatchedFile]:
     """
     Filter the diff files by the given fnmatch filters.
+    Args:
+        patch_set (PatchSet | Iterable[PatchedFile]): The diff to filter.
+        filters (str | list[str]): The fnmatch patterns to filter by.
+        exclude (bool): If True, inverse logic (exclude files matching the filters).
+    Returns:
+        PatchSet | Iterable[PatchedFile]: The filtered diff.
     """
-    assert isinstance(filters, (list, str))
+    if not isinstance(filters, (list, str)):
+        raise ValueError("Filters must be a string or a list of strings")
     if not isinstance(filters, list):
         filters = [f.strip() for f in filters.split(",") if f.strip()]
     if not filters:
@@ -238,7 +258,10 @@ def filter_diff(
     files = [
         file
         for file in patch_set
-        if any(fnmatch.fnmatch(file.path, pattern) for pattern in filters)
+        if (
+            not any(fnmatch.fnmatch(file.path, pattern) for pattern in filters) if exclude
+            else any(fnmatch.fnmatch(file.path, pattern) for pattern in filters)
+        )
     ]
     return files
 
@@ -319,6 +342,8 @@ def _prepare(
         repo=repo, what=what, against=against, use_merge_base=use_merge_base, pr=pr,
     )
     diff = filter_diff(diff, filters)
+    if cfg.exclude_files:
+        diff = filter_diff(diff, cfg.exclude_files, exclude=True)
     if not diff:
         raise NoChangesInContextError()
     lines = {
@@ -328,9 +353,9 @@ def _prepare(
                 file_diff.path,
                 cfg.max_code_tokens
                 - mc.tokenizing.num_tokens_from_string(str(file_diff)),
-                use_local_files=review_subject_is_index(what)
+                use_local_files=review_subject_is_index(what) or what == REFS_VALUE_ALL
             )
-            if file_diff.target_file != DEV_NULL and not file_diff.is_added_file
+            if file_diff.target_file != DEV_NULL or what == REFS_VALUE_ALL
             else ""
         )
         for file_diff in diff
@@ -373,6 +398,19 @@ def provide_affected_code_blocks(issues: dict, repo: Repo):
                     i["affected_code"] = block
 
 
+def _llm_response_validator(parsed_response: list[dict]):
+    """
+    Validate that the LLM response is a list of dicts that can be converted to RawIssue.
+    """
+    if not isinstance(parsed_response, list):
+        raise ValueError("Response is not a list")
+    for item in parsed_response:
+        if not isinstance(item, dict):
+            raise ValueError("Response item is not a dict")
+        RawIssue(**item)
+    return True
+
+
 async def review(
     repo: Repo = None,
     what: str = None,
@@ -382,6 +420,11 @@ async def review(
     out_folder: str | os.PathLike | None = None,
     pr: str | int = None
 ):
+    """
+    Conducts a code review.
+    Prints the review report to the console and saves it to a file.
+    """
+    reviewing_all = what == REFS_VALUE_ALL
     try:
         repo, cfg, diff, lines = _prepare(
             repo=repo,
@@ -394,25 +437,62 @@ async def review(
     except NoChangesInContextError:
         logging.error("No changes to review")
         return
+
+    def input_is_diff(file_diff: PatchedFile) -> bool:
+        """
+        In case of reviewing all changes, or added files,
+        we provide full file content as input.
+        Otherwise, we provide the diff and additional file lines separately.
+        """
+        return not reviewing_all and not file_diff.is_added_file
+
     responses = await mc.llm_parallel(
         [
             mc.prompt(
                 cfg.prompt,
-                input=file_diff,
-                file_lines=lines[file_diff.path],
+                input=(
+                    file_diff if input_is_diff(file_diff)
+                    else str(file_diff.path) + ":\n" + lines[file_diff.path]
+                ),
+                file_lines=lines[file_diff.path] if input_is_diff(file_diff) else None,
                 **cfg.prompt_vars,
             )
             for file_diff in diff
         ],
         retries=cfg.retries,
-        parse_json=True,
+        parse_json={"validator": _llm_response_validator},
+        allow_failures=True,
     )
+    processing_warnings = []
+    for i, (res_or_error, file) in enumerate(zip(responses, diff)):
+        if isinstance(res_or_error, Exception):
+            if isinstance(res_or_error, mc.LLMContextLengthExceededError):
+                message = f'File "{file.path}" was skipped due to large size: {str(res_or_error)}.'
+            else:
+                message = (
+                    f"File {file.path} was skipped due to error: "
+                    f"[{type(res_or_error).__name__}] {res_or_error}"
+                )
+                if not message.endswith('.'):
+                    message += '.'
+            processing_warnings.append(
+                ProcessingWarning(
+                    message=message,
+                    file=file.path,
+                )
+            )
+            responses[i] = []
+
     issues = {file.path: issues for file, issues in zip(diff, responses) if issues}
     provide_affected_code_blocks(issues, repo)
     exec(cfg.post_process, {"mc": mc, **locals()})
     out_folder = Path(out_folder or repo.working_tree_dir)
     out_folder.mkdir(parents=True, exist_ok=True)
-    report = Report(issues=issues, number_of_processed_files=len(diff))
+    report = Report(
+        number_of_processed_files=len(diff),
+        processing_warnings=processing_warnings,
+    )
+    report.register_issues(issues)
     ctx = Context(
         report=report,
         config=cfg,
@@ -448,6 +528,10 @@ def answer(
     pr: str | int = None,
     aux_files: list[str] = None,
 ) -> str | None:
+    """
+    Answers a question about the code changes.
+    Returns the LLM response as a string.
+    """
     try:
         repo, config, diff, lines = _prepare(
             repo=repo,
@@ -477,11 +561,11 @@ def answer(
     if aux_files or config.aux_files:
         aux_files_dict = read_files(
             repo,
-            (aux_files or list()) + config.aux_files,
+            (aux_files or []) + config.aux_files,
             config.max_code_tokens // 2
         )
     else:
-        aux_files_dict = dict()
+        aux_files_dict = {}
 
     if not prompt_file and config.answer_prompt.startswith("tpl:"):
         prompt_file = str(config.answer_prompt)[4:]

{github_code_review-3.2.0 → github_code_review-3.4.1}/gito/env.py

@@ -5,3 +5,4 @@ class Env:
     logging_level: int = 1
     verbosity: int = 1
     gito_version: str = version("gito.bot")
+    working_folder = "."

{github_code_review-3.2.0 → github_code_review-3.4.1}/gito/gh_api.py

@@ -67,9 +67,9 @@ def post_gh_comment(
     if 200 <= resp.status_code < 300:
         logging.info(f"Posted review comment to #{pr_or_issue_number} in {gh_repository}")
         return True
-    else:
-        logging.error(f"Failed to post comment: {resp.status_code} {resp.reason}\n{resp.text}")
-        return False
+
+    logging.error(f"Failed to post comment: {resp.status_code} {resp.reason}\n{resp.text}")
+    return False
 
 
 def hide_gh_comment(

{github_code_review-3.2.0 → github_code_review-3.4.1}/gito/project_config.py

@@ -33,6 +33,10 @@ class ProjectConfig:
     when referenced in code review comments.
     """
     aux_files: list[str] = field(default_factory=list)
+    exclude_files: list[str] = field(default_factory=list)
+    """
+    List of file patterns to exclude from analysis.
+    """
     pipeline_steps: dict[str, dict | PipelineStep] = field(default_factory=dict)
     collapse_previous_code_review_comments: bool = field(default=True)
     """

{github_code_review-3.2.0 → github_code_review-3.4.1}/gito/report_struct.py

@@ -1,48 +1,63 @@
 import json
 import logging
-from dataclasses import dataclass, field, asdict
+from dataclasses import field, asdict, is_dataclass
 from datetime import datetime
 from enum import StrEnum
 from pathlib import Path
 
+import textwrap
 import microcore as mc
-from colorama import Fore, Style, Back
 from microcore.utils import file_link
-import textwrap
+from colorama import Fore, Style, Back
+from pydantic.dataclasses import dataclass
 
 from .constants import JSON_REPORT_FILE_NAME, HTML_TEXT_ICON, HTML_CR_COMMENT_MARKER
 from .project_config import ProjectConfig
-from .utils import syntax_hint, block_wrap_lr, max_line_len, remove_html_comments
+from .utils import syntax_hint, block_wrap_lr, max_line_len, remove_html_comments, filter_kwargs
 
 
 @dataclass
-class Issue:
+class RawIssue:
     @dataclass
     class AffectedCode:
         start_line: int = field()
         end_line: int | None = field(default=None)
+        proposal: str | None = field(default="")
+
+    title: str = field()
+    details: str | None = field(default="")
+    severity: int | None = field(default=None)
+    confidence: int | None = field(default=None)
+    tags: list[str] = field(default_factory=list)
+    affected_lines: list[AffectedCode] = field(default_factory=list)
+
+
+@dataclass
+class Issue(RawIssue):
+    @dataclass
+    class AffectedCode(RawIssue.AffectedCode):
         file: str = field(default="")
-        proposal: str = field(default="")
         affected_code: str = field(default="")
 
         @property
         def syntax_hint(self) -> str:
             return syntax_hint(self.file)
 
-    id: str = field()
-    title: str = field()
-    details: str = field(default="")
-    severity: int | None = field(default=None)
-    confidence: int | None = field(default=None)
-    tags: list[str] = field(default_factory=list)
+    id: int | str = field(kw_only=True)
     file: str = field(default="")
     affected_lines: list[AffectedCode] = field(default_factory=list)
 
-    def __post_init__(self):
-        self.affected_lines = [
-            Issue.AffectedCode(**dict(file=self.file) | i)
-            for i in self.affected_lines
-        ]
+    @staticmethod
+    def from_raw_issue(file: str, raw_issue: RawIssue | dict, issue_id: int | str) -> "Issue":
+        if is_dataclass(raw_issue):
+            raw_issue = asdict(raw_issue)
+        params = filter_kwargs(Issue, raw_issue | {"file": file, "id": issue_id})
+        for i, obj in enumerate(params.get("affected_lines") or []):
+            d = obj if isinstance(obj, dict) else asdict(obj)
+            params["affected_lines"][i] = Issue.AffectedCode(
+                **filter_kwargs(Issue.AffectedCode, {"file": file} | d)
+            )
+        return Issue(**params)
 
     def github_code_link(self, github_env: dict) -> str:
         url = (
@@ -57,6 +72,15 @@ class Issue:
         return url
 
 
+@dataclass
+class ProcessingWarning:
+    """
+    Warning generated during code review of files
+    """
+    message: str = field()
+    file: str | None = field(default=None)
+
+
 @dataclass
 class Report:
     class Format(StrEnum):
@@ -70,6 +94,7 @@ class Report:
     created_at: str = field(default_factory=lambda: datetime.now().strftime("%Y-%m-%d %H:%M:%S"))
     model: str = field(default_factory=lambda: mc.config().MODEL)
     pipeline_out: dict = field(default_factory=dict)
+    processing_warnings: list[ProcessingWarning] = field(default_factory=list)
 
     @property
     def plain_issues(self):
@@ -79,19 +104,20 @@ class Report:
             for issue in issues
         ]
 
+    def register_issues(self, issues: dict[str, list[RawIssue | dict]]):
+        for file, file_issues in issues.items():
+            for issue in file_issues:
+                self.register_issue(file, issue)
+
+    def register_issue(self, file: str, issue: RawIssue | dict):
+        if file not in self.issues:
+            self.issues[file] = []
+        total = len(self.plain_issues)
+        self.issues[file].append(Issue.from_raw_issue(file, issue, issue_id=total + 1))
+        self.total_issues = total + 1
+
     def __post_init__(self):
-        issue_id: int = 0
-        for file in self.issues.keys():
-            self.issues[file] = [
-                Issue(
-                    **{
-                        "id": (issue_id := issue_id + 1),
-                        "file": file,
-                    } | issue
-                )
-                for issue in self.issues[file]
-            ]
-        self.total_issues = issue_id
+        self.total_issues = len(self.plain_issues)
 
     def save(self, file_name: str = ""):
         file_name = file_name or JSON_REPORT_FILE_NAME

{github_code_review-3.2.0 → github_code_review-3.4.1}/gito/utils.py

@@ -1,6 +1,8 @@
+import logging
 import re
 import sys
 import os
+from dataclasses import fields, is_dataclass
 from pathlib import Path
 import importlib.metadata
 from typing import Optional
@@ -8,7 +10,7 @@ from typing import Optional
 import typer
 import git
 from git import Repo
-
+from .env import Env
 
 _EXT_TO_HINT: dict[str, str] = {
     # scripting & languages
@@ -192,10 +194,10 @@ def detect_github_env() -> dict:
         "github_ref": ref,
     }
     # Fallback for local usage: try to get from git
-    if not repo:
+    if not repo or repo == "octocat/Hello-World":
         git_repo = None
         try:
-            git_repo = Repo(".", search_parent_directories=True)
+            git_repo = Repo(Env.working_folder, search_parent_directories=True)
             origin = git_repo.remotes.origin.url
             # e.g. git@github.com:Nayjest/ai-code-review.git -> Nayjest/ai-code-review
             match = re.search(r"[:/]([\w\-]+)/([\w\-\.]+?)(\.git)?$", origin)
@@ -240,3 +242,29 @@ def remove_html_comments(text):
     Removes all HTML comments (<!-- ... -->) from the input text.
     """
     return re.sub(r'<!--.*?-->\s*', '', text, flags=re.DOTALL)
+
+
+def filter_kwargs(cls, kwargs, log_warnings=True):
+    """
+    Filters the keyword arguments to only include those that are fields of the given dataclass.
+    Args:
+        cls: The dataclass type to filter against.
+        kwargs: A dictionary of keyword arguments.
+        log_warnings: If True, logs warnings for fields not in the dataclass.
+    Returns:
+        A dictionary containing only the fields that are defined in the dataclass.
+    """
+    if not is_dataclass(cls):
+        raise TypeError(f"{cls.__name__} is not a dataclass or pydantic dataclass")
+
+    cls_fields = {f.name for f in fields(cls)}
+    filtered = {}
+    for k, v in kwargs.items():
+        if k in cls_fields:
+            filtered[k] = v
+        else:
+            if log_warnings:
+                logging.warning(
+                    f"Warning: field '{k}' not in {cls.__name__}, dropping."
+                )
+    return filtered

{github_code_review-3.2.0 → github_code_review-3.4.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "github-code-review"
-version = "3.2.0"
+version = "3.4.1"
 description = "AI code review tool that works with any language model provider. It detects issues in GitHub pull requests or local changes—instantly, reliably, and without vendor lock-in."
 authors = ["Nayjest <mail@vitaliy.in>"]
 readme = "README.md"
@@ -21,14 +21,15 @@ packages = [
 
 [tool.poetry.dependencies]
 python = "^3.11"
-ai-microcore = "4.3.0"
+ai-microcore = "~4.5"
 GitPython = "^3.1.44"
 unidiff = "^0.7.5"
 google-generativeai = "^0.8.5"
-anthropic = "^0.57.1"
-typer = "^0.16.0"
+anthropic = ">=0.57.1,<1"
+typer = ">=0.16.0,<0.21"
 ghapi = "~=1.0.6"
 jira = "^3.8.0"
+pydantic = "^2.12.3"
 
 [tool.poetry.group.dev.dependencies]
 flake8 = "*"
@@ -40,10 +41,10 @@ pyflakes = "*"
 poetry = "*"
 
 [tool.poetry.group.test.dependencies]
-pytest = "^7.4.3"
-pytest-asyncio = ">=0.21.0"
+pytest = ">=7.4.3,<10"
+pytest-asyncio = ">=0.21.0,<2"
 pytest-mock = "^3.12.0"
-pytest-cov = "^4.1.0"
+pytest-cov = ">=4.1.0,<8"
 
 [build-system]
 requires = ["poetry-core"]