@koreanhole/opencodex 3.2.8 → 3.2.9

package/AGENTS_INSTALL.md

@@ -90,7 +90,6 @@ bun run verify:upstream
 bun run typecheck
 bun test tests/e2e/cline-only.test.ts
 bun run test:e2e:gemini
-bun run test:e2e:gemini-oauth
 bun run test:e2e:gemini-tool
 bun run test:e2e:gemini-reasoning
 bun run test:e2e:codex-restore
@@ -99,6 +98,4 @@ bun run test:e2e:codex-restore
 Report whether each evidence path used a deterministic fixture or a real
 vendor binary/account. The Gemini command above is a hard live gate: missing
 binaries, authentication, a Flash Lite model, or cleanup evidence must fail.
-The OAuth gate additionally removes API-key/Vertex environment values and the
-vendor `.env`.
 Never report a skipped or synthetic run as a real login.

package/README.md

@@ -160,7 +160,6 @@ bun run typecheck
 bun test tests
 bun run test:e2e
 bun run test:e2e:gemini
-bun run test:e2e:gemini-oauth
 bun run test:e2e:gemini-slow
 bun run test:e2e:gemini-parent-tool
 bun run privacy:scan
@@ -172,9 +171,6 @@ a real Codex client and a real authenticated Gemini CLI. It runs both the minima
 completion and a 12-second pre-event slow-start regression. It fails, not skips,
 on a missing binary, authentication, Flash Lite model, response marker, timeout,
 or child cleanup assertion. `test:e2e:gemini-slow` runs only the slow-start case.
-`test:e2e:gemini-oauth` strips API-key and Vertex environment values, removes the
-vendor `.env`, and requires a real OAuth-backed completion through Codex and the
-installed Gemini CLI.
 
 Optional overrides are `CLINE_BIN_PATH`, `GEMINI_BIN_PATH`, and
 `CODEX_BIN_PATH`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@koreanhole/opencodex",
-  "version": "3.2.8",
+  "version": "3.2.9",
   "description": "Full opencodex runtime with Cline JSON and Gemini CLI headless provider adapters",
   "type": "module",
   "main": "./bin/package-main.mjs",
@@ -31,7 +31,6 @@
     "test": "bun test tests",
     "test:e2e": "bun test tests/e2e/cline-only.test.ts",
     "test:e2e:gemini": "bun test ./tests/e2e/gemini-codex.live.ts -t \"full-stack:\"",
-    "test:e2e:gemini-oauth": "bun test ./tests/e2e/gemini-codex.live.ts -t \"uses an existing Gemini CLI OAuth login without an API key\"",
     "test:e2e:gemini-slow": "bun test ./tests/e2e/gemini-codex.live.ts -t \"full-stack: a slow Gemini start\"",
     "test:e2e:gemini-tool": "bun test ./tests/e2e/gemini-codex.live.ts -t \"one short Gemini turn leaves the workspace unchanged\"",
     "test:e2e:gemini-multiturn": "bun test ./tests/e2e/gemini-codex.live.ts -t \"reuses a live Codex multi-turn session\"",

package/src/adapter-runtime/config.ts

@@ -188,8 +188,8 @@ function isLegacyProviderRow(name: AdapterProviderId, value: unknown): boolean {
   return row?.adapter === name;
 }
 
