@howaboua/pi-codex-conversion 1.0.21 → 1.0.24

package/README.md

@@ -9,7 +9,7 @@ This package replaces Pi's default Codex/GPT experience with a narrower Codex-li
 - preserves Pi's composed system prompt and applies a narrow Codex-oriented delta on top
 - renders exec activity with Codex-style command and background-terminal labels
 - renders `apply_patch` calls with Codex-style `Added` / `Edited` / `Deleted` diff blocks and Pi-style colored diff lines
-- targets modern Pi tool/rendering APIs and is aligned with Pi `0.69.x`
+- targets modern Pi tool/rendering APIs and is aligned with Pi `0.70.x`
 
 ![Available tools](./available-tools.png)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@howaboua/pi-codex-conversion",
-  "version": "1.0.21",
+  "version": "1.0.24",
   "description": "Codex-oriented tool and prompt adapter for pi coding agent",
   "type": "module",
   "repository": {
@@ -51,15 +51,15 @@
     "access": "public"
   },
   "peerDependencies": {
-    "@mariozechner/pi-ai": "^0.69.0",
-    "@mariozechner/pi-coding-agent": "^0.69.0",
-    "@mariozechner/pi-tui": "^0.69.0",
+    "@mariozechner/pi-ai": "^0.70.0",
+    "@mariozechner/pi-coding-agent": "^0.70.0",
+    "@mariozechner/pi-tui": "^0.70.0",
     "typebox": "^1.1.24"
   },
   "devDependencies": {
-    "@mariozechner/pi-ai": "^0.69.0",
-    "@mariozechner/pi-coding-agent": "^0.69.0",
-    "@mariozechner/pi-tui": "^0.69.0",
+    "@mariozechner/pi-ai": "^0.70.0",
+    "@mariozechner/pi-coding-agent": "^0.70.0",
+    "@mariozechner/pi-tui": "^0.70.0",
     "tsx": "^4.20.5",
     "typebox": "^1.1.24",
     "typescript": "^5.9.3"

package/src/providers/openai-codex-custom-provider.ts

@@ -118,7 +118,6 @@ interface ResponsesBody {
 	input: unknown;
 	text: { verbosity: string };
 	include: string[];
-	max_output_tokens?: number;
 	prompt_cache_key?: string;
 	tool_choice: "auto";
 	parallel_tool_calls: boolean;
@@ -481,19 +480,19 @@ function clampReasoningEffort(modelId: string, effort: string): string {
 	return effort;
 }
 
-function getServiceTierCostMultiplier(serviceTier: ServiceTier): number {
+function getServiceTierCostMultiplier(model: Model<Api>, serviceTier: ServiceTier): number {
 	switch (serviceTier) {
 		case "flex":
 			return 0.5;
 		case "priority":
-			return 2;
+			return model.id === "gpt-5.5" ? 2.5 : 2;
 		default:
 			return 1;
 	}
 }
 
-function applyServiceTierPricing(usage: AssistantMessage["usage"], serviceTier: ServiceTier): void {
-	const multiplier = getServiceTierCostMultiplier(serviceTier);
+function applyServiceTierPricing(usage: AssistantMessage["usage"], serviceTier: ServiceTier, model: Model<Api>): void {
+	const multiplier = getServiceTierCostMultiplier(model, serviceTier);
 	if (multiplier === 1) return;
 	usage.cost.input *= multiplier;
 	usage.cost.output *= multiplier;
@@ -527,9 +526,10 @@ function buildRequestBody<TApi extends Api>(model: Model<TApi>, context: Context
 		parallel_tool_calls: true,
 	};
 
-	if (options?.maxTokens !== undefined) {
-		body.max_output_tokens = options.maxTokens;
-	}
+	// The Codex ChatGPT-backed endpoint rejects output-token cap fields with
+	// `Unsupported parameter: max_output_tokens`. Pi's branch summarizer passes
+	// `maxTokens`, so forwarding it breaks `/tree` summaries and extensions that
+	// use `ctx.navigateTree(..., { summarize: true })`.
 
 	if ((options as { temperature?: number } | undefined)?.temperature !== undefined) {
 		body.temperature = (options as { temperature?: number }).temperature;
@@ -848,6 +848,7 @@ async function* parseWebSocket(socket: WebSocketLike, signal: AbortSignal | unde
 	let pending: (() => void) | null = null;
 	let done = false;
 	let failed: Error | null = null;
+	let closeError: Error | null = null;
 	let sawCompletion = false;
 	let pendingMessages = 0;
 	let messageChain = Promise.resolve();
@@ -871,6 +872,7 @@ async function* parseWebSocket(socket: WebSocketLike, signal: AbortSignal | unde
 					const type = typeof parsed.type === "string" ? parsed.type : "";
 					if (type === "response.completed" || type === "response.done" || type === "response.incomplete") {
 						sawCompletion = true;
+						closeError = null;
 						done = true;
 					}
 					queue.push(parsed);
@@ -900,8 +902,8 @@ async function* parseWebSocket(socket: WebSocketLike, signal: AbortSignal | unde
 			wake();
 			return;
 		}
-		if (!failed) {
-			failed = extractWebSocketCloseError(event);
+		if (!closeError) {
+			closeError = extractWebSocketCloseError(event);
 		}
 		done = true;
 		wake();
@@ -934,6 +936,7 @@ async function* parseWebSocket(socket: WebSocketLike, signal: AbortSignal | unde
 		}
 
 		if (failed) throw failed;
+		if (closeError && !sawCompletion) throw closeError;
 		if (!sawCompletion) {
 			throw new Error("WebSocket stream closed before response.completed");
 		}
@@ -1077,7 +1080,7 @@ async function processCapturedResponsesStream<TApi extends Api>(
 	await processResponsesStream(tappedEvents as AsyncIterable<never>, output, stream, model, {
 		serviceTier: (options as { serviceTier?: ServiceTier } | undefined)?.serviceTier,
 		resolveServiceTier: resolveCodexServiceTier,
-		applyServiceTierPricing,
+		applyServiceTierPricing: (usage, serviceTier) => applyServiceTierPricing(usage, serviceTier, model as Model<Api>),
 	});
 }
 

package/src/providers/openai-responses-shared.ts

@@ -613,7 +613,8 @@ export async function processResponsesStream<TApi extends Api>(
 				output.stopReason = "toolUse";
 			}
 		} else if (event.type === "error") {
-			throw new Error(`Error Code ${event.code}: ${event.message}` || "Unknown error");
+			const details = [event.code, event.message].filter(Boolean).join(": ");
+			throw new Error(details || "Unknown error");
 		} else if (event.type === "response.failed") {
 			const error = event.response?.error;
 			const details = (event.response as { incomplete_details?: { reason?: string } } | undefined)?.incomplete_details;