From 1fa3623eae034905f7e92bf8739e9f7d0f1742ca Mon Sep 17 00:00:00 2001
From: fengyuwusong <fyws@fengyuwusong.cn>
Date: Fri, 6 Mar 2026 12:28:29 +0800
Subject: [PATCH] fix: clamp minDelayMs to timer-safe maximum

Also improve code clarity with explicit rawMinDelayMs/rawMaxDelayMs variables.
---
 src/agents/subagent-announce.ts | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/agents/subagent-announce.ts b/src/agents/subagent-announce.ts
index f02362818c2..93af932e1e2 100644
--- a/src/agents/subagent-announce.ts
+++ b/src/agents/subagent-announce.ts
@@ -76,12 +76,14 @@ type AnnounceRetryConfig = {
 
 function resolveAnnounceRetryConfig(cfg: ReturnType<typeof loadConfig>): AnnounceRetryConfig {
   const retry = cfg.agents?.defaults?.subagents?.announceRetry;
-  const minDelayMs =
+  // Clamp minDelayMs to timer-safe maximum
+  const rawMinDelayMs =
     typeof retry?.minDelayMs === "number" &&
     Number.isFinite(retry.minDelayMs) &&
     retry.minDelayMs > 0
       ? Math.floor(retry.minDelayMs)
       : DEFAULT_ANNOUNCE_RETRY_MIN_DELAY_MS;
+  const minDelayMs = Math.min(rawMinDelayMs, MAX_TIMER_SAFE_TIMEOUT_MS);
   const rawMaxDelayMs =
     typeof retry?.maxDelayMs === "number" &&
     Number.isFinite(retry.maxDelayMs) &&