@query-farm/vgi 0.32.0 → 0.33.0

package/README.md

@@ -41,7 +41,7 @@ npm install @query-farm/vgi
 Requires **Node.js ≥ 22.15** or **Bun**.
 
 > **Peer dependencies (since 0.3.0).** `@query-farm/apache-arrow` (`^21.1.1`) and
-> `@query-farm/vgi-rpc` (`^0.24.0`) are **peerDependencies** — install them directly
+> `@query-farm/vgi-rpc` (`^0.25.2`) are **peerDependencies** — install them directly
 > alongside `@query-farm/vgi`:
 >
 > ```bash

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@query-farm/vgi",
-  "version": "0.32.0",
+  "version": "0.33.0",
   "description": "TypeScript framework for extending DuckDB with user-defined functions via the Vector Gateway Interface (VGI) over Arrow IPC.",
   "license": "SEE LICENSE IN LICENSE",
   "author": "Query Farm LLC (https://query.farm)",
@@ -91,12 +91,12 @@
   },
   "peerDependencies": {
     "@query-farm/apache-arrow": "^21.1.1",
-    "@query-farm/vgi-rpc": ">=0.24.0 <0.25.0"
+    "@query-farm/vgi-rpc": ">=0.25.2 <0.26.0"
   },
   "devDependencies": {
     "@duckdb/node-api": "^1.5.5-r.4",
     "@query-farm/apache-arrow": "^21.1.1",
-    "@query-farm/vgi-rpc": "^0.24.0",
+    "@query-farm/vgi-rpc": "^0.25.2",
     "@types/bun": "latest",
     "typescript": "^7.0.2"
   },

package/src/arrow/impl-flechette/metadata.ts

@@ -16,7 +16,18 @@ import type { VgiBatch } from "../types.js";
  */
 export function withBatchMetadata(batch: VgiBatch, metadata: Map<string, string>): VgiBatch {
   const t = batch as any;
-  const clone = Object.assign(Object.create(Object.getPrototypeOf(t)), t);
+  // Copy the own properties as descriptors, not by assignment. vgi-rpc's reader
+  // pins a zero-column batch's row count as an own `numRows` over
+  // `Table.prototype.numRows`, a getter-only accessor, and `Object.assign`
+  // *assigns* each own property -- which throws in strict mode ("Attempted to
+  // assign to readonly property") over an inherited getter-only accessor.
+  const clone = Object.create(Object.getPrototypeOf(t), Object.getOwnPropertyDescriptors(t));
+  // This backend's own reader supplies that row count through a Proxy instead
+  // (see `deserializeBatch`), which no property copy carries: without this the
+  // clone would derive 0 rows from its absent columns.
+  if (clone.numRows !== t.numRows) {
+    Object.defineProperty(clone, "numRows", { value: t.numRows, configurable: true, enumerable: true });
+  }
   if (metadata && metadata.size > 0) {
     clone._vgiRecordMetadata = metadata;
     clone.metadata = metadata;

package/src/arrow/impl-flechette/normalize-type.ts

@@ -18,7 +18,7 @@
 import {
   Type,
   field as f_field,
-  nullType, bool, int, float, utf8, binary, fixedSizeBinary,
+  nullType, bool, int, float, utf8, largeUtf8, binary, largeBinary, fixedSizeBinary,
   decimal, date, time, timestamp, duration, interval,
   list, largeList, fixedSizeList, struct, map, dictionary, union,
 } from "@query-farm/flechette";
@@ -47,12 +47,19 @@ export function toFlechetteType(type: any): any {
       return aliasIntSigned(int(type.bitWidth, type.isSigned ?? type.signed ?? true));
     case Type.Float:
       return float(type.precision);
+    // Large variants stay large: flechette builds and encodes them, and
+    // folding them into Binary/Utf8 put a different type on the wire than the
+    // one declared (arrow-js writes LargeBinary/LargeUtf8). InitRequest
+    // declares `pushdown_filters` large_binary, so a flechette worker or
+    // client wrote that field as binary.
     case Type.Binary:
-    case Type.LargeBinary:
       return binary();
+    case Type.LargeBinary:
+      return largeBinary();
     case Type.Utf8:
-    case Type.LargeUtf8:
       return utf8();
+    case Type.LargeUtf8:
+      return largeUtf8();
     case Type.FixedSizeBinary:
       return fixedSizeBinary(type.byteWidth ?? type.stride);
     case Type.Decimal: