From 96b380db254d48e5dbbc2e6c683d5f3262eebf85 Mon Sep 17 00:00:00 2001
From: Wyvest <wyvestbusiness@gmail.com>
Date: Wed, 25 Feb 2026 09:31:45 -0500
Subject: [PATCH] lint pt 2

---
 apps/oneclient/desktop/src/lib.rs                    |  4 ++--
 apps/oneclient/frontend/src/routes/__root.tsx        | 12 +++++++++++-
 .../frontend/src/routes/app/account/route.tsx        |  1 +
 apps/onelauncher/desktop/src/lib.rs                  |  8 +++++---
 .../src/routes/app/browser/package.$provider.$id.tsx |  7 ++++++-
 5 files changed, 25 insertions(+), 7 deletions(-)

diff --git a/apps/oneclient/desktop/src/lib.rs b/apps/oneclient/desktop/src/lib.rs
index 71685e52..33b1901f 100644
--- a/apps/oneclient/desktop/src/lib.rs
+++ b/apps/oneclient/desktop/src/lib.rs
@@ -129,7 +129,7 @@ fn setup_window(handle: &tauri::AppHandle) -> Result<(), Box<dyn std::error::Err
 	ext::updater::init(handle)?;
 
 	let win_clone = win;
-	let app_handle = handle.clone();
+	let _app_handle = handle.clone();
 	tokio::task::spawn(async move {
 		if let Ok(state) = State::get().await {
 			let _settings = state.settings.read().await;
@@ -139,7 +139,7 @@ fn setup_window(handle: &tauri::AppHandle) -> Result<(), Box<dyn std::error::Err
 			{
 				win_clone.set_decorations(true).ok();
 				let win_weak = win_clone.clone();
-				app_handle
+				_app_handle
 					.run_on_main_thread(move || {
 						#[cfg(target_os = "macos")]
 						{
diff --git a/apps/oneclient/frontend/src/routes/__root.tsx b/apps/oneclient/frontend/src/routes/__root.tsx
index 41868d87..2b01b2ef 100644
--- a/apps/oneclient/frontend/src/routes/__root.tsx
+++ b/apps/oneclient/frontend/src/routes/__root.tsx
@@ -72,7 +72,17 @@ function useDiscordRPC() {
 	const provider = location.search.provider ?? null;
 	const packageId = location.search.packageId ?? null;
 	const { data: cluster } = useCommand(['getClusterById', clusterId], () => bindings.core.getClusterById(clusterId));
-	const { data: managedPackage } = useCommand(['getPackage', provider, packageId], () => bindings.core.getPackage(provider!, packageId!), { enabled: provider != null && packageId != null });
+
+	const { data: managedPackage } = useCommand(
+		['getPackage', provider, packageId],
+		() => {
+			if (provider == null || packageId == null)
+				return Promise.reject(new Error('Missing parameters'));
+			return bindings.core.getPackage(provider, packageId);
+		},
+		{ enabled: provider != null && packageId != null },
+	);
+
 	useEffect(() => {
 		const template = ResolvedPathNames[location.pathname as URLPath];
 		if (template)
diff --git a/apps/oneclient/frontend/src/routes/app/account/route.tsx b/apps/oneclient/frontend/src/routes/app/account/route.tsx
index be727467..5c2dcae1 100644
--- a/apps/oneclient/frontend/src/routes/app/account/route.tsx
+++ b/apps/oneclient/frontend/src/routes/app/account/route.tsx
@@ -20,6 +20,7 @@ export const Route = createFileRoute('/app/account')({
 	async beforeLoad({ context, search }) {
 		// TODO instead replace this with a refresh access_token function. Waiting on binding from @LynithDev
 		const { profile, search: validSearch } = search;
+
 		if (!profile)
 			return { profileData: null, profile: null, validSearch, playerData: null };
 
diff --git a/apps/onelauncher/desktop/src/lib.rs b/apps/onelauncher/desktop/src/lib.rs
index 5e7acfc6..6559d4ac 100644
--- a/apps/onelauncher/desktop/src/lib.rs
+++ b/apps/onelauncher/desktop/src/lib.rs
@@ -127,7 +127,7 @@ fn setup_window(handle: &tauri::AppHandle) -> Result<(), Box<dyn std::error::Err
 
 	// Initialize window decorations based on settings
 	let win_clone = win;
-	let app_handle = handle.clone();
+	let _app_handle = handle.clone();
 	tokio::task::spawn(async move {
 		if let Ok(state) = State::get().await {
 			let _settings = state.settings.read().await;
@@ -137,7 +137,7 @@ fn setup_window(handle: &tauri::AppHandle) -> Result<(), Box<dyn std::error::Err
 			{
 				win_clone.set_decorations(true).ok();
 				let win_weak = win_clone.clone();
-				app_handle
+				_app_handle
 					.run_on_main_thread(move || {
 						#[cfg(target_os = "macos")]
 						{
@@ -170,7 +170,9 @@ fn setup_window(handle: &tauri::AppHandle) -> Result<(), Box<dyn std::error::Err
 			}
 
 			#[cfg(not(target_os = "macos"))]
-			win_clone.set_decorations(settings.native_window_frame).ok();
+			win_clone
+				.set_decorations(_settings.native_window_frame)
+				.ok();
 		}
 		win_clone.show().ok();
 	});
diff --git a/apps/onelauncher/frontend/src/routes/app/browser/package.$provider.$id.tsx b/apps/onelauncher/frontend/src/routes/app/browser/package.$provider.$id.tsx
index 81f5d6a6..eb01eacb 100644
--- a/apps/onelauncher/frontend/src/routes/app/browser/package.$provider.$id.tsx
+++ b/apps/onelauncher/frontend/src/routes/app/browser/package.$provider.$id.tsx
@@ -260,7 +260,12 @@ function InstallButton() {
 		return versions.items[0];
 	}, [browserContext.cluster, versions]);
 
-	const download = useDownloadPackage(browserContext.cluster!, provider, version!, true);
+	const download = useDownloadPackage(
+		(browserContext.cluster || {}) as ClusterModel,
+		provider,
+		(version || {}) as ManagedVersion,
+		true,
+	);
 	const downloadRef = useRef(download);
 	useEffect(() => {
 		downloadRef.current.reset();