@f5-sales-demo/pi-ai 21.41.3 → 21.41.5

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@f5-sales-demo/pi-ai",
-	"version": "21.41.3",
+	"version": "21.41.5",
 	"description": "Unified LLM API with automatic model discovery and provider configuration",
 	"homepage": "https://github.com/f5-sales-demo/xcsh",
 	"author": "Can Boluk",
@@ -46,7 +46,7 @@
 		"@anthropic-ai/sdk": "0.123.0",
 		"@aws-sdk/client-bedrock-runtime": "3.1126.0",
 		"@bufbuild/protobuf": "2.14.1",
-		"@f5-sales-demo/pi-utils": "21.41.3",
+		"@f5-sales-demo/pi-utils": "21.41.5",
 		"@google/genai": "2.21.0",
 		"@sinclair/typebox": "0.34.52",
 		"@smithy/node-http-handler": "4.12.1",

package/src/providers/openai-codex/response-handler.ts

@@ -14,6 +14,7 @@ export type CodexRateLimits = {
 export type CodexErrorInfo = {
 	message: string;
 	status: number;
+	code?: string;
 	friendlyMessage?: string;
 	rateLimits?: CodexRateLimits;
 	raw?: string;
@@ -24,6 +25,7 @@ export async function parseCodexError(response: Response): Promise<CodexErrorInf
 	let message = raw || response.statusText || "Request failed";
 	let friendlyMessage: string | undefined;
 	let rateLimits: CodexRateLimits | undefined;
+	let code: string | undefined;
 
 	try {
 		const parsed = JSON.parse(raw) as { error?: Record<string, unknown> };
@@ -45,16 +47,17 @@ export async function parseCodexError(response: Response): Promise<CodexErrorInf
 				? { primary, secondary }
 				: undefined;
 
-		const code = String((err as { code?: string; type?: string }).code ?? (err as { type?: string }).type ?? "");
+		code =
+			String((err as { code?: string; type?: string }).code ?? (err as { type?: string }).type ?? "") || undefined;
 		const resetsAt = (err as { resets_at?: number }).resets_at ?? primary.resets_at ?? secondary.resets_at;
 		const mins = resetsAt ? Math.max(0, Math.round((resetsAt * 1000 - Date.now()) / 60000)) : undefined;
 
-		if (/usage_limit_reached|usage_not_included/i.test(code)) {
+		if (/usage_limit_reached|usage_not_included/i.test(code ?? "")) {
 			const planType = (err as { plan_type?: string }).plan_type;
 			const plan = planType ? ` (${String(planType).toLowerCase()} plan)` : "";
 			const when = mins !== undefined ? ` Try again in ~${mins} min.` : "";
 			friendlyMessage = `You have hit your ChatGPT usage limit${plan}.${when}`.trim();
-		} else if (/rate_limit_exceeded/i.test(code) || response.status === 429) {
+		} else if (/rate_limit_exceeded/i.test(code ?? "") || response.status === 429) {
 			const when = mins !== undefined ? ` Try again in ~${mins} min.` : "";
 			friendlyMessage = `ChatGPT rate limit exceeded.${when}`.trim();
 		}
@@ -68,6 +71,7 @@ export async function parseCodexError(response: Response): Promise<CodexErrorInf
 	return {
 		message,
 		status: response.status,
+		...(code ? { code } : {}),
 		friendlyMessage,
 		rateLimits,
 		raw: raw,

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

@@ -401,6 +401,7 @@ function resetOutputState(output: AssistantMessage): void {
 	output.content.length = 0;
 	output.usage = createEmptyUsage();
 	output.stopReason = "stop";
+	delete output.providerFailureCode;
 }
 
 function removeTransientBlockIndices(output: AssistantMessage): void {
@@ -1325,6 +1326,11 @@ async function handleCodexStreamFailure(
 	}
 	output.stopReason = context.options?.signal?.aborted ? "aborted" : "error";
 	output.errorMessage = await finalizeErrorMessage(error, context.requestContext.rawRequestDump);
+	output.providerFailureCode = sanitizeProviderFailureCode(
+		error instanceof CodexProviderStreamError
+			? error.code
+			: (error as { providerFailureCode?: unknown } | null)?.providerFailureCode,
+	);
 	output.duration = Date.now() - context.startTime;
 	if (context.firstTokenTime) {
 		output.ttft = context.firstTokenTime - context.startTime;
@@ -1937,8 +1943,10 @@ async function openCodexSseEventStream(
 	if (!response.ok) {
 		const info = await parseCodexError(response);
 		const error = new Error(info.friendlyMessage || info.message);
-		(error as { headers?: Headers; status?: number }).headers = response.headers;
-		(error as { headers?: Headers; status?: number }).status = response.status;
+		const details = error as { headers?: Headers; status?: number; providerFailureCode?: string };
+		details.headers = response.headers;
+		details.status = response.status;
+		details.providerFailureCode = sanitizeProviderFailureCode(info.code);
 		throw error;
 	}
 	if (!response.body) {
@@ -2301,6 +2309,10 @@ function getString(value: unknown): string | undefined {
 	return typeof value === "string" ? value : undefined;
 }
 
+function sanitizeProviderFailureCode(value: unknown): string | undefined {
+	return typeof value === "string" && value.length <= 128 && /^[A-Za-z0-9._-]+$/.test(value) ? value : undefined;
+}
+
 class CodexProviderStreamError extends Error {
 	readonly retryable: boolean;
 	readonly code?: string;
@@ -2309,7 +2321,7 @@ class CodexProviderStreamError extends Error {
 		super(message);
 		this.name = "CodexProviderStreamError";
 		this.retryable = retryable;
-		this.code = code;
+		this.code = sanitizeProviderFailureCode(code);
 	}
 }
 
@@ -2325,7 +2337,9 @@ function isRetryableCodexFailureEvent(rawEvent: Record<string, unknown>): boolea
 }
 
 function createCodexProviderStreamError(rawEvent: Record<string, unknown>): CodexProviderStreamError {
-	const code = getString(rawEvent.code) ?? "";
+	const response = asRecord(rawEvent.response);
+	const providerError = asRecord(rawEvent.error) ?? (response ? asRecord(response.error) : null);
+	const code = getString(providerError?.code) ?? getString(providerError?.type) ?? getString(rawEvent.code) ?? "";
 	const message = getString(rawEvent.message) ?? "";
 	const formattedMessage =
 		typeof rawEvent.type === "string" && rawEvent.type === "error"

package/src/types.ts

@@ -393,6 +393,8 @@ export interface AssistantMessage {
 	usage: Usage;
 	stopReason: StopReason;
 	errorMessage?: string;
+	/** Sanitized provider error identifier used for internal lifecycle classification. */
+	providerFailureCode?: string;
 	/** Provider-native completion reason retained for diagnostics and transport-specific recovery. */
 	rawStopReason?: string;
 	/** Provider-specific opaque payload used to reconstruct transport-native history. */