prompt-language-shell 0.7.8 → 0.8.2

package/dist/ui/Validate.js

@@ -4,35 +4,30 @@ import { Box, Text } from 'ink';
 import { ComponentStatus } from '../types/components.js';
 import { TaskType } from '../types/types.js';
 import { Colors, getTextColor } from '../services/colors.js';
-import { addDebugToTimeline } from '../services/components.js';
+import { addDebugToTimeline, createConfigStepsFromSchema, } from '../services/components.js';
+import { DebugLevel, saveConfig, unflattenConfig, } from '../services/configuration.js';
+import { saveConfigLabels } from '../services/config-labels.js';
 import { useInput } from '../services/keyboard.js';
 import { formatErrorMessage } from '../services/messages.js';
 import { ensureMinimumTime } from '../services/timing.js';
-import { DebugLevel, saveConfig, unflattenConfig, } from '../services/configuration.js';
-import { Config, StepType } from './Config.js';
+import { Config } from './Config.js';
 import { Spinner } from './Spinner.js';
 const MIN_PROCESSING_TIME = 1000;
 export function Validate({ missingConfig, userRequest, state, status, service, children, debug = DebugLevel.None, onError, onComplete, onAborted, handlers, }) {
     const isActive = status === ComponentStatus.Active;
-    const [error, setError] = useState(null);
-    const [completionMessage, setCompletionMessage] = useState(null);
-    const [configRequirements, setConfigRequirements] = useState(null);
-    const [showConfig, setShowConfig] = useState(false);
+    const [error, setError] = useState(state?.error ?? null);
+    const [completionMessage, setCompletionMessage] = useState(state?.completionMessage ?? null);
+    const [configRequirements, setConfigRequirements] = useState(state?.configRequirements ?? null);
     useInput((_, key) => {
-        if (key.escape && isActive && !showConfig) {
+        if (key.escape && isActive) {
             onAborted('validation');
         }
-    }, { isActive: isActive && !showConfig });
+    }, { isActive });
     useEffect(() => {
         // Skip processing if not active
         if (!isActive) {
             return;
         }
-        // Skip processing if no service available
-        if (!service) {
-            setError('No service available');
-            return;
-        }
         let mounted = true;
         async function process(svc) {
             const startTime = Date.now();
@@ -74,8 +69,10 @@ export function Validate({ missingConfig, userRequest, state, status, service, c
                     setConfigRequirements(withDescriptions);
                     // Save state after validation completes
                     handlers?.updateState({
+                        completionMessage: message,
                         configRequirements: withDescriptions,
                         validated: true,
+                        error: null,
                     });
                 }
             }
@@ -83,20 +80,19 @@ export function Validate({ missingConfig, userRequest, state, status, service, c
                 await ensureMinimumTime(startTime, MIN_PROCESSING_TIME);
                 if (mounted) {
                     const errorMessage = formatErrorMessage(err);
+                    setError(errorMessage);
                     // Save error state
                     handlers?.updateState({
                         error: errorMessage,
+                        completionMessage: null,
+                        configRequirements: null,
+                        validated: false,
                     });
-                    if (onError) {
-                        onError(errorMessage);
-                    }
-                    else {
-                        setError(errorMessage);
-                    }
+                    onError(errorMessage);
                 }
             }
         }
-        process(service);
+        void process(service);
         return () => {
             mounted = false;
         };
@@ -113,20 +109,33 @@ export function Validate({ missingConfig, userRequest, state, status, service, c
     if (!isActive && !completionMessage && !error && !children) {
         return null;
     }
-    // Create ConfigSteps from requirements
+    // Create ConfigSteps from requirements using createConfigStepsFromSchema
+    // to load current values from config file, then override descriptions
     const configSteps = configRequirements
-        ? configRequirements.map((req) => ({
-            description: req.description || req.path,
-            key: req.path,
-            path: req.path,
-            type: StepType.Text,
-            value: null,
-            validate: () => true,
-        }))
+        ? (() => {
+            const keys = configRequirements.map((req) => req.path);
+            const steps = createConfigStepsFromSchema(keys);
+            // Override descriptions with LLM-generated ones
+            return steps.map((step, index) => ({
+                ...step,
+                description: configRequirements[index].description ||
+                    configRequirements[index].path,
+            }));
+        })()
         : null;
     const handleConfigFinished = (config) => {
         // Convert flat dotted keys to nested structure grouped by section
         const configBySection = unflattenConfig(config);
+        // Extract and save labels to cache
+        if (configRequirements) {
+            const labels = {};
+            for (const req of configRequirements) {
+                if (req.description) {
+                    labels[req.path] = req.description;
+                }
+            }
+            saveConfigLabels(labels);
+        }
         // Save each section
         for (const [section, sectionConfig] of Object.entries(configBySection)) {
             saveConfig(section, sectionConfig);
@@ -135,7 +144,9 @@ export function Validate({ missingConfig, userRequest, state, status, service, c
         // This allows the workflow to proceed to execution
         handlers?.completeActive();
         // Invoke callback which will queue the Execute component
-        onComplete?.(configRequirements);
+        if (configRequirements) {
+            onComplete(configRequirements);
+        }
     };
     const handleConfigAborted = (operation) => {
         // Mark validation component as complete when aborted

package/dist/ui/Workflow.js

@@ -5,8 +5,8 @@ import { ComponentStatus, } from '../types/components.js';
 import { ComponentName, FeedbackType } from '../types/types.js';
 import { createFeedback, isStateless, markAsDone, } from '../services/components.js';
 import { DebugLevel } from '../services/configuration.js';
-import { exitApp } from '../services/process.js';
 import { getCancellationMessage } from '../services/messages.js';
+import { exitApp } from '../services/process.js';
 import { Component } from './Component.js';
 export const Workflow = ({ initialQueue, debug }) => {
     const [timeline, setTimeline] = useState([]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prompt-language-shell",
-  "version": "0.7.8",
+  "version": "0.8.2",
   "description": "Your personal command-line concierge. Ask politely, and it gets things done.",
   "type": "module",
   "main": "dist/index.js",
@@ -47,22 +47,22 @@
   },
   "homepage": "https://github.com/aswitalski/pls#readme",
   "dependencies": {
-    "@anthropic-ai/sdk": "^0.70.1",
-    "ink": "^6.5.1",
+    "@anthropic-ai/sdk": "^0.71.2",
+    "ink": "^6.6.0",
     "ink-text-input": "^6.0.0",
-    "react": "^19.2.0",
-    "yaml": "^2.8.1"
+    "react": "^19.2.3",
+    "yaml": "^2.8.2"
   },
   "devDependencies": {
-    "@types/node": "^24.10.1",
-    "@types/react": "^19.2.6",
-    "@vitest/coverage-v8": "^4.0.12",
-    "eslint": "^9.39.1",
+    "@types/node": "^25.0.3",
+    "@types/react": "^19.2.7",
+    "@vitest/coverage-v8": "^4.0.16",
+    "eslint": "^9.39.2",
     "husky": "^9.1.7",
     "ink-testing-library": "^4.0.0",
-    "prettier": "^3.6.2",
+    "prettier": "^3.7.4",
     "typescript": "^5.9.3",
-    "typescript-eslint": "^8.47.0",
-    "vitest": "^4.0.12"
+    "typescript-eslint": "^8.50.1",
+    "vitest": "^4.0.16"
   }
 }