Skip to content

фикс: задвоенные слеш-команды#12

Merged
iAmScienceMan merged 2 commits into
mainfrom
fix/duplicate-slash-commands
May 24, 2026
Merged

фикс: задвоенные слеш-команды#12
iAmScienceMan merged 2 commits into
mainfrom
fix/duplicate-slash-commands

Conversation

@iAmScienceMan

Copy link
Copy Markdown
Contributor

суть

команды задвоились в ui дискорда.

до f0e692d синк дерева был глобальным (tree.sync()), после переезд на guild-scoped через copy_global_to + sync(guild=GUILD). но глобально зарегистрированные на стороне дискорда команды никто не снёс — он продолжает их отдавать в ui вместе с guild-копиями.

фикс

после copy_global_to чистим глобальные из локального дерева и синкаем пустой глобальный список — это сносит их на стороне дискорда. идемпотентно, можно оставить навсегда.

test plan

  • поднять бота, проверить что в логах Synced N command(s) to guild ...
  • в дискорде открыть автокомплит слешей — каждая команда показывается ровно один раз

команды задвоились потому что до f0e692d синк был глобальным,
а после переезда на guild-scoped глобально зарегистрированные на
стороне discord команды никто не снял. дискорд показывает их в ui
вместе с guild-копиями.

копируем глобальные в guild, чистим глобальные из локального дерева
и синкаем пустой глобальный список чтобы снести их на стороне дискорда.
@codecov

codecov Bot commented May 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@insvrg3ncy insvrg3ncy self-requested a review May 24, 2026 12:01
@insvrg3ncy

Copy link
Copy Markdown
Contributor

кодеков пшел нхацй

@iAmScienceMan

Copy link
Copy Markdown
Contributor Author

стоп не мерджи

@iAmScienceMan iAmScienceMan added this pull request to the merge queue May 24, 2026
Merged via the queue into main with commit cc250b4 May 24, 2026
5 checks passed
@iAmScienceMan iAmScienceMan deleted the fix/duplicate-slash-commands branch May 24, 2026 12:03
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