dsh-memory-palace 1.8.4-alpha.1 → 1.8.4-alpha.2

package/README.md

@@ -60,8 +60,8 @@
 
 | 工具 | 层级 | 作用 |
 |---|---|---|
-| `memory_note` | 项目级 | 把约定/偏好写入当前项目全部目标 `MEMORY.md`（去重） |
-| `memory_note_user` | 用户级 | 把跨项目偏好写入 `~/.deepseek-harness/MEMORY.md`（去重） |
+| `memory_note` | 项目级 | 章节化写入当前项目全部目标 `MEMORY.md`（同 `memory_write` 格式；`section` 可选，默认「项目约定」；同章节去重） |
+| `memory_note_user` | 用户级 | 章节化写入 `~/.deepseek-harness/MEMORY.md`（与项目级同构；`section` 可选，默认「用户偏好」；同章节去重） |
 | `memory_read` | 按需 | 读取记忆。`scope` 默认 `memory`（用户级 + 项目级）；可选 `project`（仅项目级，与 `memory_write` 的写侧 scope 同名）/ `today` / `yesterday` / `daily`（近三天日志）/ `all`（长期记忆 + 日志） |
 | `memory_delete` | 用户级/项目级/每日级 | 按内容删除记忆条目（两阶段确认：先预览匹配位置与内容，用户确认后再删；删除动作经 harness 原生确认弹窗硬闸门，真人点允许才真正执行） |
 | `memory_write` | 项目级/用户级 | 章节化写入：章节存在则末尾追加，不存在则新建章节（章节化格式规范） |
@@ -120,6 +120,9 @@ dsh plugin --profile web remove dsh-memory-palace
 ## 版本历史
 
 - **1.8.4**
+    - **1.8.4-alpha.2**
+        - 🐛用户自定义指令前缀多行指令粘连修复
+        - 💥移除插件用户记忆过时工具
     - **1.8.4-alpha.1**
         - 💥用户自定义指令落盘位置迁移
         - 💪memory_reorganize 指令支持中文

package/lib/common/sections.mjs

@@ -5,7 +5,10 @@
 // - 全部返回 { ok, reason, ... }，永不抛错——LLM 产出的 ops 不可信，机器侧兜底；
 // - 只拆两级标题（`## `），`#` 文件标题与前言归 head，`###` 及更深归当前章节内容；
 // - text.mjs 的 normLine 不动；budgetClip 已在按 D2 修复重写（兑现「保留全部
-// 结构行」声明），锚点同步更新——本文件自持归一化，不依赖其内部实现。
+// 结构行」声明），锚点同步更新——章节匹配与条目归一化自持，不依赖其内部实现。
+// - sectionHasEntry 是本文件唯一的外部依赖点：条目去重口径必须与 text.mjs 的 normLine
+// 完全一致（多工具共用），故显式 import 而非复制一份。
+import { normLine } from "./text.mjs";
 
 // 章节标题归一化：剥 `#` 前缀、合空白、转小写。标题匹配全部走它，"带不带 ## 前缀"都鲁棒。
 const normTitle = (s) =>
