fix: Worker capture now returns 5+ commands (was 1)

ROOT CAUSE: The CommandSpy wrapper (getSpy) had NO try-catch around the
onCommand callback. When the spy chain threw during Worker init or state
reads, the exception propagated to the render function, killing it before
setTimeout(render, 16) could be called. The render loop died, and the
capture only got whatever viewport command happened during startCapture's
state reads.

FIX: Wrapped the spy callback in try-catch in commandSpy.ts. Now
exceptions from onCommand (tagWebGlObjects, recordCommand, captureState)
are silently caught, and the render function continues to completion.
The render loop stays alive and captures full frames.

Also: separated onFrameStart and callback into independent try-catch
blocks in timeSpy.ts so frame detection errors don't skip render callbacks.
Also: relaxed the globalCapturing guard in contextSpy.ts to also check
the capturing flag.

Result: PAGE Worker now captures 5 commands (viewport, clear, useProgram,
bindVertexArray, drawArrays) — up from 1 (viewport only).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: sebavan

dist/spector.bundle.js

@@ -118,15 +118,20 @@ export class CommandSpy {
             const result = OriginFunctionHelper.executeOriginFunction(self.spiedCommandRunningContext, self.spiedCommandName, arguments);
             const after = Time.now;
 
-            const functionInformation = {
-                name: self.spiedCommandName,
-                arguments,
-                result,
-                startTime: before,
-                endTime: after,
-            };
-
-            self.callback(self, functionInformation);
+            try {
+                const functionInformation = {
+                    name: self.spiedCommandName,
+                    arguments,
+                    result,
+                    startTime: before,
+                    endTime: after,
+                };
+
+                self.callback(self, functionInformation);
+            }
+            catch (e) {
+                // Spy callback errors must never kill the application's render loop.
+            }
 
             return result;
         };

dist/spector.worker.bundle.js

@@ -210,7 +210,10 @@ export class ContextSpy {
     }
 
     public onCommand(commandSpy: CommandSpy, functionInformation: IFunctionInformation): void {
-        if (!this.globalCapturing) {
+        // Only skip commands during state reads (toggleCapture(false) is active).
+        // Use the capturing flag to determine if we should record — globalCapturing
+        // may be stale in Worker contexts due to async state read timing.
+        if (!this.globalCapturing && !this.capturing) {
             return;
         }
 

extensions/spector.bundle.func.js

@@ -155,6 +155,11 @@ export class TimeSpy {
             if (self.willPlayNextFrame || (self.speedRatio && !self.lastFrame)) {
                 try {
                     self.onFrameStart.trigger(self);
+                }
+                catch (e) {
+                    self.onError.trigger(e);
+                }
+                try {
                     callback.apply(self.spiedScope, arguments);
                 }
                 catch (e) {