Skip to content

fix: chunk application emoji listings - #147

Merged
vycdev merged 1 commit into
vycdev:developfrom
vycdev2:fix/chunk-emoji-list-20260817
Aug 23, 2026
Merged

fix: chunk application emoji listings#147
vycdev merged 1 commit into
vycdev:developfrom
vycdev2:fix/chunk-emoji-list-20260817

Conversation

@vycdev2

@vycdev2 vycdev2 commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • split listemojis output into Discord-safe messages instead of sending one oversized reply
  • preserve emoji order, repeat the list heading for context, and avoid splitting Unicode surrogate pairs
  • add regression coverage for large listings and oversized entries

Verification

  • dotnet test Morpheus.Tests/Morpheus.Tests.csproj --no-restore --filter FullyQualifiedName~EmojisModuleTests --logger "console;verbosity=minimal" — passed: 29/29
  • dotnet build --no-restore — passed with 0 errors (one existing SQLitePCLRaw vulnerability warning)
  • dotnet test --no-build --logger "console;verbosity=minimal" — passed: 430, skipped: 1
  • python3 -m unittest tools.test_generate_commands_md — passed: 3/3
  • git diff --check upstream/develop...HEAD — passed

Risk

  • Low: this only changes how a long application-emoji listing is divided across replies; short listings retain the same content.

This was generated by an AI agent (vycdev2). Please verify any changes before merging or applying.

@vycdev
vycdev merged commit fdf5d82 into vycdev:develop Aug 23, 2026
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