Skip to content

fix(feishu): include attachments from replied messages - #24285

Closed
asdlem wants to merge 1 commit into
NousResearch:mainfrom
asdlem:fix/feishu-reply-attachments
Closed

fix(feishu): include attachments from replied messages#24285
asdlem wants to merge 1 commit into
NousResearch:mainfrom
asdlem:fix/feishu-reply-attachments

Conversation

@asdlem

@asdlem asdlem commented May 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • fetch full Feishu parent-message context for replies, including cached attachment resources
  • carry replied attachment media into the current event so text follow-ups can reference the local file
  • surface document cache paths for text events that include document media, not only document-typed events

Tests

  • /Users/hywl/.hermes/hermes-agent/venv/bin/python -m pytest -o addopts='' tests/gateway/test_feishu.py tests/gateway/test_reply_to_injection.py

@alt-glitch alt-glitch added type/bug Something isn't working comp/gateway Gateway runner, session dispatch, delivery platform/feishu Feishu / Lark adapter P2 Medium — degraded but workaround exists labels May 12, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #13115 — same fix to include parent media context for Feishu replies. Also related to #14371 (referenced media in group chat).

@asdlem

asdlem commented Jun 29, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #54570 — rebased onto current main with conflicts resolved.

@asdlem asdlem closed this Jun 29, 2026
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 P2 Medium — degraded but workaround exists platform/feishu Feishu / Lark adapter type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants