dsh-session-conductor 1.0.9 → 1.0.10

package/lib/repair.js

@@ -67,12 +67,40 @@ export function validateSessionText(text) {
         problems.push(`seq ${ev.seq} tool/result 缺 message.id`);
         continue;
       }
-      if (message.role !== "user") problems.push(`seq ${ev.seq} tool/result role 应为 user`);
       const source = message.source;
       if (typeof source !== "object" || source === null || source.kind !== "tool" || typeof source.callId !== "string" || source.callId === "") {
         problems.push(`seq ${ev.seq} tool/result source 非法`);
         continue;
       }
+      // —— tool/result 的两种合法形态（按会话格式版本区分，此处不做版本假设，直接按 role 分派）——
+      //   v3（DSH ≤0.1.x / session.v3.jsonl.zstd）：message.role = "user"，
+      //        结果包在 content[0] 的 {type:"tool-result"} wrapper 里，toolCallId 在 wrapper 上。
+      //   v4（DSH 0.2.1 起 / session.v4.jsonl.zstd）：message.role = "tool"（first-class tool-role），
+      //        toolCallId 提升到 message 层，content 直接是结果块数组（**不再有** wrapper）。
+      //        官方口径见 packages/session/session-format-v3-to-v4 的 assertV4ToolResultMessage。
+      // 【bug 修复】此前只认 v3 的 role="user"，DSH 升到 0.2.1 后 v4 日志一律被判
+      //   「role 应为 user」而校验失败 —— 表现为 set_member_model 等「写入前先校验」
+      //   的功能对 v4 会话必然失败（而日志本身完全正常）。
+      if (message.role === "tool") {
+        // v4 形态
+        if (message.toolCallId !== source.callId) problems.push(`seq ${ev.seq} v4 toolCallId 与 source.callId 不匹配`);
+        if (!Array.isArray(message.content)) {
+          problems.push(`seq ${ev.seq} tool/result content 非数组`);
+          continue;
+        }
+        if (message.content.some((b) => typeof b === "object" && b !== null && b.type === "tool-result")) {
+          problems.push(`seq ${ev.seq} v4 content 不应再含 tool-result wrapper（v4 已把它提升为原生字段）`);
+        }
+        if (message.isError !== undefined && typeof message.isError !== "boolean") {
+          problems.push(`seq ${ev.seq} v4 isError 必须是 boolean`);
+        }
+        if (ev.data?.error !== undefined && message.isError !== true) {
+          problems.push(`seq ${ev.seq} v4 带 error 元数据但 isError 不为 true`);
+        }
+        continue;
+      }
+      // v3 形态（原逻辑保留）
+      if (message.role !== "user") problems.push(`seq ${ev.seq} tool/result role 应为 user（v3）或 tool（v4），实际 ${JSON.stringify(message.role)}`);
       if (!Array.isArray(message.content)) {
         problems.push(`seq ${ev.seq} tool/result content 非数组`);
         continue;

package/lib/session-codec.js

@@ -1,14 +1,20 @@
-// dsh-session-conductor — 会话日志记录编解码（自研，适配 DSH v3 会话格式）
+// dsh-session-conductor — 会话日志记录编解码（自研）
 //
 // DSH 0.1.6-alpha.1（SESSION_FORMAT_VERSION = 3）已移除旧版公开 API：
 //   decodeStorageRecord / packChunkRuns（v0/v1 旧格式的 *-chunks 打包编解码器）。
 // 本模块提供功能等价实现，供 repair.js / seq-gap-repair.js / tools/validate-session.mjs 使用：
 //   · decodeStorageRecord(record) → 事件数组
-//       v3 格式：一行一个事件 JSON（记录即事件对象，含 type 字段）→ 返回 [record]；
+//       v3/v4 格式：一行一个事件 JSON（记录即事件对象，含 type 字段）→ 返回 [record]；
 //       旧格式容器（{events:[...]}）→ 返回 record.events；
 //       无 type 且无 events 的未知记录 → 抛错（调用方 try/catch 判定解析失败）。
 //   · packChunkRuns(events) → 记录行数组
-//       v3 格式：一行一事件，直接返回事件数组（写回时每个事件 JSON 一行）。
+//       v3/v4 格式：一行一事件，直接返回事件数组（写回时每个事件 JSON 一行）。
+//
+// 【v3 → v4 说明（DSH 0.2.1 起）】记录/帧结构未变（仍是一行一事件），
+//   变的是**事件内部的 message 结构**：tool/result 的 message.role 由 "user" 改为 "tool"
+//   （first-class tool-role），toolCallId 提升到 message 层，content 不再有 tool-result wrapper。
+//   因此本模块无需改动；按 message 结构做校验的地方（见 repair.js 的 validateSessionText）
+//   必须按 role 分派。官方口径：packages/session/session-format-v3-to-v4/src/tool-role.ts。
 
 /**
  * 把一条存储记录解码为事件数组（v3 会话格式：记录即单个事件对象）。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-session-conductor",
-  "version": "1.0.9",
+  "version": "1.0.10",
   "description": "DSH session management enhancement: archive/restore sessions, batch & conditional delete, AI auto-rename, interrupted-session auto-resume, release idle sessions, workspace grouping, undo last message, repair damaged session logs, value analysis, full-text search, per-member model switching (list_models / set_member_model), and plan/closing template injection (system-prompt + context).",
   "type": "module",
   "main": "lib/index.js",