Summary

• Closes [Bug] Subagent announce leaks raw tool output to Telegram and duplicates parent progress reply #79986
• Replaces fix(agents): stop raw tool output leaking into subagent completion announces #80049 with the maintainer patch for the same reported behavior
• This PR fixes a bug or regression

This keeps the correct core fix from #80049: selectSubagentOutputText() must not fall through to snapshot.latestRawText when a child subagent produced only tool/toolResult output. Tool output is not user-facing completion text, so the requester session should receive (no output) or a post-compaction assistant reply instead of raw command/search output.

The replacement also fixes the gaps in #80049:

• updates both public docs pages that described sanitized tool/toolResult fallback
• adds direct regression coverage for readSubagentOutput()
• strengthens announce-format e2e assertions to prove raw tool text is absent
• adds a changelog entry with contributor and reporter credit
• includes fresh Crabbox proof from a real Linux Testbox run

Verification

• pnpm test src/agents/subagent-announce-output.test.ts src/agents/subagent-announce.format.e2e.test.ts src/agents/subagent-announce-delivery.test.ts src/gateway/server-methods/agent.test.ts
• pnpm exec oxfmt --check --threads=1 CHANGELOG.md docs/automation/tasks.md docs/tools/subagents.md src/agents/subagent-announce-output.ts src/agents/subagent-announce-output.test.ts src/agents/subagent-announce.format.e2e.test.ts
• Crabbox/Testbox tbx_01kr83we61rvr3xjxxbtdme3x2, workflow run https://github.com/openclaw/openclaw/actions/runs/25620248580

Real behavior proof (required for external PRs)

• Behavior or issue addressed: Subagent completion announce could select raw child tool/toolResult content as completion text when no assistant-produced text existed. In Telegram this can surface code/search/exec dumps from the child transcript. This PR makes tool-only histories produce no selected completion output, so the existing announce fallback reports (no output) instead of leaking raw tool text.
• Real environment tested: Blacksmith Testbox via Crabbox, Linux runner, Node 24.13.0, repo branch maint/subagent-no-raw-output synced from this PR checkout.
• Exact steps or command run after this patch: Ran /Users/steipete/Projects/crabbox/bin/crabbox run --provider blacksmith-testbox --blacksmith-org openclaw --blacksmith-workflow .github/workflows/ci-check-testbox.yml --blacksmith-job check --blacksmith-ref main --idle-timeout 90m --ttl 240m --timing-json --shell -- '<probe tool-only child history>; oxfmt --check; targeted pnpm test ...'. The probe imports src/agents/subagent-announce-output.ts, stubs Gateway chat.history with assistant-empty + toolResult: raw grep output, calls readSubagentOutput("agent:main:subagent:child"), and fails if the result contains raw tool output or returns any non-undefined text.
• Evidence after fix (screenshot, recording, terminal capture, console output, redacted runtime log, linked artifact, or copied live output): Crabbox/Testbox terminal capture from tbx_01kr83we61rvr3xjxxbtdme3x2, workflow run https://github.com/openclaw/openclaw/actions/runs/25620248580:

{"scenario":"replacement-tool-only-subagent-output","result":null,"leaked":false}
Checking formatting...
All matched files use the correct format.
[test] passed 3 Vitest shards in 15.91s
blacksmith run summary sync=delegated command=1m2.483s total=1m4.837s exit=0

Pre-fix comparison from Crabbox/Testbox tbx_01kr83fbqn923dkp3zja0stp6d:

{"scenario":"prefix-tool-only-subagent-output","result":"raw grep output","leaked":true}

• Observed result after fix: Tool-only subagent completion history no longer returns raw tool output from readSubagentOutput(). The Crabbox probe result is null/undefined with leaked:false, and the announce-format tests verify requester completion handoff contains (no output) and does not contain the raw tool text.
• What was not tested: A live Telegram bot DM was not exercised. The tested path covers the runtime selector that fed raw child tool text into subagent completion announces; the separate duplicate parent progress symptom in [Bug] Subagent announce leaks raw tool output to Telegram and duplicates parent progress reply #79986 is not claimed fixed by this PR.
• Before evidence (optional but encouraged): Pre-fix Crabbox/Testbox probe returned raw tool output with leaked:true, shown above.

Root Cause

• Root cause: The completion-output selector kept a sanitized raw-text fallback for tool-only child transcripts, even though tool/toolResult content is not assistant-authored completion text.
• Missing detection / guardrail: Existing e2e coverage checked formatted completion output but did not directly assert that readSubagentOutput() rejects tool-only histories or that post-compaction assistant text still wins over raw tool text.
• Contributing context (if known): Public docs still documented tool/toolResult fallback behavior, so the selector and docs drifted together.

Regression Test Plan

• Coverage level that should have caught this: Unit test and seam/e2e announce-format test.
• Target test or file: src/agents/subagent-announce-output.test.ts and src/agents/subagent-announce.format.e2e.test.ts.
• Scenario the test should lock in: Tool-only child histories produce no selected completion text; post-compaction assistant replies still produce selected text; requester handoff does not include raw tool output.
• Why this is the smallest reliable guardrail: The bug is in the selector seam, with one e2e announce-format check to prove the user-visible fallback text.
• Existing test that already covers this (if any): None covered the selector directly before this patch.
• If no new test is added, why not: New tests are added.

User-visible / Behavior Changes

Subagent completion announces no longer surface raw tool output when the child produced no assistant text. They now use the existing (no output) fallback for that case.

Diagram

N/A