dsh-conversation-recycle-bin 0.3.20 → 0.3.21

package/CHANGELOG.md

@@ -3,6 +3,23 @@
 All notable changes to `dsh-conversation-recycle-bin` are documented here.
 The format follows Keep a Changelog; the project uses Semantic Versioning.
 
+## [0.3.21] — 2026-11-09
+
+CI 第一次真正跑起来就抓到一条**只在中文机器上通过**的用例。0.3.20 发布后推送仓库，四个矩阵任务（ubuntu/windows × Node 22.15/24）全部在 `npm test` 阶段失败，失败点相同。
+
+### Fixed
+
+- **`client-module.test.mjs` 把中文写进了断言**（`the background-sweep notice carries how long ago it ran`）：`__formatAgo` 的「刚刚」被硬编码成字面量。`client.js` 的 `currentLocaleId()` 先读宿主的 `locale` 服务、读不到才回退 `navigator.language`，而这个用例没有安装 locale 服务，于是断言跟着**运行机器**的语言走——作者的中文 Windows 通过，所有英文 locale 的 CI 与贡献者机器都失败（`'just now' !== '刚刚'`）。这正是 0.3.17 修过的同一类缺陷：**测试没有锁定 locale，产品行为其实是对的**。
+- 现在这条用例对 `en` 与 `zh-CN` **各跑一遍**，每次通过新加的 `loadClientWithLocale()` 显式安装 locale 服务，并逐段断言「刚刚 / 30 分钟前 / 3 小时前 / 2 天前」与英文文案，顺手把两套字典的相对时间文案一起钉住。
+
+### Added
+
+- **CI 把失败摘要发布成 workflow annotation**：失败任务的原始日志需要仓库凭据才能读，只看到 `exit code 1` 无法区分「locale 相关的断言断错」和「夹具缺失」。`test` 步骤现在捕获输出，失败时把失败用例的名称、`expected`/`actual` 与所在文件行号以 `::error::` 注释发出，无需凭据即可读（本条缺陷就是这样定位的）。
+
+### Tests
+
+- 用例总数不变（221），`client-module.test.mjs` 32/32。
+
 ## [0.3.20] — 2026-11-09
 
 一次只改工程效率、不改删除语义的整理：把测试墙钟从 **60.5s 压到 4.6s**，堵住「跑过测试再打包」会把本机测试残留塞进 npm 包的路径，删掉三处零引用导出，并把两个确认前的**读**循环改成有界并发。生产路径的锁预算、超时与策略判定全部不变。

package/README.md

@@ -2,7 +2,7 @@
 
 给 DeepSeek Harness (DSH) Web 增加「删除对话」：会话列表的「…」菜单与会话详情页头部各有一个删除入口；确认框内可勾选「同时删除关联记忆」（默认不勾选），并实时显示将被影响的记忆条数。删除默认进入插件回收站，可恢复；每次操作写入审计日志；批量删除有上限与限流，批量软删除需输入数量，彻底删除一键执行；事务任一步失败即回滚。
 
-> 状态：`0.3.20`，Host + Web Client 双半，零构建（plain JS ESM），只依赖宿主已发布的公开扩展点。
+> 状态：`0.3.21`，Host + Web Client 双半，零构建（plain JS ESM），只依赖宿主已发布的公开扩展点。
 > 运行要求：**Node.js ≥ 22.15**（Host 侧用到 `readdir(..., { recursive: true })`、`AbortSignal` 等 22.x 行为），DSH `>=0.2.1-rc.2 <0.3.0-0`（`package.json` 的 `engines.dsh` 已如实声明，DSH 安装校验会据此拒绝范围外版本；上界是刻意的：0.2.2 与 0.3.x 之间没有承诺兼容）。
 > 包名：`dsh-conversation-recycle-bin`（原 `dsh-delete-conversation` 因 npm 被他人占用而改名，见 CHANGELOG 0.2.0）。
 > 本 README 同时是设计说明与验收依据；安装/回滚步骤见 [INSTALL.md](INSTALL.md)；变更见 [CHANGELOG.md](CHANGELOG.md)。

package/lib/version.js

