-
-
Notifications
You must be signed in to change notification settings - Fork 79.2k
Subagent completion announce retry-limit logs hide the underlying delivery error #84272
Copy link
Copy link
Closed
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.clawsweeper:queueable-fixClawSweeper marked this issue as an existing queue_fix_pr work candidate.ClawSweeper marked this issue as an existing queue_fix_pr work candidate.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:message-lossChannel message delivery can be lost, duplicated, or misrouted.Channel message delivery can be lost, duplicated, or misrouted.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:fix-shape-clearClawSweeper found a clear likely implementation shape for this issue.ClawSweeper found a clear likely implementation shape for this issue.clawsweeper:queueable-fixClawSweeper marked this issue as an existing queue_fix_pr work candidate.ClawSweeper marked this issue as an existing queue_fix_pr work candidate.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:message-lossChannel message delivery can be lost, duplicated, or misrouted.Channel message delivery can be lost, duplicated, or misrouted.issue-rating: 🦞 diamond lobsterVery strong issue quality with high-confidence source-level or clear reproduction.Very strong issue quality with high-confidence source-level or clear reproduction.
Type
Fields
Give feedbackNo fields configured for issues without a type.
Summary
In OpenClaw 2026.5.12, failed subagent completion announcements can end with only:
The line does not include the real delivery failure. On systems where the gateway LaunchAgent discards stderr, the more useful per-attempt diagnostic can be lost, leaving no way to tell from
gateway.logwhether the failure was a gateway timeout, Slack/outbound configuration issue, routed dispatch failure, model failure, missing message-tool delivery, or no visible final reply.Environment
f066dd2)~/.openclaw/logs/gateway.log/dev/nullObserved
Live
gateway.loghad retry-limit warnings for several runs:Each run had three preceding
subagent_delivery_target firedevents withexpectsCompletionMessage=true, but the retry-limit warning had no delivery error.Historical
gateway.err.logshows the missing diagnostic previously carried the useful cause, for example:Other historical causes included
gateway timeout after ..., model/fallback failures, and routed-dispatch failures.Relevant Code Path
subagent-registry-32aElbRE.js:resumeSubagentRun()gives up after 3 retries and callsfinalizeResumedAnnounceGiveUp({ reason: "retry-limit" }).subagent-registry-32aElbRE.js:onDeliveryResultcan format and persistentry.lastAnnounceDeliveryError.subagent-announce-delivery-DzsdC5tX.js: completion messages use direct-primary delivery first, then queue fallback, and concrete failures are returned indelivery.error.subagent-announce-Cdo94lsz.js: direct announce failures can be logged per attempt.Expected
Operators should be able to diagnose a failed subagent completion announcement from
gateway.logalone, even when stderr is discarded.Proposed Fix
entry.lastAnnounceDeliveryErroror equivalent.Subagent announce give up (retry-limit)and expiry warnings.Suggested warning shape:
This is observability-only. It should not change retry policy, delivery ordering, Slack behavior, cleanup, or hook semantics.