mirrors/hermes-agent
1
0
Fork 0
mirror of https://github.com/NousResearch/hermes-agent.git synced 2026-07-26 17:38:36 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 6

fix(feishu): keep msg_type=post consistent across every chunk of a long markdown reply (#26841)

Browse source 
Transplant of PR #26848 onto the plugin adapter path
(plugins/platforms/feishu/adapter.py — the original PR targeted the
since-removed gateway/platforms/feishu.py).

``send`` classifies each chunk independently, so chunk 1 of a long
markdown reply (often plain prose) went out as msg_type=text while
later chunks rendered as post — literal **bold**/## heading markers
in the Feishu client. Lock the decision at the whole-message level:
compute prefer_post once from the full formatted message and pass it
to _build_outbound_payload per chunk.

The original PR's per-chunk table exemption is intentionally dropped:
tables now route through post/md (issue #52786 cluster fix), so the
exemption would reintroduce the raw-table downgrade.

Co-authored-by: Hermes Agent <hermes@nousresearch.com>
This commit is contained in:
xxxigm 2026-07-20 08:51:43 -07:00 • committed by Teknium
parent 17a99f6b15
commit 9403b4f8ba
2 changed files with 124 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

23
plugins/platforms/feishu/adapter.py
View file

@ -1910,11 +1910,21 @@ class FeishuAdapter(BasePlatformAdapter):
formatted = self.format_message(content)
chunks = self.truncate_message(formatted, self.MAX_MESSAGE_LENGTH)
# When chunking splits a long markdown response, an individual chunk
# can end up as plain prose that doesn't match the per-chunk hint
# regex — so it would be sent as ``msg_type=text`` and the user would
# see literal ``**bold``/``## heading``/code fences in the Feishu
# client while other chunks render correctly. Lock the markdown
# decision at the whole-message level so every chunk consistently
# uses ``post``. See #26841.
prefer_post = bool(_MARKDOWN_HINT_RE.search(formatted))
last_response = None
try:
for chunk in chunks:
msg_type, payload = self._build_outbound_payload(chunk)
msg_type, payload = self._build_outbound_payload(
chunk, prefer_post=prefer_post,
)
try:
response = await self._feishu_send_with_retry(
chat_id=chat_id,
@ -4545,14 +4555,21 @@ class FeishuAdapter(BasePlatformAdapter):
# Outbound payload construction and send pipeline
# =========================================================================
def _build_outbound_payload(self, content: str) -> tuple[str, str]:
def _build_outbound_payload(
self, content: str, *, prefer_post: bool = False,
) -> tuple[str, str]:
# Empirically (issue #52786), current Feishu clients render markdown
# tables inside ``post``-type ``md`` elements natively. The previous
# table-downgrade branch forced any table-containing message to
# ``text``, which left Feishu readers seeing the raw pipe-and-dash
# source instead of a rendered table. Trust the common markdown path
# for table content too.
if _MARKDOWN_HINT_RE.search(content):
#
# ``prefer_post`` lets ``send`` treat the chunk as part of a larger
# markdown document: when a long markdown reply is split at
# MAX_MESSAGE_LENGTH, the per-chunk regex would otherwise
# mis-classify a plain-prose chunk as ``text``. See #26841.
if prefer_post or _MARKDOWN_HINT_RE.search(content):
return "post", _build_markdown_post_payload(content)
text_payload = {"text": content}
return "text", json.dumps(text_payload, ensure_ascii=False)