Make TabletBot pass most clippy::pedantic lints

jamesbt365 · jamesbt365 · commit 6f5ee53f8ae0 · 2024-03-08T19:44:41.000Z
diff --git a/src/commands/mod.rs b/src/commands/mod.rs
@@ -27,7 +27,7 @@ pub async fn respond_embed(ctx: &Context<'_>, embed: CreateEmbed, ephemeral: boo
     let result = ctx.send(builder).await;
 
     if let Err(e) = result {
-        println!("Failed to respond: {}", e)
+        println!("Failed to respond: {e}");
     }
 }
 
@@ -55,8 +55,8 @@ pub async fn paginate_lists<U, E>(
     embed_title: &str,
 ) -> Result<(), Error> {
     let ctx_id = ctx.id();
-    let prev_button_id = format!("{}prev", ctx_id);
-    let next_button_id = format!("{}next", ctx_id);
+    let prev_button_id = format!("{ctx_id}prev");
+    let next_button_id = format!("{ctx_id}next");
 
     let colour = Colour::TEAL;
 
diff --git a/src/commands/snippets.rs b/src/commands/snippets.rs
@@ -6,6 +6,7 @@ use crate::{
 use ::serenity::futures::{Stream, StreamExt};
 use poise::serenity_prelude::{futures, CreateAttachment, CreateEmbed};
 
+#[allow(clippy::unused_async)]
 async fn autocomplete_snippet<'a>(
     ctx: Context<'a>,
     partial: &'a str,
@@ -39,7 +40,7 @@ pub async fn snippet(
     id: String,
 ) -> Result<(), Error> {
     // Lazily get snippet because this is a prefix command too.
-    if let Some(snippet) = get_snippet_lazy(&ctx, &id).await {
+    if let Some(snippet) = get_snippet_lazy(&ctx, &id) {
         let embed = snippet.embed();
 
         respond_embed(&ctx, embed, false).await;
@@ -80,7 +81,7 @@ pub async fn create_snippet(
         rwlock_guard.snippets.push(snippet.clone());
 
         rwlock_guard.snippets = rwlock_guard.snippets.clone();
-        println!("New snippet created '{}: {}'", id, title);
+        println!("New snippet created '{id}: {title}'");
         rwlock_guard.write();
 
         let mut embed = snippet.embed();
@@ -112,7 +113,7 @@ pub async fn edit_snippet(
     #[description = "The snippet's title"] title: Option<String>,
     #[description = "The snippet's content"] content: Option<String>,
 ) -> Result<(), Error> {
-    match get_snippet_lazy(&ctx, &id).await {
+    match get_snippet_lazy(&ctx, &id) {
         Some(mut snippet) => {
             if let Some(title) = title {
                 snippet.title = title;
@@ -135,7 +136,7 @@ pub async fn edit_snippet(
         None => {
             let title = &"Failed to edit snippet";
             let content = &&format!("The snippet '{id}' does not exist");
-            respond_err(&ctx, title, content).await
+            respond_err(&ctx, title, content).await;
         }
     };
 
@@ -150,17 +151,17 @@ pub async fn remove_snippet(
     #[description = "The snippet's id"]
     id: String,
 ) -> Result<(), Error> {
-    match get_snippet_lazy(&ctx, &id).await {
+    match get_snippet_lazy(&ctx, &id) {
         Some(snippet) => {
-            rm_snippet(&ctx, &snippet).await;
+            rm_snippet(&ctx, &snippet);
             let title = &"Snippet successfully removed";
             let content = &&format!("Removed snippet '{}: {}'", snippet.id, snippet.title);
             respond_ok(&ctx, title, content).await;
         }
         None => {
             let title = &"Failed to remove snippet";
             let content = &&format!("The snippet '{id}' does not exist");
-            respond_err(&ctx, title, content).await
+            respond_err(&ctx, title, content).await;
         }
     }
 
@@ -194,7 +195,7 @@ pub async fn list_snippets(ctx: Context<'_>) -> Result<(), Error> {
         .map(|snippet| (snippet.id.clone(), snippet.title.clone(), true))
         .collect::<Vec<(String, String, bool)>>()
         .chunks(25)
-        .map(|chunk| chunk.to_vec())
+        .map(<[(String, String, bool)]>::to_vec)
         .collect();
 
     super::paginate_lists(ctx, &pages, "Snippets").await?;
@@ -213,10 +214,10 @@ pub async fn export_snippet(
     #[description = "The snippet's id"]
     id: String,
 ) -> Result<(), Error> {
-    match get_snippet_lazy(&ctx, &id).await {
+    match get_snippet_lazy(&ctx, &id) {
         Some(snippet) => {
             let attachment = CreateAttachment::bytes(
-                format!("{}", &snippet.content.replace('\n', r"\n")),
+                snippet.content.replace('\n', r"\n").to_string(),
                 "snippet.txt",
             );
             let message = poise::CreateReply::default()
@@ -227,7 +228,7 @@ pub async fn export_snippet(
         None => {
             let title = &"Failed to export snippet";
             let content = &&format!("The snippet '{id}' does not exist");
-            respond_err(&ctx, title, content).await
+            respond_err(&ctx, title, content).await;
         }
     }
 
@@ -245,7 +246,7 @@ impl Embeddable for Snippet {
 }
 
 // Exact matches the snippet id and name.
-async fn _get_snippet(ctx: &Context<'_>, id: &str) -> Option<Snippet> {
+fn _get_snippet(ctx: &Context<'_>, id: &str) -> Option<Snippet> {
     let data = ctx.data();
     let rwlock_guard = data.state.read().unwrap();
 
@@ -257,7 +258,7 @@ async fn _get_snippet(ctx: &Context<'_>, id: &str) -> Option<Snippet> {
 }
 
 // Matches the snippet by checking if its starts with the id and name.
-async fn get_snippet_lazy(ctx: &Context<'_>, id: &str) -> Option<Snippet> {
+fn get_snippet_lazy(ctx: &Context<'_>, id: &str) -> Option<Snippet> {
     let data = ctx.data();
     let rwlock_guard = data.state.read().unwrap();
 
@@ -268,7 +269,7 @@ async fn get_snippet_lazy(ctx: &Context<'_>, id: &str) -> Option<Snippet> {
         .cloned()
 }
 
-async fn rm_snippet(ctx: &Context<'_>, snippet: &Snippet) {
+fn rm_snippet(ctx: &Context<'_>, snippet: &Snippet) {
     let data = ctx.data();
     let mut rwlock_guard = data.state.write().unwrap();
 
diff --git a/src/commands/utils.rs b/src/commands/utils.rs
@@ -8,6 +8,7 @@ use poise::serenity_prelude::{Colour, CreateEmbed, CreateEmbedFooter, EditMessag
 use regex::Regex;
 use serenity::futures::{self, Stream, StreamExt};
 
+#[allow(clippy::unused_async)]
 async fn autocomplete_key<'a>(
     ctx: Context<'a>,
     partial: &'a str,
@@ -65,7 +66,7 @@ pub async fn embed(
         match url.parse::<reqwest::Url>() {
             Ok(_) => {
                 if title.is_some() {
-                    embed = embed.url(url)
+                    embed = embed.url(url);
                 } else {
                     respond_err(
                         &ctx,
@@ -78,7 +79,7 @@ pub async fn embed(
             }
             Err(e) => {
                 let title = "Invalid url provided";
-                let content = &format!("The url '{}' is not a valid url: {}", url, e);
+                let content = &format!("The url '{url}' is not a valid url: {e}");
                 respond_err(&ctx, title, content).await;
                 return Ok(());
             }
@@ -231,7 +232,7 @@ pub async fn edit_embed(
             let builder = EditMessage::default().embed(embedb);
 
             match msg_clone.edit(ctx, builder).await {
-                Ok(_) => {
+                Ok(()) => {
                     respond_ok(
                         &ctx,
                         "Successfully edited embed",
@@ -241,7 +242,7 @@ pub async fn edit_embed(
                 }
                 Err(error) => {
                     // Better error handling later.
-                    respond_err(&ctx, "Error while handling message!", &format!("{}", error)).await
+                    respond_err(&ctx, "Error while handling message!", &format!("{error}")).await;
                 }
             }
         } else {
@@ -315,7 +316,7 @@ pub async fn add_repo(
     respond_ok(
         &ctx,
         "Successfully added issue token",
-        &format!("{}: {}/{}", key, owner, repository),
+        &format!("{key}: {owner}/{repository}"),
     )
     .await;
 
@@ -335,16 +336,16 @@ pub async fn remove_repo(
     // impl a solution directly into the types?
 
     // not sure why I have to do this, it won't settle otherwise.
-    let key_str = format!("The repository with the key '{}' has been removed", key);
-    match get_repo_details(&ctx, &key).await {
+    let key_str = format!("The repository with the key '{key}' has been removed");
+    match get_repo_details(&ctx, &key) {
         Some(_) => {
-            rm_repo(&ctx, &key).await;
+            rm_repo(&ctx, &key);
 
             respond_ok(&ctx, "Successfully removed repository!", &key_str).await;
         }
         None => {
             let title = "Failure to find repository";
-            let content = format!("The key '{}' does not exist.", key);
+            let content = format!("The key '{key}' does not exist.");
             respond_err(&ctx, title, &content).await;
         }
     };
@@ -385,22 +386,22 @@ pub async fn list_repos(ctx: Context<'_>) -> Result<(), Error> {
         })
         .collect::<Vec<(String, String, bool)>>()
         .chunks(25)
-        .map(|chunk| chunk.to_vec())
+        .map(<[(String, String, bool)]>::to_vec)
         .collect();
 
     super::paginate_lists(ctx, &pages, "Repositories").await?;
 
     Ok(())
 }
 
-async fn get_repo_details(ctx: &Context<'_>, key: &str) -> Option<RepositoryDetails> {
+fn get_repo_details(ctx: &Context<'_>, key: &str) -> Option<RepositoryDetails> {
     let data = ctx.data();
     let rwlock_guard = data.state.read().unwrap();
 
     rwlock_guard.issue_prefixes.get(key).cloned()
 }
 
-async fn rm_repo(ctx: &Context<'_>, key: &str) {
+fn rm_repo(ctx: &Context<'_>, key: &str) {
     let data = ctx.data();
     let mut rwlock_guard = data.state.write().unwrap();
 
diff --git a/src/events/code.rs b/src/events/code.rs
@@ -35,10 +35,10 @@ async fn get_embeds(ctx: &Context, message: &Message) -> Option<Vec<CreateEmbed>
 
     typing.stop();
 
-    if !embeds.is_empty() {
-        Some(embeds)
-    } else {
+    if embeds.is_empty() {
         None
+    } else {
+        Some(embeds)
     }
 }
 
@@ -47,12 +47,12 @@ async fn http_get_body_text(url: &String) -> Option<String> {
         Ok(res) => match res.text().await {
             Ok(content) => Some(content),
             Err(e) => {
-                println!("Failed to get text: {}", e);
+                println!("Failed to get text: {e}");
                 None
             }
         },
         Err(e) => {
-            println!("Failed to get response: {}", e);
+            println!("Failed to get response: {e}");
             None
         }
     }
@@ -95,10 +95,10 @@ impl FileReference<'_> {
             })
             .collect();
 
-        if !files.is_empty() {
-            Some(files)
-        } else {
+        if files.is_empty() {
             None
+        } else {
+            Some(files)
         }
     }
 
@@ -108,7 +108,7 @@ impl FileReference<'_> {
         if let Some(mut content) = self.display().await {
             content.shrink_to(4096 - 8 - extension.len());
 
-            let description = format!("```{}\n{}\n```", extension, content);
+            let description = format!("```{extension}\n{content}\n```");
 
             let mut default = CreateEmbed::default();
             default = default
@@ -135,7 +135,7 @@ impl FileReference<'_> {
             "https://raw.githubusercontent.com/{}/{}/{}/{}",
             self.owner, self.repo, self.git_ref, self.path
         );
-        println!("Downloading content: {}", url);
+        println!("Downloading content: {url}");
 
         if let Some(content) = http_get_body_text(&url).await {
             let lines: Vec<&str> = content.split('\n').collect();
diff --git a/src/events/issue.rs b/src/events/issue.rs
@@ -21,8 +21,8 @@ pub async fn message(data: &Data, ctx: &Context, message: &Message) {
         let typing = message.channel_id.start_typing(&ctx.http);
 
         let ctx_id = message.id.get(); // poise context isn't available here.
-        let remove_id = format!("{}remove", ctx_id);
-        let hide_body_id = format!("{}hide_body", ctx_id);
+        let remove_id = format!("{ctx_id}remove");
+        let hide_body_id = format!("{ctx_id}hide_body");
         let components = serenity::CreateActionRow::Buttons(vec![
             serenity::CreateButton::new(&remove_id)
                 .label("delete")
@@ -108,7 +108,7 @@ async fn issue_embeds(data: &Data, message: &Message) -> Option<Vec<CreateEmbed>
     let ratelimit = client.ratelimit();
 
     let regex =
-        Regex::new(r#" ?([a-zA-Z0-9-_.]+)?#([0-9]+[0-9]) ?"#).expect("Expected numbers regex");
+        Regex::new(r" ?([a-zA-Z0-9-_.]+)?#([0-9]+[0-9]) ?").expect("Expected numbers regex");
 
     let custom_repos = { data.state.read().unwrap().issue_prefixes.clone() };
 
@@ -193,7 +193,7 @@ impl Document for Issue {
 
         let mut description = String::default();
         for line in body.split('\n').take(15) {
-            description.push_str(&format!("{}\n", line));
+            description.push_str(&format!("{line}\n"));
         }
 
         description.shrink_to(4096);
@@ -208,16 +208,16 @@ impl Document for Issue {
     }
 
     fn get_labels(&self) -> Option<String> {
-        if !self.labels.is_empty() {
+        if self.labels.is_empty() {
+            None
+        } else {
             let labels = &self
                 .labels
                 .iter()
                 .map(|l| l.name.clone())
                 .collect::<Vec<String>>();
 
             Some(format!("`{}`", labels.join("`, `")))
-        } else {
-            None
         }
     }
 }
@@ -252,7 +252,7 @@ impl Embeddable for PullRequest {
             embed = embed.field("Labels", labels, true);
         }
 
-        embed.to_owned()
+        embed
     }
 }
 
@@ -269,7 +269,7 @@ impl Document for PullRequest {
 
         let mut content = String::default();
         for line in body.split('\n').take(15) {
-            content.push_str(&format!("{}\n", line));
+            content.push_str(&format!("{line}\n"));
         }
 
         content.shrink_to(4096);
diff --git a/src/formatting.rs b/src/formatting.rs
@@ -1,6 +1,6 @@
 pub fn trim_indent(lines: &[&str]) -> String {
     let base_indent = get_base_indent(lines);
-    let prefix = String::from_iter(std::iter::repeat(' ').take(base_indent));
+    let prefix = " ".repeat(base_indent);
 
     let trimmed_lines: Vec<&str> = lines
         .iter()
diff --git a/src/main.rs b/src/main.rs
diff --git a/src/structures.rs b/src/structures.rs
