ai-cr 3.2.0__py3-none-any.whl → 3.2.2__py3-none-any.whl

{ai_cr-3.2.0.dist-info → ai_cr-3.2.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: ai-cr
-Version: 3.2.0
+Version: 3.2.2
 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
 Keywords: static code analysis,code review,code quality,ai,coding,assistant,llm,github,automation,devops,developer tools,github actions,workflows,git

{ai_cr-3.2.0.dist-info → ai_cr-3.2.2.dist-info}/RECORD

@@ -14,7 +14,7 @@ gito/commands/version.py,sha256=OdAuKtjGV9Ok2_igAs-EqdJijXCKU1dcLcA5KEz1ydg,178
 gito/config.toml,sha256=51TeM_m0_v89u3X1lmafA-D5TqjiXWc-j1W_12VBppg,17777
 gito/constants.py,sha256=1ElhE4RH0EPEq3xlhwyYRUcgr38X8wXduos0gV9yPy8,819
 gito/context.py,sha256=OBfcQOREsNx8WHANsplNrnrKYrXz1PyZyne11lSfZjw,446
-gito/core.py,sha256=H4DcVFt46BdqXfy1AxYFJMFlKgzLiXzBPqY4oaO6NeM,17379
+gito/core.py,sha256=nw3eUiLFWDOQJB7N_y6UE5hUrr4cS8xD_Q_RgktepUM,17795
 gito/env.py,sha256=TVNxqrnLefqfZt5sSg495p8UenSgHaMgTImK1rRMIlY,146
 gito/gh_api.py,sha256=2yDikXr9BM2tndYpCo-weJxjoCAlEtuPh-QBinlnHtg,4052
 gito/issue_trackers.py,sha256=XYspyaIuf0ANQSvDUea5_oOdo9tQvpZZsapI5S9g78U,1551
@@ -23,7 +23,7 @@ gito/pipeline_steps/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSu
 gito/pipeline_steps/jira.py,sha256=NjFgpGFAkT5PSXi6jQ9Yr8CY9M8sa3_rMb7RFhdpoNg,1862
 gito/pipeline_steps/linear.py,sha256=6UDc8nGKGpwHruPq8VItE2QBWshWxaTapoMhu_qjN_g,2445
 gito/project_config.py,sha256=ZdHy4aNzjlVAgTUjmznrGcdd2NHMMvGX8VeSmAnzZ34,3237
-gito/report_struct.py,sha256=96gDYnw0MXhOZrrGaNOTyWstqpjjS7_cuWufd0XJzR4,4320
+gito/report_struct.py,sha256=zSMhPojsBIK34Kp9i74qKJ09DupKluuHHsubS1t9_1E,4399
 gito/tpl/answer.j2,sha256=JrBOv-a8xBqTccRJeWly2U1Y5jehc_mOBt5nSbRJldI,700
 gito/tpl/github_workflows/components/env-vars.j2,sha256=ypmf938h5PA38mXTZnP1eI4Un3AIhhmnl6wXg2X4kqI,406
 gito/tpl/github_workflows/components/installs.j2,sha256=j5wl0yVEIrXZDpAgzqBwmhXQA9End3xFspPxr2ZzHR0,693
@@ -33,9 +33,9 @@ gito/tpl/partial/aux_files.j2,sha256=lJhqnCsHBbEEocpyyOmQX27jzuLvEIuEVXY0RGqxWnY
 gito/tpl/questions/changes_summary.j2,sha256=N80OQoo9UKii0CWLuck5bOwbijul5RefvCqHJljenmE,2213
 gito/tpl/questions/release_notes.j2,sha256=OXi6o7T1bum88_2Pt4FiLHmKUe86A1t9Be_3s4mrnmU,889
 gito/tpl/questions/test_cases.j2,sha256=bB7ESjy02mwWml4zyq87DqkFDj-I0-BYpfJVgzE77cc,1410
-gito/utils.py,sha256=OSBn7IeWKjoLJoGOcdgQcJHBA69UiHUChtaYMInUeko,6852
-ai_cr-3.2.0.dist-info/LICENSE,sha256=VbdF_GbbDK24JvdTfnsxa2M6jmhsxmRSFeHCx-lICGE,1075
-ai_cr-3.2.0.dist-info/METADATA,sha256=jpHSdAUyaroAZRxlDG6rX2QfohgnCgrbCvFHslRguPA,8874
-ai_cr-3.2.0.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-ai_cr-3.2.0.dist-info/entry_points.txt,sha256=Ua1DxkhJJ8TZuLgnH-IlWCkrre_0S0dq_GtYRaYupWk,38
-ai_cr-3.2.0.dist-info/RECORD,,
+gito/utils.py,sha256=-agcMHug8nbrPxfz2nAtOKEAox7Kt-0_AIC3d-X23NE,7693
+ai_cr-3.2.2.dist-info/LICENSE,sha256=VbdF_GbbDK24JvdTfnsxa2M6jmhsxmRSFeHCx-lICGE,1075
+ai_cr-3.2.2.dist-info/METADATA,sha256=9M0y1sbIyF1v3zl96TjACPBfdKn8ha5PZWk0t_sS-B0,8874
+ai_cr-3.2.2.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+ai_cr-3.2.2.dist-info/entry_points.txt,sha256=Ua1DxkhJJ8TZuLgnH-IlWCkrre_0S0dq_GtYRaYupWk,38
+ai_cr-3.2.2.dist-info/RECORD,,

gito/core.py

@@ -121,85 +121,88 @@ def get_diff(
     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)}"
     )

gito/report_struct.py

@@ -12,7 +12,7 @@ import textwrap
 
 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
@@ -40,7 +40,7 @@ class Issue:
 
     def __post_init__(self):
         self.affected_lines = [
-            Issue.AffectedCode(**dict(file=self.file) | i)
+            Issue.AffectedCode(**filter_kwargs(Issue.AffectedCode, dict(file=self.file) | i))
             for i in self.affected_lines
         ]
 
@@ -63,7 +63,7 @@ class Report:
         MARKDOWN = "md"
         CLI = "cli"
 
-    issues: dict[str, list[Issue]] = field(default_factory=dict)
+    issues: dict[str, list[Issue | dict]] = field(default_factory=dict)
     summary: str = field(default="")
     number_of_processed_files: int = field(default=0)
     total_issues: int = field(init=False)
@@ -84,10 +84,10 @@ class Report:
         for file in self.issues.keys():
             self.issues[file] = [
                 Issue(
-                    **{
+                    **filter_kwargs(Issue, {
                         "id": (issue_id := issue_id + 1),
                         "file": file,
-                    } | issue
+                    } | issue)
                 )
                 for issue in self.issues[file]
             ]

gito/utils.py

@@ -1,6 +1,8 @@
+import logging
 import re
 import sys
 import os
+from dataclasses import fields
 from pathlib import Path
 import importlib.metadata
 from typing import Optional
@@ -240,3 +242,26 @@ 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.
+    """
+    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