Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor I/O utils; allow 'task' command line parameter in cli.py #6187

Merged
merged 24 commits into from
Feb 19, 2025

Conversation

enyst
Copy link
Collaborator

@enyst enyst commented Jan 10, 2025

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
    Add ability to use CLI with a task to execute automatically (the user doesn't need to enter in chat the first task).

Give a summary of what the PR does, explaining any non-trivial design decisions

This PR proposes:

  • dedup CLI utils in their own openhands/io.py
  • fix task argument in cli.py

Link of any specific issues this addresses
Fix #5719


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

@enyst enyst requested review from xingyaoww and rbren January 10, 2025 05:23
@enyst enyst changed the title Small refactor I/O utils, task in cli mode Small refactor I/O utils, allow 'task' command line parameter in cli.py Jan 15, 2025
@enyst enyst added the lint-fix label Jan 21, 2025
@enyst enyst changed the title Small refactor I/O utils, allow 'task' command line parameter in cli.py Refactor I/O utils; allow 'task' command line parameter in cli.py Jan 22, 2025
@mamoodi
Copy link
Collaborator

mamoodi commented Feb 10, 2025

@enyst just a gentle ping to check if this is still on going.

@enyst enyst requested a review from rbren February 10, 2025 14:56
@enyst
Copy link
Collaborator Author

enyst commented Feb 10, 2025

Sure, this is ready for re-review.

@enyst enyst added the openhands Have Openhands attempt to fix the issue (via GH App) label Feb 19, 2025
@enyst enyst merged commit eed7e2d into main Feb 19, 2025
13 checks passed
@enyst enyst deleted the enyst/task-cli branch February 19, 2025 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lint-fix openhands Have Openhands attempt to fix the issue (via GH App)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: "-t TASK" doesn't work with CLI mode
4 participants