@iservu-inc/adf-cli 0.5.5 → 0.6.0

package/CHANGELOG.md

@@ -5,6 +5,130 @@ All notable changes to `@iservu-inc/adf-cli` will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.6.0] - 2025-10-05
+
+### 🎯 Fixed Cursor Positioning with Editor Input
+
+**FIX:** Switched from inline input to editor-based input to fix cursor lag on multi-line answers.
+
+#### The Problem
+
+When typing long answers in the terminal, the text cursor would fall behind by 3-4 characters when wrapping to a second line:
+
+![Cursor Lag Issue](https://user-images.githubusercontent.com/...)
+- Text wraps to multiple lines
+- Cursor position becomes desynchronized
+- Hard to track where you're typing
+- Frustrating UX for detailed answers
+
+This is a known issue with `inquirer`'s `input` type and terminal line wrapping.
+
+#### The Solution
+
+Switched to `editor` type for all answers:
+- Opens your system's default text editor (Notepad, nano, vi, etc.)
+- Perfect for multi-line, detailed answers
+- No cursor positioning issues
+- Better editing experience (copy/paste, formatting, etc.)
+
+**New Workflow:**
+1. Question is displayed
+2. Your editor automatically opens
+3. Type your answer (multi-line, formatted, etc.)
+4. Save and close the editor
+5. Answer is captured and interview continues
+
+**To skip a question:** Leave the editor empty and save.
+
+#### Benefits
+
+- ✅ **No more cursor lag** - Editor handles display properly
+- ✅ **Better for long answers** - Multi-line editing is natural
+- ✅ **Standard CLI pattern** - Used by git, crontab, etc.
+- ✅ **Copy/paste friendly** - Full editor capabilities
+- ✅ **Markdown support** - `.md` extension for syntax highlighting
+
+#### Technical Changes
+
+- Modified: `lib/frameworks/interviewer.js`
+  - Changed answer input from `type: 'input'` to `type: 'editor'`
+  - Changed follow-up input to `type: 'editor'`
+  - Added clear instructions about editor workflow
+  - Empty answers treated as skips
+  - Updated guidance text
+
+#### User Experience
+
+**Before (inline):**
+```
+Your answer: The primary use action is to eng█ge...
+[cursor lags behind when wrapping]
+```
+
+**After (editor):**
+```
+(Your editor will open. Save and close to continue, or leave empty + save to skip)
+[Editor opens]
+[Type answer with full multi-line support]
+[Save & close]
+✓ Answer captured
+```
+
+---
+
+## [0.5.6] - 2025-10-05
+
+### 🔧 Fixed Invalid .env State Detection
+
+**FIX:** Automatically detect and reset invalid configuration where ADF_CURRENT_PROVIDER/ADF_CURRENT_MODEL are set but no API key exists.
+
+#### The Problem (Cart Before the Horse)
+
+The .env file could end up in an invalid state:
+```env
+ADF_CURRENT_PROVIDER="openrouter"
+ADF_CURRENT_MODEL="anthropic/claude-sonnet-4-5"
+# But no OPENROUTER_API_KEY present!
+```
+
+This shouldn't be possible - you can't have a "current provider" without an API key for that provider.
+
+#### The Fix
+
+Now we detect this broken state and automatically reset:
+```
+⚠️  Invalid configuration detected:
+   ADF_CURRENT_PROVIDER is set to "openrouter" but no API key found
+   Resetting provider selection...
+```
+
+The invalid ADF_CURRENT_PROVIDER and ADF_CURRENT_MODEL entries are removed from .env, forcing proper configuration.
+
+#### Why This Matters
+
+- **Data integrity**: Prevents cart-before-horse situations
+- **Better UX**: Clear feedback about what's wrong
+- **Auto-healing**: Automatically fixes the broken state
+- **Prevents confusion**: Users aren't told they have an active provider when they don't
+
+#### How It Can Happen
+
+- User manually edited/deleted API key from .env
+- Corrupted .env file
+- Configuration error during setup
+
+Now these scenarios are automatically detected and corrected!
+
+#### Technical Changes
+
+- Modified: `lib/ai/ai-config.js`
+  - Added validation after loading .env
+  - New `saveEnvFile()` helper function
+  - Refactored `saveToEnvFile()` to use `saveEnvFile()`
+  - Auto-reset invalid configuration
+
+---
+
 ## [0.5.5] - 2025-10-05
 
 ### ✅ Fixed AI Provider Configuration Display

package/lib/ai/ai-config.js

@@ -136,12 +136,9 @@ async function ensureGitignore(projectPath) {
 }
 
 /**
- * Save API key to .env file
+ * Save entire env object to .env file
  */
-async function saveToEnvFile(envPath, key, value) {
-  const env = await loadEnvFile(envPath);
-  env[key] = value;
-
+async function saveEnvFile(envPath, env) {
   const lines = [
     '# AI Provider Configuration for adf-cli',
     '# DO NOT commit this file to version control',
@@ -179,6 +176,15 @@ async function saveToEnvFile(envPath, key, value) {
   await ensureGitignore(projectPath);
 }
 
+/**
+ * Save API key to .env file
+ */
+async function saveToEnvFile(envPath, key, value) {
+  const env = await loadEnvFile(envPath);
+  env[key] = value;
+  await saveEnvFile(envPath, env);
+}
+
 /**
  * Load .env file into process.env
  */
@@ -279,8 +285,29 @@ async function configureAIProvider(projectPath = process.cwd()) {
   }
 
   // Get currently active provider and model
-  const currentProvider = process.env.ADF_CURRENT_PROVIDER || existingEnv.ADF_CURRENT_PROVIDER;
-  const currentModel = process.env.ADF_CURRENT_MODEL || existingEnv.ADF_CURRENT_MODEL;
+  let currentProvider = process.env.ADF_CURRENT_PROVIDER || existingEnv.ADF_CURRENT_PROVIDER;
+  let currentModel = process.env.ADF_CURRENT_MODEL || existingEnv.ADF_CURRENT_MODEL;
+
+  // Validate: If ADF_CURRENT_PROVIDER is set but its API key is missing, reset it
+  if (currentProvider) {
+    const currentProviderObj = AI_PROVIDERS[currentProvider.toUpperCase()];
+    if (currentProviderObj) {
+      const currentProviderHasKey = availableProviders.find(p => p.id === currentProvider);
+      if (!currentProviderHasKey) {
+        console.log(chalk.yellow(`⚠️  Invalid configuration detected:`));
+        console.log(chalk.gray(`   ADF_CURRENT_PROVIDER is set to "${currentProvider}" but no API key found`));
+        console.log(chalk.gray(`   Resetting provider selection...\n`));
+
+        // Clear invalid values from .env file
+        delete existingEnv.ADF_CURRENT_PROVIDER;
+        delete existingEnv.ADF_CURRENT_MODEL;
+        await saveEnvFile(envPath, existingEnv);
+
+        currentProvider = null;
+        currentModel = null;
+      }
+    }
+  }
 
   if (availableProviders.length > 0) {
     console.log(chalk.green('✓ Detected API keys for:'));
@@ -290,7 +317,7 @@ async function configureAIProvider(projectPath = process.cwd()) {
     console.log('');
   }
 
-  // Show current active configuration if exists
+  // Show current active configuration if exists (and is valid)
   if (currentProvider && currentModel) {
     const currentProviderObj = AI_PROVIDERS[currentProvider.toUpperCase()];
     if (currentProviderObj) {

package/lib/frameworks/interviewer.js

@@ -559,33 +559,33 @@ class Interviewer {
     console.log(chalk.gray(`💡 ${question.guidance}`));
     console.log(chalk.green(`   ✓ Good: ${question.goodExample}`));
     console.log(chalk.red(`   ✗ Bad: ${question.badExample}`));
-    console.log(chalk.gray(`   (Type "skip" to skip remaining questions in this block)\n`));
 
     // Start tracking time for this question
     if (this.skipTracker) {
       this.skipTracker.startQuestion(question.id);
     }
 
-    // Get answer
+    // Get answer using editor for better multi-line support
+    console.log(chalk.gray('   (Your editor will open. Save and close to continue, or leave empty + save to skip)'));
+
     const { answer } = await inquirer.prompt([
       {
-        type: 'input',
+        type: 'editor',
         name: 'answer',
-        message: 'Your answer:',
-        validate: (input) => {
-          if (!input || input.trim().length === 0) {
-            return 'Please provide an answer or type "skip"';
-          }
-          return true;
-        }
+        message: '',
+        waitForUseInput: false,
+        postfix: '.md',
+        default: ''
       }
     ]);
 
-    if (answer.toLowerCase() === 'skip') {
+    // Handle empty answer or "skip" as skip
+    if (!answer || answer.trim().length === 0 || answer.toLowerCase().trim() === 'skip') {
       // Record skip event
       if (this.skipTracker) {
         this.skipTracker.recordSkip(question, 'manual');
       }
+      console.log(chalk.gray('\n⏭️  Skipped\n'));
       return null; // Signal to skip remaining questions
     }
 
@@ -671,13 +671,18 @@ class Interviewer {
     }
 
     if (followUp) {
-      console.log(chalk.yellow(`\n🤖 ${followUp.message}\n`));
+      console.log(chalk.yellow(`\n🤖 ${followUp.message}`));
+      console.log(chalk.yellow(`   ${followUp.question}`));
+      console.log(chalk.gray('   (Your editor will open for follow-up answer)\n'));
 
       const { followUpAnswer } = await inquirer.prompt([
         {
-          type: 'input',
+          type: 'editor',
           name: 'followUpAnswer',
-          message: followUp.question
+          message: '',
+          waitForUseInput: false,
+          postfix: '.md',
+          default: ''
         }
       ]);
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@iservu-inc/adf-cli",
-  "version": "0.5.5",
+  "version": "0.6.0",
   "description": "CLI tool for AgentDevFramework - AI-assisted development framework with multi-provider AI support",
   "main": "index.js",
   "bin": {