@pellux/goodvibes-sdk 0.34.2 → 0.36.0

package/dist/platform/agents/wrfc-config.js

@@ -1,3 +1,7 @@
+const WRFC_COMMIT_SCOPES = ['off', 'scoped', 'all'];
+function isWrfcCommitScope(value) {
+    return typeof value === 'string' && WRFC_COMMIT_SCOPES.includes(value);
+}
 export function readWrfcConfig(configManager) {
     const wrfcConfig = configManager.getCategory('wrfc');
     // Number.isFinite (not typeof === 'number') so a NaN/Infinity config value is
@@ -6,6 +10,8 @@ export function readWrfcConfig(configManager) {
     const rawScore = configManager.get('wrfc.scoreThreshold');
     const rawMax = configManager.get('wrfc.maxFixAttempts');
     const rawHeartbeat = configManager.get('wrfc.agentHeartbeatTimeoutMs');
+    const rawTransportRetryLimit = configManager.get('wrfc.transportRetryLimit');
+    const rawTransportRetryDelayMs = configManager.get('wrfc.transportRetryDelayMs');
     return {
         scoreThreshold: Number.isFinite(rawScore)
             ? rawScore
@@ -16,15 +22,30 @@ export function readWrfcConfig(configManager) {
         autoCommit: typeof configManager.get('wrfc.autoCommit') === 'boolean'
             ? configManager.get('wrfc.autoCommit')
             : wrfcConfig?.autoCommit ?? false,
+        commitScope: isWrfcCommitScope(configManager.get('wrfc.commitScope'))
+            ? configManager.get('wrfc.commitScope')
+            : isWrfcCommitScope(wrfcConfig?.commitScope) ? wrfcConfig.commitScope : 'scoped',
         gates: Array.isArray(wrfcConfig?.gates) ? wrfcConfig.gates : [],
         agentHeartbeatTimeoutMs: Number.isFinite(rawHeartbeat)
             ? rawHeartbeat
             : Number.isFinite(wrfcConfig?.agentHeartbeatTimeoutMs) ? wrfcConfig?.agentHeartbeatTimeoutMs : 0,
+        transportRetryLimit: Number.isFinite(rawTransportRetryLimit)
+            ? rawTransportRetryLimit
+            : Number.isFinite(wrfcConfig?.transportRetryLimit) ? wrfcConfig?.transportRetryLimit : 1,
+        transportRetryDelayMs: Number.isFinite(rawTransportRetryDelayMs)
+            ? rawTransportRetryDelayMs
+            : Number.isFinite(wrfcConfig?.transportRetryDelayMs) ? wrfcConfig?.transportRetryDelayMs : 5_000,
     };
 }
 export function getWrfcAgentHeartbeatTimeoutMs(configManager) {
     return readWrfcConfig(configManager).agentHeartbeatTimeoutMs ?? 0;
 }
+export function getWrfcTransportRetryLimit(configManager) {
+    return readWrfcConfig(configManager).transportRetryLimit ?? 1;
+}
+export function getWrfcTransportRetryDelayMs(configManager) {
+    return readWrfcConfig(configManager).transportRetryDelayMs ?? 5_000;
+}
 export function getWrfcScoreThreshold(configManager) {
     return readWrfcConfig(configManager).scoreThreshold ?? 9.9;
 }
@@ -34,6 +55,9 @@ export function getWrfcMaxFixAttempts(configManager) {
 export function getWrfcAutoCommit(configManager) {
     return readWrfcConfig(configManager).autoCommit ?? false;
 }
+export function getWrfcCommitScope(configManager) {
+    return readWrfcConfig(configManager).commitScope ?? 'scoped';
+}
 export function getEnabledWrfcGates(configManager) {
     return readWrfcConfig(configManager).gates.filter((gate) => gate.enabled);
 }

package/dist/platform/agents/wrfc-controller.d.ts

@@ -139,6 +139,16 @@ export declare class WrfcController {
     private activeChildAgentId;
     private onAgentComplete;
     private onAgentFailed;
+    /**
+     * Respawn the most recently spawned child agent after a transport-classified
+     * failure, instead of failing the chain immediately. Bounded by
+     * wrfc.transportRetryLimit (default 1) and tracked via chain.transportRetryCount,
+     * kept separate from fixAttempts/reviewCycles so a transport blip never counts
+     * against the ordinary fix-cycle budget.
+     */
+    private retryTransportFailure;
+    /** Point the chain's role-specific agent-id field at a freshly (re)spawned child. */
+    private rewireChainChildAgentId;
     private onAgentCancelled;
     private startReview;
     private processReview;
@@ -152,7 +162,21 @@ export declare class WrfcController {
     private checkAndRunGatesForAll;
     private autoCommit;
     private autoCommitCandidateAgentIds;
-    private autoCommitMessage;
+    /**
+     * Chain-wide "own edit ledger": every path self-reported as created/modified/deleted by
+     * any engineer/fixer/integrator completion on this chain (including subtask completions),
+     * deduplicated. Primary source is chain.touchedPaths, an incremental accumulator appended
+     * to on every completion (see recordTouchedPaths) so fixer/re-fix passes and resumed
+     * chains are represented, not just the first pass. Falls back to deriving from the
+     * last-stored report slots (chain.engineerReport / chain.integratorReport /
+     * subtask.engineerReport) for chains serialized before touchedPaths existed.
+     *
+     * Self-reported, not ground truth — same accuracy ceiling as verifyEngineerClaims. Per-agent
+     * worktree isolation (AgentWorktree.create) is not wired up in this controller today, so
+     * there is no git-branch-diff signal to corroborate against.
+     */
+    private collectChainTouchedPaths;
+    private buildAutoCommitMessage;
     private failChain;
     private cancelRunningChildren;
     private hasRunningChild;
@@ -168,6 +192,16 @@ export declare class WrfcController {
     private createBaseChain;
     private startEngineeringChain;
     private startCompoundEngineeringChain;
+    /**
+     * Appends a completion report's self-reported filesCreated/filesModified/filesDeleted
+     * into the chain's running edit ledger (chain.touchedPaths). Called for every engineer,
+     * fixer, and integrator completion — not just the first pass — so a chain that goes
+     * through gate-fix or review-fix cycles still has the fixer's edits represented. This is
+     * why it is a standalone accumulator rather than reading the last-stored report field:
+     * chain.engineerReport / subtask.engineerReport are last-write slots that do not reliably
+     * retain every fixer pass (see collectChainTouchedPaths for the consuming side).
+     */
+    private recordTouchedPaths;
     private handleEngineerCompletion;
     private buildSubtaskEngineerTask;
     private buildCompoundIntegrationTask;
@@ -184,6 +218,20 @@ export declare class WrfcController {
     private withRouteReason;
     private completeChainAsPassed;
     private completeOwnerAgent;
+    /**
+     * Roll up token usage across every agent that has ever run under this
+     * chain (the owner plus all phase/subtask children, across every review
+     * and fix cycle — `chain.allAgentIds` already tracks the full roster for
+     * worktree cleanup, so it doubles as the usage-aggregation source). Each
+     * contributor's usage is added in, including the owner's own (normally
+     * zero, but summed rather than ignored in case it is ever populated
+     * directly). Optional fields (reasoningTokens/reasoningSummaryCount) are
+     * only included in the result if at least one contributor reported them,
+     * matching AgentUsage's undefined-means-no-data convention for those.
+     */
+    private aggregateChainUsage;
+    /** Roll up tool-call counts across every agent that has ever run under this chain. */
+    private aggregateChainToolCallCount;
     private upsertWrfcWorkPlanTask;
     private setWrfcWorkPlanTaskStatus;
     private enqueueWrfcWorkPlanTaskOperation;

package/dist/platform/agents/wrfc-controller.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"wrfc-controller.d.ts","sourceRoot":"","sources":["../../../src/platform/agents/wrfc-controller.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAC;AAUnD,OAAO,KAAK,EAGV,SAAS,EACT,sBAAsB,EAMvB,MAAM,iBAAiB,CAAC;AACzB,OAAO,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAChD,OAAO,EAAE,aAAa,EAAE,MAAM,eAAe,CAAC;AAE9C,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,sBAAsB,CAAC;AAC1D,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,yBAAyB,CAAC;AAG3D,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,2BAA2B,CAAC;AACtE,OAAO,KAAK,EAAc,eAAe,EAAE,MAAM,4BAA4B,CAAC;AAC9E,OAAO,KAAK,EACV,8BAA8B,EAE9B,8BAA8B,EAC/B,MAAM,wCAAwC,CAAC;AAUhD,OAAO,EAKL,KAAK,gBAAgB,EACtB,MAAM,kBAAkB,CAAC;AAoB1B,OAAO,EAAE,oBAAoB,EAAE,qBAAqB,EAAE,qBAAqB,EAAE,MAAM,qBAAqB,CAAC;AAEzG;;;GAGG;AACH,eAAO,MAAM,iCAAiC,IAAI,CAAC;AA4BnD,KAAK,eAAe,GAAG,IAAI,CAAC,aAAa,EAAE,OAAO,GAAG,SAAS,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC,aAAa,EAAE,mBAAmB,GAAG,aAAa,CAAC,CAAC,CAAC;AACpI,KAAK,mBAAmB,GAAG;IACzB,kBAAkB,CAAC,KAAK,EAAE,8BAA8B,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;IAC5E,kBAAkB,CAAC,KAAK,EAAE,8BAA8B,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;CAC7E,CAAC;AASF,qBAAa,cAAc;IACzB,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAgC;IACvD,OAAO,CAAC,UAAU,CAAqB;IACvC,OAAO,CAAC,aAAa,CAAyB;IAC9C,OAAO,CAAC,gBAAgB,CAAK;IAC7B,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAS;IACnC,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAc;IACtC,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAS;IACrC,OAAO,CAAC,QAAQ,CAAC,qBAAqB,CAAU;IAChD,yGAAyG;IACzG,OAAO,CAAC,QAAQ,CAAC,2BAA2B,CAAU;IACtD,OAAO,CAAC,UAAU,CAAkB;IACpC,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAyC;IACpE,OAAO,CAAC,WAAW,CAAuE;IAC1F,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAmB;IAChD,OAAO,CAAC,QAAQ,CAAC,aAAa,CAA6C;IAC3E,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAwB;IACvD,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAgC;IACjE,OAAO,CAAC,eAAe,CAAoC;IAC3D,OAAO,CAAC,QAAQ,CAAC,kBAAkB,CAAoC;IACvE,iEAAiE;IACjE,OAAO,CAAC,QAAQ,CAAC,aAAa,CAA6B;IAC3D,4CAA4C;IAC5C,OAAO,CAAC,aAAa,CAA+C;gBAGlE,UAAU,EAAE,eAAe,EAC3B,UAAU,EAAE,IAAI,CAAC,eAAe,EAAE,eAAe,CAAC,EAClD,IAAI,EAAE;QACJ,QAAQ,CAAC,YAAY,EAAE,gBAAgB,CAAC;QACxC,QAAQ,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,KAAK,GAAG,aAAa,CAAC,CAAC;QACnE,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;QAC7B,QAAQ,CAAC,WAAW,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;QAC1C,QAAQ,CAAC,cAAc,CAAC,EAAE,CAAC,MAAM,eAAe,CAAC,GAAG,SAAS,CAAC;QAC9D,QAAQ,CAAC,gBAAgB,CAAC,EAAE,sBAAsB,GAAG,SAAS,CAAC;QAC/D;;;;;WAKG;QACH,QAAQ,CAAC,qBAAqB,CAAC,EAAE,OAAO,CAAC;KAC1C;IAkBH,WAAW,CAAC,WAAW,EAAE,WAAW,GAAG,SAAS;IAyBhD,YAAY,IAAI,MAAM;IAEtB,UAAU,IAAI,WAAW;IAEzB,cAAc,CAAC,WAAW,EAAE,IAAI,CAAC,oBAAoB,EAAE,WAAW,GAAG,YAAY,CAAC,GAAG,IAAI;IAIzF,aAAa,CAAC,UAAU,EAAE,eAAe,GAAG,IAAI;IAOhD,kBAAkB,CAAC,OAAO,EAAE,mBAAmB,GAAG,IAAI,GAAG,SAAS,GAAG,IAAI;IAIzE,QAAQ,CAAC,OAAO,EAAE,MAAM,GAAG,SAAS,GAAG,IAAI;IAE3C,UAAU,IAAI,SAAS,EAAE;IAEzB,WAAW,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO;IA8ErC,qBAAqB,IAAI,MAAM;IAQ/B;;;OAGG;IACH,cAAc,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI;IAW9C;;;;;;;;;;OAUG;IACH,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG,SAAS,GAAG,IAAI;IA+ChD;;;;;;;;;OASG;IACH,WAAW,CAAC,KAAK,EAAE,SAAS,EAAE,KAAK,UAAQ,GAAG,OAAO;IAkBrD,OAAO,IAAI,IAAI;IASf,OAAO,CAAC,UAAU;IAoBlB,OAAO,CAAC,sBAAsB;IA2B9B;;gFAE4E;IAC5E;uGACmG;IACnG,OAAO,CAAC,sBAAsB;IAa9B,OAAO,CAAC,oBAAoB;IAU5B,OAAO,CAAC,oBAAoB;IAiC5B,OAAO,CAAC,aAAa;IAQrB,OAAO,CAAC,cAAc;IAmBtB,OAAO,CAAC,cAAc;IAmDtB;;;OAGG;IACH,OAAO,CAAC,aAAa;IAgBrB,2FAA2F;IAC3F,OAAO,CAAC,YAAY;IA0BpB;;;;;;;;;;;;;;;;OAgBG;IACH,OAAO,CAAC,2BAA2B;IAInC,wFAAwF;IACxF,OAAO,CAAC,kBAAkB;YAQZ,eAAe;IAiF7B,OAAO,CAAC,aAAa;IA0BrB,OAAO,CAAC,gBAAgB;IAYxB,OAAO,CAAC,WAAW;YA8CL,aAAa;IA6G3B,OAAO,CAAC,QAAQ;YAiEF,QAAQ;IAuBtB,OAAO,CAAC,qBAAqB;IA4C7B,OAAO,CAAC,mBAAmB;IAI3B,OAAO,CAAC,0BAA0B;YAIpB,kBAAkB;IA8GhC,OAAO,CAAC,oBAAoB;YASd,sBAAsB;YAqDtB,UAAU;IA0DxB,OAAO,CAAC,2BAA2B;IA6BnC,OAAO,CAAC,iBAAiB;IAKzB,OAAO,CAAC,SAAS;IA0CjB,OAAO,CAAC,qBAAqB;IAU7B,OAAO,CAAC,eAAe;IAQvB,OAAO,CAAC,WAAW;YAqCL,WAAW;IAyBzB,OAAO,CAAC,kBAAkB;IAO1B,OAAO,CAAC,cAAc;IAEtB,OAAO,CAAC,kBAAkB;IAE1B,OAAO,CAAC,mBAAmB;IA0C3B,OAAO,CAAC,mBAAmB;IAM3B,OAAO,CAAC,eAAe;IAMvB,OAAO,CAAC,mBAAmB;IAM3B,OAAO,CAAC,eAAe;IA8CvB,OAAO,CAAC,qBAAqB;IAwC7B,OAAO,CAAC,6BAA6B;IA8CrC,OAAO,CAAC,wBAAwB;IAiIhC,OAAO,CAAC,wBAAwB;IAgBhC,OAAO,CAAC,4BAA4B;IA0BpC,OAAO,CAAC,oBAAoB;YAad,8BAA8B;IAsB5C,OAAO,CAAC,gCAAgC;IA6GxC,OAAO,CAAC,0BAA0B;YAqCpB,4BAA4B;IAiF1C,OAAO,CAAC,uBAAuB;IAiD/B,OAAO,CAAC,gBAAgB;IA+BxB,OAAO,CAAC,0BAA0B;IAalC,OAAO,CAAC,kCAAkC;IAiB1C,OAAO,CAAC,cAAc;IAuCtB,OAAO,CAAC,eAAe;IAIvB,OAAO,CAAC,qBAAqB;IAc7B,OAAO,CAAC,kBAAkB;IAiC1B,OAAO,CAAC,sBAAsB;IAqD9B,OAAO,CAAC,yBAAyB;IA6BjC,OAAO,CAAC,gCAAgC;IAgCxC;;;;;;;;;;;;;;;OAeG;IACH,OAAO,CAAC,eAAe;IAWvB,OAAO,CAAC,oBAAoB;IAiB5B,OAAO,CAAC,sBAAsB;IAQ9B,OAAO,CAAC,iBAAiB;IAOzB,OAAO,CAAC,0BAA0B;IAMlC,OAAO,CAAC,eAAe;CAKxB"}
+{"version":3,"file":"wrfc-controller.d.ts","sourceRoot":"","sources":["../../../src/platform/agents/wrfc-controller.ts"],"names":[],"mappings":"AAEA,OAAO,EAAE,eAAe,EAAE,MAAM,kBAAkB,CAAC;AAUnD,OAAO,KAAK,EAGV,SAAS,EACT,sBAAsB,EAMvB,MAAM,iBAAiB,CAAC;AACzB,OAAO,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAChD,OAAO,EAAE,aAAa,EAAE,MAAM,eAAe,CAAC;AAE9C,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,sBAAsB,CAAC;AAC1D,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,yBAAyB,CAAC;AAG3D,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,2BAA2B,CAAC;AACtE,OAAO,KAAK,EAAc,eAAe,EAAE,MAAM,4BAA4B,CAAC;AAC9E,OAAO,KAAK,EACV,8BAA8B,EAE9B,8BAA8B,EAC/B,MAAM,wCAAwC,CAAC;AAUhD,OAAO,EAQL,KAAK,gBAAgB,EAEtB,MAAM,kBAAkB,CAAC;AAqB1B,OAAO,EAAE,oBAAoB,EAAE,qBAAqB,EAAE,qBAAqB,EAAE,MAAM,qBAAqB,CAAC;AAEzG;;;GAGG;AACH,eAAO,MAAM,iCAAiC,IAAI,CAAC;AA4BnD,KAAK,eAAe,GAAG,IAAI,CAAC,aAAa,EAAE,OAAO,GAAG,SAAS,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC,aAAa,EAAE,mBAAmB,GAAG,aAAa,CAAC,CAAC,CAAC;AACpI,KAAK,mBAAmB,GAAG;IACzB,kBAAkB,CAAC,KAAK,EAAE,8BAA8B,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;IAC5E,kBAAkB,CAAC,KAAK,EAAE,8BAA8B,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;CAC7E,CAAC;AASF,qBAAa,cAAc;IACzB,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAgC;IACvD,OAAO,CAAC,UAAU,CAAqB;IACvC,OAAO,CAAC,aAAa,CAAyB;IAC9C,OAAO,CAAC,gBAAgB,CAAK;IAC7B,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAS;IACnC,OAAO,CAAC,QAAQ,CAAC,OAAO,CAAc;IACtC,OAAO,CAAC,QAAQ,CAAC,WAAW,CAAS;IACrC,OAAO,CAAC,QAAQ,CAAC,qBAAqB,CAAU;IAChD,yGAAyG;IACzG,OAAO,CAAC,QAAQ,CAAC,2BAA2B,CAAU;IACtD,OAAO,CAAC,UAAU,CAAkB;IACpC,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAyC;IACpE,OAAO,CAAC,WAAW,CAAuE;IAC1F,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAmB;IAChD,OAAO,CAAC,QAAQ,CAAC,aAAa,CAA6C;IAC3E,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAwB;IACvD,OAAO,CAAC,QAAQ,CAAC,gBAAgB,CAAgC;IACjE,OAAO,CAAC,eAAe,CAAoC;IAC3D,OAAO,CAAC,QAAQ,CAAC,kBAAkB,CAAoC;IACvE,iEAAiE;IACjE,OAAO,CAAC,QAAQ,CAAC,aAAa,CAA6B;IAC3D,4CAA4C;IAC5C,OAAO,CAAC,aAAa,CAA+C;gBAGlE,UAAU,EAAE,eAAe,EAC3B,UAAU,EAAE,IAAI,CAAC,eAAe,EAAE,eAAe,CAAC,EAClD,IAAI,EAAE;QACJ,QAAQ,CAAC,YAAY,EAAE,gBAAgB,CAAC;QACxC,QAAQ,CAAC,aAAa,EAAE,IAAI,CAAC,aAAa,EAAE,KAAK,GAAG,aAAa,CAAC,CAAC;QACnE,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;QAC7B,QAAQ,CAAC,WAAW,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;QAC1C,QAAQ,CAAC,cAAc,CAAC,EAAE,CAAC,MAAM,eAAe,CAAC,GAAG,SAAS,CAAC;QAC9D,QAAQ,CAAC,gBAAgB,CAAC,EAAE,sBAAsB,GAAG,SAAS,CAAC;QAC/D;;;;;WAKG;QACH,QAAQ,CAAC,qBAAqB,CAAC,EAAE,OAAO,CAAC;KAC1C;IAkBH,WAAW,CAAC,WAAW,EAAE,WAAW,GAAG,SAAS;IAyBhD,YAAY,IAAI,MAAM;IAEtB,UAAU,IAAI,WAAW;IAEzB,cAAc,CAAC,WAAW,EAAE,IAAI,CAAC,oBAAoB,EAAE,WAAW,GAAG,YAAY,CAAC,GAAG,IAAI;IAIzF,aAAa,CAAC,UAAU,EAAE,eAAe,GAAG,IAAI;IAOhD,kBAAkB,CAAC,OAAO,EAAE,mBAAmB,GAAG,IAAI,GAAG,SAAS,GAAG,IAAI;IAIzE,QAAQ,CAAC,OAAO,EAAE,MAAM,GAAG,SAAS,GAAG,IAAI;IAE3C,UAAU,IAAI,SAAS,EAAE;IAEzB,WAAW,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO;IA8ErC,qBAAqB,IAAI,MAAM;IAQ/B;;;OAGG;IACH,cAAc,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI;IAW9C;;;;;;;;;;OAUG;IACH,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG,SAAS,GAAG,IAAI;IA+ChD;;;;;;;;;OASG;IACH,WAAW,CAAC,KAAK,EAAE,SAAS,EAAE,KAAK,UAAQ,GAAG,OAAO;IAkBrD,OAAO,IAAI,IAAI;IASf,OAAO,CAAC,UAAU;IAoBlB,OAAO,CAAC,sBAAsB;IA2B9B;;gFAE4E;IAC5E;uGACmG;IACnG,OAAO,CAAC,sBAAsB;IAa9B,OAAO,CAAC,oBAAoB;IAU5B,OAAO,CAAC,oBAAoB;IAiC5B,OAAO,CAAC,aAAa;IAQrB,OAAO,CAAC,cAAc;IAmBtB,OAAO,CAAC,cAAc;IAmDtB;;;OAGG;IACH,OAAO,CAAC,aAAa;IAgBrB,2FAA2F;IAC3F,OAAO,CAAC,YAAY;IA0BpB;;;;;;;;;;;;;;;;OAgBG;IACH,OAAO,CAAC,2BAA2B;IAInC,wFAAwF;IACxF,OAAO,CAAC,kBAAkB;YAQZ,eAAe;IAiF7B,OAAO,CAAC,aAAa;IA2CrB;;;;;;OAMG;IACH,OAAO,CAAC,qBAAqB;IAkC7B,qFAAqF;IACrF,OAAO,CAAC,uBAAuB;IAa/B,OAAO,CAAC,gBAAgB;IAYxB,OAAO,CAAC,WAAW;YA8CL,aAAa;IAiH3B,OAAO,CAAC,QAAQ;YAiEF,QAAQ;IAuBtB,OAAO,CAAC,qBAAqB;IA4C7B,OAAO,CAAC,mBAAmB;IAI3B,OAAO,CAAC,0BAA0B;YAIpB,kBAAkB;IA8GhC,OAAO,CAAC,oBAAoB;YASd,sBAAsB;YAqDtB,UAAU;IAmFxB,OAAO,CAAC,2BAA2B;IA6BnC;;;;;;;;;;;;OAYG;IACH,OAAO,CAAC,wBAAwB;IAkBhC,OAAO,CAAC,sBAAsB;IA6B9B,OAAO,CAAC,SAAS;IA2CjB,OAAO,CAAC,qBAAqB;IAU7B,OAAO,CAAC,eAAe;IAQvB,OAAO,CAAC,WAAW;YAqCL,WAAW;IAyBzB,OAAO,CAAC,kBAAkB;IAO1B,OAAO,CAAC,cAAc;IAEtB,OAAO,CAAC,kBAAkB;IAE1B,OAAO,CAAC,mBAAmB;IA0C3B,OAAO,CAAC,mBAAmB;IAM3B,OAAO,CAAC,eAAe;IAMvB,OAAO,CAAC,mBAAmB;IAM3B,OAAO,CAAC,eAAe;IAgDvB,OAAO,CAAC,qBAAqB;IAwC7B,OAAO,CAAC,6BAA6B;IA8CrC;;;;;;;;OAQG;IACH,OAAO,CAAC,kBAAkB;IAmB1B,OAAO,CAAC,wBAAwB;IAkIhC,OAAO,CAAC,wBAAwB;IAgBhC,OAAO,CAAC,4BAA4B;IA0BpC,OAAO,CAAC,oBAAoB;YAad,8BAA8B;IAsB5C,OAAO,CAAC,gCAAgC;IA8GxC,OAAO,CAAC,0BAA0B;YAqCpB,4BAA4B;IAiF1C,OAAO,CAAC,uBAAuB;IAiD/B,OAAO,CAAC,gBAAgB;IA+BxB,OAAO,CAAC,0BAA0B;IAclC,OAAO,CAAC,kCAAkC;IAiB1C,OAAO,CAAC,cAAc;IA4CtB,OAAO,CAAC,eAAe;IAIvB,OAAO,CAAC,qBAAqB;IAc7B,OAAO,CAAC,kBAAkB;IA2C1B;;;;;;;;;;OAUG;IACH,OAAO,CAAC,mBAAmB;IA2C3B,sFAAsF;IACtF,OAAO,CAAC,2BAA2B;IAQnC,OAAO,CAAC,sBAAsB;IAqD9B,OAAO,CAAC,yBAAyB;IA6BjC,OAAO,CAAC,gCAAgC;IAgCxC;;;;;;;;;;;;;;;OAeG;IACH,OAAO,CAAC,eAAe;IAWvB,OAAO,CAAC,oBAAoB;IAiB5B,OAAO,CAAC,sBAAsB;IAQ9B,OAAO,CAAC,iBAAiB;IAOzB,OAAO,CAAC,0BAA0B;IAMlC,OAAO,CAAC,eAAe;CAKxB"}

package/dist/platform/agents/wrfc-controller.js

@@ -9,7 +9,8 @@ import { completePlanItemsForAgent } from './wrfc-plan-sync.js';
 import { logger } from '../utils/logger.js';
 import { summarizeError } from '../utils/error-display.js';
 import { emitAgentCompleted, emitAgentFailed, emitAgentProgress, emitAgentRunning, emitWorkflowChainFailed, emitWorkflowFixAttempted, emitWorkflowReviewCompleted, } from '../runtime/emitters/index.js';
-import { getWrfcAutoCommit, getWrfcMaxFixAttempts, getWrfcScoreThreshold, getWrfcAgentHeartbeatTimeoutMs, } from './wrfc-config.js';
+import { getWrfcAutoCommit, getWrfcCommitScope, getWrfcMaxFixAttempts, getWrfcScoreThreshold, getWrfcAgentHeartbeatTimeoutMs, getWrfcTransportRetryLimit, getWrfcTransportRetryDelayMs, } from './wrfc-config.js';
+import { isTransportFailureMessage } from '../types/errors.js';
 import { buildEngineerConstraintAddendum, } from './wrfc-prompt-addenda.js';
 import { completeWrfcOrchestrationNode, createWrfcWorkflowContext, emitWrfcAutoCommitted, emitWrfcCascadeAbort, emitWrfcChainCreated, emitWrfcChainPassed, emitWrfcConstraintsEnumerated, emitWrfcGraphCreated, emitWrfcScoreRegression, emitWrfcStateChanged, failWrfcOrchestrationNode, startWrfcOrchestrationNode, } from './wrfc-runtime-events.js';
 import { runWrfcGateChecks } from './wrfc-gate-runtime.js';
@@ -622,7 +623,10 @@ export class WrfcController {
         const chain = this.findChainByAgentId(agentId);
         if (!chain)
             return;
-        if (agentId === chain.ownerAgentId && isChainTerminal(chain.state))
+        // A non-owner child failure on an already-terminal chain must be a no-op:
+        // mirrors onAgentCancelled and prevents duplicate WORKFLOW_CHAIN_FAILED events
+        // and passed→failed flips when a late/second child failure arrives.
+        if (isChainTerminal(chain.state))
             return;
         if (agentId === chain.ownerAgentId) {
             this.keepOwnerAgentActive(chain, 'Ignored premature owner failure event; WRFC lifecycle is still active');
@@ -637,8 +641,69 @@ export class WrfcController {
             });
             return;
         }
-        this.setWrfcWorkPlanTaskStatus(chain, agentId, 'failed', errorMessage ?? `Agent ${agentId} failed`);
-        this.failChain(chain, errorMessage ?? `Agent ${agentId} failed`);
+        const reason = errorMessage ?? `Agent ${agentId} failed`;
+        // A transport-classified failure of the most recently spawned child gets one
+        // bounded automatic retry (respawn same role/task) before the chain is failed
+        // outright — see retryTransportFailure. Guarding on lastChildSpawn.agentId
+        // matching this exact agentId avoids acting on a stale/duplicate failure event
+        // for an agent that isn't the chain's current active child.
+        if (chain.lastChildSpawn?.agentId === agentId &&
+            isTransportFailureMessage(reason) &&
+            (chain.transportRetryCount ?? 0) < getWrfcTransportRetryLimit(this.configManager)) {
+            this.retryTransportFailure(chain, chain.lastChildSpawn, reason);
+            return;
+        }
+        this.setWrfcWorkPlanTaskStatus(chain, agentId, 'failed', reason);
+        this.failChain(chain, reason, isTransportFailureMessage(reason) ? 'transport' : 'other');
+    }
+    /**
+     * Respawn the most recently spawned child agent after a transport-classified
+     * failure, instead of failing the chain immediately. Bounded by
+     * wrfc.transportRetryLimit (default 1) and tracked via chain.transportRetryCount,
+     * kept separate from fixAttempts/reviewCycles so a transport blip never counts
+     * against the ordinary fix-cycle budget.
+     */
+    retryTransportFailure(chain, spawn, reason) {
+        const limit = getWrfcTransportRetryLimit(this.configManager);
+        const delayMs = getWrfcTransportRetryDelayMs(this.configManager);
+        chain.transportRetryCount = (chain.transportRetryCount ?? 0) + 1;
+        const attempt = chain.transportRetryCount;
+        this.appendOwnerDecision(chain, 'transport_retry', `Transport failure on ${spawn.role} (attempt ${attempt}/${limit}): ${reason}. Retrying in ${Math.round(delayMs / 1000)}s.`, { agentId: spawn.agentId, role: spawn.role });
+        logger.warn('WrfcController: transport failure, retrying child agent', {
+            chainId: chain.id,
+            agentId: spawn.agentId,
+            role: spawn.role,
+            attempt,
+            limit,
+            reason,
+        });
+        const timer = setTimeout(() => {
+            // The chain may have reached a terminal state while this retry was waiting
+            // (e.g. cancelled) — don't resurrect it.
+            if (isChainTerminal(chain.state))
+                return;
+            const record = this.spawnWrfcAgent(chain, spawn.role, spawn.template, spawn.task, spawn.dangerouslyDisableWrfc, spawn.subtaskId);
+            this.registerSpawnedChild(chain, record, spawn.role, spawn.subtaskId);
+            this.rewireChainChildAgentId(chain, spawn.role, record.id);
+        }, delayMs);
+        timer.unref?.();
+    }
+    /** Point the chain's role-specific agent-id field at a freshly (re)spawned child. */
+    rewireChainChildAgentId(chain, role, agentId) {
+        switch (role) {
+            case 'engineer':
+                chain.engineerAgentId = agentId;
+                break;
+            case 'reviewer':
+                chain.reviewerAgentId = agentId;
+                break;
+            case 'fixer':
+                chain.fixerAgentId = agentId;
+                break;
+            case 'integrator':
+                chain.integratorAgentId = agentId;
+                break;
+        }
     }
     onAgentCancelled(agentId, reason) {
         const chain = this.findChainByAgentId(agentId);
@@ -749,9 +814,12 @@ export class WrfcController {
         }
         const maxFixAttempts = getWrfcMaxFixAttempts(this.configManager);
         if (chain.fixAttempts >= maxFixAttempts) {
-            const failureReason = constraintFailure && review.score >= threshold
-                ? `Unsatisfied constraints [${unsatisfiedConstraintIds.join(',')}] after ${chain.fixAttempts} fix attempt${chain.fixAttempts !== 1 ? 's' : ''}`
-                : `Score ${review.score}/10 below threshold ${threshold}/10 after ${chain.fixAttempts} fix attempt${chain.fixAttempts !== 1 ? 's' : ''} — below threshold`;
+            const attemptsLabel = `${chain.fixAttempts} fix attempt${chain.fixAttempts !== 1 ? 's' : ''}`;
+            const failureReason = chain.claimsVerified === false && review.score >= threshold && !constraintFailure
+                ? `Engineer/fixer claims could not be verified on disk (suspected phantom work) after ${attemptsLabel}`
+                : constraintFailure && review.score >= threshold
+                    ? `Unsatisfied constraints [${unsatisfiedConstraintIds.join(',')}] after ${attemptsLabel}`
+                    : `Score ${review.score}/10 below threshold ${threshold}/10 after ${attemptsLabel} — below threshold`;
             this.failChain(chain, failureReason);
             return;
         }
@@ -988,6 +1056,14 @@ export class WrfcController {
     }
     async autoCommit(chain) {
         this.transition(chain, 'committing');
+        const commitScope = getWrfcCommitScope(this.configManager);
+        if (commitScope === 'off') {
+            logger.debug('WrfcController.autoCommit: wrfc.commitScope is off, skipping commit and merge entirely', {
+                chainId: chain.id,
+            });
+            this.completeChainAsPassed(chain);
+            return;
+        }
         const commitCandidateIds = this.autoCommitCandidateAgentIds(chain);
         if (commitCandidateIds.length === 0) {
             this.failChain(chain, 'autoCommit: no write-capable WRFC agent found on chain');
@@ -1001,10 +1077,28 @@ export class WrfcController {
         const worktree = this.createWorktree();
         let completed = false;
         try {
-            const commitMessage = this.autoCommitMessage(chain);
-            const directCommitHash = worktree.commitWorkingTree
-                ? await worktree.commitWorkingTree(commitMessage)
-                : null;
+            const commitMessage = this.buildAutoCommitMessage(chain, commitScope);
+            let directCommitHash = null;
+            if (worktree.commitWorkingTree) {
+                if (commitScope === 'scoped') {
+                    const touchedPaths = this.collectChainTouchedPaths(chain);
+                    if (touchedPaths.length === 0) {
+                        // Do NOT fall back to a full-tree `--all` sweep here — an empty self-reported
+                        // ledger means we genuinely don't know what this chain touched, and committing
+                        // everything dirty in the working tree is exactly the trust bug this fixes.
+                        logger.warn('WrfcController.autoCommit: commitScope is scoped but the chain edit ledger is empty; skipping commit rather than falling back to a full-tree sweep', {
+                            chainId: chain.id,
+                        });
+                    }
+                    else {
+                        directCommitHash = await worktree.commitWorkingTree(commitMessage, touchedPaths);
+                    }
+                }
+                else {
+                    // commitScope === 'all': legacy full-tree sweep, no paths argument.
+                    directCommitHash = await worktree.commitWorkingTree(commitMessage);
+                }
+            }
             let mergedCount = 0;
             for (const agentId of commitCandidateIds) {
                 if (await worktree.merge(agentId)) {
@@ -1068,11 +1162,62 @@ export class WrfcController {
         }
         return candidates;
     }
-    autoCommitMessage(chain) {
-        const firstLine = chain.task.trim().replace(/\s+/g, ' ').slice(0, 72) || chain.id;
-        return `WRFC: ${firstLine}`;
+    /**
+     * Chain-wide "own edit ledger": every path self-reported as created/modified/deleted by
+     * any engineer/fixer/integrator completion on this chain (including subtask completions),
+     * deduplicated. Primary source is chain.touchedPaths, an incremental accumulator appended
+     * to on every completion (see recordTouchedPaths) so fixer/re-fix passes and resumed
+     * chains are represented, not just the first pass. Falls back to deriving from the
+     * last-stored report slots (chain.engineerReport / chain.integratorReport /
+     * subtask.engineerReport) for chains serialized before touchedPaths existed.
+     *
+     * Self-reported, not ground truth — same accuracy ceiling as verifyEngineerClaims. Per-agent
+     * worktree isolation (AgentWorktree.create) is not wired up in this controller today, so
+     * there is no git-branch-diff signal to corroborate against.
+     */
+    collectChainTouchedPaths(chain) {
+        const paths = new Set(chain.touchedPaths ?? []);
+        const fallbackReports = [
+            chain.engineerReport,
+            chain.integratorReport,
+            ...(chain.subtasks ?? []).map((subtask) => subtask.engineerReport),
+        ];
+        for (const report of fallbackReports) {
+            if (report && report.archetype === 'engineer') {
+                const engineerReport = report;
+                for (const path of [...engineerReport.filesCreated, ...engineerReport.filesModified, ...engineerReport.filesDeleted]) {
+                    paths.add(path);
+                }
+            }
+        }
+        return Array.from(paths);
+    }
+    buildAutoCommitMessage(chain, commitScope) {
+        const fullTask = chain.task.trim();
+        const firstLine = fullTask.replace(/\s+/g, ' ').slice(0, 72) || chain.id;
+        const subject = `WRFC: ${firstLine}`;
+        // Subject is length-capped for git log readability; the body below is never truncated —
+        // this is the fix for the "anything past 72 characters is silently discarded" bug.
+        const bodyLines = ['', fullTask || chain.id];
+        if (chain.constraints.length > 0) {
+            bodyLines.push('', `Constraints: ${chain.constraints.length}`);
+        }
+        const gateNames = (chain.gateResults ?? []).map((result) => result.gate);
+        if (gateNames.length > 0) {
+            bodyLines.push(`Gates: ${gateNames.join(', ')}`);
+        }
+        if (chain.subtasks && chain.subtasks.length > 0) {
+            bodyLines.push(`Subtasks: ${chain.subtasks.map((subtask) => subtask.title).join('; ')}`);
+        }
+        if (commitScope === 'scoped') {
+            const touchedPaths = this.collectChainTouchedPaths(chain);
+            bodyLines.push(touchedPaths.length > 0
+                ? `Staged paths (${touchedPaths.length}): ${touchedPaths.join(', ')}`
+                : 'Staged paths: (none — chain edit ledger empty)');
+        }
+        return [subject, ...bodyLines].join('\n');
     }
-    failChain(chain, reason) {
+    failChain(chain, reason, failureKind = 'other') {
         if (chain.state === 'pending') {
             this.chainQueue = this.chainQueue.filter((queued) => queued.record.id !== chain.ownerAgentId);
         }
@@ -1094,6 +1239,7 @@ export class WrfcController {
             this.activeChainCount = Math.max(0, this.activeChainCount - 1);
         }
         chain.error = reason;
+        chain.failureKind = failureKind;
         chain.completedAt = Date.now();
         this.setWrfcWorkPlanTaskStatus(chain, chain.ownerAgentId, 'failed', reason);
         this.cancelRunningChildren(chain);
@@ -1102,7 +1248,7 @@ export class WrfcController {
         });
         this.completeOwnerAgent(chain, 'failed', reason);
         this.workmap.append({ ts: new Date().toISOString(), wrfcId: chain.id, event: 'chain_failed', reason });
-        emitWorkflowChainFailed(this.runtimeBus, createWrfcWorkflowContext(this.sessionId, chain.id), { chainId: chain.id, reason });
+        emitWorkflowChainFailed(this.runtimeBus, createWrfcWorkflowContext(this.sessionId, chain.id), { chainId: chain.id, reason, failureKind });
         logger.error('WrfcController.failChain', { chainId: chain.id, reason });
         this.scheduleChainCleanup(chain);
         this.safeDequeueNext();
@@ -1271,7 +1417,9 @@ export class WrfcController {
             ownerTerminalEmitted: false,
             constraints: [],
             constraintsEnumerated: false,
+            touchedPaths: [],
             createdAt: Date.now(),
+            transportRetryCount: 0,
             ...(subtasks.length > 1 ? { subtasks } : {}),
         };
         this.chains.set(chain.id, chain);
@@ -1333,9 +1481,39 @@ export class WrfcController {
             this.upsertWrfcWorkPlanTask(chain, 'engineer', engineerRecord, 'in_progress', subtask.id);
         }
     }
+    /**
+     * Appends a completion report's self-reported filesCreated/filesModified/filesDeleted
+     * into the chain's running edit ledger (chain.touchedPaths). Called for every engineer,
+     * fixer, and integrator completion — not just the first pass — so a chain that goes
+     * through gate-fix or review-fix cycles still has the fixer's edits represented. This is
+     * why it is a standalone accumulator rather than reading the last-stored report field:
+     * chain.engineerReport / subtask.engineerReport are last-write slots that do not reliably
+     * retain every fixer pass (see collectChainTouchedPaths for the consuming side).
+     */
+    recordTouchedPaths(chain, report) {
+        if (report.archetype !== 'engineer')
+            return;
+        const engineerReport = report;
+        const claimed = [
+            ...engineerReport.filesCreated,
+            ...engineerReport.filesModified,
+            ...engineerReport.filesDeleted,
+        ];
+        if (claimed.length === 0)
+            return;
+        chain.touchedPaths ??= [];
+        const seen = new Set(chain.touchedPaths);
+        for (const path of claimed) {
+            if (!seen.has(path)) {
+                seen.add(path);
+                chain.touchedPaths.push(path);
+            }
+        }
+    }
     handleEngineerCompletion(chain, agentId, report) {
         let reportForReview = report;
         this.completeCurrentNode(chain, report.summary);
+        this.recordTouchedPaths(chain, report);
         if (chain.state === 'engineering') {
             chain.engineerReport = report;
             this.workmap.append({
@@ -1529,6 +1707,7 @@ export class WrfcController {
     handleCompoundEngineerCompletion(chain, subtask, agentId, report) {
         let reportForReview = report;
         this.completeSubtaskNode(chain, subtask, report.summary);
+        this.recordTouchedPaths(chain, report);
         if (subtask.state === 'engineering') {
             subtask.engineerReport = report;
             this.workmap.append({
@@ -1749,6 +1928,7 @@ export class WrfcController {
     }
     handleIntegratorCompletion(chain, agentId, report) {
         chain.integratorReport = report;
+        this.recordTouchedPaths(chain, report);
         this.completeCurrentNode(chain, report.summary);
         this.workmap.append({
             ts: new Date().toISOString(),
@@ -1804,6 +1984,11 @@ export class WrfcController {
         if (selectedRoute?.reason) {
             record.wrfcRouteReason = selectedRoute.reason;
         }
+        // Remember how this child was spawned so a transport-classified failure of
+        // this exact agent can be retried later by respawning with identical inputs
+        // (see retryTransportFailure). Overwritten on every spawn — only ever
+        // describes the most recent child.
+        chain.lastChildSpawn = { agentId: record.id, role, template, task, dangerouslyDisableWrfc, subtaskId };
         return record;
     }
     withRouteReason(baseReason, record) {
@@ -1833,6 +2018,15 @@ export class WrfcController {
         owner.completedAt = Date.now();
         owner.progress = message;
         owner.fullOutput = message;
+        // The owner never runs an LLM turn itself (it only supervises phase
+        // children), so its own usage/toolCallCount stay at the spawn-time zero
+        // default forever unless rolled up here from the real numbers its phase
+        // agents accumulated. This is what makes AgentManager.getStatus()/list()
+        // — the read path TUI per-agent surfaces actually use — return real data
+        // for the owner instead of the never-updated zeros (WO-305 wired the
+        // AGENT_COMPLETED.usage forwarding but nothing populated the source).
+        owner.usage = this.aggregateChainUsage(chain);
+        owner.toolCallCount = this.aggregateChainToolCallCount(chain);
         chain.ownerTerminalEmitted = true;
         const context = {
             sessionId: this.sessionId,
@@ -1846,6 +2040,7 @@ export class WrfcController {
                 durationMs: Math.max(0, owner.completedAt - owner.startedAt),
                 output: message,
                 toolCallsMade: owner.toolCallCount,
+                usage: owner.usage,
             });
         }
         else {
@@ -1857,6 +2052,66 @@ export class WrfcController {
             });
         }
     }
+    /**
+     * Roll up token usage across every agent that has ever run under this
+     * chain (the owner plus all phase/subtask children, across every review
+     * and fix cycle — `chain.allAgentIds` already tracks the full roster for
+     * worktree cleanup, so it doubles as the usage-aggregation source). Each
+     * contributor's usage is added in, including the owner's own (normally
+     * zero, but summed rather than ignored in case it is ever populated
+     * directly). Optional fields (reasoningTokens/reasoningSummaryCount) are
+     * only included in the result if at least one contributor reported them,
+     * matching AgentUsage's undefined-means-no-data convention for those.
+     */
+    aggregateChainUsage(chain) {
+        let inputTokens = 0;
+        let outputTokens = 0;
+        let cacheReadTokens = 0;
+        let cacheWriteTokens = 0;
+        let llmCallCount = 0;
+        let turnCount = 0;
+        let reasoningTokens = 0;
+        let hasReasoningTokens = false;
+        let reasoningSummaryCount = 0;
+        let hasReasoningSummaryCount = false;
+        for (const agentId of chain.allAgentIds) {
+            const usage = this.agentManager.getStatus(agentId)?.usage;
+            if (!usage)
+                continue;
+            inputTokens += usage.inputTokens;
+            outputTokens += usage.outputTokens;
+            cacheReadTokens += usage.cacheReadTokens;
+            cacheWriteTokens += usage.cacheWriteTokens;
+            llmCallCount += usage.llmCallCount;
+            turnCount += usage.turnCount;
+            if (usage.reasoningTokens !== undefined) {
+                hasReasoningTokens = true;
+                reasoningTokens += usage.reasoningTokens;
+            }
+            if (usage.reasoningSummaryCount !== undefined) {
+                hasReasoningSummaryCount = true;
+                reasoningSummaryCount += usage.reasoningSummaryCount;
+            }
+        }
+        return {
+            inputTokens,
+            outputTokens,
+            cacheReadTokens,
+            cacheWriteTokens,
+            ...(hasReasoningTokens ? { reasoningTokens } : {}),
+            llmCallCount,
+            turnCount,
+            ...(hasReasoningSummaryCount ? { reasoningSummaryCount } : {}),
+        };
+    }
+    /** Roll up tool-call counts across every agent that has ever run under this chain. */
+    aggregateChainToolCallCount(chain) {
+        let total = 0;
+        for (const agentId of chain.allAgentIds) {
+            total += this.agentManager.getStatus(agentId)?.toolCallCount ?? 0;
+        }
+        return total;
+    }
     upsertWrfcWorkPlanTask(chain, role, record, status, subtaskId) {
         if (!this.workPlanService)
             return;

package/dist/platform/agents/wrfc-types.d.ts

@@ -10,7 +10,14 @@ export type WrfcState = 'pending' | 'engineering' | 'integrating' | 'reviewing'
 /** Agent role within a WRFC chain. The owner is the durable chain orchestrator. */
 export type WrfcAgentRole = 'owner' | 'orchestrator' | 'engineer' | 'reviewer' | 'fixer' | 'integrator' | 'verifier';
 export type WrfcSubtaskState = 'pending' | 'engineering' | 'reviewing' | 'fixing' | 'passed' | 'failed';
-export type WrfcOwnerDecisionAction = 'chain_created' | 'compound_started' | 'spawn_engineer' | 'spawn_reviewer' | 'spawn_fixer' | 'spawn_integrator' | 'spawn_gate_fixer' | 'subtask_review_passed' | 'subtask_review_failed' | 'review_passed' | 'review_failed' | 'gate_passed' | 'gate_failed' | 'chain_passed' | 'chain_failed' | 'chain_cancelled' | 'owner_completion_ignored' | 'owner_failure_ignored' | 'resume_skipped' | 'resume_started';
+export type WrfcOwnerDecisionAction = 'chain_created' | 'compound_started' | 'spawn_engineer' | 'spawn_reviewer' | 'spawn_fixer' | 'spawn_integrator' | 'spawn_gate_fixer' | 'subtask_review_passed' | 'subtask_review_failed' | 'review_passed' | 'review_failed' | 'gate_passed' | 'gate_failed' | 'chain_passed' | 'chain_failed' | 'chain_cancelled' | 'owner_completion_ignored' | 'owner_failure_ignored' | 'resume_skipped' | 'resume_started' | 'transport_retry';
+/**
+ * Why a chain reached the terminal 'failed' state. Distinguishes a transport/network
+ * blip (which gets one automatic retry, see WrfcChain.transportRetryCount) from an
+ * ordinary review/gate rejection, so a consumer (e.g. the TUI) can render the two
+ * differently instead of showing every failure identically.
+ */
+export type WrfcChainFailureKind = 'transport' | 'other';
 export interface WrfcOwnerDecision {
     id: string;
     ts: string;
@@ -94,6 +101,36 @@ export interface WrfcChain {
     /** Durable audit of owner orchestration choices. */
     ownerDecisions: WrfcOwnerDecision[];
     error?: string | undefined;
+    /**
+     * Why the chain failed. Only meaningful when state is 'failed'. Optional field —
+     * absent on chains persisted before this field was introduced (deserializeChain
+     * treats it as undefined rather than requiring a schema-version bump).
+     */
+    failureKind?: WrfcChainFailureKind | undefined;
+    /**
+     * Number of times this chain has auto-retried a transport-classified child-agent
+     * failure by respawning the same role (bounded by wrfc.transportRetryLimit).
+     * Kept separate from fixAttempts/reviewCycles: a transport retry is not a fix
+     * cycle and must not count against maxFixAttempts. Optional/defaults to 0 —
+     * absent on chains persisted before this field was introduced.
+     */
+    transportRetryCount?: number | undefined;
+    /**
+     * Parameters used for the most recent spawnWrfcAgent call, kept so a
+     * transport-classified failure of that same agent can be retried by respawning
+     * with identical inputs. Overwritten on every subsequent spawn, so it only ever
+     * describes the latest child. A chain resumed from persisted JSON that hasn't
+     * spawned anything since resume simply has no retry candidate here and fails
+     * closed via failChain, same as any other missing-optional-field case.
+     */
+    lastChildSpawn?: {
+        agentId: string;
+        role: 'engineer' | 'reviewer' | 'fixer' | 'integrator';
+        template: 'engineer' | 'reviewer' | 'integrator';
+        task: string;
+        dangerouslyDisableWrfc: boolean;
+        subtaskId?: string | undefined;
+    } | undefined;
     /** Buffered agent completion — set when agent finishes while chain is still queued/pending. */
     bufferedCompletion?: {
         agentId: string;
@@ -119,6 +156,16 @@ export interface WrfcChain {
      * false = verification ran and found missing claims (phantom work detected).
      */
     claimsVerified?: boolean | undefined;
+    /**
+     * Running ledger of paths (filesCreated/filesModified/filesDeleted) self-reported by every
+     * engineer/fixer/integrator completion across the chain's lifetime — including subtask
+     * completions on compound chains. Appended to incrementally (not derived from a single
+     * "latest report" field) so it still reflects fixer/re-fix passes after resume, and so a
+     * pre-interruption pass is never lost. Consumed by collectChainTouchedPaths() to scope
+     * the auto-commit `git add` when wrfc.commitScope is 'scoped'. Self-reported, not ground
+     * truth — see verifyEngineerClaims for the same accuracy caveat.
+     */
+    touchedPaths?: string[] | undefined;
 }
 /** Quality gate definition. */
 export interface QualityGate {