fix: MaterializedExecutor sort used string comparison for bigint values

The inline sort comparator checked `typeof av === 'number'` and fell
through to `String(av).localeCompare()` for bigint, causing 10n to
sort before 9n (lexicographic). Replace with shared rowComparator()
which uses < / > operators that work correctly for all types.

Author: teamchong

src/client.ts

@@ -15,7 +15,7 @@ import type {
   VectorSearchParams,
   WindowSpec,
 } from "./types.js";
-import { NULL_SENTINEL } from "./types.js";
+import { NULL_SENTINEL, rowComparator } from "./types.js";
 import type { Operator, RowBatch } from "./operators.js";
 import { rowPassesFilters } from "./decode.js";
 import { computePartialAgg, finalizePartialAgg } from "./partial-agg.js";
@@ -963,15 +963,7 @@ export class MaterializedExecutor implements QueryExecutor {
 
     // Apply sort
     if (query.sortColumn) {
-      const col = query.sortColumn;
-      const desc = query.sortDirection === "desc";
-      rows.sort((a, b) => {
-        const av = a[col], bv = b[col];
-        if (av === null || av === undefined) return 1;
-        if (bv === null || bv === undefined) return -1;
-        if (typeof av === "number" && typeof bv === "number") return desc ? bv - av : av - bv;
-        return desc ? String(bv).localeCompare(String(av)) : String(av).localeCompare(String(bv));
-      });
+      rows.sort(rowComparator(query.sortColumn, query.sortDirection === "desc"));
     }
 
     // Apply offset + limit

src/materialized-executor.test.ts

@@ -129,6 +129,17 @@ describe("MaterializedExecutor", () => {
       expect(result.rows[4].name).toBe("Eve");
     });
 
+    it("sorts bigint values numerically (not lexicographically)", async () => {
+      const bigData = [
+        { id: 1n, val: 9n },
+        { id: 2n, val: 10n },
+        { id: 3n, val: 2n },
+        { id: 4n, val: 100n },
+      ];
+      const result = await createFromJSON(bigData).sort("val", "asc").collect();
+      expect(result.rows.map(r => r.val)).toEqual([2n, 9n, 10n, 100n]);
+    });
+
     it("pushes nulls to end", async () => {
       const dataWithNull = [
         { id: 1, val: null },