Repository navigation
Conversation
An authenticated catalog with a different response shape can then reuse the snapshot, in-flight dedupe and bounded read instead of copying them. The "models" discovery state is now a non-empty tuple and the catalog is generic over its seed, so a catalog seeded with a non-empty list always yields a non-empty one without a cast.
The startup refresh after an OpenCode Go or Zen prefetch rebuilt the catalog without the live selection, so a model switched during that rebuild kept the previous model's context window. All four prefetch refresh sites now share one helper that passes it, as the connect path already did.
A connection that drops or times out mid-body now counts as unavailable rather than malformed, so routine network trouble stays at debug while an empty or malformed catalog warns.
|
Triage: Pass No changes needed from you. A maintainer will review this pull request. Notes:
|
|
Stack merge order: #1580 → #1581 → #1579. This repo keeps merged branches, so GitHub does not retarget the next PR on its own:
|
There was a problem hiding this comment.
Review · Approve
This change gets model lists ready for one list per signed-in account, instead of a single shared list.
Fetching a provider's models and reading the response are now arguments, so a later ChatGPT list can reuse the same cache instead of copying it. Each signed-in profile picks its own models. If a fetch fails, the log says whether the catalog was empty or the connection dropped. Refreshing an OpenCode Go or Zen list no longer forgets the model you just switched to, so that model keeps the right context window.
Checked against the parent branch (#1580) only. No problems in this slice.
TheGreatAxios
left a comment
There was a problem hiding this comment.
Review · Approve
A bounded catalog takes its request and parser as inputs, an OAuth projection chooses models per profile, and a settled prefetch keeps the live model selection.
Findings
No findings.
TheGreatAxios
left a comment
There was a problem hiding this comment.
Review · Comment
Planner: the injected request and parser, the per-profile models callback, and the shared prefetch refresh fit a later authenticated, per-account catalog without widening those seams.
Findings
No findings.
Summary
Verification
bun run checkandbun run testpass.Part of CL-11727