Skip to content

Conversation

@sazary
Copy link

@sazary sazary commented Dec 3, 2025

resolves #4734

some notes about changes:

  • i think it's "cleaner" to have a way to just get new providers and return new providers using /config/providers?refresh=true, but afaik it needed much changes in the sync provider, specifically in the bootstrap method. so i did what was previously done in dialog-provider: just dispose and rebootstrap the client
  • since this adds the 3rd keybinding to the dialog-model screen, the modal width is not enough for all of them to sit in one row at the bottom of the dialog. one approach is to just wrap them, but imho it wasn't a nice ui. so i added some code to dialog-select to change from row layout to column when they don't fit in just one row. if you don't like it just tell me to remove it and add flexWrap.
  • i updated the js sdk. i didn't try to update stainless since i think it needs an account, and i updated the python sdk but the changes were way too much :) maybe i shouldn't touch that
  • i checked the desktop package to add the shortcut there too, but afaik there was no example of keyboard shortcuts there. i would be happy to add it if you tell me the convention

if there's any issues just let me know

@sazary
Copy link
Author

sazary commented Dec 8, 2025

@rekram1-node could you please approve for workflows?

@rekram1-node
Copy link
Collaborator

/review

@rekram1-node
Copy link
Collaborator

Thanks for making this also if format or sdk fail, just ignore them Dax has something he said he'd fix w/ them.

@rekram1-node
Copy link
Collaborator

PR may need some small tweaks but this functionality is needed

@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025

lgtm

@sazary
Copy link
Author

sazary commented Dec 8, 2025

sure just tell me about those tweaks

@sazary sazary force-pushed the feat/manual-refresh-models branch 2 times, most recently from a924f1d to 93207cd Compare December 24, 2025 15:13
@sazary
Copy link
Author

sazary commented Dec 24, 2025

@rekram1-node fixed merge conflicts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE]: Refresh models manually

2 participants