Robert Brennan
Robert Brennan
This is very cool! I'm a little worried about changing the footprint of `execute`, which hypothetically should be the same for every box. But I don't think it affects much...
This is looking good to me! I'm mostly worried about the complicated logic for parsing the SSH stream--nothing we can leverage from pexpect or another SSH lib?
Yes--the terminal isn't yet interactive. It's something we could add though!
See also: https://github.com/OpenDevin/OpenDevin/pull/1651
@imkebe can you try running with tag `1651-merge`? I just incorporated some of your changes here into that PR
Hey all--I think all these issues (including the bashrc timeout) have been solved at this point Thanks @imkebe for the inspiration here--it was a big help in tracking down one...
Nice! I like where this is headed
@Umpire2018 yeah we definitely need to preserve the existing serialization/deserialization (or go through a painful migration across FE, BE, and agent prompts) I think the current behavior puts everything except...
Current issue is that `source ~/.bashrc` hangs when running as root user
@xingyaoww any ideas why `source ~/.bashrc` times out when running as root? I get ``` 00:45:36 - opendevin:ERROR: ssh_box.py:231 - Command timed out, killing process... 00:46:07 - opendevin:ERROR: agent.py:125 -...