Face sensing can get stuck in the project (also the translate extension)

[nimeratus]

Expected Behavior

When all face sensing blocks and monitors have been removed, the face sensing extension is deleted from the project after reloading.

Actual Behavior

The project still loads the extension and turns on the camera. If the project also has cloud variables, they are disabled, but the warning text isn't shown on the project page.
https://scratch.mit.edu/projects/1225257855/

Steps to Reproduce

1. Go to 'https://scratch.mit.edu/projects/editor'
2. Add Face Sensing
3. Show the "face tilt" monitor
4. Move it slightly
5. Hide the monitor
6. Save and reload

It also works with the translate extension and the "language" reporter, but the face sensing has much more impact when it's stuck.

I think the bug happens because the monitor stays in the JSON so that Scratch can remember its position. This makes Scratch load the extension, even though the monitor is hidden.

System Details

I've tested it on Windows 10, Chrome 141

[cwillisf]

Nice catch! I think you're exactly right about why the problem happens. I noticed that you don't even have to move the monitor; just turning it on is enough for the hidden reporter to get "stuck" in the JSON.

[apple502j]

scratchfoundation/scratch-vm#2963, btw :)

(Also, one thing I noticed: Face Sensing l10n strings do not appear to be in the l10n repo - are they tracked elsewhere now?)

[apple502j]

@cwillisf I believe that patch still applies cleanly - should I file a PR to this repo?