[bugfix] Added a Beat Saber ends watcher to delete created symlink to avoid Oculus version getting wiped by Oculus auto updates

src/main/services/bs-launcher/abstract-launcher.service.ts

@@ -5,9 +5,9 @@ import path from "path";
 import log from "electron-log";
 
 export abstract class AbstractLauncherService {
-    
+
     protected readonly localVersions = BSLocalVersionService.getInstance();
-    
+
     constructor(){
         this.localVersions = BSLocalVersionService.getInstance();
     }
@@ -46,18 +46,20 @@ export abstract class AbstractLauncherService {
 
     }
 
-    protected launchBs(bsExePath: string, args: string[], options?: SpawnOptionsWithoutStdio): Promise<number> {
-        return new Promise<number>((resolve, reject) => {
-            const bsProcess = this.launchBSProcess(bsExePath, args, options);
+    protected launchBs(bsExePath: string, args: string[], options?: SpawnOptionsWithoutStdio): {process: ChildProcessWithoutNullStreams, exit: Promise<number>} {
+        const process = this.launchBSProcess(bsExePath, args, options);
 
-            bsProcess.on("error", reject);
-            bsProcess.on("exit", resolve);
-
-            setTimeout(() => {
-                bsProcess.removeAllListeners("error");
-                bsProcess.removeAllListeners("exit");
-                resolve(-1);
-            }, 30_000);
+        const exit = new Promise<number>((resolve, reject) => {
+            process.on("error", (err) => {
+                log.error(`Error while launching BS`, err);
+                reject(err);
+            });
+            process.on("exit", (code) => {
+                log.info(`BS process exit with code ${code}`);
+                resolve(code);
+            });
         });
+
+        return { process, exit };
     }
-}
+}

src/main/services/bs-launcher/oculus-launcher.service.ts

@@ -12,6 +12,9 @@ import { taskRunning } from "../../helpers/os.helpers";
 import { CustomError } from "../../../shared/models/exceptions/custom-error.class";
 import { InstallationLocationService } from "../installation-location.service";
 import { ensurePathNotAlreadyExist } from "../../helpers/fs.helpers";
+import { UtilsService } from "../utils.service";
+import { spawn } from "child_process";
+import { tryit } from "../../../shared/helpers/error.helpers";
 
 export class OculusLauncherService extends AbstractLauncherService implements StoreLauncherInterface {
 
@@ -26,6 +29,7 @@ export class OculusLauncherService extends AbstractLauncherService implements St
 
     private readonly oculus: OculusService;
     private readonly pathsService: InstallationLocationService;
+    private readonly util: UtilsService;
 
     private readonly oculusLib$ = new ReplaySubject<string>();
 
@@ -33,6 +37,7 @@ export class OculusLauncherService extends AbstractLauncherService implements St
         super();
         this.oculus = OculusService.getInstance();
         this.pathsService = InstallationLocationService.getInstance();
+        this.util = UtilsService.getInstance();
 
         this.oculus.tryGetGameFolder([OCULUS_BS_DIR, OCULUS_BS_BACKUP_DIR]).then(async dirPath => {
             if(dirPath){
@@ -100,6 +105,15 @@ export class OculusLauncherService extends AbstractLauncherService implements St
         return rename(bsFolderBackupPath, originalPath);
     }
 
+    private launchSymlinkCleaner(pid: number, symlinkPath: string){
+
+        log.info("Launch Symlink Cleaner", pid, symlinkPath);
+
+        const exeName = "oculus_symlink_cleaner.exe";
+        const scriptPath = this.util.getAssetsScriptsPath();
+        spawn(path.join(scriptPath, exeName), [`${pid}`, symlinkPath], { detached: true, stdio: "ignore" });
+    }
+
     public launch(launchOptions: LaunchOption): Observable<BSLaunchEventData> {
 
         const prepareOriginalVersion: () => Promise<string> = async () => {
@@ -157,7 +171,16 @@ export class OculusLauncherService extends AbstractLauncherService implements St
                 obs.next({type: BSLaunchEvent.BS_LAUNCHING});
 
                 // Launch Beat Saber
-                return this.launchBs(exePath, this.buildBsLaunchArgs(launchOptions)).catch(err => {
+                const process = this.launchBs(exePath, this.buildBsLaunchArgs(launchOptions));
+
+                if(launchOptions.version.oculus !== true && process?.process?.pid){
+                    const { error } = tryit(() => this.launchSymlinkCleaner(process.process.pid, bsPath));
+                    if(error){
+                        log.error("Error while launching symlink cleaner", error);
+                    }
+                }
+
+                return process.exit.catch(err => {
                     throw CustomError.fromError(err, BSLaunchError.BS_EXIT_ERROR);
                 });
 

src/main/services/bs-launcher/steam-launcher.service.ts

@@ -107,7 +107,7 @@ export class SteamLauncherService extends AbstractLauncherService implements Sto
             obs.next({type: BSLaunchEvent.BS_LAUNCHING});
 
             const launchPromise = !launchOptions.admin ? (
-                this.launchBs(exePath, launchArgs, { env: {...process.env, "SteamAppId": BS_APP_ID} })
+                this.launchBs(exePath, launchArgs, { env: {...process.env, "SteamAppId": BS_APP_ID} }).exit
             ) : (
                 new Promise<number>(resolve => {
                     const adminProcess = exec(`"${this.getStartBsAsAdminExePath()}" "${exePath}" ${launchArgs.join(" ")}`, { env: {...process.env, "SteamAppId": BS_APP_ID} });

src/renderer/components/modal/modal-types/original-oculus-version-backup.modal.tsx

@@ -19,11 +19,10 @@ export const OriginalOculusVersionBackupModal: ModalComponent<boolean, void> = (
         <form className="max-w-[450px] text-gray-800 dark:text-gray-200">
             <h1 className="text-3xl uppercase tracking-wide w-full text-center text-gray-800 dark:text-gray-200">{t("modals.original-version-backup-oculus.title")}</h1>
             <BsmImage className="mx-auto h-20" image={BeatConflict} />
-            
+
             <p className="text-sm italic mb-4 font-bold">{t("modals.original-version-backup-oculus.body.must-be-installed-once")}</p>
             <p className="mb-4">{t("modals.original-version-backup-oculus.body.need-backup")}</p>
             <p className="mb-4">{t("modals.original-version-backup-oculus.body.can-restore-later")}</p>
-            <p className="text-sm italic mb-4">{t("modals.original-version-backup-oculus.body.tips-launch-oculus")}</p>
 
             <div className="flex flex-row justify-start items-center gap-1.5 my-4" >
                 <BsmCheckbox className="relative z-[1] w-6 aspect-square" checked={dontShowAgain} onChange={setDontShowAgain} />