Remove now-unecessary eslint ignore rules

src/index.ts

@@ -109,10 +109,8 @@ async function initialise() {
 
         const jwt = await authenticationService.verifyToken(sanitisedToken);
 
-        /* eslint-disable @typescript-eslint/no-explicit-any */
         const email = (jwt as any)[`${config.CLIENT_URL}/email`] as string;
         const name = (jwt as any)[`${config.CLIENT_URL}/name`] as string | undefined;
-        /* eslint-enable @typescript-eslint/no-explicit-any */
 
         const { roles, id } = await userService.loadUserDetailsAndUpdateIfNecessary(email, name);
 

src/quiz/quiz.service.ts

@@ -60,9 +60,7 @@ export class QuizService {
     const quiz = await this.#persistence.getQuizByIdWithResults({
       id,
     });
-    // TODO look into modifying the upstream https://eslint.org/docs/latest/rules/no-unused-vars#ignorerestsiblings linting rule
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
-    const { images, completions, uploadedByUser, uploadedByUserId, ...quizFieldsThatDoNotRequireTransform } = quiz;
+    const { images, completions, uploadedByUser, ...quizFieldsThatDoNotRequireTransform } = quiz;
     return {
       ...quizFieldsThatDoNotRequireTransform,
       completions: completions.map((entry) => this.#quizCompletionPersistenceToQuizCompletion(entry)),
@@ -299,9 +297,7 @@ export class QuizService {
       })[];
     },
   ): Quiz {
-    // TODO modify linting tules to allow unused vars in destructuring
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
-    const { completions, uploadedByUser, uploadedByUserId, ...quizWithoutImageKey } = quiz;
+    const { completions, uploadedByUser, ...quizWithoutImageKey } = quiz;
     return {
       ...quizWithoutImageKey,
       myCompletions: completions.map((entry) => this.#quizCompletionPersistenceToQuizCompletion(entry)),

src/util/cache.ts

@@ -36,7 +36,6 @@ export interface Cache {
  * Will be replaced with a Redis implementation in the future.
  */
 export class MemoryCache implements Cache {
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
   #values: Map<string, { value: any; expiresAt?: Date }> = new Map();
   getItem<T>(key: string): Promise<T | undefined> {
     const record = this.#values.get(key);