@howaboua/pi-codex-conversion 1.5.4 → 1.5.5-dev.25.f80a775

package/src/codex-settings/ui.ts

@@ -1,65 +1,64 @@
-import { DynamicBorder, getSettingsListTheme, type ExtensionContext } from "@earendil-works/pi-coding-agent";
-import { Box, Container, SettingsList, Spacer, Text, type SettingItem } from "@earendil-works/pi-tui";
-import { DEFAULT_CODEX_CONVERSION_CONFIG, normalizeCodexVerbosity, type CodexConversionConfig } from "../adapter/config.ts";
+import { getSettingsListTheme, type ExtensionContext, type Theme } from "@earendil-works/pi-coding-agent";
+import { SettingsList, truncateToWidth, type SettingItem } from "@earendil-works/pi-tui";
+import {
+	COMPACTION_MODELS,
+	COMPACTION_REASONING_LEVELS,
+	DEFAULT_CODEX_CONVERSION_CONFIG,
+	normalizeCodexVerbosity,
+	normalizeCompactionModel,
+	normalizeCompactionReasoning,
+	type CodexConversionConfig,
+} from "../adapter/config.ts";
 import { CHANGELOG_URL, DISCORD_URL, GITHUB_URL, ISSUE_URL, openExternalUrl } from "./links.ts";
 
 export interface CodexSettingsScreenOptions {
 	initialConfig: CodexConversionConfig;
 	onChange: (nextConfig: CodexConversionConfig) => boolean;
+	initialTab?: SettingsTab;
 }
 
