refactor: apply tsconfig strict true

src/config/env.ts

@@ -14,6 +14,8 @@ const once = (fn) => {
   return function (...args) {
     if (called) return;
     called = true;
+    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+    // @ts-ignore
     return fn.apply(this, args);
   };
 };

src/services/auth/plugins/captcha/captcha.ts

@@ -25,6 +25,8 @@ export default function captchaPreHandler(
   action: RecaptchaActionType,
   options?: { shouldFail: boolean },
 ): RouteHandlerMethod {
+  // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+  // @ts-ignore
   return async (request: FastifyRequest<{ Body: { captcha: string } }>, _reply: FastifyReply) => {
     const { captcha } = request.body;
     return await validateCaptcha(request, captcha, action, options);

src/services/auth/plugins/magicLink/magicLink.controller.ts

@@ -82,6 +82,8 @@ const plugin: FastifyPluginAsyncTypebox = async (fastify) => {
       schema: auth,
       preHandler: fastifyPassport.authenticate(
         PassportStrategy.WebMagicLink,
+        // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+        // @ts-ignore
         async (request, reply, err, user?: PassportUser, info?: PassportInfo) => {
           // This function is called after the strategy has been executed.
           // It is necessary, so we match the behavior of the original implementation.

src/services/auth/plugins/passport/preHandlers.ts

@@ -79,6 +79,8 @@ export const guestAuthenticateAppsJWT = fastifyPassport.authenticate(
 export function matchOne<R extends RouteGenericInterface>(
   ...strategies: RessourceAuthorizationStrategy<R>[]
 ): RouteHandlerMethod {
+  // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+  // @ts-expect-error
   return async (req: FastifyRequest<R>) => {
     if (!strategies.some((strategy) => strategy.test(req))) {
       // If none of the strategies pass, throw an error.

src/services/auth/plugins/passport/strategies/emailChange.ts

@@ -4,7 +4,7 @@ import { Authenticator } from '@fastify/passport';
 
 import { EMAIL_CHANGE_JWT_SECRET } from '../../../../../config/secrets';
 import { db } from '../../../../../drizzle/db';
-import { MemberNotFound, UnauthorizedMember } from '../../../../../utils/errors';
+import { MemberNotFound, UnauthorizedMember, buildError } from '../../../../../utils/errors';
 import { MemberRepository } from '../../../../member/member.repository';
 import { PassportStrategy } from '../strategies';
 import type { CustomStrategyOptions, StrictVerifiedCallback } from '../types';
@@ -47,9 +47,12 @@ export default (
               false,
             );
           }
-        } catch (err) {
+        } catch (error: unknown) {
           // Exception occurred while fetching member
-          return done(options?.propagateError ? err : new UnauthorizedMember(), false);
+          return done(
+            options?.propagateError ? buildError(error) : new UnauthorizedMember(),
+            false,
+          );
         }
       },
     ),

src/services/auth/plugins/passport/strategies/jwtApps.ts

@@ -4,7 +4,7 @@ import { Authenticator } from '@fastify/passport';
 
 import { APPS_JWT_SECRET } from '../../../../../config/secrets';
 import { db } from '../../../../../drizzle/db';
-import { UnauthorizedMember } from '../../../../../utils/errors';
+import { UnauthorizedMember, buildError } from '../../../../../utils/errors';
 import { AccountRepository } from '../../../../account/account.repository';
 import { ItemRepository } from '../../../../item/item.repository';
 import { PassportStrategy } from '../strategies';
@@ -53,11 +53,14 @@ export default (
               key,
             },
           });
-        } catch (err) {
+        } catch (error: unknown) {
           // Exception occurred while fetching item
           // itemRepository.getOneOrThrow() can fail for many reasons like the item was not found, database error, etc.
           // To avoid leaking information, we prefer to return UnauthorizedMember error.
-          return done(options?.propagateError ? err : new UnauthorizedMember(), false);
+          return done(
+            options?.propagateError ? buildError(error) : new UnauthorizedMember(),
+            false,
+          );
         }
       },
     ),

src/services/auth/plugins/passport/strategies/jwtChallengeVerifier.ts

@@ -5,7 +5,12 @@ import { Authenticator } from '@fastify/passport';
 
 import { JWT_SECRET } from '../../../../../config/secrets';
 import { db } from '../../../../../drizzle/db';
-import { ChallengeFailed, MemberNotFound, UnauthorizedMember } from '../../../../../utils/errors';
+import {
+  ChallengeFailed,
+  MemberNotFound,
+  UnauthorizedMember,
+  buildError,
+} from '../../../../../utils/errors';
 import { AccountRepository } from '../../../../account/account.repository';
 import { SHORT_TOKEN_PARAM } from '../constants';
 import { PassportStrategy } from '../strategies';
@@ -57,9 +62,9 @@ export default (
               false,
             );
           }
-        } catch (err) {
+        } catch (error: unknown) {
           // Exception occurred while fetching member
-          return done(spreadException ? err : new UnauthorizedMember(), false);
+          return done(spreadException ? buildError(error) : new UnauthorizedMember(), false);
         }
       },
     ),

src/services/file/file.service.test.ts

@@ -115,8 +115,8 @@ describe('FileService', () => {
     it('upload failure will delete file', async () => {
       const uploadFileMock = jest.spyOn(s3Repository, 'uploadFiles').mockRejectedValue('error');
       const deleteFileMock = jest.spyOn(s3Repository, 'deleteFiles').mockImplementation(doNothing);
-      await expect(s3FileService.upload(member, uploadPayload)).rejects.toMatchObject(
-        new UploadFileUnexpectedError(expect.anything()),
+      await expect(s3FileService.upload(member, uploadPayload)).rejects.toThrow(
+        new UploadFileUnexpectedError({ memberId: member.id }),
       );
       expect(uploadFileMock).toHaveBeenCalled();
       expect(deleteFileMock).toHaveBeenCalled();
@@ -276,19 +276,19 @@ describe('FileService', () => {
       const copyMock = jest
         .spyOn(s3Repository, 'copyFile')
         .mockImplementation(async () => 'string');
-      await s3FileService.copy(member, copyPayload);
+      await s3FileService.copy(member.id, copyPayload);
       expect(copyMock).toHaveBeenCalled();
     });
 
     it('empty originalPath throws', async () => {
       await expect(
-        s3FileService.copy(member, { ...copyPayload, originalPath: '' }),
+        s3FileService.copy(member.id, { ...copyPayload, originalPath: '' }),
       ).rejects.toMatchObject(new CopyFileInvalidPathError(expect.anything()));
     });
 
     it('empty newFilePath throws', async () => {
       await expect(
-        s3FileService.copy(member, { ...copyPayload, newFilePath: '' }),
+        s3FileService.copy(member.id, { ...copyPayload, newFilePath: '' }),
       ).rejects.toMatchObject(new CopyFileInvalidPathError(expect.anything()));
     });
   });

src/services/file/file.service.ts

@@ -161,7 +161,7 @@ class FileService {
   }
 
   async copy(
-    member: MinimalMember,
+    memberId: MinimalMember['id'],
     data: {
       newId?: string;
       newFilePath: string;
@@ -180,7 +180,7 @@ class FileService {
 
     return this.repository.copyFile({
       newId,
-      memberId: member.id,
+      memberId,
       originalPath,
       newFilePath,
       mimetype,

src/services/file/repositories/local.ts

@@ -69,7 +69,7 @@ export class LocalFileRepository implements FileRepository {
     try {
       await access(this.buildFullPath(filepath));
     } catch (e) {
-      if (e.code === 'ENOENT') {
+      if (e !== null && typeof e === 'object' && 'code' in e && e.code === 'ENOENT') {
         throw new LocalFileNotFound({ filepath });
       }
       throw e;

src/services/file/utils/errors.ts

@@ -1,5 +1,7 @@
 import { StatusCodes } from 'http-status-codes';
 
+import { createError } from '@fastify/error';
+
 import { ErrorFactory, FAILURE_MESSAGES } from '@graasp/sdk';
 
 import { PLUGIN_NAME } from './constants';
@@ -108,32 +110,17 @@ export class S3FileNotFound extends GraaspFileError {
   }
 }
 
-export class UploadEmptyFileError extends GraaspFileError {
-  constructor(data?: unknown) {
-    super(
-      {
-        code: 'GPFERR008',
-        statusCode: StatusCodes.BAD_REQUEST,
-        message: FAILURE_MESSAGES.UPLOAD_EMPTY_FILE,
-      },
-      data,
-    );
-  }
-}
+export const UploadEmptyFileError = createError(
+  'GPFERR008',
+  FAILURE_MESSAGES.UPLOAD_EMPTY_FILE,
+  StatusCodes.BAD_REQUEST,
+);
 
-export class UploadFileUnexpectedError extends GraaspFileError {
-  constructor(data?: unknown) {
-    super(
-      {
-        code: 'GPFERR010',
-        statusCode: StatusCodes.INTERNAL_SERVER_ERROR,
-        // TODO: change message
-        message: FAILURE_MESSAGES.UPLOAD_FILE_UNEXPECTED_ERROR,
-      },
-      data,
-    );
-  }
-}
+export const UploadFileUnexpectedError = createError(
+  'GPFERR010',
+  FAILURE_MESSAGES.UPLOAD_FILE_UNEXPECTED_ERROR,
+  StatusCodes.INTERNAL_SERVER_ERROR,
+);
 
 export class DownloadFileUnexpectedError extends GraaspFileError {
   constructor(data?: unknown) {

src/services/item/item.repository.spec.ts

@@ -810,15 +810,13 @@ describe('Item Repository', () => {
       const itemCreatorIdsInDB = insertedItems.map((i) => i.creatorId);
       const itemPathsInDb = insertedItems.map((i) => i.path);
 
-      expect(itemNamesInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemNames.sort(alphabeticalOrder),
+      expect(itemNamesInDB.toSorted(alphabeticalOrder)).toEqual(
+        insertedItemNames.toSorted(alphabeticalOrder),
       );
-      expect(itemTypesInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemTypes.sort(alphabeticalOrder),
-      );
-      expect(itemCreatorIdsInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemCreatorIds.sort(alphabeticalOrder),
+      expect(itemTypesInDB.toSorted(alphabeticalOrder)).toEqual(
+        insertedItemTypes.toSorted(alphabeticalOrder),
       );
+      expect(itemCreatorIdsInDB).toEqual(insertedItemCreatorIds);
       expect(itemPathsInDb.every((path) => !path.includes('.'))).toBeTruthy();
     });
     it('post many with parent item', async () => {
@@ -848,17 +846,15 @@ describe('Item Repository', () => {
       const itemCreatorIdsInDB = insertedItems.map((i) => i.creatorId);
       const itemPathsInDB = insertedItems.map((i) => i.path);
 
-      expect(itemNamesInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemNames.sort(alphabeticalOrder),
-      );
-      expect(itemTypesInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemTypes.sort(alphabeticalOrder),
+      expect(itemNamesInDB.toSorted(alphabeticalOrder)).toEqual(
+        insertedItemNames.toSorted(alphabeticalOrder),
       );
-      expect(itemCreatorIdsInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemCreatorIds.sort(alphabeticalOrder),
+      expect(itemTypesInDB.toSorted(alphabeticalOrder)).toEqual(
+        insertedItemTypes.toSorted(alphabeticalOrder),
       );
-      expect(itemPathsInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemPaths.sort(alphabeticalOrder),
+      expect(itemCreatorIdsInDB).toEqual(insertedItemCreatorIds);
+      expect(itemPathsInDB.toSorted(alphabeticalOrder)).toEqual(
+        insertedItemPaths.toSorted(alphabeticalOrder),
       );
       expect(itemPathsInDB.every((path) => path.includes(`${parentItem.path}.`))).toBeTruthy();
     });
@@ -891,15 +887,13 @@ describe('Item Repository', () => {
       const itemCreatorIdsInDB = insertedItems.map((i) => i.creatorId);
       const itemPathsInDb = insertedItems.map((i) => i.path);
 
-      expect(itemNamesInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemNames.sort(alphabeticalOrder),
+      expect(itemNamesInDB.toSorted(alphabeticalOrder)).toEqual(
+        insertedItemNames.toSorted(alphabeticalOrder),
       );
-      expect(itemTypesInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemTypes.sort(alphabeticalOrder),
-      );
-      expect(itemCreatorIdsInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemCreatorIds.sort(alphabeticalOrder),
+      expect(itemTypesInDB.toSorted(alphabeticalOrder)).toEqual(
+        insertedItemTypes.toSorted(alphabeticalOrder),
       );
+      expect(itemCreatorIdsInDB).toEqual(insertedItemCreatorIds);
       expect(itemPathsInDb.every((path) => !path.includes('.'))).toBeTruthy();
     });
     it('post many with parent item', async () => {
@@ -933,17 +927,15 @@ describe('Item Repository', () => {
       const itemCreatorIdsInDB = insertedItems.map((i) => i.creatorId);
       const itemPathsInDB = insertedItems.map((i) => i.path);
 
-      expect(itemNamesInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemNames.sort(alphabeticalOrder),
-      );
-      expect(itemTypesInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemTypes.sort(alphabeticalOrder),
+      expect(itemNamesInDB.toSorted(alphabeticalOrder)).toEqual(
+        insertedItemNames.toSorted(alphabeticalOrder),
       );
-      expect(itemCreatorIdsInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemCreatorIds.sort(alphabeticalOrder),
+      expect(itemTypesInDB.toSorted(alphabeticalOrder)).toEqual(
+        insertedItemTypes.toSorted(alphabeticalOrder),
       );
-      expect(itemPathsInDB.sort(alphabeticalOrder)).toEqual(
-        insertedItemPaths.sort(alphabeticalOrder),
+      expect(itemCreatorIdsInDB).toEqual(insertedItemCreatorIds);
+      expect(itemPathsInDB.toSorted(alphabeticalOrder)).toEqual(
+        insertedItemPaths.toSorted(alphabeticalOrder),
       );
       expect(itemPathsInDB.every((path) => path.includes(`${parentItem.path}.`))).toBeTruthy();
     });

src/services/item/plugins/action/itemAction.controller.ts

@@ -3,8 +3,6 @@ import { StatusCodes } from 'http-status-codes';
 import type { FastifyPluginAsyncTypebox } from '@fastify/type-provider-typebox';
 import fp from 'fastify-plugin';
 
-import { type FileItemType } from '@graasp/sdk';
-
 import { resolveDependency } from '../../../../di/utils';
 import { db } from '../../../../drizzle/db';
 import { asDefined } from '../../../../utils/assertions';
@@ -13,10 +11,6 @@ import { ActionService } from '../../../action/action.service';
 import { isAuthenticated, matchOne, optionalIsAuthenticated } from '../../../auth/plugins/passport';
 import { assertIsMember } from '../../../authentication';
 import { AuthorizedItemService } from '../../../authorizedItem.service';
-import type {
-  LocalFileConfiguration,
-  S3FileConfiguration,
-} from '../../../file/interfaces/configuration';
 import { validatedMemberAccountRole } from '../../../member/strategies/validatedMemberAccountRole';
 import {
   ItemOpFeedbackErrorEvent,
@@ -34,16 +28,7 @@ import {
 import { ItemActionService } from './itemAction.service';
 import { ActionRequestExportService } from './requestExport/itemAction.requestExport.service';
 
-export interface GraaspActionsOptions {
-  shouldSave?: boolean;
-  fileItemType: FileItemType;
-  fileConfigurations: {
-    s3: S3FileConfiguration;
-    local: LocalFileConfiguration;
-  };
-}
-
-const plugin: FastifyPluginAsyncTypebox<GraaspActionsOptions> = async (fastify) => {
+const plugin: FastifyPluginAsyncTypebox = async (fastify) => {
   const { websockets } = fastify;
 
   const authorizedItemService = resolveDependency(AuthorizedItemService);

src/services/item/plugins/app/appItem.controller.ts

@@ -9,9 +9,8 @@ import { validatedMemberAccountRole } from '../../../member/strategies/validated
 import { ItemActionService } from '../action/itemAction.service';
 import { createApp, updateApp } from './app.schemas';
 import { AppItemService } from './appItemService';
-import type { AppsPluginOptions } from './types';
 
-export const plugin: FastifyPluginAsyncTypebox<AppsPluginOptions> = async (fastify) => {
+export const plugin: FastifyPluginAsyncTypebox = async (fastify) => {
   // service for item app api
   const appItemService = resolveDependency(AppItemService);
   const itemActionService = resolveDependency(ItemActionService);

src/services/item/plugins/app/appSetting/appSetting.controller.ts

@@ -2,7 +2,8 @@ import type { FastifyPluginAsyncTypebox } from '@fastify/type-provider-typebox';
 
 import { resolveDependency } from '../../../../../di/utils';
 import { type DBConnection, db } from '../../../../../drizzle/db';
-import type { AuthenticatedUser } from '../../../../../types';
+import { MinimalItemForInsert } from '../../../../../drizzle/types';
+import type { MaybeUser } from '../../../../../types';
 import { asDefined } from '../../../../../utils/assertions';
 import {
   authenticateAppsJWT,
@@ -34,9 +35,9 @@ const plugin: FastifyPluginAsyncTypebox = async (fastify) => {
 
   // copy app settings and related files on item copy
   const hook = async (
-    actor: AuthenticatedUser,
+    actor: MaybeUser,
     dbConnection: DBConnection,
-    { original, copy }: { original: ItemRaw; copy: ItemRaw },
+    { original, copy }: { original: ItemRaw; copy: MinimalItemForInsert },
   ) => {
     if (original.type !== 'app' || copy.type !== 'app') return;