Remove V2 finding compatibility for FixResult

Author: drdavella

src/codemodder/codetf/v3/codetf.py

@@ -8,7 +8,6 @@
 from ..common import Change, CodeTFWriter, Finding, FixQuality
 from ..v2.codetf import AIMetadata as AIMetadatav2
 from ..v2.codetf import CodeTF as CodeTFv2
-from ..v2.codetf import Finding as V2Finding
 from ..v2.codetf import Result
 from ..v2.codetf import Run as Runv2
 
@@ -99,7 +98,7 @@ class FixMetadata(BaseModel):
 class FixResult(BaseModel):
     """Result corresponding to a single finding"""
 
-    finding: Finding | V2Finding
+    finding: Finding
     fixStatus: FixStatus
     changeSets: list[ChangeSet] = []
     fixMetadata: Optional[FixMetadata] = None
@@ -174,7 +173,7 @@ def from_v2_result(result: Result) -> list[FixResult]:
                 )
                 fix_results.append(
                     FixResult(
-                        finding=f,
+                        finding=Finding(**f.model_dump()),
                         fixStatus=FixStatus(status=FixStatusType.fixed),
                         changeSets=[changeset],
                         fixMetadata=fix_metadata,
@@ -185,7 +184,7 @@ def from_v2_result(result: Result) -> list[FixResult]:
     for f in result.unfixedFindings or []:
         fix_results.append(
             FixResult(
-                finding=f,
+                finding=Finding(**f.model_dump()),
                 fixStatus=FixStatus(status=FixStatusType.failed, reason=f.reason),
             )
         )

tests/test_codetf.py

@@ -274,13 +274,13 @@ def test_v2_to_v3_conversion():
         # correctly associates findings to the change
         assert f.changeSets and f.changeSets[0].path == v2changeset.path
         assert f.changeSets and f.changeSets[0].diff == v2changeset.diff
-        assert isinstance(f.finding, Finding) and f.changeSets[0].changes == [
-            v2_finding_to_change[f.finding].to_common()
+        assert isinstance(f.finding, FindingV3) and f.changeSets[0].changes == [
+            v2_finding_to_change[Finding(**f.finding.model_dump())].to_common()
         ]
 
     # unfixed metadata
     assert (
         unfixed[0].fixStatus.reason
         and unfixed[0].fixStatus.reason == v2_unfixed[0].reason
     )
-    assert unfixed[0].finding == v2_unfixed[0]
+    assert unfixed[0].finding == FindingV3(**v2_unfixed[0].model_dump())