@goodandready/dsh-model-sync 0.3.10 → 0.3.11

package/README.md

@@ -107,6 +107,15 @@ dsh plugin --profile web add @goodandready/dsh-model-sync
 
 ---
 
+## 🛠️ Enhancements & Fixes in v0.3.11
+
+* 🗂️ **Streamlined Settings UI Surface**:
+  * Removed the redundant top-level settings sidebar section fallback (`settings.section`), keeping configuration strictly inside the standard DSH plugin settings card (`settings.plugin.item`).
+* 🛡️ **Hardened Cordis Service Access**:
+  * Service access for `settings` and `llm` is now performed via `ctx.get(...)` with safe fallback to direct property inspection, preventing silent resolution failures on Cordis proxy implementations.
+
+---
+
 ## 🛠️ Enhancements & Fixes in v0.3.8
 
 * ⚙️ **Reactive Settings Card Binding (`settingsScope`)**:

package/README.ru.md

@@ -84,6 +84,15 @@ dsh plugin --profile web add @goodandready/dsh-model-sync
 
 ---
 
+## 🛠️ Улучшения и исправления в v0.3.11
+
+* 🗂️ **Чистый интерфейс настроек (DSH Design Contract)**:
+  * Удалена избыточная регистрация fallback-раздела верхнего уровня бокового меню (`settings.section`), настройки доступны только через стандартную карточку в «Настройки → Плагины → Настройки плагинов» (`settings.plugin.item`).
+* 🛡️ **Надёжный доступ к сервисам Cordis**:
+  * Доступ к сервисам ядра (`settings`, `llm`) переведён на `ctx.get(...)` с безопасным fallback к прямым свойствам контекста для исключения ошибок проксирования.
+
+---
+
 ## 🛠️ Улучшения и исправления в v0.3.8
 
 * ⚙️ **Реактивное управление настройками через `settingsScope`**:

package/README.zh.md

