failsafe-llm-model-resolver 1.0.0 → 1.0.2

package/index.js

@@ -46,7 +46,15 @@ const PREFERENCE = {
   // Prefer flagship grok-*, skip fast / mini / code-fast variants when possible
   xai: {
     include: /^grok-/i,
-    exclude: /fast|mini|lite|code-fast/i,
+    // "non-reasoning" is intentionally its own term here, not "reasoning" -
+    // xAI ships paired variants like grok-4.20-0309-reasoning and
+    // grok-4.20-0309-non-reasoning. Excluding bare /reasoning/ would match
+    // BOTH strings (non-reasoning contains "reasoning" as a substring) and
+    // throw away the good one along with the bad one. non-?reasoning also
+    // catches an unhyphenated "nonreasoning" form if xAI ever ships one.
+    // Tagging/classification work benefits from the reasoning variant, so
+    // steer away from the faster-but-shallower non-reasoning one by default.
+    exclude: /fast|mini|lite|code-fast|non-?reasoning/i,
   },
   // Prefer sonnet/opus over haiku; newest first already gives us the latest
   anthropic: {
@@ -104,14 +112,28 @@ async function fetchGeminiModels(apiKey) {
   // Gemini uses query-string key and a different envelope
   const url = `https://generativelanguage.googleapis.com/v1beta/models?key=${encodeURIComponent(apiKey)}&pageSize=1000`;
   const json = await httpGet(url);
-  // Shape: { models: [ { name: "models/gemini-…", … } ] }
-  // The ID used in generateContent is the part after "models/"
+  // Shape: { models: [ { name: "models/gemini-…", supportedGenerationMethods: [...], … } ] }
+  //
+  // BUGFIX (was throwing ReferenceError on every real call): the previous
+  // version called .filter() AFTER .map() had already reduced each model
+  // object down to a bare id string - by the time the filter ran, the only
+  // thing it had to inspect was `id` (a string), yet it tried to read
+  // `m.supportedGenerationMethods` on a variable `m` that was never in scope
+  // in that callback. This wasn't just a typo to rename; the whole ordering
+  // was backwards. The fix: filter on the full model object FIRST (while
+  // supportedGenerationMethods is still available), THEN map down to the id
+  // string. No IIFE, no scope trick needed - the property we need is simply
+  // read before it's thrown away.
   const list = (json.models || [])
+    .filter(m => {
+      const methods = m.supportedGenerationMethods || m.supported_actions || [];
+      return methods.some(x => /generateContent/i.test(x));
+    })
     .map(m => {
       const name = m.name || '';
+      // The ID used in generateContent is the part after "models/"
       return name.startsWith('models/') ? name.slice(7) : name;
     })
-    .filter(id => id && /generateContent/i.test((m => (m.supportedGenerationMethods || m.supported_actions || []).join(','))(m) || 'generateContent'))
     .filter(Boolean);
   return list;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "failsafe-llm-model-resolver",
-  "version": "1.0.0",
+  "version": "1.0.2",
   "description": "Self-healing, failsafe resolver for the current frontier model across xAI, Anthropic, Gemini, and OpenRouter. Live /models fetch + preference rules + env fallbacks so a network blip never kills a batch job.",
   "main": "index.js",
   "types": "index.d.ts",