Merge 045c865904 into 0ec9f3535b

lib/services/sourced_track/sources/youtube.dart

@@ -242,30 +242,28 @@ class YoutubeSourcedTrack extends SourcedTrack {
   }) async {
     List<SiblingType> siblings = [];
 
-    final isrc = track.externalIds?.isrc;
+    final isrc = track.externalIds?.isrc.toString();
     if (isrc != null && isrc.isNotEmpty) {
       final isrcResults =
-          await ref.read(youtubeEngineProvider).searchVideos(isrc.toString());
+          await ref.read(youtubeEngineProvider).searchVideos(isrc);
       if (isrcResults.isNotEmpty) {
         final rankedResults = rankResults(
             isrcResults.map(YoutubeVideoInfo.fromVideo).toList(), track);
         final matchingResults = <YoutubeVideoInfo>[];
         for (final video in rankedResults) {
           final titleWords = video.title
+              .replaceAll(RegExp(r'\((.*)\)'), '')
               .toLowerCase()
-              .replaceAll(RegExp(r'[^a-zA-Z0-9\s]+'), '')
               .split(RegExp(r'\s+'))
-              .where((item) => item.isNotEmpty)
               .toList();
           final nameLower = track.name!
+              .replaceAll(RegExp(r'\((.*)\)'), '')
               .toLowerCase()
-              .replaceAll(RegExp(r'[^a-zA-Z0-9\s]+'), '')
               .split(RegExp(r'\s+'))
-              .where((item) => item.isNotEmpty)
               .toList();
           final matchCount =
               titleWords.where((word) => nameLower.contains(word)).length;
-          if (matchCount > nameLower.length / 2) {
+          if (matchCount >= nameLower.length) {
             matchingResults.add(video);
           }
         }
@@ -324,16 +322,18 @@ class YoutubeSourcedTrack extends SourcedTrack {
             .mapIndexed((index, info) => toSiblingType(index, info, ref)),
       ));
     }
-
+    return await (() async {
-    final seenIds = <String>{};
+      // Deduplicate siblings by info.id, keeping the first occurrence
-    // Deduplicate siblings by info.id, keeping the first occurrence
+      final seenIds = <String>{};
-    return await Future.wait(siblings.map((sibling) async {
+      final uniqueSiblings = <SiblingType>[];
-      if (!seenIds.contains(sibling.info.id)) {
+      for (final sibling in siblings) {
-        seenIds.add(sibling.info.id);
+        if (!seenIds.contains(sibling.info.id)) {
-        return sibling;
+          seenIds.add(sibling.info.id);
+          uniqueSiblings.add(sibling);
+        }
       }
-      return null;
+      return uniqueSiblings;
-    })).then((s) => s.whereType<SiblingType>().toList());
+    })();
   }
 
   @override