@victor-software-house/pi-openai-proxy 4.0.4 → 4.0.6

package/extensions/proxy.ts

@@ -99,11 +99,14 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 		};
 	}
 
-	// Resolve pi-proxy binary: try workspace node_modules, then global
+	// Resolve pi-proxy binary: try local dev build, then installed package, then PATH
 	function findProxyBinary(): string {
-		// In workspace: node_modules/pi-proxy/dist/index.mjs
-		const workspaceBin = resolve(packageRoot, "node_modules", "pi-proxy", "dist", "index.mjs");
-		if (existsSync(workspaceBin)) return workspaceBin;
+		// Local development: dist/index.mjs in the same package root
+		const localBin = resolve(packageRoot, "dist", "index.mjs");
+		if (existsSync(localBin)) return localBin;
+		// Installed as dependency: node_modules/pi-proxy/dist/index.mjs
+		const depBin = resolve(packageRoot, "node_modules", "pi-proxy", "dist", "index.mjs");
+		if (existsSync(depBin)) return depBin;
 		// Fallback: assume pi-proxy is in PATH
 		return "pi-proxy";
 	}
@@ -611,6 +614,40 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 		};
 	}
 
+	// --- Dynamic descriptions ---
+
+	const ID_MODE_DESCRIPTIONS: Record<string, string> = {
+		"collision-prefixed": "Short names; adds provider/ prefix only when models collide",
+		universal: "Short names only; fails if any model name is shared by two providers",
+		"always-prefixed": "Always provider/model-id for every model",
+	};
+
+	const EXPOSURE_MODE_DESCRIPTIONS: Record<string, string> = {
+		scoped: "Expose models from pi's configured auth (default)",
+		all: "Expose all registered models, including those without auth",
+		custom: "Expose only manually selected models",
+	};
+
+	function idModeDescription(): string {
+		return ID_MODE_DESCRIPTIONS[config.publicModelIdMode] ?? "";
+	}
+
+	function exposureModeDescription(): string {
+		return EXPOSURE_MODE_DESCRIPTIONS[config.modelExposureMode] ?? "";
+	}
+
+	function customModelsDescription(): string {
+		if (config.modelExposureMode === "custom") return "Press Enter to open model selector";
+		return "Switch exposure mode to 'custom' to select models";
+	}
+
+	function authDescription(): string {
+		if (config.authToken.length > 0) {
+			return `Token: ${config.authToken.slice(0, 8)}... (use /proxy show to copy)`;
+		}
+		return "Require bearer token for all requests";
+	}
+
 	function buildSettingItems(): SettingItem[] {
 		return [
 			// --- Server ---
@@ -638,10 +675,7 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 			{
 				id: "authToken",
 				label: "Proxy auth",
-				description:
-					config.authToken.length > 0
-						? `Token: ${config.authToken.slice(0, 8)}... (use /proxy show to copy)`
-						: "Require bearer token for all requests",
+				description: authDescription(),
 				currentValue: config.authToken.length > 0 ? "enabled" : "disabled",
 				values: ["disabled", "enabled"],
 			},
@@ -669,32 +703,50 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 			// --- Model exposure ---
 			{
 				id: "publicModelIdMode",
-				label: "Public ID mode",
-				description: "How public model IDs are generated from canonical provider/model-id",
+				label: "Model ID format",
+				description: idModeDescription(),
 				currentValue: config.publicModelIdMode,
 				values: ["collision-prefixed", "universal", "always-prefixed"],
 			},
 			{
 				id: "modelExposureMode",
 				label: "Exposure mode",
-				description:
-					"scoped = pi's available models, all = all registered, custom = manual selection",
+				description: exposureModeDescription(),
 				currentValue: config.modelExposureMode,
 				values: ["scoped", "all", "custom"],
 			},
 			{
 				id: "customModels",
 				label: "Select models",
-				description:
-					config.modelExposureMode === "custom"
-						? "Press Enter to open model selector"
-						: "Switch exposure mode to 'custom' to select models",
+				description: customModelsDescription(),
 				currentValue: customModelsDisplay(),
 				submenu: config.modelExposureMode === "custom" ? buildModelSelectorSubmenu : undefined,
 			},
 		];
 	}
 
+	/**
+	 * Update descriptions on items that change dynamically based on the current value.
+	 */
+	function refreshDescriptions(items: SettingItem[]): void {
+		for (const item of items) {
+			switch (item.id) {
+				case "publicModelIdMode":
+					item.description = idModeDescription();
+					break;
+				case "modelExposureMode":
+					item.description = exposureModeDescription();
+					break;
+				case "customModels":
+					item.description = customModelsDescription();
+					break;
+				case "authToken":
+					item.description = authDescription();
+					break;
+			}
+		}
+	}
+
 	const VALID_ID_MODES = new Set<string>(["collision-prefixed", "universal", "always-prefixed"]);
 
 	function isPublicModelIdMode(v: string): v is PublicModelIdMode {
@@ -818,8 +870,9 @@ export default function proxyExtension(pi: ExtensionAPI): void {
 							ctx.ui.notify(`Auth token: ${lastGeneratedToken}`, "info");
 						}
 
-						// Update display value in-place (no rebuild, preserves selection)
+						// Update display value and descriptions in-place (preserves selection)
 						settingsList.updateValue(id, getDisplayValue(id));
+						refreshDescriptions(items);
 
 						// When exposure mode changes, update the "Select models" item
 						if (id === "modelExposureMode") {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@victor-software-house/pi-openai-proxy",
-	"version": "4.0.4",
+	"version": "4.0.6",
 	"description": "OpenAI-compatible HTTP proxy for pi's multi-provider model registry",
 	"license": "MIT",
 	"author": "Victor Software House",