@shepai/cli 1.150.0-pr472.498e2c5 → 1.150.1

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

@@ -1 +1 @@
-{"version":3,"file":"evidence.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/evidence.node.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;GAqBG;AAIH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AAEpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAqDrD;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,cAAc,IAG3C,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAoJ7E"}
+{"version":3,"file":"evidence.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/evidence.node.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;GAqBG;AAIH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AAEpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAqDrD;;;;;GAKG;AACH,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,cAAc,IAG3C,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAqJ7E"}

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

@@ -82,8 +82,9 @@ export function createEvidenceNode(executor) {
             };
         }
         // --- Configuration ---
+        // Use feature-level state for commitEvidence; fall back to global for retries config
+        const commitEvidence = state.commitEvidence;
         const settings = hasSettings() ? getSettings() : undefined;
-        const commitEvidence = settings?.workflow.commitEvidence ?? false;
         const maxRetries = settings?.workflow.evidenceRetries ?? DEFAULT_MAX_RETRIES;
         const options = buildExecutorOptions(state);
         const tasks = parseTasks(state.specDir);

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;AAIH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AAEpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAiBrD;;;;;GAKG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,cAAc,IAGhD,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CA0F7E"}
+{"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;AAIH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AACpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAcrD;;;;;GAKG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,cAAc,IAGhD,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CA6F7E"}

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

@@ -11,14 +11,12 @@
  */
 import { execSync } from 'node:child_process';
 import { isGraphBubbleUp } from '@langchain/langgraph';
-import { createNodeLogger, buildExecutorOptions, retryExecute, getCompletedPhases, markPhaseComplete, saveEvidenceManifest, } from './node-helpers.js';
+import { createNodeLogger, buildExecutorOptions, retryExecute, getCompletedPhases, markPhaseComplete, } from './node-helpers.js';
 import { reportNodeStart } from '../heartbeat.js';
 import { recordPhaseStart, recordPhaseEnd } from '../phase-timing-context.js';
 import { updateNodeLifecycle } from '../lifecycle-context.js';
 import { buildFastImplementPrompt } from './prompts/fast-implement.prompt.js';
