Add actions for github push and send PR #1415
Conversation
There was a problem hiding this comment.
This is great, looks like we are one step closer to dogfooding!
To fix the integration test, you could refer to https://github.com/OpenDevin/OpenDevin/blob/main/tests/integration/README.md
|
|
||
| # Push the branch to the temporary remote | ||
| command = f'git push {random_remote} {self.branch}' | ||
| push_result = controller.action_manager.run_command(command, background=False) |
There was a problem hiding this comment.
Error handling for push? A common error would be permission related.
There was a problem hiding this comment.
Right now if the command fails it will return the failed command and failure message (after deleting the remote), so I think this might be OK as-is?
There was a problem hiding this comment.
Sounds good. Rate limiting error should be rare and I don't think we need to handle that. Permission error would have to be resolved by the user anyways. #1290 aims to add support for agent-user interaction, which might be helpful for users to intervene and address permission issues, if any.
|
I think we also need to modify |
…in into neubig/add_push_action
|
@li-boxuan , I added the info to the prompt, I think this is ready for another look! |
|
I have finished end-to-end testing and confirmed that this indeed works! Here is the first PR authored by OpenDevin that uses this functionality: #1438 |
|
@enyst or @li-boxuan , if you approve this is probably ready to merge. |
|
I tried locally with GPT-3.5, it failed as expected since it didn't have yet a token. 😅 |
This PR aims to give agents the ability to push to github.
GithubPushActionandGithubSendPRAction.GithubPushActiontoMonologueAgentthrough the prompt.Both of these actions require you to set
OPENDEVIN_GITHUB_TOKENas an environment variable, and uses this token to either push to github or send an API call to create a PR.I still need to extensively test the full pipeline but this should be ready for review.
Fixes #1067