@shepai/cli 1.16.0 → 1.18.0

package/dist/src/infrastructure/services/agents/feature-agent/nodes/node-helpers.js

@@ -4,7 +4,8 @@
  * Provides consistent logging, spec file reading, executor invocation,
  * and error handling so every node follows the same patterns.
  */
-import { readFileSync } from 'node:fs';
+import yaml from 'js-yaml';
+import { readFileSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';
 import { interrupt, isGraphBubbleUp } from '@langchain/langgraph';
 import { reportNodeStart } from '../heartbeat.js';
@@ -41,8 +42,34 @@ export function readSpecFile(specDir, filename) {
 export function buildExecutorOptions(state) {
     return {
         cwd: state.worktreePath || state.repositoryPath,
+        maxTurns: 50,
+        disableMcp: true,
     };
 }
+/**
+ * Sanitize YAML content to handle common AI-generated issues.
+ * Quotes unquoted list items containing `{` or `}` characters
+ * which js-yaml interprets as flow mappings.
+ */
+function sanitizeYamlBraces(content) {
+    return content.replace(/^(\s*- )(?!['"])(.+[{}].+)$/gm, (_match, prefix, value) => `${prefix}"${value.replace(/\\/g, '\\\\').replace(/"/g, '\\"')}"`);
+}
+/**
+ * Parse YAML with automatic sanitization of common AI-generated issues.
+ * First attempts normal parsing; on failure, sanitizes unquoted braces
+ * in list items and retries.
+ */
+export function safeYamlLoad(content) {
+    try {
+        return yaml.load(content);
+    }
+    catch (firstError) {
+        const sanitized = sanitizeYamlBraces(content);
+        if (sanitized === content)
+            throw firstError; // nothing to sanitize
+        return yaml.load(sanitized);
+    }
+}
 /** Nodes that are auto-approved (no interrupt) for each approval mode. */
 const AUTO_APPROVED_NODES = {
     interactive: [],
@@ -60,6 +87,99 @@ export function shouldInterrupt(nodeName, approvalMode) {
     const autoApproved = AUTO_APPROVED_NODES[approvalMode] ?? [];
     return !autoApproved.includes(nodeName);
 }
+const API_ERROR_RE = /API Error: (400|429|5\d{2})/;
+const NETWORK_ERROR_RE = /ECONNREFUSED|ETIMEDOUT|ENOTFOUND|timed out/i;
+const NON_RETRYABLE_RE = /Process exited with code|ENOENT|SyntaxError/;
+/**
+ * Classify an error message into a retry category.
+ *
+ * - **retryable-api**: Transient API errors (rate-limit, overload, bad request)
+ * - **retryable-network**: Network-level failures (DNS, timeouts, connection refused)
+ * - **non-retryable**: Logic / filesystem / syntax errors that won't resolve on retry
+ * - **unknown**: Anything unrecognised (callers may choose to retry cautiously)
+ */
+export function classifyError(errorMessage) {
+    if (API_ERROR_RE.test(errorMessage))
+        return 'retryable-api';
+    if (NETWORK_ERROR_RE.test(errorMessage))
+        return 'retryable-network';
+    if (NON_RETRYABLE_RE.test(errorMessage))
+        return 'non-retryable';
+    return 'unknown';
+}
+/**
+ * Execute a prompt via the given executor with automatic retry and
+ * exponential back-off for transient errors.
+ *
+ * Non-retryable errors are thrown immediately. Unknown errors are
+ * retried (conservative stance: could be transient).
+ */
+export async function retryExecute(executor, prompt, options, retryOpts) {
+    const maxAttempts = retryOpts?.maxAttempts ?? 3;
+    const baseDelayMs = retryOpts?.baseDelayMs ?? 2000;
+    const log = retryOpts?.logger;
+    let lastError;
+    for (let attempt = 1; attempt <= maxAttempts; attempt++) {
+        try {
+            return await executor.execute(prompt, options);
+        }
+        catch (err) {
+            lastError = err instanceof Error ? err : new Error(String(err));
+            const category = classifyError(lastError.message);
+            if (category === 'non-retryable') {
+                throw lastError;
+            }
+            if (attempt < maxAttempts) {
+                const delayMs = baseDelayMs * Math.pow(2, attempt - 1);
+                log?.info(`Attempt ${attempt}/${maxAttempts} failed (${category}), retrying in ${delayMs}ms`);
+                await new Promise((resolve) => setTimeout(resolve, delayMs));
+            }
+        }
+    }
+    throw lastError;
+}
+/**
+ * Read completed phases from feature.yaml.
+ */
+export function getCompletedPhases(specDir) {
+    const content = readSpecFile(specDir, 'feature.yaml');
+    if (!content)
+        return [];
+    try {
+        const data = yaml.load(content);
+        const status = (data?.status ?? {});
+        const phases = status.completedPhases;
+        return Array.isArray(phases) ? phases : [];
+    }
+    catch {
+        return [];
+    }
+}
+/**
+ * Mark a phase as complete in feature.yaml by adding its ID to completedPhases.
+ */
+export function markPhaseComplete(specDir, phaseId, log) {
+    try {
+        const content = readSpecFile(specDir, 'feature.yaml');
+        if (!content)
+            return;
+        const data = yaml.load(content);
+        if (!data || typeof data !== 'object')
+            return;
+        const status = (data.status ?? {});
+        const completedPhases = Array.isArray(status.completedPhases)
+            ? [...status.completedPhases]
+            : [];
+        if (!completedPhases.includes(phaseId)) {
+            completedPhases.push(phaseId);
+        }
+        data.status = { ...status, completedPhases };
+        writeFileSync(join(specDir, 'feature.yaml'), yaml.dump(data, { indent: 2, lineWidth: -1 }), 'utf-8');
+    }
+    catch (err) {
+        log?.error(`Failed to mark phase complete: ${err instanceof Error ? err.message : String(err)}`);
+    }
+}
 /**
  * Execute a node with consistent logging and error handling.
  *
@@ -104,11 +224,10 @@ export function executeNode(nodeName, executor, buildPrompt) {
             const message = err instanceof Error ? err.message : String(err);
             const elapsed = ((Date.now() - startTime) / 1000).toFixed(1);
             log.error(`${message} (after ${elapsed}s)`);
-            return {
-                currentNode: nodeName,
-                error: message,
-                messages: [`[${nodeName}] Error: ${message}`],
-            };
+            // Throw so LangGraph does NOT checkpoint this node as "completed".
+            // The worker catch block marks the run as failed, and on resume
+            // LangGraph re-executes from the last successfully checkpointed node.
+            throw new Error(`[${nodeName}] ${message}`);
         }
     };
 }

package/dist/src/presentation/cli/commands/feat/ls.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"ls.command.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/cli/commands/feat/ls.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAoEpC,wBAAgB,eAAe,IAAI,OAAO,CAqDzC"}
+{"version":3,"file":"ls.command.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/cli/commands/feat/ls.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAGH,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAsGpC,wBAAgB,eAAe,IAAI,OAAO,CAqDzC"}

package/dist/src/presentation/cli/commands/feat/ls.command.js

@@ -10,6 +10,7 @@
  * $ shep feat ls
  * $ shep feat ls --repo /path/to/project
  */
+import path from 'node:path';
 import { Command } from 'commander';
 import { container } from '../../../../infrastructure/di/container.js';
 import { ListFeaturesUseCase } from '../../../../application/use-cases/features/list-features.use-case.js';
@@ -65,6 +66,39 @@ function isProcessAlive(pid) {
         return false;
     }
 }
+/** Format a duration in ms to a compact human-readable string. */
+function formatDuration(ms) {
+    const seconds = Math.floor(ms / 1000);
+    if (seconds < 60)
+        return `${seconds}s`;
+    const minutes = Math.floor(seconds / 60);
+    if (minutes < 60)
+        return `${minutes}m`;
+    const hours = Math.floor(minutes / 60);
+    const remMin = minutes % 60;
+    if (hours < 24)
+        return remMin > 0 ? `${hours}h ${remMin}m` : `${hours}h`;
+    const days = Math.floor(hours / 24);
+    return `${days}d`;
+}
+/** Format the time column based on run status. */
+function formatTime(feature, run) {
+    const now = Date.now();
+    if (run) {
+        const isRunning = run.status === 'running' || run.status === 'pending';
+        if (isRunning && run.startedAt) {
+            const started = new Date(run.startedAt).getTime();
+            return colors.info(formatDuration(now - started));
+        }
+        if (run.status === 'completed' && run.completedAt) {
+            const completed = new Date(run.completedAt).getTime();
+            return colors.muted(`${formatDuration(now - completed)} ago`);
+        }
+    }
+    // Fallback to updatedAt
+    const updated = new Date(feature.updatedAt).getTime();
+    return colors.muted(`${formatDuration(now - updated)} ago`);
+}
 export function createLsCommand() {
     return new Command('ls')
         .description('List features')
@@ -79,15 +113,15 @@ export function createLsCommand() {
             const runs = await Promise.all(features.map((f) => f.agentRunId ? runRepo.findById(f.agentRunId) : Promise.resolve(null)));
             const rows = features.map((feature, i) => {
                 const run = runs[i];
-                const updated = feature.updatedAt instanceof Date
-                    ? feature.updatedAt.toLocaleDateString()
-                    : String(feature.updatedAt);
+                const repo = path.basename(feature.repositoryPath);
+                const agent = run?.agentType ?? colors.muted('—');
                 return [
                     feature.id.slice(0, 8),
                     feature.name.slice(0, 30),
                     formatStatus(feature, run),
-                    colors.muted(feature.branch),
-                    colors.muted(updated),
+                    colors.muted(repo),
+                    agent,
+                    formatTime(feature, run),
                 ];
             });
             renderListView({
@@ -96,8 +130,9 @@ export function createLsCommand() {
                     { label: 'ID', width: 10 },
                     { label: 'Name', width: 32 },
                     { label: 'Status', width: 20 },
-                    { label: 'Branch', width: 28 },
-                    { label: 'Updated', width: 12 },
+                    { label: 'Repo', width: 20 },
+                    { label: 'Agent', width: 14 },
+                    { label: 'Time', width: 12 },
                 ],
                 rows,
                 emptyMessage: 'No features found',

package/dist/src/presentation/tui/prompts/agent-select.prompt.d.ts

@@ -32,7 +32,7 @@ export declare function createAgentSelectConfig(): {
     }, {
         readonly name: "Cursor";
         readonly value: AgentType.Cursor;
-        readonly disabled: "(Coming Soon)";
+        readonly description: "Cursor AI coding agent";
     }];
     readonly theme: {
         readonly prefix: {

package/dist/src/presentation/tui/prompts/agent-select.prompt.js

@@ -39,7 +39,7 @@ export function createAgentSelectConfig() {
             {
                 name: 'Cursor',
                 value: AgentType.Cursor,
-                disabled: '(Coming Soon)',
+                description: 'Cursor AI coding agent',
             },
         ],
         theme: shepTheme,

package/dist/src/presentation/web/components/common/sidebar-section-header/sidebar-section-header.js

@@ -7,5 +7,5 @@ import { useSidebar } from '../../ui/sidebar';
 import { cn } from '../../../lib/utils';
 export function SidebarSectionHeader({ label, onFolderClick, onMenuClick, className, }) {
     const { state, isMobile } = useSidebar();
-    return (_jsx(SidebarGroup, { "data-testid": "sidebar-section-header", className: cn('px-2 py-1', className), children: _jsxs("div", { className: "text-sidebar-foreground flex h-7 w-full shrink-0 items-center px-2 text-sm font-medium", children: [_jsx("span", { className: "min-w-0 flex-1 truncate", children: label }), _jsxs("span", { className: "text-sidebar-foreground/70 flex shrink-0 items-center gap-0.5", children: [_jsxs(Tooltip, { children: [_jsx(TooltipTrigger, { asChild: true, children: _jsx("button", { type: "button", onClick: onFolderClick, className: "hover:bg-sidebar-accent hover:text-sidebar-accent-foreground ring-sidebar-ring flex size-7 items-center justify-center rounded-md p-0 outline-hidden transition-colors focus-visible:ring-2 [&>svg]:size-4", "aria-label": "Open features folder", children: _jsx(Folder, { className: "size-4" }) }) }), _jsx(TooltipContent, { side: "right", align: "center", hidden: state !== 'collapsed' || isMobile, children: "Open features folder" })] }), _jsxs(Tooltip, { children: [_jsx(TooltipTrigger, { asChild: true, children: _jsx("button", { type: "button", onClick: onMenuClick, className: "hover:bg-sidebar-accent hover:text-sidebar-accent-foreground ring-sidebar-ring flex size-7 items-center justify-center rounded-md p-0 outline-hidden transition-colors focus-visible:ring-2 [&>svg]:size-4", "aria-label": "Features options", children: _jsx(GripHorizontal, { className: "size-4" }) }) }), _jsx(TooltipContent, { side: "right", align: "center", hidden: state !== 'collapsed' || isMobile, children: "Features options" })] })] })] }) }));
+    return (_jsx(SidebarGroup, { "data-testid": "sidebar-section-header", className: cn('px-2 py-1', className), children: _jsxs("div", { className: "text-sidebar-foreground flex h-7 w-full shrink-0 items-center px-2 text-sm font-medium", children: [_jsx("span", { className: "min-w-0 flex-1 truncate", children: label }), _jsxs("span", { className: "text-sidebar-foreground/70 flex shrink-0 items-center gap-0.5", children: [_jsxs(Tooltip, { children: [_jsx(TooltipTrigger, { asChild: true, children: _jsx("button", { type: "button", onClick: onFolderClick, className: "ring-sidebar-ring hover:bg-sidebar-accent hover:text-sidebar-accent-foreground flex size-7 items-center justify-center rounded-md p-0 outline-hidden transition-colors focus-visible:ring-2 [&>svg]:size-4", "aria-label": "Open features folder", children: _jsx(Folder, { className: "size-4" }) }) }), _jsx(TooltipContent, { side: "right", align: "center", hidden: state !== 'collapsed' || isMobile, children: "Open features folder" })] }), _jsxs(Tooltip, { children: [_jsx(TooltipTrigger, { asChild: true, children: _jsx("button", { type: "button", onClick: onMenuClick, className: "ring-sidebar-ring hover:bg-sidebar-accent hover:text-sidebar-accent-foreground flex size-7 items-center justify-center rounded-md p-0 outline-hidden transition-colors focus-visible:ring-2 [&>svg]:size-4", "aria-label": "Features options", children: _jsx(GripHorizontal, { className: "size-4" }) }) }), _jsx(TooltipContent, { side: "right", align: "center", hidden: state !== 'collapsed' || isMobile, children: "Features options" })] })] })] }) }));
 }

package/dist/src/presentation/web/components/ui/alert.js

@@ -2,7 +2,7 @@ import { jsx as _jsx } from "react/jsx-runtime";
 import * as React from 'react';
 import { cva } from 'class-variance-authority';
 import { cn } from '../../lib/utils';
-const alertVariants = cva('[&>svg]:text-foreground relative w-full rounded-lg border px-4 py-3 text-sm [&>svg]:absolute [&>svg]:top-4 [&>svg]:left-4 [&>svg+div]:translate-y-[-3px] [&>svg~*]:pl-7', {
+const alertVariants = cva('relative w-full rounded-lg border px-4 py-3 text-sm [&>svg]:absolute [&>svg]:top-4 [&>svg]:left-4 [&>svg]:text-foreground [&>svg+div]:translate-y-[-3px] [&>svg~*]:pl-7', {
     variants: {
         variant: {
             default: 'bg-background text-foreground',

package/dist/src/presentation/web/components/ui/badge.js

@@ -1,12 +1,12 @@
 import { jsx as _jsx } from "react/jsx-runtime";
 import { cva } from 'class-variance-authority';
 import { cn } from '../../lib/utils';
-const badgeVariants = cva('focus:ring-ring inline-flex items-center rounded-md border px-2.5 py-0.5 text-xs font-semibold transition-colors focus:ring-2 focus:ring-offset-2 focus:outline-none', {
+const badgeVariants = cva('inline-flex items-center rounded-md border px-2.5 py-0.5 text-xs font-semibold transition-colors focus:ring-2 focus:ring-ring focus:ring-offset-2 focus:outline-none', {
     variants: {
         variant: {
-            default: 'bg-primary text-primary-foreground hover:bg-primary/80 border-transparent shadow',
-            secondary: 'bg-secondary text-secondary-foreground hover:bg-secondary/80 border-transparent',
-            destructive: 'bg-destructive text-destructive-foreground hover:bg-destructive/80 border-transparent shadow',
+            default: 'border-transparent bg-primary text-primary-foreground shadow hover:bg-primary/80',
+            secondary: 'border-transparent bg-secondary text-secondary-foreground hover:bg-secondary/80',
+            destructive: 'border-transparent bg-destructive text-destructive-foreground shadow hover:bg-destructive/80',
             outline: 'text-foreground',
         },
     },

package/dist/src/presentation/web/components/ui/button.js

@@ -2,12 +2,12 @@ import { jsx as _jsx } from "react/jsx-runtime";
 import { cva } from 'class-variance-authority';
 import { Slot } from 'radix-ui';
 import { cn } from '../../lib/utils';
-const buttonVariants = cva("focus-visible:border-ring focus-visible:ring-ring/50 aria-invalid:ring-destructive/20 dark:aria-invalid:ring-destructive/40 aria-invalid:border-destructive inline-flex shrink-0 items-center justify-center gap-2 rounded-md text-sm font-medium whitespace-nowrap transition-all outline-none focus-visible:ring-[3px] disabled:pointer-events-none disabled:opacity-50 [&_svg]:pointer-events-none [&_svg]:shrink-0 [&_svg:not([class*='size-'])]:size-4", {
+const buttonVariants = cva("inline-flex shrink-0 items-center justify-center gap-2 rounded-md text-sm font-medium whitespace-nowrap transition-all outline-none focus-visible:border-ring focus-visible:ring-[3px] focus-visible:ring-ring/50 disabled:pointer-events-none disabled:opacity-50 aria-invalid:border-destructive aria-invalid:ring-destructive/20 dark:aria-invalid:ring-destructive/40 [&_svg]:pointer-events-none [&_svg]:shrink-0 [&_svg:not([class*='size-'])]:size-4", {
     variants: {
         variant: {
             default: 'bg-primary text-primary-foreground hover:bg-primary/90',
-            destructive: 'bg-destructive hover:bg-destructive/90 focus-visible:ring-destructive/20 dark:focus-visible:ring-destructive/40 dark:bg-destructive/60 text-white',
-            outline: 'bg-background hover:bg-accent hover:text-accent-foreground dark:bg-input/30 dark:border-input dark:hover:bg-input/50 border shadow-xs',
+            destructive: 'bg-destructive text-white hover:bg-destructive/90 focus-visible:ring-destructive/20 dark:bg-destructive/60 dark:focus-visible:ring-destructive/40',
+            outline: 'border bg-background shadow-xs hover:bg-accent hover:text-accent-foreground dark:border-input dark:bg-input/30 dark:hover:bg-input/50',
             secondary: 'bg-secondary text-secondary-foreground hover:bg-secondary/80',
             ghost: 'hover:bg-accent hover:text-accent-foreground dark:hover:bg-accent/50',
             link: 'text-primary underline-offset-4 hover:underline',

package/dist/src/presentation/web/components/ui/sidebar.js

@@ -148,11 +148,11 @@ function SidebarMenu({ className, ...props }) {
 function SidebarMenuItem({ className, ...props }) {
     return (_jsx("li", { "data-slot": "sidebar-menu-item", "data-sidebar": "menu-item", className: cn('group/menu-item relative', className), ...props }));
 }
-const sidebarMenuButtonVariants = cva('peer/menu-button ring-sidebar-ring hover:bg-sidebar-accent hover:text-sidebar-accent-foreground active:bg-sidebar-accent active:text-sidebar-accent-foreground data-[active=true]:bg-sidebar-accent data-[active=true]:text-sidebar-accent-foreground data-[state=open]:hover:bg-sidebar-accent data-[state=open]:hover:text-sidebar-accent-foreground flex w-full items-center gap-2 overflow-hidden rounded-md p-2 text-left text-sm outline-hidden transition-[width,height,padding] group-has-data-[sidebar=menu-action]/menu-item:pr-8 group-data-[collapsible=icon]:size-8! group-data-[collapsible=icon]:p-2! focus-visible:ring-2 disabled:pointer-events-none disabled:opacity-50 aria-disabled:pointer-events-none aria-disabled:opacity-50 data-[active=true]:font-medium [&>span:last-child]:truncate [&>svg]:size-4 [&>svg]:shrink-0', {
+const sidebarMenuButtonVariants = cva('peer/menu-button flex w-full items-center gap-2 overflow-hidden rounded-md p-2 text-left text-sm ring-sidebar-ring outline-hidden transition-[width,height,padding] group-has-data-[sidebar=menu-action]/menu-item:pr-8 group-data-[collapsible=icon]:size-8! group-data-[collapsible=icon]:p-2! hover:bg-sidebar-accent hover:text-sidebar-accent-foreground focus-visible:ring-2 active:bg-sidebar-accent active:text-sidebar-accent-foreground disabled:pointer-events-none disabled:opacity-50 aria-disabled:pointer-events-none aria-disabled:opacity-50 data-[active=true]:bg-sidebar-accent data-[active=true]:font-medium data-[active=true]:text-sidebar-accent-foreground data-[state=open]:hover:bg-sidebar-accent data-[state=open]:hover:text-sidebar-accent-foreground [&>span:last-child]:truncate [&>svg]:size-4 [&>svg]:shrink-0', {
     variants: {
         variant: {
             default: 'hover:bg-sidebar-accent hover:text-sidebar-accent-foreground',
-            outline: 'bg-background hover:bg-sidebar-accent hover:text-sidebar-accent-foreground shadow-[0_0_0_1px_hsl(var(--sidebar-border))] hover:shadow-[0_0_0_1px_hsl(var(--sidebar-accent))]',
+            outline: 'bg-background shadow-[0_0_0_1px_hsl(var(--sidebar-border))] hover:bg-sidebar-accent hover:text-sidebar-accent-foreground hover:shadow-[0_0_0_1px_hsl(var(--sidebar-accent))]',
         },
         size: {
             default: 'h-8 text-sm',