[Bug]: The file editor is displaying the wrong content.

[Shimada666]

Is there an existing issue for the same bug?

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

Describe the bug

After clicking refresh icon, the displayed file content becomes incorrect. However, upon inspecting using F12, the data returned from the backend is correct. I will debug and fix this issue later.

QQ20240519-175458-HD.mp4

Current Version

main

Installation and Configuration

npm run start

Model and Agent

No response

Reproduction Steps

No response

Logs, Errors, Screenshots, and Additional Context

No response

[Shimada666]

How to reproduce:

1. Create files named aaa.sh and tmp/bbb.sh.
2. First, view the aaa.sh file, then view the tmp/bbb.sh file, and finally re-visit the aaa.sh file. You will notice that the content is incorrect.

I can confirm it's a redux issue because when I store the code using useState, the bug fixed. Later, I will see if I can refactor it.

[SmartManoj]

Working fine here. That sh file should be inside the tree. How it replaced the workspace title?

[Shimada666]

@SmartManoj
You need to create aaa.sh and tmp/bbb.sh. Please note that bbb.sh is inside the tmp folder, and then you can reproduce it. I provide a video.

QQ20240519-215938-HD.mp4

[PierrunoYT]

I had some similar issues

[SmartManoj]

@Shimada666 Yes, affecting in latest versions even without nested dirs.
If two files are there a and b,
The first time clicking a -> works
clicking b -> works
then clicking again doesn't work

[SmartManoj]

This PR #1853 introduced this?
Workaround: View files in IDE

[amanape]

It may be related to how the active file path is being updated. I can't do it today, so maybe you can take a look somewhere here?

OpenDevin/frontend/src/components/file-explorer/TreeNode.tsx

Lines 45 to 69 in 3e3dcd5

	const filename =
	fileParts[fileParts.length - 1] || fileParts[fileParts.length - 2];
	const isDirectory = path.endsWith("/");
	const refreshChildren = async () => {
	if (!isDirectory || !isOpen) {
	setChildren(null);
	return;
	}
	const files = await listFiles(path);
	setChildren(files);
	};
	React.useEffect(() => {
	refreshChildren();
	}, [refreshID, isOpen]);
	const handleClick = () => {
	if (isDirectory) {
	setIsOpen((prev) => !prev);
	} else {
	dispatch(setActiveFilepath(path));
	}
	};

[Shimada666]

@SmartManoj @amanape The latest the code does not seem to have this issue, I will close this issue.
Thank you all!