hedgequantx 1.8.48 → 2.3.0

package/src/utils/prompts.js

@@ -1,33 +1,16 @@
 /**
- * Centralized prompts utility
+ * @fileoverview Centralized prompts utility
+ * @module utils/prompts
+ * 
  * Uses native readline for reliable stdin handling
  */
 
 const inquirer = require('inquirer');
 const readline = require('readline');
 
-// Shared readline instance
+/** @type {readline.Interface|null} */
 let rl = null;
 
-// Prevent readline from exiting on SIGINT during prompts
-process.on('SIGINT', () => {
-  // Let the main app handle SIGINT
-});
-
-/**
- * Get or create readline interface
- */
-const getReadline = () => {
-  if (!rl || rl.closed) {
-    rl = readline.createInterface({
-      input: process.stdin,
-      output: process.stdout,
-      terminal: true
-    });
-  }
-  return rl;
-};
-
 /**
  * Ensure stdin is ready and flush any buffered input
  */
@@ -37,56 +20,68 @@ const prepareStdin = () => {
     if (process.stdin.isTTY && process.stdin.setRawMode) {
       process.stdin.setRawMode(false);
     }
-    // Flush any buffered input by reading without waiting
     process.stdin.read();
-  } catch (e) {}
+  } catch {
+    // Ignore stdin errors
+  }
+};
+
+/**
+ * Close existing readline if open
+ * @private
+ */
+const closeReadline = () => {
+  if (rl && !rl.closed) {
+    try {
+      rl.close();
+    } catch {
+      // Ignore close errors
+    }
+    rl = null;
+  }
 };
 
 /**
  * Native readline prompt
+ * @param {string} message - Prompt message
+ * @returns {Promise<string>}
  */
 const nativePrompt = (message) => {
   return new Promise((resolve) => {
     try {
       prepareStdin();
-      
-      // Always create a fresh readline for each prompt to avoid state issues
-      if (rl && !rl.closed) {
-        try { rl.close(); } catch (e) {}
-        rl = null;
-      }
-      
+      closeReadline();
+
       rl = readline.createInterface({
         input: process.stdin,
         output: process.stdout,
-        terminal: true
+        terminal: true,
       });
-      
+
       let answered = false;
-      
-      rl.question(message + ' ', (answer) => {
+
+      rl.question(`${message} `, (answer) => {
         answered = true;
-        try { rl.close(); } catch (e) {}
-        rl = null;
+        closeReadline();
         resolve(answer || '');
       });
-      
-      // Handle readline close (e.g., Ctrl+C)
+
       rl.on('close', () => {
         if (!answered) {
           rl = null;
           resolve('');
         }
       });
-      
-    } catch (e) {
+    } catch {
       resolve('');
     }
   });
 };
 
 /**
- * Wait for Enter
+ * Wait for Enter key
+ * @param {string} [message='Press Enter to continue...'] - Message to display
+ * @returns {Promise<void>}
  */
 const waitForEnter = async (message = 'Press Enter to continue...') => {
   await nativePrompt(message);
@@ -94,6 +89,9 @@ const waitForEnter = async (message = 'Press Enter to continue...') => {
 
 /**
  * Text input
+ * @param {string} message - Prompt message
+ * @param {string} [defaultVal=''] - Default value
+ * @returns {Promise<string>}
  */
 const textInput = async (message, defaultVal = '') => {
   const value = await nativePrompt(message);
@@ -101,48 +99,63 @@ const textInput = async (message, defaultVal = '') => {
 };
 
 /**
- * Password input
+ * Password input (masked)
+ * @param {string} message - Prompt message
+ * @returns {Promise<string>}
  */
 const passwordInput = async (message) => {
-  if (rl && !rl.closed) { rl.close(); rl = null; }
+  closeReadline();
   prepareStdin();
+
   const { value } = await inquirer.prompt([{
     type: 'password',
     name: 'value',
     message,
     mask: '*',
-    prefix: ''
+    prefix: '',
   }]);
+
   return value;
 };
 
 /**
- * Confirm - arrow keys selection
+ * Confirm prompt with arrow keys
+ * @param {string} message - Prompt message
+ * @param {boolean} [defaultVal=true] - Default value
+ * @returns {Promise<boolean>}
  */
 const confirmPrompt = async (message, defaultVal = true) => {
-  if (rl && !rl.closed) { rl.close(); rl = null; }
+  closeReadline();
   prepareStdin();
-  const choices = defaultVal 
+
+  const choices = defaultVal
     ? [{ name: 'Yes', value: true }, { name: 'No', value: false }]
     : [{ name: 'No', value: false }, { name: 'Yes', value: true }];
-  
+
   const { value } = await inquirer.prompt([{
     type: 'list',
     name: 'value',
     message,
     choices,
     prefix: '',
-    loop: false
+    loop: false,
   }]);
+
   return value;
 };
 
 /**
- * Number input
+ * Number input with validation
+ * @param {string} message - Prompt message
+ * @param {number} [defaultVal=1] - Default value
+ * @param {number} [min=1] - Minimum value
+ * @param {number} [max=1000] - Maximum value
+ * @returns {Promise<number>}
  */
 const numberInput = async (message, defaultVal = 1, min = 1, max = 1000) => {
-  if (rl && !rl.closed) { rl.close(); rl = null; }
+  closeReadline();
   prepareStdin();
+
   const { value } = await inquirer.prompt([{
     type: 'input',
     name: 'value',
@@ -150,39 +163,34 @@ const numberInput = async (message, defaultVal = 1, min = 1, max = 1000) => {
     default: String(defaultVal),
     prefix: '',
     validate: (v) => {
-      const n = parseInt(v);
+      const n = parseInt(v, 10);
       if (isNaN(n)) return 'Enter a number';
       if (n < min) return `Min: ${min}`;
       if (n > max) return `Max: ${max}`;
       return true;
-    }
+    },
   }]);
-  return parseInt(value) || defaultVal;
+
+  return parseInt(value, 10) || defaultVal;
 };
 
 /**
- * Select - arrow keys navigation
- * Supports disabled options (separators) via opt.disabled
+ * Select from options with arrow keys
+ * @param {string} message - Prompt message
+ * @param {Array<{label: string, value: any, disabled?: boolean}>} options - Options
+ * @returns {Promise<any>}
  */
 const selectOption = async (message, options) => {
-  // Close shared readline before inquirer to avoid conflicts
-  if (rl && !rl.closed) {
-    rl.close();
-    rl = null;
-  }
+  closeReadline();
   prepareStdin();
-  
+
   const choices = options.map(opt => {
     if (opt.disabled) {
-      // Use inquirer Separator for disabled items (category headers)
       return new inquirer.Separator(opt.label);
     }
-    return {
-      name: opt.label,
-      value: opt.value
-    };
+    return { name: opt.label, value: opt.value };
   });
-  
+
   const { value } = await inquirer.prompt([{
     type: 'list',
     name: 'value',
@@ -190,9 +198,9 @@ const selectOption = async (message, options) => {
     choices,
     prefix: '',
     loop: false,
-    pageSize: 20  // Increased to show more symbols
+    pageSize: 20,
   }]);
-  
+
   return value;
 };
 
@@ -203,5 +211,5 @@ module.exports = {
   passwordInput,
   confirmPrompt,
   numberInput,
-  selectOption
+  selectOption,
 };