@solidxai/core 0.1.9-beta.8 → 0.1.10-alpha.0

package/src/services/settings/default-settings-provider.service.ts

@@ -23,9 +23,27 @@ const getSolidCoreSettings = (isProd: boolean) =>
       sortOrder: 50,
       controlType: "boolean",
     },
-    { moduleName: "solid-core", key: "iamFacebookOAuthEnabled", value: false, level: SettingLevel.SystemAdminEditable },
-  { moduleName: "solid-core", key: "iamMicrosoftOAuthEnabled", value: false, level: SettingLevel.SystemAdminEditable },
-  {
+    {
+      moduleName: "solid-core",
+      key: "iamFacebookOAuthEnabled",
+      value: false,
+      level: SettingLevel.SystemAdminEditable,
+      label: "Allow Login / Signup With Facebook",
+      group: "authentication-settings",
+      sortOrder: 50,
+      controlType: "boolean",
+    },
+    {
+      moduleName: "solid-core",
+      key: "iamMicrosoftOAuthEnabled",
+      value: false,
+      level: SettingLevel.SystemAdminEditable,
+      label: "Allow Login / Signup With Microsoft",
+      group: "authentication-settings",
+      sortOrder: 50,
+      controlType: "boolean",
+    },
+    {
       moduleName: "solid-core",
       key: "authPagesLayout",
       value: "center",
@@ -277,11 +295,12 @@ const getSolidCoreSettings = (isProd: boolean) =>
       level: SettingLevel.SystemAdminEditable,
       encrypted: true,
       label: "AI Code Builder Configuration",
-      description: "Structured configuration for AI providers and model assignments.",
+      description:
+        "Structured configuration for AI providers and model assignments.",
       group: "ai-settings",
       sortOrder: 10,
       controlType: "custom",
-      settingsWidget: "solidXGenAiCodeBuilderConfigWidget"
+      settingsWidget: "solidXGenAiCodeBuilderConfigWidget",
     },
     {
       moduleName: "solid-core",
@@ -616,19 +635,63 @@ const getSolidCoreSettings = (isProd: boolean) =>
       controlType: "shortText",
     },
 
-  // facebook-oauth-settings-provider.service.ts
-  { moduleName: "solid-core", key: "FACEBOOK_CLIENT_ID", value: process.env.IAM_FACEBOOK_OAUTH_CLIENT_ID, level: SettingLevel.SystemAdminReadonly },
-  { moduleName: "solid-core", key: "FACEBOOK_CLIENT_SECRET", value: process.env.IAM_FACEBOOK_OAUTH_CLIENT_SECRET, level: SettingLevel.SystemEnv },
-  { moduleName: "solid-core", key: "FACEBOOK_CALLBACK_URL", value: process.env.IAM_FACEBOOK_OAUTH_CALLBACK_URL, level: SettingLevel.SystemAdminReadonly },
-  { moduleName: "solid-core", key: "FACEBOOK_REDIRECT_URL", value: process.env.IAM_FACEBOOK_OAUTH_REDIRECT_URL, level: SettingLevel.SystemAdminReadonly },
+    // facebook-oauth-settings-provider.service.ts
+    {
+      moduleName: "solid-core",
+      key: "FACEBOOK_CLIENT_ID",
+      value: process.env.IAM_FACEBOOK_OAUTH_CLIENT_ID,
+      level: SettingLevel.SystemAdminReadonly,
+    },
+    {
+      moduleName: "solid-core",
+      key: "FACEBOOK_CLIENT_SECRET",
+      value: process.env.IAM_FACEBOOK_OAUTH_CLIENT_SECRET,
+      level: SettingLevel.SystemEnv,
+    },
+    {
+      moduleName: "solid-core",
+      key: "FACEBOOK_CALLBACK_URL",
+      value: process.env.IAM_FACEBOOK_OAUTH_CALLBACK_URL,
+      level: SettingLevel.SystemAdminReadonly,
+    },
+    {
+      moduleName: "solid-core",
+      key: "FACEBOOK_REDIRECT_URL",
+      value: process.env.IAM_FACEBOOK_OAUTH_REDIRECT_URL,
+      level: SettingLevel.SystemAdminReadonly,
+    },
 
-  
-  // microsoft-oauth-settings-provider.service.ts
-  { moduleName: "solid-core", key: "MICROSOFT_CLIENT_ID", value: process.env.IAM_MICROSOFT_OAUTH_CLIENT_ID, level: SettingLevel.SystemAdminReadonly },
-  { moduleName: "solid-core", key: "MICROSOFT_CLIENT_SECRET", value: process.env.IAM_MICROSOFT_OAUTH_CLIENT_SECRET, level: SettingLevel.SystemEnv },
-  { moduleName: "solid-core", key: "MICROSOFT_TENANT_ID", value: process.env.IAM_MICROSOFT_OAUTH_TENANT_ID || 'common', level: SettingLevel.SystemAdminReadonly },
-  { moduleName: "solid-core", key: "MICROSOFT_CALLBACK_URL", value: process.env.IAM_MICROSOFT_OAUTH_CALLBACK_URL, level: SettingLevel.SystemAdminReadonly },
-  { moduleName: "solid-core", key: "MICROSOFT_REDIRECT_URL", value: process.env.IAM_MICROSOFT_OAUTH_REDIRECT_URL, level: SettingLevel.SystemAdminReadonly },
+    // microsoft-oauth-settings-provider.service.ts
+    {
+      moduleName: "solid-core",
+      key: "MICROSOFT_CLIENT_ID",
+      value: process.env.IAM_MICROSOFT_OAUTH_CLIENT_ID,
+      level: SettingLevel.SystemAdminReadonly,
+    },
+    {
+      moduleName: "solid-core",
+      key: "MICROSOFT_CLIENT_SECRET",
+      value: process.env.IAM_MICROSOFT_OAUTH_CLIENT_SECRET,
+      level: SettingLevel.SystemEnv,
+    },
+    {
+      moduleName: "solid-core",
+      key: "MICROSOFT_TENANT_ID",
+      value: process.env.IAM_MICROSOFT_OAUTH_TENANT_ID || "common",
+      level: SettingLevel.SystemAdminReadonly,
+    },
+    {
+      moduleName: "solid-core",
+      key: "MICROSOFT_CALLBACK_URL",
+      value: process.env.IAM_MICROSOFT_OAUTH_CALLBACK_URL,
+      level: SettingLevel.SystemAdminReadonly,
+    },
+    {
+      moduleName: "solid-core",
+      key: "MICROSOFT_REDIRECT_URL",
+      value: process.env.IAM_MICROSOFT_OAUTH_REDIRECT_URL,
+      level: SettingLevel.SystemAdminReadonly,
+    },
 
     // iam-settings-provider.service.ts
     {
@@ -661,7 +724,11 @@ const getSolidCoreSettings = (isProd: boolean) =>
       label: "Registration Validation Type",
       group: "authentication-settings",
       sortOrder: 30,
-      controlType: "shortText",
+      controlType: "selectionStatic",
+      options: [
+        { label: "Email", value: "email" },
+        { label: "Mobile", value: "mobile" },
+      ],
     },
     {
       moduleName: "solid-core",
@@ -671,7 +738,12 @@ const getSolidCoreSettings = (isProd: boolean) =>
       label: "Login Validation Type",
       group: "authentication-settings",
       sortOrder: 40,
-      controlType: "shortText",
+      controlType: "selectionStatic",
+      options: [
+        { label: "Email", value: "email" },
+        { label: "Mobile", value: "mobile" },
+        { label: "Selectable", value: "selectable" },
+      ],
     },
     {
       moduleName: "solid-core",
@@ -836,7 +908,11 @@ const getSolidCoreSettings = (isProd: boolean) =>
         (
           process.env.IAM_SEND_WELCOME_EMAIL_ON_SIGNUP ?? "false"
         ).toLowerCase() === "true",
-      level: SettingLevel.SystemEnv,
+      level: SettingLevel.SystemAdminEditable,
+      label: "Send Welcome Email On Signup",
+      group: "authentication-settings",
+      sortOrder: 180,
+      controlType: "boolean",
     },
     {
       moduleName: "solid-core",
@@ -845,7 +921,11 @@ const getSolidCoreSettings = (isProd: boolean) =>
         (
           process.env.IAM_SEND_WELCOME_SMS_ON_SIGNUP ?? "false"
         ).toLowerCase() === "true",
-      level: SettingLevel.SystemEnv,
+      level: SettingLevel.SystemAdminEditable,
+      label: "Send Welcome SMS On Signup",
+      group: "authentication-settings",
+      sortOrder: 190,
+      controlType: "boolean",
     },
     {
       moduleName: "solid-core",

package/src/services/sms/TwilioSMSService.ts

@@ -69,7 +69,7 @@ export class TwilioSMSService implements ISMS {
         try {
             const bodyTemplate = Handlebars.compile(smsTemplate.body);
             body = bodyTemplate(templateParams);
-        } catch (error) {
+        } catch (error: any) {
             throw new Error('Unable to compile sms template body');
         }
         // Finally send the email.
@@ -110,7 +110,7 @@ export class TwilioSMSService implements ISMS {
             }
 
             return r;
-        } catch (error) {
+        } catch (error: any) {
             throw new Error(error);
         }
     }

package/src/services/solid-introspect.service.ts

@@ -197,7 +197,7 @@ export class SolidIntrospectService implements OnApplicationBootstrap {
       let ds: DataSource | undefined;
       try {
         ds = this.moduleRef.get<DataSource>(token, { strict: false });
-      } catch (err) {
+      } catch (err: any) {
         this.logger.warn(`DataSource token for "${dsName ?? 'default'}" not found: ${err?.message ?? err}`);
       }
       if (!ds) {
@@ -209,7 +209,7 @@ export class SolidIntrospectService implements OnApplicationBootstrap {
       if (!ds.isInitialized) {
         try {
           await ds.initialize(); // only if you need to initialize here; in many apps datasources are created earlier
-        } catch (err) {
+        } catch (err: any) {
           this.logger.error(`Failed to initialize DataSource "${dsName}": ${err}`);
           continue;
         }

package/src/services/user-activity-history.service.ts

@@ -38,7 +38,7 @@ export class UserActivityHistoryService extends CRUDService<UserActivityHistory>
         ipAddress: ip,
         userAgent,
       });
-    } catch (err) {
+    } catch (err: any) {
       this._logger.warn(`Failed to log event "${event}" for user ${user?.id}: ${err}`);
     }
   }

package/src/services/user.service.ts

@@ -1,37 +1,68 @@
-import { BadRequestException, forwardRef, Inject, Injectable } from '@nestjs/common';
+import {
+  BadRequestException,
+  forwardRef,
+  Inject,
+  Injectable,
+} from "@nestjs/common";
 import { ModuleRef } from "@nestjs/core";
-import { InjectEntityManager, InjectRepository } from '@nestjs/typeorm';
-import { CRUDService } from 'src/services/crud.service';
-import { EntityManager, Repository } from 'typeorm';
+import { InjectEntityManager, InjectRepository } from "@nestjs/typeorm";
+import { CRUDService } from "src/services/crud.service";
+import { EntityManager, Repository } from "typeorm";
 import type { SolidCoreSetting } from "src/services/settings/default-settings-provider.service";
 
-
-import { OauthUserDto } from '../dtos/oauth-user-dto';
-import { RoleMetadata } from '../entities/role-metadata.entity';
-import { User } from '../entities/user.entity';
-import { ActiveUserData } from '../interfaces/active-user-data.interface';
-import { ERROR_MESSAGES } from 'src/constants/error-messages';
-import { UserRepository } from 'src/repository/user.repository';
-import { RoleMetadataRepository } from 'src/repository/role-metadata.repository';
-import { HashingService } from './hashing.service';
+import { OauthUserDto } from "../dtos/oauth-user-dto";
+import { RoleMetadata } from "../entities/role-metadata.entity";
+import { User } from "../entities/user.entity";
+import { ActiveUserData } from "../interfaces/active-user-data.interface";
+import { ERROR_MESSAGES } from "src/constants/error-messages";
+import { UserRepository } from "src/repository/user.repository";
+import { RoleMetadataRepository } from "src/repository/role-metadata.repository";
+import { HashingService } from "./hashing.service";
 
 @Injectable()
 export class UserService extends CRUDService<User> {
+  private buildFacebookUsernameBase(name?: string): string {
+    const normalized = (name || "")
+      .trim()
+      .toLowerCase()
+      .replace(/[^a-z0-9]+/g, "_")
+      .replace(/^_+|_+$/g, "");
+    return normalized || "facebook_user";
+  }
+
+  private async resolveUniqueUsername(
+    preferredUsername: string,
+    // fallbackUsername: string,
+  ): Promise<string> {
+    let candidate = preferredUsername;
+    let suffix = 0;
+
+    while (await this.repo.findOne({ where: { username: candidate } })) {
+      suffix += 1;
+      candidate = `${preferredUsername}_${suffix}`;
+    }
+
+    if (candidate) {
+      return candidate;
+    }
+
+    // return fallbackUsername;
+  }
+
   constructor(
     readonly hashingService: HashingService,
     @InjectEntityManager()
     readonly entityManager: EntityManager,
     // @InjectRepository(User, 'default')
     readonly repo: UserRepository,
-    @InjectRepository(User, 'default')
+    @InjectRepository(User, "default")
     readonly nonSecurityRuleAwareRepo: Repository<User>,
     // @InjectRepository(RoleMetadata)
     // private readonly roleRepository: Repository<RoleMetadata>,
     private readonly roleRepository: RoleMetadataRepository,
     readonly moduleRef: ModuleRef,
-
   ) {
-    super(entityManager, repo, 'user', 'solid-core', moduleRef);
+    super(entityManager, repo, "user", "solid-core", moduleRef);
   }
 
   override async delete(id: number, solidRequestContext: any = {}) {
@@ -44,13 +75,19 @@ export class UserService extends CRUDService<User> {
     return super.delete(id, solidRequestContext);
   }
 
-  override async deleteMany(ids: number[], solidRequestContext: any = {}): Promise<any> {
+  override async deleteMany(
+    ids: number[],
+    solidRequestContext: any = {},
+  ): Promise<any> {
     if (!ids || ids.length === 0) {
       throw new Error(ERROR_MESSAGES.DELETE_IDS_REQUIRED);
     }
 
     // ❌ If the active user is trying to delete themselves
-    if (solidRequestContext?.activeUser?.sub && ids.includes(solidRequestContext.activeUser.id)) {
+    if (
+      solidRequestContext?.activeUser?.sub &&
+      ids.includes(solidRequestContext.activeUser.id)
+    ) {
       throw new BadRequestException(ERROR_MESSAGES.DELETE_SELF_NOT_ALLOWED);
     }
 
@@ -60,9 +97,9 @@ export class UserService extends CRUDService<User> {
   async findOneByEmail(email: string): Promise<User> {
     return await this.repo.findOne({
       where: {
-        email: email
+        email: email,
       },
-      relations: {}
+      relations: {},
     });
     // if (!entity) {
     //     throw new NotFoundException(`user with email #${email} not found`);
@@ -73,18 +110,18 @@ export class UserService extends CRUDService<User> {
   async findOneByAccessCode(accessCode: string): Promise<User> {
     return await this.repo.findOne({
       where: {
-        accessCode: accessCode
+        accessCode: accessCode,
       },
-      relations: {}
+      relations: {},
     });
   }
 
   async findOneByUsername(username: string): Promise<User> {
     return await this.repo.findOne({
       where: {
-        username: username
+        username: username,
       },
-      relations: {}
+      relations: {},
     });
     // if (!entity) {
     //     throw new NotFoundException(`user with username ${username} not found`);
@@ -96,8 +133,8 @@ export class UserService extends CRUDService<User> {
     const user = await this.repo.findOne({
       where: { id: id },
       relations: {
-        roles: true
-      }
+        roles: true,
+      },
     });
     if (!user) {
       throw new Error(ERROR_MESSAGES.USER_NOT_FOUND);
@@ -113,21 +150,22 @@ export class UserService extends CRUDService<User> {
     const user = await this.repo.findOne({
       where: { username: username },
       relations: {
-        roles: true
-      }
+        roles: true,
+      },
     });
     if (!user) {
       throw new Error(ERROR_MESSAGES.USER_NOT_FOUND_BY_USERNAME(username));
     }
-    const role = await this.roleRepository.findOne({ where: { name: roleName } });
+    const role = await this.roleRepository.findOne({
+      where: { name: roleName },
+    });
     if (!role) {
       throw new Error(ERROR_MESSAGES.ROLE_NOT_FOUND(roleName));
     }
 
     if (user.roles && user.roles.length > 0) {
       user.roles.push(role);
-    }
-    else {
+    } else {
       user.roles = [role];
     }
 
@@ -137,7 +175,7 @@ export class UserService extends CRUDService<User> {
   async addRolesToUser(username: string, roleNames: string[]): Promise<User> {
     const user = await this.nonSecurityRuleAwareRepo.findOne({
       where: { username: username },
-      relations: { roles: true }
+      relations: { roles: true },
     });
 
     if (!user) {
@@ -145,43 +183,47 @@ export class UserService extends CRUDService<User> {
     }
 
     const roles = await this.roleRepository.find({
-      where: roleNames.map(roleName => ({ name: roleName }))
+      where: roleNames.map((roleName) => ({ name: roleName })),
     });
 
     if (roles.length !== roleNames.length) {
-      const foundRoleNames = roles.map(role => role.name);
-      const missingRoles = roleNames.filter(roleName => !foundRoleNames.includes(roleName));
+      const foundRoleNames = roles.map((role) => role.name);
+      const missingRoles = roleNames.filter(
+        (roleName) => !foundRoleNames.includes(roleName),
+      );
       throw new Error(ERROR_MESSAGES.ROLES_NOT_FOUND(missingRoles));
     }
 
-    const currentRoles = user.roles.map(role => role.name);
+    const currentRoles = user.roles.map((role) => role.name);
 
-    const rolesToAdd = roles.filter(role => !currentRoles.includes(role.name));
+    const rolesToAdd = roles.filter(
+      (role) => !currentRoles.includes(role.name),
+    );
 
-    const rolesToRemove = user.roles.filter(role => !roleNames.includes(role.name));
+    const rolesToRemove = user.roles.filter(
+      (role) => !roleNames.includes(role.name),
+    );
 
     if (rolesToAdd.length > 0) {
       user.roles.push(...rolesToAdd);
     }
 
     if (rolesToRemove.length > 0) {
-      user.roles = user.roles.filter(role => !rolesToRemove.includes(role));
+      user.roles = user.roles.filter((role) => !rolesToRemove.includes(role));
     }
 
     return await this.nonSecurityRuleAwareRepo.save(user);
   }
 
-
   async removeRoleFromUser(username: string, roleName: string): Promise<User> {
-
     // load the role with the respective permissions.
     const user = await this.repo.findOne({
       where: {
-        username: username
+        username: username,
       },
       relations: {
-        roles: true
-      }
+        roles: true,
+      },
     });
 
     if (!user) {
@@ -189,7 +231,7 @@ export class UserService extends CRUDService<User> {
     }
 
     // modify the permissions array.
-    user.roles = user.roles.filter(role => role.name !== roleName);
+    user.roles = user.roles.filter((role) => role.name !== roleName);
 
     return await this.repo.save(user);
   }
@@ -201,11 +243,11 @@ export class UserService extends CRUDService<User> {
         email: oauthUserDto.email,
       },
       relations: {
-        roles: true
-      }
+        roles: true,
+      },
     });
 
-    // if we are unable to find a user then we need to create one. 
+    // if we are unable to find a user then we need to create one.
     if (!user) {
       const user = new User();
       user.username = oauthUserDto.email;
@@ -220,9 +262,12 @@ export class UserService extends CRUDService<User> {
       const savedUser = await this.repo.save(user);
 
       // Initialize the user roles
-      await this.initializeRolesForNewUser([this.settingService.getConfigValue<SolidCoreSetting>('defaultRole')], savedUser);
+      await this.initializeRolesForNewUser(
+        [this.settingService.getConfigValue<SolidCoreSetting>("defaultRole")],
+        savedUser,
+      );
     }
-    // else we update the user and store the generated code & access token. 
+    // else we update the user and store the generated code & access token.
     else {
       const entity = await this.repo.preload({
         id: user.id,
@@ -240,19 +285,48 @@ export class UserService extends CRUDService<User> {
   }
 
   async resolveUserOnOauthFacebook(oauthUserDto: OauthUserDto): Promise<User> {
-    const user = await this.repo.findOne({
-      where: {
-        email: oauthUserDto.email,
-      },
-      relations: {
-        roles: true,
-      },
-    });
+    const normalizedEmail = oauthUserDto.email?.trim().toLowerCase() || null;
+    let user: User | null = null;
+
+    if (oauthUserDto.providerId) {
+      user = await this.repo.findOne({
+        where: {
+          facebookId: oauthUserDto.providerId,
+        },
+        relations: {
+          roles: true,
+        },
+      });
+    }
 
     if (!user) {
+      const facebookProviderFallback = `facebook_${oauthUserDto.providerId}`;
+      const facebookNameUsername = this.buildFacebookUsernameBase(
+        oauthUserDto.name,
+      );
+      // let username = normalizedEmail || facebookNameUsername;
+      let username = facebookNameUsername;
+
+      let email = normalizedEmail;
+
+      // Avoid clashing with local users that already own the same email/username.
+      if (normalizedEmail) {
+        const existingByEmail = await this.repo.findOne({
+          where: { email: normalizedEmail },
+        });
+        if (existingByEmail) {
+          username = facebookNameUsername;
+          email = null;
+        }
+      }
+      username = await this.resolveUniqueUsername(
+        username,
+        // facebookProviderFallback,
+      );
+
       const newUser = new User();
-      newUser.username = oauthUserDto.email;
-      newUser.email = oauthUserDto.email;
+      newUser.username = username;
+      newUser.email = email;
       newUser.fullName = oauthUserDto.name;
       newUser.lastLoginProvider = oauthUserDto.provider;
       newUser.accessCode = oauthUserDto.accessCode;
@@ -263,7 +337,7 @@ export class UserService extends CRUDService<User> {
       const savedUser = await this.repo.save(newUser);
 
       await this.initializeRolesForNewUser(
-        [this.settingService.getConfigValue<SolidCoreSetting>('defaultRole')],
+        [this.settingService.getConfigValue<SolidCoreSetting>("defaultRole")],
         savedUser,
       );
       return savedUser;
@@ -276,13 +350,11 @@ export class UserService extends CRUDService<User> {
         facebookId: oauthUserDto.providerId,
         facebookProfilePicture: oauthUserDto.picture,
       });
-
       await this.repo.save(entity);
       return entity;
     }
   }
 
-
   async resolveUserOnOauthMicrosoft(oauthUserDto: OauthUserDto): Promise<User> {
     const user = await this.repo.findOne({
       where: {
@@ -307,10 +379,9 @@ export class UserService extends CRUDService<User> {
       const savedUser = await this.repo.save(newUser);
 
       await this.initializeRolesForNewUser(
-        [this.settingService.getConfigValue<SolidCoreSetting>('defaultRole')],
+        [this.settingService.getConfigValue<SolidCoreSetting>("defaultRole")],
         savedUser,
       );
-      return savedUser;
     } else {
       const entity = await this.repo.preload({
         id: user.id,
@@ -322,25 +393,29 @@ export class UserService extends CRUDService<User> {
       });
 
       await this.repo.save(entity);
-      return entity;
     }
+    return user;
   }
 
-  async findUsersByRole(roleName: string, relations: any = {}): Promise<User[]> {
+  async findUsersByRole(
+    roleName: string,
+    relations: any = {},
+  ): Promise<User[]> {
     return await this.repo.find({
       where: {
         roles: {
-          name: roleName
-        }
+          name: roleName,
+        },
       },
-      relations: relations
+      relations: relations,
     });
   }
 
   async checkIfPermissionExists(query: any, activeUser: ActiveUserData) {
-
-    const matchingPermssions = activeUser.permissions.filter((p) => query.permissionNames.includes(p));
-    return matchingPermssions
+    const matchingPermssions = activeUser.permissions.filter((p) =>
+      query.permissionNames.includes(p),
+    );
+    return matchingPermssions;
   }
 
   async initializeRolesForNewUser(roles: string[], user: User) {
@@ -348,7 +423,7 @@ export class UserService extends CRUDService<User> {
       throw new BadRequestException(ERROR_MESSAGES.USER_MISSING_ID);
     }
     let userRoles = [];
-    // Default Internal user role assigned 
+    // Default Internal user role assigned
     userRoles.push("Internal User");
     if (roles) {
       userRoles = [...userRoles, ...roles];
@@ -372,7 +447,4 @@ export class UserService extends CRUDService<User> {
       passwordSchemeVersion: this.hashingService.currentVersion(),
     };
   }
-
-
 }
-