@goodandready/dsh-model-sync 0.2.6 → 0.2.7

package/README.md

@@ -43,3 +43,9 @@ Omit `provider` to discover all enabled API-key providers in one run. The Settin
 ```
 
 An empty `models` array means all models in the latest available catalog. The selected catalog is written to DSH's `llm-pi-ai` settings, so the standard DSH model picker refreshes from the resulting list.
+
+
+The full catalog discovered by an applied run is cached in the plugin's own settings
+separately from the model allowlist. After a restart, the model chooser can show the
+full last-known catalog while the standard DSH picker still exposes only the selected
+models. Dry-run remains read-only and does not persist this cache.

package/lib/index.js

@@ -18,6 +18,14 @@ export const Config = Schema.object({
     enabled: Schema.boolean().default(true),
   })).default([]),
   modelSelections: Schema.dict(Schema.array(Schema.string())).default({}),
+  // Full catalogs are cached separately from the allowlist applied to DSH.
+  // This keeps the model picker useful after a restart without widening the
+  // active llm-pi-ai catalog.
+  modelCatalogs: Schema.dict(Schema.array(Schema.object({
+    provider: Schema.string().required(),
+    id: Schema.string().required(),
+    name: Schema.string().required(),
+  }).loose())).default({}),
 })
 
 export function apply(ctx, config) {

package/lib/synchronizer.js

@@ -38,10 +38,13 @@ export function createModelSynchronizer(ctx, { getConfig = () => ({}), saveConfi
         ? [...new Set(config.modelSelections[row.provider].filter((id) => typeof id === 'string' && id))]
         : []
       const latest = lastResults.get(row.provider)
+      const cached = normalizeModels(row.provider, config.modelCatalogs?.[row.provider])
       return {
         ...row,
         selectedModels: selection,
-        availableModels: latest?.status === 'ok' ? latest.availableModels ?? latest.models ?? row.models : row.models,
+        availableModels: latest?.status === 'ok'
+          ? latest.availableModels ?? latest.models ?? row.models
+          : cached.length > 0 ? cached : row.models,
       }
     })
   }
@@ -70,6 +73,7 @@ export function createModelSynchronizer(ctx, { getConfig = () => ({}), saveConfi
   async function run(options = {}) {
     if (running) return running
     const config = getConfig() ?? {}
+    const cachedCatalogs = structuredClone(config.modelCatalogs ?? {})
     const dryRun = options.dryRun !== false
     const removeMissing = options.removeMissing === true
     const selected = typeof options.provider === 'string' && options.provider
@@ -101,6 +105,7 @@ export function createModelSynchronizer(ctx, { getConfig = () => ({}), saveConfi
           removeMissing: effectiveRemoveMissing,
           changed: reconciliation.changed,
         })
+        cachedCatalogs[row.provider] = discovered.models
       }
       const result = { startedAt, finishedAt: Date.now(), dryRun, removeMissing, results, applied: false }
       if (!dryRun && results.some((row) => row.status === 'ok' && row.changed)) {
@@ -117,6 +122,20 @@ export function createModelSynchronizer(ctx, { getConfig = () => ({}), saveConfi
         await settings.replace(PIAI_NS, next, descriptor.revision)
         result.applied = true
       }
+      if (!dryRun) {
+        const previousCatalogs = config.modelCatalogs ?? {}
+        if (JSON.stringify(previousCatalogs) !== JSON.stringify(cachedCatalogs)) {
+          try {
+            await saveConfig({ modelCatalogs: cachedCatalogs })
+            result.catalogCachePersisted = true
+          } catch (error) {
+            // The active DSH catalog has already been revision-checked. A
+            // cache failure must be visible but must not undo that successful
+            // provider update.
+            result.catalogCacheError = errorMessage(error)
+          }
+        }
+      }
       lastRun = result
       return result
     })()
@@ -132,7 +151,12 @@ export function createModelSynchronizer(ctx, { getConfig = () => ({}), saveConfi
     const row = providers().find((item) => item.provider === provider)
     if (!row) throw Object.assign(new Error(`provider is not configured: ${provider}`), { code: 'PROVIDER_NOT_CONFIGURED' })
     const latest = lastRun?.results?.find((item) => item.provider === provider && item.status === 'ok')
-    const available = normalizeModels(provider, latest?.availableModels ?? latest?.models ?? row.availableModels ?? row.models)
+    const cached = normalizeModels(provider, config.modelCatalogs?.[provider])
+    const latestCatalog = latest?.availableModels ?? latest?.models
+    const source = latestCatalog !== undefined
+      ? latestCatalog
+      : cached.length > 0 ? cached : row.availableModels ?? row.models
+    const available = normalizeModels(provider, source)
     const availableById = new Map(available.map((model) => [model.id, model]))
     const unknown = selectedModels.filter((id) => !availableById.has(id))
     if (unknown.length > 0) throw Object.assign(new Error(`unknown models for ${provider}: ${unknown.join(', ')}`), { code: 'UNKNOWN_MODEL' })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-model-sync",
-  "version": "0.2.6",
+  "version": "0.2.7",
   "description": "Automatic model catalog synchronization for API-key DeepSeek Harness providers.",
   "keywords": [
     "deepseek-harness",