fix(tarko): resolve infinite re-render in BrowserControlRenderer hooks (#1303)

ulivz · web-flow · commit 72785617b6f4 · 2025-08-30T02:54:56.000+08:00
diff --git a/multimodal/tarko/agent-web-ui/src/standalone/workspace/renderers/browser-control/useMousePosition.ts b/multimodal/tarko/agent-web-ui/src/standalone/workspace/renderers/browser-control/useMousePosition.ts
@@ -58,7 +58,7 @@ export const useMousePosition = ({
         );
       }
     }
-  }, [activeSessionId, toolCallId, toolResults, mousePosition]);
+  }, [activeSessionId, toolCallId, toolResults]);
 
   return { mousePosition, previousMousePosition };
 };
diff --git a/multimodal/tarko/agent-web-ui/src/standalone/workspace/renderers/browser-control/useScreenshots.ts b/multimodal/tarko/agent-web-ui/src/standalone/workspace/renderers/browser-control/useScreenshots.ts
@@ -131,8 +131,6 @@ export const useScreenshots = ({
     toolCallId,
     environmentImage,
     currentStrategy,
-    afterActionImage,
-    beforeActionImage,
   ]);
 
   return {

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ export const useMousePosition = ({`
`58`	`58`	`);`
`59`	`59`	`}`
`60`	`60`	`}`
`61`		`- }, [activeSessionId, toolCallId, toolResults, mousePosition]);`
	`61`	`+ }, [activeSessionId, toolCallId, toolResults]);`
`62`	`62`
`63`	`63`	`return { mousePosition, previousMousePosition };`
`64`	`64`	`};`