-function uniqueModels(metadata: AdapterModelMetadata | undefined): string[] {
-  return [...new Set(["default", ...(metadata?.models ?? [])].filter(Boolean))];
+function uniqueModels(name: AdapterProviderId, metadata: AdapterModelMetadata | undefined): string[] {
+  return [...new Set([...(name === "cline-cli" ? ["default"] : []), ...(metadata?.models ?? [])].filter(Boolean))];
 }
 
 export function buildManagedProviderRow(
@@ -208,7 +208,7 @@ export function buildManagedProviderRow(
     allowPrivateNetwork: true,
     liveModels: true,
     defaultModel: "default",
-    models: uniqueModels(metadata),
+    models: uniqueModels(name, metadata),
     ...(metadata.modelDisplayNames ? { modelDisplayNames: metadata.modelDisplayNames } : {}),
     ...(metadata.modelContextWindows ? { modelContextWindows: metadata.modelContextWindows } : {}),
     ...(metadata.modelInputModalities ? { modelInputModalities: metadata.modelInputModalities } : {}),

package/src/adapters/cli-agent/gemini/live-models.ts

@@ -204,6 +204,7 @@ export async function discoverGeminiLiveModels(options: GeminiLiveModelsOptions
   }
 
   const runtimeIds = runtime?.map(model => model.modelId) ?? [];
+  const runtimeAvailable = runtimeIds.length > 0;
   const ids = runtimeIds.length > 0 ? runtimeIds : Object.keys(metadata.capabilities);
   if (ids.length === 0) {
     failureCache.set(fingerprint, timestamp);
@@ -252,9 +253,11 @@ export async function discoverGeminiLiveModels(options: GeminiLiveModelsOptions
       loadSession: false,
     };
   });
-  const modelInfoWithDefault = [defaultModelInfo(metadata, modelInfo), ...modelInfo.filter(model => model.id !== "default")];
+  const modelInfoWithDefault = runtimeAvailable
+    ? modelInfo
+    : [defaultModelInfo(metadata, modelInfo), ...modelInfo.filter(model => model.id !== "default")];
   const warning = metadata.warning
-    ?? (runtimeError ? "Gemini runtime model discovery failed; using installed CLI metadata." : undefined);
+    ?? (runtimeAvailable ? undefined : "Gemini runtime model discovery failed; using installed CLI metadata.");
   cache.set(fingerprint, { models: ids, modelInfo: modelInfoWithDefault, source: "models", fetchedAt: timestamp, ...(warning ? { metadataWarning: warning } : {}) });
   failureCache.delete(fingerprint);
   return { ok: true, models: ids, modelInfo: modelInfoWithDefault, source: "models", fingerprint, ...(warning ? { metadataWarning: warning } : {}) };

package/src/lib/codex-catalog.ts

@@ -178,12 +178,14 @@ function defaultModel(): OcxProviderModel {
   return { id: "default" };
 }
 
-function normalizedModels(models: readonly OcxProviderModel[]): OcxProviderModel[] {
+function normalizedModels(models: readonly OcxProviderModel[], includeDefault = true): OcxProviderModel[] {
   const defaultInfo = models.find(model => model.id === "default");
-  const out: OcxProviderModel[] = [{ ...defaultModel(), ...(defaultInfo ?? {}), id: "default" }];
-  const seen = new Set<string>(["default"]);
+  const out: OcxProviderModel[] = includeDefault
+    ? [{ ...defaultModel(), ...(defaultInfo ?? {}), id: "default" }]
+    : [];
+  const seen = new Set<string>(includeDefault ? ["default"] : []);
   for (const model of models) {
-    if (model.id === "default" || seen.has(model.id)) continue;
+    if ((includeDefault && model.id === "default") || seen.has(model.id)) continue;
     seen.add(model.id);
     out.push(model);
   }
@@ -207,7 +209,7 @@ export async function discoverCodexCatalogModels(options: CodexCatalogDiscoveryO
   if (!gemini.ok && gemini.reason !== "not_found" && env.OPENCODEX_GEMINI_ACP?.trim() !== "0") warnings.push(`Gemini: ${gemini.detail}`);
   if (gemini.ok && gemini.metadataWarning) warnings.push(`Gemini metadata: ${gemini.metadataWarning}`);
   const clineModels = normalizedModels(cline.ok ? cline.models.map(id => ({ id })) : []);
-  const geminiModels = normalizedModels(gemini.ok ? gemini.modelInfo : []);
+  const geminiModels = normalizedModels(gemini.ok ? gemini.modelInfo : [], false);
   const geminiDefault = geminiModels.find(model => model.id === "default");
   const geminiContexts = geminiModels.flatMap(model => model.contextWindowTokens !== undefined ? [model.contextWindowTokens] : []);
   if (geminiDefault && geminiContexts.length > 0) {

package/src/server/index.ts

@@ -118,7 +118,7 @@ async function discoverModels(
   }
   if (gemini) {
     models["gemini-cli"] = gemini.ok
-      ? ["default", ...gemini.modelInfo.filter(model => model.id !== "default")]
+      ? gemini.modelInfo.map(model => model.id)
       : ["default"];
   }
   return models;

package/src/server/responses.ts

@@ -379,6 +379,16 @@ const PROVIDER_LABELS: Record<ProviderId, string> = {
   [GEMINI_PROVIDER_ID]: "Gemini CLI",
 };
 
+function listedProviderEntries(
+  known: ProviderModels,
+  providerId: ProviderId,
+): readonly ProviderModelEntry[] {
+  const entries = known[providerId] ?? [];
+  if (providerId === GEMINI_PROVIDER_ID && entries.length > 0) return entries;
+  const defaultEntry = entries.find(entry => modelIdOf(entry) === "default") ?? "default";
+  return [defaultEntry, ...entries.filter(entry => modelIdOf(entry) !== "default")];
+}
+
 export function modelList(
   known: string[] | ProviderModels,
   scope?: ProviderId,
@@ -394,8 +404,7 @@ export function modelList(
     }));
   }
   if (scope) {
-    const defaultEntry = (known[scope] ?? []).find(entry => modelIdOf(entry) === "default") ?? "default";
-    const entries = [defaultEntry, ...(known[scope] ?? []).filter(entry => modelIdOf(entry) !== "default")];
+    const entries = listedProviderEntries(known, scope);
     const rows: Array<Record<string, unknown>> = [];
     const seen = new Set<string>();
     for (const entry of entries) {
@@ -440,8 +449,7 @@ export function modelList(
     display_name: "Configured default provider",
   }];
   for (const providerId of [CLINE_PROVIDER_ID, GEMINI_PROVIDER_ID] as const) {
-    const defaultEntry = (known[providerId] ?? []).find(entry => modelIdOf(entry) === "default") ?? "default";
-    const entries = [defaultEntry, ...(known[providerId] ?? []).filter(entry => modelIdOf(entry) !== "default")];
+    const entries = listedProviderEntries(known, providerId);
     const seen = new Set<string>();
     for (const entry of entries) {
       const id = modelIdOf(entry);