Merge pull request #1065 from Hexastack/1063-issue---onapplicationbootstrap-remove-unused-settings

fix(api): resolve CLI removing settings

api/src/app.instance.ts

@@ -9,7 +9,7 @@
 import { INestApplication } from '@nestjs/common';
 
 export class AppInstance {
-  private static app: INestApplication;
+  private static app: INestApplication | null = null;
 
   static setApp(app: INestApplication) {
     this.app = app;
@@ -21,4 +21,13 @@ export class AppInstance {
     }
     return this.app;
   }
+
+  /**
+   * Checks whether the application context is initialized.
+   * This may return `false` in environments where the app instance is not set,
+   * such as when running in test env or CLI mode without a full application bootstrap.
+   */
+  static isReady(): boolean {
+    return this.app !== null;
+  }
 }

api/src/attachment/attachment.module.ts

@@ -12,6 +12,7 @@ import { Module, OnApplicationBootstrap } from '@nestjs/common';
 import { MongooseModule } from '@nestjs/mongoose';
 import { PassportModule } from '@nestjs/passport';
 
+import { AppInstance } from '@/app.instance';
 import { config } from '@/config';
 import { UserModule } from '@/user/user.module';
 
@@ -34,6 +35,10 @@ import { AttachmentService } from './services/attachment.service';
 })
 export class AttachmentModule implements OnApplicationBootstrap {
   onApplicationBootstrap() {
+    if (!AppInstance.isReady()) {
+      return;
+    }
+
     // Ensure the directories exists
     if (!existsSync(config.parameters.uploadDir)) {
       mkdirSync(config.parameters.uploadDir, { recursive: true });

api/src/extension/extension.module.ts

@@ -8,6 +8,7 @@
 
 import { Global, Module, OnApplicationBootstrap } from '@nestjs/common';
 
+import { AppInstance } from '@/app.instance';
 import { LoggerService } from '@/logger/logger.service';
 
 import { CleanupService } from './cleanup.service';
@@ -24,6 +25,11 @@ export class ExtensionModule implements OnApplicationBootstrap {
   ) {}
 
   async onApplicationBootstrap() {
+    if (!AppInstance.isReady()) {
+      // bypass in test or CLI env
+      return;
+    }
+
     try {
       await this.cleanupService.pruneExtensionSettings();
     } catch (error) {

api/src/extensions/helpers/llm-nlu/index.helper.ts

@@ -10,6 +10,7 @@ import { Injectable, OnApplicationBootstrap } from '@nestjs/common';
 import { OnEvent } from '@nestjs/event-emitter';
 import Handlebars from 'handlebars';
 
+import { AppInstance } from '@/app.instance';
 import { HelperService } from '@/helper/helper.service';
 import BaseNlpHelper from '@/helper/lib/base-nlp-helper';
 import { HelperType, LLM, NLU } from '@/helper/types';
@@ -89,6 +90,11 @@ export default class LlmNluHelper
   }
 
   async onApplicationBootstrap() {
+    if (!AppInstance.isReady()) {
+      // bypass in Test / CLI env
+      return;
+    }
+
     try {
       this.logger.log('Initializing LLM NLU helper, building prompts...');
       // Build prompts for language and trait classifiers