dsh-my-go 0.4.1 → 0.4.3

package/lib/index.js

@@ -589,19 +589,21 @@ export async function apply(ctx, config = {}) {
       // mjs files degrades to defaults instead of failing the preset mount.
       const mod = await import('@deepseek-ai/schemastery')
       const z = mod.default ?? mod
+      // schemastery 字段默认 nullable（missing→undefined），.required() 才强制；
+      // 无 .optional() 方法——直接不加 .required() 即为可选。
       const agentSchema = z.object({
-        provider: z.string(),
-        model: z.string(),
-        reasoningEffort: z.string(),
-        dsv4p0813: z.boolean(),
+        provider: z.string().default(''),
+        model: z.string().default(''),
+        reasoningEffort: z.string().default(''),
+        dsv4p0813: z.boolean().default(false),
         // 错峰路由（peak routing）：开启后按北京时间高峰/非高峰时段在
-        // 两套 provider/model 之间切换。字段平铺 optional，保证旧存储
-        // 配置（无这些字段）兼容。
-        peakRouting: z.boolean().optional(),
-        peakProvider: z.string().optional(),
-        peakModel: z.string().optional(),
-        offpeakProvider: z.string().optional(),
-        offpeakModel: z.string().optional(),
+        // 两套 provider/model 之间切换。不加 .required() 即为可选，
+        // 旧存储配置无这些字段时自动兼容（missing→undefined）。
+        peakRouting: z.boolean(),
+        peakProvider: z.string(),
+        peakModel: z.string(),
+        offpeakProvider: z.string(),
+        offpeakModel: z.string(),
       })
       settingsScope = settings.register(
         'dsh-my-go',
@@ -662,8 +664,8 @@ export async function apply(ctx, config = {}) {
           bindings = merged
         }
       })
-    } catch {
-      // Settings optional — defaults apply.
+    } catch (e) {
+      console.warn('[dsh-my-go] settings registration failed (defaults apply):', e)
     }
   }
 
@@ -883,30 +885,37 @@ export async function apply(ctx, config = {}) {
       }
       if (endpoint === 'saveSettings') {
         const draft = payload
-        if (!draft || typeof draft !== 'object') return { ok: false, error: { code: 'bad-request', message: 'payload must be an object' } }
+        if (!draft || typeof draft !== 'object') return { ok: false, error: { code: 'internal', message: 'payload must be an object', details: {} } }
         const settingsService = ctx.get('settings')
-        if (!settingsService) return { ok: false, error: { code: 'unavailable', message: 'settings service not available' } }
+        if (!settingsService) return { ok: false, error: { code: 'internal', message: 'settings service not available', details: {} } }
         try {
-          const agentTypes = ['sisyphus', 'hermes', 'explore', 'librarian', 'looker', 'hephaestus', 'prometheus', 'oracle']
-          const fields = ['provider', 'model', 'reasoningEffort', 'dsv4p0813', 'peakRouting', 'peakProvider', 'peakModel', 'offpeakProvider', 'offpeakModel']
-          const ops = []
-          for (const type of agentTypes) {
-            for (const field of fields) {
-              const val = draft[type]?.[field]
-              if (val === undefined || val === null || val === '') {
-                ops.push({ op: 'unset', path: [type, field] })
-              } else {
-                ops.push({ op: 'set', path: [type, field], value: val })
+          // 优先使用 settingsScope.replace（整段替换），比 mutate 更稳健；
+          // 降级到 mutate（path ops）兜底。
+          if (settingsScope && typeof settingsScope.replace === 'function') {
+            await settingsScope.replace(draft)
+          } else {
+            const agentTypes = ['sisyphus', 'hermes', 'explore', 'librarian', 'looker', 'hephaestus', 'prometheus', 'oracle']
+            const fields = ['provider', 'model', 'reasoningEffort', 'dsv4p0813', 'peakRouting', 'peakProvider', 'peakModel', 'offpeakProvider', 'offpeakModel']
+            const ops = []
+            for (const type of agentTypes) {
+              for (const field of fields) {
+                const val = draft[type]?.[field]
+                if (val === undefined || val === null || val === '') {
+                  ops.push({ op: 'unset', path: [type, field] })
+                } else {
+                  ops.push({ op: 'set', path: [type, field], value: val })
+                }
               }
             }
+            if (ops.length > 0) await settingsService.mutate('dsh-my-go', ops)
           }
-          if (ops.length > 0) await settingsService.mutate('dsh-my-go', ops)
           return { ok: true, value: null }
         } catch (e) {
-          return { ok: false, error: { code: 'settings-rejected', message: String(e) } }
+          console.error('[dsh-my-go] saveSettings failed:', e)
+          return { ok: false, error: { code: 'settings-rejected', message: String(e), details: { ns: 'dsh-my-go' } } }
         }
       }
-      return { ok: false, error: { code: 'bad-request', message: `unknown endpoint: ${endpoint}` } }
+      return { ok: false, error: { code: 'internal', message: `unknown endpoint: ${endpoint}`, details: {} } }
     }, { authority: 'trusted-host' })
   })
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-my-go",
-  "version": "0.4.1",
+  "version": "0.4.3",
   "description": "DSH plugin: Sisyphus agent orchestration — star topology, single-line blocking, 7 specialist sub-agents (Hermes/Explore/Librarian/Looker/Hephaestus/Prometheus/Oracle) with per-type model binding, go_work/continue/need_help/forward communication tools, Web UI tree panel and settings.",
   "author": "Tisitan",
   "license": "MIT",