@rennimars/dsh-weknora-memory 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 0.3.1 — 常驻块按用户轮次投递
+- **修（超时）**：`lib/weknora.js` 的 `request()` 改用自带 `AbortController` + `setTimeout` 计时，不再用 `AbortSignal.timeout()`——后者的定时器是 unref 的，事件循环里没有别的句柄时（Node 20 的 `node --test` 进程正是这样）它会随循环一起退出，挂起的请求永远不会结算，整份测试文件以 `cancelledByParent` 收场。现在 `finally` 里清计时器，超时原因是 `timeout`、连接错误仍是 `transport`。
+
+- **修（画像闸门）**：`writers/profile-writer.mjs` 的复核此前只要「有来源 / 不重复 / 正文 ≤60 字」就把 `proposed` 转 `active`——一晚的静默被当成了决定。现在补上 `export const ATTEST_MIN = 3`：同一话题被反复看到（`propose()` 已有的 `attest` 合并计数）至少 3 次，或卡上 `approve: true`（`--approve` / `decide()` 写入的人工批准），才允许转正，否则记 `needs_approval_or_attest_N_of_3` 保持 `proposed`。
+- **修（CI）**：`test/inject.test.mjs`、`test/layers.test.mjs` 曾 import 本机私有兄弟目录 `../../../_wk_dream/profile-writer.mjs`，本机绿、CI 上 `ERR_MODULE_NOT_FOUND`。改为仓库自带的 `../writers/profile-writer.mjs`。`test/writers.test.mjs` 的 W4 相应改成「人工批准的卡转正 + 只有 2 次证言的卡仍留 proposed」。66 个用例全绿。
+
+- **修**：常驻/画像块（`pinned_constraints`，实测约 913 字）此前在**每个 step** 都重贴一次；DSH 的 `step()` 会把注入消息 `session.append('user/message', ...)` 写进会话，同一轮的后续步既读得到又不需要第二份。实测 `logs/inject.log` 2975 条里 2386 条（80%）是 `no_user_text` 的纯重播。
+- **改**：`lib/injector.js` 新增 `pinnedForStep(sessionId, prompt)`，按本步的 prompt 文本给常驻块去重——工具续步没有 user 文本（`promptFromMessages` 返回 `""`），同一句 prompt 重复出现也算同轮。会话内第一块永远投递（`pinnedPrompt` 无该会话记录时不去重），所以「首步没有用户文本」不会让整场会话丢常驻层。
+- **配置**：新增 `pinnedOncePerTurn`（默认 `true`，`lib/config.js`）。设为 `false` 恢复旧的每步投递。
+- **日志**：被去重的步记 `pinned_skipped: true`、`pinned_chars: 0`，便于对前后数字。
+- **测试**：新增 2 项（`test/inject.test.mjs`）：同轮续步/同句重问不复投；`pinnedOncePerTurn: false` 保持旧行为。5 个测试文件全绿。
+
 ## 0.3.0 — 写入侧随包发布（可选组件）
 
 - **新增 `lib/weknora.js`**：读侧与写入侧共用的唯一 HTTP 客户端（base URL 拼接、`X-API-Key` 头、超时、`no_api_url|timeout|transport|http_<status>` 错误词表）。`lib/retrieve.js` 的两处内联 fetch 已改为调用它，59 个既有用例保持全绿。

package/README.md

@@ -25,6 +25,8 @@
 | **画像层** `profile` | soul 规则 / user 画像 / agent 工作习惯 | soul 6 张·300 字 · user 3 张·200 字 · agent 6 张·400 字，单卡 ≤80 字 | 永不打分，按类配额裁剪 |
 | **检索层** `weknora_context` | 本轮检索命中的片段 | ≤2400 字符 · 最多 3 条 | 打分 + 衰减 + 分层共同决定 |
 
+常驻/画像块按**用户轮次**投递（`pinnedOncePerTurn`，默认开）：同一轮的后续工具步只带检索片段，不再重贴同一份常驻内容——DSH 会把注入消息写进会话历史，第二份本来就是纯冗余。一个十步的回答，光这一项就省下九份约 900 字的块。
+
 三份预算是**相加而不是共享**的：检索块撑满 2400 字符也不会挤掉常驻准则（`lib/config.js:26,31`）；画像层按类型分账，避免 user 卡把 agent 层吃空（`lib/config.js:104-110`）。
 
 上限是硬编码契约，不是建议值：`timeoutMs` 被钳到 **4000ms**、`maxChars` 被钳到 **2400**（`lib/config.js:19,26` 的 `TIMEOUT_CAP_MS` / `CHARS_CAP`），手改配置文件也撑不破。
