More from coderabbit

Author: haslinghuis

src/js/msp/msp_debug_dashboard.js

@@ -464,6 +464,15 @@ export class MSPDebugDashboard {
                         this.showTestDetails(testIndex);
                     }
                 }
+
+                // Handle close details button clicks
+                if (e.target.classList.contains("close-details-btn")) {
+                    const testDetails = e.target.closest(".test-details");
+                    if (testDetails) {
+                        testDetails.remove();
+                        this.pauseUpdates(1000);
+                    }
+                }
             }
         });
 
@@ -1100,7 +1109,7 @@ export class MSPDebugDashboard {
                 `
         : ""
 }
-                <button onclick="this.parentElement.remove(); window.MSPDebug.dashboard.pauseUpdates(1000);" 
+                <button class="close-details-btn" 
                         style="margin-top: 10px; padding: 5px 10px; background: #666; color: white; border: none; border-radius: 3px; cursor: pointer;">
                     Close Details
                 </button>

src/js/msp/msp_test_runner.js

@@ -220,6 +220,11 @@ export const MSPTestRunner = {
     async quickHealthCheck() {
         console.log("🏥 Running Quick MSP Health Check...");
 
+        if (!window.MSP) {
+            console.error("MSP not available");
+            return { status: "ERROR", error: "MSP not initialized" };
+        }
+
         // Start monitoring briefly
         mspQueueMonitor.startMonitoring(100);
 
@@ -291,7 +296,12 @@ export const MSPTestRunner = {
                 const startTime = Date.now();
 
                 while (Date.now() - startTime < 5000) {
-                    promises.push(window.MSP.promise(101, null).catch(() => {}));
+                    promises.push(
+                        window.MSP.promise(101, null).catch((err) => {
+                            console.error("MSP request failed in sustained-load scenario:", err);
+                            return { error: err.message || "Unknown error" };
+                        }),
+                    );
                     await new Promise((resolve) => setTimeout(resolve, 10));
                 }
 
@@ -330,7 +340,12 @@ export const MSPTestRunner = {
                 for (let i = 0; i < 30; i++) {
                     const code = codes[i % codes.length];
                     const data = i % 4 === 0 ? new Uint8Array([i, i + 1, i + 2]) : null;
-                    promises.push(window.MSP.promise(code, data).catch(() => {}));
+                    promises.push(
+                        window.MSP.promise(code, data).catch((err) => {
+                            console.error(`MSP request failed in mixed-load scenario (code: ${code}):`, err);
+                            return { error: err.message || "Unknown error" };
+                        }),
+                    );
                 }
 
                 const startTime = Date.now();