@fro.bot/systematic 2.14.2 → 2.14.4

package/dist/index.js

@@ -717,14 +717,14 @@ function readFallbackCache() {
   if (openCodeModelsUrl) {
     const urlDerivedPath = path5.join(cacheDir, `models-${fastHash(openCodeModelsUrl)}.json`);
     const urlResult = readModelsFromCache(urlDerivedPath);
-    if (urlResult !== null) {
+    if (urlResult !== null && urlResult.size > 0) {
       return { status: "cache", models: urlResult };
     }
     return emptyAvailability();
   }
   const defaultPath = path5.join(cacheDir, MODELS_JSON_FILENAME);
   const defaultResult = readModelsFromCache(defaultPath);
-  if (defaultResult !== null) {
+  if (defaultResult !== null && defaultResult.size > 0) {
     return { status: "cache", models: defaultResult };
   }
   return emptyAvailability();
@@ -767,9 +767,13 @@ async function getAvailableModels(client, options = {}) {
   if (response.error !== undefined || response.data === undefined) {
     return readFallbackCache();
   }
+  const models = buildSetFromProviders(response.data.providers);
+  if (models.size === 0) {
+    return emptyAvailability();
+  }
   return {
     status: "api",
-    models: buildSetFromProviders(response.data.providers)
+    models
   };
 }
 
@@ -1088,6 +1092,8 @@ function createConfigHandler(deps) {
     const bundledSkills = collectSkillsAsCommands(bundledSkillsDir, systematicConfig.disabled_skills);
     const enabledSkillNames = collectEnabledSkillNames(bundledSkillsDir, systematicConfig.disabled_skills);
     const inventory = buildBundledAgentInventory(bundledAgentsDir2, systematicConfig.disabled_agents);
+    const availability = deps.client ? await getAvailableModels(deps.client) : undefined;
+    const availabilitySet = availability && availability.status !== "unknown" ? availability.models : undefined;
     assertSourceCategoryModelCoverage(inventory.categories);
     const validatedOverlays = validateAgentOverlays({
       inventory,
@@ -1096,8 +1102,6 @@ function createConfigHandler(deps) {
       enabledSkills: enabledSkillNames
     });
     const resolvedOverlays = resolveAgentOverlaySet(validatedOverlays);
-    const availability = deps.client ? await getAvailableModels(deps.client) : undefined;
-    const availabilitySet = availability && availability.status !== "unknown" ? availability.models : undefined;
     const bundledAgents = collectAgents(bundledAgentsDir2, systematicConfig.disabled_agents, nativeAgents, resolvedOverlays, availabilitySet);
     const bundledCommands = collectCommands(bundledCommandsDir, systematicConfig.disabled_commands);
     const bundledAgentKeys = new Set(Object.keys(bundledAgents));

package/dist/lib/model-availability.d.ts

@@ -22,16 +22,22 @@ export interface OpencodeClientLike {
  * Outcome of model availability discovery.
  *
  * - `api`: The OpenCode server's `/config/providers` endpoint responded with
- *   a connected-providers payload. `models` may be empty if no providers
- *   are authenticated; that is authoritative.
+ *   a connected-providers payload AND `models` is non-empty. An authoritatively
+ *   empty response (`data.providers = []`, or providers present with zero
+ *   models) collapses to `'unknown'` instead — see below — because the
+ *   operational consequence is identical and downstream consumers should treat
+ *   both cases the same way.
  * - `cache`: The API call failed (error envelope, thrown, or timed out) and
- *   the local `models.json` cache was readable. `models` reflects whatever
- *   OpenCode last wrote to disk.
- * - `unknown`: Both the API call and the cache fallback failed (cache
- *   missing, unreadable, corrupt, or schema-mismatched). Resolution should
- *   degrade gracefully — callers should treat `unknown` as a signal to
- *   skip source-default model pinning so users do not get agents pinned
- *   to inaccessible models. `models` is the empty set.
+ *   the local `models.json` cache was readable AND non-empty. `models`
+ *   reflects whatever OpenCode last wrote to disk. A cache that loads
+ *   successfully but produces a zero-size set collapses to `'unknown'` for
+ *   the same operational reason an empty API response does.
+ * - `unknown`: Either both the API call and the cache fallback failed (cache
+ *   missing, unreadable, corrupt, or schema-mismatched), OR the API call
+ *   succeeded with zero usable models. Resolution should degrade gracefully —
+ *   callers should treat `unknown` as a signal to skip source-default model
+ *   pinning so users do not get agents pinned to inaccessible models.
+ *   `models` is the empty set.
  */
 export type DiscoveryStatus = 'api' | 'cache' | 'unknown';
 export interface ModelAvailability {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fro.bot/systematic",
-  "version": "2.14.2",
+  "version": "2.14.4",
   "description": "Structured engineering workflows for OpenCode",
   "type": "module",
   "homepage": "https://fro.bot/systematic",
@@ -68,7 +68,7 @@
     "@opencode-ai/sdk": "1.14.48",
     "@types/bun": "latest",
     "@types/js-yaml": "4.0.9",
-    "@types/node": "24.12.3",
+    "@types/node": "24.12.4",
     "ajv": "8.20.0",
     "ajv-formats": "3.0.1",
     "conventional-changelog-conventionalcommits": "9.3.1",