@@ -86,6 +89,19 @@ export function locateSection(md, section) {
 // 处理：按行拆开，逐行剥行首列表符（- / *）与井号（#），去空行后以「；」连接为单行——
 // 内容零丢失、结构损坏向量消除。返回 { text, lines }（lines = 非空行数），调用方可在
 // message 里明示「已合并单行」。memory_write 与子代理 log_write_ops 共用，一处修两条路径闭环。
+// 章节内是否已存在「同归一化」条目（精确去重，防手滑重复）。
+// memory_write 与 memory_note 系列共用本判据——两个写入面口径必须一致，
+// 否则同一个事实从不同工具写进去会各存一份。
+export function sectionHasEntry(md, section, entry) {
+  const loc = locateSection(md, section);
+  if (!loc) return false;
+  const want = normLine(entry);
+  for (let i = loc.start + 1; i < loc.end; i++) {
+    if (normLine(loc.lines[i]) === want) return true;
+  }
+  return false;
+}
+
 export function flattenEntry(entry) {
   const parts = String(entry ?? "")
     .split(/\r?\n/)

package/lib/hybrid/tools.mjs

@@ -9,8 +9,8 @@ import { defineTool } from "@deepseek-ai/dsh-tools";
 import { writeFile, rename } from "node:fs/promises";
 import { join, dirname } from "node:path";
 import { readFileSync } from "node:fs";
-import { expandHome, toHomeShort, readMdSync, nowStamp, normLine } from "../common/text.mjs";
-import { appendToSectionText, upsertSectionText, replaceSectionText, markEntryDeletedText, locateSection } from "../common/sections.mjs";
+import { expandHome, toHomeShort, readMdSync, nowStamp } from "../common/text.mjs";
+import { appendToSectionText, upsertSectionText, replaceSectionText, markEntryDeletedText, locateSection, sectionHasEntry } from "../common/sections.mjs";
 import { planModeOf } from "../common/planmode.mjs";
 
 const REORG_MARK = "memory-palace:last-reorg:";
@@ -54,16 +54,8 @@ export function checkReorgGate(memFile, cfg) {
   };
 }
 
-// 同 normLine 精确去重：目标章节内已存在同归一化条目 → skip（不算去重管线，仅防手滑重复）。
-function sectionHasEntry(md, section, entry) {
-  const loc = locateSection(md, section);
-  if (!loc) return false;
-  const want = normLine(entry);
-  for (let i = loc.start + 1; i < loc.end; i++) {
-    if (normLine(loc.lines[i]) === want) return true;
-  }
-  return false;
-}
+// 同 normLine 精确去重由 common/sections.mjs 的 sectionHasEntry 提供
+// （memory_write 与 memory_note 系列必须共用同一判据）。
 
 /**
  * 注册记忆工具（MEMORY.md 章节化写入 / 整章节替换 / 全量重整）+ pre-execute 闸门。

package/lib/index.js

@@ -602,11 +602,13 @@ export function apply(ctx, config) {
           "- 用户级 MEMORY.md（跨项目）：偏好与硬约束。可写——memory_note_user，或 memory_write / memory_update_section 的 scope='user'。\n" +
           "- 项目级 MEMORY.md（本项目）：约定、决策、踩坑。可写——memory_note，或 memory_write / memory_update_section 的 scope='project'。\n" +
           "- 每日日志 memory/YYYY-MM-DD.md：记忆子代理每轮自动维护。你只读——不写、不改文件头、不重排。\n" +
+          "- 章节：按语义自己选章节名（章节不存在会自动新建），不要都塞进默认章节；memory_note / memory_note_user 省略 section 时落「项目约定」/「用户偏好」，仅当语义确实合适才省略。\n" +
           "当前项目已存在 WorkBuddy/CodeBuddy 记忆目录，本插件直接读写这些目录（不再单独创建 .deepseek-harness/memory/）。"
         : "[记忆文件]\n" +
           "- 用户级 ~/.deepseek-harness/MEMORY.md（跨项目）：偏好与硬约束。可写——memory_note_user，或 memory_write / memory_update_section 的 scope='user'。\n" +
           "- 项目级 .deepseek-harness/MEMORY.md（项目根下）：约定、决策、踩坑。可写——memory_note，或 memory_write / memory_update_section 的 scope='project'。\n" +
-          "- 每日日志 .deepseek-harness/memory/YYYY-MM-DD.md（项目根下）：记忆子代理每轮自动维护。你只读——不写、不改文件头、不重排。";
+          "- 每日日志 .deepseek-harness/memory/YYYY-MM-DD.md（项目根下）：记忆子代理每轮自动维护。你只读——不写、不改文件头、不重排。\n" +
+          "- 章节：按语义自己选章节名（章节不存在会自动新建），不要都塞进默认章节；memory_note / memory_note_user 省略 section 时落「项目约定」/「用户偏好」，仅当语义确实合适才省略。";
 
       // 段二：读取 —— 与 [记忆文件] 同一套「标签：内容」行语法（每行都能竖着扫）。
       // ⚠ 重构只改排版，不动语义：工具名、条件、分支、举例必须逐字保留。
@@ -640,10 +642,11 @@ export function apply(ctx, config) {
       // 真源是 <profile>/.memory-palace/user-commands.md（用户手改或设置页编辑）；
       // 内容恒定（改文件才变）→ 放 system 零缓存成本，不违反本节的分流原则。
       // readSync 内部按 mtimeMs 缓存，文件未变时不读盘。
+      // 前缀独占一行：多行指令若与前缀同行，首行会被当成前缀的一部分、后续行看起来游离在块外。
       const custom = userCommands.readSync().trim();
       // 本 section 已无任何动态块（日志迁出 E 投影），直接返回常量指令。
       // 返回值仍非空串（即便自定义指令为空也含 intro），避免 harness 对空 section 的处理歧义。
-      return custom ? `${introFull}\n\n[用户自定义指令] ${custom}` : introFull;
+      return custom ? `${introFull}\n\n[用户自定义指令]\n${custom}` : introFull;
     },
   });
 

package/lib/tools.mjs

@@ -1,10 +1,14 @@
 // memory-palace 记忆工具：memory_note / memory_note_user / memory_read / memory_delete
 // + 删除硬确认闸门（tools/pre-execute → harness 原生确认弹窗）。
+// memory_note 系列与 memory_write 共用章节化写入（upsertSectionText + sectionHasEntry），
+// 不再追加裸行——两个层级的条目格式与去重口径必须一致。
 // 经 registerTools 注入 ctx/config/paths/运行时状态；在 index.mjs 装配时调用一次。
 import { defineTool } from "@deepseek-ai/dsh-tools";
-import { join } from "node:path";
+import { mkdir, writeFile } from "node:fs/promises";
+import { join, dirname } from "node:path";
 import { todayISO, isoDaysAgo, expandHome, toHomeShort, readMdSync, budgetClip, stripDeletedLines } from "./common/text.mjs";
-import { appendLineDedup, findMatches, removeLineByMatch, recentLogDates } from "./common/records.mjs";
+import { findMatches, removeLineByMatch, recentLogDates } from "./common/records.mjs";
+import { upsertSectionText, sectionHasEntry } from "./common/sections.mjs";
 import { planModeOf } from "./common/planmode.mjs";
 
 /**
@@ -77,6 +81,8 @@ export function registerTools({ ctx, getConfig, paths, state }) {
         "key decision, or root-cause fix — or when the user explicitly asks you to remember it. " +
         "Do NOT log step-by-step work or one-off process (that belongs in the daily log). " +
         "Format: ONE concise line, conclusion first, objective third-person. " +
+        "It is appended under `## <section>` (created when absent) — the same chaptered format as " +
+        "memory_write; pass `section` to choose the chapter, default 「项目约定」. " +
         "CONTENT RULES: no source coordinates (`file.go:123`), no one-off material (error dumps, command output, " +
         "throwaway URLs); paths = reusable entry points only; URLs = must-reuse base addresses only. " +
         "For cross-project user-level memory, use memory_note_user instead.",
@@ -86,6 +92,10 @@ export function registerTools({ ctx, getConfig, paths, state }) {
           required: true,
           description: "The convention / preference / fact to remember, in one concise line.",
         },
+        section: {
+          type: "string",
+          description: "目标章节标题（不含 `## ` 前缀）。省略时落在「项目约定」章节；语义不符请显式指定。",
+        },
       },
       output: {
         schema: {
@@ -103,17 +113,32 @@ export function registerTools({ ctx, getConfig, paths, state }) {
         const cwd = sessionCwd(exec);
         const dirs = paths.writeDirs(cwd);
         if (!dirs.length) return { ok: false, message: "No active workspace." };
+        const entry = String(args.content ?? "").trim();
+        if (!entry) return { ok: false, message: "content 不能为空。" };
+        // 章节化：与 memory_write 同一套格式（`## 章节` + `- 条目`），默认落「项目约定」。
+        const section = String(args.section ?? "").trim() || "项目约定";
         try {
-          const line = `- ${args.content}`;
           let wrote = 0;
           let skipped = 0;
           for (const dir of dirs) {
             const mem = paths.memoryFileOf(dir, cwd);
-            if (await appendLineDedup(mem, line)) wrote++;
-            else skipped++;
+            const md = readMdSync(mem);
+            if (sectionHasEntry(md, section, entry)) {
+              skipped++;
+              continue;
+            }
+            const r = upsertSectionText(md, section, entry);
+            if (!r.ok) {
+              skipped++;
+              continue;
+            }
+            // 目录可能尚不存在（首次写入 / 新建 dsh 目录）——章节化写入用 writeFile，须自行建目录。
+            await mkdir(dirname(mem), { recursive: true });
+            await writeFile(mem, r.text, "utf8");
+            wrote++;
           }
           const dup = skipped > 0 ? ` (${skipped} already had it)` : "";
-          return { ok: true, message: `Saved to ${wrote} workspace MEMORY.md file(s)${dup}.` };
+          return { ok: true, message: `Saved to ${wrote} workspace MEMORY.md file(s) under 「${section}」${dup}.` };
         } catch (e) {
           return { ok: false, message: `Failed: ${e}` };
         }
@@ -132,6 +157,8 @@ export function registerTools({ ctx, getConfig, paths, state }) {
         "a personal preference / constraint, or a reusable fact / decision — or when the user explicitly " +
         "asks you to remember something that is not tied to this project. Not for one-off process. " +
         "Format: ONE concise line, conclusion first. " +
+        "It is appended under `## <section>` (created when absent) — the same chaptered format as " +
+        "memory_write; pass `section` to choose the chapter, default 「用户偏好」. " +
         "CONTENT RULES: no source coordinates (`file.go:123`), no one-off material (error dumps, command output, " +
         "throwaway URLs); paths = reusable entry points only; URLs = must-reuse base addresses only. " +
         "For current-project conventions, use memory_note instead.",
@@ -141,6 +168,10 @@ export function registerTools({ ctx, getConfig, paths, state }) {
           required: true,
           description: "The user-level preference / fact to remember, in one concise line.",
         },
+        section: {
+          type: "string",
+          description: "目标章节标题（不含 `## ` 前缀）。省略时落在「用户偏好」章节；语义不符请显式指定。",
+        },
       },
       output: {
         schema: {
@@ -156,17 +187,23 @@ export function registerTools({ ctx, getConfig, paths, state }) {
       async execute(args) {
         const c = cfg();
         if (!c.enabled) return { ok: false, message: "memory-palace is currently disabled in settings." };
+        const entry = String(args.content ?? "").trim();
+        if (!entry) return { ok: false, message: "content 不能为空。" };
+        // 章节化：与项目级 / memory_write 同一套格式，默认落「用户偏好」。
+        const section = String(args.section ?? "").trim() || "用户偏好";
         try {
           const file = expandHome(c.userMemoryPath);
           const fileShort = toHomeShort(file);
-          const line = `- ${args.content}`;
-          const wrote = await appendLineDedup(file, line);
-          return {
-            ok: true,
-            message: wrote
-              ? `Saved to user-level memory (${fileShort}).`
-              : `Already present in user-level memory (${fileShort}); skipped.`,
-          };
+          // 用户级目录可能尚不存在（首次写入）——mkdir 由本路径负责。
+          await mkdir(dirname(file), { recursive: true });
+          const md = readMdSync(file);
+          if (sectionHasEntry(md, section, entry)) {
+            return { ok: true, message: `Already present in user-level memory (${fileShort}); skipped.` };
+          }
+          const r = upsertSectionText(md, section, entry);
+          if (!r.ok) return { ok: false, message: `Failed: ${r.reason}` };
+          await writeFile(file, r.text, "utf8");
+          return { ok: true, message: `Saved to user-level memory (${fileShort}) under 「${section}」.` };
         } catch (e) {
           return { ok: false, message: `Failed: ${e}` };
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-memory-palace",
-  "version": "1.8.4-alpha.1",
+  "version": "1.8.4-alpha.2",
   "description": "把 WorkBuddy 的文件式记忆系统移植进 DeepSeek Harness：用户级 + 工作区的人类可读 Markdown 记忆（~/.deepseek-harness/MEMORY.md 与 <cwd>/.deepseek-harness/memory/）。",
   "type": "module",
   "main": "lib/index.js",