From adea9b3f32542e5b7069f840244762b858925abf Mon Sep 17 00:00:00 2001 From: Frank Xu Date: Thu, 16 May 2024 06:06:06 -0400 Subject: [PATCH] fix frontend browsing screenshot, allow link following in MD (#1817) --- frontend/src/services/observations.ts | 4 ++-- frontend/src/types/Message.tsx | 3 --- opendevin/runtime/browser/browser_env.py | 2 +- 3 files changed, 3 insertions(+), 6 deletions(-) diff --git a/frontend/src/services/observations.ts b/frontend/src/services/observations.ts index ea3d87263b..6677d0a236 100644 --- a/frontend/src/services/observations.ts +++ b/frontend/src/services/observations.ts @@ -17,8 +17,8 @@ export function handleObservationMessage(message: ObservationMessage) { store.dispatch(appendJupyterOutput(message.content)); break; case ObservationType.BROWSE: - if (message.screenshot) { - store.dispatch(setScreenshotSrc(message.screenshot)); + if (message.extras?.screenshot) { + store.dispatch(setScreenshotSrc(message.extras?.screenshot)); } if (message.extras?.url) { store.dispatch(setUrl(message.extras.url)); diff --git a/frontend/src/types/Message.tsx b/frontend/src/types/Message.tsx index cdcd971518..1ff375e905 100644 --- a/frontend/src/types/Message.tsx +++ b/frontend/src/types/Message.tsx @@ -21,7 +21,4 @@ export interface ObservationMessage { // A friendly message that can be put in the chat log message: string; - - // optional screenshot - screenshot?: string; } diff --git a/opendevin/runtime/browser/browser_env.py b/opendevin/runtime/browser/browser_env.py index fa19779af9..60854f434c 100644 --- a/opendevin/runtime/browser/browser_env.py +++ b/opendevin/runtime/browser/browser_env.py @@ -23,7 +23,7 @@ class BrowserEnv: def __init__(self): self.html_text_converter = html2text.HTML2Text() # ignore links and images - self.html_text_converter.ignore_links = True + self.html_text_converter.ignore_links = False self.html_text_converter.ignore_images = True # use alt text for images self.html_text_converter.images_to_alt = True