devsplain 2.2.3 → 2.3.0

package/lib/config.js

@@ -4,7 +4,7 @@ const os = require('os');
 const readline = require('readline');
 const configPath = path.join(os.homedir(), '.devsplainrc');
 
-// Asks for secret input from the user [ds]
+/** Prompts the user for sensitive input with hidden characters and raw stdin handling [ds] */
 function askSecret(query) {
   return new Promise((resolve) => {
     const rl = readline.createInterface({
@@ -22,12 +22,12 @@ function askSecret(query) {
         process.stdout.write('\n');
         rl.close();
         resolve(key);
-      } else if (char === '\u0008' || char === '\u007f') { // Backspace
+      } else if (char === '\u0008' || char === '\u007f') {
         if (key.length > 0) {
           key = key.slice(0, -1);
           process.stdout.write('\b \b');
         }
-      } else if (char === '\u0003') { // Ctrl+C
+      } else if (char === '\u0003') {
         process.stdin.setRawMode(false);
         process.exit(130);
       } else {
@@ -39,7 +39,7 @@ function askSecret(query) {
   });
 }
 
-// Migrates old config to the new format [ds]
+/** Migrates legacy single-provider config structure to the new multi-provider format [ds] */
 function migrateConfig(oldConfig) {
   if (oldConfig && typeof oldConfig === 'object' && oldConfig.provider && !oldConfig.providers) {
     return {
@@ -57,7 +57,7 @@ function migrateConfig(oldConfig) {
   return oldConfig;
 }
 
-// Gets the config, either from environment variables, the config file, or by prompting the user [ds]
+/** Resolves config from environment variables, file storage, or interactive wizard setup [ds] */
 async function getConfig(forceWizard = false) {
   if (process.env.DEVSPLAIN_API_KEY || process.env.DEVSPLAIN_PROVIDER) {
     const provider = process.env.DEVSPLAIN_PROVIDER || 'gemini';
@@ -77,7 +77,6 @@ async function getConfig(forceWizard = false) {
       const rawData = fs.readFileSync(configPath, 'utf8');
       fileConfig = migrateConfig(JSON.parse(rawData));
     } catch (e) {
-      // Ignored, file might be corrupted
     }
   }
 
@@ -104,7 +103,6 @@ async function getConfig(forceWizard = false) {
           console.log(`${i + 1}. ${p}${isActive}`);
         });
         console.log(`\n${savedProviders.length + 1}. Add/Configure a different provider`);
-        
         const c = await askQuestion(`Select (1-${savedProviders.length + 1}): `);
         const idx = parseInt(c) - 1;
 
@@ -185,7 +183,6 @@ async function getConfig(forceWizard = false) {
           provider = providerToConfig;
           const old = config.providers[provider];
           baseUrl = old.baseUrl;
-          
           let defaultModel = old.model;
           const customModel = await askQuestion(`Model name (press Enter for default '${defaultModel}'): `);
           model = customModel.trim() || defaultModel;
@@ -254,12 +251,11 @@ async function getConfig(forceWizard = false) {
             confirmed = true;
             break;
           } else if (confirm === 'n' || confirm === 'no') {
-            break; // Start over loop
+            break;
           }
           console.log("Invalid choice. Please enter 'y' or 'n'.");
         }
       } else {
-        // User just selected an existing provider and didn't want to update it
         config.activeProvider = providerToConfig;
         console.log(`\nSwitched active provider to ${config.activeProvider}.`);
         confirmed = true;