fix(gacha/search-index): search-index was generating the wrong charac…

…ter rating in community packs pools (#412)

search-index/mod.ts

@@ -144,20 +144,23 @@ const pool = async (
 
         if (!media) return undefined;
 
-        const popularity = char.popularity ?? media.node.popularity ?? 1000;
-
         const role = media.role;
 
         if (!role) return undefined;
 
-        const rating = new Rating({ role, popularity }).stars;
+        const rating = new Rating(
+          char.popularity ? { popularity: char.popularity } : {
+            role,
+            popularity: media.node.popularity ?? 1000,
+          },
+        ).stars;
 
         return new IndexedCharacter(
           `${char.packId}:${char.id}`,
           `${media.node.packId}:${media.node.id}`,
           name,
           packs.aliasToArray(media.node.title),
-          popularity,
+          char.popularity ?? media.node.popularity ?? 1000,
           rating,
           role,
         );

src/gacha.ts

@@ -184,26 +184,24 @@ export async function guaranteedPool(
   );
 
   const validate = (character: Character | DisaggregatedCharacter): boolean => {
-    if (
-      typeof character.popularity === 'number' &&
-      new Rating({ popularity: character.popularity }).stars !== guarantee
-    ) {
-      return false;
-    }
-
     // deno-lint-ignore no-non-null-assertion
     const edge = character.media && 'edges' in character.media! &&
       character.media.edges[0];
 
-    if (edge) {
+    if (
+      typeof character.popularity === 'number' &&
+      new Rating({ popularity: character.popularity }).stars === guarantee
+    ) {
+      return true;
+    } else if (edge) {
       const popularity = character.popularity || edge.node.popularity || lowest;
 
-      if (new Rating({ popularity, role: edge.role }).stars !== guarantee) {
-        return false;
+      if (new Rating({ popularity, role: edge.role }).stars === guarantee) {
+        return true;
       }
     }
 
-    return true;
+    return false;
   };
 
   return {