fix(send_message): route plugin platform media via adapters#18686
fix(send_message): route plugin platform media via adapters#18686LeonSGP43 wants to merge 1 commit into
Conversation
|
@LeonSGP43 Our custom plugin depend on this feature Can you grant me permission to edit this branch? |
|
Verified this root cause independently on Windows 11 / Telegram DM delivery. I reproduced the exact behavior this PR is addressing:
The key local observation matched this PR direction: I validated the dispatch-style fix locally in a clean worktree with a focused regression test:
I also checked the broader file on clean upstream Based on that repro, this PR is addressing a real user-facing regression, including Telegram (not just plugin-only edge cases). |
|
I compared this PR with a separate local split of plugin This PR looks like the better primary fix for #18422 because it covers the live-adapter path more broadly than my local variant: image, audio/voice, documents, and media-only sends. A few follow-up hardening points from that comparison, in case useful:
I will not open a duplicate PR for the overlapping media-routing change; these are just possible follow-ups/gaps to consider here. |
Summary
Fixes #18422
Verification