refactor: rename MFA enabled field to is_enabled

apps/client/src/ee/mfa/components/mfa-settings.tsx

@@ -25,7 +25,7 @@ export function MfaSettings() {
   }
 
   // Check if MFA is truly enabled
-  const isMfaEnabled = mfaStatus?.enabled === true;
+  const isMfaEnabled = mfaStatus?.isEnabled === true;
 
   const handleSetupComplete = () => {
     setSetupModalOpen(false);

apps/client/src/ee/mfa/types/mfa.types.ts

@@ -1,10 +1,10 @@
 export interface MfaMethod {
   type: 'totp' | 'email';
-  enabled: boolean;
+  isEnabled: boolean;
 }
 
 export interface MfaSettings {
-  enabled: boolean;
+  isEnabled: boolean;
   methods: MfaMethod[];
   backupCodesCount: number;
   lastUpdated?: string;
@@ -19,7 +19,7 @@ export interface MfaSetupState {
 }
 
 export interface MfaStatusResponse {
-  enabled?: boolean;
+  isEnabled?: boolean;
   method?: string | null;
   backupCodesCount?: number;
 }

apps/server/src/core/auth/services/auth.service.ts

@@ -207,7 +207,7 @@ export class AuthService {
     });
 
     // Check if user has MFA enabled or workspace enforces MFA
-    const userHasMfa = user?.['mfa']?.enabled || false;
+    const userHasMfa = user?.['mfa']?.isEnabled || false;
     const workspaceEnforcesMfa = workspace.enforceMfa || false;
 
     if (userHasMfa || workspaceEnforcesMfa) {

apps/server/src/database/migrations/20250715T070817-mfa.ts

@@ -9,11 +9,9 @@ export async function up(db: Kysely<any>): Promise<void> {
     .addColumn('user_id', 'uuid', (col) =>
       col.references('users.id').onDelete('cascade').notNull(),
     )
-    .addColumn('method', 'varchar(50)', (col) =>
+    .addColumn('method', 'varchar', (col) => col.notNull().defaultTo('totp'))
-      col.notNull().defaultTo('totp'),
+    .addColumn('secret', 'text', (col) => col)
-    )
+    .addColumn('is_enabled', 'boolean', (col) => col.defaultTo(false))
-    .addColumn('secret', 'varchar(255)', (col) => col)
-    .addColumn('enabled', 'boolean', (col) => col.defaultTo(false))
     .addColumn('backup_codes', sql`text[]`, (col) => col)
     .addColumn('workspace_id', 'uuid', (col) =>
       col.references('workspaces.id').onDelete('cascade').notNull(),

apps/server/src/database/repos/user/user.repo.ts

@@ -190,7 +190,7 @@ export class UserRepo {
         .select([
           'userMfa.id',
           'userMfa.method',
-          'userMfa.enabled',
+          'userMfa.isEnabled',
           'userMfa.createdAt',
         ])
         .whereRef('userMfa.userId', '=', 'users.id'),

apps/server/src/database/types/db.d.ts

@@ -250,8 +250,8 @@ export interface Spaces {
 export interface UserMfa {
   backupCodes: string[] | null;
   createdAt: Generated<Timestamp>;
-  enabled: Generated<boolean | null>;
   id: Generated<string>;
+  isEnabled: Generated<boolean | null>;
   method: Generated<string>;
   secret: string | null;
   updatedAt: Generated<Timestamp>;