fix translation logic bugs

Author: adidahiya

packages/raga-web-app/src/store/slices/audioFilesServerSlice.tsx

@@ -10,6 +10,7 @@ import { withTimeout } from "../../common/asyncUtils";
 import {
   AUDIO_FILES_SERVER_PING_TIMEOUT,
   DEFAULT_AUDIO_FILES_SERVER_PORT,
+  WRITE_AUDIO_FILE_TAG_TIMEOUT,
 } from "../../common/constants";
 import {
   type AudioFilesServerStartedEventPayload,
@@ -218,8 +219,11 @@ export const createAudioFilesServerSlice: AppStoreSliceCreator<
       } satisfies WriteAudioFileTagOptions);
 
       try {
-        yield* call(() =>
-          window.api.waitForResponse(ServerEventChannel.WRITE_AUDIO_FILE_TAG_COMPLETE),
+        yield* call(
+          window.api.waitForResponse(
+            ServerEventChannel.WRITE_AUDIO_FILE_TAG_COMPLETE,
+            WRITE_AUDIO_FILE_TAG_TIMEOUT,
+          ),
         );
 
         log.info(`[client] completed updating '${tagName}' tag for track ${trackID.toString()}`);

packages/raga-web-app/src/store/slices/librarySlice.ts

@@ -6,7 +6,7 @@ import type {
 import { call, type Operation } from "effection";
 import { Roarr as log } from "roarr";
 
-import { DEBUG } from "../../common/constants";
+import { DEBUG, LOAD_SWINSIAN_LIBRARY_TIMEOUT } from "../../common/constants";
 import { ClientErrors } from "../../common/errorMessages";
 import {
   ClientEventChannel,
@@ -201,10 +201,12 @@ export const createLibrarySlice: AppStoreSliceCreator<LibraryState & LibraryActi
         return;
       } else {
         // In desktop mode, use IPC
+        log.debug("[client] Sending load library request via IPC");
         window.api.send(ClientEventChannel.LOAD_SWINSIAN_LIBRARY, options);
-        data = yield* call(() =>
+        data = yield* call(
           window.api.waitForResponse<LoadedSwinsianLibraryEventPayload>(
             ServerEventChannel.LOADED_SWINSIAN_LIBRARY,
+            LOAD_SWINSIAN_LIBRARY_TIMEOUT,
           ),
         );
       }
@@ -275,9 +277,7 @@ export const createLibrarySlice: AppStoreSliceCreator<LibraryState & LibraryActi
     });
 
     try {
-      yield* call(() =>
-        window.api.waitForResponse(ServerEventChannel.WRITE_MODIFIED_LIBRARY_COMPLETE),
-      );
+      yield* call(window.api.waitForResponse(ServerEventChannel.WRITE_MODIFIED_LIBRARY_COMPLETE));
       set({ libraryWriteState: "none" });
     } catch {
       log.error(ClientErrors.LIBRARY_WRITE_TIMED_OUT);

packages/raga-web-app/src/webApi.ts

@@ -12,7 +12,10 @@ export interface WebContextBridgeApi {
   // Mock IPC methods
   send: (channel: ClientEventChannel, ...args: unknown[]) => void;
   handleOnce: <T = unknown>(channel: ServerEventChannel, callback: (payload: T) => void) => void;
-  waitForResponse: <T = unknown>(channel: ServerEventChannel) => Promise<T>;
+  waitForResponse: <T = unknown>(
+    channel: ServerEventChannel,
+    timeoutMs?: number,
+  ) => () => Promise<T>;
 
   // File operations
   getFilePath: (file: File | null | undefined) => string | undefined;
@@ -23,18 +26,6 @@ class WebApi implements WebContextBridgeApi {
 
   private pendingCallbacks = new Map<string, ((payload: unknown) => void)[]>();
 
-  // constructor() {
-  //   // Detect actual platform if possible
-  //   const userAgent = navigator.userAgent.toLowerCase();
-  //   if (userAgent.includes("mac")) {
-  //     this.platform = "darwin";
-  //   } else if (userAgent.includes("win")) {
-  //     this.platform = "win32";
-  //   } else if (userAgent.includes("linux")) {
-  //     this.platform = "linux";
-  //   }
-  // }
-
   send(channel: ClientEventChannel, ...args: unknown[]): void {
     console.warn(`[WebApi] IPC send called with channel: ${String(channel)}`, args);
     // In a real implementation, this could send requests to a backend API
@@ -48,10 +39,11 @@ class WebApi implements WebContextBridgeApi {
     this.pendingCallbacks.set(channelKey, callbacks);
   }
 
-  async waitForResponse<T = unknown>(channel: ServerEventChannel): Promise<T> {
+  waitForResponse<T = unknown>(channel: ServerEventChannel, _timeoutMs = 0): () => Promise<T> {
     console.warn(`[WebApi] IPC waitForResponse called with channel: ${String(channel)}`);
-    // Return a rejected promise for now - in a real implementation this would wait for backend response
-    return Promise.reject(new Error(`Web API does not support IPC channel: ${String(channel)}`));
+    // Return a function that returns a rejected promise for now - in a real implementation this would wait for backend response
+    return () =>
+      Promise.reject(new Error(`Web API does not support IPC channel: ${String(channel)}`));
   }
 
   getFilePath(file: File | null | undefined): string | undefined {