Skip to content

fix: limit choose command replies - #152

Merged
vycdev merged 1 commit into
vycdev:developfrom
vycdev2:fix/limit-choose-response-20260820
Aug 23, 2026
Merged

fix: limit choose command replies#152
vycdev merged 1 commit into
vycdev:developfrom
vycdev2:fix/limit-choose-response-20260820

Conversation

@vycdev2

@vycdev2 vycdev2 commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • truncate oversized choose options before adding the response prefix so replies stay within Discord’s 2,000-character limit
  • preserve UTF-16 surrogate pairs when truncating and add regression coverage for both the limit and emoji boundary

Verification

  • /opt/data/home/.dotnet/dotnet build --no-restore — passed: 0 errors (1 existing SQLite package vulnerability warning)
  • /opt/data/home/.dotnet/dotnet test --no-restore --logger 'console;verbosity=minimal' — passed: 430, skipped: 1 globalization-invariant culture test
  • python3 -m unittest tools.test_generate_commands_md — passed: 3 tests
  • git diff --check — passed

Risk

  • Low — only oversized options are shortened, and existing blank-line/whitespace parsing behavior remains covered.

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

@vycdev
vycdev merged commit 27e4ce4 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