Skip to content

fix(weixin): content-aware chatty reply splitting (salvage #7587) - #8230

Merged
teknium1 merged 1 commit into
mainfrom
hermes/hermes-34ffd18e
Apr 12, 2026
Merged

fix(weixin): content-aware chatty reply splitting (salvage #7587)#8230
teknium1 merged 1 commit into
mainfrom
hermes/hermes-34ffd18e

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Summary

Salvages bravohenry's content-aware splitting from PR #7587, adapted to the compact/legacy architecture from #7903.

What changed: Compact mode (default) now detects short chatty exchanges (2-6 short lines, no headings/lists/quotes) and splits them into separate WeChat bubbles. Structured content (tables, headings + body, numbered lists) stays in a single message.

Salvaged from

PR Author Contribution
#7587 @bravohenry Content-aware chatty detection + structured preservation

Test plan

  • 23/23 weixin tests pass (4 new tests for chatty splitting, structured tables, heading+body, four-line blocks)

…tructured content together

Add content-aware splitting to compact mode: short chat-like exchanges
(2-6 short lines without headings/lists/quotes) get separate message
bubbles for a natural chat feel, while structured content (tables,
headings with body, numbered lists) stays in a single message.

Cherry-picked from PR #7587 by bravohenry, adapted to the compact/legacy
split_per_line architecture from #7903.
@teknium1
teknium1 merged commit 81ac62c into main Apr 12, 2026
2 of 4 checks passed
@teknium1
teknium1 deleted the hermes/hermes-34ffd18e branch April 12, 2026 07:38
@bravohenry

Copy link
Copy Markdown
Contributor

Hey man, thanks for merging that! really, really appreciate it!!!!!!

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.

2 participants