From c87f3ffb4bddee3b1f7610bd07359e91758ae132 Mon Sep 17 00:00:00 2001
From: norbiros <norbiros@protonmail.com>
Date: Thu, 18 Jul 2024 17:54:05 +0200
Subject: [PATCH] chore: Format & Lint code

---
 .../app-frontend/src/components/ui/ExportModal.vue |  2 +-
 .../src/components/ui/InstanceCreationModal.vue    |  4 ++--
 .../app-frontend/src/components/ui/ProjectCard.vue |  2 +-
 apps/app-frontend/src/main.js                      |  1 -
 apps/app-frontend/src/pages/Browse.vue             | 14 ++++++--------
 apps/app-frontend/src/pages/instance/Logs.vue      |  9 ++++++---
 apps/app/src/api/utils.rs                          |  8 +++-----
 .../src/pages/dashboard/revenue/withdraw.vue       |  4 ++--
 packages/app-lib/src/state/profiles.rs             |  4 ++--
 packages/ui/src/components/base/Slider.vue         |  6 +++---
 packages/ui/src/components/modal/ShareModal.vue    | 13 ++++++++-----
 11 files changed, 34 insertions(+), 33 deletions(-)

diff --git a/apps/app-frontend/src/components/ui/ExportModal.vue b/apps/app-frontend/src/components/ui/ExportModal.vue
index a586dd61f..eaa256e61 100644
--- a/apps/app-frontend/src/components/ui/ExportModal.vue
+++ b/apps/app-frontend/src/components/ui/ExportModal.vue
@@ -50,7 +50,7 @@ const initFiles = async () => {
         disabled:
           folder === 'profile.json' ||
           folder.startsWith('modrinth_logs') ||
-          folder.startsWith('.fabric')
+          folder.startsWith('.fabric'),
       }))
       .filter((pathData) => !pathData.path.includes('.DS_Store'))
       .forEach((pathData) => {
diff --git a/apps/app-frontend/src/components/ui/InstanceCreationModal.vue b/apps/app-frontend/src/components/ui/InstanceCreationModal.vue
index cdd3a3fc6..9abacbc40 100644
--- a/apps/app-frontend/src/components/ui/InstanceCreationModal.vue
+++ b/apps/app-frontend/src/components/ui/InstanceCreationModal.vue
@@ -353,7 +353,7 @@ const create_instance = async () => {
   creating.value = true
   const loader_version_value =
     loader_version.value === 'other' ? specified_loader_version.value : loader_version.value
-  const loaderVersion = loader.value === 'vanilla' ? null : loader_version_value ?? 'stable'
+  const loaderVersion = loader.value === 'vanilla' ? null : (loader_version_value ?? 'stable')
 
   hide()
   creating.value = false
@@ -362,7 +362,7 @@ const create_instance = async () => {
     profile_name.value,
     game_version.value,
     loader.value,
-    loader.value === 'vanilla' ? null : loader_version_value ?? 'stable',
+    loader.value === 'vanilla' ? null : (loader_version_value ?? 'stable'),
     icon.value,
   ).catch(handleError)
 
diff --git a/apps/app-frontend/src/components/ui/ProjectCard.vue b/apps/app-frontend/src/components/ui/ProjectCard.vue
index c886cc979..1b74878b6 100644
--- a/apps/app-frontend/src/components/ui/ProjectCard.vue
+++ b/apps/app-frontend/src/components/ui/ProjectCard.vue
@@ -108,7 +108,7 @@ const install = async (e) => {
       <div
         class="banner"
         :style="{
-          'background-color': project.featured_gallery ?? project.gallery[0] ? null : toColor,
+          'background-color': (project.featured_gallery ?? project.gallery[0]) ? null : toColor,
           'background-image': `url(${
             project.featured_gallery ??
             project.gallery[0] ??
diff --git a/apps/app-frontend/src/main.js b/apps/app-frontend/src/main.js
index a0a5acde7..8d3876830 100644
--- a/apps/app-frontend/src/main.js
+++ b/apps/app-frontend/src/main.js
@@ -4,7 +4,6 @@ import App from '@/App.vue'
 import { createPinia } from 'pinia'
 import '@modrinth/assets/omorphia.scss'
 import '@/assets/stylesheets/global.scss'
-import 'floating-vue/dist/style.css'
 import FloatingVue from 'floating-vue'
 import 'floating-vue/dist/style.css'
 import { get_opening_command, initialize_state } from '@/helpers/state'
diff --git a/apps/app-frontend/src/pages/Browse.vue b/apps/app-frontend/src/pages/Browse.vue
index 598a046f0..57121fe2a 100644
--- a/apps/app-frontend/src/pages/Browse.vue
+++ b/apps/app-frontend/src/pages/Browse.vue
@@ -489,12 +489,12 @@ const [categories, loaders, availableGameVersions] = await Promise.all([
 ])
 
 const filteredLoaders = computed(() => {
-  return loaders.value.filter(loader => {
-    return (projectType.value === 'mod' || projectType.value === 'modpack')
+  return loaders.value.filter((loader) => {
+    return projectType.value === 'mod' || projectType.value === 'modpack'
       ? loader.supported_project_types[0] === 'mod'
-      : loader.supported_project_types[0] === projectType.value;
-  });
-});
+      : loader.supported_project_types[0] === projectType.value
+  })
+})
 
 const selectableProjectTypes = computed(() => {
   const values = [
@@ -527,9 +527,7 @@ const showVersions = computed(
   () => instanceContext.value === null || ignoreInstanceGameVersions.value,
 )
 
-const isModProject = computed(
-  () => ['modpack', 'mod'].includes(projectType.value)
-)
+const isModProject = computed(() => ['modpack', 'mod'].includes(projectType.value))
 
 onUnmounted(() => unlistenOffline())
 </script>
diff --git a/apps/app-frontend/src/pages/instance/Logs.vue b/apps/app-frontend/src/pages/instance/Logs.vue
index a69289628..030257092 100644
--- a/apps/app-frontend/src/pages/instance/Logs.vue
+++ b/apps/app-frontend/src/pages/instance/Logs.vue
@@ -109,7 +109,7 @@ import { createVirtualScroller } from 'vue-typed-virtual-list'
 dayjs.extend(isToday)
 dayjs.extend(isYesterday)
 
-const VirtualScroller = createVirtualScroller();
+const VirtualScroller = createVirtualScroller()
 
 const route = useRoute()
 
@@ -412,8 +412,11 @@ function handleUserScroll() {
 interval.value = setInterval(async () => {
   if (logs.value.length > 0) {
     logs.value[0] = await getLiveStdLog()
-    const logContainerElement = logContainer.value.$el;
-    const scroll = logContainerElement.scrollHeight - logContainerElement.scrollTop - logContainerElement.clientHeight
+    const logContainerElement = logContainer.value.$el
+    const scroll =
+      logContainerElement.scrollHeight -
+      logContainerElement.scrollTop -
+      logContainerElement.clientHeight
     // const scroll = logContainer.value.$el.scrollHeight - logContainer.value.$el.scrollTop - logContainer.value.$el.clientHeight
 
     // Allow resetting of userScrolled if the user scrolls to the bottom
diff --git a/apps/app/src/api/utils.rs b/apps/app/src/api/utils.rs
index 02e28b18a..d6af796c5 100644
--- a/apps/app/src/api/utils.rs
+++ b/apps/app/src/api/utils.rs
@@ -82,7 +82,7 @@ pub async fn should_disable_mouseover() -> bool {
 }
 
 #[tauri::command]
-pub fn show_in_folder(path: PathBuf) -> Result<()> {
+pub fn show_in_folder(mut path: PathBuf) -> Result<()> {
     {
         #[cfg(target_os = "windows")]
         {
@@ -100,7 +100,6 @@ pub fn show_in_folder(path: PathBuf) -> Result<()> {
         #[cfg(target_os = "linux")]
         {
             use std::fs::metadata;
-            use std::path::PathBuf;
 
             let path_string = path.to_string_lossy().to_string();
 
@@ -108,9 +107,8 @@ pub fn show_in_folder(path: PathBuf) -> Result<()> {
                 Command::new("xdg-open").arg(&path).spawn()?;
             } else if path_string.contains(',') {
                 // see https://gitlab.freedesktop.org/dbus/dbus/-/issues/76
-                let mut parent_path = PathBuf::from(path);
-                parent_path.pop();
-                Command::new("xdg-open").arg(&parent_path).spawn()?;
+                path.pop();
+                Command::new("xdg-open").arg(&path).spawn()?;
             } else {
                 Command::new("dbus-send")
                     .args([
diff --git a/apps/frontend/src/pages/dashboard/revenue/withdraw.vue b/apps/frontend/src/pages/dashboard/revenue/withdraw.vue
index 3ec3ae965..8aeec1b9c 100644
--- a/apps/frontend/src/pages/dashboard/revenue/withdraw.vue
+++ b/apps/frontend/src/pages/dashboard/revenue/withdraw.vue
@@ -258,12 +258,12 @@ const getRangeOfMethod = (method) => {
 
 const maxWithdrawAmount = computed(() => {
   const interval = selectedMethod.value.interval;
-  return interval?.standard ? interval.standard.max : interval?.fixed?.values.slice(-1)[0] ?? 0;
+  return interval?.standard ? interval.standard.max : (interval?.fixed?.values.slice(-1)[0] ?? 0);
 });
 
 const minWithdrawAmount = computed(() => {
   const interval = selectedMethod.value.interval;
-  return interval?.standard ? interval.standard.min : interval?.fixed?.values?.[0] ?? fees.value;
+  return interval?.standard ? interval.standard.min : (interval?.fixed?.values?.[0] ?? fees.value);
 });
 
 const withdrawAccount = computed(() => {
diff --git a/packages/app-lib/src/state/profiles.rs b/packages/app-lib/src/state/profiles.rs
index 575989295..cd8b41305 100644
--- a/packages/app-lib/src/state/profiles.rs
+++ b/packages/app-lib/src/state/profiles.rs
@@ -151,7 +151,7 @@ impl From<InnerProjectPathUnix> for RawProjectPath {
 pub struct ProjectPathId(pub PathBuf);
 impl ProjectPathId {
     // Create a new ProjectPathId from a full file path
-    pub async fn from_fs_path(path: &PathBuf) -> crate::Result<Self> {
+    pub async fn from_fs_path(path: &Path) -> crate::Result<Self> {
         // This is avoiding dunce::canonicalize deliberately. On Windows, paths will always be convert to UNC,
         // but this is ok because we are stripping that with the prefix. Using std::fs avoids different behaviors with dunce that
         // come with too-long paths
@@ -159,7 +159,7 @@ impl ProjectPathId {
             State::get().await?.directories.profiles_dir().await,
         )?;
         // Normal canonizing resolves symlinks, which results in "path not corresponding to profile" error
-        let path = path::absolute(path.to_path_buf())?;
+        let path = path::absolute(path)?;
         let path = path
             .strip_prefix(profiles_dir)
             .ok()
diff --git a/packages/ui/src/components/base/Slider.vue b/packages/ui/src/components/base/Slider.vue
index 31c7b8c55..cc6d4494c 100644
--- a/packages/ui/src/components/base/Slider.vue
+++ b/packages/ui/src/components/base/Slider.vue
@@ -81,12 +81,12 @@ const props = withDefaults(defineProps<Props>(), {
 const currentValue = ref(Math.max(props.min, props.modelValue))
 
 const inputValueValid = (inputValue: number) => {
-  let newValue = inputValue || props.min;
+  let newValue = inputValue || props.min
 
   if (props.forceStep) {
-    newValue -= newValue % props.step;
+    newValue -= newValue % props.step
   }
-  newValue = Math.max(props.min, Math.min(newValue, props.max));
+  newValue = Math.max(props.min, Math.min(newValue, props.max))
 
   currentValue.value = newValue
   emit('update:modelValue', currentValue.value)
diff --git a/packages/ui/src/components/modal/ShareModal.vue b/packages/ui/src/components/modal/ShareModal.vue
index 50d20f55c..c19e0da71 100644
--- a/packages/ui/src/components/modal/ShareModal.vue
+++ b/packages/ui/src/components/modal/ShareModal.vue
@@ -33,7 +33,7 @@ const props = defineProps({
   openInNewTab: {
     type: Boolean,
     default: true,
-  }
+  },
 })
 
 const shareModal = ref(null)
@@ -94,9 +94,7 @@ const sendEmail = computed(
     &body=${encodeURIComponent(content.value)}`,
 )
 
-const targetParameter = computed(
-  () => props.openInNewTab ? "_blank" : "_self"
-)
+const targetParameter = computed(() => (props.openInNewTab ? '_blank' : '_self'))
 
 const sendTweet = computed(
   () => `https://twitter.com/intent/tweet?text=${encodeURIComponent(content.value)}`,
@@ -145,7 +143,12 @@ defineExpose({
           <Button v-if="canShare" v-tooltip="'Share'" icon-only @click="share">
             <ShareIcon />
           </Button>
-          <a v-tooltip="'Send as an email'" class="btn icon-only" :href="sendEmail" :target="targetParameter">
+          <a
+            v-tooltip="'Send as an email'"
+            class="btn icon-only"
+            :href="sendEmail"
+            :target="targetParameter"
+          >
             <MailIcon />
           </a>
           <a