Flame 1.21.0 removes Components from World by itself

[dvmatyun]

What happened?

1. Updated from Flame 1.18.0 to 1.21.0 (so issue was not present on 1.18.0)
2. During game session, flame randomly can remove my components (I have no clue why that happens on 1.21 and not on 1.18)

What do you expect?

I expect to know the reason why the component is removed. Both removeFromParent or onRemove are NOT called on removed components. So they just dissapear from World children set. Maybe Flame introduced some sort of garbage collector?

How can we reproduce this?

To be honest, I have no idea, maybe you can give me a clue why that can be happening.

What steps should take to fix this?

Make World's children set immutable and all add or remove operations can be logged if debug is true?

Do have an example of where the bug occurs?

no

Relevant log output

No response

Execute in a terminal and put output into the code block below

fvm flutter doctor -v
[√] Flutter (Channel stable, 3.24.3, on Microsoft Windows [Version 10.0.22631.4317], locale ru-RU)
• Flutter version 3.24.3 on channel stable at C:\Users\dima\fvm\versions\3.24.3
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 2663184aa7 (7 weeks ago), 2024-09-11 16:27:48 -0500
• Engine revision 36335019a8
• Dart version 3.5.3
• DevTools version 2.37.3

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
• Android SDK at C:\sdk\Android\android-sdk
• Platform android-34, build-tools 34.0.0
• ANDROID_HOME = C:\sdk\Android\android-sdk
• Java binary at: C:\sdk\AndroidStudio\jbr\bin\java
• Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)
• All Android licenses accepted.

[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.7.4)
• Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
• Visual Studio Community 2022 version 17.7.34031.279
• Windows 10 SDK version 10.0.19041.0

[!] Android Studio (version 2022.3)
• Android Studio at D:\Programs\Android
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
X Unable to determine bundled Java version.
• Try updating or re-installing Android Studio.

[√] Android Studio (version 2023.1)
• Android Studio at C:\sdk\AndroidStudio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)

[√] VS Code, 64-bit edition (version 1.94.2)
• VS Code at C:\Program Files\Microsoft VS Code
• Flutter extension version 3.100.0

[√] Connected device (4 available)
• 21081111RG (mobile) • 5hmjbehqvsgqvseq • android-arm64 • Android 13 (API 33)
• Windows (desktop) • windows • windows-x64 • Microsoft Windows [Version 10.0.22631.4317]
• Chrome (web) • chrome • web-javascript • Google Chrome 130.0.6723.92
• Edge (web) • edge • web-javascript • Microsoft Edge 127.0.2651.86

[√] Network resources
• All expected network resources are available.

Affected platforms

Android, Windows

Other information

No response

Are you interested in working on a PR for this?

• I want to work on this

[dvmatyun]

Im ready to fix this if you can give me a clue who and why can be modifing World's children set without even calling onRemove or removeFromParent. (its definitely not my code)

[spydon]

Please provide a way to reproduce this, otherwise it's hard for us to investigate it.

[spydon]

If this is reproducible, it is most likely related to this change: bluefireteam/ordered_set#46

[dvmatyun]

@spydon thanks for the hint, I will try to test it and come back with the result

[dvmatyun]

@spydon so the issue is fixed if Im using old ordered set. Steps:

1. updated flame to flame: 1.21.0
2. forked ordered_set 5.0.3
3. Change signature to Iterable<C> query<C extends T>() (otherwise wont compile, this methods is overridden by flame with exactly this signature)
   Everything works. So I guess the bug is either in ordered_set, or Im using it wrong (to be honest Im using forked bonfire package, but as Ive seen nothing wrong there and I guess the bug happens during reordering the world)

[spydon]

@dvmatyun try to reproduce this for just Flame, because @RafaelBarbosatec reported something similar for Bonfire, but nobody has reported anything similar for pure Flame.