From 9997188f5ec912f801cbc1927f4c540246dbf129 Mon Sep 17 00:00:00 2001
From: icarus <eurfelux@gmail.com>
Date: Sun, 12 Oct 2025 05:48:53 +0800
Subject: [PATCH] refactor(video): extract size update logic into separate
 callback

Improve code maintainability by separating size update logic into its own useCallback hook
---
 .../src/pages/video/settings/OpenAIParamSettings.tsx     | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/renderer/src/pages/video/settings/OpenAIParamSettings.tsx b/src/renderer/src/pages/video/settings/OpenAIParamSettings.tsx
index cbae9d0461..f71f4b3d47 100644
--- a/src/renderer/src/pages/video/settings/OpenAIParamSettings.tsx
+++ b/src/renderer/src/pages/video/settings/OpenAIParamSettings.tsx
@@ -30,6 +30,13 @@ export const OpenAIParamSettings = ({ params, updateParams }: OpenAIParamSetting
     [updateParams]
   )
 
+  const updateSize = useCallback(
+    (size: VideoSize) => {
+      updateParams({ params: { size } })
+    },
+    [updateParams]
+  )
+
   return (
     <SettingsGroup>
       <SettingItem>
@@ -56,7 +63,7 @@ export const OpenAIParamSettings = ({ params, updateParams }: OpenAIParamSetting
           labelPlacement="outside"
           selectedKeys={[params.params.size ?? '720x1280']}
           onSelectionChange={(keys) => {
-            if (keys.currentKey) updateParams({ params: { size: keys.currentKey as VideoSize } })
+            if (keys.currentKey) updateSize(keys.currentKey as VideoSize)
           }}
           items={sizeItems}
           selectionMode="single"