@shepai/cli 1.120.0-pr385.7a4c87f → 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');
 }