fix: migrate lastReultHashes typo to lastResultHashes safely

Author: mukundangopalachary

backend/src/api/controllers/result.ts

@@ -402,7 +402,7 @@ export async function addResult(
   }
 
   if (req.ctx.configuration.users.lastHashesCheck.enabled) {
-    let lastHashes = user.lastReultHashes ?? [];
+    let lastHashes = user.lastResultHashes ?? user.lastReultHashes ?? [];
     if (lastHashes.includes(resulthash)) {
       void addLog(
         "duplicate_result",

backend/src/api/controllers/user.ts

@@ -509,7 +509,8 @@ type RelevantUserInfo = Omit<
   | "nameHistory"
   | "lastNameChange"
   | "_id"
-  | "lastReultHashes" //TODO fix typo
+  | "lastResultHashes"
+  | "lastReultHashes"
   | "note"
   | "ips"
   | "testActivity"
@@ -524,7 +525,8 @@ function getRelevantUserInfo(user: UserDAL.DBUser): RelevantUserInfo {
     "nameHistory",
     "lastNameChange",
     "_id",
-    "lastReultHashes", //TODO fix typo
+    "lastResultHashes",
+    "lastReultHashes",
     "note",
     "ips",
     "testActivity",

backend/src/dal/user.ts

@@ -589,7 +589,10 @@ export async function updateLastHashes(
     { uid },
     {
       $set: {
-        lastReultHashes: lastHashes, //TODO fix typo
+        lastResultHashes: lastHashes,
+      },
+      $unset: {
+        lastReultHashes: 1,
       },
     },
   );

packages/schemas/src/users.ts

@@ -243,7 +243,8 @@ export const UserSchema = z.object({
   uid: z.string(), //defined by firebase, no validation should be applied
   addedAt: z.number().int().nonnegative(),
   personalBests: PersonalBestsSchema,
-  lastReultHashes: z.array(z.string()).optional(), //TODO: fix typo (it's in the db too)
+  lastResultHashes: z.array(z.string()).optional(),
+  lastReultHashes: z.array(z.string()).optional(), //legacy typo, kept for backwards compatibility
   completedTests: z.number().int().nonnegative().optional(),
   startedTests: z.number().int().nonnegative().optional(),
   timeTyping: z