[Bug]: CodeAct repeatedly edits the wrong file

[rbren]

Is there an existing issue for the same bug?

• [X] I have checked the troubleshooting document at https://opendevin.github.io/OpenDevin/modules/usage/troubleshooting
• [X] I have checked the existing issues.

Describe the bug

I'm asking CodeAct to repeatedly switch between editing two related files. Often File A is open, then CodeAct sends an edit command intended for File B. This usually happens when I issue a new instruction.

It'd be neat to show/emphasize the currently open file in CodeAct's prompt.

Side note: maybe we should just stop making open a thing, and put filenames directly into the commands? The agent has to look back over previous commands to determine current state, instead of just having a deterministic instruction it can write.

Current Version

Main, as of `0ecba83e53c6f2abfb0fdbaf134842a0298234f6`

Installation and Configuration

docker run

Model and Agent

gpt-4-turbo, CodeAct

Reproduction Steps

repeatedly prompt codeact to edit different files

Logs, Errors, Screenshots, and Additional Context

[li-boxuan]

Side note: maybe we should just stop making open a thing, and put filenames directly into the commands?

I also had the same wonder when I saw SWE agent commands. Apparently LLM often forgets that it needs to open a file before edits it.

[xingyaoww]

Yep! I think we should plan for one major iteration of the existing file operations now that we have a good way of comparing the performance.

[edwardbrazier]

I vote for including the file name as the first parameter supplied by the AI to the edit command, as suggested above by rbren.

(This mix-up happened a bunch of times on the very first day that I used OpenDevin.)

[tobitege]

I'd take this up since I'm already on the agentskills with append_file and get to the edit_file changes for this PR afterwards.

cc @rbren @xingyaoww

[li-boxuan]

I encountered something funny with this symptom recently. The agent was trying to edit a javascript file but it forgot to open it. Since the last file it opened was an HTML file, the agent simply wrote javascript code to HTML file. When it browsed the webpage, it saw those javascript code printed out - and it thought that meant the code had been working!

[tobitege]

Proposal: right now open_file serves more as a list_file (like cat) and shouldn't be required prior to edit_file?

[li-boxuan]

right now open_file serves more as a list_file (like cat) and shouldn't be required prior to edit_file?

+1

[xingyaoww]

@tobitege i've been thinking about the same thing too -- feel free to starts a PR and add "filepath" to the edit command!

[tobitege]

@tobitege i've been thinking about the same thing too -- feel free to starts a PR and add "filepath" to the edit command!

Already done, waiting for review: https://github.com/OpenDevin/OpenDevin/pull/2346