PR #72213: fix(discord): deduplicate tool progress entries in block streaming mode [AI-assisted]

• Repository: openclaw/openclaw
• Author: luyao618
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/72213

Description (problem / solution / changelog)

🤖 AI-assisted (built with Hermes orchestration). Test level: fully tested. Prompt summary available on request.

Summary

• Problem: In Discord block streaming mode, tool progress lines show duplicate entries for the same tool (e.g., session_status appearing multiple times).
• Why it matters: Duplicate progress lines clutter the streaming preview and confuse users about actual tool activity.
• What changed: Extended the dedup check in pushPreviewToolProgress to compare against all existing entries (not just the last one), with bare name normalization (stripping the "tool: " prefix).
• What did NOT change (scope boundary): No changes to tool execution logic, event dispatching, or any other streaming mode.

Change Type (select all)

• Bug fix

Scope (select all touched areas)

• Extensions (Discord)

Linked Issue/PR

• Closes #72205
• This PR fixes a bug or regression

Root Cause

• Root cause: pushPreviewToolProgress only compared the new entry against the last entry in the array (lines[lines.length - 1] !== line). When onToolStart emits "tool: session_status" and onItemEvent later emits "session_status", they differ in the "tool: " prefix and both pass the dedup check, resulting in duplicate entries for the same tool.
• Missing detection / guardrail: No test for duplicate tool progress entries across different event sources (onToolStart vs onItemEvent).
• Contributing context: The two code paths (onToolStart at line ~990 and onItemEvent at line ~997) independently push tool names with different formatting conventions.

Regression Test Plan

• Coverage level that should have caught this:
  • Unit test
• Target test or file: extensions/discord/src/monitor/message-handler.process.test.ts
• Scenario the test should lock in: When pushPreviewToolProgress is called with entries that differ only by the "tool: " prefix (e.g., "tool: session_status" then "session_status"), only one entry should remain.
• Why this is the smallest reliable guardrail: Directly tests the dedup function with the exact input pattern that caused the bug.
• Existing test that already covers this: N/A
• If no new test is added, why not: N/A

User-visible / Behavior Changes

• Discord block streaming mode no longer shows duplicate tool names in the "Working…" progress section.

Diagram (if applicable)

N/A

Security Impact (required)

• New permissions/capabilities? No
• This fix only affects display-layer dedup logic for tool progress previews. No security-sensitive data flows are touched.

Changed files

• extensions/discord/src/monitor/message-handler.process.test.ts (modified, +22/-0)
• extensions/discord/src/monitor/message-handler.process.ts (modified, +7/-1)