feat: less bad launches

rust/src/cmd.rs

@@ -9,18 +9,21 @@ use crate::profile::Profile;
 use crate::appdata::AppData;
 use crate::{liner, start};
 
-use tauri::State;
+use tauri::{AppHandle, Manager, State};
 
 #[tauri::command]
-pub async fn startline(state: State<'_, Mutex<AppData>>) -> Result<(), String> {
+pub async fn startline(app: AppHandle) -> Result<(), String> {
     log::debug!("invoke: startline");
 
+    let app_copy = app.clone();
+    let state = app.state::<Mutex<AppData>>();
     let appd = state.lock().await;
 
     if let Some(p) = &appd.profile {
         // TODO if p.needsUpdate
         liner::line_up(p).await.expect("Line-up failed");
-        start::start(p).map_err(|e| { log::error!("Error launching: {}", e.to_string()); e.to_string() }).map(|_| ())
+        start::start(p, app_copy)
+            .map_err(|e| { log::error!("Error launching: {}", e.to_string()); e.to_string() })
         //Ok(())
     } else {
         Err("No profile".to_owned())

rust/src/pkg_store.rs

@@ -1,4 +1,4 @@
-use std::collections::HashMap;
+use std::collections::{HashMap, HashSet};
 use std::path::Path;
 use anyhow::{Result, anyhow};
 use serde::{Deserialize, Serialize};
@@ -6,7 +6,7 @@ use tauri::{AppHandle, Emitter};
 use tokio::fs;
 use tokio::task::JoinSet;
 use crate::model::rainy;
-use crate::pkg::{Package, PkgKey};
+use crate::pkg::{Package, PkgKey, Remote};
 use crate::util;
 use crate::download_handler::DownloadHandler;
 
@@ -134,12 +134,14 @@ impl PackageStore {
             pkg: key.to_owned()
         })?;
 
-        let rmt = pkg.rmt.as_ref() //clone()
+        let rmt = pkg.rmt.as_ref()
             .ok_or_else(|| anyhow!("Attempted to install a pkg without remote data"))?;
 
         if install_deps {
-            for dep in &rmt.dependencies {
-                Box::pin(self.install_package(&dep, false, true)).await?;
+            let mut set = HashSet::new();
+            self.resolve_deps(rmt.clone(), &mut set)?;
+            for dep in set {
+                Box::pin(self.install_package(&dep, false, false)).await?;
             }
         }
 
@@ -244,4 +246,18 @@ impl PackageStore {
 
         Ok(())
     }
+
+    fn resolve_deps(&self, rmt: Remote, set: &mut HashSet<PkgKey>) -> Result<()> {
+        for d in rmt.dependencies {
+            set.insert(d.clone());
+            let subrmt = self.store.get(&d)
+                .ok_or_else(|| anyhow!("Attempted to delete a nonexistent pkg"))?
+                .rmt
+                .clone()
+                .ok_or_else(|| anyhow!("Attempted to resolve deps without fetching"))?;
+            self.resolve_deps(subrmt, set)?;
+        }
+
+        Ok(())
+    }
 }

rust/src/start.rs

@@ -1,18 +1,32 @@
 use anyhow::Result;
 use tokio::process::Command;
+use tauri::{AppHandle, Emitter};
 use crate::profile::Profile;
 use crate::util;
 
 #[cfg(target_os = "linux")]
-pub fn start(p: &Profile) -> Result<()> {
-    Command::new(p.wine_runtime.as_ref().unwrap())
+pub fn start(p: &Profile, app: AppHandle) -> Result<()> {
+    let p = p.clone();
+    tauri::async_runtime::spawn(async move {
+        let rv = Command::new(p.wine_runtime.as_ref().unwrap())
             .env(
                 "SEGATOOLS_CONFIG_PATH",
                 util::profile_dir(&p).join("segatools.ini"),
             )
             .env("WINEPREFIX", p.wine_prefix.as_ref().unwrap())
             .arg(p.exe_dir.join("start.bat"))
-        .spawn()?;
+            .spawn();
+        match rv {
+            Ok(mut child) => {
+                _ = child.wait().await;
+                log::debug!("Fin");
+            },
+            Err(e) => {
+                log::error!("Fail: {}", e);
+            }
+        }
+        _ = app.emit("launch-end", "");
+    });
 
     Ok(())
 }
@@ -72,6 +86,8 @@ pub fn start(p: &Profile) -> Result<()> {
         set.join_next().await.expect("No spawn").expect("No result");
 
         log::debug!("Fin");
+
+        app.emit("launch-end", "");
     });
 
     Ok(())

rust/tauri.conf.json

@@ -42,12 +42,6 @@
     "bundle": {
         "active": true,
         "targets": "all",
-        "icon": [
-            "icons/32x32.png",
-            "icons/128x128.png",
-            "icons/128x128@2x.png",
-            "icons/icon.icns",
-            "icons/icon.ico"
-        ]
+        "icon": ["icons/slow.png"]
     }
 }

src/components/App.vue

@@ -7,6 +7,7 @@ import TabPanel from 'primevue/tabpanel';
 import TabPanels from 'primevue/tabpanels';
 import Tabs from 'primevue/tabs';
 import { invoke } from '@tauri-apps/api/core';
+import { listen } from '@tauri-apps/api/event';
 import { onOpenUrl } from '@tauri-apps/plugin-deep-link';
 import { open } from '@tauri-apps/plugin-dialog';
 import ModList from './ModList.vue';
@@ -19,6 +20,7 @@ const store = usePkgStore();
 store.setupListeners();
 
 const currentTab = ref('3');
+const startEnabled = ref(true);
 
 const loadProfile = async () => {
     await store.reloadProfile();
@@ -49,9 +51,8 @@ const loadProfile = async () => {
 const isProfileDisabled = computed(() => store.profile === null);
 
 const startline = async () => {
+    startEnabled.value = false;
     await invoke('startline');
-
-    //startDisabled.value = true;
 };
 
 onOpenUrl((urls) => {
@@ -61,6 +62,10 @@ onOpenUrl((urls) => {
 onMounted(async () => {
     await loadProfile();
 });
+
+listen('launch-end', () => {
+    startEnabled.value = true;
+});
 </script>
 
 <template>
@@ -82,7 +87,7 @@ onMounted(async () => {
                     ></Tab>
                     <div class="grow"></div>
                     <Button
-                        :disabled="false"
+                        :disabled="!startEnabled"
                         icon="pi pi-play"
                         label="START"
                         aria-label="start"

src/components/ModListEntry.vue

@@ -4,6 +4,7 @@ import ToggleSwitch from 'primevue/toggleswitch';
 import { open } from '@tauri-apps/plugin-shell';
 import InstallButton from './InstallButton.vue';
 import ModTitlecard from './ModTitlecard.vue';
+import UpdateButton from './UpdateButton.vue';
 import { usePkgStore } from '../stores';
 import { Package } from '../types';
 
@@ -21,6 +22,7 @@ const toggle = async (value: boolean) => {
 <template>
     <div class="flex items-center">
         <ModTitlecard showVersion :pkg="pkg" />
+        <UpdateButton :pkg="pkg" />
         <ToggleSwitch
             class="scale-[1.33] shrink-0"
             inputId="switch"
@@ -28,7 +30,7 @@ const toggle = async (value: boolean) => {
             :modelValue="store.isEnabled(pkg)"
             v-on:value-change="toggle"
         />
-        <InstallButton />
+        <InstallButton :pkg="pkg" />
         <Button
             rounded
             icon="pi pi-folder"

src/components/ModStore.vue

@@ -11,7 +11,12 @@ onMounted(() => {
 </script>
 
 <template>
-    <div v-for="p in pkgs.allRemote" class="flex flex-row">
+    <div
+        v-for="p in pkgs.allRemote.sort((p1, p2) =>
+            p1.name.localeCompare(p2.name)
+        )"
+        class="flex flex-row"
+    >
         <ModStoreEntry :pkg="p" />
     </div>
 </template>

src/components/ModTitlecard.vue

@@ -1,6 +1,7 @@
 <script setup lang="ts">
 import { convertFileSrc } from '@tauri-apps/api/core';
 import { Package } from '../types';
+import { needsUpdate } from '../util';
 
 const props = defineProps({
     pkg: Object as () => Package,
@@ -39,12 +40,20 @@ const iconSrc = () => {
             >
                 by&nbsp;{{ pkg.namespace }}
             </span>
+            <span class="m-2">
                 <span
                     v-if="showVersion && pkg?.loc?.version"
-                class="text-sm opacity-75 m-2"
+                    class="text-sm opacity-75"
                 >
                     {{ pkg?.loc?.version ?? '?.?.?' }}
                 </span>
+                <span
+                    v-if="showVersion && needsUpdate(pkg)"
+                    class="text-sm opacity-75"
+                >
+                    -> {{ pkg?.rmt?.version ?? '?.?.?' }}</span
+                >
+            </span>
         </div>
         <div class="text-sm opacity-75">
             {{ pkg?.description ?? 'No description' }}

src/components/UpdateButton.vue

@@ -0,0 +1,41 @@
+<script setup lang="ts">
+import Button from 'primevue/button';
+import { invoke } from '@tauri-apps/api/core';
+import { Package } from '../types';
+import { needsUpdate, pkgKey } from '../util';
+
+const props = defineProps({
+    pkg: Object as () => Package,
+});
+
+const install = async () => {
+    if (props.pkg === undefined) {
+        return;
+    }
+
+    try {
+        await invoke('install_package', { key: pkgKey(props.pkg) });
+    } catch (err) {
+        console.error(err);
+        if (props.pkg !== undefined) {
+            props.pkg.js.busy = false;
+        }
+    }
+
+    //if (rv === 'Deferred') { /* download progress */ }
+};
+</script>
+
+<template>
+    <Button
+        v-if="needsUpdate(pkg) && !pkg?.js.busy"
+        rounded
+        icon="pi pi-sync"
+        severity="success"
+        aria-label="remove"
+        size="small"
+        class="self-center mr-4"
+        style="width: 2rem; height: 2rem"
+        v-on:click="install()"
+    />
+</template>

src/util.ts

@@ -20,3 +20,23 @@ export const changePrimaryColor = (game: 'Ongeki' | 'Chunithm') => {
 };
 
 export const pkgKey = (pkg: Package) => `${pkg.namespace}-${pkg.name}`;
+
+export const needsUpdate = (pkg: Package | undefined) => {
+    const loc = pkg?.loc?.version;
+    const rmt = pkg?.rmt?.version;
+
+    if (loc === undefined || rmt === undefined) {
+        return false;
+    }
+
+    const [l1, l2, l3] = loc.split('.');
+    const [r1, r2, r3] = rmt.split('.');
+
+    if (l1 === r1) {
+        if (l2 === r2) {
+            return l3 < r3;
+        }
+        return l2 < r2;
+    }
+    return l1 < r1;
+};