@shepai/cli 1.120.0 → 1.121.0-pr386.fb524e4

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/fast-implement.node.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fast-implement.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/fast-implement.node.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAKH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AAEpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAUrD;;;;;GAKG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,cAAc,IAGhD,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAoD7E"}
+{"version":3,"file":"fast-implement.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/fast-implement.node.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAMH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AAEpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAUrD;;;;;GAKG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,cAAc,IAGhD,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CA4D7E"}

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/fast-implement.node.js

@@ -9,6 +9,7 @@
  * Follows the same factory pattern as other nodes: takes executor
  * dependency, returns async (state) => Partial<FeatureAgentState>.
  */
+import { execSync } from 'node:child_process';
 import { mkdirSync, writeFileSync } from 'node:fs';
 import { join, dirname } from 'node:path';
 import { isGraphBubbleUp } from '@langchain/langgraph';
@@ -44,6 +45,11 @@ export function createFastImplementNode(executor) {
             const durationMs = Date.now() - startTime;
             const elapsed = (durationMs / 1000).toFixed(1);
             log.info(`Complete (${result.result.length} chars, ${elapsed}s)`);
+            // Validate that the executor actually produced file changes
+            const cwd = state.worktreePath || state.repositoryPath;
+            if (!hasWorktreeChanges(cwd)) {
+                throw new Error('[fast-implement] Agent produced no file changes — it may have entered plan mode or asked questions instead of implementing. Retrying.');
+            }
             // --- Evidence sub-agent: capture proof of completion (settings-gated) ---
             const evidenceEnabled = hasSettings() && getSettings().workflow.enableEvidence;
             const evidence = evidenceEnabled ? await collectEvidence(executor, state, log) : [];
@@ -107,6 +113,21 @@ async function collectEvidence(executor, state, log) {
         return [];
     }
 }
