fix: id separator for provider, model muxing rule (#360)

* fix: id format for provider, model muxing rule

* refactor: replace separator and logic

* fix: gh action cache version

Author: peppescg

.github/actions/setup/action.yaml

@@ -13,7 +13,7 @@ runs:
 
     - name: Cache dependencies
       id: cache
-      uses: actions/cache@0c45773b623bea8c8e75f6c82b208c3cf94ea4f9 # v4
+      uses: actions/cache@v4.2.2
       with:
         path: ./node_modules
         key: modules-${{ hashFiles('package-lock.json') }}

src/features/workspace/components/workspace-models-dropdown.tsx

@@ -37,7 +37,7 @@ function groupModelsByProviderName(
     id: providerName,
     textValue: providerName,
     items: items.map((item) => ({
-      id: `${item.provider_id}:${item.name}`,
+      id: `${item.provider_id}@${item.name}`,
       textValue: item.name,
     })),
   }))
@@ -116,8 +116,10 @@ export function WorkspaceModelsDropdown({
               const selectedValue = v.values().next().value
               if (!selectedValue && typeof selectedValue !== 'string') return
               if (typeof selectedValue === 'string') {
-                const [provider_id, modelName] = selectedValue.split(':')
-                if (!provider_id || !modelName) return
+                const atIndex = selectedValue.indexOf('@')
+                const provider_id = selectedValue.slice(0, atIndex)
+                const modelName = selectedValue.slice(atIndex + 1)
+                if (atIndex === -1 && (!provider_id || !modelName)) return
                 onChange({
                   model: modelName,
                   provider_id,