chore: use typed locals instead of repeated as-casts in comparators

Author: Alp

packages/zql/src/ivm/memory-source.ts

@@ -884,23 +884,23 @@ function makeBoundComparator(sort: Ordering) {
       if (bN === null) return 1;
       // Inline compareValues for string (most common) and number
       if (typeof av === 'string') {
+        const aStr = av;
+        const bStr = bv as string;
         // Inline compareStringUTF8Fast
-        if (av === bv) return 0;
+        if (aStr === bStr) return 0;
         const len =
-          (av as string).length < (bv as string).length
-            ? (av as string).length
-            : (bv as string).length;
+          aStr.length < bStr.length ? aStr.length : bStr.length;
         for (let i = 0; i < len; i++) {
-          const ac = (av as string).charCodeAt(i);
-          const bc = (bv as string).charCodeAt(i);
+          const ac = aStr.charCodeAt(i);
+          const bc = bStr.charCodeAt(i);
           if (ac !== bc) {
             if (ac < 128 && bc < 128) return ac - bc;
-            return compareUTF8(av as string, bv as string);
+            return compareUTF8(aStr, bStr);
           }
         }
-        return (av as string).length - (bv as string).length;
+        return aStr.length - bStr.length;
       }
-      if (typeof av === 'number') return (av as number) - (bv as number);
+      if (typeof av === 'number') return av - (bv as number);
       // Fallback for other types (boolean, etc.)
       return compareValues(av as Value, bv as Value);
     };
@@ -948,7 +948,7 @@ function compareBounds(a: Bound, b: Bound): number {
   if (bN === null) {
     return 1;
   }
-  return compareValues(a as Value, b as Value);
+  return compareValues(aN, bN);
 }
 
 function* generateRows(