@victor-software-house/pi-openai-proxy 4.9.4 → 4.9.5

package/README.md

@@ -20,10 +20,10 @@ A local OpenAI-compatible HTTP proxy built on [pi](https://github.com/badlogic/p
 
 ```bash
 # Install globally
-npm install -g @victor-software-house/pi-openai-proxy
+pnpm add -g @victor-software-house/pi-openai-proxy
 
-# Or run directly with npx
-npx @victor-software-house/pi-openai-proxy
+# Or run directly
+pnpm dlx @victor-software-house/pi-openai-proxy
 ```
 
 ## Quickstart
@@ -339,17 +339,18 @@ The extension detects externally running instances and shows their status via `/
 ## Dev Workflow
 
 ```bash
-bun install           # Install dependencies
-bun run dev           # Run in development
-bun run build         # Build for npm (tsdown)
-bun run typecheck     # TypeScript strict check
-bun run lint          # Biome + oxlint (strict)
-bun test              # Run all tests
+pnpm install          # Install dependencies
+pnpm run dev          # Run in development
+pnpm run build        # Build for npm (tsdown)
+pnpm run typecheck    # TypeScript strict check
+pnpm run lint         # Biome + oxlint (strict)
+pnpm test             # Run all tests
 ```
 
 ### Tooling
 
-- **Bun** — runtime, test runner, package manager
+- **pnpm** — package manager and script runner
+- **Bun** — runtime and test runner
 - **tsdown** — npm build (ESM + .d.ts)
 - **Biome** — format + lint
 - **oxlint** — type-aware lint with strict rules (`.oxlintrc.json`)

package/dist/index.mjs

@@ -761,7 +761,7 @@ async function* streamToSSE(events, requestId, model, includeUsage) {
 	yield encodeDone();
 }
 //#endregion
-//#region node_modules/typebox/build/system/memory/metrics.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/system/memory/metrics.mjs
 /** TypeBox instantiation metrics */
 const Metrics = {
 	assign: 0,
@@ -771,7 +771,7 @@ const Metrics = {
 	update: 0
 };
 //#endregion
-//#region node_modules/typebox/build/guard/guard.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/guard/guard.mjs
 /** Returns true if this value is an array */
 function IsArray(value) {
 	return Array.isArray(value);
@@ -816,7 +816,7 @@ function Keys(value) {
 	return Object.getOwnPropertyNames(value);
 }
 //#endregion
-//#region node_modules/typebox/build/system/memory/clone.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/system/memory/clone.mjs
 function IsGuard(value) {
 	return IsObject(value) && HasPropertyKey(value, "~guard");
 }
@@ -857,7 +857,7 @@ function Clone(value) {
 	return FromValue(value);
 }
 //#endregion
-//#region node_modules/typebox/build/system/settings/settings.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/system/settings/settings.mjs
 const settings = {
 	immutableTypes: false,
 	maxErrors: 8,
@@ -871,7 +871,7 @@ function Get() {
 	return settings;
 }
 //#endregion
-//#region node_modules/typebox/build/system/memory/create.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/system/memory/create.mjs
 function MergeHidden(left, right) {
 	for (const key of Object.keys(right)) Object.defineProperty(left, key, {
 		configurable: true,
@@ -900,7 +900,7 @@ function Create(hidden, enumerable, options = {}) {
 	return settings.immutableTypes ? Object.freeze(withHidden) : withHidden;
 }
 //#endregion
-//#region node_modules/typebox/build/system/memory/update.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/system/memory/update.mjs
 /**
 * Updates a value with new properties while preserving property enumerability. Use this function to modify
 * existing types without altering their configuration.
@@ -924,12 +924,12 @@ function Update(current, hidden, enumerable) {
 	return result;
 }
 //#endregion
-//#region node_modules/typebox/build/type/types/schema.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/schema.mjs
 function IsSchema(value) {
 	return IsObject(value);
 }
 //#endregion
-//#region node_modules/typebox/build/type/types/_optional.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/_optional.mjs
 /** Adds Optional to the given type. */
 function OptionalAdd(type) {
 	return Update(type, { "~optional": true }, {});
@@ -943,7 +943,7 @@ function IsOptional(value) {
 	return IsSchema(value) && HasPropertyKey(value, "~optional");
 }
 //#endregion
-//#region node_modules/typebox/build/type/types/array.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/array.mjs
 /** Creates an Array type. */
 function _Array_(items, options) {
 	return Create({ "~kind": "Array" }, {
@@ -952,37 +952,36 @@ function _Array_(items, options) {
 	}, options);
 }
 //#endregion
-//#region node_modules/typebox/build/type/types/properties.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/properties.mjs
 /** Creates a RequiredArray derived from the given TProperties value. */
 function RequiredArray(properties) {
 	return Keys(properties).filter((key) => !IsOptional(properties[key]));
 }
 //#endregion
-//#region node_modules/typebox/build/type/types/object.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/object.mjs
 /** Creates an Object type. */
 function _Object_(properties, options = {}) {
 	const requiredKeys = RequiredArray(properties);
-	const required = requiredKeys.length > 0 ? { required: requiredKeys } : {};
 	return Create({ "~kind": "Object" }, {
 		type: "object",
-		...required,
+		...requiredKeys.length > 0 ? { required: requiredKeys } : {},
 		properties
 	}, options);
 }
 //#endregion
-//#region node_modules/typebox/build/type/types/union.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/union.mjs
 /** Creates a Union type. */
 function Union(anyOf, options = {}) {
 	return Create({ "~kind": "Union" }, { anyOf }, options);
 }
 //#endregion
-//#region node_modules/typebox/build/type/types/unknown.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/unknown.mjs
 /** Creates an Unknown type. */
 function Unknown(options) {
 	return Create({ ["~kind"]: "Unknown" }, {}, options);
 }
 //#endregion
-//#region node_modules/typebox/build/system/hashing/hash.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/system/hashing/hash.mjs
 var ByteMarker;
 (function(ByteMarker) {
 	ByteMarker[ByteMarker["Array"] = 0] = "Array";
@@ -1000,27 +999,26 @@ var ByteMarker;
 	ByteMarker[ByteMarker["TypeArray"] = 12] = "TypeArray";
 	ByteMarker[ByteMarker["Undefined"] = 13] = "Undefined";
 })(ByteMarker || (ByteMarker = {}));
-const [Prime, Size] = [BigInt("1099511628211"), BigInt("18446744073709551616")];
 Array.from({ length: 256 }).map((_, i) => BigInt(i));
 const F64 = new Float64Array(1);
 new DataView(F64.buffer);
 new Uint8Array(F64.buffer);
 new TextEncoder();
 //#endregion
-//#region node_modules/typebox/build/type/types/boolean.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/boolean.mjs
 /** Creates a Boolean type. */
 function Boolean(options) {
 	return Create({ "~kind": "Boolean" }, { type: "boolean" }, options);
 }
 //#endregion
-//#region node_modules/typebox/build/type/types/integer.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/integer.mjs
 const IntegerPattern = "-?(?:0|[1-9][0-9]*)";
 /** Creates a Integer type. */
 function Integer(options) {
 	return Create({ "~kind": "Integer" }, { type: "integer" }, options);
 }
 //#endregion
-//#region node_modules/typebox/build/type/types/literal.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/literal.mjs
 var InvalidLiteralValue = class extends Error {
 	constructor(value) {
 		super(`Invalid Literal value`);
@@ -1045,30 +1043,30 @@ function Literal(value, options) {
 	}, options);
 }
 //#endregion
-//#region node_modules/typebox/build/type/types/null.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/null.mjs
 /** Creates a Null type. */
 function Null(options) {
 	return Create({ "~kind": "Null" }, { type: "null" }, options);
 }
 //#endregion
-//#region node_modules/typebox/build/type/types/number.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/number.mjs
 const NumberPattern = "-?(?:0|[1-9][0-9]*)(?:.[0-9]+)?";
 /** Creates a Number type. */
 function Number$1(options) {
 	return Create({ "~kind": "Number" }, { type: "number" }, options);
 }
 //#endregion
-//#region node_modules/typebox/build/type/types/string.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/string.mjs
 /** Creates a String type. */
 function String$1(options) {
 	return Create({ "~kind": "String" }, { type: "string" }, options);
 }
 //#endregion
-//#region node_modules/typebox/build/type/types/record.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/types/record.mjs
 const IntegerKey = `^${IntegerPattern}$`;
 `${NumberPattern}`;
 //#endregion
-//#region node_modules/typebox/build/type/script/token/internal/char.mjs
+//#region node_modules/.pnpm/typebox@1.1.38/node_modules/typebox/build/type/script/token/internal/char.mjs
 function Range(start, end) {
 	return Array.from({ length: end - start + 1 }, (_, i) => String.fromCharCode(start + i));
 }

package/extensions/proxy.ts

@@ -20,8 +20,10 @@
  */
 
 import { type ChildProcess, spawn } from "node:child_process";
-import { existsSync, mkdirSync, readFileSync, unlinkSync, writeFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
+import { constants, existsSync, mkdirSync, readFileSync, unlinkSync, writeFileSync } from "node:fs";
+import { access } from "node:fs/promises";
+import { homedir } from "node:os";
+import { delimiter, dirname, join, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 import type { Api, Model } from "@mariozechner/pi-ai";
 import {
@@ -129,6 +131,41 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 		return "pi-proxy";
 	}
 
+	async function isExecutable(path: string): Promise<boolean> {
+		try {
+			await access(path, constants.X_OK);
+			return true;
+		} catch {
+			return false;
+		}
+	}
+
+	async function resolveBunExecutable(): Promise<string> {
+		if ("bun" in process.versions) return process.execPath;
+
+		const bunInstall = process.env["BUN_INSTALL"];
+		const pathEntries = (process.env["PATH"] ?? "")
+			.split(delimiter)
+			.filter((entry) => entry.length > 0);
+		const candidates = [
+			process.env["PI_OPENAI_PROXY_BUN"],
+			bunInstall === undefined ? undefined : join(bunInstall, "bin", "bun"),
+			...pathEntries.map((entry) => join(entry, "bun")),
+			join(homedir(), ".bun", "bin", "bun"),
+			join(homedir(), ".local", "share", "mise", "installs", "bun", "latest", "bin", "bun"),
+			"/opt/homebrew/bin/bun",
+			"/usr/local/bin/bun",
+		];
+
+		for (const candidate of candidates) {
+			if (candidate !== undefined && (await isExecutable(candidate))) return candidate;
+		}
+
+		throw new Error(
+			"Bun runtime not found. Install Bun or set PI_OPENAI_PROXY_BUN to the Bun executable path.",
+		);
+	}
+
 	function proxyUrl(): string {
 		return `http://${config.host}:${String(config.port)}`;
 	}
