pi-openai-codex-compat 0.0.1 → 0.0.3

package/extensions/openai-codex-compat/settings-pane.ts

@@ -10,11 +10,15 @@ import {
   type SettingItem,
   SettingsList,
   Text,
+  truncateToWidth,
 } from "@earendil-works/pi-tui";
 import {
+  CONFIG_ENVIRONMENT_VARIABLES,
   configLayer,
   loadConfig,
+  parseEnvironmentConfig,
   saveConfig,
+  withoutEnvironmentOverrides,
   writableConfigPath,
   type CodexCompatConfig,
   type ConfigLayer,
@@ -48,8 +52,11 @@ function thresholdValue(value: number | undefined): string {
   return value === undefined ? "Pi default" : `${value}%`;
 }
 
-export function settingItems(config: CodexCompatConfig): SettingItem[] {
-  return [
+export function settingItems(
+  config: CodexCompatConfig,
+  environmentConfig: ConfigLayer = {},
+): SettingItem[] {
+  const items: SettingItem[] = [
     {
       id: "fastMode",
       label: "Fast mode",
@@ -130,6 +137,18 @@ export function settingItems(config: CodexCompatConfig): SettingItem[] {
       values: ["Pi default", "75%", "80%", "85%", "90%", "95%"],
     },
   ];
+
+  return items.map((item) => {
+    const id = item.id as SettingId;
+    if (!Object.hasOwn(environmentConfig, id)) return item;
+
+    const variable = CONFIG_ENVIRONMENT_VARIABLES[id];
+    const lockedItem = { ...item };
+    lockedItem.currentValue = `${item.currentValue} (env)`;
+    lockedItem.description = `${item.description} Locked by ${variable}.`;
+    delete lockedItem.values;
+    return lockedItem;
+  });
 }
 
 export function settingPatch(id: string, value: string): ConfigLayer | undefined {
@@ -212,18 +231,21 @@ async function showSettings(
   }
 
   let config = callbacks.getConfig?.(ctx) ?? loadConfig(ctx.cwd, ctx.isProjectTrusted());
+  let persistedConfig = { ...config };
   let revision = 0;
   let savedRevision = 0;
   let saveQueue = Promise.resolve();
   const filePath = writableConfigPath(ctx.cwd, ctx.isProjectTrusted());
+  const environmentConfig = parseEnvironmentConfig();
 
-  await ctx.ui.custom((tui, theme, _keybindings, done) => {
+  await ctx.ui.custom((tui, theme, keybindings, done) => {
+    let closing = false;
     const container = new Container();
     container.addChild(
       new Text(theme.fg("accent", theme.bold("OpenAI Codex Compatibility Settings")), 1, 1),
     );
     container.addChild(
-      new Text(theme.fg("dim", `Session-only. Ctrl+S saves to ${filePath}`), 1, 0),
+      new Text(theme.fg("dim", `Ctrl+S saves without closing to ${filePath}`), 1, 0),
     );
     const saveStatus = new Text(
       theme.fg("dim", "Changes apply to this session immediately."),
@@ -232,10 +254,61 @@ async function showSettings(
     );
     container.addChild(saveStatus);
 
+    const queueSave = (closeAfterSave: boolean): void => {
+      if (closing) return;
+      if (closeAfterSave) closing = true;
+
+      const snapshotConfig = { ...config };
+      const snapshot = withoutEnvironmentOverrides(configLayer(snapshotConfig), environmentConfig);
+      const snapshotRevision = revision;
+      saveStatus.setText(theme.fg("dim", closeAfterSave ? "Saving and closing…" : "Saving…"));
+      saveQueue = saveQueue.then(async () => {
+        try {
+          const savedPath = await saveConfig(ctx.cwd, ctx.isProjectTrusted(), snapshot);
+          if (snapshotRevision >= savedRevision) {
+            persistedConfig = snapshotConfig;
+            savedRevision = snapshotRevision;
+          }
+          if (closeAfterSave) {
+            done(undefined);
+            return;
+          }
+          saveStatus.setText(
+            savedRevision === revision
+              ? theme.fg("success", `Saved to ${savedPath}`)
+              : theme.fg("warning", "Unsaved session changes."),
+          );
+        } catch (error) {
+          if (closeAfterSave) closing = false;
+          const message = error instanceof Error ? error.message : String(error);
+          saveStatus.setText(theme.fg("error", message));
+          ctx.ui.notify(message, "error");
+        }
+        tui.requestRender();
+      });
+      tui.requestRender();
+    };
+
+    const discardAndClose = (): void => {
+      if (closing) return;
+      closing = true;
+      saveStatus.setText(theme.fg("dim", "Discarding unsaved changes…"));
+      saveQueue = saveQueue.then(() => {
+        if (revision !== savedRevision) {
+          config = { ...persistedConfig };
+          revision = savedRevision;
+          callbacks.onChange?.(config, ctx);
+        }
+        done(undefined);
+      });
+      tui.requestRender();
+    };
+
+    const listTheme = getSettingsListTheme();
     const list = new SettingsList(
-      settingItems(config),
+      settingItems(config, environmentConfig),
       12,
-      getSettingsListTheme(),
+      listTheme,
       (id, value) => {
         const patch = settingPatch(id, value);
         if (!patch) return;
@@ -246,36 +319,42 @@ async function showSettings(
         saveStatus.setText(theme.fg("warning", "Unsaved session changes."));
         tui.requestRender();
       },
-      () => done(undefined),
+      discardAndClose,
       { enableSearch: true },
     );
-    container.addChild(list);
+    container.addChild({
+      render: (width: number) => {
+        const lines = list.render(width);
+        if (lines.length > 0) {
+          lines[lines.length - 1] = truncateToWidth(
+            listTheme.hint(
+              "  Type to search · Space changes · Enter save & close · Esc discard & close",
+            ),
+            width,
+            "",
+          );
+        }
+        return lines;
+      },
+      invalidate: () => list.invalidate(),
+      handleInput: (data: string) => list.handleInput(data),
+    });
 
     return {
       render: (width: number) => container.render(width),
       invalidate: () => container.invalidate(),
       handleInput: (data: string) => {
+        if (closing) return;
         if (matchesKey(data, Key.ctrl("s"))) {
-          const snapshot = configLayer(config);
-          const snapshotRevision = revision;
-          saveStatus.setText(theme.fg("dim", "Saving…"));
-          saveQueue = saveQueue.then(async () => {
-            try {
-              const savedPath = await saveConfig(ctx.cwd, ctx.isProjectTrusted(), snapshot);
-              savedRevision = Math.max(savedRevision, snapshotRevision);
-              saveStatus.setText(
-                savedRevision === revision
-                  ? theme.fg("success", `Saved to ${savedPath}`)
-                  : theme.fg("warning", "Unsaved session changes."),
-              );
-            } catch (error) {
-              const message = error instanceof Error ? error.message : String(error);
-              saveStatus.setText(theme.fg("error", message));
-              ctx.ui.notify(message, "error");
-            }
-            tui.requestRender();
-          });
-          tui.requestRender();
+          queueSave(false);
+          return;
+        }
+        if (matchesKey(data, Key.enter)) {
+          queueSave(true);
+          return;
+        }
+        if (matchesKey(data, Key.escape) || keybindings.matches(data, "tui.select.cancel")) {
+          discardAndClose();
           return;
         }
         list.handleInput(data);

package/extensions/openai-codex-compat/web-run.ts

@@ -116,6 +116,13 @@ export default function registerWebRun(
     name: WEB_RUN_TOOL_NAME,
     label: WEB_RUN_TOOL_NAME,
     description: WEB_RUN_DESCRIPTION,
+    promptSnippet: "Search and browse the internet",
+    promptGuidelines: [
+      "Use `web.run` when the user explicitly asks to browse or when answering requires current, niche, high-stakes, or precisely sourced information, including recommendations that may change over time.",
+      'Batch independent `web.run` operations in one call, pass only required parameters, and keep `search_query` to at most four queries; use `response_length: "medium"` or `"long"` when sending four.',
+      "For technical research, prefer primary sources; for OpenAI product questions, inspect local code first and restrict fallback browsing to official OpenAI sites.",
+      "Cite supported claims with direct Markdown links near the relevant text, never expose internal reference IDs, and respect the description's quotation and source word limits.",
+    ],
     parameters: WEB_RUN_PARAMETERS,
     executionMode: "parallel",
     renderShell: "self",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-openai-codex-compat",
-  "version": "0.0.1",
+  "version": "0.0.3",
   "description": "OpenAI Codex compatibility for Pi with native compaction, fast mode, and Codex-optimized capabilities",
   "keywords": [
     "pi-package"
@@ -31,6 +31,7 @@
   "scripts": {
     "check": "hk check --all --check",
     "test": "node --test --test-concurrency=1 test/*.test.ts",
+    "test:live:codex": "PI_CODEX_LIVE_TEST=1 node --test --test-concurrency=1 test/codex-transport.live.test.ts",
     "fmt": "oxfmt",
     "lint": "oxlint",
     "lint:fix": "oxlint --fix"