dsh-my-go 0.1.13 → 0.1.14

package/dist/client.core.js

@@ -199,12 +199,11 @@ function apply(ctx) {
     const [available, setAvailable] = React.useState({ providers: [], models: {} });
     React.useEffect(() => {
       if (!sp) return;
-      try {
-        setDraft(sp.read());
-      } catch {
-        setDraft({});
-      }
       if (connection && connection.rpc && typeof connection.rpc.call === "function") {
+        connection.rpc.call("/dsh-my-go", "loadSettings", {}).then((res) => {
+          if (res && res.ok && res.value) setDraft(res.value);
+          else setDraft({});
+        }).catch(() => setDraft({}));
         connection.rpc.call("/dsh-my-go", "listModels", {}).then((res) => {
           if (res && res.ok && res.value && Array.isArray(res.value.providers)) setAvailable(res.value);
         }).catch(() => {
@@ -246,17 +245,6 @@ function apply(ctx) {
         setSaving(false);
       }
     };
-    const autoSaveTimer = React.useRef(null);
-    React.useEffect(() => {
-      if (!draft || !sp) return;
-      if (autoSaveTimer.current) clearTimeout(autoSaveTimer.current);
-      autoSaveTimer.current = setTimeout(() => {
-        void save();
-      }, 500);
-      return () => {
-        if (autoSaveTimer.current) clearTimeout(autoSaveTimer.current);
-      };
-    }, [draft, sp]);
     const selectStyle = { background: "var(--surface, #1e1e1e)", color: "var(--text, #e0e0e0)", border: "1px solid var(--separator, #333)", borderRadius: 4, padding: "4px 8px", fontSize: 13, width: "100%", boxSizing: "border-box" };
     const labelStyle = { fontSize: 12, color: "var(--text-secondary, #888)", marginBottom: 2 };
     const cardStyle = { border: "1px solid var(--separator, #333)", borderRadius: 8, padding: 12, marginBottom: 12 };

package/dist/client.js

@@ -204,12 +204,11 @@ function apply(ctx) {
     const [available, setAvailable] = React.useState({ providers: [], models: {} });
     React.useEffect(() => {
       if (!sp) return;
-      try {
-        setDraft(sp.read());
-      } catch {
-        setDraft({});
-      }
       if (connection && connection.rpc && typeof connection.rpc.call === "function") {
+        connection.rpc.call("/dsh-my-go", "loadSettings", {}).then((res) => {
+          if (res && res.ok && res.value) setDraft(res.value);
+          else setDraft({});
+        }).catch(() => setDraft({}));
         connection.rpc.call("/dsh-my-go", "listModels", {}).then((res) => {
           if (res && res.ok && res.value && Array.isArray(res.value.providers)) setAvailable(res.value);
         }).catch(() => {
@@ -251,17 +250,6 @@ function apply(ctx) {
         setSaving(false);
       }
     };
-    const autoSaveTimer = React.useRef(null);
-    React.useEffect(() => {
-      if (!draft || !sp) return;
-      if (autoSaveTimer.current) clearTimeout(autoSaveTimer.current);
-      autoSaveTimer.current = setTimeout(() => {
-        void save();
-      }, 500);
-      return () => {
-        if (autoSaveTimer.current) clearTimeout(autoSaveTimer.current);
-      };
-    }, [draft, sp]);
     const selectStyle = { background: "var(--surface, #1e1e1e)", color: "var(--text, #e0e0e0)", border: "1px solid var(--separator, #333)", borderRadius: 4, padding: "4px 8px", fontSize: 13, width: "100%", boxSizing: "border-box" };
     const labelStyle = { fontSize: 12, color: "var(--text-secondary, #888)", marginBottom: 2 };
     const cardStyle = { border: "1px solid var(--separator, #333)", borderRadius: 8, padding: 12, marginBottom: 12 };

package/lib/index.js

@@ -376,6 +376,16 @@ export async function apply(ctx, config = {}) {
         }
         return { ok: true, value: { providers, models } }
       }
+      if (endpoint === 'loadSettings') {
+        const settingsService = ctx.get('settings')
+        if (!settingsService) return { ok: true, value: {} }
+        try {
+          const stored = settingsService.get('dsh-my-go')
+          return { ok: true, value: stored && typeof stored === 'object' ? stored : {} }
+        } catch (e) {
+          return { ok: true, value: {} }
+        }
+      }
       if (endpoint === 'saveSettings') {
         const draft = payload
         if (!draft || typeof draft !== 'object') return { ok: false, error: { code: 'bad-request', message: 'payload must be an object' } }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-my-go",
-  "version": "0.1.13",
+  "version": "0.1.14",
   "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": "daizihan233",
   "license": "MIT",

package/src/client.js

@@ -187,8 +187,12 @@ export function apply(ctx) {
 
     React.useEffect(() => {
       if (!sp) return
-      try { setDraft(sp.read()) } catch { setDraft({}) }
+      // Load settings via host RPC (DSH SettingsScope doesn't support nested reads)
       if (connection && connection.rpc && typeof connection.rpc.call === 'function') {
+        connection.rpc.call('/dsh-my-go', 'loadSettings', {}).then((res) => {
+          if (res && res.ok && res.value) setDraft(res.value)
+          else setDraft({})
+        }).catch(() => setDraft({}))
         connection.rpc.call('/dsh-my-go', 'listModels', {}).then((res) => {
           if (res && res.ok && res.value && Array.isArray(res.value.providers)) setAvailable(res.value)
         }).catch(() => {})
@@ -212,6 +216,7 @@ export function apply(ctx) {
       })
     }
 
+    // Manual save only — auto-save risks infinite loops with settings/updated events
     const save = async () => {
       setSaving(true); setMsg(null)
       try {
@@ -229,15 +234,6 @@ export function apply(ctx) {
       } finally { setSaving(false) }
     }
 
-    // Auto-save on draft change with debounce
-    const autoSaveTimer = React.useRef(null)
-    React.useEffect(() => {
-      if (!draft || !sp) return
-      if (autoSaveTimer.current) clearTimeout(autoSaveTimer.current)
-      autoSaveTimer.current = setTimeout(() => { void save() }, 500)
-      return () => { if (autoSaveTimer.current) clearTimeout(autoSaveTimer.current) }
-    }, [draft, sp])
-
     const selectStyle = { background: 'var(--surface, #1e1e1e)', color: 'var(--text, #e0e0e0)', border: '1px solid var(--separator, #333)', borderRadius: 4, padding: '4px 8px', fontSize: 13, width: '100%', boxSizing: 'border-box' }
     const labelStyle = { fontSize: 12, color: 'var(--text-secondary, #888)', marginBottom: 2 }
     const cardStyle = { border: '1px solid var(--separator, #333)', borderRadius: 8, padding: 12, marginBottom: 12 }