@victor-software-house/pi-openai-proxy 4.3.0 → 4.3.1

package/dist/config.d.mts

@@ -50,10 +50,12 @@ interface ProxyConfig {
   readonly providerPrefixes: Readonly<Record<string, string>>;
 }
 declare const DEFAULT_CONFIG: Readonly<ProxyConfig>;
+declare function isPublicModelIdMode(value: string): value is PublicModelIdMode;
+declare function isModelExposureMode(value: string): value is ModelExposureMode;
 declare function normalizeConfig(raw: unknown): ProxyConfig;
 declare function getConfigPath(): string;
 declare function loadConfigFromFile(): ProxyConfig;
 declare function saveConfigToFile(config: ProxyConfig): void;
 declare function configToEnv(config: ProxyConfig): Record<string, string>;
 //#endregion
-export { DEFAULT_CONFIG, ModelExposureMode, ProxyConfig, PublicModelIdMode, configToEnv, getConfigPath, loadConfigFromFile, normalizeConfig, saveConfigToFile };
+export { DEFAULT_CONFIG, ModelExposureMode, ProxyConfig, PublicModelIdMode, configToEnv, getConfigPath, isModelExposureMode, isPublicModelIdMode, loadConfigFromFile, normalizeConfig, saveConfigToFile };

package/dist/config.mjs

@@ -1,118 +1,3 @@
 #!/usr/bin/env bun
