prompt-language-shell 0.8.2 → 0.8.4

package/dist/ui/Schedule.js

@@ -71,7 +71,7 @@ function taskToListItem(task, highlightedChildIndex = null, isDefineTaskWithoutS
     }
     return item;
 }
-export function Schedule({ message, tasks, state, status, debug = DebugLevel.None, handlers, onSelectionConfirmed, }) {
+export function Schedule({ message, tasks, state, status, debug = DebugLevel.None, stateHandlers, lifecycleHandlers, errorHandlers, onSelectionConfirmed, }) {
     const isActive = status === ComponentStatus.Active;
     // isActive passed as prop
     const [highlightedIndex, setHighlightedIndex] = useState(state?.highlightedIndex ?? null);
@@ -95,7 +95,7 @@ export function Schedule({ message, tasks, state, status, debug = DebugLevel.Non
             const concreteTasks = tasks.filter((task) => task.type !== TaskType.Ignore && task.type !== TaskType.Discard);
             // Complete the selection phase - it goes to timeline
             // Callback will create a new Plan showing refined tasks (pending) + Confirm (active)
-            handlers?.completeActive();
+            lifecycleHandlers?.completeActive();
             void onSelectionConfirmed(concreteTasks);
         }
     }, [
@@ -103,7 +103,7 @@ export function Schedule({ message, tasks, state, status, debug = DebugLevel.Non
         defineTaskIndices.length,
         tasks,
         onSelectionConfirmed,
-        handlers,
+        lifecycleHandlers,
     ]);
     useInput((input, key) => {
         // Don't handle input if not active or no define task
@@ -111,7 +111,7 @@ export function Schedule({ message, tasks, state, status, debug = DebugLevel.Non
             return;
         }
         if (key.escape) {
-            handlers?.onAborted('task selection');
+            errorHandlers?.onAborted('task selection');
             return;
         }
         if (key.downArrow) {
@@ -170,12 +170,12 @@ export function Schedule({ message, tasks, state, status, debug = DebugLevel.Non
                 if (onSelectionConfirmed) {
                     // Complete the selection phase - it goes to timeline
                     // Callback will create a new Plan showing refined tasks (pending) + Confirm (active)
-                    handlers?.completeActive();
+                    lifecycleHandlers?.completeActive();
                     void onSelectionConfirmed(refinedTasks);
                 }
                 else {
                     // No selection callback, just complete normally
-                    handlers?.completeActive();
+                    lifecycleHandlers?.completeActive();
                 }
             }
         }
@@ -183,7 +183,7 @@ export function Schedule({ message, tasks, state, status, debug = DebugLevel.Non
     // Sync state back to component definition
     // This ensures timeline can render historical state and tests can validate behavior
     useEffect(() => {
-        handlers?.updateState({
+        stateHandlers?.updateState({
             highlightedIndex,
             currentDefineGroupIndex,
             completedSelections,
@@ -192,7 +192,7 @@ export function Schedule({ message, tasks, state, status, debug = DebugLevel.Non
         highlightedIndex,
         currentDefineGroupIndex,
         completedSelections,
-        handlers,
+        stateHandlers,
     ]);
     const listItems = tasks.map((task, idx) => {
         // Find which define group this task belongs to (if any)

package/dist/ui/Validate.js

@@ -4,7 +4,7 @@ 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, createConfigStepsFromSchema, } from '../services/components.js';
+import { 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';
@@ -13,7 +13,7 @@ import { ensureMinimumTime } from '../services/timing.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, }) {
+export function Validate({ missingConfig, userRequest, state, status, service, children, debug = DebugLevel.None, onError, onComplete, onAborted, stateHandlers, lifecycleHandlers, workflowHandlers, }) {
     const isActive = status === ComponentStatus.Active;
     const [error, setError] = useState(state?.error ?? null);
     const [completionMessage, setCompletionMessage] = useState(state?.completionMessage ?? null);
@@ -39,7 +39,9 @@ export function Validate({ missingConfig, userRequest, state, status, service, c
                 await ensureMinimumTime(startTime, MIN_PROCESSING_TIME);
                 if (mounted) {
                     // Add debug components to timeline if present
-                    addDebugToTimeline(result.debug, handlers);
+                    if (result.debug?.length) {
+                        workflowHandlers?.addToTimeline(...result.debug);
+                    }
                     // Extract CONFIG tasks with descriptions from result
                     const configTasks = result.tasks.filter((task) => task.type === TaskType.Config);
                     // Build ConfigRequirements with descriptions
@@ -68,7 +70,7 @@ export function Validate({ missingConfig, userRequest, state, status, service, c
                     setCompletionMessage(message);
                     setConfigRequirements(withDescriptions);
                     // Save state after validation completes
-                    handlers?.updateState({
+                    stateHandlers?.updateState({
                         completionMessage: message,
                         configRequirements: withDescriptions,
                         validated: true,
@@ -82,7 +84,7 @@ export function Validate({ missingConfig, userRequest, state, status, service, c
                     const errorMessage = formatErrorMessage(err);
                     setError(errorMessage);
                     // Save error state
-                    handlers?.updateState({
+                    stateHandlers?.updateState({
                         error: errorMessage,
                         completionMessage: null,
                         configRequirements: null,
@@ -142,7 +144,7 @@ export function Validate({ missingConfig, userRequest, state, status, service, c
         }
         // Mark validation component as complete before invoking callback
         // This allows the workflow to proceed to execution
-        handlers?.completeActive();
+        lifecycleHandlers?.completeActive();
         // Invoke callback which will queue the Execute component
         if (configRequirements) {
             onComplete(configRequirements);
@@ -150,10 +152,10 @@ export function Validate({ missingConfig, userRequest, state, status, service, c
     };
     const handleConfigAborted = (operation) => {
         // Mark validation component as complete when aborted
-        handlers?.completeActive();
+        lifecycleHandlers?.completeActive();
         onAborted(operation);
     };
-    return (_jsxs(Box, { alignSelf: "flex-start", flexDirection: "column", children: [isActive && !completionMessage && !error && (_jsxs(Box, { marginLeft: 1, children: [_jsxs(Text, { color: getTextColor(isActive), children: ["Validating configuration requirements.", ' '] }), _jsx(Spinner, {})] })), completionMessage && (_jsx(Box, { marginLeft: 1, children: _jsx(Text, { color: getTextColor(isActive), children: completionMessage }) })), error && (_jsx(Box, { marginTop: 1, children: _jsxs(Text, { color: Colors.Status.Error, children: ["Error: ", error] }) })), configSteps && configSteps.length > 0 && !error && (_jsx(Box, { marginTop: 1, children: _jsx(Config, { steps: configSteps, status: status, debug: debug, onFinished: handleConfigFinished, onAborted: handleConfigAborted, handlers: handlers }) })), configSteps && configSteps.length === 0 && !error && (_jsx(Box, { marginTop: 1, marginLeft: 1, children: _jsx(Text, { color: Colors.Status.Error, children: "Error: No configuration steps generated. Please try again." }) })), children] }));
+    return (_jsxs(Box, { alignSelf: "flex-start", flexDirection: "column", children: [isActive && !completionMessage && !error && (_jsxs(Box, { marginLeft: 1, children: [_jsxs(Text, { color: getTextColor(isActive), children: ["Validating configuration requirements.", ' '] }), _jsx(Spinner, {})] })), completionMessage && (_jsx(Box, { marginLeft: 1, children: _jsx(Text, { color: getTextColor(isActive), children: completionMessage }) })), error && (_jsx(Box, { marginTop: 1, children: _jsxs(Text, { color: Colors.Status.Error, children: ["Error: ", error] }) })), configSteps && configSteps.length > 0 && !error && (_jsx(Box, { marginTop: 1, children: _jsx(Config, { steps: configSteps, status: status, debug: debug, onFinished: handleConfigFinished, onAborted: handleConfigAborted }) })), configSteps && configSteps.length === 0 && !error && (_jsx(Box, { marginTop: 1, marginLeft: 1, children: _jsx(Text, { color: Colors.Status.Error, children: "Error: No configuration steps generated. Please try again." }) })), children] }));
 }
 /**
  * Build prompt for VALIDATE tool

package/dist/ui/Workflow.js

@@ -5,6 +5,7 @@ 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 { getWarnings } from '../services/logger.js';
 import { getCancellationMessage } from '../services/messages.js';
 import { exitApp } from '../services/process.js';
 import { Component } from './Component.js';
@@ -35,11 +36,8 @@ export const Workflow = ({ initialQueue, debug }) => {
             return { active: null, pending };
         });
     }, []);
-    // Global handlers for all stateful components
-    const handlers = useMemo(() => ({
-        addToQueue: (...items) => {
-            setQueue((queue) => [...queue, ...items]);
-        },
+    // Focused handler instances - segregated by responsibility
+    const stateHandlers = useMemo(() => ({
         updateState: (newState) => {
             setCurrent((curr) => {
                 const { active, pending } = curr;
@@ -56,12 +54,41 @@ export const Workflow = ({ initialQueue, debug }) => {
                 return { active: updated, pending };
             });
         },
+    }), []);
+    const lifecycleHandlers = useMemo(() => ({
         completeActive: (...items) => {
             moveActiveToPending();
             if (items.length > 0) {
                 setQueue((queue) => [...items, ...queue]);
             }
         },
+    }), [moveActiveToPending]);
+    const queueHandlers = useMemo(() => ({
+        addToQueue: (...items) => {
+            setQueue((queue) => [...queue, ...items]);
+        },
+    }), []);
+    const errorHandlers = useMemo(() => ({
+        onAborted: (operation) => {
+            moveActiveToTimeline();
+            // Add feedback to queue
+            const message = getCancellationMessage(operation);
+            setQueue((queue) => [
+                ...queue,
+                createFeedback(FeedbackType.Aborted, message),
+            ]);
+        },
+        onError: (error) => {
+            moveActiveToTimeline();
+            // Add feedback to queue
+            setQueue((queue) => [
+                ...queue,
+                createFeedback(FeedbackType.Failed, error),
+            ]);
+        },
+    }), [moveActiveToTimeline]);
+    // Workflow handlers - used for timeline/queue management
+    const workflowHandlers = useMemo(() => ({
         completeActiveAndPending: (...items) => {
             setCurrent((curr) => {
                 const { active, pending } = curr;
@@ -83,24 +110,7 @@ export const Workflow = ({ initialQueue, debug }) => {
         addToTimeline: (...items) => {
             setTimeline((prev) => [...prev, ...items]);
         },
-        onAborted: (operation) => {
-            moveActiveToTimeline();
-            // Add feedback to queue
-            const message = getCancellationMessage(operation);
-            setQueue((queue) => [
-                ...queue,
-                createFeedback(FeedbackType.Aborted, message),
-            ]);
-        },
-        onError: (error) => {
-            moveActiveToTimeline();
-            // Add feedback to queue
-            setQueue((queue) => [
-                ...queue,
-                createFeedback(FeedbackType.Failed, error),
-            ]);
-        },
-    }), [moveActiveToPending, moveActiveToTimeline]);
+    }), []);
     // Global Esc handler removed - components handle their own Esc individually
     // Move next item from queue to active
     useEffect(() => {
@@ -138,6 +148,14 @@ export const Workflow = ({ initialQueue, debug }) => {
         }
         // Stateful components stay in active until handlers move them to pending
     }, [current]);
+    // Check for accumulated warnings and add them to timeline
+    useEffect(() => {
+        const warningMessages = getWarnings();
+        if (warningMessages.length > 0) {
+            const warningComponents = warningMessages.map((msg) => markAsDone(createFeedback(FeedbackType.Warning, msg)));
+            setTimeline((prev) => [...prev, ...warningComponents]);
+        }
+    }, [timeline, current]);
     // Move final pending to timeline and exit when all done
     useEffect(() => {
         const { active, pending } = current;
@@ -171,23 +189,77 @@ export const Workflow = ({ initialQueue, debug }) => {
         if (isStateless(active)) {
             return _jsx(Component, { def: active, debug: debug }, active.id);
         }
-        // For stateful components, inject global handlers
+        // For stateful components, inject focused handlers
         const statefulActive = active;
         const wrappedDef = {
             ...statefulActive,
             props: {
                 ...statefulActive.props,
-                handlers,
+                stateHandlers,
+                lifecycleHandlers,
+                queueHandlers,
+                errorHandlers,
+                workflowHandlers,
             },
         };
         return _jsx(Component, { def: wrappedDef, debug: debug }, active.id);
-    }, [current, debug, handlers]);
+    }, [
+        current,
+        debug,
+        stateHandlers,
+        lifecycleHandlers,
+        queueHandlers,
+        errorHandlers,
+        workflowHandlers,
+    ]);
     const pendingComponent = useMemo(() => {
         const { pending } = current;
         if (!pending)
             return null;
-        // Pending components don't receive input
-        return _jsx(Component, { def: pending, debug: debug }, pending.id);
-    }, [current, debug]);
-    return (_jsxs(Box, { flexDirection: "column", children: [_jsx(Static, { items: timeline, children: (item) => (_jsx(Box, { marginTop: 1, children: _jsx(Component, { def: item, debug: DebugLevel.None }) }, item.id)) }, "timeline"), pendingComponent && _jsx(Box, { marginTop: 1, children: pendingComponent }), activeComponent && _jsx(Box, { marginTop: 1, children: activeComponent })] }));
+        // For stateless components, render as-is
+        if (isStateless(pending)) {
+            return _jsx(Component, { def: pending, debug: debug }, pending.id);
+        }
+        // For stateful components, inject focused handlers (they may have useEffect hooks)
+        const statefulPending = pending;
+        const wrappedDef = {
+            ...statefulPending,
+            props: {
+                ...statefulPending.props,
+                stateHandlers,
+                lifecycleHandlers,
+                queueHandlers,
+                errorHandlers,
+                workflowHandlers,
+            },
+        };
+        return _jsx(Component, { def: wrappedDef, debug: debug }, pending.id);
+    }, [
+        current,
+        debug,
+        stateHandlers,
+        lifecycleHandlers,
+        queueHandlers,
+        errorHandlers,
+        workflowHandlers,
+    ]);
+    return (_jsxs(Box, { flexDirection: "column", children: [_jsx(Static, { items: timeline, children: (item) => {
+                    // For stateful timeline components, inject handlers (useEffect hooks may still run)
+                    let def = item;
+                    if (!isStateless(item)) {
+                        const statefulItem = item;
+                        def = {
+                            ...statefulItem,
+                            props: {
+                                ...statefulItem.props,
+                                stateHandlers,
+                                lifecycleHandlers,
+                                queueHandlers,
+                                errorHandlers,
+                                workflowHandlers,
+                            },
+                        };
+                    }
+                    return (_jsx(Box, { marginTop: 1, children: _jsx(Component, { def: def, debug: DebugLevel.None }) }, item.id));
+                } }, "timeline"), pendingComponent && _jsx(Box, { marginTop: 1, children: pendingComponent }), activeComponent && _jsx(Box, { marginTop: 1, children: activeComponent })] }));
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prompt-language-shell",
-  "version": "0.8.2",
+  "version": "0.8.4",
   "description": "Your personal command-line concierge. Ask politely, and it gets things done.",
   "type": "module",
   "main": "dist/index.js",
@@ -51,7 +51,8 @@
     "ink": "^6.6.0",
     "ink-text-input": "^6.0.0",
     "react": "^19.2.3",
-    "yaml": "^2.8.2"
+    "yaml": "^2.8.2",
+    "zod": "^4.2.1"
   },
   "devDependencies": {
     "@types/node": "^25.0.3",