+type SettingsTab = "general" | "compaction";
+
+const TAB_ORDER: readonly SettingsTab[] = ["general", "compaction"];
+
 export async function openCodexSettingsScreen(ctx: ExtensionContext, options: CodexSettingsScreenOptions): Promise<void> {
 	let draft = { ...options.initialConfig };
+	let activeTab: SettingsTab = options.initialTab ?? "general";
+
 	await ctx.ui.custom<void>((tui, theme, _kb, done) => {
-		const buildItems = (): SettingItem[] => [
-			{ id: "useOnAllModels", label: "Use on all models", currentValue: draft.useOnAllModels ? "on" : "off", values: ["off", "on"] },
-			{ id: "fast", label: "Fast mode", currentValue: draft.fast ? "on" : "off", values: ["off", "on"] },
-			{ id: "webSearch", label: "Web search", currentValue: draft.webSearch ? "on" : "off", values: ["off", "on"] },
-			{ id: "imageGeneration", label: "Image generation", currentValue: draft.imageGeneration ? "on" : "off", values: ["off", "on"] },
-			{ id: "verbosity", label: "Verbosity", currentValue: draft.verbosity, values: ["low", "medium", "high"] },
-		];
+		let settingsList = createSettingsList(activeTab, draft, options, (nextDraft) => {
+			draft = nextDraft;
+		}, done, () => tui.requestRender());
 
-		const container = new Container();
-		const panel = new Box(1, 0);
-		panel.addChild(new DynamicBorder((text) => theme.fg("accent", text)));
-		let settingsList: SettingsList;
-		settingsList = new SettingsList(buildItems(), 6, getSettingsListTheme(), (id, value) => {
-			const nextDraft = { ...draft };
-			const previousValue = buildItems().find((item) => item.id === id)?.currentValue;
-			if (id === "useOnAllModels") nextDraft.useOnAllModels = value === "on";
-			if (id === "fast") nextDraft.fast = value === "on";
-			if (id === "webSearch") nextDraft.webSearch = value === "on";
-			if (id === "imageGeneration") nextDraft.imageGeneration = value === "on";
-			if (id === "verbosity") nextDraft.verbosity = normalizeCodexVerbosity(value) ?? DEFAULT_CODEX_CONVERSION_CONFIG.verbosity;
-			if (options.onChange(nextDraft)) {
+		const switchTab = () => {
+			const currentIndex = TAB_ORDER.indexOf(activeTab);
+			activeTab = TAB_ORDER[(currentIndex + 1) % TAB_ORDER.length] ?? "general";
+			settingsList = createSettingsList(activeTab, draft, options, (nextDraft) => {
 				draft = nextDraft;
-			} else if (previousValue !== undefined) {
-				settingsList.updateValue(id, previousValue);
-			}
+			}, done, () => tui.requestRender());
 			tui.requestRender();
-		}, () => done(undefined));
-		panel.addChild(settingsList);
-		panel.addChild(new DynamicBorder((text) => theme.fg("dim", text)));
-		panel.addChild(
-			new Text(
-				[
-					`${theme.bold("g")} github  ${theme.fg("dim", GITHUB_URL)}`,
-					`${theme.bold("c")} changes ${theme.fg("dim", CHANGELOG_URL)}`,
-					`${theme.bold("d")} discord ${theme.fg("dim", DISCORD_URL)}`,
-					`${theme.bold("i")} issue   ${theme.fg("dim", ISSUE_URL)}`,
-				].join("\n"),
-				0,
-				0,
-			),
-		);
-		panel.addChild(new DynamicBorder((text) => theme.fg("accent", text)));
-		container.addChild(new Spacer(1));
-		container.addChild(panel);
+		};
 
 		return {
-			render: (width: number) => container.render(width),
-			invalidate: () => container.invalidate(),
+			render: (width: number) =>
+				[
+					rule(width, theme, "accent"),
+					formatTabs(activeTab, theme),
+					rule(width, theme, "borderMuted"),
+					...(activeTab === "compaction" ? formatCompactionNotes(theme) : []),
+					"",
+					...settingsList.render(width),
+					rule(width, theme, "borderMuted"),
+					...formatLinks(theme),
+					rule(width, theme, "accent"),
+					theme.fg("dim", "  Tab to switch sections · g/c/d/i open links"),
+				].map((line) => truncateToWidth(line, width, "")),
+			invalidate: () => settingsList.invalidate(),
 			handleInput: (data: string) => {
+				if (data === "\t") {
+					switchTab();
+					return;
+				}
 				if (handleLinkKey(data, ctx)) return;
 				settingsList.handleInput?.(data);
 				tui.requestRender();
@@ -68,6 +67,87 @@ 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."),
+	];
+}
+
+function rule(width: number, theme: Theme, color: "accent" | "borderMuted"): string {
+	return theme.fg(color, "─".repeat(Math.max(0, width)));
+}
+
+function createSettingsList(
+	tab: SettingsTab,
+	draft: CodexConversionConfig,
+	options: CodexSettingsScreenOptions,
+	onDraftChanged: (draft: CodexConversionConfig) => void,
+	done: (value?: void) => void,
+	requestRender: () => void,
+): SettingsList {
+	let settingsList: SettingsList;
+	settingsList = new SettingsList(buildItems(tab, draft), 8, getSettingsListTheme(), (id, value) => {
+		const nextDraft = applySettingChange(id, value, draft);
+		const previousValue = buildItems(tab, draft).find((item) => item.id === id)?.currentValue;
+		if (options.onChange(nextDraft)) {
+			onDraftChanged(nextDraft);
+			draft = nextDraft;
+		} else if (previousValue !== undefined) {
+			settingsList.updateValue(id, previousValue);
+		}
+		requestRender();
+	}, () => done(undefined));
+	return settingsList;
+}
+
+function buildItems(tab: SettingsTab, draft: CodexConversionConfig): SettingItem[] {
+	if (tab === "compaction") {
+		return [
+			{ id: "responsesCompaction", label: "Responses compaction", currentValue: (draft.responsesCompaction ?? false) ? "on" : "off", values: ["off", "on"] },
+			{ id: "compactionModel", label: "Model", currentValue: draft.compactionModel, values: [...COMPACTION_MODELS] },
+			{ id: "compactionReasoning", label: "Reasoning", currentValue: draft.compactionReasoning, values: [...COMPACTION_REASONING_LEVELS] },
+		];
+	}
+
+	return [
+		{ id: "useOnAllModels", label: "Use on all models", currentValue: draft.useOnAllModels ? "on" : "off", values: ["off", "on"] },
+		{ id: "statusLine", label: "Statusline", currentValue: draft.statusLine ? "on" : "off", values: ["off", "on"] },
+		{ id: "fast", label: "Fast mode", currentValue: draft.fast ? "on" : "off", values: ["off", "on"] },
+		{ id: "webSearch", label: "Web search", currentValue: draft.webSearch ? "on" : "off", values: ["off", "on"] },
+		{ id: "imageGeneration", label: "Image generation", currentValue: draft.imageGeneration ? "on" : "off", values: ["off", "on"] },
+		{ id: "verbosity", label: "Verbosity", currentValue: draft.verbosity, values: ["low", "medium", "high"] },
+	];
+}
+
+function applySettingChange(id: string, value: string, draft: CodexConversionConfig): CodexConversionConfig {
+	const nextDraft = { ...draft };
+	if (id === "useOnAllModels") nextDraft.useOnAllModels = value === "on";
+	if (id === "statusLine") nextDraft.statusLine = value === "on";
+	if (id === "fast") nextDraft.fast = value === "on";
+	if (id === "webSearch") nextDraft.webSearch = value === "on";
+	if (id === "imageGeneration") nextDraft.imageGeneration = value === "on";
+	if (id === "responsesCompaction") nextDraft.responsesCompaction = value === "on";
+	if (id === "compactionModel") nextDraft.compactionModel = normalizeCompactionModel(value) ?? DEFAULT_CODEX_CONVERSION_CONFIG.compactionModel;
+	if (id === "compactionReasoning") nextDraft.compactionReasoning = normalizeCompactionReasoning(value) ?? DEFAULT_CODEX_CONVERSION_CONFIG.compactionReasoning;
+	if (id === "verbosity") nextDraft.verbosity = normalizeCodexVerbosity(value) ?? DEFAULT_CODEX_CONVERSION_CONFIG.verbosity;
+	return nextDraft;
+}
+
+function formatTabs(activeTab: SettingsTab, theme: Theme): string {
+	const renderTab = (tab: SettingsTab, label: string) => activeTab === tab ? theme.bold(label) : theme.fg("dim", label);
+	return `  ${renderTab("general", "General")}  ${theme.fg("dim", "/")}  ${renderTab("compaction", "Compaction")}`;
+}
+
+function formatLinks(theme: Theme): string[] {
+	return [
+		`${theme.bold("g")} github  ${theme.fg("dim", GITHUB_URL)}`,
+		`${theme.bold("c")} changes ${theme.fg("dim", CHANGELOG_URL)}`,
+		`${theme.bold("d")} discord ${theme.fg("dim", DISCORD_URL)}`,
+		`${theme.bold("i")} issue   ${theme.fg("dim", ISSUE_URL)}`,
+	];
+}
+
 function handleLinkKey(data: string, ctx: ExtensionContext): boolean {
 	const target = getLinkTarget(data);
 	if (!target) return false;

package/src/index.ts

@@ -1,23 +1,23 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import { Box, Text, truncateToWidth } from "@earendil-works/pi-tui";
 import { getCodexRuntimeShell } from "./adapter/runtime-shell.ts";
 import { clearApplyPatchRenderState, registerApplyPatchTool } from "./tools/apply-patch-tool.ts";
 import { createExecCommandTracker } from "./tools/exec-command-state.ts";
 import { registerExecCommandTool } from "./tools/exec-command-tool.ts";
 import { createExecSessionManager } from "./tools/exec-session-manager.ts";
-import {
-	IMAGE_SAVE_DISPLAY_MESSAGE_TYPE,
-	WEB_SEARCH_ACTIVITY_MESSAGE_TYPE,
-	registerOpenAICodexCustomProvider,
-} from "./providers/openai-codex-custom-provider.ts";
+import { registerOpenAICodexCustomProvider } from "./providers/openai-codex-custom-provider.ts";
 import { registerImageGenerationTool } from "./tools/image-generation-tool.ts";
 import { buildCodexSystemPrompt, extractPiPromptSkills, resolvePromptSkills } from "./prompt/build-system-prompt.ts";
 import { registerViewImageTool, supportsOriginalImageDetail } from "./tools/view-image-tool.ts";
-import { registerWebSearchTool, WEB_SEARCH_SESSION_NOTE_TYPE } from "./tools/web-search-tool.ts";
+import { registerWebSearchTool } from "./tools/web-search-tool.ts";
 import { registerWriteStdinTool } from "./tools/write-stdin-tool.ts";
 import { ensureBundledApplyPatchOnPath } from "./tools/apply-patch-binary.ts";
 import { readCodexConversionConfig } from "./adapter/config.ts";
 import { syncAdapter, mergeAdapterTools, restoreTools, stripAdapterTools, shouldUseCodexAdapter } from "./adapter/activation.ts";
 import { rewriteCodexProviderRequest } from "./adapter/provider-request.ts";
+import { handleCodexSessionBeforeCompact } from "./adapter/compaction.ts";
+import { isNativeCompactionDetails, NATIVE_COMPACTION_DISPLAY_MESSAGE_TYPE, NATIVE_COMPACTION_DISPLAY_TEXT } from "./adapter/types.ts";
+import { isAdapterContextExcludedCustomMessage } from "./adapter/context-filter.ts";
 import { getCodexSkillPaths } from "./adapter/skills.ts";
 import type { AdapterState } from "./adapter/state.ts";
 import { registerCodexCommand } from "./codex-settings/command.ts";
@@ -44,6 +44,19 @@ export default function codexConversion(pi: ExtensionAPI) {
 	const tracker = createExecCommandTracker();
 	const state: AdapterState = { enabled: false, cwd: process.cwd(), promptSkills: [], config: readCodexConversionConfig() };
 	const sessions = createExecSessionManager();
+	let nativeWebSearchRegistered = false;
+	let nativeImageGenerationRegistered = false;
+
+	function ensureOptionalNativeToolsRegistered(config = state.config): void {
+		if (config.webSearch && !nativeWebSearchRegistered) {
+			registerWebSearchTool(pi);
+			nativeWebSearchRegistered = true;
+		}
+		if (config.imageGeneration && !nativeImageGenerationRegistered) {
+			registerImageGenerationTool(pi);
+			nativeImageGenerationRegistered = true;
+		}
+	}
 
 	registerOpenAICodexCustomProvider(pi, {
 		getCurrentCwd: () => state.cwd,
@@ -51,9 +64,18 @@ export default function codexConversion(pi: ExtensionAPI) {
 	registerApplyPatchTool(pi);
 	registerExecCommandTool(pi, tracker, sessions);
 	registerWriteStdinTool(pi, sessions);
-	registerImageGenerationTool(pi);
-	registerWebSearchTool(pi);
-	registerCodexCommand(pi, state);
+	ensureOptionalNativeToolsRegistered();
+	registerCodexCommand(pi, state, ensureOptionalNativeToolsRegistered);
+
+	pi.registerMessageRenderer(NATIVE_COMPACTION_DISPLAY_MESSAGE_TYPE, (message, _options, theme) => {
+		const box = new Box(1, 1, (text) => theme.bg("customMessageBg", text));
+		box.addChild(new Text(theme.fg("customMessageLabel", theme.bold("[compaction]")), 0, 0));
+		const content = typeof message.content === "string" ? message.content : NATIVE_COMPACTION_DISPLAY_TEXT;
+		box.addChild(new Text(`\n${theme.fg("customMessageText", content)}`, 0, 0));
+		const render = box.render.bind(box);
+		box.render = (width) => render(width).map((line) => truncateToWidth(line, width, ""));
+		return box;
+	});
 
 	sessions.onSessionExit((sessionId) => {
 		tracker.recordSessionFinished(sessionId);
@@ -62,6 +84,7 @@ export default function codexConversion(pi: ExtensionAPI) {
 	pi.on("session_start", async (_event, ctx) => {
 		state.cwd = ctx.cwd;
 		state.config = readCodexConversionConfig();
+		ensureOptionalNativeToolsRegistered();
 		state.promptSkills = extractPiPromptSkills(ctx.getSystemPrompt());
 		registerViewImageTool(pi, { allowOriginalDetail: supportsOriginalImageDetail(ctx.model) });
 		clearApplyPatchRenderState();
@@ -125,19 +148,25 @@ export default function codexConversion(pi: ExtensionAPI) {
 		return rewriteCodexProviderRequest(event.payload, ctx, state);
 	});
 
-	pi.on("context", async (event) => {
-		return {
-			messages: event.messages.filter(
-				(message) =>
-					!(
-						message.role === "custom" &&
-						(message.customType === WEB_SEARCH_SESSION_NOTE_TYPE ||
-							message.customType === WEB_SEARCH_ACTIVITY_MESSAGE_TYPE ||
-							message.customType === IMAGE_SAVE_DISPLAY_MESSAGE_TYPE)
-					),
-			),
-		};
+	pi.on("session_before_compact", async (event, ctx) => {
+		state.cwd = ctx.cwd;
+		return handleCodexSessionBeforeCompact(event, ctx, state, pi);
 	});
+
+	pi.on("session_compact", async (event) => {
+		if (!event.fromExtension || !isNativeCompactionDetails(event.compactionEntry.details)) return;
+		pi.sendMessage(
+			{
+				customType: NATIVE_COMPACTION_DISPLAY_MESSAGE_TYPE,
+				content: NATIVE_COMPACTION_DISPLAY_TEXT,
+				display: true,
+				details: { compactionEntryId: event.compactionEntry.id },
+			},
+			{ triggerTurn: false },
+		);
+	});
+
+	pi.on("context", async (event) => ({ messages: event.messages.filter((message) => !isAdapterContextExcludedCustomMessage(message)) }));
 }
 
 export { getCodexSkillPaths, mergeAdapterTools, restoreTools, stripAdapterTools };

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

@@ -17,6 +17,7 @@ import type { ResponseCreateParamsStreaming } from "openai/resources/responses/r
 import {
 	convertResponsesMessages,
 	convertResponsesTools,
+	CODEX_TOOL_CALL_PROVIDERS,
 	processResponsesStream,
 } from "./openai-responses-shared.ts";
 
@@ -28,7 +29,6 @@ const OPENAI_CODEX_IMAGE_DIR = ".pi/openai-codex-images";
 const OPENAI_CODEX_LATEST_IMAGE_NAME = "latest.png";
 const MAX_RETRIES = 3;
 const BASE_DELAY_MS = 1000;
-const CODEX_TOOL_CALL_PROVIDERS = new Set(["openai", "openai-codex", "opencode"]);
 const CODEX_RESPONSE_STATUSES = new Set(["completed", "incomplete", "failed", "cancelled", "queued", "in_progress"]);
 const OPENAI_BETA_RESPONSES_WEBSOCKETS = "responses_websockets=2026-02-06";
 const WEBSOCKET_MESSAGE_TOO_BIG_CLOSE_CODE = 1009;

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

@@ -40,6 +40,8 @@ interface ConvertResponsesToolsOptions {
 	strict?: boolean | null;
 }
 
+export const CODEX_TOOL_CALL_PROVIDERS = new Set(["openai", "openai-codex", "opencode"]);
+
 function shortHash(str: string): string {
 	let h1 = 0xdeadbeef;
 	let h2 = 0x41c6ce57;