From a9e7c59845ceba75fc19175cb5d04943c747992e Mon Sep 17 00:00:00 2001 From: 777genius Date: Sat, 9 May 2026 12:22:06 +0300 Subject: [PATCH] fix(team): checkpoint follow-up delivery edits --- ...pencode-runtime-delivery-advisory-phase-1-2-plan.md | 10 ++++++++++ src/main/services/team/ChangeExtractorService.ts | 3 ++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/docs/team-management/opencode-runtime-delivery-advisory-phase-1-2-plan.md b/docs/team-management/opencode-runtime-delivery-advisory-phase-1-2-plan.md index e6aa2931..318b3afe 100644 --- a/docs/team-management/opencode-runtime-delivery-advisory-phase-1-2-plan.md +++ b/docs/team-management/opencode-runtime-delivery-advisory-phase-1-2-plan.md @@ -1192,6 +1192,8 @@ File: ```txt test/main/services/team/OpenCodeRuntimeDeliveryAdvisoryPolicy.test.ts +test/main/services/team/opencode/OpenCodeRuntimeDeliveryProofMatching.test.ts +test/main/services/team/opencode/OpenCodeRuntimeDeliveryProofReader.test.ts ``` Cases: @@ -1214,6 +1216,14 @@ it('suppresses when a newer terminal success exists', () => {}); it('treats payload mismatch as immediate hard error', () => {}); ``` +Add extraction-preservation tests around existing delivery recovery if they are not already covered: + +```ts +it('keeps visible reply recovery by observed message id behavior unchanged after helper extraction', async () => {}); +it('keeps visible reply recovery by taskRefs behavior unchanged after helper extraction', async () => {}); +it('keeps lead-recipient user fallback candidate behavior unchanged after helper extraction', async () => {}); +``` + ### Member advisory service tests Update: diff --git a/src/main/services/team/ChangeExtractorService.ts b/src/main/services/team/ChangeExtractorService.ts index 91ae02b2..6515aa79 100644 --- a/src/main/services/team/ChangeExtractorService.ts +++ b/src/main/services/team/ChangeExtractorService.ts @@ -47,6 +47,7 @@ import type { AgentChangeSet, ChangeStats, TaskChangeSetV2, + TeamConfig, TeamTaskChangeSummariesResponse, TeamTaskChangeSummaryItem, TeamTaskChangeSummaryRequest, @@ -132,7 +133,7 @@ export class ChangeExtractorService { this.taskChangeComputer = new TaskChangeComputer(logsFinder, boundaryParser); } - private readConfigForObservation(teamName: string) { + private readConfigForObservation(teamName: string): Promise { return typeof this.configReader.getConfigSnapshot === 'function' ? this.configReader.getConfigSnapshot(teamName) : this.configReader.getConfig(teamName);