feat(issuecards): add permission checks to repository commands and add global repository configuration

issuecards/issuecards.py

@@ -177,12 +177,10 @@ class IssueCards(commands.Cog):
     @commands.group(name="issuecards")
     async def issuecards(self, ctx: commands.Context) -> None:
         """Manage the IssueCards cog configuration."""
-        pass
 
     @issuecards.group(name="providers")
     async def issuecards_providers(self, ctx: commands.Context) -> None:
         """Manage IssueCards providers."""
-        pass
 
     @issuecards_providers.command(name="list")
     @commands.bot_has_permissions(embed_links=True)
@@ -219,10 +217,10 @@ class IssueCards(commands.Cog):
     @issuecards.group(name="repositories", aliases=["repository", "repo", "repos"])
     async def issuecards_repositories(self, ctx: commands.Context) -> None:
         """Manage IssueCards repositories."""
-        pass
 
     @issuecards_repositories.command(name="add")
-    async def issuecards_repositories_add(self, ctx: commands.Context, owner: str, name: str, provider_id: str, prefix: str | None = None) -> None:
+    @commands.admin_or_permissions(manage_guild=True)
+    async def issuecards_repositories_add(self, ctx: commands.Context, owner: str, name: str, provider_id: str, prefix: str | None = None, is_global: bool = False) -> None:
         """Add a repository to the IssueCards configuration.
 
         **Arguments**
@@ -232,10 +230,15 @@ class IssueCards(commands.Cog):
           Use `[p]issuecards providers list` to get a list of valid providers.
         * `prefix`: The prefix used for determining which repository to retrieve an issue from when using the `<prefix>#<issue-num>` syntax.
           Defaults to `None`, disabling the prefix functionality.
+        * `is_global`: Whether the repository is global or not. If this is set to True, you must be the bot owner to create the repository.
         """
+        if is_global is True and not await self.bot.is_owner(ctx.author):
+            await ctx.send("Only the bot owner can create global repositories.")
+            return
+
         prefix = prefix.lower() if prefix else None
         try:
-            provider = await Provider.from_config(provider_id, ctx.guild)
+            provider = await Provider.from_config(provider_id, None if is_global else ctx.guild)
         except ValueError:
             await ctx.send(f"Invalid provider ID. Please use {inline(f'{ctx.prefix}issuecards providers list')} for a list of valid providers.")
             return
@@ -247,12 +250,20 @@ class IssueCards(commands.Cog):
         except ValueError:
             pass
 
-        repository = Repository(cog=self, guild=ctx.guild, owner=owner, name=name, provider=provider, prefix=prefix)
+        try:
+            await Repository.from_prefix(cog=self, guild=ctx.guild, prefix=prefix or "")
+            await ctx.send(f"Repository with prefix `{prefix}` already exists in the configuration.")
+            return
+        except ValueError:
+            pass
+
+        repository = Repository(cog=self, guild=None if is_global else ctx.guild, owner=owner, name=name, provider=provider, prefix=prefix)
         await repository.to_config()
         await ctx.send(f"Repository `{owner}/{name}` {f'with prefix `{prefix}` ' if prefix is not None else ' '}has been added to the configuration.")
 
     @issuecards_repositories.command(name="remove")
-    async def issuecards_repositories_remove(self, ctx: commands.Context, owner: str, name: str, provider_id: str) -> None:
+    @commands.admin_or_permissions(manage_guild=True)
+    async def issuecards_repositories_remove(self, ctx: commands.Context, owner: str, name: str, provider_id: str, is_global: bool = False) -> None:
         """Remove a repository from the IssueCards configuration.
 
         **Arguments**
@@ -260,7 +271,12 @@ class IssueCards(commands.Cog):
         * `name`: The name of the repository.
         * `provider_id`: The ID of the provider that contains this repository.
           Use `[p]issuecards providers list` to get a list of valid providers.
+        * `is_global`: Whether the repository is global or not. If this is set to True, you must be the bot owner to remove the repository.
         """
+        if is_global is True and not await self.bot.is_owner(ctx.author):
+            await ctx.send("Only the bot owner can remove global repositories.")
+            return
+
         try:
             provider = await Provider.from_config(provider_id, ctx.guild)
         except ValueError:
@@ -272,6 +288,9 @@ class IssueCards(commands.Cog):
             await ctx.send(f"Repository `{owner}/{name}` does not exist in the configuration.")
             return
 
+        if not is_global and repository.guild is None:
+            await ctx.send(f"Repository `{owner}/{name}` is a global repository. You must be the bot owner to remove it.")
+
         await repository.remove_config()
         await ctx.send(f"Repository `{owner}/{name}` has been removed from the configuration.")