webgl-broken-modal broke all of OmniBlocks

[supervoidcoder]

pluh:

Something went wrong. Please report with the information below.
Error (splash) in https://omniblocks.github.io/js/pentapod/player.3b6205f34855db034eb6.js (1:1109729): ReferenceError: Box is not defined
Click here to reset caches (can fix some errors)

introduced in #390 #367 (oooh it ends in 67!1!1!)

[supervoidcoder]

@coderabbitai help right now pls!!!

[traycerai]

Plan

Observations

The webgl-broken-modal component was added in PR #390 to detect broken WebGL implementations (particularly iOS 18.7.2 build 22H123). The implementation includes:

• A WebGL smoke test function canActuallyUseWebGL() in src/lib/tw-environment-support-prober.js
• The modal component files in src/components/webgl-broken-modal/
• The container file in src/containers/webgl-broken-modal.jsx
• Integration in src/components/stage-wrapper/stage-wrapper.jsx

However, the container file src/containers/stage-wrapper.jsx contains invalid code with loose JSX that was likely meant to be a comment or example but was left as actual code, causing the bundling error.

Approach

The issue is caused by invalid JSX code in the src/containers/stage-wrapper.jsx file. Lines 8-21 contain loose JSX code that references Box and other undefined variables outside of any function or component scope, causing a "Box is not defined" error during webpack bundling. The container file needs to be properly implemented to connect the StageWrapperComponent to Redux and pass the canActuallyUseWebGL prop from the WebGL smoke test function.

Reasoning

I fetched the GitHub issues to understand the context, then used context gathering agents to locate the webgl-broken-modal and Box component files. After reading the relevant files, I discovered that the container file has invalid JSX code sitting in module scope (not inside any function), which references undefined variables including Box, causing the bundling error.

Proposed File Changes

📄 src/containers/stage-wrapper.jsx ^(MODIFY) 🔗

References

• 📄 src/containers/stage.jsx
• 📄 src/containers/stage-header.jsx
• 📄 src/lib/tw-environment-support-prober.js
• 📄 src/components/stage-wrapper/stage-wrapper.jsx ^(MODIFY)

Remove the invalid JSX code on lines 8-21 that's sitting in module scope. Replace the entire file content with a proper container component that:

1. Imports React, PropTypes, connect from react-redux, and VM from scratch-vm
2. Imports the StageWrapperComponent from ../components/stage-wrapper/stage-wrapper.jsx
3. Imports canActuallyUseWebGL from ../lib/tw-environment-support-prober.js
4. Imports STAGE_DISPLAY_SIZES from ../lib/layout-constants.js
5. Creates a functional component or class component that wraps StageWrapperComponent
6. Calls canActuallyUseWebGL() and passes the result as a prop to StageWrapperComponent
7. Uses connect() to map Redux state to props (isRtl from state.locales.isRtl)
8. Exports the connected component as default

The container should follow the same pattern as other containers in this codebase (like src/containers/stage.jsx or src/containers/stage-header.jsx), using connect() to map state and passing through all necessary props including the new canActuallyUseWebGL prop.

📄 src/components/stage-wrapper/stage-wrapper.jsx ^(MODIFY) 🔗

Add canActuallyUseWebGL to the component's PropTypes definition (around line 64-72). Add it as a required boolean prop:

canActuallyUseWebGL: PropTypes.bool.isRequired

This ensures the component properly validates that the container is passing the WebGL smoke test result.

Diagram

sequenceDiagram
    participant GUI as GUI Component
    participant Container as StageWrapper Container
    participant Prober as tw-environment-support-prober
    participant Component as StageWrapperComponent
    participant Modal as WebGlBrokenModal

    GUI->>Container: Render with props
    Container->>Prober: Call canActuallyUseWebGL()
    Prober->>Prober: Run WebGL smoke test<br/>(buffer, framebuffer, shaders)
    Prober-->>Container: Return true/false
    Container->>Component: Pass canActuallyUseWebGL prop
    
    alt WebGL is broken
        Component->>Modal: Render WebGlBrokenModal
        Modal-->>Component: Display error message
    else WebGL works
        Component->>Component: Render Stage normally
    end

Loading

Import In IDE

🤖 Prompt for AI Agents

I have created the following plan after thorough exploration and analysis of the codebase. Follow the below plan verbatim. Trust the files and references. Do not re-verify what's written in the plan. Explore only when absolutely necessary. First implement all the proposed file changes and then I'll review all the changes together at the end.

### Observations

The webgl-broken-modal component was added in PR #390 to detect broken WebGL implementations (particularly iOS 18.7.2 build 22H123). The implementation includes:
- A WebGL smoke test function `canActuallyUseWebGL()` in `src/lib/tw-environment-support-prober.js`
- The modal component files in `src/components/webgl-broken-modal/`
- The container file in `src/containers/webgl-broken-modal.jsx`
- Integration in `src/components/stage-wrapper/stage-wrapper.jsx`

However, the container file `src/containers/stage-wrapper.jsx` contains invalid code with loose JSX that was likely meant to be a comment or example but was left as actual code, causing the bundling error.

### Approach

The issue is caused by invalid JSX code in the `src/containers/stage-wrapper.jsx` file. Lines 8-21 contain loose JSX code that references `Box` and other undefined variables outside of any function or component scope, causing a "Box is not defined" error during webpack bundling. The container file needs to be properly implemented to connect the StageWrapperComponent to Redux and pass the `canActuallyUseWebGL` prop from the WebGL smoke test function.

