Remove unused deps, improve build script, update auth token name

.github/workflows/ci.yml

@@ -9,6 +9,9 @@ on:
     branches:
       - main
 
+env:
+  RUSTFLAGS: "-D warnings"
+
 jobs:
   build-test:
     name: Build and Test

.github/workflows/release.yml

@@ -113,7 +113,8 @@ jobs:
       - name: Build
         run: cargo build --all-features --release --verbose --target ${{ matrix.target }}
         env:
-          AUTH_TOKEN: ${{ secrets.AUTH_TOKEN }}
+          ARGON_TOKEN: ${{ secrets.ARGON_TOKEN }}
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
 
       - name: Archive
         shell: bash

Cargo.toml

@@ -27,12 +27,11 @@ rbx_reflection_database = "0.2.10"
 config-derive = { version = "*", path = "crates/config-derive" }
 profiling = { version = "*", path = "crates/profiling/profiling" }
 
-clap = { version = "4.4.1", features = ["derive", "cargo"] }
 serde = { version = "1.0.189", features = ["derive"] }
-tokio = { version = "1.32.0", features = ["full"] }
-reqwest = { version = "0.11.23", features = ["blocking", "rustls-tls"] }
 rmpv = {version = "1.3.0", features = ["with-serde"] }
-self_update = { version = "0.39.0", features = [
+clap = { version = "4.4.1", features = ["derive", "cargo"] }
+reqwest = { version = "0.11.23", default-features = false, features = ["blocking", "rustls-tls"] }
+self_update = { version = "0.39.0", default-features = false, features = [
 	"compression-zip-deflate",
 	"rustls",
 ] }
@@ -85,15 +84,10 @@ winsafe = { version = "0.0.20", features = ["user"] }
 
 [build-dependencies]
 anyhow = "1.0.79"
-self_update = { version = "0.39.0", features = [
-	"compression-zip-deflate",
-	"rustls",
-] }
-
+self_update = { version = "0.39.0", default-features = false, features = ["rustls"]}
 
 [patch.crates-io]
 notify-debouncer-full = { path = "crates/notify-debouncer-full" }
 self_update = { path = "crates/self_update" }
-
 # Temporary until `InstanceBuilder::with_referent` is released
 rbx_dom_weak = { path = "crates/rbx_dom_weak" }

build.rs

@@ -1,4 +1,4 @@
-use anyhow::Result;
+use anyhow::{Context, Result};
 use self_update::backends::github::Update;
 use std::{env, fs::File, path::PathBuf};
 
@@ -10,14 +10,25 @@ fn main() -> Result<()> {
 		return Ok(());
 	}
 
-	Update::configure()
+	let mut builder = Update::configure();
+
+	if let Ok(token) = env::var("GITHUB_TOKEN") {
+		builder.auth_token(&token);
+	} else {
+		println!("cargo:warning=GITHUB_TOKEN not set, rate limits may apply!")
+	}
+
+	builder
 		.repo_owner("argon-rbx")
 		.repo_name("argon-roblox")
 		.bin_name("Argon.rbxm")
 		.bin_install_path(out_path)
-		.target("")
+		.target("");
+
+	builder
 		.build()?
-		.download()?;
+		.download()
+		.context("Failed to download Argon plugin from GitHub!")?;
 
 	Ok(())
 }

src/cli/exec.rs

@@ -4,7 +4,7 @@ use reqwest::{blocking::Client, header::CONTENT_TYPE};
 use serde::Serialize;
 use std::{fs, path::MAIN_SEPARATOR};
 
-use crate::{argon_error, argon_info, sessions, studio};
+use crate::{argon_error, argon_info, sessions};
 
 /// Execute Luau code in Roblox Studio (requires running session)
 #[derive(Parser)]
@@ -71,7 +71,7 @@ impl Exec {
 
 				#[cfg(target_os = "windows")]
 				if self.focus {
-					studio::focus(None)?;
+					crate::studio::focus(None)?;
 				}
 			} else {
 				argon_error!("Code execution failed: running session does not have an address");

src/server/stop.rs

@@ -4,14 +4,10 @@ use std::process;
 
 use crate::util;
 
-async fn stop() {
+#[post("/stop")]
+async fn main() -> impl Responder {
 	trace!("Stopping Argon!");
-	// We need to kill all child processes as well
 	util::kill_process(process::id());
-}
 
-#[post("/stop")]
-async fn main() -> impl Responder {
-	tokio::spawn(stop());
 	HttpResponse::Ok().body("Argon stopped successfully")
 }

src/stats.rs

@@ -113,7 +113,7 @@ pub fn track() -> Result<()> {
 	let mut tracker = get_tracker()?;
 
 	if tracker.last_synced.elapsed()?.as_secs() > 3600 && tracker.stats.total() > 10 {
-		if let Some(token) = option_env!("AUTH_TOKEN") {
+		if let Some(token) = option_env!("ARGON_TOKEN") {
 			let stats = tracker.stats;
 			let remainder = stats.minutes_used % 60;
 
@@ -138,7 +138,7 @@ pub fn track() -> Result<()> {
 
 			set_tracker(&tracker)?;
 		} else {
-			warn!("This Argon build has no `AUTH_TOKEN` set, stats will not be uploaded")
+			warn!("This Argon build has no `ARGON_TOKEN` set, stats will not be uploaded")
 		}
 	} else {
 		debug!("Stats already synced within the last hour or too few stats to sync");