Tighten custom provider review fixes

Remove debug logging of OpenAI-compatible conversation and SSE payloads. Resolve provider selections through canonicalized IDs so stale casing or spacing maps back to the stored provider.

Author: PeterDaveHello

src/popup/sections/api-modes-provider-utils.mjs

@@ -97,13 +97,16 @@ export function createProviderId(providerName, existingProviders, reservedProvid
 export function resolveSelectableProviderId(providerId, providers, fallbackProviderId = '') {
   const normalizedProviderId = normalizeText(providerId)
   if (!normalizedProviderId) return fallbackProviderId
-  const hasMatchedProvider =
+  const normalizedLookupProviderId = normalizeProviderId(normalizedProviderId)
+  const matchedProvider =
     Array.isArray(providers) &&
-    providers.some(
+    providers.find(
       (provider) =>
-        normalizeText(provider?.id) === normalizedProviderId && provider?.enabled !== false,
+        normalizeProviderId(provider?.id) === normalizedLookupProviderId &&
+        provider?.enabled !== false,
     )
-  return hasMatchedProvider ? normalizedProviderId : fallbackProviderId
+  if (!matchedProvider) return fallbackProviderId
+  return normalizeText(matchedProvider?.id) || normalizedProviderId
 }
 
 export function resolveEditingProviderSelection(

src/services/apis/openai-compatible-core.mjs

@@ -112,7 +112,6 @@ export async function generateAnswersWithOpenAICompatible({
     if (finished) return
     finished = true
     pushRecord(session, question, answer)
-    console.debug('conversation history', { content: session.conversationRecords })
     port.postMessage({ answer: null, done: true, session: session })
   }
 
@@ -122,7 +121,6 @@ export async function generateAnswersWithOpenAICompatible({
     headers: buildHeaders(apiKey, extraHeaders),
     body: JSON.stringify(requestBody),
     onMessage(message) {
-      console.debug('sse message', message)
       if (finished) return
       if (message.trim() === '[DONE]') {
         finish()

tests/unit/popup/api-modes-provider-utils.test.mjs

@@ -371,6 +371,12 @@ test('resolveSelectableProviderId falls back when provider is missing or invalid
   assert.equal(resolveSelectableProviderId('   ', providers, fallbackId), fallbackId)
 })
 
+test('resolveSelectableProviderId matches normalized provider ids and returns canonical id', () => {
+  const providers = [{ id: 'my-proxy' }]
+
+  assert.equal(resolveSelectableProviderId(' My Proxy ', providers, ''), 'my-proxy')
+})
+
 test('resolveEditingProviderSelection keeps legacy provider selection for empty or legacy values', () => {
   const providers = [{ id: 'myproxy' }]