fix: move empty id check before visited set, add parentheses for operator precedence

- Move !id guard before visited.add() to prevent empty strings being added
- Add parentheses around range checks in CSS escape fallback for correct precedence

Agent-Logs-Url: https://github.com/antvis/Infographic/sessions/289607cf-6b2e-4c1c-b79d-ec538a1b098c

Co-authored-by: Aarebecca <25787943+Aarebecca@users.noreply.github.com>

Author: Copilot

src/exporter/svg.ts

@@ -489,13 +489,10 @@ function collectDefElements(svg: SVGSVGElement, ids: Set<string>) {
 
   while (queue.length) {
     const id = queue.shift()!;
+    if (!id) continue;
     if (visited.has(id)) continue;
     visited.add(id);
 
-    if (!id) {
-      continue;
-    }
-
     const selector = `#${escapeCssId(id)}`;
     const target = svg.querySelector(selector);
     if (!target) continue;
@@ -567,9 +564,9 @@ function escapeCssId(id: string) {
 
       // Characters that are safe to use unescaped
       if (
-        codeUnit >= 0x0030 && codeUnit <= 0x0039 || // 0-9
-        codeUnit >= 0x0041 && codeUnit <= 0x005a || // A-Z
-        codeUnit >= 0x0061 && codeUnit <= 0x007a || // a-z
+        (codeUnit >= 0x0030 && codeUnit <= 0x0039) || // 0-9
+        (codeUnit >= 0x0041 && codeUnit <= 0x005a) || // A-Z
+        (codeUnit >= 0x0061 && codeUnit <= 0x007a) || // a-z
         codeUnit === 0x002d || // -
         codeUnit === 0x005f // _
       ) {