fix: prevent duplicate LM Studio models in provider configuration

- Use consistent model.path as key for both downloaded and loaded models
- Fixes issue where models appeared twice with different identifiers
- Update tests to reflect the new consistent key usage

Fixes #6954

Author: roomote

src/api/providers/fetchers/__tests__/lmstudio.test.ts

@@ -140,7 +140,7 @@ describe("LMStudio Fetcher", () => {
 			expect(mockListLoaded).toHaveBeenCalledTimes(1)
 
 			const expectedParsedModel = parseLMStudioModel(mockRawModel)
-			expect(result).toEqual({ [mockRawModel.modelKey]: expectedParsedModel })
+			expect(result).toEqual({ [mockRawModel.path]: expectedParsedModel })
 		})
 
 		it("should use default baseUrl if an empty string is provided", async () => {

src/api/providers/fetchers/lmstudio.ts

@@ -87,8 +87,9 @@ export async function getLMStudioModels(baseUrl = "http://localhost:1234"): Prom
 		})) as Array<LLMInstanceInfo>
 
 		for (const lmstudioModel of loadedModels) {
-			models[lmstudioModel.modelKey] = parseLMStudioModel(lmstudioModel)
-			modelsWithLoadedDetails.add(lmstudioModel.modelKey)
+			// Use path as the key to avoid duplicates when a model is both downloaded and loaded
+			models[lmstudioModel.path] = parseLMStudioModel(lmstudioModel)
+			modelsWithLoadedDetails.add(lmstudioModel.path)
 		}
 	} catch (error) {
 		if (error.code === "ECONNREFUSED") {