OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

fix(frontend): Fix `actionHasObservationPair` function to correctly identify pairs

Open amanape opened this issue 7 months ago • 1 comments

  • [ ] This change is worth documenting at https://docs.all-hands.dev/
  • [ ] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

End-user friendly description of the problem this fixes or functionality this introduces. We would show actions + observations instead of replace actions with their obsevation

This was because some actions come with thoughts, which we want to render. But we didnt take into account actions that didnt include them, so we rendered those instead of replace them with thoughts

Summarize what the PR does, explaining any non-trivial design decisions.


Link of any specific issues this addresses:


To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:437b471-nikolaik   --name openhands-app-437b471   docker.all-hands.dev/all-hands-ai/openhands:437b471

amanape avatar May 23 '25 14:05 amanape

Looks like there are a few issues preventing this PR from being merged!

  • GitHub Actions are failing:
    • Run Python Unit Tests

If you'd like me to help, just leave a comment, like

@OpenHands please fix the failing actions on PR #8661

Feel free to include any additional details that might help me get this PR into a better state.

You can manage your notification settings

openhands-staging[bot] avatar May 23 '25 18:05 openhands-staging[bot]