Skip to content

Conversation

@yotam-frid
Copy link

@yotam-frid yotam-frid commented Aug 27, 2025

In Godot 4.5, the scene:inspect_object remote debugger command was replaced with a multi-object inspect scene:inspect_objects command.

The extension currently works with the v4.4 schema but hangs indefinitely when inspecting objects in v4.5 projects.

This fixes operations like hover-inspecting variables, or seeing an object's members.

@DaelonSuzuka
Copy link
Collaborator

What version specifically was this change made? I feel like this is a recent change and was not present in 4.0.

@yotam-frid
Copy link
Author

yotam-frid commented Aug 27, 2025

What version specifically was this change made? I feel like this is a recent change and was not present in 4.0.

You're right, it was only merged 5 months ago and was added to the 4.5dev1 upstream:
godotengine/godot@5c66129

The proposal's been around since last year. Guess it was missed it was a breaking change.

Looks like it's going to be part of version 4.5. So it would be wiser to either add a backwards-compatible implementation, or re-add the old command in the godotengine repo

@yotam-frid yotam-frid changed the title Debugger: fix inspecting objects/variables in Godot v4+ Debugger: fix inspecting objects/variables in Godot v4.5 Aug 28, 2025
@yotam-frid
Copy link
Author

I created a backwards compatibility layer fix - godotengine/godot#110043
If merged, PR can be closed. Otherwise, a forking logic should be added.

@DaelonSuzuka
Copy link
Collaborator

There's no reason to create an issue on the engine, they can change the APIs if they want to. However, this extension explicitly supports all Godot versions after maybe 3.2, so this PR isn't usable as-is.

Detecting the connected version and forking the internal logic is the correct solution.

@yotam-frid
Copy link
Author

There's no reason to create an issue on the engine, they can change the APIs if they want to. However, this extension explicitly supports all Godot versions after maybe 3.2, so this PR isn't usable as-is.

Detecting the connected version and forking the internal logic is the correct solution.

OK - I did just that 🚀

@Calinou Calinou added the bug label Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants