Skip to content

fix(deps): add dingtalk-stream and httpx to messaging extras - #2065

Closed
ygd58 wants to merge 3 commits into
NousResearch:mainfrom
ygd58:fix/dingtalk-missing-from-messaging-extras
Closed

fix(deps): add dingtalk-stream and httpx to messaging extras#2065
ygd58 wants to merge 3 commits into
NousResearch:mainfrom
ygd58:fix/dingtalk-missing-from-messaging-extras

Conversation

@ygd58

@ygd58 ygd58 commented Mar 19, 2026

Copy link
Copy Markdown
Contributor

Fixes #2062

Problem

gateway/platforms/dingtalk.py requires dingtalk-stream but the package was not listed anywhere in pyproject.toml, causing an ImportError for users who installed with .[all] or .[messaging].

Fix

  • Added a dedicated dingtalk = ["dingtalk-stream>=1.0.0"] optional-dependency group
  • Added hermes-agent[dingtalk] to the [all] extras group

This follows the same pattern as other platform-specific extras (slack, sms, homeassistant, etc.) and allows users to install only DingTalk deps with pip install hermes-agent[dingtalk].

How to Test

pip install -e ".[all]"
pip show dingtalk-stream # should be installed

@iborazzi iborazzi mentioned this pull request Mar 19, 2026
1 task
teknium1 added a commit that referenced this pull request Mar 22, 2026
Cherry-picked from PR #2065 by @ygd58. Fixes #2062.

dingtalk-stream was required by gateway/platforms/dingtalk.py but not
listed in pyproject.toml, causing ImportError on pip install .[all].
Adds dingtalk extras group following the same pattern as slack/sms/etc.
@teknium1

Copy link
Copy Markdown
Contributor

Merged via PR #2452. Cherry-picked with authorship preserved. Thanks!

@teknium1 teknium1 closed this Mar 22, 2026
outsourc-e pushed a commit to outsourc-e/hermes-agent that referenced this pull request Mar 26, 2026
Cherry-picked from PR NousResearch#2065 by @ygd58. Fixes NousResearch#2062.

dingtalk-stream was required by gateway/platforms/dingtalk.py but not
listed in pyproject.toml, causing ImportError on pip install .[all].
Adds dingtalk extras group following the same pattern as slack/sms/etc.
angelburgosrosado pushed a commit to angelburgosrosado/hermes-agent that referenced this pull request Apr 27, 2026
Cherry-picked from PR NousResearch#2065 by @ygd58. Fixes NousResearch#2062.

dingtalk-stream was required by gateway/platforms/dingtalk.py but not
listed in pyproject.toml, causing ImportError on pip install .[all].
Adds dingtalk extras group following the same pattern as slack/sms/etc.
02356abc pushed a commit to 02356abc/hermes-agent that referenced this pull request May 14, 2026
Cherry-picked from PR NousResearch#2065 by @ygd58. Fixes NousResearch#2062.

dingtalk-stream was required by gateway/platforms/dingtalk.py but not
listed in pyproject.toml, causing ImportError on pip install .[all].
Adds dingtalk extras group following the same pattern as slack/sms/etc.
gweeteve pushed a commit to gweeteve/hermes-agent that referenced this pull request Jun 2, 2026
Cherry-picked from PR NousResearch#2065 by @ygd58. Fixes NousResearch#2062.

dingtalk-stream was required by gateway/platforms/dingtalk.py but not
listed in pyproject.toml, causing ImportError on pip install .[all].
Adds dingtalk extras group following the same pattern as slack/sms/etc.
waefrebeorn pushed a commit to waefrebeorn/slermes that referenced this pull request Jul 2, 2026
Cherry-picked from PR NousResearch#2065 by @ygd58. Fixes NousResearch#2062.

dingtalk-stream was required by gateway/platforms/dingtalk.py but not
listed in pyproject.toml, causing ImportError on pip install .[all].
Adds dingtalk extras group following the same pattern as slack/sms/etc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: 钉钉无法连接。

2 participants