fix: ts strict

src/common/@types/classes/common.response.ts

@@ -2,14 +2,14 @@ export class AuthenticationResponse {
   /**
    * @example fb9eac5f-eb94-489b-8fca-24324558be18
    */
-  user: {
-    idx?: string
+  user!: {
+    idx: string
   };
 
   /**
    * @example eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXKYjj.eyJ
    */
-  accessToken: string;
+  accessToken!: string;
 
   /**
    * @example eyJh3d06e6e3e152ae839a6623c3cb6f961a.eyJ

src/common/@types/classes/cursor.response.ts

@@ -7,19 +7,19 @@ export class CursorMeta {
    * @example AdVxY2F0ZWdvcnlfaWQ9MjMx
    */
   @ApiProperty()
-    nextCursor: string;
+    nextCursor!: string;
 
   /**
    * @example false
    */
   @ApiProperty()
-    hasNextPage: boolean;
+    hasNextPage!: boolean;
 
   /**
    * @example true
    */
   @ApiProperty()
-    hasPreviousPage: boolean;
+    hasPreviousPage!: boolean;
 
   /**
    * @example "lorem ipsum"
@@ -31,8 +31,8 @@ export class CursorMeta {
 export class CursorPaginationResponse<T> implements PaginationAbstractResponse<T, CursorMeta> {
   @IsArray()
   @ApiProperty({ isArray: true })
-  readonly data: T[];
+  readonly data!: T[];
 
   @ApiProperty({ type: () => CursorMeta })
-  readonly meta: CursorMeta;
+  readonly meta!: CursorMeta;
 }

src/common/@types/interfaces/mail.interface.ts

@@ -2,7 +2,7 @@ import type { EmailTemplate } from "../enums";
 
 export interface MailPayload {
   template: string
-  replacements?: Record<string, string>
+  replacements: Record<string, string>
   to: string
   subject: string
   from: string

src/common/constant/string.constants.ts

@@ -15,7 +15,7 @@ const packageJson = readPackageSync();
 
 export const APP_NAME = packageJson.name;
 export const SWAGGER_API_CURRENT_VERSION = packageJson.version;
-export const SWAGGER_DESCRIPTION = packageJson.description;
+export const SWAGGER_DESCRIPTION = packageJson.description!;
 export const SWAGGER_TITLE = `${capitalize(APP_NAME)} API Documentation`;
 
 export const SWAGGER_API_ENDPOINT = "doc";

src/common/database/base.repository.ts

@@ -487,7 +487,7 @@ export class BaseRepository<T extends BaseEntity> extends EntityRepository<T> {
       countWhere["$and"] = this.getFilters("createdAt", decoded, oppositeOrder);
       previousCount = await repo.count(countWhere);
 
-      // eslint-disable-next-line ts/dot-notation
+      // eslint-disable-next-line ts/dot-notation\
       where["$and"] = this.getFilters("createdAt", decoded, queryOrder);
     }
 

src/common/database/mikro-orm.encrypted.ts

@@ -6,7 +6,7 @@ export class EncryptedType extends Type {
   private readonly encKey = process.env.ENC_KEY;
   private readonly encIV = process.env.ENC_IV;
 
-  convertToDatabaseValue(value: string | undefined, _platform: Platform): string {
+  convertToDatabaseValue(value: string , _platform: Platform): string {
     if (value && !(typeof value.valueOf() === "string"))
       throw ValidationError.invalidType(EncryptedType, value, "JS");
 

src/common/database/user.subscriber.ts

@@ -13,12 +13,14 @@ export class UserSubscriber implements EventSubscriber<User> {
   }
 
   async beforeCreate(arguments_: EventArgs<User>): Promise<void> {
-    if (arguments_.changeSet.payload?.password)
+    if (arguments_.changeSet?.payload?.password)
       arguments_.entity.password = await HelperService.hashString(arguments_.entity.password);
   }
 
+
   async beforeUpdate(arguments_: EventArgs<User>): Promise<void> {
-    if (arguments_.changeSet.payload?.password)
-      arguments_.entity.password = await HelperService.hashString(arguments_.entity.password);
-  }
+      if (arguments_.changeSet?.payload?.password)
+        arguments_.entity.password = await HelperService.hashString(arguments_.entity.password);
+    }
+
 }

src/common/decorators/api-file.decorator.ts

@@ -27,7 +27,7 @@ interface ApiFilesOptions extends ApiFileOptions {
  * @returns A function that returns a decorator.
  */
 export function ApiFile(options_?: ApiFileOptions) {
-  const options: ApiFileOptions = { fieldName: "file", required: false, ...options_ };
+  const options = { fieldName: "file", required: false, ...options_ } satisfies ApiFilesOptions
 
   return applyDecorators(
     UseInterceptors(FileInterceptor(options.fieldName, options.localOptions)),
@@ -54,12 +54,12 @@ export function ApiFile(options_?: ApiFileOptions) {
  * @returns A function that returns a decorator.
  */
 export function ApiFiles(options_?: ApiFilesOptions) {
-  const options: ApiFilesOptions = {
+  const options = {
     fieldName: "files",
     required: false,
     maxCount: 10,
     ...options_,
-  };
+  } satisfies ApiFilesOptions;
 
   return applyDecorators(
     UseInterceptors(

src/common/decorators/auth.decorator.ts

@@ -17,11 +17,11 @@ interface AuthGuard {
  */
 
 export function Auth(options_?: AuthGuard) {
-  const options: AuthGuard = {
+  const options = {
     guards: [JwtAuthGuard, PoliciesGuard],
     unauthorizedResponse: API_UNAUTHORISED_RESPONSE,
     ...options_,
-  };
+  } satisfies AuthGuard;
 
   return applyDecorators(
     UseGuards(...options.guards),

src/common/decorators/validation/is-number-field.decorator.ts

@@ -21,7 +21,7 @@ import type { NumberFieldOptions } from "@common/@types";
  */
 
 export function IsNumberField(options_?: NumberFieldOptions) {
-  const options: NumberFieldOptions = {
+  const options = {
     min: 1,
     required: true,
     each: false,

src/common/decorators/validation/min-max-length.decorator.ts

@@ -9,7 +9,7 @@ import { validationI18nMessage } from "@lib/i18n";
  * @returns A function that takes in a target, propertyKey, and descriptor
  */
 export function MinMaxLength(options_?: MinMaxLengthOptions) {
-  const options: MinMaxLengthOptions = { minLength: 2, maxLength: 500, each: false, ...options_ };
+  const options = { minLength: 2, maxLength: 500, each: false, ...options_ };
 
   return applyDecorators(
     MinLength(options.minLength, {

src/common/dtos/pagination.dto.ts

@@ -23,7 +23,7 @@ export abstract class PaginationDto {
    *  The search query
    */
   @IsStringField({ required: false, minLength: 1, maxLength: 100 })
-    search: string;
+    search?: string;
 
   /**
    * The `withDeleted` property is a boolean flag that

src/entities/conversation.entity.ts

@@ -5,7 +5,7 @@ import { Message, User } from "./index";
 @Entity()
 export class Conversation extends BaseEntity {
   @Property({ index: true })
-    chatName: string;
+    chatName!: string;
 
   @ManyToMany(() => User, user => user.conversations, { index: true })
     users = new Collection<User>(this);

src/entities/message.entity.ts

@@ -12,7 +12,7 @@ export class Message extends BaseEntity {
     eager: false,
     index: true,
   })
-    sender: Rel<Ref<Post>>;
+    sender!: Rel<Ref<Post>>;
 
   @ManyToOne({
     eager: false,

src/entities/news-letter.entity.ts

@@ -4,7 +4,7 @@ import { BaseEntity } from "@common/database";
 @Entity()
 export class NewsLetter extends BaseEntity {
   @Property({ index: true, unique: true })
-    name: string;
+    name!: string;
 
   @Property({ columnType: "text" })
     content!: string;

src/entities/otp-log.entity.ts

@@ -18,7 +18,7 @@ export class OtpLog extends BaseEntity {
     eager: false,
     index: true,
   })
-    user: Rel<Ref<User>>;
+    user!: Rel<Ref<User>>;
 
   @Property()
     isUsed? = false;

src/entities/refresh-token.entity.ts

@@ -11,7 +11,7 @@ export class RefreshToken extends BaseEntity {
   @ManyToOne({
     eager: false,
   })
-    user: Rel<Ref<User>>;
+    user!: Rel<Ref<User>>;
 
   @Property()
     isRevoked? = false;

src/lib/casl/casl-ability.factory.ts

@@ -16,7 +16,7 @@ export class CaslAbilityFactory {
 
     /* Giving the user the ability to read and write to everything if they are an admin. */
 
-    if (user.roles.includes(Roles.ADMIN))
+    if (user.roles!.includes(Roles.ADMIN))
       can(Action.Manage, "all"); // read-write access to everything
     else
       can(Action.Read, "all"); // read-only access to everything

src/lib/config/configs/minio.config.ts

@@ -11,7 +11,7 @@ export const minioConfigValidationSchema = {
 
 export const minio = registerAs("minio", () => ({
   host: process.env.MINIO_HOST,
-  port: Number.parseInt(process.env.MINIO_PORT, 10),
+  port: Number.parseInt(process.env.?MINIO_PORT, 10),
   accessKey: process.env.MINIO_ACCESS_KEY,
   secretKey: process.env.MINIO_SECRET_KEY,
   useSSl: process.env.MINIO_USE_SSL === "true",

src/lib/firebase-admin/firebase.service.ts

@@ -11,7 +11,7 @@ interface NestFirebase {
 
 @Injectable()
 export class NestFirebaseService implements NestFirebase {
-  private _firebaseConnection: admin.app.App;
+  private _firebaseConnection!: admin.app.App;
 
   constructor(
   @Inject(MODULE_OPTIONS_TOKEN) private _NestFirebaseOptions: FirebaseModuleOptions,

src/lib/minio.module.ts

@@ -9,7 +9,7 @@ import { NestMinioModule } from "nestjs-minio";
       inject: [ConfigService],
       isGlobal: true,
       useFactory: async (configService: ConfigService) => ({
-        endPoint: configService.get("minio.host"),
+        endPoint: configService.get("minio.host")!,
         port: configService.get("minio.port"),
         accessKey: configService.get("minio.accessKey"),
         secretKey: configService.get("minio.secretKey"),

src/main.ts

@@ -96,7 +96,7 @@ async function bootstrap() {
     module.hot.dispose(() => app.close());
   }
 
-  const port = process.env.PORT ?? configService.get("app.port", { infer: true });
+  const port = process.env.PORT ?? configService.get("app.port", { infer: true })!;
 
   await app.listen(port);
 

src/modules/auth/auth.controller.ts

@@ -184,6 +184,6 @@ export class AuthController {
   ): Observable<User> {
     return fromAll
       ? this.authService.logoutFromAll(user)
-      : this.authService.logout(user, refreshToken.refreshToken);
+      : this.authService.logout(user, refreshToken!.refreshToken);
   }
 }

src/modules/auth/auth.service.ts

@@ -6,6 +6,7 @@ import {
   ForbiddenException,
   Injectable,
   NotFoundException,
+  UnauthorizedException,
 } from "@nestjs/common";
 import { ConfigService } from "@nestjs/config";
 import { init } from "@paralleldrive/cuid2";
@@ -79,7 +80,7 @@ export class AuthService {
         }
 
         return user && isPasswordLogin
-          ? HelperService.verifyHash(user.password, pass).pipe(
+          ? HelperService.verifyHash(user.password, pass!).pipe(
             map((isValid) => {
               if (isValid)
                 return omit(user, ["password"]);
@@ -198,7 +199,7 @@ export class AuthService {
             const otp = this.otpRepository.create({
               user: userExists,
               otpCode: otpNumber,
-              expiresIn: new Date(Date.now() + (protocol.otpExpiryInMinutes * 60_000)), // prettier-ignore
+              expiresIn: new Date(Date.now() + (protocol?.otpExpiryInMinutes ?? 5 * 60_000)), // prettier-ignore
             });
 
             return from(
@@ -245,6 +246,19 @@ export class AuthService {
       ),
     ).pipe(
       switchMap((details) => {
+
+        if(!details) {
+
+          return throwError(
+            () =>
+              new NotFoundException(
+                translate("exception.itemDoesNotExist", {
+                  args: { item: "Otp" },
+                }),
+              ),
+          );
+        }
+
         const user = details.user.getEntity();
         this.userRepository.assign(user, { password });
 
@@ -332,6 +346,19 @@ export class AuthService {
       }),
     ).pipe(
       switchMap((userDetails) => {
+
+        if(!userDetails) {
+
+          return throwError(
+            () =>
+              new NotFoundException(
+                translate("exception.itemDoesNotExist", {
+                  args: { item: "Account" },
+                }),
+              ),
+          );
+        }
+
         return HelperService.verifyHash(userDetails.password, oldPassword).pipe(
           switchMap((isValid) => {
             if (!isValid) {
@@ -354,6 +381,10 @@ export class AuthService {
   }
 
   async findUser(condition: FilterQuery<User>): Promise<User> {
-    return this.userRepository.findOne(condition);
+    const user = await this.userRepository.findOne(condition);
+
+    if (!user)
+      throw new UnauthorizedException();
+    return user
   }
 }

src/modules/auth/strategies/facebook.strategy.ts

@@ -43,14 +43,14 @@ export class FacebookStrategy extends PassportStrategy(Strategy, "facebook") {
   ): Promise<any> {
     const { name, emails, username, photos } = profile;
     const user: OauthResponse = {
-      email: emails[0].value,
+      email: emails![0]!.value,
       firstName: name?.givenName,
       lastName: name?.familyName,
       accessToken,
     };
     // Check if the user already exists in your database
     const existingUser = await this.userRepo.findOne({
-      email: emails[0].value,
+      email: emails![0]!.value,
       isDeleted: false,
     });
 
@@ -62,7 +62,7 @@ export class FacebookStrategy extends PassportStrategy(Strategy, "facebook") {
       // If the user doesn't exist, create a new user
       const newUser = this.userRepo.create({
         ...omit(user, ["accessToken"]),
-        avatar: photos[0].value,
+        avatar: photos?.[0]?.value ?? '',
         username,
         password: randomString({ length: 10, symbols: true, numbers: true }),
       });