@@ -343,9 +380,9 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 			const bin = findProxyBinary();
 
 			if (config.lifetime === "detached") {
-				startDetached(bin, proxyEnv);
+				await startDetached(bin, proxyEnv);
 			} else {
-				startSessionTied(ctx, bin, proxyEnv);
+				await startSessionTied(ctx, bin, proxyEnv);
 			}
 
 			const ready = await waitForReady(3000);
@@ -366,9 +403,9 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 		}
 	}
 
-	function startDetached(bin: string, env: Record<string, string>): void {
+	async function startDetached(bin: string, env: Record<string, string>): Promise<void> {
 		const usesBun = bin.endsWith(".mjs");
-		const cmd = usesBun ? "bun" : bin;
+		const cmd = usesBun ? await resolveBunExecutable() : bin;
 		const cmdArgs = usesBun ? ["run", bin] : [];
 
 		const child = spawn(cmd, cmdArgs, {
@@ -377,6 +414,10 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 			env: { ...process.env, ...env },
 		});
 
+		child.once("error", () => {
+			if (child.pid !== undefined) removePid();
+		});
+
 		if (child.pid === undefined) {
 			throw new Error("No PID returned from spawn");
 		}
@@ -385,14 +426,18 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 		writePid(child.pid);
 	}
 
-	function startSessionTied(ctx: ExtensionContext, bin: string, env: Record<string, string>): void {
+	async function startSessionTied(
+		ctx: ExtensionContext,
+		bin: string,
+		env: Record<string, string>,
+	): Promise<void> {
 		if (sessionProcess !== undefined) {
 			ctx.ui.notify("Session proxy already running", "info");
 			return;
 		}
 
 		const usesBun = bin.endsWith(".mjs");
-		const cmd = usesBun ? "bun" : bin;
+		const cmd = usesBun ? await resolveBunExecutable() : bin;
 		const cmdArgs = usesBun ? ["run", bin] : [];
 
 		sessionProcess = spawn(cmd, cmdArgs, {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@victor-software-house/pi-openai-proxy",
-	"version": "4.9.4",
+	"version": "4.9.5",
 	"description": "OpenAI-compatible HTTP proxy for pi's multi-provider model registry",
 	"license": "MIT",
 	"author": "Victor Software House",
@@ -28,6 +28,7 @@
 	"engines": {
 		"node": ">=20"
 	},
+	"packageManager": "pnpm@11.0.0-dev.1005",
 	"type": "module",
 	"exports": {
 		".": {
@@ -60,7 +61,7 @@
 		"build": "tsdown",
 		"start": "bun dist/index.mjs",
 		"typecheck": "tsc --noEmit",
-		"lint": "bun run lint:biome && bun run lint:oxlint",
+		"lint": "pnpm run lint:biome && pnpm run lint:oxlint",
 		"lint:biome": "biome check .",
 		"lint:oxlint": "oxlint --import-plugin --type-aware --tsconfig=./tsconfig.json .",
 		"lint:fix": "biome check --write .",
@@ -68,36 +69,37 @@
 		"test": "bun test",
 		"test:ci": "bun test test/unit/ test/integration/",
 		"prepare": "tsdown",
-		"prepack": "bun run build",
-		"prepublishOnly": "bun test && bun run build"
+		"prepack": "pnpm run build",
+		"prepublishOnly": "pnpm test && pnpm run build"
 	},
 	"dependencies": {
 		"@mariozechner/pi-ai": "^0.73.0",
 		"@mariozechner/pi-coding-agent": "^0.73.0",
-		"citty": "^0.1.6",
-		"hono": "^4.12.8",
+		"@mariozechner/pi-tui": "^0.73.0",
+		"citty": "^0.2.2",
+		"hono": "^4.12.18",
 		"jsonc-parser": "^3.3.1",
-		"zod": "^4.3.6"
+		"zod": "^4.4.3"
 	},
 	"devDependencies": {
-		"@biomejs/biome": "^2.4.8",
-		"@commitlint/cli": "^20.5.0",
-		"@commitlint/config-conventional": "^20.5.0",
+		"@biomejs/biome": "^2.4.14",
+		"@commitlint/cli": "^20.5.3",
+		"@commitlint/config-conventional": "^20.5.3",
 		"@limegrass/eslint-plugin-import-alias": "^1.6.1",
 		"@semantic-release/changelog": "^6.0.3",
 		"@semantic-release/git": "^10.0.1",
 		"@semantic-release/github": "^12.0.6",
 		"@semantic-release/npm": "^13.1.5",
-		"@types/bun": "^1.3.11",
-		"@types/node": "^25.5.0",
-		"eslint-plugin-zod": "^3.5.0",
-		"lefthook": "^2.1.4",
-		"openai": "^6.26.0",
-		"oxlint": "^1.56.0",
-		"oxlint-tsgolint": "^0.17.1",
+		"@types/bun": "^1.3.13",
+		"@types/node": "^25.6.0",
+		"eslint-plugin-zod": "^3.12.0",
+		"lefthook": "^2.1.6",
+		"openai": "^6.36.0",
+		"oxlint": "^1.63.0",
+		"oxlint-tsgolint": "^0.22.1",
 		"semantic-release": "^25.0.3",
-		"tsdown": "^0.21.4",
+		"tsdown": "^0.21.10",
 		"typebox": "^1.1.38",
-		"typescript": "^5.9.3"
+		"typescript": "^6.0.3"
 	}
 }