@@ -57,7 +59,7 @@ agent/pre-step
 | `DSH-Session-Ingest` | 每小时 | 把**已结束且空闲 ≥10 分钟**的会话转录**原样**入库，跳过子代理会话；单篇 ≤200,000 字符，最多保留 150 篇；失败重试 3 次，入库后轮询索引 ≤600 秒 | 库① 会话记忆 |
 | `DSH-Session-Distill` | 每日 04:30 | 从最近会话里抽取结论 / 约束 / 偏好 / 教训，一次最多 3 个会话、3 篇文档，输入 ≤30,000 字符、每轮 ≤1500 字符、每会话 ≤8 条，语义去重阈值 0.66 | 库② 知识 |
 | `DSH-Memory-Dream` | 每日 03:30 | **夜间做梦**：五阶段 S0 收拢 → S1 分组 → S2 裁决 → S3 落账 → S4 报告。裁决同主题事实卡的取代 / 冲突 / 重复，**只改 header 里的状态字段，正文逐字保留**；单夜 ≤60 次 LLM 调用、≤20 次更新；支持 `--dry-run` / `--no-llm` 零成本演练 | 库② 知识（事实卡状态） |
-| `profile-writer` | 紧接做梦之后，且**只在做梦 rc=0 时** | `data/profile.json` 的**唯一写入者**：新卡一律 `proposed`，至少等一夜才复核；复核必须同时满足「有来源 / 不重复 / 不冲突 / 正文 ≤60 字」才转正，否则保持 `proposed` 并记日志；被插件标了 degrade 的 `active` 卡转 `disputed`，下一轮起不再进常驻块 | 常驻层画像 |
+| `profile-writer` | 紧接做梦之后，且**只在做梦 rc=0 时** | `data/profile.json` 的**唯一写入者**：新卡一律 `proposed`，至少等一夜才复核；复核必须同时满足「有来源 / 不重复 / 不冲突 / 正文 ≤60 字」，且同一话题被反复看到至少 3 次（`attest ≥ 3`，`ATTEST_MIN`）或人显式 `--approve` 批准，才转正；否则保持 `proposed` 并记日志；被插件标了 degrade 的 `active` 卡转 `disputed`，下一轮起不再进常驻块 | 常驻层画像 |
 | `DSH-Memory-Weekly` | 每周 | 体检：跨库重复、失效文档、状态异常 | 三库 |
 
 三库的分工不是随手划的：
@@ -68,7 +70,7 @@ agent/pre-step
 
 **为什么写入侧必须独立于插件：** 写要 LLM、要数据库权限、要能被人工审计；而读侧每轮都跑，必须零依赖、可静默失败。两者混在一起，任何一次写入故障都会变成一次会话故障。
 
-**为什么画像文件只允许一个写入者：** 常驻内容每轮注入、跨会话传播——一旦被污染，后面没有任何一次检索能把它纠正回来。所以插件侧只读，任何新内容只能走 `proposed → 隔夜复核 → active` 这条闸门。
+**为什么画像文件只允许一个写入者：** 常驻内容每轮注入、跨会话传播——一旦被污染，后面没有任何一次检索能把它纠正回来。所以插件侧只读，任何新内容只能走 `proposed → 隔夜复核（≥3 次证言或人工批准）→ active` 这条闸门——一晚的静默本身不是决定。
 
 ### 本包自带的写入侧参考实现（`writers/`，默认不启用）
 
@@ -201,3 +203,22 @@ npm test          # node --test test/*.mjs，59 个用例
 ## License
 
 MIT © RenniMars
+
+## 发布流程
+
+版本号、CHANGELOG 与 git tag 三处同步，发布由 CI 完成，不在本机手工 `npm publish`：
+
+1. 改代码 → 在 `CHANGELOG.md` 顶部加一节（版本号 + 变更）→ `package.json` 的 `version` 改成同一个号。
+2. `npm test` 全绿。
+3. `git commit` → `git tag vX.Y.Z` → `git push origin main --follow-tags`。
+4. `.github/workflows/release.yml` 校验 tag 与 `package.json` 版本一致后，跑测试、`npm publish --provenance`（用的是仓库 secret `NPM_TOKEN`），再发 GitHub Release。
+5. `.github/workflows/ci.yml` 在每次 push / PR 上跑 Node 20 与 22 两个版本的测试。
+
+回滚：npm 侧 `npm deprecate @rennimars/dsh-weknora-memory@X.Y.Z "原因"` 并发布修正版；GitHub 侧 `git revert` 后重新打 tag，不移动已有 tag。
+
+## 路线图 / Roadmap
+
+本项目的演进计划分阶段推进：先给"检索准不准"一把可量化的尺子，再让事实卡的冲突能被裁决、遗忘能被解释、主题能被归并，最后才谈写入门控。
+完整计划（13 节，含每阶段的目标、改动点、验收与回滚）：[docs/ROADMAP.md](docs/ROADMAP.md)
+当前版本 0.3.1；计划可分批实现、每批独立验收，欢迎在 Issue 区按阶段提方案。
+想改这份计划，请先开 Issue 讨论，达成一致后再改文档，不要直接改 ROADMAP.md。

