四个各自独立的生产缺陷,共同的根源都是「本该属于会话的属性没有存在会话上」。 ## 1. dsh 指定工作目录完全失效(所有会话落进「未分组」) 插件建会话时用的 cwd 是自己拼的 `~/.dsh/mail-sessions/mail-<uuid>` —— 每封邮件一个全新的空目录。DSH 与 opencode 都按 cwd 给会话分组,于是所有 邮件会话既不属于任何项目、彼此也不同组。 而 Gateway 从来没把地址里的 path 位发给插件:`notifyRecipients` 的 payload 只有 mail_id/session_id/from_name/subject,`to_workspace` 虽然入库了却不在 SSE 事件里,插件即使想用也拿不到。 - SSE `new_mail` 事件加 `to_workspace`。**每个收件方拿到自己那个地址的 path**, 不是主收件人的 —— 抄送给 opencode@/a 与主发给 dsh@/b 是两个工作区 - 两个插件的 cwd 都改为取寻址的 path 位;不存在的目录**不创建**而是回退到 兜底目录(一个笔误不该在磁盘上落下真目录,Agent 会在里面一无所获地干活) - 拒绝相对路径:cwd 的相对基准是 harness 进程的启动目录,systemd 下通常是 `/` ## 2. 会话别名列不出工作区下的历史会话(无法选择) workspace 只存在于 `mails.to_workspace` 上,「这个工作区下有哪些会话」必须 JOIN mails 再从收发双方的 workspace 里猜。而 Agent 回信时 from_workspace 填的是 **Agent 名**而不是路径,旧条件 `to_workspace = $p OR from_workspace = $p` 在只剩 Agent 回信可匹配时两边都对不上。 - `sessions.workspace` 新列,`CreateSession` 从地址的 path 位带入 - `SuggestSessionCandidates` 取代 `SuggestSessionsFor`:以会话自己的 workspace 为权威,历史会话(该列为空)回退到 mails 反推 —— 升级后老会话不该消失 - `FindOrCreateDefaultSession` 同步改用会话的 workspace ## 3. 平台侧会话在补全里根本不存在 人直接在 opencode/DSH 界面上开的会话,Gateway 一无所知。 新增 `agent_platform_sessions` 镜像表,插件在心跳里上报快照。 **上报而非 Gateway 反向拉取**:当前架构是单向的(Agent 持密钥主动连 Gateway, Gateway 从不外呼),反向拉取需要它保存各平台的地址与凭证,那是另一套信任模型。 - 与 sessions 表分开存:镜像里是别人家的会话,id 属于平台的 id 空间,没有 本侧的 owner/预算/邮件。混进 sessions 会让每一处「按会话鉴权」都要先判断 这条到底是不是真的本侧会话 - **整表替换而非增量合并**:平台侧删掉的会话必须从候选里消失 —— session 位是 三态语义,指向不存在的会话直接 404 - **nil 与空数组语义不同**:插件拉不到列表时省略该字段(保留镜像), 而不是传空数组把镜像抹掉 - **subagent 子会话不上报**:实测 DSH 的 list 里混着 49 条子会话,标题就是 派活的提示词前缀(九条都叫 "You are auditing ONE file"),slug 全撞名; 它们是父 agent 内部的工作单元,人往里发邮件毫无意义 - **slug 撞名只留最近那条**:服务端只能取其中一条,上报同名项只会让补全里 出现几个点哪个都不确定的候选 - DSH 插件此前**完全没有心跳** —— Gateway 靠 last_seen 判在线,一直靠注册撑着 补全候选带标题与来源:`suggestions` 保留纯字符串数组(不打破已部署的前端与 第三方客户端),新增同序的 `candidates`。过滤时标题也参与匹配 —— 人记得的是 「缓存选型」而不是 brisk-harbor 这种随机短名。 ## 4. 对话树看不见抄送与转发产生的分支 旧实现从锚点分「祖先链 + 子树」两路展开,而**兄弟节点既不是锚点的祖先也不是 它的子孙**:一封抄送给两个 Agent 的邮件收到两个回复,从其中一个看树永远看不到 另一个;挂在原件上的转发分支同理。 改为先 `ThreadRootOf` 上溯到线索根,再从根整树 BFS。只剩一个加载方向, 因此不再需要滚动位置补偿。前端补上抄送人列表与转发标记 —— 树上两个兄弟节点 为什么并列,唯一的解释就是父邮件抄送给了两个人。 ## 5. DSH 插件(Phase 7.7) 卡了一下午的 `Cannot read properties of undefined (reading 'kind')` 根因是 `followup()` 的参数形状:DSH 要完整的 UserMessage(content + source), 而我照抄了 opencode 的 parts 数组。错误抛在 agent-loop 内部,不指向调用点。 - `agent/status` → idle 时自动转发最后一条 assistant 消息(对应 opencode 的 session.idle),复用 relay-dedup 让位于模型的主动回信,走免配额通道 - `approval/request` 权限询问转邮件问人。与 opencode 的差异:那边的 permission.ask 是同步钩子只能立即返回 ask,DSH 这边是异步 waterfall, 可以真的等人 —— 拆插件时未决询问一律 fail closed,否则 await 永不返回 - 会话别名由模型标题派生(保留中文,去掉 `.` `@` `/` 等寻址分隔符 —— 留在别名里会让它自己被解析器切开) - 逻辑放 lib/ 下的纯函数并加测试:三类约定都是「错了不当场报错、只在深处 炸一个无关错误」 ## 其他 - `deploy/reset-demo.sh`:清空演示邮件数据,保留账号与密钥。备份用 `.backup` 而非 cp(WAL 下 cp 拿到的是缺尾巴的库);手工按依赖顺序删(SQLite 的 foreign_keys 默认关,声明了 REFERENCES 也不级联);只在目标是默认库时才碰 systemd(演练时误停过一次生产服务) - 插件 dist/ 不进版本库,install.sh 负责构建 - `permission_decision` 事件补 session_id:插件重启丢了待决映射时要靠它定位会话
180 lines
6.8 KiB
JavaScript
180 lines
6.8 KiB
JavaScript
/**
|
||
* dsh-mail-bridge 纯函数测试。
|
||
*
|
||
* 重点不是覆盖率,而是钉住几条「错了不当场报错」的约定:
|
||
* 1. userMessage() 的形状 —— 传错会在 agent-loop 深处抛一个不指向调用点的
|
||
* `Cannot read properties of undefined (reading 'kind')`,实测卡了一下午
|
||
* 2. modelTitle() 必须拒绝 fallback 占位标题 —— 否则会把插件自己的提示词
|
||
* 「你收到一封新邮件(AgentMail)」当成会话标题回写给 AgentMail
|
||
* 3. lastAssistantText() 只取 text 块 —— reasoning 是思考过程,不该进邮件
|
||
*
|
||
* node --test test/
|
||
*/
|
||
|
||
import { test } from 'node:test';
|
||
import assert from 'node:assert/strict';
|
||
import {
|
||
userMessage,
|
||
stripRe,
|
||
replySubject,
|
||
lastAssistantText,
|
||
modelTitle,
|
||
} from '../lib/message.js';
|
||
|
||
// ─── userMessage:DSH followup() 的唯一合法形状 ───
|
||
|
||
test('userMessage 产出 content + source 两个字段', () => {
|
||
const m = userMessage('你好');
|
||
assert.deepEqual(m, {
|
||
content: [{ type: 'text', text: '你好' }],
|
||
source: { kind: 'user' },
|
||
});
|
||
});
|
||
|
||
test('不变量:userMessage 必须带 source.kind —— agent-loop 的 preStep 直接读它', () => {
|
||
// 这一条是本文件存在的理由。少了 source.kind,DSH 抛的错落在 agent-loop 内部
|
||
// (`Cannot read properties of undefined (reading 'kind')`),既不指向调用点,
|
||
// 也不说是哪个字段,turn 会一 start 就 end、模型请求根本不发出去。
|
||
for (const text of ['x', '', '多行\n文本', '🙂']) {
|
||
const m = userMessage(text);
|
||
assert.equal(typeof m.source?.kind, 'string', 'source.kind 必须是字符串');
|
||
assert.equal(m.source.kind, 'user');
|
||
assert.ok(Array.isArray(m.content), 'content 必须是数组');
|
||
}
|
||
});
|
||
|
||
test('不变量:userMessage 返回的不是裸数组(opencode 的 parts 形状)', () => {
|
||
// opencode 的 promptAsync 收 parts 数组,DSH 收完整 UserMessage。
|
||
// 把 opencode 的写法照抄过来正是那次故障的起因。
|
||
const m = userMessage('x');
|
||
assert.ok(!Array.isArray(m), 'followup() 不接受裸数组');
|
||
});
|
||
|
||
test('userMessage 把非字符串转成字符串', () => {
|
||
assert.equal(userMessage(42).content[0].text, '42');
|
||
});
|
||
|
||
// ─── stripRe / replySubject ───
|
||
|
||
test('stripRe 去掉单个与叠加的 Re: 前缀', () => {
|
||
assert.equal(stripRe('Re: 主题'), '主题');
|
||
assert.equal(stripRe('Re: Re: Re: 主题'), '主题');
|
||
assert.equal(stripRe('RE: 主题'), '主题');
|
||
assert.equal(stripRe('主题'), '主题');
|
||
});
|
||
|
||
test('stripRe 不动正文里的 Re:', () => {
|
||
assert.equal(stripRe('关于 Re: 这个写法'), '关于 Re: 这个写法');
|
||
});
|
||
|
||
test('replySubject 只加一层 Re:', () => {
|
||
assert.equal(replySubject('缓存选型'), 'Re: 缓存选型');
|
||
assert.equal(replySubject('Re: 缓存选型'), 'Re: 缓存选型');
|
||
assert.equal(replySubject('Re: Re: 缓存选型'), 'Re: 缓存选型');
|
||
});
|
||
|
||
test('replySubject 空主题走兜底而不是产出裸 "Re: "', () => {
|
||
assert.equal(replySubject(''), 'DSH 回复');
|
||
assert.equal(replySubject(' '), 'DSH 回复');
|
||
assert.equal(replySubject(undefined), 'DSH 回复');
|
||
assert.equal(replySubject('', '自定义'), '自定义');
|
||
});
|
||
|
||
// ─── lastAssistantText ───
|
||
|
||
const assistantMsg = (blocks) => ({
|
||
type: 'assistant/message',
|
||
data: { message: { role: 'assistant', content: blocks } },
|
||
});
|
||
|
||
test('lastAssistantText 取最后一条 assistant 消息', () => {
|
||
const events = [
|
||
assistantMsg([{ type: 'text', text: '第一轮' }]),
|
||
{ type: 'tool/call', data: {} },
|
||
assistantMsg([{ type: 'text', text: '第二轮' }]),
|
||
];
|
||
assert.equal(lastAssistantText(events), '第二轮');
|
||
});
|
||
|
||
test('lastAssistantText 丢掉 reasoning 块', () => {
|
||
const events = [assistantMsg([
|
||
{ type: 'reasoning', text: '让我想想……用户要的是' },
|
||
{ type: 'text', text: '结论:可以。' },
|
||
])];
|
||
assert.equal(lastAssistantText(events), '结论:可以。');
|
||
});
|
||
|
||
test('lastAssistantText 拼接多个 text 块', () => {
|
||
const events = [assistantMsg([
|
||
{ type: 'text', text: '第一段' },
|
||
{ type: 'text', text: '第二段' },
|
||
])];
|
||
assert.equal(lastAssistantText(events), '第一段\n第二段');
|
||
});
|
||
|
||
test('lastAssistantText 只有 tool-call 时返回空串(没有可回信的内容)', () => {
|
||
const events = [assistantMsg([
|
||
{ type: 'reasoning', text: '先读收件箱' },
|
||
{ type: 'tool-call', id: 'c1', name: 'read_inbox' },
|
||
])];
|
||
assert.equal(lastAssistantText(events), '');
|
||
});
|
||
|
||
test('lastAssistantText 容错:空日志、非数组、结构缺失', () => {
|
||
assert.equal(lastAssistantText([]), '');
|
||
assert.equal(lastAssistantText(undefined), '');
|
||
assert.equal(lastAssistantText(null), '');
|
||
assert.equal(lastAssistantText([{ type: 'assistant/message' }]), '');
|
||
assert.equal(lastAssistantText([{ type: 'assistant/message', data: {} }]), '');
|
||
});
|
||
|
||
test('lastAssistantText 跳过非 assistant/message 事件', () => {
|
||
const events = [
|
||
assistantMsg([{ type: 'text', text: '正文' }]),
|
||
{ type: 'step/end', data: {} },
|
||
{ type: 'turn/end', data: {} },
|
||
];
|
||
assert.equal(lastAssistantText(events), '正文');
|
||
});
|
||
|
||
// ─── modelTitle ───
|
||
|
||
const titleEvent = (title, kind) => ({
|
||
type: 'session/title',
|
||
data: { title, source: { kind } },
|
||
});
|
||
|
||
test('modelTitle 取模型生成的标题', () => {
|
||
const events = [titleEvent('处理新邮件任务并回复', 'provider')];
|
||
assert.equal(modelTitle(events), '处理新邮件任务并回复');
|
||
});
|
||
|
||
test('不变量:modelTitle 拒绝 fallback 占位标题', () => {
|
||
// DSH 在模型生成真标题之前会先落一个 fallback 标题,内容是用户第一句话的截断。
|
||
// 而「用户第一句话」是插件自己拼的提示词,回写过去等于把
|
||
// 「你收到一封新邮件(AgentMail)」当成会话标题。
|
||
const events = [titleEvent('你收到一封新邮件(AgentMail)', 'fallback')];
|
||
assert.equal(modelTitle(events), '');
|
||
});
|
||
|
||
test('modelTitle 取最后一次 session/title —— fallback 之后的 provider 标题算', () => {
|
||
const events = [
|
||
titleEvent('你收到一封新邮件(AgentMail)', 'fallback'),
|
||
{ type: 'assistant/chunk', data: {} },
|
||
titleEvent('缓存层选型评估邮件回复', 'provider'),
|
||
];
|
||
assert.equal(modelTitle(events), '缓存层选型评估邮件回复');
|
||
});
|
||
|
||
test('modelTitle 容错:无标题事件、结构缺失、非字符串', () => {
|
||
assert.equal(modelTitle([]), '');
|
||
assert.equal(modelTitle(undefined), '');
|
||
assert.equal(modelTitle([{ type: 'session/title' }]), '');
|
||
assert.equal(modelTitle([{ type: 'session/title', data: {} }]), '');
|
||
assert.equal(modelTitle([{ type: 'session/title', data: { title: 42 } }]), '');
|
||
});
|
||
|
||
test('modelTitle 修掉标题两端空白', () => {
|
||
assert.equal(modelTitle([titleEvent(' 带空白的标题 ', 'provider')]), '带空白的标题');
|
||
});
|