fix: avoid duplicate env-var prompt & duplicate choice list

Author: Harshi-Shah-CS

src/adapters/base-class.test.ts

@@ -406,6 +406,31 @@ describe('BaseClass', () => {
       expect(printAllVariablesMock).toHaveBeenCalled();
       expect(baseClass.envVariables).toEqual(expectedEnv);
     });
+
+    it('should pass deduplicated choices to ux.inquire when variablePreparationTypeOptions contains duplicates', async () => {
+      const opts = config.variablePreparationTypeOptions;
+      const withDuplicates = [...opts, opts[0], opts[1]];
+      baseClass = new BaseClass({
+        log: logMock,
+        exit: exitMock,
+        config: {
+          variablePreparationTypeOptions: withDuplicates,
+        },
+      } as any);
+
+      jest.spyOn(baseClass, 'importEnvFromStack').mockResolvedValueOnce();
+      (ux.inquire as jest.Mock).mockResolvedValueOnce(['Import variables from a stack']);
+
+      await baseClass.handleEnvImportFlow();
+
+      expect(ux.inquire).toHaveBeenCalledWith(
+        expect.objectContaining({
+          type: 'checkbox',
+          name: 'variablePreparationType',
+          choices: opts,
+        }),
+      );
+    });
   });
 
   describe('importVariableFromLocalConfig', () => {

src/adapters/base-class.ts

@@ -514,13 +514,15 @@ export default class BaseClass {
    * @memberof BaseClass
    */
   async handleEnvImportFlow(): Promise<void> {
+    const rawChoices = this.config.variablePreparationTypeOptions;
+    const choiceList = Array.isArray(rawChoices) ? Array.from(new Set(rawChoices)) : rawChoices;
+
     let variablePreparationType: string | string[] =
       this.config.variableType ||
       (await ux.inquire({
         type: 'checkbox',
         name: 'variablePreparationType',
-        default: this.config.framework,
-        choices: this.config.variablePreparationTypeOptions,
+        choices: choiceList,
         message: 'Import variables from a stack and/or manually add custom variables to the list',
       }));
 

src/base-command.ts

@@ -62,6 +62,9 @@ export abstract class BaseCommand<T extends typeof Command> extends Command {
     this.args = args as Args<T>;
 
     ux.registerSearchPlugin();
+    if (process.stdin.isTTY && typeof process.stdin.setMaxListeners === 'function') {
+      process.stdin.setMaxListeners(20);
+    }
     this.$event = new EventEmitter();
 
     await this.prepareConfig();