Fix issue #5015: [Bug]: Headless mode awaits for requested user feedb…

[enyst]

End-user friendly description of the problem this fixes or functionality that this introduces

• [ ] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below Headless mode must not stop to ask the user for input.

Explain what this PR does

When the app is running in headless mode, it must not ask the user for input. This means two things:

• if the agent is nevertheless waiting for input, send it automatically some message, apparently from a fake "user", to force it to continue instead of waiting.
• do not raise.

Read the comments in the original issue: you will find an example of how headless mode is implemented silently in external scripts.

You have a unit test for headless mode set. Read it and fix it to conform to this behavior. Then add a test for headless mode not set.

---

Link of any specific issues this addresses Fix #5015

---

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:200d70d-nikolaik   --name openhands-app-200d70d   docker.all-hands.dev/all-hands-ai/openhands:200d70d

[github-actions[bot]]

OpenHands started fixing the pr! You can monitor the progress here.

[enyst]

The PR you are fixing is: https://github.com/All-Hands-AI/OpenHands/pull/5246

Read all its contents before starting.

[enyst]

@openhands-agent End-user friendly description of the problem this fixes or functionality that this introduces: Headless mode must not stop to ask the user for input.

Explain what this PR does

When the app is running in headless mode, it must not ask the user for input. This means two things:

• if the agent is nevertheless waiting for input, send it automatically some message, apparently from a fake "user", to force it to continue instead of waiting.
• do not raise.

Read the comments in the original issue: you will find an example of how headless mode is implemented silently in external scripts.

[github-actions[bot]]

OpenHands started fixing the pr! You can monitor the progress here.

[enyst]

openhands-agent, the previous comments were mostly solved. We have a few more things to do now:

• review where the AgentController is initialized, and if headless mode is set, set a fake user response there
• HOWEVER, do not overwrite a fake user response coming from a caller, if that is the case!
• verify what fake user responses exist in the codebase. Hint: ./evaluation subfolders.
• only if none is provided, and the mode is headless, have some default that is simple, and tells the agent something similar with the existing ones
• unit tests: fix the test on this issue, and add a test for when the mode isn't headless.

[github-actions[bot]]

OpenHands started fixing the pr! You can monitor the progress here.

[enyst]

@openhands-agent Python linting is failing on this PR in CI. You know how to run python linting in this project. Fix it and commit the fix.

[github-actions[bot]]

The workflow to fix this issue encountered an error. Please check the workflow logs for more information.

[enyst]

@openhands-agent Look carefully at the diff of this PR. Undo the exact changes you made to traffic control. Only those, and commit; the rest is fine.

[github-actions[bot]]

OpenHands started fixing the pr! You can monitor the progress here.