Update runtime and scratch desktop

[dylanmccall]

In addition to updating scratch-desktop and its ilk, along with the runtime (which all happened together due to requiring a newer version of NodeJS), I ended up doing a few other things while I was here:

• I cleaned up a few details in edu.mit.Scratch.yaml, removing some old comments and unused code.
• The newer version of Electron means we can drop the filesystem=home permission.
• I noticed Turbowarp was handling *.sb3 files from Scratch, so I added the same kind of support to the Scratch app. The mime type is a bit odd (application/x.scratch.sb3 as opposed to application/x-scratch-sb3), but that appears to be the format Scratch upstream uses, as well as Turbowarp.

I ran into one strange issue where the URL for the scratch-gui git repo in generated-sources.json turned out to be a git+ssh:// style URL, which Flathub wasn't impressed with. I'd love to explore why that happened, but for now, I manually changed it to an https:// URL.

Also, note that I'm only updating to org.freedesktop.Platform//22.08 for now. I'd like to bump all the way to 23.08 soon, but this was already tricky to line up (due to also needing a different version of NodeJS), so I think it's easier doing that separately.

I'm happy to split this all into different pull requests if you prefer.

[flathubbot]

Started test build 82833

[flathubbot]

Build 82833 failed

[dylanmccall]

There's one last (I think) thing to do here, which is scratch-gui added a prepublish script. A prepublish script is yet another diabolical way a nodejs project can ruin the day of anyone who cares about reproducible builds. This one runs this glob of code: https://github.com/scratchfoundation/scratch-gui/blob/develop/scripts/prepublish.mjs#L113-L126. I was hoping to just ignore it and not download this file and call it a day, //but// it breaks the build, so i need to add some extra handling to the project's Python script that generates "asset-sources.json".

[flathubbot]

Started test build 82966

[flathubbot]

Build 82966 failed

[flathubbot]

Started test build 82970

[flathubbot]

Build 82970 failed

[flathubbot]

Started test build 83019

[flathubbot]

Build 83019 failed

[flathubbot]

Started test build 83021

[flathubbot]

Build 83021 failed

[flathubbot]

Started test build 83023

[flathubbot]

Build 83023 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/65657/edu.mit.Scratch.flatpakref

[flathubbot]

Started test build 83134

[flathubbot]

Build 83134 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/65769/edu.mit.Scratch.flatpakref

[refi64]

Thanks!

I ran into one strange issue where the URL for the scratch-gui git repo in generated-sources.json turned out to be a git+ssh:// style URL, which Flathub wasn't impressed with. I'd love to explore why that happened, but for now, I manually changed it to an https:// URL.

Might be worth mentioning in the readme if that needs to be redone when flatpak-node-generator gets run?

I was hoping to just ignore it and not download this file and call it a day, //but// it breaks the build, so i need to add some extra handling to the project's Python script that generates "asset-sources.json".

I don't see any changes there other than the URL update, was this no longer necessary or did they get accidentally left out of the commits?

[dylanmccall]

Might be worth mentioning in the readme if that needs to be redone when flatpak-node-generator gets run?

Yeah, I can do that :) Maybe I'll add a note to run flatpak-builder-lint, since there's a lot happening here. The problem I ran into was probably this one you were working on: flatpak/flatpak-builder-tools#381. The lockfile is indeed lockfile version 2 now. So that's, err, concerning, but also nice it's only the one thing that breaks here 🤔

I don't see any changes there other than the URL update, was this no longer necessary or did they get accidentally left out of the commits?

Yep, I ended up squashing that change into c5126b6. Mostly this part: c5126b6#diff-1660a6484fcf48528c7f07a34e072c0405e15616909a7ddf1c8df4dfd9189273R99-R107, and the addition of npm_config_ignore_scripts: 'true'.

[flathubbot]

Started test build 83168

[flathubbot]

Build 83168 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/65803/edu.mit.Scratch.flatpakref

[wjt]

bot, build

[flathubbot]

Queued test build for edu.mit.Scratch.

[flathubbot]

Started test build 83853

[flathubbot]

Build 83853 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/66494/edu.mit.Scratch.flatpakref

[wjt]

I'm just a person but this looks good to me.

[wjt]

Interesting, today I learned how mime types are exported by Flatpak apps.

[wjt]

I'm sorry to report that this extension doesn't work, but that's not a regression.