[serjic]

Summary

Addresses IJAI-1783.

An organization can restrict the models its JetBrains gateway serves, but a session routed through providers/set offered Codex's whole catalogue, so the picker showed models the gateway then refused. providers/set now lists the gateway's models before it responds, and the adapter reports only the Codex models the gateway lists.

• GET {baseUrl}/models with the headers Codex sends to the gateway (X-Client-Feature-ID: codex plus the providers/set headers); OpenAI format, data[].id. The listing is stored with the gateway config, so a provider restart reuses it, and setting the same configuration again does not list again.
• fetchAvailableModels narrows model/list, so new, resumed, loaded and forked sessions and unstable_setSessionModel all see the same list. Ids match without case and date suffix; exactly one narrowed model stays isDefault.
• A thread model the gateway does not list is replaced by the narrowed default, also on turn/start; a model the gateway lists but Codex's catalogue lacks is kept. Sessions open across providers/set or providers/disable take the models of the restarted app-server, and the client gets a config_option_update. A listing that matches no Codex model leaves the session with no model: there is no model config option at all.
• A 4xx answer means the gateway has no listing: providers/set succeeds and nothing is narrowed. Any other failed listing (network error, 5xx, malformed body, 10 s timeout) fails providers/set with internal_error, and the current routing stays. A listing that finishes after a newer providers/set or providers/disable was applied is dropped (a newer request that failed drops nothing), and the routing goes to the client a provider restart installed. A baseUrl that is not an http(s) URL, accepted on v1 before, now fails with invalid_params; an invalid header or a base URL with credentials fails without repeating the value.
• The gateway auth method and providers from the Codex config (config.toml, CODEX_CONFIG) are not listed. readme-dev.md describes the behaviour.

Not covered

Title generation (gpt-5.6-luna), subagents, /review and compaction choose their own model and are not checked against the listing.

Verification

npm run typecheck
npm run build
npm test   # 1755 passed, 36 skipped

gateway-model-discovery.test.ts covers the request and headers, narrowing and the default, the replaced thread model on turn/start, failing providers/set (HTTP error, malformed body, network error) with the routing kept, the empty list, no listing without providers/set or for the gateway auth method, providers/disable, reuse, and the provider restart. Tests that call providers/set stub the listing (stubGatewayModelListing). Checked locally against the JetBrains gateway with an earlier build of this branch: the adapter's log shows /models answering gpt-5.6-sol, gpt-6-sol, gpt-6.1-sol in one session and an empty list with only Bedrock configured; the empty-list and failing-listing behaviour of the final build is covered by the tests only.

🤖 Generated with Claude Code

[EugeneTheDev]

High: a providers/set can be lost during a provider restart. setProvider awaits this.codexAcpClient.setProvider(...), and that call now awaits the network listing. The receiver is bound before the await. If a restart queued by an earlier providers/set swaps this.codexAcpClient while this listing is in flight, the new routing is written to the old client, which has already been replaced. This queued update then runs adoptRoutingFrom(this.codexAcpClient) and copies the previous routing from the current client.

I reproduced it with a temporary test: set A (one session, replacement initialize held), then set B with a deferred fetch, release the restart, then resolve B. Both calls succeed, but the final client reports baseUrl A. Before this PR, the queued update re-applied params on the replacement, so B won.

Suggestion: take the listing first (it does not depend on the client), then apply the config synchronously to whatever client is current, and/or keep passing the built GatewayConfig into the queued update instead of copying from this.codexAcpClient.

[EugeneTheDev]

Medium: a providers/disable (or a newer providers/set) sent while this listing is in flight gets overwritten. previous is read before the await, and gatewayConfig is assigned unconditionally after it. So set(A) → disable → listing resolves leaves A active, even though the client's last request was the disable and both requests succeeded. I confirmed this with a temporary test: after set then disable, listProviders() still reports A's baseUrl. Two overlapping sets with different configs behave the same way: the one whose listing finishes last wins, not the one sent last.

A generation counter (or serializing the providers/* calls) would let a stale listing drop its result instead of applying it.

[EugeneTheDev]

This replaces any thread model that is not in the narrowed Codex catalogue, even when the gateway itself lists it. Say config.toml sets model = "gpt-6.1-sol", the gateway lists gpt-6.1-sol, and Codex's bundled catalogue doesn't have it yet. The session silently switches to the narrowed default. That undoes the custom-provider behaviour described in the comment just below (keep an uncatalogued configured model). Should this branch run only when the gateway does not list modelId either? For example, check listed.has(normalizeModelId(modelId)), which would need a small helper exported from GatewayModelDiscovery.

[EugeneTheDev]

Low: when the gateway lists no Codex models, the model select option is returned with options: [] but currentValue is still the thread model (gpt-5.5 in the test). A select whose currentValue is not among its options is inconsistent, and clients may reject it or render it oddly. Would it be better to omit the model option entirely in this case?