chore(devforum-rank-verification): Add a roblox_verified role to make an early check and avoid making unnecessary API calls

Author: Archasion

README.md

@@ -12,6 +12,8 @@ The following is an example of the file structure:
 roles:
   devforum_member: "ROLE_ID"
   devforum_regular: "ROLE_ID"
+  # Optional, allows the bot to avoid making unnecessary API calls
+  roblox_verified: "ROLE_ID"
 ```
 
 [ci badge]:https://img.shields.io/github/actions/workflow/status/archasion/discord-bot-rs/ci.yml?branch=main&event=push&label=CI

bot/src/components/verify_devforum_rank.rs

@@ -36,8 +36,37 @@ impl ComponentHandler for VerifyDevForumRank<'_> {
     async fn exec(&self, ctx: crate::Context) -> anyhow::Result<()> {
         let guild_id = self.cmd.guild_id.context("get guild id")?;
         let author_id = self.cmd.author_id().context("get interaction author id")?;
+        let member_roles = ctx
+            .http
+            .guild_member(guild_id, author_id)
+            .await
+            .context("get guild member")?
+            .model()
+            .await?
+            .roles;
+
+        // Respond early if the user doesn't have the verified role
+        // this is a quick check to avoid unnecessary API calls
+        if let Some(r_id) = ctx.cfg.roles.roblox_verified {
+            if !member_roles.contains(&r_id) {
+                ctx.http
+                    .interaction(self.cmd.application_id)
+                    .create_response(self.cmd.id, &self.cmd.token, &InteractionResponse {
+                        kind: InteractionResponseType::ChannelMessageWithSource,
+                        data: Some(
+                            InteractionResponseDataBuilder::new()
+                                .flags(MessageFlags::EPHEMERAL)
+                                .content("You must be Roblox verified to use this command.")
+                                .build(),
+                        ),
+                    })
+                    .await
+                    .context("respond to interaction")?;
+                return Ok(());
+            }
+        }
 
-        // Defer the interaction response
+        // Defer the interaction response since the API calls may take some time
         ctx.http
             .interaction(self.cmd.application_id)
             .create_response(self.cmd.id, &self.cmd.token, &InteractionResponse {

bot/src/config.rs

@@ -12,6 +12,7 @@ pub(crate) struct Config {
 pub(crate) struct ConfigRoles {
     pub(crate) devforum_member: Id<RoleMarker>,
     pub(crate) devforum_regular: Id<RoleMarker>,
+    pub(crate) roblox_verified: Option<Id<RoleMarker>>,
 }
 
 #[tracing::instrument(ret)]