-import { buildEvidencePrompt } from './prompts/evidence-prompts.js';
-import { parseEvidenceRecords } from './evidence-output-parser.js';
-import { hasSettings, getSettings } from '../../../settings.service.js';
+import { createEvidenceNode } from './evidence.node.js';
 /**
  * Factory that creates the fast-implement node function.
  *
@@ -62,11 +60,15 @@ export function createFastImplementNode(executor) {
             if (!hasWorktreeChanges(cwd) && !hasNewCommits(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) : [];
-            if (!evidenceEnabled) {
-                log.info('Evidence collection disabled via settings — skipping');
+            // --- Evidence sub-agent: capture proof of completion (feature-gated) ---
+            let evidence = [];
+            if (state.enableEvidence) {
+                const evidenceNode = createEvidenceNode(executor);
+                const evidenceResult = await evidenceNode(state);
+                evidence = evidenceResult.evidence ?? [];
+            }
+            else {
+                log.info('Evidence collection disabled — skipping');
             }
             await recordPhaseEnd(timingId, durationMs, {
                 inputTokens: result.usage?.inputTokens,
@@ -106,39 +108,6 @@ export function createFastImplementNode(executor) {
         }
     };
 }
-/**
- * Sub-agent call to collect evidence after fast implementation completes.
- * Graceful degradation: returns empty array on any failure so evidence
- * collection never blocks the workflow.
- */
-async function collectEvidence(executor, state, log) {
-    try {
-        log.info('Collecting evidence (sub-agent)');
-        const commitEvidence = hasSettings() && getSettings().workflow.commitEvidence;
-        const prompt = buildEvidencePrompt(state, { commitEvidence });
-        const options = buildExecutorOptions(state);
-        const result = await retryExecute(executor, prompt, options, { logger: log });
-        try {
-            const evidence = parseEvidenceRecords(result.result);
-            log.info(`Parsed ${evidence.length} evidence record(s)`);
-            saveEvidenceManifest(state, evidence, log);
-            return evidence;
-        }
-        catch (parseErr) {
-            const msg = parseErr instanceof Error ? parseErr.message : String(parseErr);
-            log.error(`Warning: evidence parsing failed: ${msg} — continuing with empty evidence`);
-            return [];
-        }
-    }
-    catch (err) {
-        // Re-throw LangGraph control-flow exceptions
-        if (isGraphBubbleUp(err))
-            throw err;
-        const msg = err instanceof Error ? err.message : String(err);
-        log.error(`Evidence collection failed: ${msg} — continuing without evidence`);
-        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.

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

@@ -1 +1 @@
-{"version":3,"file":"implement.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/implement.node.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAMH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AACpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAqErD,wBAAgB,mBAAmB,CAAC,QAAQ,EAAE,cAAc,IAG5C,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAiQ7E"}
+{"version":3,"file":"implement.node.d.ts","sourceRoot":"","sources":["../../../../../../../../../packages/core/src/infrastructure/services/agents/feature-agent/nodes/implement.node.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAMH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,+DAA+D,CAAC;AACpG,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,aAAa,CAAC;AAoErD,wBAAgB,mBAAmB,CAAC,QAAQ,EAAE,cAAc,IAG5C,OAAO,iBAAiB,KAAG,OAAO,CAAC,OAAO,CAAC,iBAAiB,CAAC,CAAC,CAiQ7E"}

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

@@ -16,7 +16,6 @@ import { recordPhaseStart, recordPhaseEnd, recordApprovalWaitStart, updatePhaseP
 import { updateNodeLifecycle } from '../lifecycle-context.js';
 import { buildImplementPhasePrompt, } from './prompts/implement.prompt.js';
 import { createEvidenceNode } from './evidence.node.js';
-import { hasSettings, getSettings } from '../../../settings.service.js';
 /**
  * Update feature.yaml with current implementation progress.
  * Silently no-ops if the file is missing or unparseable.
@@ -190,8 +189,8 @@ export function createImplementNode(executor) {
             log.info(`All phases complete — ${completedTasks}/${totalTasks} tasks (${elapsed}s)`);
             updateFeatureProgress(state.specDir, totalTasks, totalTasks, 'implementation-complete', null, log);
             messages.push(`[implement] Complete: ${totalTasks} tasks across ${totalPhases} phases (${elapsed}s)`);
-            // --- Evidence sub-agent: capture proof of completion (settings-gated) ---
-            const evidenceEnabled = hasSettings() && getSettings().workflow.enableEvidence;
+            // --- Evidence sub-agent: capture proof of completion (feature-gated) ---
+            const evidenceEnabled = state.enableEvidence;
             let evidenceResult = {};
             if (evidenceEnabled) {
                 const evidenceNode = createEvidenceNode(executor);

package/dist/src/presentation/web/app/actions/get-merge-review-data.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"get-merge-review-data.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/get-merge-review-data.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EACV,eAAe,EAEhB,MAAM,sDAAsD,CAAC;AAK9D,KAAK,wBAAwB,GAAG,eAAe,GAAG;IAAE,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC;AAqCpE,wBAAsB,kBAAkB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,wBAAwB,CAAC,CA2G7F"}
+{"version":3,"file":"get-merge-review-data.d.ts","sourceRoot":"","sources":["../../../../../../src/presentation/web/app/actions/get-merge-review-data.ts"],"names":[],"mappings":"AAQA,OAAO,KAAK,EACV,eAAe,EAEhB,MAAM,sDAAsD,CAAC;AAK9D,KAAK,wBAAwB,GAAG,eAAe,GAAG;IAAE,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC;AAqCpE,wBAAsB,kBAAkB,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,wBAAwB,CAAC,CAmG7F"}

package/dist/src/presentation/web/app/actions/get-merge-review-data.js

@@ -92,16 +92,7 @@ export async function getMergeReviewData(featureId) {
                 const manifestPath = join(evidenceDir, 'manifest.json');
                 if (existsSync(manifestPath)) {
                     const raw = JSON.parse(readFileSync(manifestPath, 'utf-8'));
-                    const normalized = normalizeEvidencePaths(raw, evidenceDir);
-                    // Deduplicate: same type + relativePath means the same evidence entry
-                    const seen = new Set();
-                    evidence = normalized.filter((e) => {
-                        const key = `${e.type}:${e.relativePath}`;
-                        if (seen.has(key))
-                            return false;
-                        seen.add(key);
-                        return true;
-                    });
+                    evidence = normalizeEvidencePaths(raw, evidenceDir);
                 }
             }
             catch {