Skip to content

[pull] master from monkeytypegame:master#726

Merged
pull[bot] merged 3 commits intoUncodedtech:masterfrom
monkeytypegame:master
Dec 14, 2025
Merged

[pull] master from monkeytypegame:master#726
pull[bot] merged 3 commits intoUncodedtech:masterfrom
monkeytypegame:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull Bot commented Dec 14, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

Miodec and others added 3 commits December 13, 2025 23:55
Move ui code to test ui, remove unused code, remove duplicated code,
merge the two config event listeners, merge a lot of the ui code to make
it easier to grasp.
)

### Description
  
* `update()` could hit a race where settings became `null` between
checks, causing a`TypeError` at runtime.

* Async callbacks (setTimeout) accessed the global settings after it was
cleared, leading to runaway errors.
<img width="1887" height="755" alt="Screenshot 2025-12-12 135316"
src="https://github.com/user-attachments/assets/ba6bd13c-c052-4870-ba7c-cfeae916b6dc"
/>


**fix**  
settings once (currentSettings) at the start of update() and use that
for all property access and scheduling, so the loop never touches a
null/stale reference.
@pull pull Bot locked and limited conversation to collaborators Dec 14, 2025
@pull pull Bot added the ⤵️ pull label Dec 14, 2025
@pull pull Bot merged commit 331ca1a into Uncodedtech:master Dec 14, 2025
2 of 4 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants