OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

Strip unnecessary tags from browser output

Open FreightCompanionDavid opened this issue 1 year ago • 4 comments

    # - Added HTML content preprocessing to remove `<script>`, `<style>`, and other non-textual elements.
    # - Improved working directory tracking for enhanced context in `cd` command executions.
    # - Refined memory management by optimizing the condensation of older memories for better efficiency.
    # - Set a timeout for `run` actions to address potential hang-ups with interactive commands.
    # - Laid the foundation for handling background processes, allowing for server initiation and management of long-running tasks.

FreightCompanionDavid avatar Mar 30 '24 19:03 FreightCompanionDavid

@FreightCompanionDavid thanks for the contribution!

The browser stripping tags is great--let's put it in opendevin/actions/browse.py so that other agents can use it.

I don't see these two updates:

  • Set a timeout for run actions to address potential hang-ups with interactive commands.
  • Laid the foundation for handling background processes, allowing for server initiation and management of long-running tasks.

We already have both of these in the controller.

The other stuff I think we should probably revert--I don't think it adds much to the agent. LMK if you think otherwise.

rbren avatar Mar 30 '24 23:03 rbren

I just did my best. 😁 I started to get a little bit lost when I was way deep in it so sorry about any accuracies

FreightCompanionDavid avatar Mar 31 '24 03:03 FreightCompanionDavid

@FreightCompanionDavid are you open to just splitting out the browser changes, and putting them in browser.py?

rbren avatar Apr 01 '24 17:04 rbren

Yes, I'm on it.

FreightCompanionDavid avatar Apr 01 '24 20:04 FreightCompanionDavid