dsh-tdai-memory 0.2.7 → 0.2.8

package/README.md

@@ -79,6 +79,35 @@ tdai-memory:
       sendDimensions: false
 ```
 
+## Install
+
+```bash
+dsh plugin --profile web add dsh-tdai-memory
+```
+
+then mount it in `$DSH_HOME/profiles/web/cordis.patch.yml`:
+
+```yaml
+- insert:
+    - id: tdai-memory
+      name: 'dsh-tdai-memory'
+      config: {}          # keys can live in settings.yaml instead
+```
+
+and restart `dsh web`. LLM/embedding API keys can be set in the Web UI
+settings page (记忆 / Memory) or directly in `settings.yaml` under
+`tdai-memory:`.
+
+> **Note for users**
+> - `dsh plugin` prints "declares no dsh.bundle — installed as a plain
+>   dependency" — **expected**: this plugin mounts via `cordis.patch.yml`.
+> - The settings section needs the `dsh-host-apiproxy` namespace allowlist;
+>   the plugin patches it automatically on first start — **restart `dsh web`
+>   once more** and the section appears. A dsh update overwrites the patch;
+>   the next plugin start re-applies it.
+> - Settings changes apply **after a restart** (TdaiCore is built at startup).
+> - Tested against DSH `0.1.0-rc.6`.
+
 ## Known trade-offs
 
 - **Extraction model**: `mimo-v2.5` extracts correctly but takes 20-30s per

package/client.js

@@ -32,7 +32,7 @@ window.__ModuleLoader__.load({
       ".__tm_btn{border:1px solid var(--dsw-alias-border-l2);background:var(--dsw-alias-bg-layer-3);color:var(--dsw-alias-label-primary);border-radius:8px;padding:6px 14px;font:inherit;font-size:13px;cursor:pointer}" +
       ".__tm_btn:hover:not(:disabled){border-color:var(--dsw-alias-state-business-primary)}" +
       ".__tm_btn:disabled{opacity:.5;cursor:default}" +
-      ".__tm_btnPrimary{border-color:var(--dsw-alias-state-business-primary);background:var(--dsw-alias-state-business-primary);color:var(--dsw-alias-label-on-accent)}" +
+      ".__tm_btnPrimary{border-color:var(--dsw-alias-state-business-primary, #3964fe);background:var(--dsw-alias-state-business-primary, #3964fe);color:#fff}" +
       ".__tm_status{font-size:12px;color:var(--dsw-alias-label-tertiary)}" +
       ".__tm_error{font-size:12px;color:var(--dsw-alias-state-error-primary)}" +
       ".__tm_unavailable{font-size:13px;color:var(--dsw-alias-label-tertiary)}";

package/index.js

@@ -74,6 +74,20 @@ const Config = z.object({
   toolsEnabled: z.boolean().default(true),
 });
 
+/**
+ * Whether a derived message is real human input.
+ *
+ * A `"user"` role alone is not enough: DSH injects synthetic user-role
+ * context (system reminders, runtime context, skill catalogs, goal rounds,
+ * tool results, …) that must never be treated as user input. Real input is
+ * distinguished by `message.source.kind === "user"` (see dsh-llm Message
+ * types). Messages without a `source` (older harnesses) are treated as real
+ * input to stay compatible.
+ */
+function isRealUserMessage(m) {
+  return m?.role === "user" && (m.source == null || m.source.kind === "user");
+}
+
 /** Extract plain text from a dsh message content (block array or string). */
 function extractText(content) {
   if (typeof content === "string") return content.trim();
@@ -227,11 +241,19 @@ function apply(ctx, config) {
   async function captureTurn(session, startedAt) {
     if (!tdaiConfig?.capture.enabled) return;
     const derived = session.deriveMessages();
-    const lastUserIdx = derived.map((m) => m.role).lastIndexOf("user");
+    // Start from the last REAL user message. DSH injects synthetic user-role
+    // context (system reminders, runtime context, skill catalogs, goal
+    // rounds, …) whose `source.kind` is not "user"; those must not pollute
+    // L0 memory (issue #1).
+    const lastUserIdx = derived.findLastIndex(isRealUserMessage);
     if (lastUserIdx < 0) return;
     const messages = derived
       .slice(lastUserIdx)
       .map((m) => {
+        // Keep assistant replies and real user input only; drop synthetic
+        // user-role messages (plugin context, tool results, …) injected
+        // after the last real user message.
+        if (m.role !== "assistant" && !isRealUserMessage(m)) return null;
         const text = extractText(m.content);
         if (!text) return null;
         return {
@@ -268,7 +290,7 @@ function apply(ctx, config) {
         agent.ctx.on("system-prompt/assemble", async (assembly, context) => {
           try {
             if (!tdaiConfig?.recall.enabled) return assembly;
-            const lastUser = [...session.deriveMessages()].reverse().find((m) => m.role === "user");
+            const lastUser = [...session.deriveMessages()].reverse().find(isRealUserMessage);
             if (!lastUser) return assembly;
             const text = extractText(lastUser.content);
             if (!text) return assembly;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name":  "dsh-tdai-memory",
-    "version":  "0.2.7",
+    "version":  "0.2.8",
     "description":  "DeepSeek Harness 的 TencentDB Agent Memory 移植：L0 对话捕获 → L1 结构化记忆提取 → L2 场景/L3 画像，自动召回注入 + 记忆/对话搜索工具；复用现有 ~/.memory-tencentdb/memory-tdai 数据；附 Web UI 设置栏。",
     "type":  "module",
     "main":  "index.js",

package/recall-inject.js

@@ -38,7 +38,13 @@ function apply(ctx, config) {
       if (!agent) return assembly;
       const session = agent.session;
       if (!session) return assembly;
-      const lastUser = [...session.deriveMessages()].reverse().find((m) => m.role === "user");
+      // Only real human input triggers recall. DSH injects synthetic
+      // user-role context (system reminders, runtime context, skill
+      // catalogs, …) whose source.kind is not "user"; recalling on those
+      // would pollute the prompt with irrelevant memories (issue #1).
+      const lastUser = [...session.deriveMessages()]
+        .reverse()
+        .find((m) => m.role === "user" && (m.source == null || m.source.kind === "user"));
       if (!lastUser) return assembly;
       const text = extractText(lastUser.content);
       if (!text || text.length < config.minUserTextChars) return assembly;