bug: fix last signed in at cache (#184)

Author: lukepolo

src/auth/auth.controller.ts

@@ -73,7 +73,7 @@ export class AuthController {
           __typename: true,
         },
       });
-      await this.cache.put(lastSignInKey, true, 60 * 60 * 1000);
+      await this.cache.put(lastSignInKey, true, 60 * 60);
       user.last_sign_in_at = now;
     }
 

src/hasura/hasura.service.ts

@@ -41,7 +41,7 @@ export class HasuraService {
   }
 
   public static PLAYER_LAST_SIGN_IN_CACHE_KEY(steamId: bigint | string) {
-    return `user:last_sign_in_at:${steamId.toString()}`;
+    return `user:last_sign_in_at:v2:${steamId.toString()}`;
   }
 
   public checkSecret(secret: string) {

src/type-sense/type-sense.service.ts

@@ -94,6 +94,8 @@ export class TypeSenseService {
         type: "string",
         symbols_to_index: ["~"],
         optional: true,
+        sort: true,
+        index: true,
       },
     ];
 
@@ -112,18 +114,37 @@ export class TypeSenseService {
 
     const collection = await this.client.collections("players").retrieve();
 
-    const missingFields = fields.filter((field) => {
-      return !collection.fields.find((_existingField) => {
-        return _existingField.name === field.name;
-      });
-    });
+    const fieldUpdates: Array<Record<string, unknown>> = [];
+    let needsRefresh = false;
+
+    for (const field of fields) {
+      const existing = collection.fields.find((f) => f.name === field.name);
+
+      if (!existing) {
+        fieldUpdates.push(field);
+        needsRefresh = true;
+        continue;
+      }
 
-    if (missingFields.length > 0) {
+      const sortChanged = Boolean(existing.sort) !== Boolean(field.sort);
+      const indexChanged = Boolean(existing.index) !== Boolean(field.index);
+      const typeChanged = existing.type !== field.type;
+
+      if (sortChanged || indexChanged || typeChanged) {
+        fieldUpdates.push({ name: field.name, drop: true });
+        fieldUpdates.push(field);
+        needsRefresh = true;
+      }
+    }
+
+    if (fieldUpdates.length > 0) {
       await this.client.collections("players").update({
-        fields: missingFields,
+        fields: fieldUpdates as CollectionFieldSchema[],
       });
 
-      await this.queue.add(RefreshAllPlayersJob.name, {});
+      if (needsRefresh) {
+        await this.queue.add(RefreshAllPlayersJob.name, {});
+      }
     }
   }