Merge pull request #36 from Nayjest/bug_32

#32 Bug: Wrong calculation of reviewed files qty

Author: Nayjest

README.md

@@ -41,7 +41,7 @@ jobs:
       uses: actions/setup-python@v5
       with: { python-version: "3.13" }
     - name: Install AI Code Review tool
-      run: pip install ai-code-review==0.4.0
+      run: pip install ai-code-review==0.4.1
     - name: Run AI code review
       env:
         LLM_API_KEY: ${{ secrets.LLM_API_KEY }}

ai_code_review/.ai-code-review.toml

@@ -5,11 +5,10 @@ report_template_md = """
     {{ report.summary }}
 {%- endif %}
 
-**
-{%- if report.total_issues > 0 %}⚠️{% endif -%}
-Total issues: `{{ report.total_issues }}`**
+**{%- if report.total_issues > 0 %}⚠️{% endif -%}
+Total issues: `{{ report.total_issues }}`
 {{- ' ' -}}
-**reviewed files: `{{ report.issues | count }}`**
+in `{{ report.number_of_processed_files }}` files**
 
 {%- for issue in report.plain_issues -%}
     {{"\n"}}## `#{{ issue.id}}`  {{ issue.title -}}
@@ -66,7 +65,7 @@ Respond with a valid JSON array of issues in the following format:
         "affected_lines": [ // optional; list of affected lines
             {
                 "start_line": <start_line:int>,
-                "end_line": <end_line:int>
+                "end_line": <end_line:int>,
                 "proposal": "<proposed code to replace the affected lines (optional)>"
             },
             ...

ai_code_review/core.py

@@ -106,7 +106,7 @@ async def review(filters: str | list[str] = ""):
                         f_lines[i["start_line"]: i["end_line"]+1]
                     )
     exec(cfg.post_process, {"mc": mc, **locals()})
-    report = Report(issues=issues)
+    report = Report(issues=issues, number_of_processed_files=len(diff))
     report.summary = make_cr_summary(cfg, report, diff)
     report.save()
     report_text = report.render(cfg, Report.Format.MARKDOWN)

ai_code_review/report_struct.py

@@ -60,6 +60,7 @@ class Format(StrEnum):
 
     issues: dict = field(default_factory=dict)
     summary: str = field(default="")
+    number_of_processed_files: int = field(default=0)
     total_issues: int = field(init=False)
     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)

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "ai-code-review"
-version = "0.4.0"
+version = "0.4.1"
 description = "LLM-agnostic GitHub AI Code Review Tool with integration to GitHub actions"
 authors = ["Nayjest <mail@vitaliy.in>"]
 readme = "README.md"

tests/test_report_struct.py

@@ -58,6 +58,7 @@ def test_report_save_load(tmp_path):
             ]
         },
         "summary": "SUMMARY",
+        "number_of_processed_files": 2,
     }
     file_name = tmp_path / "report.json"
     report = Report(**data)
@@ -67,9 +68,11 @@ def test_report_save_load(tmp_path):
     with open(file_name, "r") as f:
         loaded = json.load(f)
     assert loaded["summary"] == "SUMMARY"
+    assert loaded["number_of_processed_files"] == 2
     # test reload with .load
     loaded_report = Report.load(file_name)
     assert loaded_report.summary == "SUMMARY"
+    assert loaded_report.number_of_processed_files == 2
     assert loaded_report.total_issues == 1
     assert loaded_report.issues["file.py"][0].title == "Bug"