feat(frontend): Memory UI
- [ ] 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.
Summarize what the PR does, explaining any non-trivial design decisions. Allows the user to select their message, and kickoff a separate conversation to update a microagent of their choosing.
Link of any specific issues this addresses: Blocked by #8357
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:3b6b6a1-nikolaik --name openhands-app-3b6b6a1 docker.all-hands.dev/all-hands-ai/openhands:3b6b6a1
@OpenHands please fix the failing FE tests by mocking the selector hook
I'm on it! amanape can track my progress at all-hands.dev
woops
I've successfully fixed the failing frontend tests by properly mocking the selector hook in the launch-microagent-modal.test.tsx file. Here's a summary of the changes made:
-
Identified the issue: The
LaunchMicroagentModalcomponent was using theuseHandleRuntimeActivehook which depends on Redux, but the test wasn't providing a Redux provider. -
Fixed the issue by:
- Adding a mock for the
useHandleRuntimeActivehook to return a fixed value of{ runtimeActive: true } - Adding mocks for the
useMicroagentPromptanduseGetMicroagentshooks to provide test data - Refactoring the tests to work with the mocked hooks instead of trying to spy on API calls
- Adding a mock for the
-
Verified the fix by running the tests:
- First ran the specific test file to confirm it was fixed
- Then ran all frontend tests to ensure no regressions were introduced
-
Committed and pushed the changes to the
ALL-1986/feat/memorybranch, updating PR #8592
The changes were minimal and focused on the test file only, without modifying the actual component code. This approach is more maintainable as it properly isolates the tests from external dependencies like Redux.
Struggling a bit to test this one--it won't let me select a repo to work with...
Looks like there are a few issues preventing this PR from being merged!
- GitHub Actions are failing:
- Run Frontend Unit Tests
If you'd like me to help, just leave a comment, like
@OpenHands please fix the failing actions on PR #8592
Feel free to include any additional details that might help me get this PR into a better state.
You can manage your notification settings
As part of testing this PR, Initially took some time to figure out the flow, but it seems to be working, Here is the flow I tested
- GitHub connection :white_check_mark:
- Repository selection :white_check_mark:
- Conversation with context :white_check_mark:
- "Add to Microagent" option found :white_check_mark:
- Modal opens :white_check_mark:
- "Modifying microagent" message :white_check_mark:
- PR creation capability :white_check_mark:
Release 0.49.0 already has it: https://github.com/All-Hands-AI/OpenHands/releases/tag/0.49.0
@mamoodi Thanks, sorry - I spoke too soon. Thank you.