Merge pull request #5663 from 7se7en72025/test/improve-error-message-scope

test: Enhance error scope validation in AST2BlockList tests

Author: omsuneri

js/js-export/__tests__/ast2blocklist.test.js

@@ -58,8 +58,11 @@ describe("AST2BlockList Class", () => {
         try {
             AST2BlockList.toBlockList(AST, config);
         } catch (e) {
-            //TODO: error message should isolate to smallest scope
+            // Verify error message provides context about unsupported statement
             expect(e.prefix).toEqual("Unsupported statement: ");
+            // Error should include position information for scope isolation
+            expect(e.start).toBeDefined();
+            expect(e.end).toBeDefined();
         }
     });
 

js/loader.js

@@ -301,15 +301,14 @@ requirejs(["i18next", "i18nextHttpBackend"], function (i18next, i18nextHttpBacke
                 function () {
                     // Give scripts a moment to finish executing and set globals
                     setTimeout(function () {
-                        // Log verification status for debugging (non-blocking)
+                        // Verify core dependencies are loaded
                         const verificationStatus = {
                             createjs: typeof window.createjs !== "undefined",
                             createDefaultStack: typeof window.createDefaultStack !== "undefined",
                             Logo: typeof window.Logo !== "undefined",
                             Blocks: typeof window.Blocks !== "undefined",
                             Turtles: typeof window.Turtles !== "undefined"
                         };
-                        console.log("Core module verification:", verificationStatus);
 
                         // Check critical dependencies (only createjs is truly critical)
                         if (typeof window.createjs === "undefined") {
@@ -325,7 +324,6 @@ requirejs(["i18next", "i18nextHttpBackend"], function (i18next, i18nextHttpBacke
                             ["activity/activity"],
                             function () {
                                 // Activity loaded successfully
-                                console.log("Activity module loaded successfully");
                             },
                             function (err) {
                                 console.error("Failed to load activity/activity:", err);