@f5-sales-demo/pi-ai 21.15.0 → 21.16.0

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@f5-sales-demo/pi-ai",
-	"version": "21.15.0",
+	"version": "21.16.0",
 	"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.115",
 		"@aws-sdk/client-bedrock-runtime": "^3",
 		"@bufbuild/protobuf": "^2.11",
-		"@f5-sales-demo/pi-utils": "21.15.0",
+		"@f5-sales-demo/pi-utils": "21.16.0",
 		"@google/genai": "^2.15",
 		"@sinclair/typebox": "^0.34",
 		"@smithy/node-http-handler": "^4.4",

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

@@ -63,7 +63,12 @@ import {
 	transformRequestBody,
 } from "./openai-codex/request-transformer";
 import { parseCodexError } from "./openai-codex/response-handler";
-import { encodeTextSignatureV1, mapOpenAIResponsesStopReason, parseTextSignature } from "./openai-responses-shared";
+import {
+	encodeTextSignatureV1,
+	mapOpenAIResponsesStopReason,
+	parseTextSignature,
+	resolveAssistantMessagePhase,
+} from "./openai-responses-shared";
 import { transformMessages } from "./transform-messages";
 
 export interface OpenAICodexResponsesOptions extends StreamOptions {
@@ -747,11 +752,21 @@ function handleCodexStreamEvent(args: {
 		runtime.currentBlock = createOutputBlockForItem(item);
 		if (!runtime.currentBlock) return firstTokenTime;
 		output.content.push(runtime.currentBlock);
-		stream.push({
-			type: getOutputBlockStartEventType(runtime.currentBlock),
-			contentIndex: blockIndex(),
-			partial: output,
-		});
+		if (runtime.currentBlock.type === "text") {
+			stream.push({
+				type: "text_start",
+				contentIndex: blockIndex(),
+				phase: runtime.currentBlock.phase ?? "final_answer",
+				partial: output,
+			});
+		} else {
+			const type = getOutputBlockStartEventType(runtime.currentBlock);
+			if (type === "thinking_start") {
+				stream.push({ type, contentIndex: blockIndex(), partial: output });
+			} else {
+				stream.push({ type: "toolcall_start", contentIndex: blockIndex(), partial: output });
+			}
+		}
 		return firstTokenTime;
 	}
 
@@ -838,7 +853,7 @@ function createOutputBlockForItem(item: CodexEventItem): CodexOutputBlock | null
 		return { type: "thinking", thinking: "" };
 	}
 	if (item.type === "message") {
-		return { type: "text", text: "" };
+		return { type: "text", text: "", phase: resolveAssistantMessagePhase(item.phase) };
 	}
 	if (item.type === "function_call") {
 		return {
@@ -986,12 +1001,14 @@ function handleOutputItemDone(
 		runtime.currentBlock.text = item.content
 			.map(content => (content.type === "output_text" ? content.text : content.refusal))
 			.join("");
-		const phase = item.phase === "commentary" || item.phase === "final_answer" ? item.phase : undefined;
+		const phase = resolveAssistantMessagePhase(item.phase);
+		runtime.currentBlock.phase = phase;
 		runtime.currentBlock.textSignature = encodeTextSignatureV1(item.id, phase);
 		stream.push({
 			type: "text_end",
 			contentIndex: blockIndex(),
 			content: runtime.currentBlock.text,
+			phase,
 			partial: output,
 		});
 		runtime.currentBlock = null;
@@ -2175,7 +2192,7 @@ function convertMessages(model: Model<"openai-codex-responses">, context: Contex
 						content: [{ type: "output_text", text: textBlock.text.toWellFormed(), annotations: [] }],
 						status: "completed",
 						id: msgId,
-						phase: parsedSignature?.phase,
+						phase: parsedSignature?.phase ?? textBlock.phase,
 					} satisfies ResponseOutputMessage);
 					continue;
 				}

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

@@ -14,6 +14,7 @@ import { calculateCost } from "../models";
 import type {
 	Api,
 	AssistantMessage,
+	AssistantMessagePhase,
 	ImageContent,
 	Model,
 	StopReason,
@@ -33,6 +34,10 @@ export function encodeTextSignatureV1(id: string, phase?: TextSignatureV1["phase
 	return JSON.stringify(payload);
 }
 
+export function resolveAssistantMessagePhase(phase: unknown): AssistantMessagePhase {
+	return phase === "commentary" ? "commentary" : "final_answer";
+}
+
 export function parseTextSignature(
 	signature: string | undefined,
 ): { id: string; phase?: TextSignatureV1["phase"] } | undefined {
@@ -152,7 +157,7 @@ export function convertResponsesAssistantMessage<TApi extends Api>(
 				content: [{ type: "output_text", text: block.text.toWellFormed(), annotations: [] }],
 				status: "completed",
 				id: msgId,
-				phase: parsedSignature?.phase,
+				phase: parsedSignature?.phase ?? block.phase,
 			} satisfies ResponseOutputMessage);
 			continue;
 		}
@@ -262,9 +267,14 @@ export async function processResponsesStream<TApi extends Api>(
 				stream.push({ type: "thinking_start", contentIndex: blockIndex(), partial: output });
 			} else if (item.type === "message") {
 				currentItem = item;
-				currentBlock = { type: "text", text: "" };
+				currentBlock = { type: "text", text: "", phase: resolveAssistantMessagePhase(item.phase) };
 				output.content.push(currentBlock);
-				stream.push({ type: "text_start", contentIndex: blockIndex(), partial: output });
+				stream.push({
+					type: "text_start",
+					contentIndex: blockIndex(),
+					phase: currentBlock.phase ?? "final_answer",
+					partial: output,
+				});
 			} else if (item.type === "function_call") {
 				currentItem = item;
 				currentBlock = {
@@ -380,11 +390,13 @@ export async function processResponsesStream<TApi extends Api>(
 				currentBlock.text = item.content
 					.map(part => (part.type === "output_text" ? (part.text ?? "") : (part.refusal ?? "")))
 					.join("");
-				currentBlock.textSignature = encodeTextSignatureV1(item.id, item.phase ?? undefined);
+				currentBlock.phase = resolveAssistantMessagePhase(item.phase);
+				currentBlock.textSignature = encodeTextSignatureV1(item.id, currentBlock.phase);
 				stream.push({
 					type: "text_end",
 					contentIndex: blockIndex(),
 					content: currentBlock.text,
+					phase: currentBlock.phase ?? "final_answer",
 					partial: output,
 				});
 				currentBlock = null;

package/src/types.ts

@@ -258,10 +258,12 @@ export type StreamFunction<TApi extends Api> = (
 	options: OptionsForApi<TApi>,
 ) => AssistantMessageEventStream;
 
+export type AssistantMessagePhase = "commentary" | "final_answer";
+
 export interface TextSignatureV1 {
 	v: 1;
 	id: string;
-	phase?: "commentary" | "final_answer";
+	phase?: AssistantMessagePhase;
 }
 
 /**
@@ -288,6 +290,8 @@ export interface WebCitation {
 export interface TextContent {
 	type: "text";
 	text: string;
+	/** Semantic role supplied by Responses. Absence has final-answer semantics. */
+	phase?: AssistantMessagePhase;
 	textSignature?: string; // e.g., for OpenAI responses, message metadata (legacy id string or TextSignatureV1 JSON)
 	/** Structured citations for this span, when the provider ran a server-side search. */
 	citations?: WebCitation[];
@@ -468,9 +472,15 @@ export interface Context {
 
 export type AssistantMessageEvent =
 	| { type: "start"; contentIndex?: undefined; partial: AssistantMessage }
-	| { type: "text_start"; contentIndex: number; partial: AssistantMessage }
+	| { type: "text_start"; contentIndex: number; phase?: AssistantMessagePhase; partial: AssistantMessage }
 	| { type: "text_delta"; contentIndex: number; delta: string; partial: AssistantMessage }
-	| { type: "text_end"; contentIndex: number; content: string; partial: AssistantMessage }
+	| {
+			type: "text_end";
+			contentIndex: number;
+			content: string;
+			phase?: AssistantMessagePhase;
+			partial: AssistantMessage;
+	  }
 	| { type: "thinking_start"; contentIndex: number; partial: AssistantMessage }
 	| { type: "thinking_delta"; contentIndex: number; delta: string; partial: AssistantMessage }
 	| { type: "thinking_end"; contentIndex: number; content: string; partial: AssistantMessage }