+/**
+ * Check whether the worktree has any uncommitted changes (new, modified, or deleted files).
+ * Uses `git status --porcelain` which outputs one line per changed file, or empty if clean.
+ * Returns false if the git command fails (e.g. not a git repo).
+ */
+function hasWorktreeChanges(cwd) {
+    try {
+        const output = execSync('git status --porcelain', { cwd, encoding: 'utf-8' });
+        return output.trim().length > 0;
+    }
+    catch {
+        // If git command fails, assume no changes (conservative — will trigger the error)
+        return false;
+    }
+}
 /**
  * Save evidence manifest to the shep home evidence folder so the
  * merge review UI can read it without accessing graph state.

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/prompts/fast-implement.prompt.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fast-implement.prompt.d.ts","sourceRoot":"","sources":["../../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/prompts/fast-implement.prompt.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAMH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,gBAAgB,CAAC;AAwIxD;;;;;;;;;;;GAWG;AACH,wBAAgB,wBAAwB,CAAC,KAAK,EAAE,iBAAiB,GAAG,MAAM,CAoEzE"}
+{"version":3,"file":"fast-implement.prompt.d.ts","sourceRoot":"","sources":["../../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/prompts/fast-implement.prompt.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAMH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,gBAAgB,CAAC;AAwIxD;;;;;;;;;;;GAWG;AACH,wBAAgB,wBAAwB,CAAC,KAAK,EAAE,iBAAiB,GAAG,MAAM,CAuEzE"}

package/dist/packages/core/src/infrastructure/services/agents/feature-agent/nodes/prompts/fast-implement.prompt.js

@@ -211,6 +211,9 @@ ${dirListing}
 - Follow existing codebase conventions and patterns
 - Do NOT commit, push, or create pull requests
 - Do NOT modify any spec YAML files
-- Keep changes focused and minimal`);
+- Keep changes focused and minimal
+- Do NOT enter plan mode — implement directly without planning phases
+- Do NOT ask the user questions or use AskUserQuestion — make reasonable decisions and proceed
+- You MUST create or modify actual code files — a plan or summary alone is not acceptable output`);
     return sections.join('\n\n');
 }

package/dist/src/presentation/web/components/common/feature-list-item/feature-list-item.stories.d.ts

@@ -6,6 +6,7 @@ type Story = StoryObj<typeof meta>;
 export declare const ActionNeeded: Story;
 export declare const InProgress: Story;
 export declare const Done: Story;
+export declare const Pending: Story;
 export declare const Blocked: Story;
 export declare const Error: Story;
 export declare const WithClickHandler: Story;

package/dist/src/presentation/web/components/common/feature-list-item/feature-list-item.stories.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"feature-list-item.stories.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-list-item/feature-list-item.stories.tsx"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAEvD,OAAO,EAAE,eAAe,EAAE,MAAM,qBAAqB,CAAC;AAEtD,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,OAAO,eAAe,CAgBtC,CAAC;AAEF,eAAe,IAAI,CAAC;AACpB,KAAK,KAAK,GAAG,QAAQ,CAAC,OAAO,IAAI,CAAC,CAAC;AAEnC,eAAO,MAAM,YAAY,EAAE,KAO1B,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,KAQxB,CAAC;AAEF,eAAO,MAAM,IAAI,EAAE,KAQlB,CAAC;AAEF,eAAO,MAAM,OAAO,EAAE,KAOrB,CAAC;AAEF,eAAO,MAAM,KAAK,EAAE,KAOnB,CAAC;AAEF,eAAO,MAAM,gBAAgB,EAAE,KAQ9B,CAAC;AAEF,eAAO,MAAM,YAAY,EAAE,KAM1B,CAAC"}
+{"version":3,"file":"feature-list-item.stories.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-list-item/feature-list-item.stories.tsx"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAEvD,OAAO,EAAE,eAAe,EAAE,MAAM,qBAAqB,CAAC;AAEtD,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,OAAO,eAAe,CAgBtC,CAAC;AAEF,eAAe,IAAI,CAAC;AACpB,KAAK,KAAK,GAAG,QAAQ,CAAC,OAAO,IAAI,CAAC,CAAC;AAEnC,eAAO,MAAM,YAAY,EAAE,KAO1B,CAAC;AAEF,eAAO,MAAM,UAAU,EAAE,KAQxB,CAAC;AAEF,eAAO,MAAM,IAAI,EAAE,KAQlB,CAAC;AAEF,eAAO,MAAM,OAAO,EAAE,KAOrB,CAAC;AAEF,eAAO,MAAM,OAAO,EAAE,KAOrB,CAAC;AAEF,eAAO,MAAM,KAAK,EAAE,KAOnB,CAAC;AAEF,eAAO,MAAM,gBAAgB,EAAE,KAQ9B,CAAC;AAEF,eAAO,MAAM,YAAY,EAAE,KAM1B,CAAC"}

package/dist/src/presentation/web/components/common/feature-list-item/feature-list-item.stories.js

@@ -39,6 +39,14 @@ export const Done = {
         modelId: 'gemini-2.5-pro',
     },
 };
+export const Pending = {
+    args: {
+        name: 'Dark Mode Toggle',
+        status: 'pending',
+        agentType: 'claude-code',
+        modelId: 'claude-sonnet-4-6',
+    },
+};
 export const Blocked = {
     args: {
         name: 'Dependency Resolver',

package/dist/src/presentation/web/components/common/feature-status-badges/feature-status-badges.stories.js

@@ -15,31 +15,31 @@ const meta = {
 export default meta;
 export const AllStatuses = {
     args: {
-        counts: { 'action-needed': 2, 'in-progress': 3, blocked: 1, error: 1, done: 5 },
+        counts: { 'action-needed': 2, 'in-progress': 3, pending: 2, blocked: 1, error: 1, done: 5 },
     },
 };
 export const ActionNeededOnly = {
     args: {
-        counts: { 'action-needed': 4, 'in-progress': 0, blocked: 0, error: 0, done: 0 },
+        counts: { 'action-needed': 4, 'in-progress': 0, pending: 0, blocked: 0, error: 0, done: 0 },
     },
 };
 export const InProgressOnly = {
     args: {
-        counts: { 'action-needed': 0, 'in-progress': 2, blocked: 0, error: 0, done: 0 },
+        counts: { 'action-needed': 0, 'in-progress': 2, pending: 0, blocked: 0, error: 0, done: 0 },
     },
 };
 export const DoneOnly = {
     args: {
-        counts: { 'action-needed': 0, 'in-progress': 0, blocked: 0, error: 0, done: 7 },
+        counts: { 'action-needed': 0, 'in-progress': 0, pending: 0, blocked: 0, error: 0, done: 7 },
     },
 };
 export const LargeCounts = {
     args: {
-        counts: { 'action-needed': 12, 'in-progress': 99, blocked: 5, error: 3, done: 150 },
+        counts: { 'action-needed': 12, 'in-progress': 99, pending: 8, blocked: 5, error: 3, done: 150 },
     },
 };
 export const Empty = {
     args: {
-        counts: { 'action-needed': 0, 'in-progress': 0, blocked: 0, error: 0, done: 0 },
+        counts: { 'action-needed': 0, 'in-progress': 0, pending: 0, blocked: 0, error: 0, done: 0 },
     },
 };

package/dist/src/presentation/web/components/common/feature-status-config.d.ts

@@ -1,5 +1,5 @@
 import type { LucideIcon } from 'lucide-react';
-export type FeatureStatus = 'action-needed' | 'in-progress' | 'blocked' | 'error' | 'done';
+export type FeatureStatus = 'action-needed' | 'in-progress' | 'pending' | 'blocked' | 'error' | 'done';
 export interface FeatureStatusConfig {
     icon: LucideIcon;
     iconClass: string;

package/dist/src/presentation/web/components/common/feature-status-config.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"feature-status-config.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/components/common/feature-status-config.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,cAAc,CAAC;AAE/C,MAAM,MAAM,aAAa,GAAG,eAAe,GAAG,aAAa,GAAG,SAAS,GAAG,OAAO,GAAG,MAAM,CAAC;AAE3F,MAAM,WAAW,mBAAmB;IAClC,IAAI,EAAE,UAAU,CAAC;IACjB,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,MAAM,CAAC;CACf;AAED,eAAO,MAAM,mBAAmB,EAAE,MAAM,CAAC,aAAa,EAAE,mBAAmB,CA+B1E,CAAC;AAEF,eAAO,MAAM,kBAAkB,EAAE,aAAa,EAM7C,CAAC"}
+{"version":3,"file":"feature-status-config.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/components/common/feature-status-config.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,cAAc,CAAC;AAE/C,MAAM,MAAM,aAAa,GACrB,eAAe,GACf,aAAa,GACb,SAAS,GACT,SAAS,GACT,OAAO,GACP,MAAM,CAAC;AAEX,MAAM,WAAW,mBAAmB;IAClC,IAAI,EAAE,UAAU,CAAC;IACjB,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,MAAM,CAAC;CACf;AAED,eAAO,MAAM,mBAAmB,EAAE,MAAM,CAAC,aAAa,EAAE,mBAAmB,CAqC1E,CAAC;AAEF,eAAO,MAAM,kBAAkB,EAAE,aAAa,EAO7C,CAAC"}

package/dist/src/presentation/web/components/common/feature-status-config.js

@@ -1,4 +1,4 @@
-import { CircleAlert, Loader2, CircleCheck, Ban, CircleX } from 'lucide-react';
+import { CircleAlert, Loader2, CircleCheck, Ban, CircleX, Clock } from 'lucide-react';
 export const featureStatusConfig = {
     'action-needed': {
         icon: CircleAlert,
@@ -12,6 +12,12 @@ export const featureStatusConfig = {
         bgClass: 'bg-blue-500/10',
         label: 'In Progress',
     },
+    pending: {
+        icon: Clock,
+        iconClass: 'text-slate-400',
+        bgClass: 'bg-slate-400/10',
+        label: 'Pending',
+    },
     blocked: {
         icon: Ban,
         iconClass: 'text-gray-400',
@@ -36,5 +42,6 @@ export const featureStatusOrder = [
     'error',
     'blocked',
     'in-progress',
+    'pending',
     'done',
 ];

package/dist/src/presentation/web/components/common/feature-status-group/feature-status-group.stories.d.ts

@@ -5,5 +5,6 @@ export default meta;
 type Story = StoryObj<typeof meta>;
 export declare const InProgress: Story;
 export declare const Done: Story;
+export declare const Pending: Story;
 export declare const ActionNeeded: Story;
 //# sourceMappingURL=feature-status-group.stories.d.ts.map

package/dist/src/presentation/web/components/common/feature-status-group/feature-status-group.stories.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"feature-status-group.stories.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-status-group/feature-status-group.stories.tsx"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAGvD,OAAO,EAAE,kBAAkB,EAAE,MAAM,wBAAwB,CAAC;AAE5D,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,OAAO,kBAAkB,CAgBzC,CAAC;AAEF,eAAe,IAAI,CAAC;AACpB,KAAK,KAAK,GAAG,QAAQ,CAAC,OAAO,IAAI,CAAC,CAAC;AAEnC,eAAO,MAAM,UAAU,EAAE,KAWxB,CAAC;AAEF,eAAO,MAAM,IAAI,EAAE,KAYlB,CAAC;AAEF,eAAO,MAAM,YAAY,EAAE,KAW1B,CAAC"}
+{"version":3,"file":"feature-status-group.stories.d.ts","sourceRoot":"","sources":["../../../../../../../src/presentation/web/components/common/feature-status-group/feature-status-group.stories.tsx"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAGvD,OAAO,EAAE,kBAAkB,EAAE,MAAM,wBAAwB,CAAC;AAE5D,QAAA,MAAM,IAAI,EAAE,IAAI,CAAC,OAAO,kBAAkB,CAgBzC,CAAC;AAEF,eAAe,IAAI,CAAC;AACpB,KAAK,KAAK,GAAG,QAAQ,CAAC,OAAO,IAAI,CAAC,CAAC;AAEnC,eAAO,MAAM,UAAU,EAAE,KAWxB,CAAC;AAEF,eAAO,MAAM,IAAI,EAAE,KAYlB,CAAC;AAEF,eAAO,MAAM,OAAO,EAAE,KAWrB,CAAC;AAEF,eAAO,MAAM,YAAY,EAAE,KAW1B,CAAC"}

package/dist/src/presentation/web/components/common/feature-status-group/feature-status-group.stories.js

@@ -28,6 +28,13 @@ export const Done = {
         children: (_jsxs(_Fragment, { children: [_jsx(FeatureListItem, { name: "Settings Page", status: "done", duration: "2h" }), _jsx(FeatureListItem, { name: "User Profile", status: "done", duration: "1h" }), _jsx(FeatureListItem, { name: "Auth Module", status: "done", duration: "30m" })] })),
     },
 };
+export const Pending = {
+    args: {
+        label: 'Pending',
+        count: 2,
+        children: (_jsxs(_Fragment, { children: [_jsx(FeatureListItem, { name: "Dark Mode", status: "pending" }), _jsx(FeatureListItem, { name: "Search Filters", status: "pending" })] })),
+    },
+};
 export const ActionNeeded = {
     args: {
         label: 'Action Needed',

package/dist/src/presentation/web/hooks/sidebar-features-context.js

@@ -9,7 +9,7 @@ const stateMapping = {
     running: 'in-progress',
     done: 'done',
     blocked: 'blocked',
-    pending: 'blocked',
+    pending: 'pending',
     error: 'error',
     creating: null,
     deleting: null,