Skip to content

Conversation

@ibetitsmike
Copy link
Contributor

Summary

Adds inline editing for custom models in the Settings > Models section, with duplicate prevention and a loading state.

Changes

  • Model editing: Click the pencil icon to edit a model ID inline
    • Enter to save, Escape to cancel
    • Check/X buttons for mouse users
  • Duplicate prevention: Shows error when adding or renaming to an existing model ID
  • Loading state: Shows spinner with "Loading settings..." while fetching config
  • UX polish: Edit/delete buttons disabled while another model is being edited

Deletion safety

Deleting a model from settings only removes it from the dropdown—it doesn't invalidate workspace selections. Model strings are stored in localStorage and passed directly to providers, so workspaces keep working until the user changes their selection.

Generated with mux

- Add inline editing for custom models with pencil icon button
- Prevent duplicate model additions with error messages
- Support keyboard shortcuts: Enter to save, Escape to cancel
- Disable edit/delete buttons while another model is being edited

Deletion of models used by workspaces is safe by design:
model selections are stored as strings and passed to providers,
so removing a model from settings only removes the shortcut.

_Generated with `mux`_
@ibetitsmike ibetitsmike force-pushed the settings-model-editing branch from 4f756ef to fd91d9f Compare November 30, 2025 13:02
@ibetitsmike ibetitsmike added this pull request to the merge queue Nov 30, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 30, 2025
@ibetitsmike ibetitsmike added this pull request to the merge queue Nov 30, 2025
Merged via the queue into main with commit 0085b92 Nov 30, 2025
13 checks passed
@ibetitsmike ibetitsmike deleted the settings-model-editing branch November 30, 2025 13:28
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.

1 participant