Skip to content

fix: preserve Feishu markdown around tables - #46638

Closed
xue2han wants to merge 5 commits into
NousResearch:mainfrom
xue2han:main
Closed

fix: preserve Feishu markdown around tables#46638
xue2han wants to merge 5 commits into
NousResearch:mainfrom
xue2han:main

Conversation

@xue2han

@xue2han xue2han commented Jun 15, 2026

Copy link
Copy Markdown

Summary

  • convert outbound Markdown table blocks to fenced text code blocks for Feishu posts
  • keep the rest of the message rendered as rich Markdown instead of downgrading the whole reply to plain text
  • add Feishu outbound payload regression tests

Test Plan

  • python -m pytest tests/gateway/test_feishu.py -k 'markdown_table_reply_stays_rich_post or plain_text_reply_remains_text_message' -q -o 'addopts='
  • python -m pytest tests/gateway/test_feishu.py -q -o 'addopts='
  • manual payload check: table reply now emits msg_type=post with table block fenced as ```text

xue2han added 5 commits June 8, 2026 00:24
Reapply the local Feishu gateway routing, custom fast-provider, auxiliary client cache, and Codex commentary filtering fixes on top of NousResearch/main without merge commits.
@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 duplicate This issue or pull request already exists labels Jun 15, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #21872 — same code-fence approach to Feishu markdown table rendering (convert tables to fenced text, keep rest as rich post) within the saturated Feishu table cluster (#22259, #27469). Note: this PR's diff also carries unrelated stacked-branch changes (agent/codex_*, cron/scheduler.py, cli.py, gateway/config.py, a tinker-atropos submodule) — the actual fix is gateway/platforms/feishu.py only.

@xue2han

xue2han commented Jun 15, 2026

Copy link
Copy Markdown
Author

Closing per request.

@xue2han xue2han closed this Jun 15, 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 duplicate This issue or pull request already exists 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