fix(renderer): Prevent Renderer Crash on Texture Update by Preloading Assets

[TKanX]

Summary:

Fixed a critical issue where the renderer could crash if a new texture pack failed to load during a call to updateTextures. The previous implementation modified the live AssetsManager instance, which could lead to an inconsistent state if the new assets were invalid or unreachable.

Refactors the updateTextures method to use a "preload-and-swap" strategy. It now creates a temporary AssetsManager instance to load the new textures in the background. Only after the new assets are successfully loaded is the main AssetsManager instance replaced. This ensures that the renderer remains in a stable state and does not crash if a texture update fails.

Changes:

• Refactored updateTextures for Safe Asset Loading:
  • The updateTextures method in renderer.js no longer modifies the active AssetsManager directly.
  • It now instantiates a temporary AssetsManager to handle the loading of the new texture index and atlas.
  • If the temporary manager successfully loads the new assets, the renderer's primary _assetsManager is swapped with the temporary one.
  • If any part of the loading process fails within the temporary instance, an error is thrown, and the original, stable _assetsManager remains untouched, preventing a renderer crash.
• Improved Error Handling:
  • Simplified the try...catch block and error propagation, making the method more robust.
• Forced Re-render:
  • After a successful texture update and swap, a full, forced re-render of all layers is now triggered by calling this._layersManager.render(true) to immediately reflect the new textures.

[Copilot]

Pull Request Overview

This PR fixes a critical issue in the renderer where texture pack updates could cause crashes if new assets failed to load. The solution implements a "preload-and-swap" strategy to ensure the renderer maintains a stable state during texture updates.

• Implements safe asset loading by creating temporary instances before swapping
• Moves frameRequested initialization to constructor for better state management
• Improves error handling to prevent renderer crashes during failed texture updates

Comments suppressed due to low confidence (1)

src/renderer.js:305

• [nitpick] The variable name 'tempTextures' could be more descriptive. Consider renaming it to 'newTexturePack' or 'candidateTextures' to better reflect that this represents the new texture pack being validated before swapping.

      const tempTextures = new TexturePack(