dsh-instruction-memory 1.3.0 → 1.3.1

package/CHANGELOG.md

@@ -7,6 +7,22 @@
 > 1.0.13 起由 CI 经 OIDC Trusted Publishing 发布（首个带 provenance 的版本）。
 > 这些历史 tag **不应回填**——npm 上的版本号已被占用。
 
+## [1.3.1] — 2026-10-15
+
+1.3.0 发布当天的评审跟进：一个披露计数回归与三个展示层小修正。
+
+### 修复
+
+- **ui-test 去竞态**：F7 真实定时器冒烟中「暂停后立刻断言注入已静音」与 50ms 定时器存在调度竞态
+  （高负载 runner 上定时器可能已过期），改为确定性地断言到期时刻已写入存储文件，并把到期等待
+  从 300ms 放宽到 1s。仅测试基建，不影响已发布的包内容。
+- **导入回退去重后的计数披露**：「新增指令 N 条 / 新增用户形象 N 个」按实际落库数（staged）
+  统计，不再包含被回退去重跳过的项；截断披露同样只统计实际落库项。80f94f5 引入的口径回归，
+  contract-test 新增断言钉住「2 通过 − 1 回退重复 = 新增指令 1 条」。
+- **暂停倒计时与复制路径的三个小修正**：新起暂停时先取当前时刻再起秒表（否则长开面板的
+  首帧剩余时间可能偏大最多 1 秒，下一秒自愈）；他窗发起暂停时复位暂停菜单展开态；
+  剪贴板 execCommand 抛错路径也移除游离的辅助 textarea。均为 1.3.0 评审发现，仅展示层。
+
 ## [1.3.0] — 2026-10-15
 
 两个审查修复与两项体验增强（F12 / F7）。

package/lib/client.js

@@ -1600,6 +1600,13 @@ window.__ModuleLoader__.load({
         && view.injection.muted === true
       React.useEffect(() => {
         if (!mutedNow) return undefined
+        // A pause started from ANOTHER window replaces the menu row; the open
+        // flag must not resurface when the row returns after expiry.
+        setPauseMenuOpen(false)
+        // Snapshot the clock the moment the pause lands: nowTick may be up to
+        // a second stale from a PREVIOUS pause, which would render the first
+        // frame with an inflated remaining time.
+        setNowTick(Date.now())
         const tick = setInterval(() => setNowTick(Date.now()), 1000)
         return () => clearInterval(tick)
       }, [mutedNow])
@@ -1984,8 +1991,14 @@ window.__ModuleLoader__.load({
           const host = document.body || document.head
           if (host && typeof host.appendChild === 'function') host.appendChild(area)
           if (typeof area.select === 'function') area.select()
-          const ok = typeof document.execCommand === 'function' && document.execCommand('copy') === true
-          if (host && typeof host.removeChild === 'function' && area.parentNode === host) host.removeChild(area)
+          // execCommand may THROW (not merely return false); the detached
+          // helper must leave the DOM on every path.
+          let ok = false
+          try {
+            ok = typeof document.execCommand === 'function' && document.execCommand('copy') === true
+          } finally {
+            if (host && typeof host.removeChild === 'function' && area.parentNode === host) host.removeChild(area)
+          }
           if (ok) {
             setNotice(t('diag.copied'))
             return

package/lib/index.js

@@ -3372,8 +3372,6 @@ export function apply(ctx) {
     const seenIds = new Set(existing.map((entry) => entry.setId + '\u0000' + entry.id))
     const seenBodies = new Set(existing.map((entry) => bodyKey(entry.setId, entry.title, entry.content)))
     const accepted = []
-    const cutFields = new Set()
-    let truncatedCount = 0
     let skipped = 0
     for (const candidate of (rawEntries || [])) {
       const entry = sanitizeEntry(candidate)
@@ -3390,17 +3388,12 @@ export function apply(ctx) {
         skipped += 1
         continue
       }
+      // Truncation travels per item and is only REPORTED for items that are
+      // actually staged: an item later skipped as a fallback duplicate must
+      // not inflate the truncation disclosure (or the inserted-count).
       const cut = truncatedFields(candidate)
-      if (cut.length > 0) {
-        // sanitizeEntry enforces the limits silently, which is fine for the
-        // editor (its inputs carry the same maxLength) but wrong here: an
-        // imported 7,000-character instruction would come back as 6,000 with
-        // nothing said about the loss.
-        truncatedCount += 1
-        for (const field of cut) cutFields.add(field)
-      }
       entry.updatedAt = Date.now()
-      accepted.push({ entry, wanted })
+      accepted.push({ entry, wanted, cut })
       seenIds.add(wanted + '\u0000' + entry.id)
       seenBodies.add(bodyKey(wanted, entry.title, entry.content))
     }
@@ -3411,8 +3404,6 @@ export function apply(ctx) {
     const seenProfileIds = new Set(existingProfiles.map((profile) => profile.setId + '\u0000' + profile.id))
     const seenProfileBodies = new Set(existingProfiles.map((profile) => profile.setId + '\u0000' + profile.name + '\u0000' + profile.content))
     const acceptedProfiles = []
-    const cutProfileFields = new Set()
-    let truncatedProfileCount = 0
     let skippedProfiles = 0
     for (const candidate of (rawProfiles || [])) {
       const profile = sanitizeProfile(candidate)
@@ -3431,12 +3422,8 @@ export function apply(ctx) {
         continue
       }
       const cut = truncatedProfileFields(candidate)
-      if (cut.length > 0) {
-        truncatedProfileCount += 1
-        for (const field of cut) cutProfileFields.add(field)
-      }
       profile.updatedAt = Date.now()
-      acceptedProfiles.push({ profile, wanted })
+      acceptedProfiles.push({ profile, wanted, cut })
       seenProfileIds.add(wanted + '\u0000' + profile.id)
       seenProfileBodies.add(wanted + '\u0000' + profile.name + '\u0000' + profile.content)
     }
@@ -3499,7 +3486,7 @@ export function apply(ctx) {
           finalSeenBodies.add(bodyKey(finalId, item.entry.title, item.entry.content))
         }
         item.entry.setId = finalId
-        stagedEntries.push(item.entry)
+        stagedEntries.push(item)
       }
       const finalSeenProfileIds = new Set(existingProfiles.map((profile) => profile.setId + '\u0000' + profile.id))
       const finalSeenProfileBodies = new Set(existingProfiles.map((profile) => profile.setId + '\u0000' + profile.name + '\u0000' + profile.content))
@@ -3522,7 +3509,7 @@ export function apply(ctx) {
           finalSeenProfileBodies.add(finalId + '\u0000' + item.profile.name + '\u0000' + item.profile.content)
         }
         item.profile.setId = finalId
-        stagedProfiles.push(item.profile)
+        stagedProfiles.push(item)
       }
     }
     if (stagedEntries.length === 0 && stagedProfiles.length === 0) {
@@ -3538,8 +3525,8 @@ export function apply(ctx) {
     }
     const saved = await commit(() => {
       state.data.sets.push(...createdSets)
-      for (const entry of stagedEntries) state.data.entries.push(entry)
-      for (const profile of stagedProfiles) state.data.profiles.push(profile)
+      for (const item of stagedEntries) state.data.entries.push(item.entry)
+      for (const item of stagedProfiles) state.data.profiles.push(item.profile)
     })
     const parts = []
     if (createdSets.length > 0) {
@@ -3551,19 +3538,38 @@ export function apply(ctx) {
     if (dedupedOnFallback > 0) {
       parts.push('另有 ' + dedupedOnFallback + ' 项因回退到当前配置集后与已有内容重复被跳过')
     }
-    if (accepted.length > 0) {
-      const truncated = truncatedCount > 0
-        ? '；其中 ' + truncatedCount + ' 条超过长度上限，' + [...cutFields].join('、') + '已被截断'
+    if (stagedEntries.length > 0) {
+      // Counts quote what actually LANDED: staged ⊆ accepted, and the gap is
+      // the fallback duplicates already disclosed on their own line — an
+      // accepted-based count would claim inserts that were skipped.
+      const stagedCut = new Set()
+      let stagedTruncated = 0
+      for (const item of stagedEntries) {
+        if (item.cut.length > 0) {
+          stagedTruncated += 1
+          for (const field of item.cut) stagedCut.add(field)
+        }
+      }
+      const truncated = stagedTruncated > 0
+        ? '；其中 ' + stagedTruncated + ' 条超过长度上限，' + [...stagedCut].join('、') + '已被截断'
         : ''
-      parts.push('新增指令 ' + accepted.length + ' 条，跳过 ' + skipped + ' 条' + truncated)
+      parts.push('新增指令 ' + stagedEntries.length + ' 条，跳过 ' + skipped + ' 条' + truncated)
     } else if (skipped > 0) {
       parts.push('指令跳过 ' + skipped + ' 条（已存在或无效）')
     }
-    if (acceptedProfiles.length > 0) {
-      const truncated = truncatedProfileCount > 0
-        ? '；其中 ' + truncatedProfileCount + ' 个超过长度上限，' + [...cutProfileFields].join('、') + '已被截断'
+    if (stagedProfiles.length > 0) {
+      const stagedCut = new Set()
+      let stagedTruncated = 0
+      for (const item of stagedProfiles) {
+        if (item.cut.length > 0) {
+          stagedTruncated += 1
+          for (const field of item.cut) stagedCut.add(field)
+        }
+      }
+      const truncated = stagedTruncated > 0
+        ? '；其中 ' + stagedTruncated + ' 个超过长度上限，' + [...stagedCut].join('、') + '已被截断'
         : ''
-      parts.push('新增用户形象 ' + acceptedProfiles.length + ' 个，跳过 ' + skippedProfiles + ' 个' + truncated)
+      parts.push('新增用户形象 ' + stagedProfiles.length + ' 个，跳过 ' + skippedProfiles + ' 个' + truncated)
     } else if (skippedProfiles > 0) {
       parts.push('用户形象跳过 ' + skippedProfiles + ' 个（已存在或无效）')
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-instruction-memory",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "User-only instruction memory for DSH: long-term instructions maintained in the settings UI and auto-injected into every conversation — the model has no write access. 指令记忆：只由用户在设置页维护的长期指令，自动注入此后每轮对话；模型没有任何写入口。",
   "license": "MIT",
   "type": "module",