fixed the bug in the amount of scroll schema while scrolling inside the container

Author: Jeomon

src/agent/web/__init__.py

@@ -130,7 +130,6 @@ async def action(self,state:AgentState):
             'informative_elements':browser_state.dom_state.informative_elements_to_string(),
             'scrollable_elements':browser_state.dom_state.scrollable_elements_to_string()
         })
-        print(browser_state.dom_state.scrollable_nodes)
         messages=[AIMessage(action_prompt),ImageMessage(text=observation_prompt,image_obj=image_obj) if self.use_vision and image_obj is not None else HumanMessage(observation_prompt)]
         return {**state,'messages':messages,'prev_observation':observation}
 

src/agent/web/tools/__init__.py

@@ -56,12 +56,12 @@ async def scroll_tool(direction:Literal['up','down']='up',index:int=None,amount:
     page=await context.get_current_page()
     if index is not None:
         element=await context.get_element_by_index(index=index)
-        handle=await context.get_handle_by_xpath(element.xpath)
-        frame=await context.get_frame_by_xpath(element.xpath)
+        handle=await context.get_handle_by_xpath(xpath=element.xpath)
+        frame=await context.get_frame_by_xpath(xpath=element.xpath)
         if direction=='up':
-            await frame.evaluate(f'element=> element.scrollBy(0,-{amount})', handle)
+            await frame.evaluate(f'(element)=> element.scrollBy(0,{-amount})', handle)
         elif direction=='down':
-            await frame.evaluate(f'element=> element.scrollBy(0,{amount})', handle)
+            await frame.evaluate(f'(element)=> element.scrollBy(0,{amount})', handle)
         else:
             raise ValueError('Invalid direction')
         return f'Scrolled {direction} inside the element at label {index} by {amount}'

src/agent/web/tools/views.py

@@ -21,7 +21,7 @@ class Wait(SharedBaseModel):
 
 class Scroll(SharedBaseModel):
     direction: Literal['up','down'] = Field(description="The direction to scroll content", examples=['up','down'], default='up')
-    amount: int = Field(description="Number of pixels to scroll, if None then scrolls by page/container height", examples=[100, 250, 500], default=None)
+    amount: int = Field(description="Number of pixels to scroll, if None then scrolls by page/container height. Must required for scrollable container elements and the amount should be small", examples=[100, 25, 50], default=None)
     index: int = Field(description="Index of specific scrollable container element, if None then scrolls the entire page", examples=[0, 5, 12], default=None)
 
 class GoTo(SharedBaseModel):