0.7.1

A few more tweaks before going live

Author: c-git

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "bazooka-bot"
-version = "0.7.0"
+version = "0.7.1"
 edition = "2024"
 publish = false
 

Cargo.toml

@@ -3,5 +3,4 @@ ignore = [
   # "RUSTSEC-0000-0000",
   # { id = "RUSTSEC-0000-0000", reason = "you can specify a reason the advisory is ignored" },
   { id = "RUSTSEC-2024-0388", reason = "Tracking issue https://github.com/serenity-rs/poise/issues/334" },
-  { id = "RUSTSEC-2025-0057", reason = "Tracking issue https://github.com/serenity-rs/serenity/issues/3420" },
 ]

Shuttle.toml

@@ -26,11 +26,12 @@ impl StartupConfig {
     pub fn try_new(clap_config: &ClapConfig) -> anyhow::Result<Self> {
         let guild_id = clap_config
             .registration_guild_id
-            .parse::<u64>()
-            .context("failed to parse guild id")
-            .map(GuildId::new)?;
+            .as_ref()
+            .map(|x| x.parse::<u64>().map(GuildId::new))
+            .transpose()
+            .context("failed to parse guild id")?;
 
-        // TODO 4 - See if we can split this in clap
+        // TODO 5 - See if we can split this in clap
         let owners: HashSet<UserId> = clap_config
             .owners
             .split(',')
@@ -41,10 +42,10 @@ impl StartupConfig {
             })
             .collect::<anyhow::Result<HashSet<UserId>>>()?;
 
-        let is_production = std::env::var("SHUTTLE").is_ok();
+        let is_production = std::env::var("IS_PROD").is_ok();
 
         Ok(Self {
-            registration_guild_id: Some(guild_id),
+            registration_guild_id: guild_id,
             owners,
             is_production,
         })

deny.toml

@@ -92,7 +92,7 @@ pub struct ClapConfig {
 
     /// Used mostly for testing to register the commands directly for the guild
     #[arg(long, env = "REGISTRATION_GUILD_ID")]
-    pub registration_guild_id: String,
+    pub registration_guild_id: Option<String>,
 
     /// The RoleId of the role that can run privileged commands
     #[arg(long, env = "AUTH_ROLE_ID")]

src/config.rs

@@ -1,4 +1,4 @@
-use anyhow::Context as _;
+use anyhow::{Context as _, bail};
 use bazooka_bot::{ClapConfig, Data, SharedConfig, StartupConfig, commands_list, heartbeat};
 use poise::serenity_prelude::{ClientBuilder, GatewayIntents};
 use secrecy::ExposeSecret;
@@ -14,7 +14,7 @@ use version::version;
 use clap::Parser;
 
 #[tokio::main]
-async fn main() {
+async fn main() -> anyhow::Result<()> {
     tracing_subscriber::registry()
         .with(fmt::layer().with_span_events(FmtSpan::NEW | FmtSpan::CLOSE))
         .with(
@@ -27,14 +27,17 @@ async fn main() {
 
     // Load setup values
     info!("Loading environment variables");
-    loadenv::load().expect("failed to load .env file");
+    match loadenv::load() {
+        Ok(was_found) => debug!(".env file was found: {was_found}"),
+        Err(err_msg) => bail!("failed to load .env file: {err_msg:?}"),
+    }
     let clap_config = ClapConfig::parse();
     debug!("ClapConfig: {:?}", clap_config);
 
     let startup_config =
-        StartupConfig::try_new(&clap_config).expect("failed to create setup config");
+        StartupConfig::try_new(&clap_config).context("failed to create setup config")?;
     let shared_config =
-        SharedConfig::try_new(&clap_config).expect("failed to created shared_config");
+        SharedConfig::try_new(&clap_config).context("failed to created shared_config")?;
     let discord_token = clap_config.discord_token;
 
     let framework = poise::Framework::builder()
@@ -101,9 +104,10 @@ async fn main() {
     )
     .framework(framework)
     .await
-    .expect("Error creating client");
+    .context("Error creating client")?;
 
-    if let Err(why) = client.start().await {
-        error!("Client error: {why:?}");
-    }
+    client
+        .start()
+        .await
+        .context("failed to start discord client")
 }