@juspay/neurolink 12.41.7 → 12.41.9

package/dist/cli/commands/models.js

@@ -5,6 +5,7 @@
  */
 import { ModelResolver, formatSearchResults, formatRecommendation, formatComparison, } from "../../models/modelResolver.js";
 import { getAllModels, getModelsByProvider, getAvailableProviders, formatModelForDisplay, } from "../../models/modelRegistry.js";
+import { getCatalogJsonEntries } from "../../providers/catalog/loader.js";
 import chalk from "chalk";
 import ora from "ora";
 import { logger } from "../../utils/logger.js";
@@ -334,9 +335,23 @@ export class ModelsCommandFactory {
                 try {
                     const { AIProviderFactory } = await import("../../core/factory.js");
                     const provider = (await AIProviderFactory.createProvider(providerName));
-                    const liveIds = typeof provider.getAvailableModels === "function"
+                    const rawLiveIds = typeof provider.getAvailableModels === "function"
                         ? await provider.getAvailableModels()
                         : [];
+                    // Project against the JSON catalog's non-retired ids, mirroring
+                    // getAllModels()'s listing filter: a catalog-backed provider's
+                    // vendor `/models` endpoint often keeps listing an id long after
+                    // it 404s on chat completions (see e.g. fireworks.json's
+                    // `evidence.liveMatrix`), so an unfiltered live listing would
+                    // print a retired id as a usable "live" result. Providers with no
+                    // catalog entry (litellm, ollama, openai-compatible gateways,
+                    // lm-studio, llamacpp) have nothing to project against, so their
+                    // live discovery stays unrestricted — that is the whole point of
+                    // this fallback for them.
+                    const catalogEntry = getCatalogJsonEntries().find((entry) => entry.id === providerName);
+                    const liveIds = catalogEntry
+                        ? rawLiveIds.filter((id) => catalogEntry.models.catalog[id]?.status !== "retired")
+                        : rawLiveIds;
                     if (liveIds.length > 0) {
                         if (spinner) {
                             spinner.succeed(`Found ${liveIds.length} models (live from ${providerName})`);

package/dist/providers/catalog/fireworks.json

@@ -23,7 +23,7 @@
                 "enumMember": "KIMI_K2P6",
                 "vision": false,
                 "status": "retired",
-                "description": "Retired 2026-09 — still on the Fireworks roster, but a real chat call returns 404 'not deployed' (since the 2026-09-26 nightly)"
+                "description": "Retired 2026-09 — still listed on the Fireworks /models roster, but the chat endpoint 404s 'Model not found, inaccessible, and/or not deployed' (since the 2026-09-26 nightly); superseded by kimi-k3, same Moonshot family"
             },
             "accounts/fireworks/models/gpt-oss-120b": {
                 "enumMember": "GPT_OSS_120B",
@@ -107,7 +107,8 @@
         "topModels": [
             "accounts/fireworks/models/kimi-k3",
             "accounts/fireworks/models/gpt-oss-120b",
-            "accounts/fireworks/models/qwen3p8-max"
+            "accounts/fireworks/models/qwen3p8-max",
+            "accounts/fireworks/models/glm-5p3"
         ]
     },
     "capabilities": {
@@ -146,7 +147,7 @@
         },
         "liveMatrix": {
             "date": "2026-09-28",
-            "result": "kimi-k2p6 is still listed on /models but a real chat call now returns 404 'not deployed', which failed the nightly model-liveness gate from 2026-09-26 and sent the matrix vision test down the text-only fallback. kimi-k3 answers a text call and a call carrying a PNG with 200, so it becomes the default; gpt-oss-120b, qwen3p8-max and glm-5p3 also answer text calls. Earlier (2026-08-30): deepseek-v4-pro was listed but hung past 120s, six ids returned 404 'not deployed', and glm-5p3 answers 'does not support image inputs'."
+            "result": "Issue #1796: the 2026-09-26 nightly model-liveness gate reported the default, kimi-k2p6, as dead, sending the matrix vision test down the text-only fallback. Confirmed live — GET /models still lists kimi-k2p6, but POST /chat/completions 404s 'Model not found, inaccessible, and/or not deployed', the same still-listed-but-unservable pattern deepseek-v4-pro hit in 2026-08 (hung past 120s rather than 404ing). Moved the default to kimi-k3, the same Moonshot family's newest generation: 200 OK on both a text-only call and a call carrying a PNG, and a full-length call returns a proper OpenAI-shaped tool_calls entry with finish_reason=tool_calls, confirming vision and tool support both survive the swap. kimi-k2p6 marked retired rather than deleted, per the frozen-enum rule. gpt-oss-120b, qwen3p8-max and glm-5p3 (the rest of fallbacks/topModels) were re-probed alive at the same time; glm-5p3 still only answers text calls, not image ones, as found 2026-08-30."
         },
         "addedInPR": "https://github.com/juspay/neurolink/pull/1587"
     }

package/dist/utils/modelChoices.d.ts

@@ -30,6 +30,12 @@ export declare function getTopModelChoices(provider: AIProviderName, limit?: num
  * Get all available models for a provider
  * Returns all values from the provider's model enum
  *
+ * For a JSON-catalog provider, excludes retired ids. The `catalog` map keeps
+ * retired entries as raw table keys (they still resolve for a caller who
+ * pins that id explicitly — see `isValidModel`'s dedicated retired-id
+ * allowance below), but a listing/selection surface should not offer them.
+ * Mirrors the `selectableModels` filter in catalog/schema.ts.
+ *
  * @param provider - The AI provider to get models for
  * @returns Array of model identifier strings
  */
@@ -57,6 +63,20 @@ export declare function getDefaultModel(provider: AIProviderName): string | unde
 /**
  * Check if a model is valid for a given provider
  *
+ * `getAllModels` excludes retired catalog ids (it is a listing/selection
+ * surface), but a caller who pins a retired id explicitly must not have
+ * that pin start failing validation just because it dropped out of the
+ * listing — so this falls back to the full catalog (all statuses) before
+ * rejecting a JSON-catalog provider's model.
+ *
+ * The empty-listing case is checked against catalog membership, not
+ * accepted outright: a catalog provider whose every entry happens to be
+ * retired still has a real catalog to validate against, and a fabricated
+ * id must not slip through just because the *selectable* listing is empty.
+ * Only a provider with no catalog entry at all (openai-compatible,
+ * lm-studio, llamacpp — providers without a strict model list) keeps the
+ * permissive "allow any model" behavior.
+ *
  * @param provider - The AI provider
  * @param model - The model identifier to check
  * @returns true if the model exists in the provider's model enum

package/dist/utils/modelChoices.js

@@ -510,13 +510,21 @@ export function getTopModelChoices(provider, limit = 5) {
  * Get all available models for a provider
  * Returns all values from the provider's model enum
  *
+ * For a JSON-catalog provider, excludes retired ids. The `catalog` map keeps
+ * retired entries as raw table keys (they still resolve for a caller who
+ * pins that id explicitly — see `isValidModel`'s dedicated retired-id
+ * allowance below), but a listing/selection surface should not offer them.
+ * Mirrors the `selectableModels` filter in catalog/schema.ts.
+ *
  * @param provider - The AI provider to get models for
  * @returns Array of model identifier strings
  */
 export function getAllModels(provider) {
     const catalogEntry = catalogEntryFor(provider);
     if (catalogEntry) {
-        return Object.keys(catalogEntry.models.catalog);
+        return Object.entries(catalogEntry.models.catalog)
+            .filter(([, spec]) => spec.status !== "retired")
+            .map(([model]) => model);
     }
     const modelEnum = MODEL_ENUMS[provider];
     if (!modelEnum) {
@@ -557,17 +565,42 @@ export function getDefaultModel(provider) {
 /**
  * Check if a model is valid for a given provider
  *
+ * `getAllModels` excludes retired catalog ids (it is a listing/selection
+ * surface), but a caller who pins a retired id explicitly must not have
+ * that pin start failing validation just because it dropped out of the
+ * listing — so this falls back to the full catalog (all statuses) before
+ * rejecting a JSON-catalog provider's model.
+ *
+ * The empty-listing case is checked against catalog membership, not
+ * accepted outright: a catalog provider whose every entry happens to be
+ * retired still has a real catalog to validate against, and a fabricated
+ * id must not slip through just because the *selectable* listing is empty.
+ * Only a provider with no catalog entry at all (openai-compatible,
+ * lm-studio, llamacpp — providers without a strict model list) keeps the
+ * permissive "allow any model" behavior.
+ *
  * @param provider - The AI provider
  * @param model - The model identifier to check
  * @returns true if the model exists in the provider's model enum
  */
 export function isValidModel(provider, model) {
     const models = getAllModels(provider);
-    if (models.length === 0) {
-        // For providers without strict model lists (like openai-compatible), allow any model
+    if (models.includes(model)) {
         return true;
     }
-    return models.includes(model);
+    const catalogEntry = catalogEntryFor(provider);
+    if (catalogEntry) {
+        // Falls back to the FULL catalog (all statuses) so an explicitly pinned
+        // retired id still validates, and — critically — so an all-retired
+        // catalog (getAllModels() === []) is checked against real membership
+        // instead of accepting any string. Object.hasOwn (not `in`) so an
+        // inherited Object.prototype member name (e.g. "toString") is never
+        // mistaken for a real catalog key.
+        return Object.hasOwn(catalogEntry.models.catalog, model);
+    }
+    // No JSON catalog and no strict MODEL_ENUMS entry (openai-compatible,
+    // lm-studio, llamacpp): keep the permissive "allow any model" behavior.
+    return models.length === 0;
 }
 /**
  * Get model choices formatted for inquirer prompts with a specific default