Script changes made with an external editor are not live reloading

[fhd]

Godot version

3.51.stable.mono.official

System information

Ubuntu 22.04.2 LTS, GLES3, GeForce RTX 3090

Issue description

Live reloading changes made by an external editor doesn't seem to work for me, tried this on two different machines. In my understanding, since #51828 (which aims to address #10946) has been merged quite a while ago, it should.

I don't see any errors in the console or debugger, just nothing happens after I save the file with an external editor. I tried it with and without setting Use External Editor.

Update: Did some debugging (full story in the comments) and figured out that it does reload, but only if the editor window is focused after the file has been saved. I'd love a truly automatic reload though, without having to touch the editor first.

Steps to reproduce

I made a project to reproduce this, but it's pretty simple:

1. Create a new project
2. Create a new User Interface scene
3. Add a Label node
4. Attach a script to the Label node
5. Add a _process function with $Label.text = "Test"

Steps to reproduce once the project is in place:

1. Run the project
2. Change the string "Test" to something else in the Godot script editor and observe that it live reloads
3. Change the string in an external editor and observe that it does not live reload

Minimal reproduction project

external-editor-live-reload.zip

[fhd]

Also just tested this in 4.0 rc3, and it doesn't work either. That said, I'm just not 100% sure how/if it's supposed to work. I found a setting Auto Reload Scripts On External Change in both 3.5 and 4.0, but turning it off doesn't seem to help (nor would I expect it to, I guess).

[fhd]

Took some time to test this on a Windows 10 machine - same behaviour. Makes me wonder even more if this is even supposed to work.

[fhd]

Doesn't appear to be a regression either - just tested in 3.4.2, the release that first included this change, as well as my own build of 19301d4 (the commit that fixed this in the 3.x branch) - no luck.

Started to look into the code to try and figure out what's going on here - currently trying to understand all the cases in which update_modified_scripts_for_external_editor is being called, since that's what ultimately calls _trigger_live_script_reload after the PR that addressed this back then. One of these would have to be triggered by a file change on disk, I presume.

[fhd]

Alright, from reading the code, I think I figured it out! The changes are only reloaded if the editor receives focus after the changes are saved. Looks like that's the behaviour as implemented, but I would argue it would be more ideal if it wasn't necessary to change focus to the editor first.

[KoBeWi]

What's the use for automatic reload?
If the editor is not focused, you are not using it. And there is no way to save a file externally without unfocusing the editor AFAIK.

[PathogenDavid]

What's the use for automatic reload?

My use-case is that I have multiple monitors. I have VS Code open on my left monitor, the editor on my middle monitor, and the running game on the right monitor.

Since I can still see the game running ideally I'd like to see my changes reflected as soon as I save. (Which is how it works if I use the built-in script editor.)

[sztomi]

I'm seeing the same behavior on Linux in 4.0.stable (GDScript version).

[OmarShehata]

It would be great to fix this. The editor is going to be NOT focused 100% of the time when you are editing code with an external editor, which you need to be doing if you want to see your code side by side with the scene editor.

[goblinJoel]

On Godot 3.5.3, Windows 10:
I'd like to note that, on my own project, I don't get live reloading of scripts for a running game even when I switch focus from VS Code back to the editor after saving the change. It does reload tool scripts within the editor immediately though, no focus switch necessary.

It works fine if I edit the script from within Godot -- changes apply in a running game immediately.

I've tried with and without Editor > Text Editor > External checked.

[OmarShehata]

The fix was implemented here #82986. It does seem to work in v4.2.1.stable.official [b09f793f5]. I thought I saw it was backported to some 3.x version but I can't find that now.