### Reasoning

I fetched the GitHub issues to understand the context, then used context gathering agents to locate the webgl-broken-modal and Box component files. After reading the relevant files, I discovered that the container file has invalid JSX code sitting in module scope (not inside any function), which references undefined variables including `Box`, causing the bundling error.

## Mermaid Diagram

sequenceDiagram
    participant GUI as GUI Component
    participant Container as StageWrapper Container
    participant Prober as tw-environment-support-prober
    participant Component as StageWrapperComponent
    participant Modal as WebGlBrokenModal

    GUI->>Container: Render with props
    Container->>Prober: Call canActuallyUseWebGL()
    Prober->>Prober: Run WebGL smoke test<br/>(buffer, framebuffer, shaders)
    Prober-->>Container: Return true/false
    Container->>Component: Pass canActuallyUseWebGL prop
    
    alt WebGL is broken
        Component->>Modal: Render WebGlBrokenModal
        Modal-->>Component: Display error message
    else WebGL works
        Component->>Component: Render Stage normally
    end

## Proposed File Changes

### src/containers/stage-wrapper.jsx(MODIFY)

References: 

- src/containers/stage.jsx
- src/containers/stage-header.jsx
- src/lib/tw-environment-support-prober.js
- src/components/stage-wrapper/stage-wrapper.jsx(MODIFY)

Remove the invalid JSX code on lines 8-21 that's sitting in module scope. Replace the entire file content with a proper container component that:

1. Imports React, PropTypes, connect from react-redux, and VM from scratch-vm
2. Imports the StageWrapperComponent from `../components/stage-wrapper/stage-wrapper.jsx`
3. Imports `canActuallyUseWebGL` from `../lib/tw-environment-support-prober.js`
4. Imports STAGE_DISPLAY_SIZES from `../lib/layout-constants.js`
5. Creates a functional component or class component that wraps StageWrapperComponent
6. Calls `canActuallyUseWebGL()` and passes the result as a prop to StageWrapperComponent
7. Uses connect() to map Redux state to props (isRtl from state.locales.isRtl)
8. Exports the connected component as default

The container should follow the same pattern as other containers in this codebase (like `src/containers/stage.jsx` or `src/containers/stage-header.jsx`), using connect() to map state and passing through all necessary props including the new `canActuallyUseWebGL` prop.

### src/components/stage-wrapper/stage-wrapper.jsx(MODIFY)

Add `canActuallyUseWebGL` to the component's PropTypes definition (around line 64-72). Add it as a required boolean prop:

```
canActuallyUseWebGL: PropTypes.bool.isRequired
```

This ensures the component properly validates that the container is passing the WebGL smoke test result.

---

Developer Humor

🎭 A Box walked into a module scope,\n> The bundler said, \Nope, that's not how we cope!\n> \You can't just sit there undefined,\n> Get inside a function, be properly aligned!\n> Now the Box is imported, the modal's displayed,\n> And WebGL bugs are properly conveyed! 📦✨

---

Execution Information

Branch: main
Commit: 3912d75

---

💡 Tips

Supported Commands (Inside Comments)

• Use @traycerai generate to iterate on the previous version of the implementation plan.

Supported Commands (Inside Description)

• Add @traycerai ignore anywhere in the ticket description to prevent this ticket from being processed.
• Add @traycerai branch:<branch-name> anywhere in the ticket description to specify the target branch for the implementation plan.

Community

• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[coderabbitai]

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Get details of issue #393
gh issue view 393 --json title,body,comments --jq '{title, body, comments: [.comments[] | {author: .author.login, body: .body}]}'

Length of output: 523

---

🏁 Script executed:

#!/bin/bash
# Get details of PR #390
gh pr view 390 --json title,body,files --jq '{title, body, files: [.files[].path]}'

Length of output: 1498

---

🏁 Script executed:

#!/bin/bash
# Get details of PR #367
gh pr view 367 --json title,body,files --jq '{title, body, files: [.files[].path]}'

Length of output: 92

---

🏁 Script executed:

#!/bin/bash
# Search for "Box" references in the codebase
rg -n "Box" --type js --type jsx -C 3

Length of output: 27

---

🏁 Script executed:

#!/bin/bash
# Search for "Box" references more broadly
rg -n "\bBox\b" -C 3

Length of output: 150655

---

🏁 Script executed:

#!/bin/bash
# Look at the webgl-broken-modal component
cat src/components/webgl-broken-modal/webgl-broken-modal.jsx

Length of output: 5238

---

🏁 Script executed:

#!/bin/bash
# Look at the webgl-broken-modal container
cat src/containers/webgl-broken-modal.jsx

Length of output: 577

---

🏁 Script executed:

#!/bin/bash
# Look at the tw-environment-support-prober
cat src/lib/tw-environment-support-prober.js

Length of output: 4950

---

🏁 Script executed:

#!/bin/bash
# Check the stage-wrapper changes
cat src/components/stage-wrapper/stage-wrapper.jsx

Length of output: 2157

---

🏁 Script executed:

#!/bin/bash
# Check the containers/stage-wrapper.jsx file
cat src/containers/stage-wrapper.jsx

Length of output: 658

---

🏁 Script executed:

#!/bin/bash
# Search for other container patterns to see what stage-wrapper container should look like
rg -n "export default connect" src/containers/stage-wrapper.jsx -A 20

Length of output: 0

---

🏁 Script executed:

