openclaw - 💡(How to fix) Fix fix(a2a): resolve direct/dm announce target from session key fallback [1 pull requests]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…

A2A announce target resolution may fail for direct-session keys shaped like agent:main:<channel>:direct:<peerId> (including dm alias) when the primary session parser path returns null.

Root Cause

A2A announce target resolution may fail for direct-session keys shaped like agent:main:<channel>:direct:<peerId> (including dm alias) when the primary session parser path returns null.

Fix Action

Fixed

RAW_BUFFERClick to expand / collapse

Summary

A2A announce target resolution may fail for direct-session keys shaped like agent:main:<channel>:direct:<peerId> (including dm alias) when the primary session parser path returns null.

Impact

Direct announce delivery in A2A flows can miss the intended recipient for certain session-key formats.

Repro

  1. Prepare a direct session key: agent:main:wecom:direct:<peerId> (or ...:dm:<peerId>).
  2. Trigger announce target resolution from session key.
  3. Observe unresolved/incorrect direct target in fallback cases.

Expected

Announce target resolves correctly for direct/dm key variants and stays safe when plugin-level delivery target resolution returns null.

Proposed Fix

  • Add fallback parsing for agent-scoped direct/dm session keys.
  • Reuse channel plugin resolveDeliveryTarget when available.
  • Fallback to safe normalized user:<peerId> target when plugin resolution returns null.
  • Add focused tests for direct, dm alias, and null delivery-target fallback.

Notes

  • Scope is intentionally limited to announce target resolution.
  • No behavior change for group/thread paths.

Evidence

Manual verification screenshots are available and will be attached in the linked PR with redaction where needed.

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING