From df01ed926a7b8198ee009c3543a3f3afc0551341 Mon Sep 17 00:00:00 2001
From: Tianyi Cui <53024+tianyicui@users.noreply.github.com>
Date: Sun, 2 Aug 2026 12:39:35 +0800
Subject: [PATCH] fix(subagent): type the schema-resolved reportDelivery shape
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Config() applies the schemastery default at runtime, but its return type
keeps the input's optional field, so assert the resolved shape at the
seam — keeping the dead fallback branch gone.
---
 packages/client/ui-subagent/tests/conversation-ui.spec.tsx | 2 +-
 packages/subagent/tool-subagent-report/src/index.ts        | 5 ++++-
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/packages/client/ui-subagent/tests/conversation-ui.spec.tsx b/packages/client/ui-subagent/tests/conversation-ui.spec.tsx
index 2e90893244..1687b1ffc1 100644
--- a/packages/client/ui-subagent/tests/conversation-ui.spec.tsx
+++ b/packages/client/ui-subagent/tests/conversation-ui.spec.tsx
@@ -69,7 +69,7 @@ function props(
   // The zh dictionary is the source of truth for this spec's assertions:
   // the stub interpolates `{name}` params like the locale service does.
   const t = ((key: SubagentKey, params?: Record<string, unknown>): string => {
-    let text = zh[key]
+    let text: string = zh[key]
     for (const [name, value] of Object.entries(params ?? {})) {
       text = text.replaceAll(`{${name}}`, String(value))
     }
diff --git a/packages/subagent/tool-subagent-report/src/index.ts b/packages/subagent/tool-subagent-report/src/index.ts
index 6f6160dc85..962d8cf382 100644
--- a/packages/subagent/tool-subagent-report/src/index.ts
+++ b/packages/subagent/tool-subagent-report/src/index.ts
@@ -88,7 +88,10 @@ export function installReportTool(
  * @param config - deployment scheduling policy.
  */
 export function apply(ctx: Context, config: Config = {}): void {
-  const { reportDelivery } = Config(config)
+  // Config() applies the schema default ('quiet') at runtime; the schemastery
+  // return type keeps the input's optional shape, so assert the resolved
+  // shape here — no runtime fallback exists or is wanted.
+  const { reportDelivery } = Config(config) as { reportDelivery: SubagentReportDelivery }
   ctx.subagents.registerContinuableSetup(childCtx =>
     installReportTool(childCtx, ctx, reportDelivery))
 }