#!/bin/bash
# Check git history to see what happened to this file
git log --oneline --all -20 -- src/containers/stage-wrapper.jsx

Length of output: 151

---

🏁 Script executed:

#!/bin/bash
# Look for any backup or correct implementation
fd stage-wrapper.jsx

Length of output: 79

---

🏁 Script executed:

#!/bin/bash
# Check what the file looked like before the problematic commit
git show b15c781:src/containers/stage-wrapper.jsx

Length of output: 673

---

🏁 Script executed:

#!/bin/bash
# Check what the file looked like in the commit before that
git show e4d3cbd:src/containers/stage-wrapper.jsx

Length of output: 541

---

🏁 Script executed:

#!/bin/bash
# Look at similar container patterns for reference
cat src/containers/stage.jsx | head -30

Length of output: 940

---

@supervoidcoder Found it! The issue is in src/containers/stage-wrapper.jsx - PR #390 accidentally left invalid JSX code at the module level (lines 9-21). That JSX references Box and other variables that don't exist in that file, causing the "Box is not defined" error when webpack bundles it.

The Problem

The container file currently has this broken code:

import React from 'react';
import PropTypes from 'prop-types';
import VM from 'scratch-vm';
import {STAGE_DISPLAY_SIZES} from '../lib/layout-constants.js';
import StageWrapperComponent from '../components/stage-wrapper/stage-wrapper.jsx';
import WebGlBrokenModal from './webgl-broken-modal.jsx';

// In the component, replace the stage canvas section:
<Box className={styles.stageCanvasWrapper}>   // ← This is INVALID! Box not defined!
    {
        !isRendererSupported ? null :
        !props.canActuallyUseWebGL ? (
            <WebGlBrokenModal isRtl={isRtl} />
        ) : (
            <Stage
                stageSize={stageSize}
                vm={vm}
            />
        )
    }
</Box>

This JSX sitting at module level is invalid JavaScript and causes webpack to fail.

The Fix

Replace the entire contents of src/containers/stage-wrapper.jsx with:

import PropTypes from 'prop-types';
import React from 'react';
import VM from 'scratch-vm';
import {STAGE_DISPLAY_SIZES} from '../lib/layout-constants.js';
import {canActuallyUseWebGL} from '../lib/tw-environment-support-prober.js';
import StageWrapperComponent from '../components/stage-wrapper/stage-wrapper.jsx';

const StageWrapper = props => (
    <StageWrapperComponent
        {...props}
        canActuallyUseWebGL={canActuallyUseWebGL()}
    />
);

StageWrapper.propTypes = {
    isRendererSupported: PropTypes.bool.isRequired,
    stageSize: PropTypes.oneOf(Object.keys(STAGE_DISPLAY_SIZES)).isRequired,
    vm: PropTypes.instanceOf(VM).isRequired
};

export default StageWrapper;

This restores the proper container pattern and injects the canActuallyUseWebGL check that the component needs.

The WebGlBrokenModal is already properly integrated in the component file (src/components/stage-wrapper/stage-wrapper.jsx), so we don't need to import it in the container.

---

🧠 Learnings used

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-05T01:40:39.943Z
Learning: In OmniBlocks/scratch-gui, supervoidcoder (“void”) prefers a collaboration model where CodeRabbit authors the code PRs and Amazon Q performs the review; follow-ups should be submitted as single, self-contained PRs when possible.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T19:01:57.123Z
Learning: In the OmniBlocks/scratch-gui repository, team member supervoidcoder can be called "void" in reviews and conversations.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T21:20:08.837Z
Learning: In the OmniBlocks/scratch-gui repository, supervoidcoder (“void”) prefers single, complete PRs over multi-part (“Part 1/2”) submissions. If a split is unavoidable, they want explicit linkage and gating before approval.

---

