Skip to content

fix(discord): expose /reasoning levels + reset|show|hide as native slash choices (salvage #55803) - #65732

Merged
teknium1 merged 2 commits into
mainfrom
fix/discord-reasoning-choices
Jul 16, 2026
Merged

fix(discord): expose /reasoning levels + reset|show|hide as native slash choices (salvage #55803)#65732
teknium1 merged 2 commits into
mainfrom
fix/discord-reasoning-choices

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Summary

Discord's native /reasoning slash UI now presents an explicit choices dropdown — effort levels plus reset/show/hide — instead of a free-text box that hid every subcommand and required memorizing valid values.

Salvage of #55803 by @sam7894604, cherry-picked with authorship preserved, plus a follow-up widening the choices to max/ultra (the contributor's list predates #62650).

Changes

  • plugins/platforms/discord/adapter.py: free-text effort param → app_commands.choices dropdown covering all 8 effort levels + reset/show/hide (@sam7894604; max/ultra added in follow-up). Typed-text path unchanged — --global persistence still reachable.
  • scripts/release.py: AUTHOR_MAP entry.

Validation

Before After
Discord /reasoning UI free-text box native dropdown, 11 choices (cap is 25)
reset / show / hide undiscoverable listed
max / ultra absent from contributor branch included

tests/gateway/test_discord_slash_auth.py + tests/plugins/platforms/ — 152 passed.

Closes #55803. Refs #61110 (Telegram picker parity — separate feature).

Infographic

discord-reasoning-choices

@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/plugins Plugin system and bundled plugins platform/discord Discord bot adapter labels Jul 16, 2026
sam7894604 and others added 2 commits July 16, 2026 08:47
The Discord /reasoning command declared a single free-text `effort`
parameter, so the native UI funneled every invocation into that one box
and never surfaced the reset / show / hide subcommands the gateway
handler already supports. Replace the free-text param with an explicit
choices dropdown covering the effort levels plus reset/show/hide,
mirroring the existing /tokens and /voice commands. --global persistence
stays reachable by typing the command as plain text.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…s to max/ultra

The salvaged choices list predates the max/ultra effort levels (#62650);
add them so the Discord dropdown matches the canonical ladder. Discord
caps choices at 25 — we're at 11, plenty of headroom.
@teknium1
teknium1 force-pushed the fix/discord-reasoning-choices branch from 25dc12c to 7e45ccf Compare July 16, 2026 15:47
@teknium1
teknium1 merged commit 6cd5a2c into main Jul 16, 2026
33 checks passed
@teknium1
teknium1 deleted the fix/discord-reasoning-choices branch July 16, 2026 15:58

@tonydwb tonydwb left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review Summary

Verdict: Approved

Fix exposes /reasoning levels plus reset|show|hide as native Discord slash choices (+22 -2). Clean small fix salvaging a prior PR.


Reviewed by Hermes Agent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/plugins Plugin system and bundled plugins P3 Low — cosmetic, nice to have platform/discord Discord bot adapter type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants