update model filter for current crop of models

Author: sysread

src/gpt.js

@@ -219,9 +219,52 @@ export async function fetchAndStream(port, messages, options = {}) {
 //------------------------------------------------------------------------------
 // Filters the list of OpenAI models to only those that are useful for our
 // purposes.
+//
+// Notes:
+// - We use the Chat Completions endpoint (/v1/chat/completions), not Responses.
+// - /v1/models returns many model IDs (embeddings, audio, moderation, etc.). We
+//   only want models intended for text generation.
 //------------------------------------------------------------------------------
 export function wantModel(model) {
-  return [/^o\d-mini$/, /^gpt-\d(\.\d)?o(-mini)?$/].some((re) => re.test(model));
+  // Prefers undated aliases, but date preference is enforced in model list construction.
+  // (OpenAI tends to provide `foo-YYYY-MM-DD` *and* a dateless `foo` alias.)
+
+  // Allow chat/text model families.
+  const allow = [/^gpt-/i, /^o\d/i].some((re) => re.test(model));
+
+  // Exclude models that are not intended for this extension’s text summaries.
+  const denyRules = [
+    // Explicit non-text model families.
+    /^text-embedding-/i,
+    /^embedding-/i,
+    /^whisper-/i,
+    /\btts-/i,
+    /-tts\b/i,
+    /^omni-moderation-/i,
+    /^text-moderation-/i,
+    /^moderation-/i,
+    /^dall-e-/i,
+
+    // Known "not what we want" tokens/families.
+    /(^|-)instruct($|-)/i,
+    /(^|-)preview($|-)/i,
+    // Exclude turbo variants (e.g. gpt-4-turbo) but keep gpt-3.5-turbo.
+    // (We do this with a regex rather than an explicit model list.)
+    /(^|-)pro($|-)/i,
+    /(^|-)image($|-)/i,
+    /(^|-)realtime($|-)/i,
+    /(^|-)audio($|-)/i,
+    /(^|-)transcribe($|-)/i,
+    /(^|-)diarize($|-)/i,
+    /(^|-)search($|-)/i,
+    /(^|-)codex($|-)/i,
+    /(^|-)deep-research($|-)/i,
+  ];
+  const isTurboVariant = /(^|-)turbo($|-)/i.test(model);
+  const isAllowedTurbo = /^gpt-3\.5-turbo($|-)/i.test(model);
+  const deny = denyRules.some((re) => re.test(model)) || (isTurboVariant && !isAllowedTurbo);
+
+  return allow && !deny;
 }
 
 //------------------------------------------------------------------------------

src/pages/config.js

@@ -284,18 +284,38 @@ document.addEventListener('DOMContentLoaded', async () => {
       });
 
       if (!response.ok) {
+        const errorMsg = response.statusText;
         console.error('Error fetching models:', response);
-        throw new Error(`Error fetching models: ${response.statusText}`);
+        return { models: [], error: `Error fetching models: ${errorMsg}` };
       }
 
       const data = await response.json();
-      const models = data.data.map((model) => model.id).filter(wantModel);
-      models.sort();
-
-      return models;
+      const candidateIds = data.data.map((model) => model.id);
+      const filtered = candidateIds.filter(wantModel);
+      const withoutNumericSuffix = filtered.filter(id => !/-\d{4,}$/.test(id));
+      const DATE_SUFFIX_RE = /-\d{4}-\d{2}-\d{2}$/;
+      // Build per-base-model selection map
+      const modelMap = {};
+      for (const id of withoutNumericSuffix) {
+        const base = id.replace(DATE_SUFFIX_RE, '');
+        if (!modelMap[base]) {
+          modelMap[base] = { undated: null, dated: null };
+        }
+        if (DATE_SUFFIX_RE.test(id)) {
+          // Keep first dated if multiple exist
+          if (modelMap[base].dated === null) {
+            modelMap[base].dated = id;
+          }
+        } else {
+          modelMap[base].undated = id;
+        }
+      }
+      const chosen = Object.values(modelMap).map(({undated, dated}) => undated || dated);
+      chosen.sort();
+      return { models: chosen, error: null };
     } catch (error) {
       console.error(error);
-      return [];
+      return { models: [], error: error.message };
     }
   }
 
@@ -315,10 +335,19 @@ document.addEventListener('DOMContentLoaded', async () => {
         return;
       }
 
-      const models = await fetchAvailableModels(apiKeyValue);
+      const result = await fetchAvailableModels(apiKeyValue);
+
+      if (result.error) {
+        showError(`Failed to refresh models: ${result.error}`);
+        return;
+      }
+
+      const models = result.models;
 
       if (models.length === 0) {
-        showError('No models retrieved. Please check your API key and try again.');
+        showError(
+          'No usable chat models were returned. This can happen if OpenAI returned only models that this extension filters out (preview/turbo/pro/dated/audio/etc).'
+        );
         return;
       }
 
@@ -341,6 +370,7 @@ document.addEventListener('DOMContentLoaded', async () => {
       } else {
         modelSelect.value = defaultModel;
       }
+      toggleReasoningOptions();
     } catch (error) {
       showError(`Failed to refresh models: ${error.message}`);
     } finally {