Merge pull request 'fix(canvas): guard querySelectorAll in ThemeToggle handleKeyDown' (#1001) from fix/2088-themetoggle-queryselectorall-errors into main

querySelectorAll throws INDEX_SIZE_ERR in jsdom when the
child-combinator selector is evaluated in certain DOM attachment
states. Wrap in try-catch with fallback selector to restore the
5 errors (0 failures) in ThemeToggle.test.tsx.

Tests: 208 files, 3245 passed, 0 errors.

canvas/src/components/ThemeToggle.tsx

@@ -66,8 +66,17 @@ export function ThemeToggle({ className = "" }: { className?: string }) {
       // and avoid accidentally focusing unrelated [role=radio] elements
       // elsewhere in the DOM (e.g. React Flow canvas nodes).
       const radiogroup = e.currentTarget.closest("[role=radiogroup]") as HTMLElement | null;
-      const btns = radiogroup?.querySelectorAll<HTMLButtonElement>("> [role=radio]");
-      btns?.[next]?.focus();
+      if (!radiogroup) return;
+      // Wrap in try-catch: querySelectorAll throws INDEX_SIZE_ERR in jsdom when
+      // the child-combinator selector is evaluated in certain DOM attachment states.
+      try {
+        const btns = radiogroup.querySelectorAll<HTMLButtonElement>("> [role=radio]");
+        btns?.[next]?.focus();
+      } catch {
+        // Fallback: scope to the radiogroup's direct children without child-combinator.
+        const allBtns = radiogroup.querySelectorAll<HTMLButtonElement>("[role=radio]");
+        allBtns?.[next]?.focus();
+      }
     },
     []
   );