hermes - 💡(How to fix) Fix [Bug]: _send_via_adapter ignores media_files parameter — file attachments broken for qqbot [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…

Error Message

Additional Logs / Traceback (optional)

Root Cause

Root Cause Analysis (optional)

Fix Action

Fixed

Code Example

qqbot

---
RAW_BUFFERClick to expand / collapse

Bug Description

File: tools/send_message_tool.py, line 461 Bug: adapter.send(chat_id=chat_id, content=chunk) — media_files and force_document are ignored Impact: send_message(media="/path/to/file.pdf") sends text only, no attachment, on all platforms (qqbot, telegram, etc.) Fix: pass media_files and force_document to adapter.send()

Steps to Reproduce

send_message(media="/tmp/xxx.pdf")

Expected Behavior

get a pdf file

Actual Behavior

url only

Affected Component

Gateway (Telegram/Discord/Slack/WhatsApp)

Messaging Platform (if gateway-related)

No response

Debug Report

qqbot

Operating System

Ubuntu 24.04

Python Version

No response

Hermes Version

No response

Additional Logs / Traceback (optional)

Root Cause Analysis (optional)

No response

Proposed Fix (optional)

send_message_tool.py:461 result = await adapter.send(chat_id=chat_id, content=chunk, media_files=media_files, force_document=force_document)

Are you willing to submit a PR for this?

  • I'd like to fix this myself and submit a PR

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