feat: fix pr

Author: MinimaxLanbo

apps/webuiapps/src/components/ChatPanel/index.tsx

@@ -425,7 +425,10 @@ const ChatPanel: React.FC<{ onClose: () => void; visible?: boolean }> = ({
     loadMemories(sessionPath).then(setMemories);
   }, [sessionPath, modCollection, seedPrologue]);
 
-  // Load configs from file (async override)
+  // Load configs from file (async override).
+  // Empty deps [] is intentional: configs (character collection, mod collection,
+  // chat config, image-gen config) are loaded inside the effect and written to
+  // state — they are not external dependencies that should trigger re-runs.
   useEffect(() => {
     loadConfig().then((fileConfig) => {
       if (fileConfig) setConfig(fileConfig);

apps/webuiapps/src/lib/characterManager.ts

@@ -187,8 +187,8 @@ function migrateOldFormat(): CharacterCollection | null {
         return collection;
       }
     }
-  } catch {
-    // ignore
+  } catch (e) {
+    console.warn('[CharacterManager] migrateOldFormat failed:', e);
   }
   return null;
 }
@@ -203,8 +203,8 @@ export async function loadCharacterCollection(): Promise<CharacterCollection | n
         return data as CharacterCollection;
       }
     }
-  } catch {
-    // API not available
+  } catch (e) {
+    console.warn('[CharacterManager] loadCharacterCollection API not available:', e);
   }
   return null;
 }
@@ -216,8 +216,8 @@ export function loadCharacterCollectionSync(): CharacterCollection | null {
       const parsed = JSON.parse(raw);
       if (parsed.activeId && parsed.items) return parsed as CharacterCollection;
     }
-  } catch {
-    // ignore
+  } catch (e) {
+    console.warn('[CharacterManager] loadCharacterCollectionSync failed:', e);
   }
   return migrateOldFormat();
 }
@@ -230,8 +230,8 @@ export async function saveCharacterCollection(collection: CharacterCollection):
       headers: { 'Content-Type': 'application/json' },
       body: JSON.stringify(collection),
     });
-  } catch {
-    // Silently ignore
+  } catch (e) {
+    console.warn('[CharacterManager] saveCharacterCollection failed:', e);
   }
 }
 
@@ -367,11 +367,6 @@ export function clearEmotionVideoCache(characterId?: string): void {
   }
 }
 
-/** @deprecated Use resolveEmotionMedia instead */
-export function resolveEmotionImage(config: CharacterConfig, emotion?: string): string | undefined {
-  return resolveEmotionMedia(config, emotion)?.url;
-}
-
 export function getCharacterPromptContext(config: CharacterConfig): string {
   return (
     `You are ${config.character_name}, a ${config.character_gender_desc} character.\n` +

apps/webuiapps/src/lib/diskStorage.ts

@@ -42,7 +42,8 @@ export async function listFiles(dirPath: string): Promise<{
       return await res.json();
     }
     return { files: [], not_exists: true };
-  } catch {
+  } catch (e) {
+    console.warn('[diskStorage] listFiles failed:', e);
     return { files: [], not_exists: true };
   }
 }
@@ -60,7 +61,8 @@ export async function getFile(filePath: string): Promise<unknown> {
     } catch {
       return text;
     }
-  } catch {
+  } catch (e) {
+    console.warn('[diskStorage] getFile failed:', e);
     return null;
   }
 }
@@ -81,8 +83,8 @@ export async function putTextFilesByJSON(data: {
         headers: { 'Content-Type': 'text/plain' },
         body: file.content || '',
       });
-    } catch {
-      // silently ignore
+    } catch (e) {
+      console.warn('[diskStorage] putTextFilesByJSON write failed:', e);
     }
   });
   await Promise.all(promises);
@@ -140,7 +142,8 @@ export async function searchFiles(data: { query: string }): Promise<unknown[]> {
         parentId: null,
         metadata: { size: f.size || 0 },
       }));
-  } catch {
+  } catch (e) {
+    console.warn('[diskStorage] searchFiles failed:', e);
     return [];
   }
 }

apps/webuiapps/src/lib/modManager.ts

@@ -440,7 +440,10 @@ export class ModManager {
 
     for (const id of targetIds) {
       if (validTargets.includes(id) && !this.state.completed_targets.includes(id)) {
-        this.state.completed_targets.push(id);
+        this.state = {
+          ...this.state,
+          completed_targets: [...this.state.completed_targets, id],
+        };
         newlyCompleted.push(id);
       }
     }
@@ -460,10 +463,13 @@ export class ModManager {
 
     // Stage complete — advance
     const completedStage = { index: stage.stage_index, name: stage.stage_name };
-    this.state.current_stage_index++;
+    this.state = {
+      ...this.state,
+      current_stage_index: this.state.current_stage_index + 1,
+    };
 
     if (this.state.current_stage_index >= this.config.stage_count) {
-      this.state.is_finished = true;
+      this.state = { ...this.state, is_finished: true };
       return {
         message: `Stage "${stage.stage_name}" completed! All stages finished!`,
         stageCompleted: true,