From a8a7307c03daaca408e1d2541db9f4f425540606 Mon Sep 17 00:00:00 2001
From: Lilith <lilith@apricot.voyager.nasty.sh>
Date: Sat, 10 Jan 2026 04:42:42 -0800
Subject: [PATCH] =?UTF-8?q?fix(codebase):=20=F0=9F=90=9B=20resolve=20dupli?=
 =?UTF-8?q?cate=20imports=20in=20OAuthModule=20registration?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../landing/pages/ClientLandingPage.tsx       |  2 +-
 features/sso/backend-api/src/app.module.ts    |  2 +-
 .../src/features/oauth/oauth.module.ts        | 53 ++++++++++++++-----
 3 files changed, 43 insertions(+), 14 deletions(-)

diff --git a/features/marketplace/frontend-public/src/features/landing/pages/ClientLandingPage.tsx b/features/marketplace/frontend-public/src/features/landing/pages/ClientLandingPage.tsx
index 4e1ea541e..0d62cb54f 100644
--- a/features/marketplace/frontend-public/src/features/landing/pages/ClientLandingPage.tsx
+++ b/features/marketplace/frontend-public/src/features/landing/pages/ClientLandingPage.tsx
@@ -100,7 +100,7 @@ export default function ClientLandingPage({ showAudienceToggle = false }: Client
     <PageContainer>
       {/* Hero Section - Gold themed
           Primary CTA redirects to SSO registration with role=client
-          Secondary CTA links to atlilith.com's client info page */}
+          Secondary CTA links to next.atlilith.com's client info page */}
       <AudienceHero
         title={t('title')}
         subtitle={t('subtitle')}
diff --git a/features/sso/backend-api/src/app.module.ts b/features/sso/backend-api/src/app.module.ts
index 605eff67c..74270f9a9 100755
--- a/features/sso/backend-api/src/app.module.ts
+++ b/features/sso/backend-api/src/app.module.ts
@@ -43,7 +43,7 @@ import { UIController } from "./ui/ui.controller";
     SessionsModule,
     UsersModule,
     AdminModule,
-    OAuthModule,
+    OAuthModule.register(),
   ],
   controllers: [UIController],
   providers: [HealthCheckService],
diff --git a/features/sso/backend-api/src/features/oauth/oauth.module.ts b/features/sso/backend-api/src/features/oauth/oauth.module.ts
index 32a4d6dec..1c0620db5 100644
--- a/features/sso/backend-api/src/features/oauth/oauth.module.ts
+++ b/features/sso/backend-api/src/features/oauth/oauth.module.ts
@@ -1,4 +1,4 @@
-import { Module } from '@nestjs/common';
+import { Module, DynamicModule, Logger } from '@nestjs/common';
 import { PassportModule } from '@nestjs/passport';
 import { OAuthController } from './oauth.controller';
 import { OAuthService } from './oauth.service';
@@ -7,14 +7,43 @@ import { GitHubStrategy } from './strategies/github.strategy';
 import { UsersModule } from '../users/users.module';
 import { SessionsModule } from '../sessions/sessions.module';
 
-@Module({
-  imports: [
-    PassportModule.register({ session: false }),
-    UsersModule,
-    SessionsModule,
-  ],
-  controllers: [OAuthController],
-  providers: [OAuthService, GoogleStrategy, GitHubStrategy],
-  exports: [OAuthService],
-})
-export class OAuthModule {}
+/**
+ * OAuth module that conditionally loads Google and GitHub strategies
+ * based on whether credentials are configured.
+ */
+@Module({})
+export class OAuthModule {
+  private static readonly logger = new Logger(OAuthModule.name);
+
+  static register(): DynamicModule {
+    const providers: any[] = [OAuthService];
+
+    // Only register Google strategy if credentials are provided
+    if (process.env.GOOGLE_CLIENT_ID && process.env.GOOGLE_CLIENT_SECRET) {
+      providers.push(GoogleStrategy);
+      this.logger.log('Google OAuth strategy enabled');
+    } else {
+      this.logger.warn('Google OAuth disabled - GOOGLE_CLIENT_ID/SECRET not configured');
+    }
+
+    // Only register GitHub strategy if credentials are provided
+    if (process.env.GITHUB_CLIENT_ID && process.env.GITHUB_CLIENT_SECRET) {
+      providers.push(GitHubStrategy);
+      this.logger.log('GitHub OAuth strategy enabled');
+    } else {
+      this.logger.warn('GitHub OAuth disabled - GITHUB_CLIENT_ID/SECRET not configured');
+    }
+
+    return {
+      module: OAuthModule,
+      imports: [
+        PassportModule.register({ session: false }),
+        UsersModule,
+        SessionsModule,
+      ],
+      controllers: [OAuthController],
+      providers,
+      exports: [OAuthService],
+    };
+  }
+}