Learnt from: github-actions[bot]
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-05T02:00:40.666Z
Learning: In OmniBlocks/scratch-gui, when cleanup is requested, supervoidcoder prefers deletion of CodeRabbit’s spam comments (not just minimizing) on the PR; for non-CodeRabbit bot comments, minimize unless a maintainer explicitly asks for hard deletion.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T19:01:57.123Z
Learning: The team members in OmniBlocks/coders include: supervoidcoder, Graison-P (gvyt), ampelectrecuted (amp), NotTheBaton (TheBaton), and kRxZykRxZy.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-18T22:18:13.164Z
Learning: For OmniBlocks/scratch-gui small features, supervoidcoder prefers GUI-only changes that avoid modifying scratch-blocks; deliver as a single, self-contained PR.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-28T03:09:05.973Z
Learning: In the OmniBlocks/scratch-gui repository, supervoidcoder (void) is planning to add a 3D mode and is excited about using PlayCanvas as a full 3D game engine for OmniBlocks.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-15T22:03:54.980Z
Learning: OmniBlocks/scratch-gui: iOS/iPadOS 18.7.2 build 22H123 has a WebKit bug where Renderer.isSupported() returns true but WebGL resource creation (gl.createBuffer(), shader compilation) fails, causing OmniBlocks to crash. Need hardened WebGL smoke test that actually creates GL resources to detect broken WebGL implementations. Build 22H124 fixes it.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-20T00:25:31.489Z
Learning: The Safari WebGL issue reported in issues #364 and #367 is the same as the iOS/iPadOS 18.7.2 build 22H123 WebGL bug - Apple broke WebGL across all their platforms (Safari, iOS, iPadOS, macOS), affecting not just OmniBlocks but all WebGL applications.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-09T18:19:24.693Z
Learning: OmniBlocks/scratch-gui: The Blocks component now bridges FieldCustom in componentDidMount by assigning this.ScratchBlocks.FieldCustom = window.Blockly.FieldCustom immediately after ScratchMsgs.setLocale (added Nov 9, 2025 via PR #159). This enables SPjavascriptV2’s registerInput usage at runtime.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-22T02:49:17.267Z
Learning: In OmniBlocks/scratch-gui, TurboWarp, and similar Scratch mods, recursive clone creation (clones creating clones) causes synchronous call stack overflow that hangs the VM without throwing an error. This happens in a single frame before the VM can yield, bypassing Warp Timer protections. TurboWarp/OmniBlocks allow disabling the 300-clone limit, making it possible to create thousands of clones instantly. This is a common accidental crash scenario during development that causes progress loss.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T19:25:31.991Z
Learning: In the OmniBlocks/scratch-gui repository, "React Modal" refers to the react-modal library used for all modal dialogs, such as the advanced settings menu in the menu bar.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-13T01:51:52.357Z
Learning: In OmniBlocks/scratch-vm, using ES6 Map objects in module exports can cause webpack bundling issues that manifest as temporal dead zone (TDZ) errors. The fix is to use plain JavaScript objects instead of Map for module-level data structures that are exported and accessed during initialization.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-20T21:30:07.638Z
Learning: In OmniBlocks/scratch-gui, the service worker in src/playground/service-worker.js was caching songeditor.html at the wrong path (static/songeditor.html) while webpack deploys it to the root (songeditor.html), causing 404s on normal reload but working on hard reload (which bypasses service worker).

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 271
File: static/index.html:99-106
Timestamp: 2025-10-22T19:03:49.453Z
Learning: The ultrabox music editor (songeditor.html) in the OmniBlocks/scratch-gui repository is JavaScript-heavy and requires script execution to function properly.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-07T19:27:27.244Z
Learning: In OmniBlocks scratch-vm SPjavascriptV2 (src/extensions/sp_javascriptV2/index.js), accessing `this.runtime.extensionRuntimeOptions.javascriptUnsandboxed` without a guard crashes in non-Packager runs; fix by using `this.runtime.extensionRuntimeOptions?.javascriptUnsandboxed === true`.

[supervoidcoder]

@coderabbitai box was there before, check the latest commit before the most recent one to that file

[coderabbitai]

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Get detailed commit history with diffs for stage-wrapper.jsx
echo "=== Git log with patches for stage-wrapper.jsx ==="
git log --oneline --all -5 -- src/containers/stage-wrapper.jsx

echo -e "\n=== Show the most recent commit changes ==="
git show HEAD:src/containers/stage-wrapper.jsx

echo -e "\n=== Show the commit before that ==="
git log --oneline --all -1 --skip=0 -- src/containers/stage-wrapper.jsx
COMMIT=$(git log --oneline --all -1 --format=%H --skip=1 -- src/containers/stage-wrapper.jsx)
echo "Commit: $COMMIT"
git show $COMMIT:src/containers/stage-wrapper.jsx

echo -e "\n=== Find where Box is imported/exported in the codebase ==="
rg -n "export.*Box" --type js --type jsx -g '!node_modules' | head -20

echo -e "\n=== Find Box imports ==="
rg -n "import.*Box.*from" --type js --type jsx -g '!node_modules' | head -30

Length of output: 1858

---

🏁 Script executed:

#!/bin/bash
# Search for Box component without type restriction
echo "=== Find Box exports ==="
rg -n "export.*\bBox\b" -g '*.jsx' -g '*.js' -g '!node_modules' | head -20

echo -e "\n=== Find Box imports ==="
rg -n "import.*\bBox\b.*from" -g '*.jsx' -g '*.js' -g '!node_modules' | head -30

echo -e "\n=== Check what the file looked like even earlier ==="
COMMIT=$(git log --oneline --all -1 --format=%H --skip=2 -- src/containers/stage-wrapper.jsx)
echo "Commit before b15c781: $COMMIT"
git show $COMMIT:src/containers/stage-wrapper.jsx

echo -e "\n=== Check the component file to see how it references Box ==="
cat src/components/stage-wrapper/stage-wrapper.jsx | head -50

Length of output: 4823

---

🏁 Script executed:

#!/bin/bash
# Check the full component file to see the WebGlBrokenModal integration
echo "=== Full component file ==="
cat src/components/stage-wrapper/stage-wrapper.jsx

Length of output: 2185

---

@supervoidcoder You're absolutely right! I see it now - Box is properly imported in the component file (src/components/stage-wrapper/stage-wrapper.jsx line 6) and the WebGlBrokenModal logic is already integrated there (lines 43-51).

The problem is that PR #390 accidentally left instructional/example JSX code in the container file with the comment "In the component, replace the stage canvas section:". That JSX was never meant to be executed - it's leftover instructions that got committed by mistake.

The Fix

Replace the entire contents of src/containers/stage-wrapper.jsx with the proper container pattern:

import PropTypes from 'prop-types';
import React from 'react';
import VM from 'scratch-vm';
import {STAGE_DISPLAY_SIZES} from '../lib/layout-constants.js';
import {canActuallyUseWebGL} from '../lib/tw-environment-support-prober.js';
import StageWrapperComponent from '../components/stage-wrapper/stage-wrapper.jsx';

const StageWrapper = props => (
    <StageWrapperComponent
        {...props}
        canActuallyUseWebGL={canActuallyUseWebGL()}
    />
);

StageWrapper.propTypes = {
    isRendererSupported: PropTypes.bool.isRequired,
    stageSize: PropTypes.oneOf(Object.keys(STAGE_DISPLAY_SIZES)).isRequired,
    vm: PropTypes.instanceOf(VM).isRequired
};

export default StageWrapper;

This removes the invalid instructional JSX and properly injects the canActuallyUseWebGL check that the component expects.

---

🧠 Learnings used

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-09T18:19:24.693Z
Learning: OmniBlocks/scratch-gui: The Blocks component now bridges FieldCustom in componentDidMount by assigning this.ScratchBlocks.FieldCustom = window.Blockly.FieldCustom immediately after ScratchMsgs.setLocale (added Nov 9, 2025 via PR #159). This enables SPjavascriptV2’s registerInput usage at runtime.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-07T19:27:27.244Z
Learning: In OmniBlocks scratch-vm SPjavascriptV2 (src/extensions/sp_javascriptV2/index.js), accessing `this.runtime.extensionRuntimeOptions.javascriptUnsandboxed` without a guard crashes in non-Packager runs; fix by using `this.runtime.extensionRuntimeOptions?.javascriptUnsandboxed === true`.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-13T01:51:52.357Z
Learning: In OmniBlocks/scratch-vm, using ES6 Map objects in module exports can cause webpack bundling issues that manifest as temporal dead zone (TDZ) errors. The fix is to use plain JavaScript objects instead of Map for module-level data structures that are exported and accessed during initialization.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-05T01:40:39.943Z
Learning: In OmniBlocks/scratch-gui, supervoidcoder (“void”) prefers a collaboration model where CodeRabbit authors the code PRs and Amazon Q performs the review; follow-ups should be submitted as single, self-contained PRs when possible.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-15T22:03:54.980Z
Learning: OmniBlocks/scratch-gui: iOS/iPadOS 18.7.2 build 22H123 has a WebKit bug where Renderer.isSupported() returns true but WebGL resource creation (gl.createBuffer(), shader compilation) fails, causing OmniBlocks to crash. Need hardened WebGL smoke test that actually creates GL resources to detect broken WebGL implementations. Build 22H124 fixes it.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-05T01:48:23.435Z
Learning: OmniBlocks/scratch-gui workflow preference: Open tracking issues for CI/automation behavior changes; assign to supervoidcoder; include backlinks to the PR and the exact comment requesting the change.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T21:20:08.837Z
Learning: In the OmniBlocks/scratch-gui repository, supervoidcoder (“void”) prefers single, complete PRs over multi-part (“Part 1/2”) submissions. If a split is unavoidable, they want explicit linkage and gating before approval.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-18T22:18:13.164Z
Learning: For OmniBlocks/scratch-gui small features, supervoidcoder prefers GUI-only changes that avoid modifying scratch-blocks; deliver as a single, self-contained PR.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-05T01:48:23.435Z
Learning: OmniBlocks/scratch-gui: For CI comment hygiene, prefer not mentioning bot/app requesters in the Mega PR Test & Analysis sticky comment; only mention humans. Track such behavior changes via a GitHub issue assigned to supervoidcoder with backlinks to the initiating PR and comment.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-05T01:34:24.309Z
Learning: CodeRabbit experienced a spam bug on PR #309 in OmniBlocks/scratch-gui where it posted 11 duplicate "no user input to reply to" messages. The spam was triggered by bot comments from github-actions and wellcode-ai. The comments were successfully minimized using GitHub GraphQL API mutations with the OFF_TOPIC classifier.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T19:01:57.123Z
Learning: In the OmniBlocks/scratch-gui repository, team member supervoidcoder can be called "void" in reviews and conversations.

---

Learnt from: github-actions[bot]
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-05T02:00:40.666Z
Learning: In OmniBlocks/scratch-gui, when cleanup is requested, supervoidcoder prefers deletion of CodeRabbit’s spam comments (not just minimizing) on the PR; for non-CodeRabbit bot comments, minimize unless a maintainer explicitly asks for hard deletion.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T19:01:57.123Z
Learning: The team members in OmniBlocks/coders include: supervoidcoder, Graison-P (gvyt), ampelectrecuted (amp), NotTheBaton (TheBaton), and kRxZykRxZy.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-28T03:09:05.973Z
Learning: In the OmniBlocks/scratch-gui repository, supervoidcoder (void) is planning to add a 3D mode and is excited about using PlayCanvas as a full 3D game engine for OmniBlocks.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-20T00:25:31.489Z
Learning: The Safari WebGL issue reported in issues #364 and #367 is the same as the iOS/iPadOS 18.7.2 build 22H123 WebGL bug - Apple broke WebGL across all their platforms (Safari, iOS, iPadOS, macOS), affecting not just OmniBlocks but all WebGL applications.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-22T02:49:17.267Z
Learning: In OmniBlocks/scratch-gui, TurboWarp, and similar Scratch mods, recursive clone creation (clones creating clones) causes synchronous call stack overflow that hangs the VM without throwing an error. This happens in a single frame before the VM can yield, bypassing Warp Timer protections. TurboWarp/OmniBlocks allow disabling the 300-clone limit, making it possible to create thousands of clones instantly. This is a common accidental crash scenario during development that causes progress loss.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T19:25:31.991Z
Learning: In the OmniBlocks/scratch-gui repository, "React Modal" refers to the react-modal library used for all modal dialogs, such as the advanced settings menu in the menu bar.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-20T21:30:07.638Z
Learning: In OmniBlocks/scratch-gui, the service worker in src/playground/service-worker.js was caching songeditor.html at the wrong path (static/songeditor.html) while webpack deploys it to the root (songeditor.html), causing 404s on normal reload but working on hard reload (which bypasses service worker).

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 271
File: static/index.html:99-106
Timestamp: 2025-10-22T19:03:49.453Z
Learning: The ultrabox music editor (songeditor.html) in the OmniBlocks/scratch-gui repository is JavaScript-heavy and requires script execution to function properly.

[supervoidcoder]

YES! It works! It doesn’t show the extra info about the iPad version, maybe the user agent isn’t checked right. Yknow browsers can’t even tell between iPad and macOS, just look at the webgl test txt I put in root.

WebGL Appears to be Broken

Your browser reports WebGL support, but it appears to be malfunctioning. This is likely a browser bug or graphics driver issue.

Back

For more information, see the Status and Issues page.

[supervoidcoder]

@coderabbitai i also can confirm it still works as intended on places where webgl is not crippled
also the link to the status and issues page is typoed (it links to a missing wiki page on here, github, not the actual omniblocks wiki page: https://omniblocks.miraheze.org/wiki/Status_and_Issues
can you look into that?

[coderabbitai]

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Find the Status and Issues link in webgl-broken-modal
echo "=== Checking webgl-broken-modal component ==="
cat src/components/webgl-broken-modal/webgl-broken-modal.jsx

echo -e "\n=== Checking for any other references to Status and Issues ==="
rg -n "Status.*Issues" -g '*.jsx' -g '*.js' -g '!node_modules'

echo -e "\n=== Check the tw-environment-support-prober for user agent detection ==="
cat src/lib/tw-environment-support-prober.js

Length of output: 10966

---

@supervoidcoder Great to hear it's working! Found the link issue - it's pointing to a non-existent GitHub wiki instead of the Miraheze wiki.

The Link Fix

Line 88 in src/components/webgl-broken-modal/webgl-broken-modal.jsx has the wrong URL. Here's the fix:

href="https://omniblocks.miraheze.org/wiki/Status_and_Issues"

Change line 88 from:

href="https://github.com/OmniBlocks/scratch-gui/wiki/Status_and_Issues"

About the iPad User Agent Detection

You're absolutely right about browsers not being able to distinguish iPad from macOS. The code does attempt to handle this on lines 18-21:

const isIOS = /iPad|iPhone|iPod/.test(navigator.userAgent) || 
    (navigator.platform === 'MacIntel' && navigator.maxTouchPoints > 1);

This catches modern iPads that report as MacIntel (the second condition), but the iOS version detection regex on line 22 won't work for those iPads:

const iosMatch = navigator.userAgent.match(/OS (\d+)[._](\d+)[._]?(\d+)?/);

When iPadOS reports as Mac (which it does by default since iPadOS 13+), the user agent string doesn't contain the "OS 18_7_2" pattern - it just looks like a Mac. This is an intentional Apple design to make iPads get desktop websites.

Result: The modal will show for broken WebGL on modern iPads, but the special iOS 18.7.2 callout won't appear because the version can't be detected. This is a limitation of web APIs, not a bug in the code.

If you want to test this, check navigator.userAgent on your iPad - it probably looks like a Mac Safari user agent string with no iOS version info.

---

🧠 Learnings used

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-15T22:03:54.980Z
Learning: OmniBlocks/scratch-gui: iOS/iPadOS 18.7.2 build 22H123 has a WebKit bug where Renderer.isSupported() returns true but WebGL resource creation (gl.createBuffer(), shader compilation) fails, causing OmniBlocks to crash. Need hardened WebGL smoke test that actually creates GL resources to detect broken WebGL implementations. Build 22H124 fixes it.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-20T00:25:31.489Z
Learning: The Safari WebGL issue reported in issues #364 and #367 is the same as the iOS/iPadOS 18.7.2 build 22H123 WebGL bug - Apple broke WebGL across all their platforms (Safari, iOS, iPadOS, macOS), affecting not just OmniBlocks but all WebGL applications.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-05T01:48:23.435Z
Learning: OmniBlocks/scratch-gui workflow preference: Open tracking issues for CI/automation behavior changes; assign to supervoidcoder; include backlinks to the PR and the exact comment requesting the change.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-20T21:30:07.638Z
Learning: In OmniBlocks/scratch-gui, the service worker in src/playground/service-worker.js was caching songeditor.html at the wrong path (static/songeditor.html) while webpack deploys it to the root (songeditor.html), causing 404s on normal reload but working on hard reload (which bypasses service worker).

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-20T00:25:31.489Z
Learning: OmniBlocks status page is hosted at https://omniblocks.miraheze.org/wiki/Status_and_Issues on Miraheze wiki.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-05T01:40:39.943Z
Learning: In OmniBlocks/scratch-gui, supervoidcoder (“void”) prefers a collaboration model where CodeRabbit authors the code PRs and Amazon Q performs the review; follow-ups should be submitted as single, self-contained PRs when possible.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T19:01:57.123Z
Learning: In the OmniBlocks/scratch-gui repository, team member supervoidcoder can be called "void" in reviews and conversations.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T21:20:08.837Z
Learning: In the OmniBlocks/scratch-gui repository, supervoidcoder (“void”) prefers single, complete PRs over multi-part (“Part 1/2”) submissions. If a split is unavoidable, they want explicit linkage and gating before approval.

---

Learnt from: github-actions[bot]
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-05T02:00:40.666Z
Learning: In OmniBlocks/scratch-gui, when cleanup is requested, supervoidcoder prefers deletion of CodeRabbit’s spam comments (not just minimizing) on the PR; for non-CodeRabbit bot comments, minimize unless a maintainer explicitly asks for hard deletion.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T19:01:57.123Z
Learning: The team members in OmniBlocks/coders include: supervoidcoder, Graison-P (gvyt), ampelectrecuted (amp), NotTheBaton (TheBaton), and kRxZykRxZy.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-18T22:18:13.164Z
Learning: For OmniBlocks/scratch-gui small features, supervoidcoder prefers GUI-only changes that avoid modifying scratch-blocks; deliver as a single, self-contained PR.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-28T03:09:05.973Z
Learning: In the OmniBlocks/scratch-gui repository, supervoidcoder (void) is planning to add a 3D mode and is excited about using PlayCanvas as a full 3D game engine for OmniBlocks.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-09T18:19:24.693Z
Learning: OmniBlocks/scratch-gui: The Blocks component now bridges FieldCustom in componentDidMount by assigning this.ScratchBlocks.FieldCustom = window.Blockly.FieldCustom immediately after ScratchMsgs.setLocale (added Nov 9, 2025 via PR #159). This enables SPjavascriptV2’s registerInput usage at runtime.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-22T02:49:17.267Z
Learning: In OmniBlocks/scratch-gui, TurboWarp, and similar Scratch mods, recursive clone creation (clones creating clones) causes synchronous call stack overflow that hangs the VM without throwing an error. This happens in a single frame before the VM can yield, bypassing Warp Timer protections. TurboWarp/OmniBlocks allow disabling the 300-clone limit, making it possible to create thousands of clones instantly. This is a common accidental crash scenario during development that causes progress loss.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T19:25:31.991Z
Learning: In the OmniBlocks/scratch-gui repository, "React Modal" refers to the react-modal library used for all modal dialogs, such as the advanced settings menu in the menu bar.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-13T01:51:52.357Z
Learning: In OmniBlocks/scratch-vm, using ES6 Map objects in module exports can cause webpack bundling issues that manifest as temporal dead zone (TDZ) errors. The fix is to use plain JavaScript objects instead of Map for module-level data structures that are exported and accessed during initialization.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 271
File: static/index.html:99-106
Timestamp: 2025-10-22T19:03:49.453Z
Learning: The ultrabox music editor (songeditor.html) in the OmniBlocks/scratch-gui repository is JavaScript-heavy and requires script execution to function properly.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-07T19:27:27.244Z
Learning: In OmniBlocks scratch-vm SPjavascriptV2 (src/extensions/sp_javascriptV2/index.js), accessing `this.runtime.extensionRuntimeOptions.javascriptUnsandboxed` without a guard crashes in non-Packager runs; fix by using `this.runtime.extensionRuntimeOptions?.javascriptUnsandboxed === true`.

[supervoidcoder]

@coderabbitai look at the user agent in heer:
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/18.7.2 Safari/605.1.15
WebGL VENDOR: null
WebGL VERSION: null
WebGL RENDERER: null
Unmasked VENDOR: undefined
Unmasked RENDERER: undefined
WebGL R/G/B/A/Depth/Stencil bits (default config): null/null/null/null/null/null

[coderabbitai]

Tip

For best results, initiate chat on the files or code changes.

@supervoidcoder Perfect! That confirms it - the Safari Version/18.7.2 in the user agent is the iOS version. When iPad reports as Mac, we need to parse the Safari version instead of the OS string.

The Fix

Update src/lib/tw-environment-support-prober.js to detect the iOS version from the Safari Version/ string when the device is an iPad masquerading as Mac:

Lines 18-30 should be replaced with:

const isIOS = /iPad|iPhone|iPod/.test(navigator.userAgent) || 
    (navigator.platform === 'MacIntel' && navigator.maxTouchPoints > 1);

let iosVersion = null;
if (isIOS) {
    // Try standard iOS user agent pattern first (iPhone, old iPads)
    let iosMatch = navigator.userAgent.match(/OS (\d+)[._](\d+)[._]?(\d+)?/);
    
    // If no match, iPad is likely reporting as Mac - try Safari Version/ instead
    if (!iosMatch) {
        iosMatch = navigator.userAgent.match(/Version\/(\d+)\.(\d+)(?:\.(\d+))?/);
    }
    
    if (iosMatch) {
        iosVersion = {
            major: parseInt(iosMatch[1], 10),
            minor: parseInt(iosMatch[2], 10),
            patch: iosMatch[3] ? parseInt(iosMatch[3], 10) : 0
        };
    }
}

Then update lines 33-39 to use the new structure:

const isAffectedIOSVersion = iosVersion && 
    iosVersion.major === 18 && 
    iosVersion.minor === 7 && 
    iosVersion.patch === 2;

// Get build version for iOS 18.7.2 devices
const iosBuildMatch = isAffectedIOSVersion && 
    navigator.userAgent.match(/Safari\/(\d+\.\d+\.\d+)/);

This will now correctly detect your iPad as iOS 18.7.2 and show the special iOS callout in the modal!

  (\\_/)
 (='.'=)  *excited twitching*
 (")_(")

---

✏️ Learnings added

Learnt from: supervoidcoder
URL: https://github.com/OmniBlocks/scratch-gui/issues/393

Timestamp: 2025-11-22T03:57:11.049Z
Learning: On iPadOS 13+, when Safari reports the device as Macintosh (user agent shows "Macintosh; Intel Mac OS X"), the Safari Version/ token in the user agent (e.g., "Version/18.7.2") actually represents the iOS/iPadOS version, not just the Safari version. The Version/ number can be used to extract the iOS version when the OS pattern match fails.

Note: Learnings are effective only in the context of similar code segments. To apply general coding guidelines, please configure review instructions. You can manage existing learnings in the app.

---

🧠 Learnings used

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-15T22:03:54.980Z
Learning: OmniBlocks/scratch-gui: iOS/iPadOS 18.7.2 build 22H123 has a WebKit bug where Renderer.isSupported() returns true but WebGL resource creation (gl.createBuffer(), shader compilation) fails, causing OmniBlocks to crash. Need hardened WebGL smoke test that actually creates GL resources to detect broken WebGL implementations. Build 22H124 fixes it.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-20T00:25:31.489Z
Learning: The Safari WebGL issue reported in issues #364 and #367 is the same as the iOS/iPadOS 18.7.2 build 22H123 WebGL bug - Apple broke WebGL across all their platforms (Safari, iOS, iPadOS, macOS), affecting not just OmniBlocks but all WebGL applications.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-05T01:40:39.943Z
Learning: In OmniBlocks/scratch-gui, supervoidcoder (“void”) prefers a collaboration model where CodeRabbit authors the code PRs and Amazon Q performs the review; follow-ups should be submitted as single, self-contained PRs when possible.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T19:01:57.123Z
Learning: In the OmniBlocks/scratch-gui repository, team member supervoidcoder can be called "void" in reviews and conversations.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T21:20:08.837Z
Learning: In the OmniBlocks/scratch-gui repository, supervoidcoder (“void”) prefers single, complete PRs over multi-part (“Part 1/2”) submissions. If a split is unavoidable, they want explicit linkage and gating before approval.

---

Learnt from: github-actions[bot]
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-05T02:00:40.666Z
Learning: In OmniBlocks/scratch-gui, when cleanup is requested, supervoidcoder prefers deletion of CodeRabbit’s spam comments (not just minimizing) on the PR; for non-CodeRabbit bot comments, minimize unless a maintainer explicitly asks for hard deletion.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T19:01:57.123Z
Learning: The team members in OmniBlocks/coders include: supervoidcoder, Graison-P (gvyt), ampelectrecuted (amp), NotTheBaton (TheBaton), and kRxZykRxZy.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-18T22:18:13.164Z
Learning: For OmniBlocks/scratch-gui small features, supervoidcoder prefers GUI-only changes that avoid modifying scratch-blocks; deliver as a single, self-contained PR.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-28T03:09:05.973Z
Learning: In the OmniBlocks/scratch-gui repository, supervoidcoder (void) is planning to add a 3D mode and is excited about using PlayCanvas as a full 3D game engine for OmniBlocks.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-09T18:19:24.693Z
Learning: OmniBlocks/scratch-gui: The Blocks component now bridges FieldCustom in componentDidMount by assigning this.ScratchBlocks.FieldCustom = window.Blockly.FieldCustom immediately after ScratchMsgs.setLocale (added Nov 9, 2025 via PR #159). This enables SPjavascriptV2’s registerInput usage at runtime.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-22T02:49:17.267Z
Learning: In OmniBlocks/scratch-gui, TurboWarp, and similar Scratch mods, recursive clone creation (clones creating clones) causes synchronous call stack overflow that hangs the VM without throwing an error. This happens in a single frame before the VM can yield, bypassing Warp Timer protections. TurboWarp/OmniBlocks allow disabling the 300-clone limit, making it possible to create thousands of clones instantly. This is a common accidental crash scenario during development that causes progress loss.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-22T19:25:31.991Z
Learning: In the OmniBlocks/scratch-gui repository, "React Modal" refers to the react-modal library used for all modal dialogs, such as the advanced settings menu in the menu bar.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-13T01:51:52.357Z
Learning: In OmniBlocks/scratch-vm, using ES6 Map objects in module exports can cause webpack bundling issues that manifest as temporal dead zone (TDZ) errors. The fix is to use plain JavaScript objects instead of Map for module-level data structures that are exported and accessed during initialization.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-11-20T21:30:07.638Z
Learning: In OmniBlocks/scratch-gui, the service worker in src/playground/service-worker.js was caching songeditor.html at the wrong path (static/songeditor.html) while webpack deploys it to the root (songeditor.html), causing 404s on normal reload but working on hard reload (which bypasses service worker).

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 271
File: static/index.html:99-106
Timestamp: 2025-10-22T19:03:49.453Z
Learning: The ultrabox music editor (songeditor.html) in the OmniBlocks/scratch-gui repository is JavaScript-heavy and requires script execution to function properly.

---

Learnt from: supervoidcoder
Repo: OmniBlocks/scratch-gui PR: 0
File: :0-0
Timestamp: 2025-10-07T19:27:27.244Z
Learning: In OmniBlocks scratch-vm SPjavascriptV2 (src/extensions/sp_javascriptV2/index.js), accessing `this.runtime.extensionRuntimeOptions.javascriptUnsandboxed` without a guard crashes in non-Packager runs; fix by using `this.runtime.extensionRuntimeOptions?.javascriptUnsandboxed === true`.

[supervoidcoder]

@coderabbitai uh those aren't the right lines can you check the file