Robert Brennan
Robert Brennan
I think the best solution here is to give a good error message back to the LLM--something like "You must only edit files inside the workspace"
@xingyaoww I vaguely remember you doing something like this already @coffeecupjapan we should at least extract out the magic constant, so we can tune this a bit
See also: https://github.com/OpenDevin/OpenDevin/pull/1421
@coffeecupjapan I'm going to close this one, but feel free to open up an issue where we can discuss a more long-term solution!
Hmm thanks for the report @isavita. This looks like the issue described here: https://opendevin.github.io/OpenDevin/modules/usage/troubleshooting#unable-to-connect-to-llm Are you sure you're using the exact same environment variables when you run `main` vs my...
Thanks all! @enyst it looks like your issue was a bad LLM response (it set `action: user_message`), so probably unrelated. But it does look like it broke the loop instead...
Turns out the bad action was being sent by the UI--maybe a merge issue? It sent me down a huge rabbit hole of fixes for the interactive "waiting for user...
> Integration tests for sandboxes were executing with almost all config vars set to None My guess is this was true when running locally, but they were set in the...
Looks like this is good to go! Just need to fix the conflicts
I think having defaults for the paths makes sense. And yeah--integration is probably the right word for those 😄