Update scratch version to 13

[abcampo-iry]

Updated Scratch to version 13 and update to react-redux 8 is needed since it's a required peer dependency.
Also for react-redux 8 we need redux as a peer dependency too, so we need to provide it ourselves.

Updating the version didn't change the HTML preview behavior directly, but it changed the timing enough to expose an existing race in the HTML runner internal-link flow.

Test change explanation:

When an internal link was clicked, the app updated previewFile and triggered a rerun in the same tick.
In CI runs, because it can be slower, the rerun could happen before React had actually applied the new previewFile, so the iframe sometimes re-rendered new.html again instead of index.html.

What changed:

• In externalLinkHelper internal-link handling now only updates previewFile. In HtmlRunner, the rerun is deferred until after previewFile has actually changed, via reloadAfterPreviewChange.
• Also same-file reloads are now handled explicitly, so if the requested preview file is already the current one, we rerun immediately instead of leaving a pending reload behind.
• In spec-html.cy, getIframeBody() doesn't use .then(cy.wrap), so Cypress keeps re-querying the live iframe body instead of holding a stale body element across iframe navigation.
• The assertion in allows internal links was simplified to getIframeBody().should("contain.text", "hello world"), which matches the user-visible behavior and avoids depending on a specific node during reload.

Now the assertion is less coupled to the exact DOM shape and retries against the current iframe document, which is what we want for navigation/reload behavior.

There was also an issue exposed in WebComponentLoader after the dependency update.
loadRemix was being kept as derived state through an effect, and after the upgrade that became unstable.
Now loadRemix is derived directly from user and remixLoadFailed, which removes the extra state sync and avoids the loader getting stuck.

In WebComponentLoader.test, the mocked changeLanguage now resolves instead of returning a never-ending promise, which matches the real behavior better and avoids the test hanging.

[Copilot]

Pull request overview

Updates the project’s Scratch/React-Redux dependency stack and adjusts HTML runner internal-link handling to avoid a race condition exposed by the Scratch v13 timing changes.

Changes:

• Bump @scratch/scratch-gui to v13 and upgrade to react-redux v8, adding redux as an explicit dependency.
• Fix HTML runner internal-link flow by deferring reruns until after previewFile state changes.
• Make Cypress HTML iframe assertions resilient to iframe navigation by re-querying the live body.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
yarn.lock	Lockfile updates reflecting Scratch v13, React-Redux v8, and new transitive dependency graph.
package.json	Updates @scratch/scratch-gui and react-redux, adds redux peer dependency.
src/utils/externalLinkHelper.js	Removes immediate rerun dispatch for internal-link handling (now rerun is coordinated by HtmlRunner).
src/components/Editor/Runners/HtmlRunner/HtmlRunner.jsx	Adds reloadAfterPreviewChange deferral to avoid internal-link race when switching preview files.
cypress/e2e/spec-html.cy.js	Updates iframe body helper and assertion to avoid stale DOM references across iframe reload/navigation.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.