dsh-dupguard 1.6.2 → 1.6.3

package/CHANGELOG.md

@@ -2,6 +2,22 @@
 
 本文件遵循 [Keep a Changelog](https://keepachangelog.com/zh-CN/1.1.0/)，版本号遵循 [SemVer](https://semver.org/lang/zh-CN/)。
 
+## [1.6.3] - 2026-09-24
+
+### Fixed
+
+- **写入成功却被判为「保存失败」**：`remote.settings.mutate()` 返回的命名空间视图是**写入前**的快照
+  （配置由 loader 异步重载后才更新），此前的实现拿它与本地期望值做回读比对，于是每次写入都
+  必然不一致——面板报错、看起来「没保存」，而宿主其实已经落盘。现改为**只以宿主应答为准**
+  （`response.ok === false` 才算失败），与官方 `ConfigForm.mutate` 的语义一致；恢复默认同样按
+  「全部 unset 是否未被拒绝」判定。
+
+### Added
+
+- 设置页底部常驻诊断行 `设置通道：<状态>｜<诊断>｜构建 <标记>`，并在写入失败时附带宿主原始原因
+  与构建标记；浏览器控制台同步打印 `[dupguard] mutate → …` / `mutate 被拒…`，用于快速区分
+  「宿主拒绝」「通道未就绪」「回读不一致」三类问题。
+
 ## [1.6.2] - 2026-09-24
 
 ### Fixed

package/lib/client.js

@@ -26,6 +26,8 @@ window.__ModuleLoader__.load({
     const React = require('react')
 
     const NS = 'dsh-dupguard'
+    /** 构建标记：显示在设置页底部，用于判断浏览器实际加载的是哪一版（排查缓存/旧包问题）。 */
+    const BUILD_MARK = '1.6.3'
     /** 旧版（≤ 0.1.6）register 出来的命名空间名。 */
     const SETTINGS_NS = 'dsh-dupguard'
     /** 新版（≥ 0.1.7）命名空间取自 loader entry id：本 bundle 插入的行 id 为 dupguard。 */
@@ -344,28 +346,34 @@ window.__ModuleLoader__.load({
           }
         : null
 
-      // ---- 写路径：settingsScope 控制器（DSH 自己的设置写通道，自动携带 revision、
-      // 串行化并发写、把宿主应答折叠回镜像）。DSH 0.1.2 起客户端不再暴露
-      // connection.api，控制器接口自 0.1.1 起稳定，故不直接触碰 wire 面。 ----
+      // ---- 写路径：设置控制器（新版 remote.settings，旧版 settingsScope）。 ----
+      // 写成功与否**只以宿主应答为准**：宿主返回的视图是写入前的快照（配置由 loader
+      // 异步重载），拿它跟本地期望值比对必然不一致——这正是此前「明明保存成功却报
+      // 保存失败、重开后新值不见了」的原因。官方实现同样是信任 response.ok。
       const snapshotValue = () => {
         const current = controller.getSnapshot()
         return current !== undefined && current.value !== undefined ? current.value : {}
       }
-      const sameList = (a, b) => Array.isArray(a) && Array.isArray(b) &&
-        a.length === b.length && a.every((item, index) => item === b[index])
       const currentForm = () => (form === null ? remoteForm : form)
-      const runWrite = (operation, verify) => {
+      const runWrite = (operation) => {
         setWriteState('saving')
+        // 失败时带上宿主返回的原始原因（writeReason）与构建标记：
+        // 标记同时用于判断浏览器加载的是哪一版（排查旧包/缓存）。
+        const failureText = () => {
+          const reason = typeof props.writeReason === 'function' ? props.writeReason() : null
+          const suffix = reason === null || reason === undefined ? '（宿主未给出原因）' : String(reason)
+          return t('saveFailed') + ' [diag ' + BUILD_MARK + '] ' + suffix
+        }
         Promise.resolve()
           .then(() => operation())
-          .then(() => setWriteState(verify() ? 'saved' : 'error:' + t('saveFailed')), (error) => {
+          .then((accepted) => setWriteState(accepted === false ? 'error:' + failureText() : 'saved'), (error) => {
             setWriteState('error:' + String((error && error.message) || error))
           })
       }
       const commitList = (next) => {
         dirty.current.ignoredChars = true
         setForm({ ...currentForm(), ignoredChars: next })
-        runWrite(() => controller.set('ignoredChars', next), () => sameList(snapshotValue().ignoredChars, next))
+        runWrite(() => controller.set('ignoredChars', next))
       }
       const commitNumber = (field) => {
         const message = fieldError(field)
@@ -376,7 +384,7 @@ window.__ModuleLoader__.load({
         const value = parsed[field.key]
         setErrors((prev) => ({ ...prev, [field.key]: null }))
         // 值未变化时不写入：避免仅仅聚焦/失焦就把默认值写进用户层，
-        // 污染 settings.yaml 并让「恢复默认」失去意义。
+        // 污染配置文件并让「恢复默认」失去意义。
         if (snapshotValue()[field.key] === value) {
           dirty.current[field.key] = false
           setForm({ ...currentForm(), [field.key]: String(value) })
@@ -384,27 +392,20 @@ window.__ModuleLoader__.load({
         }
         dirty.current[field.key] = true
         setForm({ ...currentForm(), [field.key]: String(value) })
-        runWrite(() => controller.set(field.key, value), () => snapshotValue()[field.key] === value)
+        runWrite(() => controller.set(field.key, value))
       }
       const commitBool = (key) => {
         const value = shown[key] !== true
         dirty.current[key] = true
         setForm({ ...currentForm(), [key]: value })
-        runWrite(() => controller.set(key, value), () => snapshotValue()[key] === value)
+        runWrite(() => controller.set(key, value))
       }
       const resetAll = () => {
         dirty.current = {}
         setErrors({})
-        runWrite(
-          () => Promise.all(ALL_FIELDS.map((key) => controller.unset(key))),
-          () => {
-            setForm(toForm(snapshotValue()))
-            const current = controller.getSnapshot()
-            const user = current !== undefined ? current.user : undefined
-            if (user === undefined || user === null) return true
-            return ALL_FIELDS.every((key) => user[key] === undefined)
-          },
-        )
+        // 恢复默认 = 逐字段 unset；接受与否同样只看宿主应答。
+        runWrite(() => Promise.all(ALL_FIELDS.map((key) => controller.unset(key)))
+          .then((results) => results.every((item) => item !== false)))
       }
 
       const add = () => {
@@ -513,6 +514,11 @@ window.__ModuleLoader__.load({
               : writeState === 'saved'
                 ? React.createElement('p', { className: 'dg-note' }, t('saved'))
                 : React.createElement('p', { className: 'dg-note dg-error' }, writeState),
+          React.createElement('p', { className: 'dg-note' },
+            fmt(t('loadingDiag'), {
+              state: mirrorSnap ? String(mirrorSnap.status) : 'unknown',
+              diag: (mirrorSnap && mirrorSnap.diag ? String(mirrorSnap.diag) : '') + '｜构建 ' + BUILD_MARK,
+            })),
         ),
       )
     }
@@ -657,6 +663,32 @@ window.__ModuleLoader__.load({
             if (!closed && mine === generation) loaded = true
           }
         }
+        /** 写入失败时面板上展示的宿主原始原因（诊断用）。 */
+        let lastWriteError = null
+
+        /** 只尝试属于本插件的命名空间名：describe 报出的 ns 与其去掉组合前缀的形式。 */
+        const namespaceCandidates = () => {
+          const list = [resolvedNamespace]
+          const cut = resolvedNamespace.lastIndexOf(':')
+          if (cut !== -1 && cut + 1 < resolvedNamespace.length) {
+            const stripped = resolvedNamespace.slice(cut + 1)
+            if (list.indexOf(stripped) === -1) list.push(stripped)
+          }
+          return list
+        }
+
+        const attempt = async (namespace, owned, expected) => {
+          let response
+          try {
+            response = await settings.mutate(namespace, owned, expected)
+          } catch (error) {
+            return { ok: false, error: errorText(error) }
+          }
+          if (response === null || typeof response !== 'object') return { ok: false, error: '宿主应答无效' }
+          if (response.ok === true) return { ok: true, value: response.value }
+          return { ok: false, error: responseError(response, '宿主拒绝了该写入') }
+        }
+
         const mutate = (ops) => {
           const owned = ops.map((op) => (op.op === 'set'
             ? { op: 'set', path: [...op.path], value: op.value }
@@ -665,26 +697,40 @@ window.__ModuleLoader__.load({
             if (closed || channel === null) return false
             // 首次写入前先完成一次 describe：否则命名空间与 revision 都还是初始值。
             if (!loaded) await load()
-            let response
-            try {
-              response = await settings.mutate(resolvedNamespace, owned, revision)
-            } catch (_error) {
-              await load()
-              return false
-            }
-            if (response === null || typeof response !== 'object' || response.ok !== true) {
-              await load() // 写失败：回读宿主真实状态，避免界面停留在错误的本地值
-              return false
+            const failures = []
+            for (const namespace of namespaceCandidates()) {
+              console.info('[dupguard] mutate → ' + namespace + ' revision=' + String(revision) +
+                ' paths=' + JSON.stringify(owned.map((op) => op.path)))
+              let result = await attempt(namespace, owned, revision)
+              if (!result.ok) {
+                // 常见于并发写入导致的 revision 过期：回读后按新 revision 重试一次。
+                console.warn('[dupguard] mutate 被拒（' + namespace + '）：' + result.error + '，回读后重试')
+                await load()
+                result = await attempt(namespace, owned, revision)
+              }
+              if (result.ok) {
+                lastWriteError = null
+                if (!closed) acceptView(result.value)
+                publish(undefined, { status: mirror.status, error: null, diag: '已写入命名空间 ' + namespace })
+                return true
+              }
+              failures.push(namespace + '：' + result.error)
             }
-            if (!closed) acceptView(response.value)
-            return true
+            lastWriteError = failures.join('；')
+            // 写失败：回读宿主真实状态，避免界面停留在错误的本地值；并把原因显示到面板。
+            await load()
+            publish(undefined, { status: mirror.status, error: '写入被拒绝 —— ' + lastWriteError, diag: 'mutate 失败' })
+            return false
           })
           tail = task.then(() => {}, () => {})
           return task
         }
+        /** 供组件展示的最近一次写入失败原因。 */
+        const writeError = () => lastWriteError
         return {
           load,
           mutate,
+          writeError,
           close: () => {
             closed = true
           },
@@ -763,12 +809,20 @@ window.__ModuleLoader__.load({
         controller: {
           subscribe,
           getSnapshot: () => snapshot,
-          set: (field, value) => (channel === null
-            ? Promise.resolve(false)
-            : channel.mutate([{ op: 'set', path: [field], value }])),
-          unset: (field) => (channel === null
-            ? Promise.resolve(false)
-            : channel.mutate([{ op: 'unset', path: [field] }])),
+          set: (field, value) => {
+            if (channel === null) {
+              lastWriteError = '设置通道未就绪（channel=null）'
+              return Promise.resolve(false)
+            }
+            return channel.mutate([{ op: 'set', path: [field], value }])
+          },
+          unset: (field) => {
+            if (channel === null) {
+              lastWriteError = '设置通道未就绪（channel=null）'
+              return Promise.resolve(false)
+            }
+            return channel.mutate([{ op: 'unset', path: [field] }])
+          },
         },
         mirror: {
           subscribe,
@@ -781,6 +835,8 @@ window.__ModuleLoader__.load({
         },
         /** 更新面板上的通道诊断文本（不改变数据状态）。 */
         note: (text) => publish(undefined, { status: mirror.status, error: mirror.error, diag: text }),
+        /** 最近一次写入失败的宿主原因（旧版通道不提供时返回 null）。 */
+        writeReason: () => (channel !== null && typeof channel.writeError === 'function' ? channel.writeError() : null),
         disposed: () => disposed,
         /** 是否已有可用通道（旧版接上后即可停止新版探测）。 */
         hasChannel: () => channel !== null,
@@ -921,6 +977,7 @@ window.__ModuleLoader__.load({
         // 传函数而非快照：远程/只读状态可能由 describe 的 writable 或
         // remote.$host.isLoopback 稍后确定，渲染时必须取最新值。
         isLoopback: () => bridge.isLoopback(),
+        writeReason: () => bridge.writeReason(),
       })
       ctx.slots.inject('settings.section', () => ctx.slots.register({
         name: 'settings.section',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-dupguard",
-  "version": "1.6.2",
+  "version": "1.6.3",
   "description": "Real-time repetition guard for DeepSeek Harness (DSH): stops model generation when the same string repeats >=10 times in the streamed output. 实时检测 DSH 大模型流式输出中的重复内容，同一字符串重复十次以上立即停止生成。",
   "keywords": [
     "dsh-plugin",