prompt-language-shell 0.6.6 → 0.7.0

package/dist/{config/VALIDATE.md → skills/validate.md}

@@ -1,8 +1,12 @@
 ## Overview
 
-You are the validation component of "pls" (please), responsible for validating skill requirements and generating natural language descriptions for missing configuration values.
+You are the validation component of "pls" (please), responsible for
+validating skill requirements and generating natural language descriptions
+for missing configuration values.
 
-Your role is to help users understand what configuration values are needed and why, using context from skill descriptions to create clear, helpful prompts.
+Your role is to help users understand what configuration values are needed
+and why, using context from skill descriptions to create clear, helpful
+prompts.
 
 ## Input
 
@@ -21,18 +25,22 @@ Generate a response with two required fields:
 
 For each CONFIG task, create a natural language description that:
 
-1. **Explains what the value is for** using context from the skill's description
+1. **Explains what the value is for** using context from the skill's
+   description
 2. **Keeps it SHORT** - one brief phrase (3-6 words max)
-3. **Does NOT include the config path** - the path will be shown separately in debug mode
+3. **Does NOT include the config path** - the path will be shown
+   separately in debug mode
 
-**CRITICAL**: You MUST include both the `message` field (set to empty string) and the `tasks` array in your response.
+**CRITICAL**: You MUST include both the `message` field (set to empty
+string) and the `tasks` array in your response.
 
 ## Description Format
 
 **Format:** "Brief description" (DO NOT include {config.path}!)
 
 The description should:
