fix: enable websocket transport by default

Author: lidge-jun

devlog/133_websocket-default-on/00_verification.md

@@ -0,0 +1,48 @@
+# 133.00 — WebSocket Default-On Release
+
+## Decision
+
+After Phase 132 hardened the Responses WebSocket bridge and the user verified local Codex behavior,
+WebSocket advertisement is restored to default-on for `1.9.1`.
+
+## Behavior
+
+- Missing `websockets` now means enabled.
+- Fresh `~/.opencodex/config.json` writes `"websockets": true`.
+- Explicit `"websockets": false` still suppresses provider/catalog `supports_websockets`.
+
+## Changed
+
+- `/Users/jun/Developer/new/700_projects/opencodex/src/config.ts`
+- `/Users/jun/Developer/new/700_projects/opencodex/src/types.ts`
+- `/Users/jun/Developer/new/700_projects/opencodex/tests/codex-inject.test.ts`
+- `/Users/jun/Developer/new/700_projects/opencodex/tests/codex-catalog.test.ts`
+
+## Verification Plan
+
+- `bun test tests/codex-inject.test.ts tests/codex-catalog.test.ts`
+- `bun test tests`
+- `bun x tsc --noEmit`
+- `ocx start` smoke:
+  - `/healthz` returns HTTP 200.
+  - `/v1/models?client_version=0.141.0` advertises `supports_websockets` by default.
+  - `ocx stop` leaves the proxy stopped before release.
+
+## Verification Results
+
+- `bun test tests/codex-inject.test.ts tests/codex-catalog.test.ts`
+  - 17 pass, 0 fail, 107 assertions.
+- `bun test tests`
+  - 85 pass, 0 fail, 277 assertions.
+- `bun x tsc --noEmit`
+  - passed with exit 0.
+- Local source smoke via `bun src/cli.ts start`
+  - `/healthz` returned HTTP 200.
+  - `/v1/models?client_version=0.141.0` returned `supports_websockets: true` for:
+    - `gpt-5.5`
+    - `opencode-go/kimi-k2.7-code`
+    - `opencode-go/minimax-m3`
+  - Context values stayed correct for the checked routed models:
+    - `opencode-go/kimi-k2.7-code`: `262144`, auto compact `235929`
+    - `opencode-go/minimax-m3`: `512000`, auto compact `460800`
+  - `ocx stop` stopped the proxy and final `ocx status` returned `Proxy not running`.

src/codex-catalog.ts

