@howaboua/pi-codex-conversion 1.5.11-dev.45.7d9b450 → 1.5.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@howaboua/pi-codex-conversion",
-  "version": "1.5.11-dev.45.7d9b450",
+  "version": "1.5.11",
   "description": "Codex-oriented tool and prompt adapter for pi coding agent",
   "type": "module",
   "repository": {

package/src/adapter/compaction.ts

@@ -142,6 +142,28 @@ function notifyNativeCompactionFallback(ctx: ExtensionContext, state: AdapterSta
 	ctx.ui.notify(`${message}; Pi compaction will run.${stashed ? " Previous native compacted window will be included in Pi compaction fallback." : ""}`, "error");
 }
 
+function textFromResponsesContent(content: unknown): string {
+	if (typeof content === "string") return content;
+	if (!Array.isArray(content)) return "";
+	return content
+		.map((item) => isRecord(item) && item.type === "input_text" && typeof item.text === "string" ? item.text : "")
+		.join("\n");
+}
+
+function isPiCompactionSummarizationPayload(payload: ResponsesCompatibleRequestPayload): boolean {
+	const instructions = typeof payload.instructions === "string" ? payload.instructions : "";
+	if (/compact|summar/i.test(instructions)) return true;
+
+	return payload.input.some((item) => {
+		if (!isRecord(item)) return false;
+		const role = item.role;
+		const text = textFromResponsesContent(item.content);
+		if ((role === "system" || role === "developer") && /compact|summar/i.test(text)) return true;
+		if (role === "user" && /<conversation>|previous compaction summary|summary/i.test(text)) return true;
+		return false;
+	});
+}
+
 export async function handleCodexSessionBeforeCompact(event: SessionBeforeCompactEvent, ctx: ExtensionContext, state: AdapterState, pi: ExtensionAPI) {
 	if (!state.config.responsesCompaction || !shouldUseCodexAdapter(ctx, state.config)) {
 		return undefined;
@@ -165,6 +187,9 @@ async function handleCodexSessionBeforeCompactInner(event: SessionBeforeCompactE
 
 	const resolution = await resolveNativeCompactionEnvironment(ctx, { enabled: true });
 	if (!resolution.ok) {
+		if (resolution.reason === "unsupported-provider" || resolution.reason === "unsupported-api") {
+			return undefined;
+		}
 		ctx.ui.notify(`OpenAI native compaction is enabled but unavailable (${resolution.reason}); Pi compaction was not run.`, "error");
 		return { cancel: true };
 	}
@@ -306,11 +331,15 @@ export async function injectPendingNativeWindowIntoPiCompactionRequest(payload:
 		state.pendingPiCompactionNativeWindow = undefined;
 		return undefined;
 	}
+	if (!isPiCompactionSummarizationPayload(payload)) return undefined;
 
 	const resolution = await resolveNativeCompactionEnvironment(ctx, { enabled: true }, payload);
 	if (!resolution.ok) return undefined;
 	const runtime = resolution.runtime;
-	if (pending.provider !== runtime.provider || pending.api !== runtime.api || pending.baseUrl !== runtime.baseUrl) return undefined;
+	if (pending.provider !== runtime.provider || pending.api !== runtime.api || pending.baseUrl !== runtime.baseUrl) {
+		state.pendingPiCompactionNativeWindow = undefined;
+		return undefined;
+	}
 
 	const input = [...payload.input];
 	let insertAt = 0;
@@ -320,6 +349,7 @@ export async function injectPendingNativeWindowIntoPiCompactionRequest(payload:
 		insertAt++;
 	}
 
+	state.pendingPiCompactionNativeWindow = undefined;
 	return {
 		...payload,
 		input: [

package/src/adapter/payload-rewrite.ts

@@ -390,6 +390,15 @@ function createReplayVariants<TApi extends Api>(args: {
 	return [contextSet, createReplayMessageSet(args.model, piMessages)];
 }
 
+function clonePayloadConversationInput(args: {
+	payloadInput: readonly unknown[];
+	freshPreamble: FreshAuthoritativePreamble;
+}): ResponsesInputItem[] | undefined {
+	const tailEndIndex = args.payloadInput.length - args.freshPreamble.trailingInput.length;
+	if (tailEndIndex < args.freshPreamble.leadingInput.length) return undefined;
+	return cloneResponsesInputSlice(args.payloadInput.slice(args.freshPreamble.leadingInput.length, tailEndIndex));
+}
+
 function findReplayMatch<TApi extends Api>(args: {
 	model: Model<TApi>;
 	payloadInput: readonly unknown[];
@@ -506,21 +515,57 @@ function buildNativeReplaySegmentsInternal<TApi extends Api>(args: {
 		};
 	}
 
-	const newerCompactionEntry = args.branchEntries
-		.slice(boundaryIndex + 1)
-		.some((entry) => entry.type === "compaction");
-	if (newerCompactionEntry) {
+	const compactedWindow = cloneOpaqueCompactedWindow(args.compactionEntry.details?.compactedWindow ?? []);
+	if (!compactedWindow) {
 		return {
 			ok: false,
-			reason: "unexpected-compaction-after-boundary",
+			reason: "invalid-compacted-window",
 		};
 	}
 
-	const compactedWindow = cloneOpaqueCompactedWindow(args.compactionEntry.details?.compactedWindow ?? []);
-	if (!compactedWindow) {
+	const newerCompactionEntry = args.branchEntries
+		.slice(boundaryIndex + 1)
+		.some((entry) => entry.type === "compaction");
+	if (newerCompactionEntry) {
+		const conversationInput = clonePayloadConversationInput({ payloadInput: args.payload.input, freshPreamble });
+		const originalPiReplayInput = cloneResponsesInputSlice(args.payload.input);
+		if (!conversationInput || !originalPiReplayInput) {
+			return {
+				ok: false,
+				reason: "unexpected-compaction-after-boundary",
+			};
+		}
+
 		return {
-			ok: false,
-			reason: "invalid-compacted-window",
+			ok: true,
+			segments: {
+				boundaryIndex,
+				firstKeptEntryIndex,
+				instructions: freshPreamble.instructions,
+				freshPreamble: freshPreamble.leadingInput,
+				trailingPreamble: freshPreamble.trailingInput,
+				compactionSummary: [],
+				preCompactionKeptWindow: createReplaySlice([], [], []),
+				compactedWindow,
+				postCompactionTail: createReplaySlice(args.branchEntries.slice(boundaryIndex + 1), [], conversationInput),
+				originalPiReplayInput,
+				replayInput: [
+					...freshPreamble.leadingInput,
+					...compactedWindow,
+					...conversationInput,
+					...freshPreamble.trailingInput,
+				],
+			},
+			rewrittenPayload: {
+				...args.payload,
+				...(freshPreamble.instructions !== undefined ? { instructions: freshPreamble.instructions } : {}),
+				input: [
+					...freshPreamble.leadingInput,
+					...compactedWindow,
+					...conversationInput,
+					...freshPreamble.trailingInput,
+				],
+			},
 		};
 	}
 

package/src/adapter/types.ts

@@ -9,7 +9,7 @@ export const NATIVE_COMPACTION_DISPLAY_TEXT = [
 	"",
 	"The compaction result is encrypted by OpenAI and is not human-readable in Pi.",
 	"",
-	"Warning: do not turn Responses compaction off mid-session; old context may be much less reliable.",
+	"Warning: do not turn Responses compaction off or switch providers mid-session; old context may be much less reliable.",
 ].join("\n");
 
 export type NativeCompactionStrategy = typeof NATIVE_COMPACTION_STRATEGY;

package/src/codex-settings/ui.ts

@@ -92,7 +92,8 @@ export async function openCodexSettingsScreen(ctx: ExtensionContext, options: Co
 function formatCompactionNotes(theme: Theme): string[] {
 	return [
 		theme.fg("dim", "  Beta: native OpenAI Responses compaction is experimental. Please report any issues."),
-		theme.fg("error", "  Warning: do not turn this off mid-session; old context may be much less reliable."),
+		theme.fg("error", "  Warning: do not turn this off or switch providers mid-session; old context may be much less reliable."),
+		theme.fg("warning", "  If native compaction recovery fails, go back below 90% context and compact from there."),
 	];
 }