dsh-my-go 0.4.1 → 0.4.2

package/lib/index.js

@@ -662,8 +662,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 +883,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.2",
   "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",