Not static, guild specific hybrid commands

[Kadethra]

Hey,
So I would like to make commands customisable for users, so they can decide which one they have on their guild and which not. I am using hybrid commands which makes a bit more difficult. I tried using the guilds parameter in bot.add_cog, but it doesn't work with hybrids so I opened a feature request. I asked if it's possible to add that hybrid commands, or at least the app_command part of them can be affected bythe guilds parameter and got the answer, that one can already do it with decorators (app_commands.guilds).
Now my question: Since user can change the options, so it's not static, how can I apply it?

Hope my problem is understandable so far.

[LostLuma]

You can call CommandTree.add_command / CommandTree.remove_command and then sync the tree for the guild(s) you added / removed the command to at any point, allowing you to change where commands are available on the fly.

This means to make your command available dynamically you can go multiple routes:

The @guilds decorator can be used without passing in any guilds, allowing you to specify which guilds the command is for later before syncing:

@commands.hybrid_command()
@app_commands.guilds()
async def example(context: commands.Context) -> None:
   ...

# Somewhere before syncing the tree
bot.tree.add_command(example.app_command, guilds=...)

You can also simply register your command as a global command, but before you sync the tree you remove it and add it for the guilds it should be in:

@commands.hybrid_command()
async def example(context: commands.Context) -> None:
   ...

# Somewhere before syncing the tree
bot.tree.remove_command('example')
bot.tree.add_command(example.app_command, guilds=...)

[Kadethra]

Here. I shortened it pretty much and it's still not working for me. You should be able to run this version if u want to.
https://gist.github.com/SuchtiZockerLP/06c77e59b84a83ce9a73520d88fa7952

[LostLuma]

Thanks. Your code copies the same list of all guild IDs as the settings for every extension instead of the modified one as you can see here. You will want to use perms.update({ext: guilds}) instead and then it will work.

[Kadethra]

Sry rewrite error because it was another function. Doesn't work even with the change.

[LostLuma]

I see now. I assumed the library will add app commands from cogs to the tree before calling the cog_load function, but it does it in the other order.

If we go back to my original comment, the first solution of adding an empty @guilds decorator and not needing the remove_command call at all does the trick, you can use this in your cog.

[Kadethra]

Thx a lot!