cody icon indicating copy to clipboard operation
cody copied to clipboard

Simplify model fetching observables - part 1 of N

Open pkukielka opened this issue 10 months ago • 0 comments

Changes

Main change is in the structure of observables in the sync.ts method syncModels. Previously it was possible to get multiple updates of auth/config/clientConfig/etc and various promises could be at different stage of updates. E.g. userPreferences could be updated based on older version of config, and remoteModels based on newer version. Also config and auth should always go in sync, otherwise we can get them for two different endpoints.

In general this PR aims to make model syncing more predictable and less error prone.

There will be more PRs following this one.

Test plan

N/A - covered by automated tests

pkukielka avatar Apr 17 '25 11:04 pkukielka