-import { existsSync, mkdirSync, readFileSync, renameSync, unlinkSync, writeFileSync } from "node:fs";
-import { dirname, resolve } from "node:path";
-//#region src/config/schema.ts
-/**
-* Proxy configuration schema -- single source of truth.
-*
-* Used by both the proxy server and the pi extension.
-* The server reads the JSON config file as defaults, with env vars and CLI args as overrides.
-* The pi extension reads and writes the JSON config file via the /proxy config panel.
-*/
-const DEFAULT_CONFIG = {
-	host: "127.0.0.1",
-	port: 4141,
-	authToken: "",
-	remoteImages: false,
-	maxBodySizeMb: 50,
-	upstreamTimeoutSec: 120,
-	lifetime: "detached",
-	publicModelIdMode: "collision-prefixed",
-	modelExposureMode: "scoped",
-	scopedProviders: [],
-	customModels: [],
-	providerPrefixes: {}
-};
-function isRecord(value) {
-	return value !== null && value !== void 0 && typeof value === "object" && !Array.isArray(value);
-}
-function clampInt(raw, min, max, fallback) {
-	if (typeof raw !== "number" || !Number.isFinite(raw)) return fallback;
-	return Math.max(min, Math.min(max, Math.round(raw)));
-}
-const VALID_PUBLIC_ID_MODES = new Set([
-	"collision-prefixed",
-	"universal",
-	"always-prefixed"
-]);
-function isPublicModelIdMode(value) {
-	return VALID_PUBLIC_ID_MODES.has(value);
-}
-const VALID_EXPOSURE_MODES = new Set([
-	"all",
-	"scoped",
-	"custom"
-]);
-function isModelExposureMode(value) {
-	return VALID_EXPOSURE_MODES.has(value);
-}
-function normalizeStringArray(raw) {
-	if (!Array.isArray(raw)) return [];
-	const result = [];
-	for (const item of raw) if (typeof item === "string" && item.length > 0) result.push(item);
-	return result;
-}
-function normalizeStringRecord(raw) {
-	if (!isRecord(raw)) return {};
-	const result = {};
-	for (const [key, val] of Object.entries(raw)) if (typeof val === "string" && val.length > 0) result[key] = val;
-	return result;
-}
-function normalizeConfig(raw) {
-	const v = isRecord(raw) ? raw : {};
-	const rawHost = v["host"];
-	const rawAuthToken = v["authToken"];
-	const rawRemoteImages = v["remoteImages"];
-	const rawPublicIdMode = v["publicModelIdMode"];
-	const rawExposureMode = v["modelExposureMode"];
-	return {
-		host: typeof rawHost === "string" && rawHost.length > 0 ? rawHost : DEFAULT_CONFIG.host,
-		port: clampInt(v["port"], 1, 65535, DEFAULT_CONFIG.port),
-		authToken: typeof rawAuthToken === "string" ? rawAuthToken : DEFAULT_CONFIG.authToken,
-		remoteImages: typeof rawRemoteImages === "boolean" ? rawRemoteImages : DEFAULT_CONFIG.remoteImages,
-		maxBodySizeMb: clampInt(v["maxBodySizeMb"], 1, 500, DEFAULT_CONFIG.maxBodySizeMb),
-		upstreamTimeoutSec: clampInt(v["upstreamTimeoutSec"], 5, 600, DEFAULT_CONFIG.upstreamTimeoutSec),
-		lifetime: v["lifetime"] === "session" ? "session" : "detached",
-		publicModelIdMode: typeof rawPublicIdMode === "string" && isPublicModelIdMode(rawPublicIdMode) ? rawPublicIdMode : DEFAULT_CONFIG.publicModelIdMode,
-		modelExposureMode: typeof rawExposureMode === "string" && isModelExposureMode(rawExposureMode) ? rawExposureMode : DEFAULT_CONFIG.modelExposureMode,
-		scopedProviders: normalizeStringArray(v["scopedProviders"]),
-		customModels: normalizeStringArray(v["customModels"]),
-		providerPrefixes: normalizeStringRecord(v["providerPrefixes"])
-	};
-}
-function getConfigPath() {
-	return resolve(process.env["PI_CODING_AGENT_DIR"] ?? resolve(process.env["HOME"] ?? "~", ".pi", "agent"), "proxy-config.json");
-}
-function loadConfigFromFile() {
-	const p = getConfigPath();
-	if (!existsSync(p)) return { ...DEFAULT_CONFIG };
-	try {
-		return normalizeConfig(JSON.parse(readFileSync(p, "utf-8")));
-	} catch {
-		return { ...DEFAULT_CONFIG };
-	}
-}
-function saveConfigToFile(config) {
-	const p = getConfigPath();
-	const normalized = normalizeConfig(config);
-	const tmp = `${p}.tmp`;
-	try {
-		mkdirSync(dirname(p), { recursive: true });
-		writeFileSync(tmp, `${JSON.stringify(normalized, null, "	")}\n`, "utf-8");
-		renameSync(tmp, p);
-	} catch {
-		if (existsSync(tmp)) unlinkSync(tmp);
-	}
-}
-function configToEnv(config) {
-	const env = {};
-	env["PI_PROXY_HOST"] = config.host;
-	env["PI_PROXY_PORT"] = String(config.port);
-	if (config.authToken.length > 0) env["PI_PROXY_AUTH_TOKEN"] = config.authToken;
-	env["PI_PROXY_REMOTE_IMAGES"] = String(config.remoteImages);
-	env["PI_PROXY_MAX_BODY_SIZE"] = String(config.maxBodySizeMb * 1024 * 1024);
-	env["PI_PROXY_UPSTREAM_TIMEOUT_MS"] = String(config.upstreamTimeoutSec * 1e3);
-	return env;
-}
-//#endregion
-export { DEFAULT_CONFIG, configToEnv, getConfigPath, loadConfigFromFile, normalizeConfig, saveConfigToFile };
+import { a as isPublicModelIdMode, c as saveConfigToFile, i as isModelExposureMode, n as configToEnv, o as loadConfigFromFile, r as getConfigPath, s as normalizeConfig, t as DEFAULT_CONFIG } from "./schema-x6mps-hM.mjs";
+export { DEFAULT_CONFIG, configToEnv, getConfigPath, isModelExposureMode, isPublicModelIdMode, loadConfigFromFile, normalizeConfig, saveConfigToFile };