@@ -73,3 +73,13 @@ dsh plugin --profile web add @goodandready/dsh-model-sync
 ## 📄 开源协议
 
 MIT © [GooDAnDReaDY](https://github.com/GooDAnDReaDY)
+
+## 🛠️ v0.3.11 改进与修复
+
+* 🗂️ **规范化设置界面**:
+  * 移除侧边栏顶层多余的 `settings.section` fallback，统一使用标准插件设置卡片 `settings.plugin.item`。
+* 🛡️ **增强 Cordis 服务访问稳定性**:
+  * 使用 `ctx.get(...)` 安全获取 `settings` 与 `llm` 服务，避免代理对象属性解析异常。
+
+---
+

package/lib/client.js

@@ -49,7 +49,6 @@ window.__ModuleLoader__.load({
     }
     const API = '/dsh-model-sync'
     const CAPABILITY_LABELS = { vision: { en: 'Vision', ru: 'Зрение' }, tools: { en: 'Tools', ru: 'Инструменты' }, reasoning: { en: 'Reasoning', ru: 'Рассуждение' }, embeddings: { en: 'Embeddings', ru: 'Эмбеддинги' } }
-    function capabilityKeys(models) { return [...new Set((models ?? []).flatMap((model) => Object.entries(model?.capabilities ?? {}).filter(([, enabled]) => enabled === true).map(([key]) => key)))].sort() }
     function filterModelsByCapabilities(models, selected) { const required = [...new Set((selected ?? []).filter((key) => typeof key === 'string' && key))]; return (models ?? []).filter((model) => required.every((key) => model?.capabilities?.[key] === true)) }
     const NS = 'dsh-model-sync'
     function useLocale(ctx) {
@@ -86,7 +85,7 @@ window.__ModuleLoader__.load({
       const doTry = (provider, modelId) => { setTryState((s)=>({ ...s, [provider+':'+modelId]: { loading: true } })); fetch(`${API}/try`, { method: 'POST', headers: { 'content-type': 'application/json' }, body: JSON.stringify({ provider, model: modelId }) }).then(async r=>{ const b=await r.json(); if(!r.ok) throw new Error(b?.error?.message||`HTTP ${r.status}`); return b }).then(b=>setTryState((s)=>({ ...s, [provider+':'+modelId]: { ok: b.ok, latency: b.latencyMs, status: b.status, error: b.error } }))).catch(e=>setTryState((s)=>({ ...s, [provider+':'+modelId]: { error: String(e.message||e) } }))) }
       const load = React.useCallback(() => { setError(''); return fetch(`${API}/status`).then((r) => r.json()).then(setStatus).catch((e) => setError(String(e?.message ?? e))) }, [])
       const loadCredentials = React.useCallback(() => { setError(''); return fetch(`${API}/credentials`).then(async (r) => { const body = await r.json(); if (!r.ok) throw new Error(body?.error?.message ?? `HTTP ${r.status}`); return body }).then(setCredentialStatus).catch((e) => setError(String(e?.message ?? e))) }, [])
-      React.useEffect(() => { ensureStyles(); load(); loadCredentials(); const timer = setInterval(load, 15000); return () => clearInterval(timer) }, [load, loadCredentials])
+      React.useEffect(() => { ensureStyles(); load(); loadCredentials(); const timer = setInterval(() => { if (typeof document !== 'undefined' && document.hidden) return; load() }, 15000); return () => clearInterval(timer) }, [load, loadCredentials])
       const execute = (targetProvider, dryRunOverride = dryRun) => { if (!dryRunOverride && removeMissing && !window.confirm(t.removeMissing)) return; setBusy(true); setError(''); fetch(`${API}/run`, { method: 'POST', headers: { 'content-type': 'application/json' }, body: JSON.stringify({ provider: targetProvider || undefined, dryRun: dryRunOverride, removeMissing }) }).then(async (r) => { const body = await r.json(); if (!r.ok) throw new Error(body?.error?.message ?? `HTTP ${r.status}`); return body }).then((body) => { setStatus(body); if (dryRunOverride) setPreview({ ...body, provider: targetProvider || '' }); else setPreview(null) }).catch((e) => setError(String(e?.message ?? e))).finally(() => setBusy(false)) }
       const checkHealth = (targetProvider) => { setBusy(true); setError(''); fetch(`${API}/health`, { method: 'POST', headers: { 'content-type': 'application/json' }, body: JSON.stringify(targetProvider ? { provider: targetProvider } : {}) }).then(async (r) => { const body = await r.json(); if (!r.ok) throw new Error(body?.error?.message ?? `HTTP ${r.status}`); return body }).then(setStatus).catch((e) => setError(String(e?.message ?? e))).finally(() => setBusy(false)) }
       const checkCredentials = (targetProvider) => { setBusy(true); setError(''); fetch(`${API}/credentials/check`, { method: 'POST', headers: { 'content-type': 'application/json' }, body: JSON.stringify(targetProvider ? { provider: targetProvider } : {}) }).then(async (r) => { const body = await r.json(); if (!r.ok) throw new Error(body?.error?.message ?? `HTTP ${r.status}`); return body }).then((body) => { setCredentialStatus(body.credentials); setStatus((current) => ({ ...(current ?? {}), lastHealth: body.health })) }).catch((e) => setError(String(e?.message ?? e))).finally(() => setBusy(false)) }
@@ -184,7 +183,7 @@ window.__ModuleLoader__.load({
     function ModelSyncCard({ ctx }) {
       const locale = useLocale(ctx); const t = locale === 'ru' ? ru : en; const [open, setOpen] = React.useState(false)
       const [cardStatus, setCardStatus] = React.useState(null)
-      React.useEffect(() => { ensureStyles(); const load = () => fetch(`${API}/status`).then(r=>r.json()).then(setCardStatus).catch(()=>{}); load(); const id=setInterval(load,15000); return ()=>clearInterval(id)}, [])
+      React.useEffect(() => { ensureStyles(); const load = () => { if (typeof document !== 'undefined' && document.hidden) return; fetch(`${API}/status`).then(r=>r.json()).then(setCardStatus).catch(()=>{}) }; load(); if (open) return; const id = setInterval(load, 15000); return () => clearInterval(id) }, [open])
       const cardBadge = (cardStatus?.lastRun?.results?.reduce((s,r)=>s+(r.diff?.added?.length||0),0) || 0)
       return React.createElement('li', { className: 'dms-card' + (open ? ' dms-card-open' : '') },
         React.createElement('button', { type: 'button', className: 'dms-card-header', 'aria-expanded': open, 'aria-label': `${t[open ? 'hideSettings' : 'showSettings']}: ${t.title}`, onClick: () => setOpen((value) => !value) },
@@ -198,15 +197,9 @@ window.__ModuleLoader__.load({
         try { ctx.locale?.register?.(NS, { en, ru }) } catch {}
         return () => {}
       }, 'dsh-model-sync: dictionaries')
-      const registerPluginCard = () => {
-        try {
-          ctx.slots.inject('settings.plugin.item', () => ctx.slots.register({ name: 'settings.plugin.item', key: NS, locale: NS, inject: () => ({ ctx }) }, (props) => React.createElement(ModelSyncCard, { ...props, ctx })))
-          return true
-        } catch { return false }
-      }
-      if (!registerPluginCard()) {
-        ctx.slots.inject('settings.section', () => ctx.slots.register({ name: 'settings.section', id: 'dsh-model-sync', order: 25, label: () => 'Model Sync' }, (props) => React.createElement(ModelSyncSection, { ...props, ctx })))
-      }
+      try {
+        ctx.slots.inject('settings.plugin.item', () => ctx.slots.register({ name: 'settings.plugin.item', key: NS, locale: NS, inject: () => ({ ctx }) }, (props) => React.createElement(ModelSyncCard, { ...props, ctx })))
+      } catch {}
     }
     module.exports = { apply, inject: ['slots', 'locale', 'settingsScope'], filterModelsByCapabilities }; return module.exports
   },

package/lib/index.js

@@ -137,7 +137,7 @@ export function apply(ctx, config) {
   try { settings = ctx.get('settings') } catch {}
   if (!bindSettings(settings, ctx)) {
     ctx.inject(['settings'], (sctx) => {
-      bindSettings(sctx.settings, sctx)
+      bindSettings((typeof sctx.get === 'function' ? sctx.get('settings') : null) ?? sctx.settings, sctx)
     })
   }
 

package/lib/provider-service.js

@@ -3,13 +3,22 @@ import { listApiKeyProviders } from './inventory.js'
 export function createProviderService(ctx) {
   const readSettings = () => {
     try {
-      return ctx.get('settings')
+      return (typeof ctx.get === 'function' ? (ctx.get('settings') ?? ctx.settings) : ctx.settings)
     } catch {
-      return undefined
+      return ctx.settings
     }
   }
 
-  const listProviders = () => listApiKeyProviders(ctx.llm, readSettings())
+  const readLlm = () => {
+    try {
+      const llm = typeof ctx.get === 'function' ? ctx.get('llm') : null
+      return (llm && typeof llm === 'object' && (llm.listConfigurableProviders || llm.listProviders)) ? llm : ctx.llm
+    } catch {
+      return ctx.llm
+    }
+  }
+
+  const listProviders = () => listApiKeyProviders(readLlm(), readSettings())
 
   return Object.freeze({
     listProviders,