[80] Improve episode match algorithm

Only compare defined InterfaceID objects, to prevent blank ID comparisons causing a failure to match

Author: CollinHeist

backend/.version

@@ -1 +1 @@
-v2.16.0-webui79
+v2.16.0-webui80

backend/app/info/episode.py

@@ -155,7 +155,7 @@ def __repr__(self) -> str:
         abs_string = (
             ''
             if self.absolute_number is None
-            else '_A' + str(self.absolute_number)
+            else '-A' + str(self.absolute_number)
         )
 
         return (
@@ -238,23 +238,41 @@ def __eq__(self, other: 'EpisodeInfo | PlexEpisode | object') -> bool:
             (self.tvdb_id, other.tvdb_id),
             (self.tvrage_id, other.tvrage_id),
         ]:
+            # Interface ID comparison
             if (isinstance(this_id, InterfaceID)
-                and isinstance(other_id, InterfaceID)):
+                and isinstance(other_id, InterfaceID)
+            ):
+                # If neither ID is actually defined, skip this
+                if not this_id or not other_id:
+                    continue
                 if this_id.equals(other_id):
                     id_matches += 1
+                # Do not increment mismatch because interface IDs are
+                # not strictly comparable
+                continue
+
+            # Regular ID comparison - if this ID is defined and matches,
+            # count it
             if this_id is not None and this_id == other_id:
                 id_matches += 1
-            elif (this_id is not None and other_id is not None
-                and this_id != other_id):
+            # Both IDs are defined but do not match, count as mismatch
+            elif (this_id is not None
+                and other_id is not None
+                and this_id != other_id
+            ):
                 id_mismatches += 1
 
-        # More than one ID match is equality, more than one mismatch is
-        # an inequality
-        if id_matches > 1:
-            return True
-        if (id_matches == 1
-            and self.season_number == other.season_number
-            and self.episode_number == other.episode_number):
+        # More than one ID match is an equality, one match and index
+        # match is equality, and more than one ID mismatch is not a 
+        # match
+        if (
+            (id_matches > 1 and id_matches > id_mismatches)
+            or (
+                id_matches == 1
+                and self.season_number == other.season_number
+                and self.episode_number == other.episode_number
+            )
+        ):
             return True
         if id_mismatches > 1:
             return False

backend/app/info/util.py

@@ -155,8 +155,9 @@ def match_episode_infos(
 
     ref_infos = [_to_episode_info(ref) for ref in references]
     match_lists: list[list[_SearchT]] = [[] for _ in references]
-    # Parallel lists tracking the normalized EpisodeInfo for each match/unmatch
-    # entry so they can be rendered in the summary table without re-normalizing.
+    # Parallel lists tracking the normalized EpisodeInfo for each
+    # match/unmatch entry so they can be rendered in the summary table
+    # without re-normalizing
     match_infos: list[list[EpisodeInfo]] = [[] for _ in references]
     unmatched: list[_SearchT] = []
     unmatched_infos: list[EpisodeInfo] = []