Robert Brennan

@openhands update this test ``` =================================== FAILURES =================================== _____ test_run_controller_with_context_window_exceeded_without_truncation ______ [gw3] linux -- Python 3.12.4 /home/runner/.cache/pypoetry/virtualenvs/openhands-ai-2f2V0C4o-py3.12/bin/python mock_agent = mock_runtime = mock_memory = test_event_stream = EventStream(sid='test', file_store=, user_id=None, cur_id=6, cache_size=25)...

@openhands update these tests ``` ================================== FAILURES =================================== _______________________ test_exception_during_execution _______________________ windows_bash_session = def test_exception_during_execution(windows_bash_session): """Test handling of exceptions during command execution.""" # Patch the PowerShell class itself within the...

@openhands fix the merge conflicts. There are a lot of them--fix them ALL

@openhands fix the failed tests ``` [gw1] FAILED tests/unit/resolver/github/test_send_pull_request.py::test_send_pull_request[ready-None-Custom PR Title] tests/unit/resolver/github/test_send_pull_request.py::test_send_pull_request[draft-develop-Another Custom Title] [gw0] PASSED tests/unit/frontend/test_translation_completeness.py::TestTranslationCompleteness::test_translation_completeness_check_runs tests/unit/resolver/github/test_guess_success.py::test_guess_success_multiline_explanation [gw0] PASSED tests/unit/resolver/github/test_guess_success.py::test_guess_success_multiline_explanation tests/unit/resolver/github/test_guess_success.py::test_pr_handler_guess_success_with_thread_comments [gw2] FAILED tests/unit/resolver/gitlab/test_gitlab_send_pull_request.py::test_send_pull_request[branch-None-None] tests/unit/resolver/gitlab/test_gitlab_send_pull_request.py::test_send_pull_request[draft-None-None] [gw0] PASSED tests/unit/resolver/github/test_guess_success.py::test_pr_handler_guess_success_with_thread_comments tests/unit/resolver/github/test_guess_success.py::test_pr_handler_guess_success_only_review_comments...

@openhands follow the instructions in REFACTOR.md. Follow each step in order. Commit and push each time you finish with a particular step. But DO NOT PUSH until the pre-commit is...

@openhands fix these tests ``` =========================== short test summary info ============================ FAILED tests/unit/test_cli.py::test_main_without_task FAILED tests/unit/test_cli.py::test_main_with_session_name_passes_name_to_run_session FAILED tests/unit/test_cli_settings.py::TestModifyLLMSettingsBasic::test_modify_llm_settings_basic_success FAILED tests/unit/test_cli_settings.py::TestModifyLLMSettingsBasic::test_modify_llm_settings_basic_user_cancels FAILED tests/unit/test_cli_settings.py::TestModifyLLMSettingsBasic::test_modify_llm_settings_basic_invalid_input ============ 5 failed, 1141 passed, 23 skipped in 75.92s...

Hmm I'm not sure we need a new websocket or EventStream here. Instead, I'd like to figure out why errors aren't propagating immediately over the current WS/ES--is there some reason...

Yeah we probably only want to unzip when doing the project upload. Guessing this is a long-tail issue though, not high priority

Yeah @amanape let's add the TOS to the all-hands.dev marketing site, rather than the OSS project

Closing thanks to the alternate fix