Skip to content

Shorten descriptions to fit command length limit#22

Merged
getBoolean merged 2 commits into
prodfrom
master
Jun 22, 2026
Merged

Shorten descriptions to fit command length limit#22
getBoolean merged 2 commits into
prodfrom
master

Conversation

@getBoolean

Copy link
Copy Markdown
Owner

No description provided.

@getBoolean getBoolean enabled auto-merge June 22, 2026 07:00
@cursor

cursor Bot commented Jun 22, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Changes are mostly validation, test infrastructure, and shorter Discord UI copy; registration now fails fast instead of silently on Discord API errors.

Overview
Keeps slash commands under Discord’s 8000-character combined limit so bulk registration no longer fails with APPLICATION_COMMAND_TOO_LARGE.

Adds command-size.utils and blocks registerCommands when any built command is oversized, with a clear error listing offenders. Vitest (commands.test.ts) asserts every command builds, stays under the limit, and has unique names; db.ts uses an in-memory SQLite DB when VITEST is set so loading the command graph does not touch data/main.sqlite.

/events add/set option descriptions for announcement, room ping, and Discord event templates are shortened to point admins at /events help instead of long inline placeholder lists. Removes a duplicate VoiceCommand entry in commands.loader.ts (also covered by the duplicate-name test).

Reviewed by Cursor Bugbot for commit 83feaeb. Bugbot is set up for automated code reviews on this repo. Configure here.

@getBoolean getBoolean merged commit 1c49b5e into prod Jun 22, 2026
7 checks passed
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.

1 participant