@query-farm/vgi 0.24.0 → 0.24.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@query-farm/vgi",
-  "version": "0.24.0",
+  "version": "0.24.1",
   "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)",
@@ -69,15 +69,18 @@
   "files": [
     "dist",
     "src",
+    "scripts/postinstall.cjs",
     "!src/**/__tests__/**",
     "!src/**/*.test.ts",
     "!src/**/*.spec.ts"
   ],
+  "//imports": "`browser` maps to arrow-js DELIBERATELY, not flechette. Measured on the client surface (VgiClient + Arguments + batch helpers), a minified browser bundle is 84 KB gzip via arrow-js vs 116 KB via flechette — arrow-js tree-shakes to the subset the client touches, while flechette links as a unit. Flechette still wins for `worker-cf`, which exercises far more of the Arrow API and cannot take a peer dep. Keep `browser` AFTER `workerd`/`worker`: conditions match in key order, so a workerd build (which sets both) must hit flechette first.",
   "imports": {
     "#arrow-impl": {
       "flechette": "./src/arrow/impl-flechette/index.ts",
       "workerd": "./src/arrow/impl-flechette/index.ts",
       "worker": "./src/arrow/impl-flechette/index.ts",
+      "browser": "./src/arrow/impl-arrowjs/index.ts",
       "default": "./src/arrow/impl-arrowjs/index.ts"
     }
   },
@@ -87,12 +90,12 @@
   },
   "peerDependencies": {
     "@query-farm/apache-arrow": "^21.1.1",
-    "@query-farm/vgi-rpc": "^0.17.0"
+    "@query-farm/vgi-rpc": ">=0.18.1 <0.19.0"
   },
   "devDependencies": {
     "@duckdb/node-api": "^1.5.2-r.1",
     "@query-farm/apache-arrow": "^21.1.1",
-    "@query-farm/vgi-rpc": "^0.17.0",
+    "@query-farm/vgi-rpc": "^0.18.1",
     "@types/bun": "latest",
     "typescript": "^6.0.3"
   },

package/scripts/postinstall.cjs

@@ -0,0 +1,47 @@
+#!/usr/bin/env node
+// Copyright 2025, 2026 Query Farm LLC - https://query.farm
+// Postinstall: deduplicate @query-farm/apache-arrow across @query-farm/vgi-rpc
+// to avoid dual-package instanceof issues.
+//
+// This is a LOCAL DEVELOPMENT-only fixup for the vgi-typescript monorepo, where
+// @query-farm/vgi-rpc is a `file:` link with its own nested copy of apache-arrow.
+// It must NOT run when this package is installed as a dependency of someone
+// else's project — so it no-ops unless the sibling vgi-rpc-typescript checkout
+// is present next to this repo. (Published consumers get a single hoisted
+// apache-arrow from the registry, so no dedup is needed there.)
+const fs = require("fs");
+const path = require("path");
+
+const PKG = "@query-farm/apache-arrow";
+
+// Resolve everything relative to the repo root (this file lives in <repo>/scripts),
+// not the process cwd, so behavior doesn't depend on where the installer runs us.
+const repoRoot = path.resolve(__dirname, "..");
+
+// Dev-monorepo signal: the sibling vgi-rpc source checkout. When this package is
+// installed as a dependency, that path resolves inside the consumer's node_modules
+// and won't exist — so we bail and leave their tree untouched.
+const devSibling = path.resolve(repoRoot, "..", "vgi-rpc-typescript", "package.json");
+if (!fs.existsSync(devSibling)) {
+  process.exit(0);
+}
+
+// Deduplicate: replace vgi-rpc's nested copy with a symlink to ours.
+//   Without this, two separate DataType classes exist at runtime and
+//   instanceof checks fail across package boundaries.
+//   Bun uses per-file symlinks for file: deps, so module resolution follows
+//   the real path. We must target the REAL node_modules of vgi-rpc-typescript.
+const ourArrow = path.join(repoRoot, "node_modules", PKG);
+const rpcPkgLink = path.join(repoRoot, "node_modules", "@query-farm", "vgi-rpc", "package.json");
+try {
+  const realPkgPath = fs.realpathSync(rpcPkgLink);
+  const rpcRoot = path.dirname(realPkgPath);
+  const rpcArrow = path.join(rpcRoot, "node_modules", PKG);
+  const st = fs.lstatSync(rpcArrow);
+  if (!st.isSymbolicLink()) {
+    fs.rmSync(rpcArrow, { recursive: true });
+    fs.symlinkSync(ourArrow, rpcArrow);
+  }
+} catch (e) {
+  if (e.code !== "ENOENT") throw e;
+}