@@ -5,4 +5,4 @@
  * asserted against it by `test/manifest.test.mjs` so the two can never drift
  * (they did: the constant stayed at 0.1.0 while the package reached 0.1.4).
  */
-export const PLUGIN_VERSION = '0.3.20'
+export const PLUGIN_VERSION = '0.3.21'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-conversation-recycle-bin",
-  "version": "0.3.20",
+  "version": "0.3.21",
   "description": "DSH conversation deletion with an optional, previewed cascade into conversation-associated memory: soft delete + recycle bin, audit trail, batch rate limits and per-session rollback. 删除对话（可选一并删除关联记忆）：软删除与回收站、审计留痕、批量限流、失败回滚。",
   "repository": {
     "type": "git",

package/test/client-module.test.mjs

@@ -81,6 +81,32 @@ async function loadClient({ withReactDom = false } = {}) {
   return { exported, registrations, style, moduleId, body, portalCalls }
 }
 
+/**
+ * Load the Client with an explicit active locale.
+ *
+ * `client.js` resolves the locale from the host's `locale` service and only
+ * then falls back to `navigator.language`, so a suite that just calls
+ * `loadClient()` silently follows whatever language the MACHINE is set to.
+ * That is how a Chinese literal in an assertion passed on a zh-CN laptop and
+ * failed on every English-locale CI runner. Anything that asserts user-visible
+ * text must pin the locale here instead.
+ */
+async function loadClientWithLocale(localeId) {
+  const { exported } = await loadClient()
+  exported.apply({
+    slots: {
+      inject(key, callback) { return callback() ?? (() => {}) },
+      register() { return () => {} },
+    },
+    effect(fn) { fn() },
+    get(name) {
+      if (name === 'locale') return { getSnapshot: () => ({ active: localeId }) }
+      return undefined
+    },
+  })
+  return exported
+}
+
 test('client: module id equals the package name the boot graph keys rows by', async () => {
   const { moduleId } = await loadClient()
   const pkg = JSON.parse(readFileSync(new URL('../package.json', import.meta.url), 'utf8'))
@@ -1092,11 +1118,22 @@ test('client module: an incomplete entry offers purge when the Host allows it',
 })
 
 test('client module: the background-sweep notice carries how long ago it ran', async () => {
-  const { exported } = await loadClient()
   const now = Date.now()
-  assert.equal(exported.__formatAgo(now - 5_000, now), '刚刚')
-  assert.match(exported.__formatAgo(now - 30 * 60_000, now), /分钟前|min ago/)
-  assert.match(exported.__formatAgo(now - 3 * 3_600_000, now), /小时前|h ago/)
-  assert.match(exported.__formatAgo(now - 2 * 86_400_000, now), /天前|d ago/)
-  assert.equal(exported.__formatAgo(undefined, now), '')
+  // Both languages, each with an explicitly installed locale service: a single
+  // Chinese literal here (`'刚刚'`) made this assertion depend on the machine's
+  // `navigator.language`, so it failed on every English-locale runner while
+  // passing on the author's zh-CN laptop. Pinning both tables also proves the
+  // relative-time wording exists on both sides.
+  const cases = [
+    ['en', { justNow: 'just now', minutes: /^30 min ago$/, hours: /^3 h ago$/, days: /^2 d ago$/ }],
+    ['zh-CN', { justNow: '刚刚', minutes: /^30 分钟前$/, hours: /^3 小时前$/, days: /^2 天前$/ }],
+  ]
+  for (const [localeId, expected] of cases) {
+    const client = await loadClientWithLocale(localeId)
+    assert.equal(client.__formatAgo(undefined, now), '', `${localeId}: no timestamp renders nothing`)
+    assert.equal(client.__formatAgo(now - 5_000, now), expected.justNow, `${localeId}: seconds`)
+    assert.match(client.__formatAgo(now - 30 * 60_000, now), expected.minutes, `${localeId}: minutes`)
+    assert.match(client.__formatAgo(now - 3 * 3_600_000, now), expected.hours, `${localeId}: hours`)
+    assert.match(client.__formatAgo(now - 2 * 86_400_000, now), expected.days, `${localeId}: days`)
+  }
 })