@query-farm/vgi 0.34.0 → 0.35.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@query-farm/vgi",
-  "version": "0.34.0",
+  "version": "0.35.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.25.2 <0.26.0"
+    "@query-farm/vgi-rpc": ">=0.25.4 <0.26.0"
   },
   "devDependencies": {
     "@duckdb/node-api": "^1.5.5-r.4",
     "@query-farm/apache-arrow": "^21.1.1",
-    "@query-farm/vgi-rpc": "^0.25.2",
+    "@query-farm/vgi-rpc": "^0.25.4",
     "@types/bun": "latest",
     "typescript": "^7.0.2"
   },

package/src/client/client.ts

@@ -3,7 +3,11 @@
 // Works with any RpcClient (subprocess or HTTP transport).
 
 import { type VgiSchema, schema as schema_, type VgiField, field, type VgiBatch, type VgiDataType, utf8, binary, list } from "../arrow/index.js";
-import { type RpcClient, type StreamSession } from "@query-farm/vgi-rpc";
+// `import type`, not `import { type … }`: the latter keeps the statement and
+// emits `import {} from "@query-farm/vgi-rpc"`, a side-effect import of the
+// package root — which re-exports the whole framework, server included, into
+// every browser bundle. See the note in ./errors.
+import type { RpcClient, StreamSession } from "@query-farm/vgi-rpc";
 import {
   serializeBindRequest,
   deserializeBindResponse,

package/src/client/errors.ts

@@ -2,7 +2,16 @@
 // VgiClient error class + RpcClient wrapper that translates RpcError into
 // VgiClientError, preserving the worker's remote traceback.
 
-import { RpcError, type RpcClient } from "@query-farm/vgi-rpc";
+// RpcError comes from the `/connect` subpath, not the package root, and the
+// distinction is worth ~160 kB in every browser bundle. The root re-exports the
+// whole framework — protocol, dispatch, access log, the server — and a bundler
+// cannot drop it, so importing one error class from there shipped `RpcServer`
+// to browsers to satisfy an `instanceof`. `RpcClient` is a pure `import type`
+// for the same reason: `import { type X }` leaves the statement behind, and a
+// bare `import {} from "@query-farm/vgi-rpc"` is a side-effect import of that
+// same graph.
+import { RpcError } from "@query-farm/vgi-rpc/connect";
+import type { RpcClient } from "@query-farm/vgi-rpc";
 
 /** Error thrown by VgiClient when an RPC call fails or returns unexpected data. */
 export class VgiClientError extends Error {