Make progressUpdate one channel

The Frontend already filters this to only registered appNames
Ideally this would be refactored to go in a Zustand store, but one thing at a
time

src/backend/api/library.ts

@@ -1,4 +1,3 @@
-import { GameStatus } from 'common/types'
 import {
   makeListenerCaller as lc,
   makeHandlerInvoker as hi,
@@ -19,15 +18,7 @@ export const importGame = hi('importGame')
 
 export const handleGameStatus = fls('gameStatusUpdate')
 
-export const onProgressUpdate = (
-  appName: string,
-  onChange: (e: Electron.IpcRendererEvent, status: GameStatus) => void
-) => {
-  ipcRenderer.on(`progressUpdate-${appName}`, onChange)
-  return () => {
-    ipcRenderer.removeListener(`progressUpdate-${appName}`, onChange)
-  }
-}
+export const onProgressUpdate = fls('progressUpdate')
 
 export const handleLaunchGame = fls('launchGame')
 

src/backend/utils.ts

@@ -1040,7 +1040,7 @@ export async function moveOnWindows(
       const filenameMatch = data.match(/([\w.:\\]+)$/)?.[1]
       if (filenameMatch) currentFile = filenameMatch
 
-      sendFrontendMessage(`progressUpdate-${gameInfo.app_name}`, {
+      sendFrontendMessage(`progressUpdate`, {
         appName: gameInfo.app_name,
         runner: gameInfo.runner,
         status: 'moving',
@@ -1146,7 +1146,7 @@ export async function moveOnUnix(
           }
         }
 
-        sendFrontendMessage(`progressUpdate-${gameInfo.app_name}`, {
+        sendFrontendMessage(`progressUpdate`, {
           appName: gameInfo.app_name,
           runner: gameInfo.runner,
           status: 'moving',
@@ -1250,7 +1250,7 @@ function sendGameStatusUpdate(payload: GameStatus) {
 }
 
 function sendProgressUpdate(payload: GameStatus) {
-  sendFrontendMessage(`progressUpdate-${payload.appName}`, payload)
+  sendFrontendMessage(`progressUpdate`, payload)
   backendEvents.emit(`progressUpdate-${payload.appName}`, payload)
 }
 

src/common/ipc/types.ts

@@ -327,7 +327,7 @@ export interface FrontendMessages {
   progressOfWineManager: (version: string, progress: WineManagerStatus) => void
   'installing-winetricks-component': (component: string) => void
 
-  [key: `progressUpdate${string}`]: (progress: GameStatus) => void
+  progressUpdate: (progress: GameStatus) => void
 
   // Used inside tests, so we can be a bit lenient with the type checking here
   message: (...params: unknown[]) => void

src/frontend/hooks/hasProgress.ts

@@ -41,10 +41,8 @@ export const hasProgress = (appName: string) => {
         })
       }
     }
-    const setGameStatusRemoveListener = window.api.onProgressUpdate(
-      appName,
-      handleProgressUpdate
-    )
+    const setGameStatusRemoveListener =
+      window.api.onProgressUpdate(handleProgressUpdate)
 
     return () => {
       setGameStatusRemoveListener()