Skip to content

feat(gateway): Updated the skill description & adding native media attachment support for the Yuanbao platform#17411

Closed
loongfay wants to merge 3 commits into
NousResearch:mainfrom
YuanbaoTeam:dev
Closed

feat(gateway): Updated the skill description & adding native media attachment support for the Yuanbao platform#17411
loongfay wants to merge 3 commits into
NousResearch:mainfrom
YuanbaoTeam:dev

Conversation

@loongfay

Copy link
Copy Markdown
Contributor
  1. SKILL.md: Updated the skill description from the original "Yuanbao (元宝) groups: @mention users, query info/members." to a more concise "Yuanbao (元宝) group interaction — @mention users, query group info and members".

  2. send_message_tool.py: Major changes adding native media attachment support for the Yuanbao platform:

  • Added media_files parameter support in YUANBAO platform processing logic

  • Added conditional logic to use specialized sending logic when the platform is YUANBAO and media files are present

  • Implemented chunked message sending with media attachments included in the final chunk

These modifications primarily enhance the media file sending capability of the Yuanbao platform, enabling better handling of messages with attachments.

@alt-glitch alt-glitch added type/feature New feature or request P3 Low — cosmetic, nice to have comp/tools Tool registry, model_tools, toolsets comp/gateway Gateway runner, session dispatch, delivery tool/skills Skills system (list, view, manage) labels Apr 29, 2026
teknium1 pushed a commit that referenced this pull request Apr 29, 2026
_send_yuanbao() already supported media_files= and the user-facing
error strings already advertised yuanbao support, but there was no
dispatch branch in _send_to_platform() actually routing to it. Target
yuanbao in send_message previously fell through to
"Direct sending not yet implemented".

- Add yuanbao media-chunk branch (mirrors Signal/Matrix: media on
  final chunk only).
- Add yuanbao elif in the non-media loop.

Salvage of #17411; SKILL.md description change and redundant
sidebars.ts entry dropped, indentation/trailing-whitespace cleaned up.
@teknium1

Copy link
Copy Markdown
Contributor

Salvaged via #17424 — your tools/send_message_tool.py commit was cherry-picked onto current main with authorship preserved (commit ecaf800). Dropped the SKILL.md description change (the original 51-char version fits our tighter <=60ch target) and the website/sidebars.ts yuanbao entry (already on main). Also cleaned up indentation + trailing whitespace on the Yuanbao comment block. Thanks for the contribution!

@teknium1 teknium1 closed this Apr 29, 2026
donald131 pushed a commit to donald131/hermes-agent that referenced this pull request May 2, 2026
_send_yuanbao() already supported media_files= and the user-facing
error strings already advertised yuanbao support, but there was no
dispatch branch in _send_to_platform() actually routing to it. Target
yuanbao in send_message previously fell through to
"Direct sending not yet implemented".

- Add yuanbao media-chunk branch (mirrors Signal/Matrix: media on
  final chunk only).
- Add yuanbao elif in the non-media loop.

Salvage of NousResearch#17411; SKILL.md description change and redundant
sidebars.ts entry dropped, indentation/trailing-whitespace cleaned up.
02356abc pushed a commit to 02356abc/hermes-agent that referenced this pull request May 14, 2026
_send_yuanbao() already supported media_files= and the user-facing
error strings already advertised yuanbao support, but there was no
dispatch branch in _send_to_platform() actually routing to it. Target
yuanbao in send_message previously fell through to
"Direct sending not yet implemented".

- Add yuanbao media-chunk branch (mirrors Signal/Matrix: media on
  final chunk only).
- Add yuanbao elif in the non-media loop.

Salvage of NousResearch#17411; SKILL.md description change and redundant
sidebars.ts entry dropped, indentation/trailing-whitespace cleaned up.
jsboige pushed a commit to jsboige/hermes-agent that referenced this pull request May 14, 2026
_send_yuanbao() already supported media_files= and the user-facing
error strings already advertised yuanbao support, but there was no
dispatch branch in _send_to_platform() actually routing to it. Target
yuanbao in send_message previously fell through to
"Direct sending not yet implemented".

- Add yuanbao media-chunk branch (mirrors Signal/Matrix: media on
  final chunk only).
- Add yuanbao elif in the non-media loop.

Salvage of NousResearch#17411; SKILL.md description change and redundant
sidebars.ts entry dropped, indentation/trailing-whitespace cleaned up.
dannyJ848 pushed a commit to dannyJ848/hermes-agent that referenced this pull request May 17, 2026
_send_yuanbao() already supported media_files= and the user-facing
error strings already advertised yuanbao support, but there was no
dispatch branch in _send_to_platform() actually routing to it. Target
yuanbao in send_message previously fell through to
"Direct sending not yet implemented".

- Add yuanbao media-chunk branch (mirrors Signal/Matrix: media on
  final chunk only).
- Add yuanbao elif in the non-media loop.

Salvage of NousResearch#17411; SKILL.md description change and redundant
sidebars.ts entry dropped, indentation/trailing-whitespace cleaned up.
gweeteve pushed a commit to gweeteve/hermes-agent that referenced this pull request Jun 2, 2026
_send_yuanbao() already supported media_files= and the user-facing
error strings already advertised yuanbao support, but there was no
dispatch branch in _send_to_platform() actually routing to it. Target
yuanbao in send_message previously fell through to
"Direct sending not yet implemented".

- Add yuanbao media-chunk branch (mirrors Signal/Matrix: media on
  final chunk only).
- Add yuanbao elif in the non-media loop.

Salvage of NousResearch#17411; SKILL.md description change and redundant
sidebars.ts entry dropped, indentation/trailing-whitespace cleaned up.
Egavasyug pushed a commit to Egavasyug/hermes-agent that referenced this pull request Jun 10, 2026
_send_yuanbao() already supported media_files= and the user-facing
error strings already advertised yuanbao support, but there was no
dispatch branch in _send_to_platform() actually routing to it. Target
yuanbao in send_message previously fell through to
"Direct sending not yet implemented".

- Add yuanbao media-chunk branch (mirrors Signal/Matrix: media on
  final chunk only).
- Add yuanbao elif in the non-media loop.

Salvage of NousResearch#17411; SKILL.md description change and redundant
sidebars.ts entry dropped, indentation/trailing-whitespace cleaned up.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/gateway Gateway runner, session dispatch, delivery comp/tools Tool registry, model_tools, toolsets P3 Low — cosmetic, nice to have tool/skills Skills system (list, view, manage) type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants