fix: resolve RequireJS AMD race conditions and dependency issues (sugarlabs#2632) (sugarlabs#5433)

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

Author: vanshika2720

js/__tests__/loader.test.js

@@ -306,7 +306,10 @@ class Activity {
         this.loadAnimationIntervalId = null;
 
         // Initialize GIF animator
-        this.gifAnimator = new GIFAnimator();
+        // Check if GIFAnimator is available before using it
+        if (typeof GIFAnimator !== "undefined") {
+            this.gifAnimator = new GIFAnimator();
+        }
 
         // Dirty flag for canvas rendering optimization
         // When true, the stage needs to be redrawn on the next animation frame
@@ -5683,7 +5686,9 @@ class Activity {
             this.update = true;
         };
 
-        this.__showAltoAccidentals = () => {};
+        this.__showAltoAccidentals = () => {
+            // No-op for Alto clef
+        };
 
         /*
          * Shows musical alto staff

js/activity.js

@@ -7164,3 +7164,17 @@ class Blocks {
         };
     }
 }
+// Export Blocks
+if (typeof define === "function" && define.amd) {
+    define([], function () {
+        return Blocks;
+    });
+}
+
+if (typeof module !== "undefined" && module.exports) {
+    module.exports = Blocks;
+}
+
+if (typeof window !== "undefined") {
+    window.Blocks = Blocks;
+}

js/blocks.js

@@ -246,3 +246,7 @@ class GIFAnimator {
 if (typeof module !== "undefined" && module.exports) {
     module.exports = GIFAnimator;
 }
+
+if (typeof window !== "undefined") {
+    window.GIFAnimator = GIFAnimator;
+}