@yemi33/minions 0.1.2424 → 0.1.2426

package/engine/model-discovery.js

@@ -75,9 +75,18 @@ function listAllRuntimes() {
  * @param {object}  opts.config Pass `getConfig()` result so the helper can
  *                              honour `config.engine.disableModelDiscovery`
  *                              without re-reading the file on every request.
+ * @param {boolean} opts.persist Write refreshed catalogs to the runtime cache.
+ * @param {boolean} opts.throwOnUnavailable Surface discovery failures instead
+ *                              of returning the free-text fallback.
  * @returns {Promise<{runtime: string, models: object[]|null, cachedAt: string|null}>}
  */
-async function getRuntimeModels(runtimeName, { force = false, ttlMs = MODEL_CACHE_TTL_MS, config = null } = {}) {
+async function getRuntimeModels(runtimeName, {
+  force = false,
+  ttlMs = MODEL_CACHE_TTL_MS,
+  config = null,
+  persist = true,
+  throwOnUnavailable = false,
+} = {}) {
   // Throws "Unknown runtime ..." for unregistered names. Caller turns this into
   // a 404; bubbling the throw is intentional so misconfigurations surface loudly
   // (mirrors the registry contract documented in engine/runtimes/index.js).
@@ -138,10 +147,15 @@ async function getRuntimeModels(runtimeName, { force = false, ttlMs = MODEL_CACH
       // want that to be mutable through the response we return.
       models = result.map(m => ({ ...m }));
     }
-  } catch { /* swallow — network/transient errors map to null */ }
+  } catch (error) {
+    if (throwOnUnavailable) throw error;
+  }
+  if (throwOnUnavailable && models === null) {
+    throw new Error(`Runtime "${runtimeName}" returned no model catalog`);
+  }
 
   const cachedAt = new Date().toISOString();
-  if (cachePath) {
+  if (cachePath && persist) {
     _writeCacheFile(cachePath, {
       runtime: runtimeName,
       models,