dsh-tool-vision 0.3.5 → 0.3.7

package/client.js

@@ -116,10 +116,7 @@ window.__ModuleLoader__.load({
     function VisionSection(props) {
       var t = props.t;
       var scope = props.scope;
-      var snapshot = react.useSyncExternalStore(
-        function (cb) { return scope.subscribe(cb); },
-        function () { return scope.getSnapshot(); }
-      );
+      var [snapshot, setSnapshot] = react.useState(function () { return scope.getSnapshot(); });
       var ready = snapshot.status === "ready" && snapshot.value !== void 0;
       var [draft, setDraft] = react.useState({});
       var [busy, setBusy] = react.useState(false);
@@ -128,13 +125,19 @@ window.__ModuleLoader__.load({
 
       react.useEffect(function () {
         scope.load();
-        return function () { scope.dispose(); };
+        var alive = true;
+        var sync = function () { if (alive) setSnapshot(scope.getSnapshot()); };
+        var un = typeof scope.subscribe === "function" ? scope.subscribe(sync) : null;
+        return function () { alive = false; if (un) un(); if (scope.dispose) scope.dispose(); };
       }, [scope]);
-      // Re-seed the draft when the snapshot value changes (after a save or a host change).
+      // Seed the draft ONLY when the snapshot becomes ready — never on value
+      // churn. settingsScope.getSnapshot() returns a fresh object per call,
+      // so depending on snapshot.value would reset user input on every render
+      // (typing appears dead).
       react.useEffect(function () {
         if (ready) setDraft(function (prev) { return Object.assign({}, prev, valueToDraft(snapshot.value)); });
         // eslint-disable-next-line react-hooks/exhaustive-deps
-      }, [ready, snapshot.value]);
+      }, [ready]);
 
       if (snapshot.status === "unavailable") {
         return h("p", { className: "__tv_unavailable" }, t("unavailable"));
@@ -184,15 +187,34 @@ window.__ModuleLoader__.load({
         });
         Promise.all(writes).then(function () {
           setBusy(false); setNotice(t("saved"));
+          if (scope.load) scope.load();
         }).catch(function (e) {
           setBusy(false); setError(t("error") + ": " + String(e && e.message || e));
         });
       }
 
+      function reseedDraft() {
+        if (typeof scope.load === "function") {
+          var p = scope.load();
+          if (p && typeof p.then === "function") {
+            p.then(function () {
+              var fresh = scope.getSnapshot();
+              if (fresh.status === "ready" && fresh.value !== void 0) setDraft(Object.assign({}, valueToDraft(fresh.value)));
+            }).catch(function () {});
+            return;
+          }
+        }
+        setTimeout(function () {
+          var fresh = scope.getSnapshot();
+          if (fresh.status === "ready" && fresh.value !== void 0) setDraft(Object.assign({}, valueToDraft(fresh.value)));
+        }, 120);
+      }
+
       function onReset() {
         setBusy(true); setNotice(null); setError(null);
         Promise.all(FIELDS.map(function (f) { return scope.unset(f.key); })).then(function () {
           setBusy(false); setNotice(t("saved"));
+          reseedDraft();
         }).catch(function (e) {
           setBusy(false); setError(t("error") + ": " + String(e && e.message || e));
         });

package/package.json

@@ -1,6 +1,6 @@
 {
     "name":  "dsh-tool-vision",
-    "version":  "0.3.5",
+    "version":  "0.3.7",
     "description":  "DeepSeek Harness 外置视觉模型插件：inspect_image 把本地图片或 http(s) 图片 URL 发给任意 OpenAI 兼容端点，视觉模型看图的文字回答直接带回对话；附 Web UI 设置栏。",
     "type":  "module",
     "main":  "index.js",
@@ -19,7 +19,8 @@
                   "client.js",
                   "README.md",
                   "README.zh.md",
-                  "LICENSE"
+                  "LICENSE",
+                  "vendor"
               ],
     "homepage":  "https://github.com/Scorp1o117/dsh-tool-vision",
     "keywords":  [

package/vendor/dsh-settings-expose.js

@@ -0,0 +1,75 @@
+/**
+ * dsh-host-apiproxy exposes only namespaces listed in its hard-coded
+ * WEB_SETTINGS_NAMESPACES allowlist to the Web settings client. A namespace
+ * registered by a third-party plugin answers `settings-not-exposed` even
+ * though it is registered — upstream explicitly defers letting plugins
+ * expose their own configuration.
+ *
+ * This module idempotently patches that allowlist in the dsh installation
+ * actually loaded by the host process, so the plugin's settings section
+ * becomes visible in the Web UI without manual edits. A dsh update
+ * overwrites the file; the next plugin start re-patches it.
+ */
+import { existsSync, readFileSync, writeFileSync } from "node:fs";
+import { createRequire } from "node:module";
+import { dirname, join, sep } from "node:path";
+
+/**
+ * Ensure `nsName` is present in dsh-host-apiproxy's WEB_SETTINGS_NAMESPACES.
+ * No-op when already exposed or when the file cannot be located/patched.
+ * @param {import("cordis").Context} ctx
+ * @param {string} nsName - settings namespace short name (e.g. "tdai-memory").
+ * @param {{info?: Function, warn?: Function}} logger - dsh logger.
+ */
+export function ensureSettingsNamespaceExposed(ctx, nsName, logger) {
+  void ctx;
+  try {
+    const target = findApiproxyIndex();
+    if (!target) {
+      logger?.warn?.(`[settings-expose] could not locate dsh-host-apiproxy; add "${nsName}" to WEB_SETTINGS_NAMESPACES in dsh-host-apiproxy/lib/index.js to get a Web settings section`);
+      return;
+    }
+    let src;
+    try {
+      src = readFileSync(target, "utf8");
+    } catch (error) {
+      logger?.warn?.(`[settings-expose] cannot read ${target}: ${String(error)}`);
+      return;
+    }
+    const body = src.match(/const WEB_SETTINGS_NAMESPACES = \[([\s\S]*?)\];/)?.[1] ?? "";
+    if (body.includes(`"${nsName}"`)) return; // already exposed (manual or previous patch)
+    const patched = src.replace(/(const WEB_SETTINGS_NAMESPACES = \[[\s\S]*?)(\n\s*\];)/, (_, pre, post) => {
+      const trailingComma = /,\s*$/.test(pre) ? "" : ",";
+      const sep2 = pre.trimEnd().endsWith("[") ? "" : trailingComma;
+      return `${pre}${sep2}\n\t"${nsName}"${post}`;
+    });
+    if (patched === src) {
+      logger?.warn?.(`[settings-expose] allowlist pattern not found in ${target}; add "${nsName}" to WEB_SETTINGS_NAMESPACES manually`);
+      return;
+    }
+    writeFileSync(target, patched, "utf8");
+    logger?.info?.(`[settings-expose] added "${nsName}" to WEB_SETTINGS_NAMESPACES (${target}) — restart dsh web for the settings section to appear`);
+  } catch (error) {
+    logger?.warn?.(`[settings-expose] failed: ${String(error)}`);
+  }
+}
+
+function findApiproxyIndex() {
+  // 1) The host process has already loaded dsh-host-apiproxy: read the real
+  //    module path from the CommonJS module cache (any install layout).
+  try {
+    const Module = createRequire(import.meta.url)("module");
+    const cache = Module._cache ?? {};
+    for (const key of Object.keys(cache)) {
+      if (key.includes(`${sep}dsh-host-apiproxy${sep}`) && key.endsWith(`${sep}index.js`)) return key;
+    }
+  } catch { /* fall through */ }
+  // 2) Fallback: sibling of @deepseek-ai/dsh-settings (dsh's nested layout).
+  try {
+    const require = createRequire(import.meta.url);
+    const settingsEntry = require.resolve("@deepseek-ai/dsh-settings");
+    const candidate = join(dirname(dirname(dirname(settingsEntry))), "dsh-host-apiproxy", "lib", "index.js");
+    if (existsSync(candidate)) return candidate;
+  } catch { /* fall through */ }
+  return "";
+}