package/lib/config.js

@@ -110,6 +110,13 @@ export const DEFAULT_CONFIG = Object.freeze({
    * a ceiling, not a target -- a file with fewer cards renders a shorter block.
    */
   pinnedMaxChars: 1500,
+  /**
+   * The pinned block is configuration, not retrieval: it renders identically on every step
+   * of one user turn. DSH persists each injected message into the session, so one copy per
+   * turn is still readable while later steps would only repeat it. true = once per turn,
+   * false = the old per-step behaviour.
+   */
+  pinnedOncePerTurn: true,
   /**
    * Standing profile facts (data/profile.json). The plugin only ever reads this file:
    * every write belongs to the nightly task (_wk_dream/profile-writer.mjs). That is the
@@ -507,6 +514,7 @@ export function loadConfig(overrides = {}, options = {}) {
     panelEnabled: merged.panelEnabled !== false,
     pinned: normalizePinned(merged.pinned),
     pinnedMaxChars: clampInt(merged.pinnedMaxChars, DEFAULT_CONFIG.pinnedMaxChars, 120, PINNED_MAX_CHARS_CAP),
+    pinnedOncePerTurn: merged.pinnedOncePerTurn !== false,
     profile,
     decay: normalizeDecay(merged.decay, packageRoot),
     tier: normalizeTier(merged.tier, env),

package/lib/injector.js

@@ -341,6 +341,14 @@ export function createWeKnoraInjector(options = {}) {
   const retrieve = options.retrieve ?? searchWeKnora;
   const injected = new Map();
   const lastPrompt = new Map();
+  /**
+   * The prompt each session's pinned block was last injected for. The pinned layer is
+   * configuration: it cannot change between two steps of one user turn, and DSH appends
+   * every injected message to the session, so the step-1 copy stays readable from history.
+   * Re-injecting it per step therefore buys no information and costs the whole block each
+   * time -- measured at 80% of all injection records being pinned-only repeats.
+   */
+  const pinnedPrompt = new Map();
 
   const seenFor = (sessionId) => {
     let set = injected.get(sessionId);
@@ -396,6 +404,35 @@ export function createWeKnoraInjector(options = {}) {
     return { ...pinned, fields };
   };
 
+  /**
+   * The pinned block for this step: built once per user turn, empty on a repeat.
+   *
+   * "Once per user turn" is keyed on the step's own prompt text, because that is the only
+   * turn identity this hook sees: a tool-continuation step carries no user-role text at
+   * all. The first step of a session always gets the block (pinnedPrompt has no entry for
+   * it yet), so a step that arrives without user text can never cost a session its
+   * standing rules; after that, an empty prompt or an identical one is a repeat.
+   * pinnedOncePerTurn: false restores the old behaviour of injecting on every step.
+   */
+  const pinnedForStep = (sessionId, prompt) => {
+    if (config.pinnedOncePerTurn !== false && pinnedPrompt.has(sessionId)
+      && (prompt === "" || pinnedPrompt.get(sessionId) === prompt)) {
+      return {
+        text: "",
+        skipped: true,
+        fields: { pinned_chars: 0, pinned_count: 0, pinned_truncated: false, pinned_skipped: true },
+      };
+    }
+    const pinned = pinnedFor();
+    if (config.pinnedOncePerTurn !== false) {
+      if (pinnedPrompt.size >= MAX_SESSIONS && !pinnedPrompt.has(sessionId)) {
+        pinnedPrompt.delete(pinnedPrompt.keys().next().value);
+      }
+      pinnedPrompt.set(sessionId, prompt);
+    }
+    return pinned;
+  };
+
   const enhance = async (input, decision) => {
     if (!decision || decision.kind !== "enter") return decision;
     if (config.enabled === false) return decision;
@@ -408,7 +445,9 @@ export function createWeKnoraInjector(options = {}) {
     // A card promoted by the nightly task has to appear without a restart: the profile
     // file is re-read only when its mtime moved, so this is one stat() on the common path.
     refreshProfileCards(config?.profile);
-    const pinned = pinnedFor();
+    // The prompt is read before the pinned block: which block this step gets depends on it.
+    const prompt = promptFromMessages(decision.messages);
+    const pinned = pinnedForStep(sessionId, prompt);
     /**
      * Close the step: compose pinned first, retrieval second, and log exactly once.
      * record carries this path's outcome and accounting; when the composed body is empty
@@ -470,7 +509,6 @@ export function createWeKnoraInjector(options = {}) {
       return { ...decision, messages: [...messages, injectionMessage(body)] };
     };
 
-    const prompt = promptFromMessages(decision.messages);
     if (!prompt) {
       return finish({ outcome: "skip", reason: "no_user_text", session: sessionId }, "", []);
     }
@@ -557,6 +595,7 @@ export function createWeKnoraInjector(options = {}) {
     forget: (sessionId) => {
       injected.delete(sessionId);
       lastPrompt.delete(sessionId);
+      pinnedPrompt.delete(sessionId);
     },
   };
 }

package/lib/weknora.js

@@ -59,8 +59,16 @@ export async function request(config, method, path, options = {}) {
   const base = baseUrlOf(config);
   if (!base) return { ok: false, reason: "no_api_url" };
 
+  // 自带计时器而不是 AbortSignal.timeout()：后者的定时器是 unref 的，事件循环里没有别的
+  // 句柄时它会随循环一起退出，pending 的 promise 就再也不会结算（Node 20 的测试进程里正好如此）。
   const signals = [];
-  if (Number.isFinite(timeoutMs) && timeoutMs > 0) signals.push(AbortSignal.timeout(timeoutMs));
+  let deadline = null;
+  let deadlineTimer = null;
+  if (Number.isFinite(timeoutMs) && timeoutMs > 0) {
+    deadline = new AbortController();
+    deadlineTimer = setTimeout(() => deadline.abort(), timeoutMs);
+    signals.push(deadline.signal);
+  }
   if (signal) signals.push(signal);
 
   const init = { method, headers: headersFor(config, { json: body !== undefined }) };
@@ -75,6 +83,9 @@ export async function request(config, method, path, options = {}) {
     return { ok: true, status: response.status, body: await response.json() };
   } catch (error) {
     const name = error?.name;
-    return { ok: false, reason: name === "TimeoutError" || name === "AbortError" ? "timeout" : "transport" };
+    const reason = (Number.isFinite(timeoutMs) && timeoutMs > 0 && deadline?.signal.aborted) || name === "TimeoutError" || name === "AbortError" ? "timeout" : "transport";
+    return { ok: false, reason };
+  } finally {
+    if (deadlineTimer) clearTimeout(deadlineTimer);
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rennimars/dsh-weknora-memory",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Layered context engine for DeepSeek Harness: per-kb WeKnora retrieval fused by RRF, recency decay and hot/warm/cold tiering, standing profile cards and fact-conflict surfacing — budgeted, auditable, editable.",
   "type": "module",
   "main": "./lib/index.js",

package/writers/profile-writer.mjs

@@ -53,6 +53,9 @@ export const REVIEW_TEXT_MAX = 60;
  */
 export const DEGRADE_AFTER = 50;
 
+/** 同一话题的重复出现算一次'证言'；攒够这么多次、或人工批准，才允许 proposed -> active。 */
+export const ATTEST_MIN = 3;
+
 const clean = (value) => String(value ?? '').replace(/\s+/g, ' ').trim();
 const charLen = (value) => Array.from(clean(value)).length;
 const KINDS = new Set(['user', 'agent']);
@@ -228,6 +231,13 @@ export function review(profile, { today = dayOf(), stats = {}, degrade = true }
         decisions.push({ id: card.id, action: 'kept_proposed', reason: 'conflict:' + conflict });
         continue;
       }
+      // 一晚的静默不是决定：只有同一话题被反复看到（attest 攒够），或人显式批准，才转正。
+      const approved = card.approve === true;
+      const attest = Number(card.attest) || 0;
+      if (!approved && attest < ATTEST_MIN) {
+        decisions.push({ id: card.id, action: 'kept_proposed', reason: 'needs_approval_or_attest_' + attest + '_of_' + ATTEST_MIN });
+        continue;
+      }
       card.status = 'active';
       decisions.push({ id: card.id, action: 'activated', reason: 'source+unique+under_' + REVIEW_TEXT_MAX });
       continue;

package/writers/register-tasks.ps1

@@ -10,14 +10,14 @@
     .\register-tasks.ps1 -Prefix WK   # rename the task group (default DSH-Memory)
 
   Tasks are registered for the current user only: no admin rights, no system-wide
-  service. Times can be changed in -Times like @{ Ingest='hourly'; Distill='04:30'; Dream='03:30' }.
+  service. Times can be changed in -Times like @{ Ingest='hourly'; Distill='00:00'; Dream='00:30' }.
 #>
 [CmdletBinding()]
 param(
   [switch]$Unregister,
   [string]$Prefix = 'DSH-Memory',
   [string]$Config = '',
-  [hashtable]$Times = @{ Ingest = 'hourly'; Distill = '04:30'; Dream = '03:30' }
+  [hashtable]$Times = @{ Ingest = 'hourly'; Distill = '00:00'; Dream = '00:30' }
 )
 
 $ErrorActionPreference = 'Stop'