refactor: enable no-unsafe-call rule (@Miodec) (#5883)

### Description

<!-- Please describe the change(s) made in your PR -->

### Checks

- [ ] Adding quotes?
- [ ] Make sure to include translations for the quotes in the
description (or another comment) so we can verify their content.
- [ ] Adding a language or a theme?
- [ ] If is a language, did you edit `_list.json`, `_groups.json` and
add `languages.json`?
  - [ ] If is a theme, did you add the theme.css?
- Also please add a screenshot of the theme, it would be extra awesome
if you do so!
- [ ] Check if any open issues are related to this PR; if so, be sure to
tag them below.
- [ ] Make sure the PR title follows the Conventional Commits standard.
(https://www.conventionalcommits.org for more info)
- [ ] Make sure to include your GitHub username prefixed with @ inside
parentheses at the end of the PR title.

<!-- label(optional scope): pull request title (@your_github_username)
-->

<!-- I know I know they seem boring but please do them, they help us and
you will find out it also helps you.-->

Closes #

<!-- the issue(s) your PR resolves if any (delete if that is not the
case) -->
<!-- please also reference any issues and or PRs related to your pull
request -->
<!-- Also remove it if you are not following any issues. -->

<!-- pro tip: you can mention an issue, PR, or discussion on GitHub by
referencing its hash number e.g:
[#1234](#1234) -->

<!-- pro tip: you can press . (dot or period) in the code tab of any
GitHub repo to get access to GitHub's VS Code web editor Enjoy! :) -->

Author: Miodec

backend/src/api/controllers/dev.ts

@@ -241,7 +241,7 @@ async function updateUser(uid: string): Promise<void> {
       timestamp: best.timestamp,
     } as PersonalBest;
 
-    personalBests[mode.mode][mode.mode2].push(entry);
+    (personalBests[mode.mode][mode.mode2] as PersonalBest[]).push(entry);
 
     if (mode.mode === "time") {
       if (lbPersonalBests[mode.mode][mode.mode2] === undefined)

backend/src/dal/new-quotes.ts

@@ -1,4 +1,4 @@
-import { simpleGit } from "simple-git";
+import { SimpleGit, simpleGit } from "simple-git";
 import { Collection, ObjectId } from "mongodb";
 import path from "path";
 import { existsSync, writeFileSync } from "fs";
@@ -24,7 +24,7 @@ type QuoteData = {
 
 const PATH_TO_REPO = "../../../../monkeytype-new-quotes";
 
-let git;
+let git: SimpleGit | undefined;
 try {
   git = simpleGit(path.join(__dirname, PATH_TO_REPO));
 } catch (e) {

backend/src/services/weekly-xp-leaderboard.ts

@@ -96,7 +96,8 @@ export class WeeklyXpLeaderboard {
       timeTypedSeconds + (currentEntryTimeTypedSeconds ?? 0);
 
     const [rank] = await Promise.all([
-      // @ts-expect-error
+      // @ts-expect-error we are doing some weird file to function mapping, thats why its any
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-call
       connection.addResultIncrement(
         2,
         weeklyXpLeaderboardScoresKey,
@@ -128,7 +129,8 @@ export class WeeklyXpLeaderboard {
     const { weeklyXpLeaderboardScoresKey, weeklyXpLeaderboardResultsKey } =
       this.getThisWeeksXpLeaderboardKeys();
 
-    // @ts-expect-error
+    // @ts-expect-error we are doing some weird file to function mapping, thats why its any
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call
     const [results, scores] = (await connection.getResults(
       2, // How many of the arguments are redis keys (https://redis.io/docs/manual/programmability/lua-api/)
       weeklyXpLeaderboardScoresKey,

backend/src/utils/daily-leaderboards.ts

@@ -74,7 +74,8 @@ export class DailyLeaderboard {
 
     const resultScore = kogascore(entry.wpm, entry.acc, entry.timestamp);
 
-    // @ts-expect-error
+    // @ts-expect-error we are doing some weird file to function mapping, thats why its any
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call
     const rank = (await connection.addResult(
       2,
       leaderboardScoresKey,
@@ -120,7 +121,8 @@ export class DailyLeaderboard {
     const { leaderboardScoresKey, leaderboardResultsKey } =
       this.getTodaysLeaderboardKeys();
 
-    // @ts-expect-error
+    // @ts-expect-error we are doing some weird file to function mapping, thats why its any
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call
     const [results] = (await connection.getResults(
       2,
       leaderboardScoresKey,
@@ -213,7 +215,8 @@ export async function purgeUserFromDailyLeaderboards(
     return;
   }
 
-  // @ts-expect-error
+  // @ts-expect-error we are doing some weird file to function mapping, thats why its any
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-call
   await connection.purgeResults(0, uid, dailyLeaderboardNamespace);
 }
 

backend/src/utils/pb.ts

@@ -55,7 +55,7 @@ export function checkAndUpdatePb(
     const didUpdate = updatePersonalBest(personalBestMatch, result);
     isPb = didUpdate;
   } else {
-    userPb[mode][mode2].push(buildPersonalBest(result));
+    (userPb[mode][mode2] as PersonalBest[]).push(buildPersonalBest(result));
   }
 
   if (!_.isNil(lbPersonalBests)) {
@@ -186,15 +186,17 @@ export function updateLeaderboardPersonalBests(
   lbPb[mode] ??= {};
   lbPb[mode][mode2] ??= {};
   const bestForEveryLanguage = {};
-  userPersonalBests[mode][mode2].forEach((pb: PersonalBest) => {
-    const language = pb.language;
-    if (
-      bestForEveryLanguage[language] === undefined ||
-      bestForEveryLanguage[language].wpm < pb.wpm
-    ) {
-      bestForEveryLanguage[language] = pb;
+  (userPersonalBests[mode][mode2] as PersonalBest[]).forEach(
+    (pb: PersonalBest) => {
+      const language = pb.language;
+      if (
+        bestForEveryLanguage[language] === undefined ||
+        bestForEveryLanguage[language].wpm < pb.wpm
+      ) {
+        bestForEveryLanguage[language] = pb;
+      }
     }
-  });
+  );
   _.each(bestForEveryLanguage, (pb: PersonalBest, language: string) => {
     const languageDoesNotExist = lbPb[mode][mode2][language] === undefined;
     const languageIsEmpty = _.isEmpty(lbPb[mode][mode2][language]);

frontend/src/ts/controllers/ad-controller.ts

@@ -175,6 +175,7 @@ export async function checkCookieblocker(): Promise<void> {
         return;
       }
       //@ts-expect-error
+      // eslint-disable-next-line @typescript-eslint/no-unsafe-call
       window.__tcfapi("getTCData", 2, (tcData, success) => {
         if (success as boolean) {
           if (tcData.eventStatus === "cmpuishown") {
@@ -268,11 +269,13 @@ export function updateFooterAndVerticalAds(visible: boolean): void {
 export function showConsentPopup(): void {
   if (choice === "eg") {
     //@ts-expect-error
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call
     window.__tcfapi("displayConsentUi", 2, function () {
       //
     });
   } else {
     //@ts-expect-error
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call
     ramp.showCmpModal();
   }
 }

frontend/src/ts/controllers/captcha-controller.ts

@@ -13,7 +13,7 @@ export function render(
   }
 
   //@ts-expect-error
-  // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-call
   const widgetId = grecaptcha.render(element, {
     sitekey: siteKey,
     callback,
@@ -28,6 +28,7 @@ export function reset(id: string): void {
   }
 
   //@ts-expect-error
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-call
   grecaptcha.reset(captchas[id]);
 }
 
@@ -37,6 +38,6 @@ export function getResponse(id: string): string {
   }
 
   //@ts-expect-error
-  // eslint-disable-next-line @typescript-eslint/no-unsafe-return
+  // eslint-disable-next-line @typescript-eslint/no-unsafe-return, @typescript-eslint/no-unsafe-call
   return grecaptcha.getResponse(captchas[id]);
 }

frontend/src/ts/controllers/eg-ad-controller.ts

@@ -15,6 +15,7 @@ export function init(): void {
 export function renderResult(widerThanBreakpoint: boolean): void {
   if (widerThanBreakpoint) {
     //@ts-expect-error
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call
     window.egAps.render([
       "ad-result",
       "ad-vertical-left",
@@ -23,6 +24,7 @@ export function renderResult(widerThanBreakpoint: boolean): void {
     ]);
   } else {
     //@ts-expect-error
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call
     window.egAps.render([
       "ad-result-small",
       "ad-vertical-left",
@@ -35,6 +37,7 @@ export function renderResult(widerThanBreakpoint: boolean): void {
 export function reinstate(): boolean {
   try {
     //@ts-expect-error
+    // eslint-disable-next-line @typescript-eslint/no-unsafe-call
     window.egAps.reinstate();
     return true;
   } catch (e) {

frontend/src/ts/controllers/pw-ad-controller.ts

@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-unsafe-call */
 //@ts-nocheck
 
 import Config from "../config";

frontend/src/ts/controllers/sound-controller.ts

@@ -412,10 +412,8 @@ export async function previewError(val: string): Promise<void> {
   const errorSoundIds = Object.keys(safeErrorSounds);
   if (!errorSoundIds.includes(val)) return;
 
-  //@ts-expect-error
-  errorClickSounds[val][0].sounds[0].seek(0);
-  //@ts-expect-error
-  errorClickSounds[val][0].sounds[0].play();
+  errorSounds?.[val]?.[0]?.sounds[0]?.seek(0);
+  errorSounds?.[val]?.[0]?.sounds[0]?.play();
 }
 
 let currentCode = "KeyA";