-- Start with what the config value represents (e.g., "Path to...", "URL for...", "Name of...")
+- Start with what the config value represents (e.g., "Path to...", "URL
+  for...", "Name of...")
 - Be SHORT and direct - no extra details or variant explanations
 - NEVER include the config path in curly brackets like {config.path}
 
@@ -97,11 +105,16 @@ tasks: [
 
 ## Guidelines
 
-1. **Use skill context**: Read the skill's Description section to understand what the variant represents
-2. **Be specific**: Don't just say "Repository path" - say "Path to Alpha repository"
-3. **Add helpful details**: Include information from the description when relevant
-4. **Keep it concise**: One brief phrase that clearly explains what's needed
-5. **Never include the path**: Do not append `{config.path}` - it's shown separately in debug mode
+1. **Use skill context**: Read the skill's Description section to
+   understand what the variant represents
+2. **Be specific**: Don't just say "Repository path" - say "Path to Alpha
+   repository"
+3. **Add helpful details**: Include information from the description when
+   relevant
+4. **Keep it concise**: One brief phrase that clearly explains what's
+   needed
+5. **Never include the path**: Do not append `{config.path}` - it's shown
+   separately in debug mode
 
 ## Common Config Types
 
@@ -137,4 +150,5 @@ tasks: [
 - Descriptions should be helpful and contextual, not just technical
 - Use information from Available Skills section to provide context
 - Keep descriptions to one brief phrase (3-6 words)
-- NEVER include the config path in the action/description - it's shown separately
+- NEVER include the config path in the action/description - it's shown
+  separately

package/dist/types/types.js

@@ -3,6 +3,7 @@ export var ComponentName;
     ComponentName["Welcome"] = "welcome";
     ComponentName["Config"] = "config";
     ComponentName["Message"] = "message";
+    ComponentName["Debug"] = "debug";
     ComponentName["Command"] = "command";
     ComponentName["Plan"] = "plan";
     ComponentName["Refinement"] = "refinement";

package/dist/ui/Command.js

@@ -8,7 +8,7 @@ import { createPlanDefinition } from '../services/components.js';
 import { formatErrorMessage } from '../services/messages.js';
 import { useInput } from '../services/keyboard.js';
 import { handleRefinement } from '../services/refinement.js';
-import { routeTasksWithConfirm } from '../services/task-router.js';
+import { routeTasksWithConfirm } from '../services/router.js';
 import { ensureMinimumTime } from '../services/timing.js';
 import { Spinner } from './Spinner.js';
 import { UserQuery } from './UserQuery.js';
@@ -48,6 +48,10 @@ export function Command({ command, state, status, service, handlers, onAborted,
                 }
                 await ensureMinimumTime(startTime, MIN_PROCESSING_TIME);
                 if (mounted) {
+                    // Add debug components to timeline if present
+                    if (result.debug && result.debug.length > 0) {
+                        handlers?.addToTimeline(...result.debug);
+                    }
                     // Save result to state for timeline display
                     handlers?.updateState({
                         message: result.message,

package/dist/ui/Component.js

@@ -5,6 +5,7 @@ import { Answer } from './Answer.js';
 import { Command } from './Command.js';
 import { Confirm } from './Confirm.js';
 import { Config } from './Config.js';
+import { Debug } from './Debug.js';
 import { Execute } from './Execute.js';
 import { Feedback } from './Feedback.js';
 import { Introspect } from './Introspect.js';
@@ -28,6 +29,8 @@ export const Component = memo(function Component({ def, debug, }) {
             return _jsx(Feedback, { ...def.props, status: def.status });
         case ComponentName.Message:
             return _jsx(Message, { ...def.props, status: def.status });
+        case ComponentName.Debug:
+            return _jsx(Debug, { ...def.props, status: def.status });
         case ComponentName.Refinement:
             return (_jsx(Refinement, { ...def.props, state: def.state, status: def.status }));
         case ComponentName.Confirm:

package/dist/ui/Config.js

@@ -4,6 +4,7 @@ import { Box, Text, useFocus } from 'ink';
 import TextInput from 'ink-text-input';
 import { ComponentStatus } from '../types/components.js';
 import { Colors } from '../services/colors.js';
+import { DebugLevel } from '../services/configuration.js';
 import { useInput } from '../services/keyboard.js';
 export var StepType;
 (function (StepType) {
@@ -58,7 +59,7 @@ function SelectionStep({ options, selectedIndex, isCurrentStep, }) {
             return (_jsx(Box, { marginRight: 2, children: _jsx(Text, { dimColor: !isSelected || !isCurrentStep, bold: isSelected, children: option.label }) }, option.value));
         }) }));
 }
-export function Config({ steps, state, status, debug, handlers, onFinished, onAborted, }) {
+export function Config({ steps, state, status, debug = DebugLevel.None, handlers, onFinished, onAborted, }) {
     const isActive = status === ComponentStatus.Active;
     const [step, setStep] = useState(!isActive ? (state?.completedStep ?? steps.length) : 0);
     const [values, setValues] = useState(() => {
@@ -243,6 +244,6 @@ export function Config({ steps, state, status, debug, handlers, onFinished, onAb
             if (!shouldShow) {
                 return null;
             }
-            return (_jsxs(Box, { flexDirection: "column", marginTop: index === 0 ? 0 : 1, children: [_jsxs(Box, { children: [_jsx(Text, { children: stepConfig.description }), _jsx(Text, { children: ": " }), debug && stepConfig.path && (_jsxs(Text, { color: Colors.Type.Define, children: ['{', stepConfig.path, '}'] }))] }), _jsxs(Box, { children: [_jsx(Text, { children: " " }), _jsx(Text, { color: Colors.Action.Select, dimColor: !isCurrentStep, children: ">" }), _jsx(Text, { children: " " }), renderStepInput(stepConfig, isCurrentStep)] })] }, stepConfig.path || stepConfig.key));
+            return (_jsxs(Box, { flexDirection: "column", marginTop: index === 0 ? 0 : 1, children: [_jsxs(Box, { children: [_jsx(Text, { children: stepConfig.description }), _jsx(Text, { children: ": " }), debug !== DebugLevel.None && stepConfig.path && (_jsxs(Text, { color: Colors.Type.Define, children: ['{', stepConfig.path, '}'] }))] }), _jsxs(Box, { children: [_jsx(Text, { children: " " }), _jsx(Text, { color: Colors.Action.Select, dimColor: !isCurrentStep, children: ">" }), _jsx(Text, { children: " " }), renderStepInput(stepConfig, isCurrentStep)] })] }, stepConfig.path || stepConfig.key));
         }) }));
 }

package/dist/ui/Debug.js

@@ -0,0 +1,8 @@
+import { jsx as _jsx, jsxs as _jsxs } from "react/jsx-runtime";
+import { Box, Text } from 'ink';
+const MIN_CONTENT_WIDTH = 80;
+const HORIZONTAL_PADDING = 2;
+const BORDER_WIDTH = 1;
+export const Debug = ({ title, content, color }) => {
+    return (_jsxs(Box, { flexDirection: "column", paddingX: HORIZONTAL_PADDING, paddingY: 1, borderStyle: "single", borderColor: color, alignSelf: "flex-start", minWidth: MIN_CONTENT_WIDTH + 2 * HORIZONTAL_PADDING + 2 * BORDER_WIDTH, children: [_jsx(Text, { color: color, children: title }), _jsx(Text, { color: color, children: content })] }));
+};

package/dist/ui/Execute.js

@@ -7,8 +7,8 @@ import { useInput } from '../services/keyboard.js';
 import { formatErrorMessage } from '../services/messages.js';
 import { formatDuration } from '../services/utils.js';
 import { ExecutionStatus, executeCommands, } from '../services/shell.js';
-import { replacePlaceholders } from '../services/placeholder-resolver.js';
-import { loadUserConfig } from '../services/config-loader.js';
+import { replacePlaceholders } from '../services/resolver.js';
+import { loadUserConfig } from '../services/loader.js';
 import { ensureMinimumTime } from '../services/timing.js';
 import { Spinner } from './Spinner.js';
 const MINIMUM_PROCESSING_TIME = 400;

package/dist/ui/Introspect.js

@@ -4,6 +4,7 @@ import { Box, Text } from 'ink';
 import { ComponentStatus, } from '../types/components.js';
 import { Colors, getTextColor } from '../services/colors.js';
 import { createReportDefinition } from '../services/components.js';
+import { DebugLevel } from '../services/configuration.js';
 import { useInput } from '../services/keyboard.js';
 import { formatErrorMessage } from '../services/messages.js';
 import { ensureMinimumTime } from '../services/timing.js';
@@ -51,7 +52,7 @@ function parseCapabilityFromTask(task) {
         isIncomplete,
     };
 }
-export function Introspect({ tasks, state, status, service, children, debug = false, handlers, }) {
+export function Introspect({ tasks, state, status, service, children, debug = DebugLevel.None, handlers, }) {
     const isActive = status === ComponentStatus.Active;
     // isActive passed as prop
     const [error, setError] = useState(null);
@@ -83,7 +84,7 @@ export function Introspect({ tasks, state, status, service, children, debug = fa
                     // Parse capabilities from returned tasks
                     let capabilities = result.tasks.map(parseCapabilityFromTask);
                     // Filter out internal capabilities when not in debug mode
-                    if (!debug) {
+                    if (debug === DebugLevel.None) {
                         capabilities = capabilities.filter((cap) => cap.name.toUpperCase() !== 'PLAN' &&
                             cap.name.toUpperCase() !== 'VALIDATE' &&
                             cap.name.toUpperCase() !== 'REPORT');

package/dist/ui/Main.js

@@ -3,18 +3,32 @@ import { useEffect, useState } from 'react';
 import { FeedbackType } from '../types/types.js';
 import { createAnthropicService, } from '../services/anthropic.js';
 import { createCommandDefinition, createConfigDefinitionWithKeys, createFeedback, createMessage, createWelcomeDefinition, } from '../services/components.js';
-import { getConfigurationRequiredMessage, getMissingConfigKeys, loadConfig, loadDebugSetting, saveAnthropicConfig, saveDebugSetting, } from '../services/configuration.js';
+import { DebugLevel, getConfigurationRequiredMessage, getMissingConfigKeys, loadConfig, loadDebugSetting, saveAnthropicConfig, saveDebugSetting, } from '../services/configuration.js';
 import { registerGlobalShortcut } from '../services/keyboard.js';
+import { initializeLogger, setDebugLevel } from '../services/logger.js';
 import { Workflow } from './Workflow.js';
 export const Main = ({ app, command }) => {
     const [service, setService] = useState(null);
     const [initialQueue, setInitialQueue] = useState(null);
-    const [isDebug, setIsDebug] = useState(() => loadDebugSetting());
+    const [debug, setDebugLevelState] = useState(() => loadDebugSetting());
+    // Initialize logger on mount
+    useEffect(() => {
+        initializeLogger();
+    }, []);
+    // Update logger when debug level changes
+    useEffect(() => {
+        setDebugLevel(debug);
+    }, [debug]);
     // Register global keyboard shortcuts
     useEffect(() => {
         registerGlobalShortcut('shift+tab', () => {
-            setIsDebug((prev) => {
-                const newValue = !prev;
+            setDebugLevelState((prev) => {
+                // Cycle through: None -> Info -> Verbose -> None
+                const newValue = prev === DebugLevel.None
+                    ? DebugLevel.Info
+                    : prev === DebugLevel.Info
+                        ? DebugLevel.Verbose
+                        : DebugLevel.None;
                 saveDebugSetting(newValue);
                 return newValue;
             });
@@ -90,5 +104,5 @@ export const Main = ({ app, command }) => {
     if (initialQueue === null) {
         return null;
     }
-    return _jsx(Workflow, { initialQueue: initialQueue, debug: isDebug });
+    return _jsx(Workflow, { initialQueue: initialQueue, debug: debug });
 };

package/dist/ui/Plan.js

@@ -3,6 +3,7 @@ import { useEffect, useState } from 'react';
 import { Box } from 'ink';
 import { ComponentStatus } from '../types/components.js';
 import { TaskType } from '../types/types.js';
+import { DebugLevel } from '../services/configuration.js';
 import { getTaskColors } from '../services/colors.js';
 import { useInput } from '../services/keyboard.js';
 import { Label } from './Label.js';
@@ -50,7 +51,7 @@ function taskToListItem(task, highlightedChildIndex = null, isDefineTaskWithoutS
     }
     return item;
 }
-export function Plan({ message, tasks, state, status, debug = false, handlers, onSelectionConfirmed, }) {
+export function Plan({ message, tasks, state, status, debug = DebugLevel.None, handlers, onSelectionConfirmed, }) {
     const isActive = status === ComponentStatus.Active;
     // isActive passed as prop
     const [highlightedIndex, setHighlightedIndex] = useState(state?.highlightedIndex ?? null);
@@ -201,5 +202,5 @@ export function Plan({ message, tasks, state, status, debug = false, handlers, o
             isActive;
         return taskToListItem(task, childIndex, isDefineWithoutSelection, isActive);
     });
-    return (_jsxs(Box, { flexDirection: "column", children: [message && (_jsx(Box, { marginBottom: 1, marginLeft: 1, children: _jsx(Label, { description: message, taskType: TaskType.Plan, showType: debug, isCurrent: isActive }) })), _jsx(Box, { marginLeft: 1, children: _jsx(List, { items: listItems, highlightedIndex: currentDefineTaskIndex >= 0 ? highlightedIndex : null, highlightedParentIndex: currentDefineTaskIndex, showType: debug }) })] }));
+    return (_jsxs(Box, { flexDirection: "column", children: [message && (_jsx(Box, { marginBottom: 1, marginLeft: 1, children: _jsx(Label, { description: message, taskType: TaskType.Plan, showType: debug !== DebugLevel.None, isCurrent: isActive }) })), _jsx(Box, { marginLeft: 1, children: _jsx(List, { items: listItems, highlightedIndex: currentDefineTaskIndex >= 0 ? highlightedIndex : null, highlightedParentIndex: currentDefineTaskIndex, showType: debug !== DebugLevel.None }) })] }));
 }

package/dist/ui/Validate.js

@@ -7,11 +7,11 @@ import { Colors, getTextColor } from '../services/colors.js';
 import { useInput } from '../services/keyboard.js';
 import { formatErrorMessage } from '../services/messages.js';
 import { ensureMinimumTime } from '../services/timing.js';
-import { saveConfig, unflattenConfig } from '../services/configuration.js';
+import { DebugLevel, saveConfig, unflattenConfig, } from '../services/configuration.js';
 import { Config, StepType } from './Config.js';
 import { Spinner } from './Spinner.js';
 const MIN_PROCESSING_TIME = 1000;
-export function Validate({ missingConfig, userRequest, state, status, service, children, debug, onError, onComplete, onAborted, handlers, }) {
+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);
@@ -128,9 +128,15 @@ export function Validate({ missingConfig, userRequest, state, status, service, c
         for (const [section, sectionConfig] of Object.entries(configBySection)) {
             saveConfig(section, sectionConfig);
         }
+        // Mark validation component as complete before invoking callback
+        // This allows the workflow to proceed to execution
+        handlers?.completeActive();
+        // Invoke callback which will queue the Execute component
         onComplete?.(configRequirements);
     };
     const handleConfigAborted = (operation) => {
+        // Mark validation component as complete when aborted
+        handlers?.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 && !error && (_jsx(Box, { marginTop: 1, children: _jsx(Config, { steps: configSteps, status: status, debug: debug, onFinished: handleConfigFinished, onAborted: handleConfigAborted, handlers: handlers }) })), children] }));

package/dist/ui/Workflow.js

@@ -4,6 +4,7 @@ import { Box, Static } from 'ink';
 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 { Component } from './Component.js';
@@ -188,5 +189,5 @@ export const Workflow = ({ initialQueue, debug }) => {
         // 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: false }) }, item.id)) }, "timeline"), pendingComponent && _jsx(Box, { marginTop: 1, children: pendingComponent }), activeComponent && _jsx(Box, { marginTop: 1, children: activeComponent })] }));
+    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 })] }));
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prompt-language-shell",
-  "version": "0.6.6",
+  "version": "0.7.0",
   "description": "Your personal command-line concierge. Ask politely, and it gets things done.",
   "type": "module",
   "main": "dist/index.js",
@@ -13,7 +13,7 @@
   "scripts": {
     "clean": "rm -rf dist",
     "typecheck": "tsc --project tsconfig.eslint.json",
-    "build": "npm run typecheck && npm run clean && tsc && chmod +x dist/index.js && mkdir -p dist/config && cp src/config/*.md dist/config/",
+    "build": "npm run typecheck && npm run clean && tsc && chmod +x dist/index.js && mkdir -p dist/skills && cp src/skills/*.md dist/skills/",
     "dev": "npm run build && tsc --watch",
     "prepare": "husky",
     "prepublishOnly": "npm run check",

package/dist/services/skill-expander.js

@@ -1,87 +0,0 @@
-/**
- * Parse skill reference from execution line
- * Returns skill name if line is a reference, otherwise null
- */
-export function parseSkillReference(line) {
-    const match = line.match(/^\[(.+)\]$/);
-    return match ? match[1].trim() : null;
-}
-/**
- * Check if execution line is a skill reference
- */
-export function isSkillReference(line) {
-    return /^\[.+\]$/.test(line.trim());
-}
-/**
- * Expand skill references in execution commands
- * Returns expanded execution lines with references replaced
- * Throws error if circular reference detected
- */
-export function expandSkillReferences(execution, skillLookup, visited = new Set()) {
-    const expanded = [];
-    for (const line of execution) {
-        const skillName = parseSkillReference(line);
-        if (!skillName) {
-            // Not a reference, keep as-is
-            expanded.push(line);
-            continue;
-        }
-        // Check for circular reference
-        if (visited.has(skillName)) {
-            throw new Error(`Circular skill reference detected: ${Array.from(visited).join(' → ')} → ${skillName}`);
-        }
-        // Look up referenced skill
-        const skill = skillLookup(skillName);
-        if (!skill) {
-            // Referenced skill not found, keep as-is
-            expanded.push(line);
-            continue;
-        }
-        // Recursively expand referenced skill's execution
-        const newVisited = new Set(visited);
-        newVisited.add(skillName);
-        const referencedExecution = expandSkillReferences(skill.execution, skillLookup, newVisited);
-        expanded.push(...referencedExecution);
-    }
-    return expanded;
-}
-/**
- * Get all skill names referenced in execution (including nested)
- * Returns unique set of skill names
- */
-export function getReferencedSkills(execution, skillLookup, visited = new Set()) {
-    const referenced = new Set();
-    for (const line of execution) {
-        const skillName = parseSkillReference(line);
-        if (!skillName || visited.has(skillName)) {
-            continue;
-        }
-        referenced.add(skillName);
-        const skill = skillLookup(skillName);
-        if (skill) {
-            const newVisited = new Set(visited);
-            newVisited.add(skillName);
-            const nested = getReferencedSkills(skill.execution, skillLookup, newVisited);
-            for (const name of nested) {
-                referenced.add(name);
-            }
-        }
-    }
-    return referenced;
-}
-/**
- * Validate skill references don't form cycles
- * Returns true if valid, false if circular reference detected
- */
-export function validateNoCycles(execution, skillLookup, visited = new Set()) {
-    try {
-        expandSkillReferences(execution, skillLookup, visited);
-        return true;
-    }
-    catch (error) {
-        if (error instanceof Error && error.message.includes('Circular')) {
-            return false;
-        }
-        throw error;
-    }
-}