fix: resolve CI issues - push complete implementation with tests

src/gdiffer/models.py

@@ -1,37 +1,31 @@
-"""Data models for git diff parsing and analysis."""
-
 from dataclasses import dataclass, field
 
 
 @dataclass
 class DiffHunk:
-    """Represents a single hunk (chunk) of changes in a diff."""
     old_start: int
     old_lines: int
     new_start: int
     new_lines: int
-    old_lines_content: list[str] = field(default_factory=list)
-    new_lines_content: list[str] = field(default_factory=list)
+    old_lines_content: list = field(default_factory=list)
+    new_lines_content: list = field(default_factory=list)
     header: str = ""
 
-    def get_added_lines(self) -> list[tuple[int, str]]:
-        """Return tuples of (line_number, content) for added lines."""
+    def get_added_lines(self):
         result = []
         for i, line in enumerate(self.new_lines_content):
-            if line.startswith('+') and not line.startswith('+++'):
+            if line.startswith("+") and not line.startswith("+++"):
                 result.append((self.new_start + i, line[1:]))
         return result
 
-    def get_removed_lines(self) -> list[tuple[int, str]]:
-        """Return tuples of (line_number, content) for removed lines."""
+    def get_removed_lines(self):
         result = []
         for i, line in enumerate(self.old_lines_content):
-            if line.startswith('-') and not line.startswith('---'):
+            if line.startswith("-") and not line.startswith("---"):
                 result.append((self.old_start + i, line[1:]))
         return result
 
-    def get_modified_lines(self) -> list[tuple[int, str, str]]:
-        """Return tuples of (old_line, old_content, new_content) for modified lines."""
+    def get_modified_lines(self):
         result = []
         added = self.get_added_lines()
         removed = self.get_removed_lines()
@@ -45,71 +39,63 @@ class DiffHunk:
 
 @dataclass
 class DiffFile:
-    """Represents a file in the diff with its changes."""
-    old_path: str | None
-    new_path: str | None
-    new_file_mode: str | None = None
-    deleted_file_mode: str | None = None
-    similarity_index: str | None = None
-    rename_from: str | None = None
-    rename_to: str | None = None
-    hunks: list[DiffHunk] = field(default_factory=list)
+    old_path: str
+    new_path: str
+    new_file_mode: str = None
+    deleted_file_mode: str = None
+    similarity_index: str = None
+    rename_from: str = None
+    rename_to: str = None
+    hunks: list = field(default_factory=list)
     change_type: str = "modify"
 
     @property
-    def filename(self) -> str:
-        """Get the current filename."""
+    def filename(self):
         if self.new_path:
             return self.new_path
         return self.old_path or ""
 
     @property
-    def is_new(self) -> bool:
-        """Check if this is a new file."""
+    def is_new(self):
         return self.new_file_mode is not None or self.old_path in [None, "/dev/null"]
 
     @property
-    def is_deleted(self) -> bool:
-        """Check if this file was deleted."""
+    def is_deleted(self):
         return self.deleted_file_mode is not None
 
     @property
-    def is_rename(self) -> bool:
-        """Check if this file was renamed."""
+    def is_rename(self):
         return self.rename_from is not None
 
     @property
-    def extension(self) -> str:
-        """Get the file extension."""
+    def extension(self):
         filename = self.filename
-        if '.' in filename:
-            return filename.rsplit('.', 1)[-1].lower()
+        if "." in filename:
+            return filename.rsplit(".", 1)[-1].lower()
         return ""
 
 
 @dataclass
 class CodeChange:
-    """Represents a code change with context."""
     file: DiffFile
-    hunk: DiffHunk | None
+    hunk: DiffHunk
     old_code: str
     new_code: str
     language: str = "unknown"
     summary: str = ""
-    issues: list[dict] = field(default_factory=list)
-    suggestions: list[str] = field(default_factory=list)
+    issues: list = field(default_factory=list)
+    suggestions: list = field(default_factory=list)
 
 
 @dataclass
 class DiffAnalysis:
-    """Complete analysis result for a diff."""
-    files: list[DiffFile] = field(default_factory=list)
+    files: list = field(default_factory=list)
     total_files: int = 0
     files_added: int = 0
     files_deleted: int = 0
     files_modified: int = 0
     files_renamed: int = 0
     total_changes: int = 0
-    language_breakdown: dict[str, int] = field(default_factory=dict)
-    all_issues: list[dict] = field(default_factory=list)
-    all_suggestions: list[str] = field(default_factory=list)
+    language_breakdown: dict = field(default_factory=dict)
+    all_issues: list = field(default_factory=list)
+    all_suggestions: list = field(default_factory=list)