OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

[Feature]: Teach OpenDevin agents their name and email address

Open neubig opened this issue 1 year ago • 7 comments

What problem or use case are you trying to solve?

We recently added the ability to push to github: #1415

However, when doing so, our agents don't know their name and email address, so we should probably prompt them with a name and email address to use for PRs.

Do you have thoughts on the technical implementation?

This could be achieved by adding name and email to the config and defaults , with defaults like OpenDevin and some email address we create.

This could then be referenced in the prompts provided to the agents, starting with MonologueAgent.

neubig avatar Apr 29 '24 00:04 neubig

Hey, I would like to work on this!

ADTmux avatar Apr 30 '24 09:04 ADTmux

Great @ADTmux , please take a look!

neubig avatar Apr 30 '24 12:04 neubig

Fixed by https://github.com/OpenDevin/OpenDevin/pull/1842

neubig avatar May 16 '24 21:05 neubig

Hmm, it seems like this is still not fixed...

I tried to do #1904 with OpenDevin, and it still doesn't have the appropriate username and email.

neubig avatar May 19 '24 14:05 neubig

@neubig weird... Did the agent actually sets the git config by itself though?

xingyaoww avatar May 19 '24 14:05 xingyaoww

Yes, because the git config is not set, the agent does it by itself. But this results in unpredictable behavior, where the name is different every time.

neubig avatar May 19 '24 14:05 neubig

@neubig I actually wonder what's the sandbox version you are using? I just tried with ghcr.io/opendevin/sandbox:main and it seems to work well for me: https://github.com/OpenDevin/OpenDevin/pull/1928

xingyaoww avatar May 20 '24 16:05 xingyaoww

@neubig Any chance you can try this with main and report back?

mamoodi avatar Jun 08 '24 17:06 mamoodi

Recently I haven't noticed any issues with this, so I think it might be fixed?

neubig avatar Jun 08 '24 17:06 neubig