From 98ccbade7edefa3c4e19d7a1ba30319a055df044 Mon Sep 17 00:00:00 2001 From: Tianyi Cui <53024+tianyicui@users.noreply.github.com> Date: Sun, 2 Aug 2026 12:28:50 +0800 Subject: [PATCH] fix(subagent): drop the dead reportDelivery destructure default MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit `apply()` resolved the deployment config through schemastery's `Config()`, which always fills the schema default (`quiet`, pinned by the config test), so the `= 'quiet'` destructure fallback was dead at runtime on every path — and as a defaulted parameter it formed a branch no test could ever exercise against the per-file coverage gate. Remove the fallback and let the schema be the single home of the default. --- packages/subagent/tool-subagent-report/src/index.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/subagent/tool-subagent-report/src/index.ts b/packages/subagent/tool-subagent-report/src/index.ts index e83d65f830..6f6160dc85 100644 --- a/packages/subagent/tool-subagent-report/src/index.ts +++ b/packages/subagent/tool-subagent-report/src/index.ts @@ -88,7 +88,7 @@ export function installReportTool( * @param config - deployment scheduling policy. */ export function apply(ctx: Context, config: Config = {}): void { - const { reportDelivery = 'quiet' } = Config(config) + const { reportDelivery } = Config(config) ctx.subagents.registerContinuableSetup(childCtx => installReportTool(childCtx, ctx, reportDelivery)) }