@@ -206,7 +206,7 @@ function deriveEntry(template: RawEntry | null, slug: string, desc: string, prio
  * catalog sync and the proxy `/v1/models?client_version` branch.
  * Native gpt slugs stay bare; routed models are namespaced `<provider>/<model>`.
  */
-export function buildCatalogEntries(template: RawEntry | null, gptSlugs: string[], goModels: CatalogModel[], featured?: string[], wsEnabled = false): RawEntry[] {
+export function buildCatalogEntries(template: RawEntry | null, gptSlugs: string[], goModels: CatalogModel[], featured?: string[], wsEnabled = true): RawEntry[] {
   // Codex's models-manager sorts by `priority` ASC and advertises the first 5 picker-visible
   // models to spawn_agent (sort_by_key(priority) + MAX_MODEL_OVERRIDES_IN_SPAWN_AGENT=5). Catalog
   // ARRAY order is discarded — so "featuring" a model = giving it the LOWEST priority (0..N-1) so

src/codex-inject.ts

@@ -14,7 +14,7 @@ const OCX_SECTION_MARKER = "# Auto-injected by opencodex";
  * whatever `[table]` happened to be open last (e.g. `[plugins."chrome@openai-bundled"]`), so Codex
  * never saw a global model_provider and silently fell back to the `openai` (ChatGPT) provider.
  */
-export function buildProviderTableBlock(port: number, supportsWebsockets = false): string {
+export function buildProviderTableBlock(port: number, supportsWebsockets = true): string {
   const lines = [
     "",
     OCX_SECTION_MARKER,

src/config.ts

@@ -58,7 +58,7 @@ export function saveConfig(config: OcxConfig): void {
 }
 
 export function websocketsEnabled(config: Pick<OcxConfig, "websockets">): boolean {
-  return config.websockets === true;
+  return config.websockets !== false;
 }
 
 export function getDefaultConfig(): OcxConfig {
@@ -76,7 +76,7 @@ export function getDefaultConfig(): OcxConfig {
     },
     defaultProvider: "openai",
     subagentModels: [...DEFAULT_SUBAGENT_MODELS],
-    websockets: false,
+    websockets: true,
   };
 }
 

src/types.ts

@@ -175,7 +175,7 @@ export interface OcxConfig {
   subagentModels?: string[];
   /** Routed model ids ("<provider>/<model>") hidden from Codex (excluded from the catalog + /v1/models). */
   disabledModels?: string[];
-  /** Advertise supports_websockets so Codex opens the WS endpoint. Default false; set true to opt into WS. */
+  /** Advertise supports_websockets so Codex opens the WS endpoint. Default true; set false to force HTTPS/SSE. */
   websockets?: boolean;
   /** Freshness window (ms) for the per-provider live `/models` cache. Defaults to 5 min. */
   modelCacheTtlMs?: number;

tests/codex-catalog.test.ts

@@ -62,7 +62,7 @@ describe("Codex catalog routed normalization", () => {
     expect(routed).not.toHaveProperty("tool_mode");
     expect(routed).not.toHaveProperty("multi_agent_version");
     expect(routed).not.toHaveProperty("use_responses_lite");
-    expect(routed).not.toHaveProperty("supports_websockets");
+    expect(routed?.supports_websockets).toBe(true);
     expect(routed).not.toHaveProperty("additional_speed_tiers");
     expect(routed).not.toHaveProperty("service_tier");
     expect(routed).not.toHaveProperty("service_tiers");
@@ -99,24 +99,28 @@ describe("Codex catalog routed normalization", () => {
     expect(native?.tool_mode).toBe("code");
     expect(native?.multi_agent_version).toBe("v2");
     expect(native?.use_responses_lite).toBe(true);
-    // Phase 132: websocket advertisement is explicit opt-in, not inherited from templates.
-    expect(native).not.toHaveProperty("supports_websockets");
+    // Phase 133: websocket advertisement is default-on after Phase 132 hardening.
+    expect(native?.supports_websockets).toBe(true);
     expect(native?.web_search_tool_type).toBe("text_and_image");
     expect(native?.supports_search_tool).toBe(true);
     expect(native?.service_tier).toBe("priority");
     expect(native?.service_tiers).toEqual([{ id: "priority" }]);
   });
 
-  test("buildCatalogEntries advertises supports_websockets only on explicit opt-in", () => {
+  test("buildCatalogEntries advertises supports_websockets by default with explicit opt-out", () => {
     const goModels = [{ provider: "anthropic", id: "claude-sonnet-4-6", owned_by: "anthropic" }];
 
-    const off = buildCatalogEntries(nativeTemplate(), ["gpt-5.5"], goModels);
-    expect(off.find(e => e.slug === "gpt-5.5")).not.toHaveProperty("supports_websockets");
-    expect(off.find(e => e.slug === "anthropic/claude-sonnet-4-6")).not.toHaveProperty("supports_websockets");
+    const defaultOn = buildCatalogEntries(nativeTemplate(), ["gpt-5.5"], goModels);
+    expect(defaultOn.find(e => e.slug === "gpt-5.5")?.supports_websockets).toBe(true);
+    expect(defaultOn.find(e => e.slug === "anthropic/claude-sonnet-4-6")?.supports_websockets).toBe(true);
 
     const on = buildCatalogEntries(nativeTemplate(), ["gpt-5.5"], goModels, undefined, true);
     expect(on.find(e => e.slug === "gpt-5.5")?.supports_websockets).toBe(true);
     expect(on.find(e => e.slug === "anthropic/claude-sonnet-4-6")?.supports_websockets).toBe(true);
+
+    const off = buildCatalogEntries(nativeTemplate(), ["gpt-5.5"], goModels, undefined, false);
+    expect(off.find(e => e.slug === "gpt-5.5")).not.toHaveProperty("supports_websockets");
+    expect(off.find(e => e.slug === "anthropic/claude-sonnet-4-6")).not.toHaveProperty("supports_websockets");
   });
 
   test("fallback routed entries still receive explicit search metadata", () => {

tests/codex-inject.test.ts

@@ -2,12 +2,18 @@ import { describe, expect, test } from "bun:test";
 import { buildProviderTableBlock, stripOpencodexConfig } from "../src/codex-inject";
 
 describe("Codex config injection", () => {
-  test("omits provider-level Responses WebSocket support by default", () => {
+  test("advertises provider-level Responses WebSocket support by default", () => {
     const block = buildProviderTableBlock(10100);
 
     expect(block).toContain("[model_providers.opencodex]");
     expect(block).toContain('wire_api = "responses"');
     expect(block).toContain("requires_openai_auth = true");
+    expect(block).toContain("supports_websockets = true");
+  });
+
+  test("can suppress provider-level Responses WebSocket support for explicit opt-out", () => {
+    const block = buildProviderTableBlock(10100, false);
+
     expect(block).not.toContain("supports_websockets");
   });