package/dist/exposure.mjs

@@ -2,7 +2,11 @@
 //#region src/openai/model-exposure.ts
 function filterExposedModels(available, allRegistered, config) {
 	switch (config.modelExposureMode) {
-		case "scoped": return [...available];
+		case "scoped": {
+			if (config.scopedProviders.length === 0) return [...available];
+			const allowed = new Set(config.scopedProviders);
+			return available.filter((m) => allowed.has(m.provider));
+		}
 		case "all": return [...allRegistered];
 		case "custom": {
 			const allowed = new Set(config.customModels);

package/dist/index.mjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env bun
-import { getConfigPath, loadConfigFromFile } from "./config.mjs";
+import { l as isRecord, o as loadConfigFromFile, r as getConfigPath } from "./schema-x6mps-hM.mjs";
 import { computeModelExposure, resolveExposedModel } from "./exposure.mjs";
 import { AuthStorage, ModelRegistry } from "@mariozechner/pi-coding-agent";
 import { randomBytes } from "node:crypto";
@@ -476,6 +476,8 @@ function parseAndValidateDataUri(uri) {
 }
 //#endregion
 //#region src/openai/models.ts
+/** Unix timestamp (seconds) from when the module was first loaded. */
+const MODULE_CREATED = Math.floor(Date.now() / 1e3);
 /**
 * Convert an ExposedModel to an OpenAI model object.
 */
@@ -483,7 +485,7 @@ function toOpenAIModel(exposed) {
 	return {
 		id: exposed.publicId,
 		object: "model",
-		created: 0,
+		created: MODULE_CREATED,
 		owned_by: exposed.provider
 	};
 }
@@ -549,6 +551,7 @@ function buildChatCompletion(requestId, canonicalModelId, message) {
 		object: "chat.completion",
 		created: Math.floor(Date.now() / 1e3),
 		model: canonicalModelId,
+		system_fingerprint: null,
 		choices: [{
 			index: 0,
 			message: messageBody,
@@ -724,9 +727,31 @@ async function* streamToSSE(events, requestId, model, includeUsage) {
 }
 //#endregion
 //#region src/openai/json-schema-to-typebox.ts
-function isRecord$2(value) {
-	return value !== null && typeof value === "object" && !Array.isArray(value);
-}
+/**
+* JSON Schema -> TypeBox conversion for OpenAI function tool parameters.
+*
+* Phase 2 contract:
+* - Support a documented subset of JSON Schema only
+* - Reject unsupported schema constructs with ConversionError
+* - Do not silently downgrade complex schemas
+*
+* Supported subset:
+* - type: object, string, number, integer, boolean, array, null
+* - properties, required
+* - enum (string enums)
+* - arrays with supported item schema
+* - nullable via type: [T, "null"]
+* - anyOf for nullable types and simple unions (max 10 branches)
+* - description on any schema node
+*
+* Rejected:
+* - $ref
+* - oneOf, allOf
+* - recursive schemas
+* - additionalProperties as a schema (boolean true/false allowed)
+* - patternProperties
+* - if/then/else
+*/
 /**
 * Unsupported JSON Schema keywords that we reject explicitly.
 * Note: `anyOf` is handled separately for common patterns (nullable types, simple unions).
@@ -747,7 +772,7 @@ const REJECTED_KEYWORDS = [
 * Returns a ConversionError for unsupported constructs.
 */
 function jsonSchemaToTypebox(schema, path = "") {
-	if (!isRecord$2(schema)) return {
+	if (!isRecord(schema)) return {
 		ok: false,
 		message: "Schema must be an object",
 		path
@@ -894,7 +919,7 @@ function convertObject(schema, path, opts) {
 		ok: true,
 		schema: Type.Object({}, opts)
 	};
-	if (!isRecord$2(rawProperties)) return {
+	if (!isRecord(rawProperties)) return {
 		ok: false,
 		message: `'properties' must be an object at ${path || "root"}`,
 		path
@@ -1123,16 +1148,13 @@ const rejectedFields = [
 *
 * Phase 2 contract: unknown fields -> 422, rejected fields -> 422
 */
-function isRecord$1(value) {
-	return value !== null && typeof value === "object" && !Array.isArray(value);
-}
 /**
 * Validate a raw request body against the Phase 1 schema.
 *
 * Also checks for known rejected fields to give friendly errors.
 */
 function validateChatRequest(body) {
-	if (isRecord$1(body)) {
+	if (isRecord(body)) {
 		for (const field of rejectedFields) if (body[field] !== void 0) return {
 			ok: false,
 			status: 422,
@@ -1170,9 +1192,6 @@ function validateChatRequest(body) {
 }
 //#endregion
 //#region src/pi/complete.ts
-function isRecord(value) {
-	return value !== null && typeof value === "object" && !Array.isArray(value);
-}
 /**
 * Map OpenAI reasoning_effort to pi ThinkingLevel.
 *
@@ -1276,7 +1295,7 @@ async function piComplete(model, context, request, options) {
 	return completeSimple(model, context, await buildStreamOptions(model, request, options));
 }
 /**
-* Streaming completion: returns an async iterable of events.
+* Streaming completion: returns an event stream with abort capability.
 */
 async function piStream(model, context, request, options) {
 	return streamSimple(model, context, await buildStreamOptions(model, request, options));

package/dist/schema-x6mps-hM.mjs

@@ -0,0 +1,127 @@
+#!/usr/bin/env bun
+import { existsSync, mkdirSync, readFileSync, renameSync, unlinkSync, writeFileSync } from "node:fs";
+import { dirname, resolve } from "node:path";
+//#region src/utils/guards.ts
+/**
+* Shared type guard utilities.
+*/
+/**
+* Narrow `unknown` to `Record<string, unknown>`.
+* Rejects null, undefined, and arrays.
+*/
+function isRecord(value) {
+	return value !== null && typeof value === "object" && !Array.isArray(value);
+}
+//#endregion
+//#region src/config/schema.ts
+/**
+* Proxy configuration schema -- single source of truth.
+*
+* Used by both the proxy server and the pi extension.
+* The server reads the JSON config file as defaults, with env vars and CLI args as overrides.
+* The pi extension reads and writes the JSON config file via the /proxy config panel.
+*/
+const DEFAULT_CONFIG = {
+	host: "127.0.0.1",
+	port: 4141,
+	authToken: "",
+	remoteImages: false,
+	maxBodySizeMb: 50,
+	upstreamTimeoutSec: 120,
+	lifetime: "detached",
+	publicModelIdMode: "collision-prefixed",
+	modelExposureMode: "scoped",
+	scopedProviders: [],
+	customModels: [],
+	providerPrefixes: {}
+};
+function clampInt(raw, min, max, fallback) {
+	if (typeof raw !== "number" || !Number.isFinite(raw)) return fallback;
+	return Math.max(min, Math.min(max, Math.round(raw)));
+}
+const VALID_PUBLIC_ID_MODES = new Set([
+	"collision-prefixed",
+	"universal",
+	"always-prefixed"
+]);
+function isPublicModelIdMode(value) {
+	return VALID_PUBLIC_ID_MODES.has(value);
+}
+const VALID_EXPOSURE_MODES = new Set([
+	"all",
+	"scoped",
+	"custom"
+]);
+function isModelExposureMode(value) {
+	return VALID_EXPOSURE_MODES.has(value);
+}
+function normalizeStringArray(raw) {
+	if (!Array.isArray(raw)) return [];
+	const result = [];
+	for (const item of raw) if (typeof item === "string" && item.length > 0) result.push(item);
+	return result;
+}
+function normalizeStringRecord(raw) {
+	if (!isRecord(raw)) return {};
+	const result = {};
+	for (const [key, val] of Object.entries(raw)) if (typeof val === "string" && val.length > 0) result[key] = val;
+	return result;
+}
+function normalizeConfig(raw) {
+	const v = isRecord(raw) ? raw : {};
+	const rawHost = v["host"];
+	const rawAuthToken = v["authToken"];
+	const rawRemoteImages = v["remoteImages"];
+	const rawPublicIdMode = v["publicModelIdMode"];
+	const rawExposureMode = v["modelExposureMode"];
+	return {
+		host: typeof rawHost === "string" && rawHost.length > 0 ? rawHost : DEFAULT_CONFIG.host,
+		port: clampInt(v["port"], 1, 65535, DEFAULT_CONFIG.port),
+		authToken: typeof rawAuthToken === "string" ? rawAuthToken : DEFAULT_CONFIG.authToken,
+		remoteImages: typeof rawRemoteImages === "boolean" ? rawRemoteImages : DEFAULT_CONFIG.remoteImages,
+		maxBodySizeMb: clampInt(v["maxBodySizeMb"], 1, 500, DEFAULT_CONFIG.maxBodySizeMb),
+		upstreamTimeoutSec: clampInt(v["upstreamTimeoutSec"], 5, 600, DEFAULT_CONFIG.upstreamTimeoutSec),
+		lifetime: v["lifetime"] === "session" ? "session" : "detached",
+		publicModelIdMode: typeof rawPublicIdMode === "string" && isPublicModelIdMode(rawPublicIdMode) ? rawPublicIdMode : DEFAULT_CONFIG.publicModelIdMode,
+		modelExposureMode: typeof rawExposureMode === "string" && isModelExposureMode(rawExposureMode) ? rawExposureMode : DEFAULT_CONFIG.modelExposureMode,
+		scopedProviders: normalizeStringArray(v["scopedProviders"]),
+		customModels: normalizeStringArray(v["customModels"]),
+		providerPrefixes: normalizeStringRecord(v["providerPrefixes"])
+	};
+}
+function getConfigPath() {
+	return resolve(process.env.PI_CODING_AGENT_DIR ?? resolve(process.env.HOME ?? "~", ".pi", "agent"), "proxy-config.json");
+}
+function loadConfigFromFile() {
+	const p = getConfigPath();
+	if (!existsSync(p)) return { ...DEFAULT_CONFIG };
+	try {
+		return normalizeConfig(JSON.parse(readFileSync(p, "utf-8")));
+	} catch {
+		return { ...DEFAULT_CONFIG };
+	}
+}
+function saveConfigToFile(config) {
+	const p = getConfigPath();
+	const normalized = normalizeConfig(config);
+	const tmp = `${p}.tmp`;
+	try {
+		mkdirSync(dirname(p), { recursive: true });
+		writeFileSync(tmp, `${JSON.stringify(normalized, null, "	")}\n`, "utf-8");
+		renameSync(tmp, p);
+	} catch {
+		if (existsSync(tmp)) unlinkSync(tmp);
+	}
+}
+function configToEnv(config) {
+	const env = {};
+	env["PI_PROXY_HOST"] = config.host;
+	env["PI_PROXY_PORT"] = String(config.port);
+	if (config.authToken.length > 0) env["PI_PROXY_AUTH_TOKEN"] = config.authToken;
+	env["PI_PROXY_REMOTE_IMAGES"] = String(config.remoteImages);
+	env["PI_PROXY_MAX_BODY_SIZE"] = String(config.maxBodySizeMb * 1024 * 1024);
+	env["PI_PROXY_UPSTREAM_TIMEOUT_MS"] = String(config.upstreamTimeoutSec * 1e3);
+	return env;
+}
+//#endregion
+export { isPublicModelIdMode as a, saveConfigToFile as c, isModelExposureMode as i, isRecord as l, configToEnv as n, loadConfigFromFile as o, getConfigPath as r, normalizeConfig as s, DEFAULT_CONFIG as t };

package/extensions/proxy.ts

@@ -76,18 +76,19 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 	const extensionDir = dirname(fileURLToPath(import.meta.url));
 	const packageRoot = resolve(extensionDir, "..");
 
-	// --- Model registry access (for verify/show/selectors) ---
+	// --- Model registry access (cached, refreshed per call) ---
+
+	const cachedAuth = AuthStorage.create();
+	const cachedRegistry = new ModelRegistry(cachedAuth);
 
 	function getAvailableModels(): Model<Api>[] {
-		const auth = AuthStorage.create();
-		const registry = new ModelRegistry(auth);
-		return registry.getAvailable();
+		cachedRegistry.refresh();
+		return cachedRegistry.getAvailable();
 	}
 
 	function getAllRegisteredModels(): Model<Api>[] {
-		const auth = AuthStorage.create();
-		const registry = new ModelRegistry(auth);
-		return registry.getAll();
+		cachedRegistry.refresh();
+		return cachedRegistry.getAll();
 	}
 
 	function buildExposureConfig(): ModelExposureConfig {
@@ -256,8 +257,13 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 
 	async function probe(): Promise<RuntimeStatus> {
 		try {
+			const headers: Record<string, string> = {};
+			if (config.authToken.length > 0) {
+				headers["authorization"] = `Bearer ${config.authToken}`;
+			}
 			const res = await fetch(`${proxyUrl()}/v1/models`, {
 				signal: AbortSignal.timeout(2000),
+				headers,
 			});
 			if (res.ok) {
 				const body = (await res.json()) as { data?: unknown[] };
@@ -639,8 +645,10 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 			{ enableSearch: true },
 		);
 
-		// SettingsList has no public selectedIndex setter.
-		// Access internals via bracket notation for provider jumping.
+		// HACK: SettingsList has no public API for jumping to an index.
+		// Accesses private fields via bracket notation for provider jumping.
+		// Pinned to pi-tui behavior as of @mariozechner/pi-coding-agent ^0.62.0.
+		// Remove when SettingsList exposes a jumpTo/setSelectedIndex method.
 		function jumpProvider(direction: "prev" | "next"): void {
 			const sl = list as unknown as Record<string, unknown>;
 			const idx = sl["selectedIndex"] as number;
@@ -837,16 +845,14 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 		}
 	}
 
-	const VALID_ID_MODES = new Set<string>(["collision-prefixed", "universal", "always-prefixed"]);
-
+	// Local type guards — the extension resolves against the built dist, so it
+	// cannot import these from the source config module during development.
 	function isPublicModelIdMode(v: string): v is PublicModelIdMode {
-		return VALID_ID_MODES.has(v);
+		return v === "collision-prefixed" || v === "universal" || v === "always-prefixed";
 	}
 
-	const VALID_EXPOSURE_MODES = new Set<string>(["all", "scoped", "custom"]);
-
 	function isModelExposureMode(v: string): v is ModelExposureMode {
-		return VALID_EXPOSURE_MODES.has(v);
+		return v === "all" || v === "scoped" || v === "custom";
 	}
 
 	function applySetting(id: string, value: string): void {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@victor-software-house/pi-openai-proxy",
-	"version": "4.3.0",
+	"version": "4.3.1",
 	"description": "OpenAI-compatible HTTP proxy for pi's multi-provider model registry",
 	"license": "MIT",
 	"author": "Victor Software House",
@@ -63,6 +63,7 @@
 		"format": "biome format --write .",
 		"test": "bun test",
 		"test:ci": "bun test test/unit/ test/integration/",
+		"prepare": "tsdown",
 		"prepack": "bun run build",
 		"prepublishOnly": "bun test && bun run build"
 	},