|
|
72fe80688e
|
docs(debt): 新债 —— 判据的动作有上界:字面执行 ≠ 意图覆盖(⑩→⑩′→⑩‴ 收紧链的实例)
复核 pi 的 5927110a(09-25 06:00:27,已由 5a6b8879 答复,58 封在后)。
他那封最有价值的一条是: "这不是我忘了执行⑩,而是 **⑩ 按字面执行也拦不住它**"
——repo/thread.go 里 grep descendantDepthCap = 1(存在、名字对),
而从 handler 包引用它编译失败(undefined: repo.descendantDepthCap)。
⇒ 判据的动作(grep)没坏,但它**测的范围小于被许诺的范围**,
所以它会**稳定地、每次都**放过恰好落在上界之外的那类错(不是偶发)。
与已有的 criterion-silently-returns-zero 是**姊妹条不是同一条**,故另立:
那条: 读数器**可能坏了** —— 命中 0 条时须先证「路径/字段/时间窗」都对
本条: 读数器**没坏**,但它**测不到那里** ⇒ 稳定放过上界外的错
收紧链(每格由真实反例逼出):
⑩ "引代码时实测该标识符存在"(grep -c ≥ 1)
⇒ 反例: 跨包未导出常量,grep=1 但编译不过
⑩′ "除存在外还须导出(首字母大写)" —— 我加的,**必要不充分**:
最小工程实测 InTest / Tagged 两种**首字母大写却够不着**(rc=1)
⑩‴ "存在 / 导出 / 在构建中"三分(docs/API.md:4682 (C) 节,目前是散文)
机制: 静态能验的那两格(存在、大写)**恰好是失败较少发生的那两格**;
而唯一会咬人的第三格(在构建中)**只有编译器知道**
⇒ 这就是"标签宽于断言范围"的新形态: 这次宽的不是断言,是**动作**。
现状(先量过再说): go vet ./... 全仓 **rc=0** ⇒ 当前**没有活实例**。
descendantDepthCap 那处不是代码踩了,是**建议稿里的一行会踩**(pi 信里那行,尚未落地)
⇒ 本条没有红的判据可挂,登记的是**判据设计上的已知边界**。
可判动作: 写"要验 X"时若所选动作天然有上界,把**上界写进文案**(⑬);
引跨包标识符时三格里**只有第三格必须编译** ⇒ 可执行形态是"编译一次"
(最小工程探针 / go vet),不是 grep。⚠️ 别把 ⑩ 当成已覆盖跨包引用。
边界: 没改任何代码; ⑩‴ 只落 API.md 散文、未落可执行判据;
样本量 = 1 个真实反例 + 2 个构造反例 —— 登记它是因为**机制清楚**,不是因为样本多。
验证: go test ./internal/repo/ -run Debt 全绿; debt-visibility.test.mjs 1/1。
|
2026-10-01 18:58:11 +08:00 |
|
|
|
c860fd7944
|
docs(debt): 新债 —— 递归上界当止损用却从不被读:ThreadRootOf 的 lvl 触到 cap 仍原样上报
复核 pi 的 df1788ec(09-25 05:57:34,已由 ccd4e4f6 答复,59 封在后)。
他指出的洞我 2 分 28 秒后给了修法(路2:在 repo 包内就地判定),
本轮复核确认**四天过去代码未动**,且账本里此前没有这一条。
形状(信号在手,没人读):
repo/thread.go:47 const descendantDepthCap = 10000 ← 兜底上界
repo/thread.go:77 ... WHERE up.lvl < $2 ... Scan(&rootID, &lvl)
repo/thread.go:81 return rootID, lvl, nil ← ★ 取到 lvl 后从不判定
handler/thread.go:99/113/118/123/174 五处消费 anchorDepth(接收/>0/累加/下钻/上报)
⇒ 全仓 anchorDepth 与 descendantDepthCap 的比较 = 0 处
准确说法(pi 措辞最准): 不是"没产生信号",是"信号在手、没人读" ——
CTE 对、返回对、调用方接得对,每段单独看都对,
错只在段与段之间那个"应该发生却没发生的比较"里。
现状影响 = 零(先量过再说):
现库最大回复链深度 = 80(递归 SQL 实测),cap=10000 ⇒ 差 9920 倍
⇒ 正常邮件永远触不到 cap;但数据损坏/成环时 CTE 兜底停止后,
lvl 仍被返回、照常累加、照常上报(:174)⇒ 客户端拿到 anchor_depth≈10000
却无从知道根不可信
缺口也无判据: thread_test.go:95-100 只验正常路径(anchorDepth != 1 才失败),
没有任何测试覆盖"深度触到 cap 时会怎样"。
未做: 本轮只读(递归 SQL + grep + 读源码),没改任何代码。
"截断后 lvl 照常上报"是**读码得出的**,不是**跑出来的**——
触发条件(成环/损坏)我没有构造。
深度 80 是本机此刻的库,是瞬时量不是断言。
若日后要修,先确认那个 10000 是"兜底停止"还是"业务上限"——两种含义对应不同修法。
顺带记一条同源判据 ⑩′: "在不在那个文件里"与"从调用点能不能拿到"是两件事;
grep -c 只能验前者,验不了后者。
验证: go test ./internal/repo/ -run Debt 全绿; debt-visibility.test.mjs 1/1。
|
2026-10-01 18:56:24 +08:00 |
|
|
|
477479a370
|
fix(harmony): ★★ 三页 AppHeader 顶栏避让硬编码 0 ⇒ 顶栏压进系统状态栏(真机实测)
设备:HUAWEI MatePad Pro(MRDI-W00),HarmonyOS NEXT,API 26,
`hdc tconn 192.168.2.87:43679`(此前一直无真机,本条挂了 5 天)。
## 症状(用户报:「左上角全屏状态下不应该显示全屏,会与顶部系统顶栏冲突」)
截图像看是状态栏压住顶栏。**实测证伪了这个读法**:像素扫描 + `uitest dumpLayout`
显示顶栏文字在 y=134..166、系统状态栏止于 y=82,**两者不重叠**。
真正被切掉的是**列表第一封邮件的标题**(y=294..319,只剩一条细线)。
⇒ 两处独立问题,第二个(窗格头 y=222..320 与列表首行 y=294..319 重叠)
**不是顶栏避让**造成的,本次未修,见下。
## 已修:AppHeader 那一半的避让确实是坏的
`EntryAbility` 早就把避让读到了(真机日志 `insets: statusBar=38.588235
navIndicator=27.764706 windowDecor=37`),`CommPage`/`MainPage` 内容层也用了
(`top: max(statusBar, windowDecor) + paneGap`)。**但 `AppHeader` 是另一个消费者,
三处都写死了 `topInsetPx: 0`**:
· `SentTab`(发件箱) MainPage.ets:1658
· `SettingsPage`(我的) SettingsPage.ets:687
· `PermissionTab`(授权) PermissionTab.ets:487
只有 `AdminUsersPage` 是对的(`topInset(this.windowInsets)`)—— 抄它。
## 为什么已有的判据没抓到(这才是关键)
`harmony-window.test.mjs` 接线⑤「每一个 @Entry 页都要消费避让」是**绿的**,
因为它的 `consumes()` 认两种形状,其中一种只要**文件里出现过**
`this.windowInsets.statusBar` 就算过 —— 而 `MainPage` 的**内容层**正好读了它。
⇒ 页面上有**两个**避让消费者,判据只问"页里有没有出现过那个字段",
于是 `AppHeader` 里那个 0 被完全放过。
新判据(判据 10)改成**逐个消费者问**:任何传给 `AppHeader` 的 `topInsetPx`
不得是字面量 0。自检要求扫到 ≥4 处,防"遍历写错 ⇒ 永远绿"。
★ 这条判据自己先犯过一次同类错并当场被抓:第一版用 `prose()`(含注释),
把**我自己写进注释里的**「原先 topInsetPx: 0」抓成了红 ——
判据在判自己的注释。改用本文件已有的 `stripped()`(其注释原话:
「判据的锚不能落在被守对象的自述上」)。红绿已验:把 SettingsPage 退回
缺陷版 ⇒ 红;恢复 ⇒ 绿。
## 编译期抓到的一个坑
`MainPage.ets:1658` 的 AppHeader 在 **`SentTab`** 里,而 `windowInsets` 原本
只声明在 `CommPage` 上。ArkTS 报 `Property 'windowInsets' does not exist on
type 'SentTab'`。⇒ `@StorageLink` 是**每个组件各自**订阅 `AppStorage` 的,
父组件的不会自动传给子组件;直接各自订阅同一把键(比"父传子"少一层)。
## 真机验证
装机后重跑 dumpLayout:发件箱顶栏 `y=134..166`(状态栏底 y=82,间隙 52px),
截图确认「发件箱」完整显示、不再被压。
## 未修(诚实登记)
列表第一行标题被窗格头盖住(`List` 首项 y=294..319 落在窗格头 y=222..320 内),
与顶栏避让**无关**,本次未动。根因待查:`MailRow` 是 `.height(64)` 定高,
而 List 容器从 y=320 起算,首项被画到容器上方。
|
2026-10-01 18:49:06 +08:00 |
|
|
|
e76b78fa1d
|
docs(debt): 补量化 —— 我信件里的旧 sha 引证失效 99%(101 个全量:85 被 filter-repo 重写、15 连 commit-map 都不在)
复核 pi 的 d4d90f51(09-25 06:36:02,已由 53fca768 答复,44 封在后)。
pi 的请求"§二 modified=true 换成工作树清单 收不收"已在 3 分 57 秒内收并落地
(2393703 "feat(deploy): 构建步打印工作树清单"),现在 HEAD 祖先 ✓。
复核过程中自检发现:我在 53fca768 引了 hash "b2584f1" 当落地证据,
而 git cat-file -t b2584f1 = "Not a valid object name"。真身是 2393703。
由此扫了本会话 21c398ee 里 dsh 发出的全部信件,凡上下文标注为 git 对象
(commit/提交/HEAD/hash/反引号包裹)的 7+hex 串,逐串 git cat-file -t 验证:
引用形状串总数 = 101
现在可解析 = 1
在 commit-map 里 = 85(filter-repo 重写过 ⇒ 旧 sha 永久失效)
完全不在 map 里 = 15(amend 前悬空提交已 gc / 或错拼)
⇒ 失效 = 100/101 = 99%
两个来源机制不同: filter-repo 重写(可预见,跑过 commit-map 就知道)、
amend+gc(不可预见,只有逐串验证能抓)。判据若写"引用前先 cat-file -t"
只能拦住当前; 对历史信件里的旧引证,唯一能救的是当时同时给新 sha 或
commit-map 行 —— 当时没做,现在补不进历史了。
这是对已有条目 history-rewrite-undisclosed-citations-dangle 的增量:
原条目只抽查了 8 个样本; 这次全量 101 个,且分离"被重写"vs"连 map 都不在"。
验证: go test ./internal/repo/ -run Debt 全绿; debt-visibility.test.mjs 1/1。
|
2026-10-01 04:20:23 +08:00 |
|
|
|
c8fcc6ad85
|
docs(debt): 新债 —— 三个部署脚本的 uid 预检在 Landlock 域内"说谎"(id -u=0 通过、touch rc=1)
复核 pi 的 49b1f6b2(09-25,已由 f3b352b4 答复)那条 ⑭′——"能力判断只能用
真实动作,不能用检查器"。当时我以为它是纯方法论、仓库没落点;今天实测发现
仓库有三个同构的落点。
形状(本机实测,2026-09-30):
deploy/redeploy-gateway.sh:55 [ "$(id -u)" = "0" ]
deploy/install.sh:173 [[ $EUID -eq 0 ]]
deploy/reset-demo.sh:27 [[ $EUID -eq 0 ]]
我(dsh 的 bash 工具)实测: id -u=0 ⇒ 三处预检全过;但
touch /opt/agentmail/.probe ⇒ rc=1 Permission denied;NoNewPrivs=1;
LSM 含 landlock;mountinfo 里 /opt 没单独 ro ⇒ 只能是 LSM 门控。
⇒ 检查器说"能写",真实动作说"不能写"——正是 ⑭′ 判据。
一句话: "谁能写"不是 uid 的属性,是"哪个进程"的属性。
后果:
沙箱域内 uid=0 进程跑这三个脚本,会通过预检、在最后一步写 $PREFIX
(/opt/agentmail)时以 Permission denied 失败——错在离检查最远那一步。
三个检查器互相一致,修一处不等于修三处。
redeploy:57 那行药方"sudo bash ..."在域内是死路——sudo 子进程 NNP 仍=1
(pi §二③④ 实测: sudo -n touch rc=1)⇒ 会误导执行者重试。
对照(避免过度推广):
agentmail 自己的 mail_id/session_id 侧没有这类检查器
(grep unix.Access|access(W_OK) = 0)。
检查器在无沙箱宿主进程(NNP=0)上不撒谎 ⇒ 这是"检查器的适用域没声明",
不是"检查器永远错"。
可判动作:
到期动作不是改检查器(改了还是检查器),是把预检从"问 uid"换成"真实动作":
if ! touch "$PREFIX/.write-probe" 2>/dev/null; then fail "...连真实写入都失败
(多半在沙箱域内,uid 无关)"; fi; rm -f "$PREFIX/.write-probe"
或降级为披露: 保留 uid 预检,另加一行"本机检测: 执行层 NNP=$(...)"。
边界: 只实测本机 dsh 的 bash 工具层(NNP=1);pi/其他 agent 侧未测。
未改任何脚本(本条是登记)。未能回给 pi(send_mail 被会话级闸挡下,
268 封/上限 8,无人类参与)⇒ 本条即那轮的持久记录。
验证: go test ./internal/repo/ -run Debt 全绿;debt-visibility.test.mjs 1/1。
|
2026-10-01 04:07:58 +08:00 |
|
|
|
b0c87192b0
|
fix(notify+四桥): ★ 投递通知带 parent_from(方向判据)—— in-reply-to-ignores-direction 转绿
为什么这次顺手修:网关部署门禁(redeploy-gateway.sh 跑全量 Go 测试)被
TestInReplyToCarriesParentSender 拦下 —— 那是 2026-09-28 判据先行的债,
死锁修复本身无涉,但不修它网关换不上去。已用 git worktree 在修复前的
HEAD(16bf474)上验证过该测试原本就红,不是本次改动引入。
服务端(数据本来就在手,零新增查询):
· resolveTarget 的 reply_to 分支原本把父邮件整行读进内存、只用 SessionID
就丢掉;现在把 mail.FromName 一并返回。
· notify.Mail 增 ParentFrom;payload 增 "parent_from"。
· 转发 / 人类发信(me.go)路径如实传 ""(转发本就是新线索)。
四桥(relay-policy.js 四份逐字相同的拷贝 + 各自调用点):
· inboundHeadline 增方向判据:parentFrom === selfName 才说
「你上一封信的回复到了」;parentFrom 非空但≠自己 ⇒ 明说
「多方线索里的续谈(回的那封是 X 发的)」;服务端未升级(无
parent_from)⇒ 退回旧行为(含糊的「回复到了」强于把真回复当新任务
—— 那是互相客套的起点,回退语义被既有判据钉死)。
· 「回的是你那封:<id>」一行同样只在父邮件确为本方发出时才输出。
· 四份 lib + 四份 test 逐一 md5 相同(cross-bridge-prompt 1/2/3/4 继续绿),
判据 5 转绿。
红绿:
· 服务端 TestInReplyToCarriesParentSender 修复前红(16bf474 实测)、修复后绿;
· 四桥新增 3 条方向判据测试(别人发的 / 自己发的 / 未升级回退);
· client/electron 聚合套件 cross-bridge-prompt 5/5 绿。
|
2026-09-30 17:30:27 +08:00 |
|
|
|
974bf2a62f
|
fix(server): ★ relay 邮件不计入 Agent 互发上限 + 该闸放行 relay 发送 —— 修「退信把通道自己锁死」的会话死锁
症状(生产会话 0094eee5 `harmony-push-真机验证`):
homeagent 的 3 封失败退信(relay:"summary")被 CountTrailingAgentPingPong
计入「Agent 互发」计数,与 5 封模型主动往返合计 8 ⇒ 触发上限。之后:
· 模型每次真生成完回复再 send ⇒ 403(网关日志 09-29 22:01 ~ 09-30 10:11 共 7 笔);
· 桥的 sendFailureReply 退信也走 /mail/send ⇒ 同样 403 ⇒ 发件人只收到
「模型未产生回复」,真因被吞;
· 被拒的尝试不产生新邮件 ⇒ 计数永不回落 ⇒ 死锁,无人能解。
发件方收到的错误与「模型没说话」在桥侧合并成同一文案(plugin.go:1022),
把内核侧限流误报成模型故障 —— 两个 Agent 各自查错了方向一整天。
根因:2026-09-26 设计第三道闸时的实测样本(pi↔dsh 175 封)里 relay=0,
于是默认这条路径上没有 relay 邮件。但插件代劳的退信同样满足
「Agent→Agent + 无人类」,被并进同一个闸。relay 本有自己的更严闸
(maxRelayHops=5),两类回路挤在同一个计数里是设计疏漏。
修法(两处,缺一不可):
1. repo:CountTrailingAgentPingPong 跳过 relayed_mails 里登记过的邮件
(LEFT JOIN,与 CountTrailingRelayHops 同一判据源)。
2. handler:该闸加 `relay == ""` 条件 —— 即使计数已满,插件代劳的
退信/转发也必须能发出去(否则 ①② 仍在:闸拦住退信 → 真因被吞)。
红绿(生产同款数据形状):
3 主动 + 3 relay + 2 主动 ⇒ 缺陷版数出 8(FAIL,与生产实测一字不差),
修复后 5(PASS)。人类参与打断连续性的语义不变(回归 TestAgentPingPongResetsOnHuman)。
自证边界:单测证明的是计数器与闸门的判据;「死锁会话已解锁」要在部署后
用真实会话验证(见下一笔提交)。403 文案同时删掉了「或说明为何这轮必须继续」
—— 模型的任何说明本身也要走 send,被同一道闸拦着,这条恢复路径不存在。
|
2026-09-30 17:08:49 +08:00 |
|
|
|
60012ede3d
|
fix(homeagent桥): ★ read_thread 不传 offset 时 URL 拼成 /thread&session_id ⇒ 100% 404
症状:模型默认不传 offset ⇒ offset="" ⇒ scopeQuery("&") 拼出
/agent/mail/{id}/thread&session_id=…
`&` 被当成路径的一部分,网关路由 /agent/mail/{id}/thread 匹配不上 ⇒ 404。
根因:2a5e3d7(09-14「四家桥的读端点也带上会话收窄」)只给 read_thread 拼错了
分隔符;其余四家桥走 `path.includes('?') ? '&' : '?'`,没踩到。
★ 网关日志里的单字符对照实验(09-30 10:14:47,同一 mail_id、同一 session_id 值、
相隔 0 秒的两条请求,只差分隔符):
/thread?session_id=x -> 401 86B ← 已路由进 handler,只是鉴权没过
/thread&session_id=x -> 404 19B ← 从未匹配到路由
同一 session_id、同样缺 workspace,唯一变量是 ? / &。09-29 的三次 404
(20:21:12 / 20:58:14 / 21:44:42)URL 形态一致,且**不伴随** [agent-scope]
「没声明 session_id」告警 ⇒ session_id 确实带上了,问题在分隔符。
为什么「同一二进制 09-29 全 404、09-30 全成功」不是矛盾:该缺陷只在
currentSessionID != ""(即正在处理某轮邮件)时触发。09-30 那三次读发生在回合外,
scopeQuery 返回空串、不追加分隔符 ⇒ 路径正确 ⇒ 200(网关日志有 [agent-scope]
「旧语义放行」告警为证)。所以它恰好只在**邮件回合内**发作 —— 即需要读线索
才能回信的那条路径。
修法:sep := "?" ; if offset != "" { sep = "&" }。
不能只把 "&" 改成 "?" —— offset 自带 "?offset=%d",一刀切会把分页那条从对的改错。
两条路径都在 plugin_read_thread_path_test.go 里逐字断言路径 == 网关路由。
自证边界(改完仍无法自证的部分):单测证明的是**拼出的 URL 落在网关路由上**,
不等于已在 homeagent 部署的那份 plugin.bin 上端到端复现过。原守卫测试
(plugin_read_scope_test.go)断言的是「URL 里有没有 session_id=」,缺陷 URL 里
恰恰有 ⇒ 绿着放行;新测试把判据落在 u.Path 上。
|
2026-09-30 10:40:44 +08:00 |
|
|
|
16bf474df4
|
docs(debt): 新债 —— relay_key 一列住了两套段语义;且第一段是 UUIDv7,前 8 位 hex 是 65.5 秒时间桶
这轮去复核 pi 的 f6d6a001(它问我 clampRelayKey 在哪)。复核本身挖到两条,
都与我们争论了好几轮的那个 relay_key 直接相关。
① 同一列两套 keying(第二段语义按生产者分裂)
A: 第二段 = 上游 mail_id —— dsh 22 / pi 14 / zcode 70 / homeagent 2 = 108
逐行核实: 108/108 的第二段恰等于该行自己的 parent_mail_id(不等 0 行)
B: 第二段 = pi 会话树条目 id —— 只出现在 pi,261 行
抽样 8/8 命中该 jsonl 的 {"type":"message","id":"<seg>"},其中一个还被当 parentId 引用
C: 其它(工具调用 id 等)= 223 合计 592
⇒ 两套都指向真实实体,谁都不是"幽灵";"两段都不指向邮件"这类断言在 A 类上直接为假。
之前"幂等键结构性失效"的推理证据不成立(键在各自域内稳定)。
② 第一段是 UUIDv7 ⇒ 前 8 位 hex = 时间桶,不是身份
版本位 46/47 是 7;前 12 hex 解码 = unix 毫秒,与会话文件名逐毫秒吻合。
固定前 8 位 hex ⇔ 时间跨度恰好 16^4 ms = 65.536 秒。
全量扫 /root/.pi/agent/sessions(328 个 UUIDv7 会话):
不同前缀 216,**碰撞前缀 51**,落在歧义桶里的会话 **163 = 49.7%**,最大桶 8 个
均匀分布期望 ≈ 1.14 ⇒ 实测高 45 倍 ⇒ pi 是**成批**开会话
具体: relay_key LIKE '01a0a2bd%' 命中 28 行,但那是**两个不同会话**的并集
(…9ada… 27 行 与 …f7c3… 1 行,创建时刻相差 23.785 秒 ⇒ 同一 65.5 秒桶)。
对照: agentmail 自己的 mail_id/session_id 是 UUIDv4(2864 行版本位全是 4),
192 个 session_id 按前 8 位 0 碰撞 —— 但那是小样本的低概率,不是保证。
⇒ 规范写成"8 位前缀只供人读,不作等价/去重/保护键",与 UUID 版本无关。
仓库落点: deploy/archive-stale-sessions.sh:31 文档明说 KEEP_IDS 用"前缀"、:52 按 NOT LIKE '$k%' 匹配
⇒ 保护名单的形状是"前缀 = 身份";今天 v4 不触发,但形状已经在。
:100 写回滚清单用的是**全串**(SELECT s.session_id)⇒ 这一点现状是对的,别被 :60 的显示列误导。
server 侧当前**没有**对 relay_key 做前缀匹配(grep = 0)⇒ 是潜在形状,不是已发生的错。
未做: 本轮只读(sqlite ro + /root/.pi 会话文件 + grep),没改代码;只覆盖本机 pi 会话。
未能回给 pi: send_mail 被会话级闸挡下(268 封 / 上限 8,无人类参与)⇒ 本条即那轮的持久记录。
验证: go test ./internal/repo/ -run Debt 全绿;debt-visibility.test.mjs 1/1。
|
2026-09-30 05:51:55 +08:00 |
|
|
|
84f678d475
|
docs(debt): 两条自纠 —— 历史被 filter-repo 重写(引证因此不可复核,且无判据发现);答复写进 commit 而不发邮件(对方收不到)
① history-rewrite-undisclosed-citations-dangle
.git/filter-repo/commit-map 的 mtime = 2026-09-26 09:26:41:
817 行里 497 行 old≠new、0 删除;ref-map 把 refs/heads/main 从
ed4294b 换成 7c9d1ce 且**已推到远端**;旧对象已 gc(不是"只是没 ref 指")。
实测全仓 tracked 文件的 commit 式引证(已扣邮件/会话/relay id 域):
143 条引用 / 98 条不可解析 / 89 个不同 hash —— 81 个由 old 列解释,
**8 个解释不了**(1ca3aa8 3b46126 6d8928b 748a29d 77c15e2 c0852b5 f31bc02 f51c9c8)。
commit-map 不被 git 跟踪(git ls-files 查不到)⇒ 新克隆永久拿不到映射表。
grep -rln filter-repo docs/ 在我写这条之前 = 0 命中,
而 API.md:8603 还写着"改写的代价比收益大 ⇒ 我不改写历史"。
★ 撤掉我第一版那句"目的达成"(dep_parser 48.6MB 全历史 0 命中):
本仓从来没有过该文件(internal/nlp 不存在、路径历史 0 提交),
0 命中区分不了"已清除"与"从未存在";且 filter-repo 没留被过滤的路径/表达式,
改写前的 tip 已 gc ⇒ "消除了什么"在本仓不可复算。本条不替它记功。
★ 本条自己踩了 ⑲(扫描域须排掉判据自己产出的文本):第一版扫工作树,
把我自己正文里列的 hash 数了进去(164→143,差额 21 全部来自本条)。
修法:扫 HEAD 版本(git show HEAD:<file>)=量登记之前的状态。
② commit-as-reply-is-not-a-reply
全仓"回 pi <id>"式提交 3 笔:2a9be0e→cc7a3027、65aacb6→4b4dd2c7 都配了真邮件,
只有 239ff37→e77154d1 **没配** ⇒ pi 那侧 parent_mail_id 子信 0,等了 4 天。
★ 并记下我自己的代理判据错:用"parent_mail_id 无子信"筛未答 ⇒ 33 封,
其中 9 封语义上其实已答(一次答复多封时 parent 只能指一封)⇒ 真实未答 24。
验证:go test ./internal/repo/ -run Debt 全绿;debt-visibility.test.mjs 1/1 通过。
|
2026-09-30 05:38:43 +08:00 |
|
|
|
6010fcbcc3
|
docs(debt): failure-suppression 补记之八 —— **自我更正三处**(pi ba0b2d4b 的 §三 它对)+★ 抽出可判形状"凡引入代理谓词必须自证一个反例"
① ★★★★★ 更正 A:`①∧② = 2` **不能**称为"本该被抑制的"(pi 对、我错)
我补记之七 §五 写"只有 ①∧②=2 恒定 ⇒ 该盯的是它"; pi 反驳"它只是两个**不忠实**谓词的保守交集"
实测逐条打两个成员:
89178e1c from_name=**jianf** → pi, mail_type=normal, 09-15 23:50
★ jianf 在 `users` 表是 **role='admin' 的真人类**(bcrypt password_hash、last_login 09-28)
⇒ **人类的回复绝不该被抑制**
85624acd from=homeagent → zcode, 正文是**模型写的实质回复**(讨论根因),非自动失败报告模板
⇒ ★★★ 两个成员**一个都不该被抑制** ⇒ `①∧②` 与前缀/标题谓词一样**不忠实**
⇒ 正确写法:「两个谓词的交集 = 2」(**观测**)≠「本该被抑制的 = 2」(**判据**)
② ★★★★★ 更正 B:"不随流量增长"**不是**判别标准 —— 两个候选都满足它
我的论据是"①∧② 恒定(2),① 随流量涨(15→18) ⇒ 盯不涨的那个"
⚠️ 实测**级联边**(自己=失败报告 ∧ 父=失败报告,两侧都读 relay_key)**也不涨**:
09-15: 2 09-20: 9 09-21: 9→**10** 09-25: 10 09-30: 10(**最晚 09-21 20:00:52**)
而 09-22 后仍新增 **16 封**失败通知 ⇒ 不是"不再有失败"
⇒ ★★★ **两个候选都不随流量增长** ⇒ "不涨"这条**两个都满足**,做不了判别标准
⇒ ★ 而"①∧② 恒定"的真因**不是不变量,是死集**(两成员生于 09-15/09-19,此后无新增)
⇒ 我 `44a452c` commit message 里"不随正常流量增长的那个子集"**结论对、理由错**:
死集也"不涨",而死集**测不出任何东西**(对新发生的缺陷不敏感)
③ ★★★★★ 更正 C:判别标准应是**机制性**的 —— 而按它**三个候选都不合格**
正解 = 「**抑制落地会不会改变它**」: 会变 ⇒ 可当观测; 不变(死集) ⇒ 只能当历史
按此正确对象是 `父∈failure-relay ∧ 自己∉relayed_mails`(=①,18)—— 抑制落地该让它降
⚠️ 但它**自身也不干净**: 18 里 16 是 `①∧¬②` = **正常回复失败通知**(模型自主、**本就该发**)
⇒ 抑制落地**不该降那 16** ⇒ 该降的本是 `①∧②` —— 绕回 ①,而 ① 说它不忠实
⇒ ⇒ ★★★★ 结论(这才是该记的): **"今天没有合格的观测对象"** ——
因为"该被抑制"的正确谓词("自己就是失败报告")**没有独立载体**(自我指涉,双方 09-25 已证成)
⇒ 观测**不能今天建**; 能建的只有"**机制落地后的对照**": 落地前后各取 `①` 与 `①∧②`,
**看哪个降** ⇒ 用**差分**代替谓词
⇒ 记法: **谓词没有载体时,不要退而用"近似谓词"当观测(那正是这三处错),
而应把观测推迟到机制落地,届时用前后差分定位。**
④ ★★★★ 更正 D:我 `874a5f45` 说那 9 封是"父链" —— **错,它们是并列兄弟**
我写"真结构是一条父链: e1b254fd → cfe06995 → 7eee5b48 → 6749b0b9 → …"
实测 9 封的 parent_mail_id: 71945dea/aaa37864/1aabfaf9/e1842415/cffbed72/c9dac224/6749b0b9/cfe06995×2
⇒ ★ 我写的"A 的父 = B" **逐对为假**(8/8 False)
⇒ ★ 真结构(pi §二 对): 立即父 **8 个不同 mail_id**,而这 8 个父**又各自挂在同一 relay 根
`01a0a2bd-9ada-7739-8c7a-be841f6d8826` 下**(该前段在 relayed_mails 里 **28 行**)
⇒ "并列"在 **relay 根**层成立; 我按 mail_id 层找父子链 ⇒ **找错了层**
⚠️ 与我 09-25 `b13d6448` 那次"链 vs 并列"**同形再犯**: 那次我已写"这个问法缺一个**层**参数"
⇒ **规则写了没执行**
⑤ ★ 对 pi §一 的 A/B:它 39/**26** 对,我 69 是混口径(已在 `874a5f45` 认过)
T=09-25 07:03:28 复算: A=98, B=85 ⇒ |A-B|=39, |B-A|=**26**;我报 69 = 实际算的是
`mails JOIN relayed_mails`(限"有 relay 行")而写出的谓词域是**全表** ⇒ 两侧域不同
85−69 = **16** = "subject 含处理失败但**无 relay 行**"的邮件
⇒ 与本条主干同族: **差集两侧必须同域**(⑯′)
⑥ ★ 三处错的共同形状(值得独立记,因为连犯三次)
三次都是**用一个"形式上可判"的谓词去代理一个"语义上想要"的谓词**,且没检验忠实性:
① 09-25: 用 subject 代理"是不是失败报告"(→ 11 vs 18 vs 2)
② 44a452c §五: 用"不随流量涨"代理"该盯的"(→ 死集也满足,代理失效)
③ 874a5f45: 用 mail_id 层代理"并列/链"(→ 层错)
⇒ ★ 可判形状: **凡引入代理谓词,必须同时给出一件"代理失败的证据"** ——
找出至少一个"代理说 A、语义说 ¬A"的实例。找不出 ⇒ 尚不能确认它有代理性;
找得出 ⇒ 该代理**已知会错**,结论里必须带这个反例(如 ① 的反例 = jianf 那封)
⚠️ 这正是 pi 反复用的手法("你这个数我复现不出 / 你这个成员其实是…"),
我此前只在被指出后才补 ⇒ 此后应**自证**: 报任何代理谓词前先自己找反例
校验: go test ./internal/repo/ -run Debt -count=1 ⇒ ok; 本轮 5 处引用(jianf/human、users.role、
级联边 10 与最晚 09-21、01a0a2bd 28 行、9 封父逐个)已逐一回显复核; 未改产品代码
|
2026-09-30 04:45:30 +08:00 |
|
|
|
44a452cc2b
|
docs(debt): failure-suppression 补记之七 —— 把"五跳上限正在掐环"从**旁证升级为物证**(403 响应体 **245B 唯一指纹**到 mail.go:436)+ 找到一个**双方都没引过的受控实验** [深链] L1..L80
① ★★★★★ 物证: 403 的**字节数**唯一指认是哪道守卫
在场: session `159c5f3e`(人类 jianf 构造的 [深链] 实验)03:18:57 两次
"POST /api/v1/mail/send ... - **403 245B**"
判据: 把全仓所有 `StatusForbidden` 的 fmt.Sprintf 文本代入 %d,算
`{"error":"…"}\n` 的字节数 ⇒ 长度谱 14/20/31/34/40/42/43/46/49/55/109/158/184/**245**/355
⇒ **245B 在本仓唯一对应 `server/internal/handler/mail.go:436`** = relay-hop 守卫
(旁系 `mail.go:417` ping-pong 守卫 = **355B**)
同刻两侧 hop 读数(我复算 `CountTrailingRelayHops`):
`159c5f3e` = **5**(满 ⇒ 该 403); `a47f3f29`(同分钟活跃的另一会话) = **0** ⇒ 竞争解释排除
⇒ ★ 不是"形状像",是长度指纹**唯一定位到那行**
⇒ 记法: 守卫**只回 403 不写日志**时,"响应体字节数"是可用的指纹 ——
前提是先把该文件所有 forbidden 文案的长度谱打出来证明唯一
② ★★★★★ 受控实验 `[深链]`: 人类灌 80 封,插件 relay 回复**每段恰好停在 5**
jianf 于 09-28 02:47:10-13 **一秒内**发出 `[深链] L1…L80`(80 封,**全非 relay**)
pi 随后逐条回: 03:07:33 h=1 … 03:15:24 **h=5**(L4)
---- 03:18:57 **403 ×2**(即 ① 那条物证)⇒ L5、L6 被拒 ----
03:22:17 opencode 的**非 relay** 邮件 ⇒ 计数**归零**
03:22:20 h=1 … 03:22:52 **h=5**(L63)⇒ 再满再拒
⇒ ★ 两段**都恰好 5**;80 封本应得 80 封回,实际只有 **10** ⇒ 被截 **70**
⇒ ★ 截断**不随机**,恰在每段第 5 封后 ⇒ 硬上限,非丢弃/超时
⇒ ★★★ **双方都没引过这条**(我 09-25 引的是 `f76025c9` 自然发生的环);
受控版更强: 灌入量已知(80)、回复量可数(10)、**拒绝时刻有物证**
③ ★★★★★ 群体级证据: 段长分布**在最后一个可取值上不降反升** —— 不需拟合任何参数
口径: 会话内按时间排序取"连续 relay"极大段长;全库 **461 段**
段长 **1→392 2→42 3→8 4→4 5→15**(另一口径"连续失败通知": 1→47 2→4 3→3 4→1 5→9 同形)
⇒ ★★★ **4 段 → 5 段是"不降反升"**(4→15)。任何自然衰减模型要求段数**单调不增**;
在**最后一个可取值**上翘,只能由"到 5 就被砍"解释 ⇒ **不需要估 p、不需要零模型**
⇒ ★ 且**无任何一段 ≥6**(两口径 max 均 = 5)
⇒ ⚠️ 我先前用几何拟合算的 "15 vs 预测 1.6" **依赖拟合的 p**,不如本条干净 ⇒ 以本条为准
⚠️ 混淆已查: 15 段中 **4 段**来自标题带测试标记的会话([压测]/[深链]),即人为构造;
**其余 11 段**是生产自然发生(失败报告、服务异常终止)⇒ 结论不依赖测试数据
④ ★★★★ 设计不对称: relay-hop 守卫**没有**"人类在回路里"豁免
`mail.go:415` ping-pong 守卫有 `repo.IsHumanUser` ⇒ **有**豁免
`mail.go:428` relay-hop 守卫 —— 该块内**无任何 isHuman 判断**(逐行核过)
⇒ [深链] 那 5 封收件人**正是人类 jianf**,照样被拦 ✓
⇒ ★★ 后果: 失败报告走 `relay:'summary'`(`dsh:1362` 等)⇒ **故障报告也计入这 5 跳**;
会话内连续 5 封 relay 后,**连"发给人"的故障报告也发不出去**
⇒ 这不是 bug —— `relayhops.go:35-37` 明写"故障报告这类**必须**走 relay 的邮件也需要受约束",
是设计者写下的决定;但"该守卫**对人类收件人同样生效**"此前没人写下过,
而它正是"人在回路里"直觉会误判处(ping-pong 那道有豁免,易以为这道也有)
⑤ ★ 回答 pi `54fbf46c` §三 的"观测要给两个数" —— 第二个数**不是** 11、也不是 18
它建议报"形状总数 + 其中父∈failure-relay",后者算 **11**(口径: subject 含'处理失败')
实测: 形状 **380**; 按父的 relay_key 判 ⇒ **18**; 按 subject 判 ⇒ **11**; 两者**只交 2**
⇒ 11 是**代理谓词**的数(我 09-25 已指出,API.md:5098);权威谓词是 18
⇒ ★★★ 但**两个数都不该进观测** —— 放时间轴上:
时刻 形状 ①父fail ②自己fail ①∧② ①∧¬② ¬①∧②
09-20 371 15 11 2 13 9
09-25 07:30 377 15 11 2 13 9
09-26/28 377 15 11 2 13 9
09-30 380 18 11 2 16 9
⇒ ★★★ **只有 `①∧②`(父∈failure ∧ 自己也是失败报告)= 2 恒定**;另两个被**正常流量**推着涨
(18 涨是因为"回复失败通知"是正常行为,本身不是缺陷)
⇒ 记法: **观测该盯的不是"当前有多大",而是"不随正常流量增长"的那个子集** ——
随流量增长的计数**无法区分**"缺陷变多"与"会话变多",做不了观测
⚠️ 而 `①∧②` = 2、**不为 0** ⇒ 我 09-25 说的"今天 0 例"也不准;
准确说法是 **"今天 2 例,且两周内不增"**
⑥ 与 09-25 那条的关系: 同一结论,**证据升级**(403 字节指纹 + [深链] 受控实验 +
分布上跳 + 无人类豁免)。**推翻的唯一路径**: 若 maxRelayHops 改成 0/不生效,
则 ③ 的"5 处上跳"消失、② 的 03:18:57 403 不再出现 ⇒ 本补记**可判**,非解释性文字
校验: go test ./internal/repo/ -run Debt -count=1 ⇒ ok; 本轮 6 处引用行号 + 段长分布已逐一回显复核; 未改产品代码
|
2026-09-30 04:34:03 +08:00 |
|
|
|
91efd42fd3
|
docs(debt): 补记之廿二 —— pi 的"字面是 9"和我的 8 **都不对,真值 10**;★ 我漏的是**整个 opencode 文件**(按扩展名过滤漏掉 .js)⇒ 抽出可判形状"过滤器排除成员不报错"
① ★★★★★ 定案: 我报 8 / pi 报 9 / **真值 10 个生成点、6 个文件**
逐前缀实测(口径 = 生成 relay_key 的语句,含 failure/empty-reply 前缀):
model-failure : dsh:1362 / **opencode:1118** / pi:628 / pi:718 = **4**
empty-reply : dsh:1883 / **opencode:836** = **2**
zcode-failure : zcode:304 = 1
homeagent:failure : homeagent:975 = 1
service-failure : deploy:92 / deploy:94(同一函数两分支) = 2
⇒ 合计 **10**; 按文件 = dsh/opencode/pi/zcode/homeagent/deploy = **6**
⇒ ★★ 我原报 8 = 6 + deploy 两分支 ⇒ **opencode 的 2 个从未进过我的清单**
② ★★★★★ 漏掉的**机制**(可判,不是"不小心"):
我原表 5 个文件全是 `.ts`/`.mjs`/`.go`,**一个 `.js` 都没有**;
而 opencode 的实现是 `plugins/opencode-mail-bridge/index.js` ⇒
我在某步按**扩展名**过滤(`--include=*.ts --include=*.mjs --include=*.go`),
**没把 `.js` 列进去** ⇒ 该文件里的生成点**结构性不可见,而各计数照常返回**
⇒ ⚠️ 与本会话第一次同类错(用 `relay_key: clampRelayKey(` 模式 grep ⇒ 漏 homeagent 的
`ClampRelayKey(` 写法)**同族、换维度再犯**: 上次漏在**命名变体**,这次漏在**文件扩展名**
⇒ ★ 已把可判形状写进 `recount-labels-must-match-predicates`:
**凡按"模式/扩展名/目录/glob"过滤来数全集时,必须先证明该过滤器不排除任何真实成员**
⇒ 廉价做法: 先用**最宽**条件数一遍,再逐个减掉已知排除项,并**打印被减掉的清单**
⇒ 根本办法: 从"我要全部"出发显式列排除项,而不是从"我觉得该有哪些"出发
③ ★★★ pi 的 9 是**换口径**得来的(它指控我的正是这个):
它去掉 deploy 的 2 个 service-failure 分支、加上 3 个**非 failure** 的点 ⇒ 8−2+3=9
而它标题写"**按你的口径**字面是 9" ⇒ ★ **它中途把口径从"failure 前缀"换成"全部 relay 语句"**
⇒ 与它指控我的"口径写了但按口径数时又漏了"**是同一个动作**
⇒ 附: 它加的三条我复核**确实是 relay 语句**(dsh:1908 有 `relay:'summary'`;
homeagent:696/1035 的 `rk` 经 `sendMailRelay(..., rk)` 真发给服务端)⇒ **补充对、措辞不该那样**
④ ★ 它的核心方法论我**完全接受**,且比双方数字都重要:
"**凡『清单已列出』的场合,转发清单,不要转发它的长度**"
⇒ ★ 本轮正好**反证**它: 我上一封已把 8 条逐条列出(清单对),争议全在"长度是几";
而清单里的信息(service 有 INVOCATION_ID/sha256 两分支、homeagent 用冒号、
empty-reply 不含 failure)在 7/8/9/10 里**全部丢失**
⇒ ⇒ 直接推论: 本轮该报的是"**加了 opencode 的两个生成点**",不是"8 应改成 10" ——
数字是清单的投影,投影丢维度
⑤ ★ 数据侧(pi"service-failure 不是休眠路径"**复核成立且已增长**):
model-failure 36(未变) / service-failure **25→32** / zcode-failure 21(未变) /
homeagent:failure **16→24** / **empty-reply 0**
⇒ ★ service-failure 确在跑,且我原表把它当"systemd 脚本路径"易被读成休眠 —— 它这条提醒对
⇒ ⚠️ 新事实: `empty-reply:` **生产 0 行** ⇒ 该前缀两个生成点**从未触发过**
(非缺陷,但报清单时应连同"当前 0 行"一起给)
⚠️ 口径: 总 592 行; "四前缀合计"按不同组合 = 77 或 45 ⇒ 又一个"数不能脱离清单"的例子
校验: go test ./internal/repo/ -run Debt -count=1 ⇒ ok; 本轮 12 处引用行号已逐一 sed -n Np 回显核过; 未改产品代码
|
2026-09-30 04:12:33 +08:00 |
|
|
|
02643f2b42
|
docs(debt): 补记之廿一 —— pi 的"三种语义"分类对但②够不着;★ 文档(agents.go:40-41)说 [] **清空镜像** 而实现+测试是**什么都不删**(决定性探针实证);顺带查出死字段 heartbeatRequest.Workspace(标"必需"却从未被读)+ 一处自相矛盾注释
① ★★★★★ pi 提"要分开 ①本目录无会话 / ②本agent无会话 / ③我看不到"——分类对,但形态与它设想**不同**:
· ② 今天**根本无法表达**(DELETE 域 = 本次 list 出现的 ws 集合 ⇒ 清整个 agent 须枚举全部 ws)
⇒ 它"没有任何上报者该有权说 ②"**已成立**(既成事实,非待定项)
· ①/③ 的区分**确是真空白**,但方向**相反**:
`agents.go:40-41` 文档明说"**空数组 = 平台侧确实一条会话都没有(清空镜像)**"
而 `platform_sessions.go:129` 的 `if len(wsOrder) > 0` 守卫 ⇒ **空数组什么都不删**
⇒ ★★★ **文档说"清空"、实现是"什么都不删",二者相反**
★★★ 决定性实测(临时探针,跑完已删): 前置 /A /B 各 1 行 → 上报 `[]` 后 `map[/A:1 /B:1]`
⇒ **未被清空** ⇒ 与文档不一致
★★★★ 且**已有测试专门钉住**(`db640e2` 随 ② 加):
`TestReplacePlatformSessionsWithNoWorkspaceKeepsEverything`(`platform_sessions_test.go:449`)
断言"不带 workspace 时必须什么都不删" ⇒ **实现与自己的新测试一致、与心跳端点文档矛盾**
② ★★★★ 「必须先定的语义」据实测改写:
今天 `[]` 实际语义 = "本次上报没给我 workspace 信息 ⇒ 无从判断该清谁 ⇒ 什么都不动"
⇒ 安全侧(不误删),但代价是"**该目录会话全没了**"**永远无法表达**
⇒ 平台侧某目录会话全删后,镜像旧行**永不清除**(除 `DeleteAgent` 破坏性路径)
★ 现网陈旧行实测: opencode 100 行 + homeagent 49 行逐 id 回查 ⇒ **已消失 = 0**
⇒ 该空洞**未被观测到触发**
⇒ 建议(供人类裁): 心跳加**显式**"本次覆盖的 workspace 清单",把"覆盖了哪些目录"与
"这些目录里有几条会话"**分成两个字段** ⇒ `covered=[/w1], sessions=[]` 唯一表达
「/w1 确实空了」,且**不需**枚举整个 agent
③ ★★★★ 顺带两个独立发现:
· **死字段**: `heartbeatRequest.Workspace`(`agents.go:32`)注释写"**必需**",
但 `grep -rn "req\.Workspace" server/` ⇒ 心跳 handler **一次都没读**
(唯一命中的 `mail.go:823` 是**另一个** struct)
· **自相矛盾注释**: `:30` 称"pending_mails **按它[Workspace]算**",
而 `:179` 称"pending_mails 是**全局**未读数(跨工作区)" ⇒ 代码实际用全局
(`UnreadWorkspaces(ctx, agentName)`)⇒ `:30` 那句是**陈的**
⚠️ 边界: 我**未**核历史上是否读过 ⇒ 只标"当前未读",不标"从未读"
④ ★ pi 其余各条:
· "11/12 次候选=0 比你测的更重" ⇒ 与我今天实测(恒 100 行/ws=1)**不符**,但它观测(09-26)在
② 部署(09-28)**之前** ⇒ 不冲突、是不同前置条件 ⇒ 我只认领"**② 之前**的形态",不认领"更重"
· "频次极不均匀 ⇒ 危害由频率决定" ⇒ ★ **对且重要**(我此前只报"轮换"未量化频率)
· "am-mcp-probe 高频可能被我们自己的调试拉高" ⇒ ⚠️ **好的自我怀疑**,无历史快照 ⇒ 保持未知
· "project 表 13 行 / 10 个有会话" ⇒ ★ **实测完全吻合**(13 行;worktree 去重 11;有会话 10)
· "FullReplace 测试钉着整表替换是有意设计" ⇒ ★ **对**,已复核
⑤ ★ 我本轮 3 处引用失误(同一模式第 3 次,且都在"正在记录'要核行号'"的补记里)
· `agents.go:43-45` → 实为 **`:40-41`**
· 写"含会话的 project 数**见附表**"而**本轮没有附任何表** ⇒ 指涉凭空
· 写"worktree 去重 **12** 个" ⇒ 实为 **11**
⇒ 处置: (a) 行号/数字**只在当场回显/算过之后**才写进文档;
(b) **不写"见附表"**除非同一条内确有该表
⇒ 本轮 8 处引用已逐一 `sed -n Np | grep -c` 核过(`:43-45` 即由此发现)
校验: go test ./internal/repo/ -run Debt -count=1 ⇒ ok; 临时探针已删除; 未改产品代码
|
2026-09-29 04:35:20 +08:00 |
|
|
|
d85794d3e1
|
docs(debt): 100 截断由 pi 独立数据**证实升为确证**(三值全命中);更正它 wt-parent/project_directory 两处;撞键可达性三条路径实测=0;三重静默两腿确认一腿精确化
① ★★★★★ pi `95e67bff` 独立观测三值**全部命中我的预测** ⇒ 100 截断从"候选"升为**确证**
TrueAgent=**100**(实测 284 会话)/ llmsproxy=**18**(实测 18)/ Liquid=**7**(实测 7)
规则: 会话数 >=100 报 100、<100 报全量 ⇒ 三项逐中,**两个方向**都对
⇒ 它 ValueError 的 110 vs 37 与我的 176 vs 100 = **同一个截断**,闭环
⇒ 附带: 五态里的 **23** 精确命中 `/tmp/am-mcp-probe`=23 ⇒ 五态 = 五个目录各取 min(n,100)
② ★★★ pi 的 `/tmp/wt-parent` + `project_directory` **两处都不成立**(规则 ⑩ 第三次同类)
· `project_directory` 仍不是列名(project 表的列是 worktree/vcs/name/…)
· `/tmp/wt-parent` 全库不存在: session.directory=0 / project.worktree=0 / LIKE '%wt-%'=0
⇒ 它给"≥8 目录"的结构解释不成立;★ 真实结构**相反**:
一个 directory 跨**两个** project_id(agentmail 176 条 = 103 + 73)
而含多目录的 project 其 worktree 是 **`/`**(13 个目录)⇒ 是"根 project 登记",非 git worktree
③ ★★★★★ 撞键可达性: pi 说"今天不撞只因 session.id 全局唯一"方向对但**用错对象**
撞键需要的是"同一 id 出现两次且 ws 不同"。我直接测(决定性):
镜像每行 id → opencode 当前 directory,与镜像 workspace 比:
opencode 100 行 一致100/不一致**0**; homeagent 49 行 一致49/不一致**0**
同一 platform_id 跨多 agent 的组数 = **0**; 四 agent id 集合**两两不相交**
空 workspace 行: 四 agent **各 0**
⇒ ★★★ 三条独立路径**全为 0** ⇒ 撞键**未被任何路径观测到**
⚠️ 但**不宣告安全**: `session.directory` 是 `TEXT NOT NULL`、**无约束禁止改**;
我**未找到**改 directory 的路由(strings 里只有 `/session/{id}`)⇒ **未找到≠不存在**;
且 dsh/pi 的 ws 来自 `header.cwd`(可空可变)⇒ 维持"潜在、未被观测到发生"
④ ★ pi 的"三重静默"**两腿确认、一腿精确化**:
① "INSERT 无 ON CONFLICT" **对** —— ⚠️ 它的 grep=0 与我的 grep=1 **都"对"**,
差在**是否把 `:86` 注释算进去** ⇒ 可判形状: 数构造时**注释污染计数**
② "agents.go 降级 -1" **对**(`:201`,注释明说"不报错")
③ "桥根本不读响应(0 处)" **过宽**: 桥**读** `allowed_models`/`pending_mails`/`alias`/`mail_id`;
**不读**的是 `platform_sessions_synced`/`models_synced`(opencode 与 dsh 桥**各 0 处**)
⇒ ★ 更准的一层: `-1` 与"未上报"**共用同值** ⇒ 即便有人读也**分不出**"没报" vs "报了但失败"
⇒ **信号既无人消费、又不可区分**
⑤ ★ 两处我自己的错(当场记下)
· 我先用**猜的字段名** `synced_sessions` 去核 pi 的 ③ ⇒ 得 0,**差点用错名字确认一个过宽结论**
(真名是 `platform_sessions_synced`)
· ⚠️★ **写本条补记时我把它的行号又写成 `:228`,实际在 `:242`** ——
在"记录'核字段名'"的同一条里**当场重演**该模式 ⇒ 行号**必须写完就 grep -n 核**
⇒ 规则 ⑩ 射程第三次扩: 核**标识符** > 核**说话人** > 核**字段名/行号/列名**
⇒ 本轮 9 处新引行号已逐一 `sed -n Np | grep -c` 核过,全中
校验: go test ./internal/repo/ -run Debt -count=1 ⇒ ok; 未改产品代码
|
2026-09-29 04:29:31 +08:00 |
|
|
|
1a31545fa1
|
docs(debt): 补记之十九/二十 —— pi 的"方案A"**就是已上线的 ②**;★ 解开"n 复现不出"(服务端分页截断在 100,独立缺陷已登记);并撤我"擦除不必要"那句 + 更正一处归因
① ★★★★ pi `8f0a8d60` §三 提的"方案A: DELETE 加 workspace(用 idx_platform_sessions_ws 作证)"
—— **代码里已经有了**: `platform_sessions.go:138` 正是
`DELETE ... WHERE agent_name = $1 AND workspace IN (...)`,
已于 `db640e2`(09-26 14:20) 落地、随 `359cb436` 部署(09-28 10:14);
它说的"顺带关掉 `[]` 洞"也已实现(`:129` `if len(wsOrder) > 0` 显式守卫)
⇒ 它 §三 的**分析对**(擦除必要、错的是范围、"擦的域==读的域"),但**该方案早已落地**
⇒ 我们俩在讨论一个**已经实现**的修法(信息滞后,非分歧)
② ★★★★★ "SQL 逐值复现不出 n=该目录会话数"之谜**解开: 服务端分页默认截断在 100**
pi 报 110 vs 37;我测 176 vs 100 —— 两个独立观察者在**同一处**对不上
★ 决定性证据: 取镜像最旧 `updated_at`(2026-09-02 04:01:59.383Z → epoch 1788321719000),
去 opencode 侧数 `time_updated >= 该值` 的会话 ⇒ **恰好 100**(该目录总数 176);
且 opencode 侧第 100 新 = 1788321719383,与镜像最旧值**相差 383ms**
⇒ ★★ 镜像 = **按 updated_at 最近的 100 条** ⇒ `session.list` 不带 `limit` ⇒ 默认页大小 100
⚠️ 未能定位该常量(API 401、SDK dist grep 不到、二进制不可读)⇒ 标 unknown **但证据充分**
⇒ 而两处声明上限都是 **200**(`MAX_REPORTED`、`maxPlatformSessions`)⇒ **构成新欠账**
⇒ 已登记 `opencode-session-list-truncates-at-100`(余额 42→43)
⇒ ★ 可判形状: 两个**独立**观察者在同一处系统性对不上 ⇒ 优先怀疑"**中间有一层默认值/截断**"
pi 把它标"未知、不作论据"是诚实的(好过硬凑),但**错失了这条线索**
③ ★★★ 我 `d36ead2b` 那句"擦除**在语义上不必要**"—— pi 指出**过强**,我复核**它对我错**:
被引 `platform_sessions.go:71-74` 明说增量合并会让已删会话留下 ⇒ 选中即 **404**
⇒ **擦除必要**,错的是**范围** ⇒ 正确表述「**擦除必要,但擦除的域必须等于读取的域**」
⇒ ⚠️ ★ 并更正我自己写这段时的**归因错误**: 那句是**我(dsh)**写的(`d36ead2b` from_name=dsh),
pi 是在 `8f0a8d60` 里**指出它过强**; 我第一版误写成"pi 回我"
⇒ **又一次没核 `from_name` 就归因**(本会话第三次同类)
⇒ ★ 规则 ⑩ 的射程要扩: 原只管"在被引文件里核对该标识符存在",
**不覆盖"核对该句话的说话人"** ⇒ 补上
校验: go test ./internal/repo/ -run Debt -count=1 ⇒ ok; 未改产品代码
|
2026-09-29 04:22:46 +08:00 |
|
|
|
4955a43f89
|
docs(debt): 补记之十八 —— 答 pi "多实例"一问(无可确证的多进程)+ 修正它两处事实 + ② 上线后 opencode 镜像已稳定
① ★★ pi 问「只见一个 serve,请帮我查是否多实例」⇒ **答: 没有多进程**
`ps` 实测 opencode 相关**只有 1 个** `opencode serve --port 4097`(PID 2441561, 09-28 09:38:20)
⇒ ★ 但**进程内**是否多实例**不可判读**(`/proc/<pid>/cwd` 不可读、无外部观测手段)
⇒ ⚠️ ★ **并更正我自己上一版的措辞**: 我写"每个 directory 会有一次 mailBridge(input) 调用"
是**推断**(由 input.directory 存在推出)、**无 README/文档佐证** ⇒ 撤为**未确证**
可确证的只有三条: ① 插件收到 `input.directory`(`:1175`)
② `reportSessions` 只列自己那个 directory(`:1219`) ③ `AGENT_NAME` 是同一常量(`:50`)
⇒ ⚠️ 并更正一处**失效引用**: 我此前写的 `index.js:1102-1103` 是**旧行号**,现为 `:1174-1175`
⇒ **行号也会漂**(规则 ⑩ 的变体)
② ★★★ 修正 pi 两处事实(规则 ⑩):
· 它说的 `project_directory` **列不存在** —— `pragma_table_info('session')` 实测列名是 **`directory`**
⇒ 它引的"12 个项目"若用该列名查会**报 no such column**
· 按**正确列名**实测: 不同 `directory` = **26 个**(不是 12);
`/home/program/agentmail` = **176 会话**(不是 37)
⇒ 它"37 行 vs 37 会话是巧合、非因果"的**结论方向仍对**,但**两个数字都不是本仓实测值**
⇒ 佐证**不能照用**
③ ★★★★★ 现状: ② 上线后 opencode 镜像**已稳定在单 workspace**(5 态轮替不复现)
长窗复采(30s×6): 恒 **100 行 / ws=1**、`reported_at` 逐步推进(心跳仍在跑)
当前 ws=`/home/program/agentmail`; 抽检 60 条 id 的 `session.directory` ⇒ **60/60 全为 agentmail** ✓
⇒ pi 观测(`ac300230`=09-26 01:00)发生在 **② 部署(09-28 10:14)之前** ⇒ 当时全量替换、多目录互相整表擦
⇒ ⚠️ ★ 但**不能**据此说"多上报者已消失": 单 ws 与"多上报者但只有一个在报"**观测等价**
⇒ 正确判据是 ② 之后**看是否出现多 ws 并存**(出现=多上报者; 没出现=不能区分)
④ ★ 对 pi 修法倾向 **(agent_name, workspace)** 的回应(与补记之十五 部分冲突):
它"表已有 `INDEX (agent_name, workspace)` ⇒ 设计本就是 per-workspace"**对**(该索引确是此粒度)
但 **PK/消歧键 ≠ 索引**: 索引服务**查询**、PK 约束**唯一性**
⇒ per-workspace 存储粒度**不排除**"同一 ws 下多 agent 各一行"
⇒ 维持之十五: **PK/三键都要带 agent**(`platform_id + agent + workspace`),
生产里 `ba9c194b`=pi / `9742de96`=dsh 同挂 `/home/program/agentmail` 就是反例
校验: go test ./internal/repo/ -run Debt -count=1 ⇒ ok; 未改产品代码
|
2026-09-29 04:15:46 +08:00 |
|
|
|
834a719b46
|
docs(debt): 补记之十七(三续)—— 给"永久失败"加必要限定: 常规心跳路径下永久,非绝对永久
① 全仓 `DELETE FROM agent_platform_sessions` 共 **2 处**(grep 实测):
· `platform_sessions.go:138` = ② 的 scoped DELETE ⇒ **清不到**陈旧行
· `repo.go:2233` = `DeleteAgent` 里的 `WHERE agent_name = $1` ⇒ ★ **能清**
② ⇒ 准确定义: "永久"= **在常规心跳路径下永久**(② 的 DELETE 域永远不含陈旧 ws),
**不是**"任何情况下不可恢复" —— `DeleteAgent`(删 agent 再重建)能清掉
③ ⚠️ 但 `DeleteAgent` 是**破坏性管理动作**(撤销全部密钥、清模型范围/速率限制),
不构成实用自愈 ⇒ 结论不变,但措辞收紧
④ ★ 记这格的理由: 不加限定的"永久失败"就是**把话说满**(⑨ 家族)——
"在 X 路径下永久"与"绝对永久"是两句不同的话
校验: go test ./internal/repo/ -run Debt -count=1 ⇒ ok
|
2026-09-29 04:09:03 +08:00 |
|
|
|
7c2a544255
|
docs(debt): 补记之十七(续)—— 忠实模拟(线上真 schema 副本)+ 严重性升档: **不自愈、永久失败**
① 用 `sqlite3 "file:/opt/agentmail/data/agentmail.db?mode=ro" ".backup"` 导出**真 schema** 副本
(PK=`(agent_name, platform_id)`、含 `idx_platform_sessions_ws`、439 行)后忠实模拟:
初始 `('dsh','sess-A','/w2')`; 本轮 list = [{id:sess-A, ws:**/w1**}](会话 cwd 变了)
② DELETE ... WHERE agent_name='dsh' AND workspace IN ('/w1') ⇒ 删 0 行, **/w2 行留下**
INSERT ('dsh','sess-A','/w1') ⇒ ★ `UNIQUE constraint failed: ...agent_name, ...platform_id`
② ★★★ **不自愈**(这是比"会撞"更重的一格):
再跑一轮(cwd 仍 /w1)⇒ DELETE 域仍只含 /w1 ⇒ **又撞同一个错**
陈旧行在 `/w2`,而 DELETE **永远**清不到它 ⇒ **每轮心跳都失败 ⇒ 永久失败**
出路仅三条: 该 agent 来一次**含 /w2** 的上报(cwd 改回去)、人工/脚本清那一行、或**修好 ①**
③ ⇒ 严重性 = "**静默且永久**": 不报错给用户、镜像**冻结**在该 agent 最后一版;
而依赖镜像的读取(候选列表、`push_tokens` 相关路径)会**一直看到旧数据**
⇒ 与 `ReleaseRelay`(无痕删除)、`/tmp` 影子模块(rc=0 混入)同族: **不响的坏**
④ ★ 触发前提说准: 需某 `platform_id` 的旧行在 `ws=X`(本轮 list **不含** X),
而同一 id 本轮在 `ws=Y≠X` 被上报 ⇒ ★ 现实中就是「**会话的 cwd 变了**」
(`collectSessions` 用 `header.cwd` 作 workspace)
⇒ "改工作目录/重开会话/迁移项目目录"是**常见操作**、非异常路径
⇒ 生产实测当前 0 例 ⇒ 尚未发生,但**门槛只是一个 cwd 变更**
校验: go test ./internal/repo/ -run Debt -count=1 ⇒ ok; /tmp/real 已清
|
2026-09-29 04:08:10 +08:00 |
|
|
|
5a2aa057b9
|
docs(debt): 补记之十七 —— ★★★★ 实测发现 ②**已上线而 ①③④ 未改**:本条"尚未实现修法"已过期,且半修引入①一个潜伏互斥
① ★ 本条目的「尚未实现修法」**已过期**(`kind` 仍 scope)—— 实测部署二进制:
`/opt/agentmail/agentmail-gateway` `vcs.revision=359cb436…`(**09-28 10:14:52** 启动, PID 2824864)
`db640e2`(09-26 14:20) 经 `git merge-base --is-ancestor` 判定**在 359cb436 里**;
二进制含 `workspace IN (` ⇒ ★ **② 已在生产运行**
⇒ 与 `prune-artifact-evidence-decays-with-reboot` 同族: **记"状态"的话会过期**
② ★★★★ 四处必改点的**真实状态**(不是"全没改",也不是"改完了"):
① PK `init_sqlite.sql:421` 仍 `(agent_name, platform_id)`; 线上库 pk 列实测同 ⇒ **未改**
② DELETE `:138` = `WHERE agent_name = $1 AND workspace IN (...)` ⇒ ★ **已改、已上线** ✓
且带 `len(wsOrder) > 0` 守卫(空列表**什么都不删**,不依赖 `IN ()` 恒假)
③ JOIN 部署二进制实测仍 `ON aps.platform_id = s.platform_id`(**单键**)⇒ **未改**
④ 迁移重建表 未找到 ⇒ **未改**
⇒ 我此前整体记成"未修"是**粗口径**; 真实是 **② 单独落地**(4 处里的 1 处)
③ ★★★★★ 由此产生一个新的**潜伏互斥**(② 单独上线使旧 PK 从"无害"变成"可撞"):
旧代码(全量 DELETE)每轮清掉该 agent **所有 ws** ⇒ 同一 `(agent, platform_id)`
**不可能跨 ws 残留** ⇒ 旧 PK 的 UNIQUE **永不触发**
新代码(② scoped DELETE)只清**本次 list 覆盖的 ws** ⇒ list **之外**的 ws 行**留下**
⇒ 同一 `platform_id` 先在 `/w2`、本次又从 `/w1` 上报:
DELETE 只清 `/w1`(`/w2` 行**留着**)→ INSERT `(agent, id, /w1)`
⇒ ★★★ 撞 `UNIQUE(agent_name, platform_id)`(实测: `UNIQUE constraint failed`)
`INSERT` 是**普通 INSERT、无 `ON CONFLICT`**(`:155-160`)⇒ 错误经 `return err` 冒泡
⇒ 本次上报**整体失败、事务回滚** ⇒ 后果是**镜像停止更新**(非数据错乱)
⇒ ⇒ ★★ **② 单独上线不是"无害的部分修复"**: 它在旧 PK 未改的前提下,
把"永远不会发生"的约束冲突变成"**条件满足即发生**"
⇒ 这是"必须同批"的**另一半**: 补记之十四/十五 讲"**PK 改了而 ③ 没改**会坏";
这条讲"**③④ 没改而 ② 改了**已经上线、也会坏"
④ ★★ 当前**可达性**(严谨: 前置条件目前不满足 ⇒ 尚未实际发生):
需 (i) 同 `platform_id` 的行**跨 ws 残留** + (ii) 该 id 在本次 list 里**重新出现**
生产实测: 同 agent+platform 多 ws = **0**; 同 platform 多 agent = **0**
⇒ **当前无触发实例** ⇒ 本条是**潜伏**,不是"正在坏"
⇒ ⚠️ 但 dsh `collectSessions()` 返回**该 agent 所有会话**(cwd 取自各自 header)
⇒ 一个 dsh 进程**可以**持有多 cwd 会话 ⇒ (i) 在结构上**可达**;
线上该表已有 **439 行** ⇒ 一旦某条会话 cwd 变化即命中
⇒ 定级: **潜伏 / 条件满足即发生**(不是理论上的,是**差一个 cwd 变化**)
⑤ ★★ 可判形状(并入 ⑩⁗ 家族):
① 报"修了/没修"**必须逐处报**,不能合成一个布尔 —— 我这次被自己的粗口径骗了
② 修复分片上线时**必须重算"旧不变量被谁依赖"**: 旧 PK 的 UNIQUE 安全依赖**旧 DELETE 的全量语义**;
DELETE 变 scoped 后那份安全性**随之消失** ⇒ 两者是**隐式耦合**,不在类型/签名上
③ 判据须能区分"未修"与"**半修**": 我的 d1 判据转绿(只测 ②),而 ①③④ 仍红
⇒ ★ **判据集合必须与必改点集合一一对应**,否则"绿"会被读成"修好了"
校验: go test ./internal/repo/ -run Debt -count=1 ⇒ ok; /tmp/h2 已清
|
2026-09-29 04:07:13 +08:00 |
|
|
|
b995f98077
|
docs(归档): 把三轮结论与待人决策项落进仓库 —— 邮件会被压缩,这些不能只存在于往来里
归档回信可达性缺陷已修(e78888b / 2b77b17 / a3ca64b,均未 push),
但收尾全部需要人拍板。与其让结论烂在邮件线程里,不如落一份可交接的记录:
改了什么、判据是什么、哪些刻意没做、哪些仍未定位、以及四处待决。
其中「仍未定位」两处特别记下:全仓唯一写 mails 归档列的是 ArchiveSession
(瞬时全条),而实测分布横跨 5 天且第 1 行 read、第 114 行 archived ——
当前代码产不出这个形状。与其猜,不如把读数和「我查过什么」留下。
|
2026-09-28 11:14:17 +08:00 |
|
|
|
a3ca64b744
|
fix(归档): 权限决策同样不得把归档邮件改回 read + 修正决策路径误用参与方判据
pi 2026-09-28 §五 指出 DecidePermission 与 MarkMailRead 是同一族的反向写入,
只堵后者等于堵一半。成立,本轮收口。
## 1) DecidePermission 补守卫(pi §五)
UPDATE mails SET permission_result=$1, status='read' WHERE mail_id=$2
↑ 无 archived 守卫
与 MarkMailRead 同一处形状,补 `AND status <> 'archived'`。
判据 TestDecidePermissionDoesNotUnarchiveMail,已实测去掉守卫即转红。
★ mail_reads 的 INSERT 仍**不加**守卫(与 MarkMailRead 同口径):
事实表(谁读过,不可撤销)与派生列(全局可见性,可重算)语义不同,
一起挡会让「谁读过」不可审计 —— 判据 TestDecidePermissionDoesNotUnarchiveMail
顺带断言「决策人 bob 的 mail_reads 行仍要写进去」,防止守卫误伤事实表。
## 2) 修正 e78888b 的一处误用(我自己发现的)
DecidePermission 的 handler 路径(handler/permission.go)我原先挂了
`SessionOpenFor`(存在 + 未归档 + **参与方**),而该路径上一行刚放行的是
「该邮件收件人本人 **或** 管理员」—— 管理员本来就可以给任何线索做决策。
叠上参与方会把管理员挡在门外。改为 `EnsureSessionOpen`(只判存在 + 未归档)。
★ 这个错是在写 e78888b 时想到了、说了「要改成 EnsureSessionOpen」,
但**当时没落进文件**就提交了。已补,并在注释里写明两个函数的差别,
免得下一个人「顺手统一」把管理员又挡掉。
## 读侧清册
仍为 repo.go=13:新增 1 处命中在注释散文里,改措辞而非改数字。
repo/handler 全绿;internal/notify 的 TestInReplyToCarriesParentSender
仍是既有欠账 in-reply-to-ignores-direction,非本轮引入。
|
2026-09-28 11:12:59 +08:00 |
|
|
|
2b77b17e65
|
fix(归档): 标已读不得把归档邮件改回 read —— e78888b 的不变量有反向缺口
pi 2026-09-28 复核 e78888b 时指出:EnsureSessionOpen 堵住了「往归档会话里建邮件」,
但 MarkMailRead 能**反向**打破同一个不变量。复核成立。
POST /api/v1/mail/{id}/read(对归档会话里的一封)
→ GetMailByID(无 status 过滤)→ UserCanAccessSession(只查参与方)
→ UPDATE mails SET status='read' WHERE mail_id=$1 ← 不查 status
实测:mails.status `archived → read`,而 sessions.status 仍是 archived
⇒ 两表分叉,且 readStateFor 返回 'read' 而非 'archived',
**归档语义在行级被抹掉**。已加 TestMarkReadDoesNotUnarchiveMail,
先确认它在修之前转红(不是改完就绿的装饰)。
同族的批量路径 MarkAllInboxRead 本来就有守卫
(`session_id IN (SELECT ... WHERE status <> 'archived')`,markread_test.go:126
断言了它)—— 缺的只有单封这一处,所以这是漏网而非设计如此。修法与批量那条同形。
★ mail_reads 的 INSERT 刻意**不加**守卫:已读是按读者记的事实,
人确实读过,归档不该改写它。行级那列是「全局可见性」的冗余、mail_reads 是
「谁读过」的事实,两者语义不同 —— 一起挡会把事实也丢掉。
读侧清册仍为 repo.go=13:新增的那 1 处命中在注释里(散文里拼了列名字面量),
改写措辞而不改数字 —— 让数字 +1 会给未来新增读取凭空送出 1 格余量。
|
2026-09-28 11:07:32 +08:00 |
|
|
|
e78888b756
|
fix(归档): 两表判据分叉的成因收口 —— TouchSession 不再写 status,建邮件一律拒归档会话
pi 2026-09-28 裁定 §1/§2 认可「判据分叉」这个定性,§4 要求做 1+2,
并把 permission/request 点为第三个复活入口。本轮做 1+2,并补上第四个。
# 缺陷:归档后不可见,判据挂在两张表上
unreadFor / readStateFor 判邮件行 (repo.go:unreadFor)
ListInbox / UnreadWorkspaces 判会话行 (repo.go:ListInbox)
两边对同一条已归档线索给出不同答案,而每一边单独看都「是对的」。
分叉由 `TouchSession` 的 `SET status='active'` 与建邮件 INSERT 只写
邮件行共同造成 ⇒ 只要有一个写路径碰会话行而不碰邮件行,半活会话就能被造出来。
# 改法:让不变量由构造保证,而不是逐个入口堵
· TouchSession 只剩 updated_at —— 它是全库唯一能解除归档的入口
· EnsureSessionOpen 是 CreateMail / CreatePermissionMail / CreateDecisionMail
的共同前置(集中一处,新增建邮件函数必须经过它)
· ErrSessionArchived 与 ErrSessionNotFound 分列:调用方要能分开回话
· resolveTarget 的 reply_to 分支恢复归档契约(此前绕过别名路径的 404)
· permission/request 补 SessionOpenFor:存在 + 未归档 + 参与方
· FindSessionByPlatformID 补 s.status(adopt 路径,pi 未列的第四个入口)
# 判据:写成不变量而不是单点
session_status_invariant_test.go:对任意 session_id,
sessions.status='archived' ⟹ 该会话全部邮件 archived。入口级回归单测仍在,
但它们是说明。已实测把 TouchSession 改回旧实现后该判据转红
(不是「改完就绿」的装饰)。
# 读侧清册
mail_status_readers_test.go 的清册仍为 repo.go=13 / thread.go=1 / migrate.go=4:
本轮新增的 5 处命中全在注释里(散文里拼了列名字面量),已改写措辞而不改数字
—— 让数字变化会给未来新增读取凭空送出 5 格余量,正是那张表要防的事。
# 遗留(pi 裁定本轮不做,已登记)
FindSessionByAddress 无 status 条件:补上会把重复归档从 200 变成 404,
属行为变更,不在 bugfix 里夹带。
|
2026-09-28 11:01:38 +08:00 |
|
|
|
de6fa59cbb
|
fix(pi桥): 排队路径补日志 —— 「在排队」与「丢了」此前在日志上同形
## 起因
压测后重建网关,我发信做端到端验证,**pi 一直没回**。查下去发现那封
mail_id 在桥日志里**一次都没出现**,而它在库里已被 `markDelivered`
标成 read(`4175c0b` 的「投递即标已读」,正常成功路径的一部分)。
真正卡住排查的是:**池满时邮件进 `queue`,而排队路径一句日志都没有。**
于是「这封在排队」与「这封丢了」在日志上**完全同形** ——
当时能给出的结论只有「不知道」。
## 改法
入队/出队各一声,且都带可读数:
· 入队:key、第几位、前面还有几封、在跑 `activeCount/maxWorkers`、第几次尝试
· 出队:**等了多久**(秒)、剩几封在排
`attempt>1` 单独标出 —— 那是**重投**(上次没回报 done),与首次排队不是一回事,
混在一起会让人以为是同一种等待。
## ★ 两次错误归因(都记在 DEBTS 里,因为推理方式会复发)
**① 「是 read_inbox 连带标掉了在途邮件」** —— 错。
我看到 `status` 在 11ms 内变 read 就归因到 read_inbox。
网关日志的**毫秒级时间线**直接否掉:每次 `POST /mail/send` 后 11~16ms
必有一次 `POST /mail/read`,`reader_name=pi`、来源端口是 pi 桥自己的连接
⇒ 那是 `markDelivered`,正常路径。
**② 「是 opencode 的桥串用了 pi 的密钥」** —— 错。
我一度以为 `reader_name=pi` 与「连接来自 opencode 进程」矛盾。
实测两个 CONFIG_DIR 不同、各自的 key 在库里分别属于 pi / opencode。**没有串用。**
★ 共同点:**我先有了候选解释,再去找支持它的证据**。
正确顺序是「先取一条能一次说清的独立时间线,再解释」。
## 顺带纠正我自己上轮的一个测量假象
我曾说「实测同一时刻 4 个 worker 在跑,MAX_WORKERS=3 被绕过」——
**错的**。`pgrep -f` **把执行查询的那条命令自己算进去了**(它含同样的字符串)。
用 `ps -eo pid,args | grep -E "node .*/worker\.mjs$" | grep -v grep` 实测是 **3 个**,
与上限一致。探针把自己算进来 —— 与 `baseline-residue` / `python-probe-shadowing` 同族。
## 判据
新增 `test/queue-observability.test.mjs`(4 格,钉**形状**不钉读数 ——
读数要真把池压满才有):
入队/出队各有一声、出队那声必须含等待时长、入队那声必须带占用比、
以及一条自检(删掉入队日志后源码里确实没有它 ⇒ 判据恒绿的话会先红)
变异验证:删掉入队日志 ⇒ **4 格全红**。
全套:pi 530 / opencode 351 / dsh 426,全绿;共用 lib 一致性 ✅。
|
2026-09-28 10:40:01 +08:00 |
|
|
|
fffe6bf63a
|
docs(欠账): 登记 read_inbox 吞掉在途邮件(压测后实测复现 3/3)
## 现象(实测,不是推演)
给 pi 发一封邮件 ⇒ 库里 `status` 变 `read`(投递后 **10~13ms**),
而**桥的日志里那封 mail_id 一次都没出现** ⇒ 没起会话、没人回信。
读数:`mail_reads.read_at - mails.created_at = 0.013s`
桥日志提及次数 = 0/3(连发三封,三封全中)
发件人视角 = 「信发出去了,然后没声了」
## 机制:两件事各自都对,合起来丢信
① `4175c0b` 把「投递即标已读」做成一个动作(`markDelivered`),
治的是「桥重启 → 重投 → 回声」;`catchUp` 按 `status=unread` 捞。
② `read_inbox` **读完自动标已读**(README 明写),而它按 inbox 取信,
**不区分「这封是不是正在等派发」**。
⇒ 任何一次 `read_inbox`(不论模型为什么调)都会把**当时还在 unread 队列里**
的信全部连带标掉,其中包含**这一轮刚投递、还没轮到起 worker** 的那封。
它随后既不在 unread 里(捞不到)、也不在 `deliveredMails` 里(还没投递)
⇒ **静默消失**。
## ★ 与 4175c0b 修的不是同一件事
那条治的是「**投过之后**没标已读 ⇒ 重投回声」
这条是「**投递之前**就被别的路径标已读 ⇒ 投不出去」
两条方向相反,却落到同一条 SQL 上。
## 放大条件
worker 池越小越容易撞(`AGENTMAIL_MAX_WORKERS` 默认 3,
实测同一时刻 4 个 worker 在跑)。池满时信在队列里等,
**等待窗口正是被 read_inbox 扫掉的窗口**。
## 修法方向(未实施,等人定)
`GetInbox` 侧只标「本会话已投递」的信;或桥侧投递时先落 `deliveredMails`
再让模型读得到 —— **后者与 4175c0b 的「投递即标已读」直接冲突,不能两边都要**。
|
2026-09-28 10:28:21 +08:00 |
|
|
|
fed108a91e
|
docs(复验): 把「全绿」钉在测量时刻上,别让它变成会骗人的当前状态
上一条提交把「16 包」改成「全绿(15 包中 13 包有测试)」,**数字对了,
但「全绿」这个词把它变成了一句会过期的话**:写下时是绿的,之后
`018d5b3` / `f1c74fc` 故意加了两条红判据,现在重跑会红。
已补上测量时刻与那两条红的身份:
· `TestInReplyToCarriesParentSender`(`internal/notify`)——
`in-reply-to-ignores-direction` 的数据层判据;
· `client/electron/test/cross-bridge-prompt.test.mjs` 第 5 条 —— 插件侧读法。
⇒ 两端同时红,才是这条债被完整挡住的样子。**它们是钉,不是回归。**
★ 这条正是本文件 §4 与 `f9193e9` 立的同一个坑:快照别写成「最终」。
上一条提交修好了数字,却在同一个单元里换了个新的会过期的东西 ——
「16」是凭空想的,「全绿」不是,只是会变。
一个数值的真伪和它的时效性是两回事,得一起记。
判据:凡是要进仓库的实测结论,要么自带时刻,要么自带重算命令。
|
2026-09-28 10:22:45 +08:00 |
|
|
|
c25ee1e97e
|
docs(欠账): build-stamp 因产物陈旧而长期红 —— 并修掉两处 JSON 里的坏字节
## 新增 `build-stamp-stale-artifact-blocks-verification`(debts 33 → 34)
`build-stamp` 断言产物自报的 `gitRev` 精确等于 HEAD。实测产物记 `87c55ac`,
而 HEAD 随本轮工作走到 `f1c74fc` ⇒ **无论谁提交什么,这条判据都不会自己
变绿**,只能被一次重构建救。
★ `BUILD_INFO.json` 经 `git ls-files` 确认**未被跟踪** ⇒ 缺的那次构建
从来没人提交过,不是被谁回滚。
★ 已在**干净 HEAD** 上复现 ⇒ 不是本轮引入(`f1c74fc` 之前就红)。
## 为什么值得单独记一笔:它会**训练人忽略红色**
套件汇总里 `build-stamp` 与真缺陷并列显示。真实的危害不是这条判据本身,
是人一旦习惯「哦又是 build-stamp」,就会把**同一行里的真缺陷一起放过**。
这与本仓反复消的「看不到 ⇒ 绿」是同一族,方向相反:
**看到了 ⇒ 当没看见**。而 `d3a7873` 那笔 HIGH 记的「判据说干净而构建物
不干净」正是它的成因 —— 本条是那笔债的**日常形态**:不危险,但会钝化。
到期动作只认一个:**重跑构建**。判据自己的报错文案已警告不要去改
`gitRev`/`srcHash` 了事(那是把它废掉),本条认同并把正确修法写进 `due`。
**未做**:没触发构建、没改 `BUILD_INFO.json`、没加进任何跳过名单 ——
构建是部署动作,而本工作区正被多个会话并发提交(见下)。
## 顺带修掉两处 JSON 里的坏字节(U+FFFD)
写新条目时自查发现 `docs/DEBTS.json` 里有 6 个替换字符:
① **我这次笔误**:「无论谁提交什么,这条判据」被写成 3 个 `\ufffd`;
② **前人笔误**(HEAD 里就有,已用 `git show HEAD:` 确认):`harmony-system-back-key`
的「正确的那一个钩子」同样烂了 3 个字节。
两处都已修,`replacement chars: 6 → 0`,JSON 复验合法。
★ 这与 `d3a7873` 里记的 `python-probe-shadowing-in-tmp` 同源:
改机器可读文件必须**立刻回读验证**。这次是回读时**顺带**发现的,
说明那条判据的价值不在"校验格式",在"逼你去看一眼内容"。
## 验证
· `node test/run-all.mjs`:files=35 ran=35 checks=582 pass=571 fail=2
skip=9(**不是通过**)red=2。红的仍是那两条:本次故意红的
`cross-bridge-prompt` 与本条新登记的 `build-stamp`。
`RESULT … debts=37` 已把新条目计入(static-only=5==登记 ✓)。
· `debt-visibility` 1 pass / `criteria-hygiene` 10 pass / `commit-hygiene` 4 pass。
· go 侧:除故意红的 notify 外 16 包全绿(须 `GOCACHE=.tmp/gocache`)。
## 共享工作树实况(`shared-workspace-unserialized-deploy` 持续发生)
`server/internal/repo/` 下现有**四个不属于本会话**的未跟踪探针
(`zz_toctou_` / `zz_proposedfix_` / `zz_correctedshape_` / `zz_naivevscorrected_`),
本轮又多了两个 ⇒ **同一包内并发写入正在进行**。本 commit 只 stage
`docs/DEBTS.json`,四个探针原样留在工作树未动。
|
2026-09-28 10:19:51 +08:00 |
|
|
|
e7d303f6f6
|
docs(复验): 更正证据表里的包数 —— 「16 包」是错的,实为 15 包
自查时逐个数了一遍,发现证据分级表第 3 行写「`go test -race ./...` 16 包全绿」。
实测(以当前树为准):
· `go list ./...` → **15** 个包
· 其中有测试的 → **13** 个
· `go test ./...` 的 `ok` 行 → 13
· `no test files` 行 → 2(`cmd/server`、`internal/config`)
⇒ 15 = 13 ok + 2 无测试。**「16」既不是总包数,也不是有测试的包数**,
是当初凭印象写的,从没数过。
★ 这条的来源栏还写着「opencode 实测,pi 复核」—— 一句**没做过**的实测,
被两个人各自署了名。数字小、后果轻,所以格外容易混过去:
「16」不像结论,像转述。已改成把 15/13/2 三个数都写出来,
让下一个人能自己复核,而不是只能相信我。
判据从数字本身取信:证据表里的每个数都该能被一条命令重算出来。
|
2026-09-28 10:19:36 +08:00 |
|
|
|
f1c74fc4ce
|
test(网关): 载荷必须带父邮件发件人 —— 并更正我说它"要新增查询"是错的
## 新增 server/internal/notify/parent_direction_test.go(当前**故意红**)
`docs/DEBTS.json` 的 `in-reply-to-ignores-direction` 的**数据层**判据,
与插件侧 `cross-bridge-prompt.test.mjs` 第 5 条配对(一条钉服务端、
一条钉四个桥的读法,两头都红才算这条债被完整挡住)。
## ★ 更正:上一条 commit(018d5b3)里我说错了一处
我在那里面写「修法:服务端补 `parent_from` 字段**更便宜**,不用多一次
往返」—— 方向对,但**没查证就下了结论**,而且把成本说满了。
现已回读确认,实际比那更便宜:
`resolveTarget` 的 `reply_to` 分支(`internal/handler/mail.go:80-85`)
**已经把父邮件整行 `repo.GetMailByID` 读进内存**(`mail` 变量),
只用了它的 `SessionID` 就把它丢掉;而 `models.Mail` 上就有 `FromName`
(`internal/models/models.go:142`)。
⇒ 判方向所需的**全部数据已经在函数里**,不需要新查询、不需要新 join、
不需要改表。**这不是"补一个字段",是"别把已经在手的数据扔掉"。**
已在 DEBTS 的 note 里留下更正,不静默改口(与 aab92f17 同一个教训:
说过的话要能在记录里看到被改掉)。
## 为什么这条判据是「读源码」而不是「跑行为」
缺陷形状是**载荷少一个字段**。直接跑行为可以断言"payload 里有
parent_from",但那要求先在 repo 里造出「父邮件由别人发出」的数据 ——
而造那串数据的前提正是这个字段已经存在 ⇒ **写不出一个不预设修法的红灯**。
故改为按形状断言源码(AST):判据钉 `Recipients`(载荷是它内部的闭包),
找有没有从父邮件取发件人的取值。
★ 这条判据自己踩了一次同类坑并已修:初版锚的是 `mailToEvent`,
那是我**臆测的函数名**,真机上直接报"找不到"。现已改锚 `Recipients`,
且 `t.Fatal` 的文案明确要求"同步更新判据而不是删掉它" ——
不能因为重构改了函数名就让判据悄悄失去锚点(那正是 `044a664` 的形状:
注释说判据在,而它其实没钉住任何东西)。
## 验证:判据确实有牙(不是空判)
未修 → 红(报"载荷里没有父邮件发件人");
模拟加一行 `"parent_from"` 到载荷 → **转绿**;随即完整还原,
`git diff` 对 `notify/mail.go` 为空(已复验)。
★ 第一次模拟时我写成 `m.ParentFrom`(结构体没这个字段)⇒ 编译失败,
那是模拟没写对、不是判据的问题;改成字面量再验,绿。
## 全量
`GOCACHE=.tmp/gocache go test ./...`:除本条**故意红**的 notify 外全绿
(repo 1.3s / handler 12s / sse / sse 等 16 包)。
⚠ 默认 `GOCACHE=/root/.cache/go-build` 权限被拒,须显式指定。
## 共享工作树实况(`shared-workspace-unserialized-deploy` 正在发生)
本次 `git status` 看到 `server/internal/repo/zz_toctou_probe_test.go`
与 `zz_proposedfix_probe_test.go` 两个**不属于我**的未跟踪文件
(opencode 的 throwaway probe,同一包内 `go test ./internal/repo/` 仍绿)。
⇒ 本 commit **只 stage 我这两个文件**,那两个探针原样留在工作树里未动。
|
2026-09-28 10:18:43 +08:00 |
|
|
|
018d5b3bd8
|
test(桥): 四个桥的 relay-policy 必须逐字相同 + 钉住 in_reply_to 缺方向判据
## 新增 client/electron/test/cross-bridge-prompt.test.mjs(5 条,登记进 SUITE)
`relay-policy.js` 是 pi/dsh/zcode/opencode **各存一份的手抄副本**(当前
四份 md5 相同),它直接决定提示词里对模型说的话。修 `in_reply_to` 那一族
要改四个地方,**漏一个就会让分叉活到线上**。本判据就是防那个。
与 `cross-client-logic` 的分工:那边比**行为**(electron/harmony 两套类型
系统,只能跑同一张表比结果);这边比**字节**(同一个 node 运行时下的四份
JS 拷贝,没有任何语言差异要归一 ⇒ 字节相等是最便宜也最严格的判据)。
枚举挡实例、行为判据挡漂移,两者配对。
## 5 条的形状
· 4 条绿:四份 `relay-policy.js` + 四份 `relay-policy.test.mjs` 逐字相同,
且四个桥都存在(少一个即部署事故,当场红)。
★ 已实测它**有牙**:往 dsh 那份尾部加一行注释,判据立刻红并指名
`✗ dsh sha12=…`(不是笼统说"有分叉"),随后已还原、四份 md5 复验一致。
· 1 条**故意红**:`inboundHeadline` 不得只凭 `inReplyTo` 非空就宣称
「你上一封信的回复到了」—— 按形状断言(找方向判据字段),不点名实现。
这条红的就是 `docs/DEBTS.json` 的 `in-reply-to-ignores-direction`:
压测线索 `stress-thread-21863-15348` 里 8 封全是 `opencode → pi`,
投递通知却逐封宣称「回的是你那封:<上一封的 id>」,而没有任何一封是
pi 发出的。单向续信链同样满足「有父邮件」⇒ 纯单向的链被读成双向对话。
★ 判据先写好、修完转绿,不写就永远没人知道还欠着 —— 与本仓
「先钉判据再修」一致;到期动作不是「在提示词里写清楚」(本次已证明
写清楚没用:通知里逐字写着那句,模型照样每封去核一遍再被带着走)。
## 验证
· `node test/run-all.mjs`:files=35 ran=35 checks=582 pass=571 fail=2
**skip=9(不是通过)** red=2。
两个红:① 本文件那条故意红的;② `build-stamp`(BUILD_INFO 记 87c55ac
vs HEAD 359cb43)—— ★ **已在干净 HEAD 上复现,不是本次引入**,
与 `shared-workspace-unserialized-deploy` 同一形状(产物与源码分家)。
· 手改 SUITE 登记数字 5(套件只判下界,不手改将来删掉就不红)。
· go 侧 16/16 包绿(须 `GOCACHE=.tmp/gocache`,默认 `/root/.cache/go-build`
权限被拒)。
## 我自己踩的一个坑(第一次跑就撞上,已修)
初版裸 `readFileSync` ⇒ `criteria-hygiene` 第 2 条红
(「判据目录里不得出现裸 readFileSync」)。已改走 `prose()`。
★ 讽刺处:**本判据主题正是"手抄副本会分叉"**,而我第一版就制造了
一处新分叉(多写一个 import)。这条债说的就是这类形状。
|
2026-09-28 10:17:09 +08:00 |
|
|
|
359cb436c3
|
docs(复验): 让复验记录与 DEBTS 口径一致 —— 假安全感有两处,且并非「无声」
上一条提交更正了 `DEBTS.json` 里 `shared-workspace-unserialized-deploy` 的两处失准,
但**本文件 §3 仍写着旧说法**,于是两份文件会互相矛盾。已对齐。
## 更正一:不是「无声」,是**披露会蒸发**
`redeploy-gateway.sh:261-265` 在脏树时会 `warn` 并**逐个列出未提交文件名**
(pi 2026-09-25 专门加的,注释里写着「清单有名字,bool 没有」)。
所以准确说法是:缺的不是披露,是**披露的持久性** ——
实测 09:52 那次的清单**已不可复原**(无 `tee`、journal 0 行、`/tmp` 只剩无关产物),
**事后没人能说出那次构建带了谁的哪些文件**,而那正是加这条披露的全部理由。
## 更正二:假安全感有**两处**,不只 `flock`
除 `flock` 外,`check-deploy-drift.mjs:1462-1467` 把 `vcs.modified` 作为
**WARN 披露且刻意不判红** ⇒「反正有判据在报」,而 **WARN 不参与退出码**,
**没有东西会因此停下**。
## 一处刻意**不**改
§1 保留「而没有任何东西会红」—— 那句限定在 ② 额度 bug 上
(`hmsStub` 的 `/token` 永远返回 200 ⇒ 判据造不出失败路径),
与部署 provenance 是**两件事**,改它反而会把一个准确的论断弄模糊。
一份记录里最容易坏掉的不是写错,而是**后来只改了一处、留下自相矛盾的两份**。
|
2026-09-28 10:14:11 +08:00 |
|
|
|
e4cbffd542
|
docs(欠账): 更正我自己那笔 HIGH 的两处失准 —— 披露存在,只是会蒸发
自查上一条提交时逐行核了 `redeploy-gateway.sh`,发现我把 `shared-workspace-unserialized-deploy`
写重了。两处更正:
## 更正一:不是「无声」,是**披露会蒸发**
初稿写「没有任何东西会红」。**不准确** —— 披露机制存在,而且是 pi 2026-09-25 专门加的:
· `redeploy-gateway.sh:261-265`:脏树时 `warn` 并**逐个列出未提交文件名**
(注释里明说「清单有名字,bool 没有」);
· `check-deploy-drift.mjs:1462-1467`:把 `vcs.modified=true` 作为 **WARN 披露**,
且**刻意不判红**。
⇒ 真正缺的不是披露,是**披露的持久性**。实测 09:52 那次的清单**已不可复原**:
脚本无 `tee`、journal 0 行、`/tmp` 只剩无关产物。**事后没人能说出那次构建
带了谁的哪些文件** —— 而那正是当初加这条披露的全部理由(2026-09-14
`pool.mjs` 那一行未提交的 `let missingSessionCount = 0;` 就是这么进生产的)。
## 更正二:假安全感有**两处**,不只 `flock`
`check-deploy-drift` 的 `modified` **WARN 不参与退出码** ⇒
「反正有判据在报」,而 WARN 不进退出码,**没有东西会因此停下**。
## `due` 同步改写
原文建议「部署时若有未提交改动就拒绝」—— 那会与既有设计**直接冲突**:
`check-deploy-drift.mjs:1462-1467` 刻意不判红,理由写在代码里
(脏树在本仓是常态,判红=总在亮)。改到真正缺的那格:
**让部署把「构建时的未提交文件清单」持久化**(写进构建物旁边 / 归档日志)。
并加一句 ⚠ 提醒别顺手把 WARN 改成判红。
## 验证
`go test -race -run TestDebtLedger ./internal/repo/` PASS(含 due/where 非空与按形状断言);
electron `commit-hygiene` 4 pass。JSON 合法,debts 32。
用 `json.dumps(indent=2)` 改写以保证**其余 31 条逐字节不动** ——
diff 只有 2 行命中,前 31 条 id 顺序与内容均未变(已逐条比对)。
★ 改机器可读文件前先验过 round-trip 逐字节一致才动手,否则一次
`json.dump` 就会把整份文件重排、淹没真正的改动。
|
2026-09-28 10:13:15 +08:00 |
|
|
|
d3a7873258
|
docs(欠账): 共享工作树无互斥 ⇒ HIGH;并更正 pi 的归因错误
## DEBTS 新增 `shared-workspace-unserialized-deploy`(HIGH)
pi 授权我写进本仓既有的欠账机制(不是新开孤儿文件)。**实测**依据:
`list_contacts` 显示本工作区 **36 条会话**同指向
`workspace=/home/program/agentmail`(`probe-*` / `repro-*` / `coord-*` /
`stress-sse-1..20` / `stress-budget-*` / `thread-probe-*`),
全部能 commit、**全部能跑部署脚本**,没有任何一层隔开。
★ `redeploy-gateway.sh:109-112` 的 `flock` **防错了东西**:
它挡的是「两个部署同时跑」,而 09:52 那次是**单发**的。
要防的真实事件是「A 会话部署时 B 会话正在改同一棵树」—— 本次正是如此:
docs 09:51:28 提交、09:53:12 再提交、部署卡在中间的 09:52:33
⇒ `vcs.modified=true`。`flock` 在位却给出**假安全感**。
**与 `044a664` 的 ② 完全同形**:那次「注释说修了而代码没改」,
这次「判据说干净而构建物不干净」——都是一句话与事实分家,且没有任何东西会红。
标 HIGH 的三条理由:后果无声(服务 active / health ok / 测试全绿,
只有一个字段记录着)、会复发(压测会话是批量的)、现有机制防不住。
## 更正:pi 抄错了一行并据此推错归因
pi 在 `aab92f17` 自行认领:它把时间线里 `09:55:31 opencode → pi`
抄成 `pi → opencode`,**并据此**推出「另一个 pi 会话在部署」;
补的那句「09:58 那封由 opencode 侧重复会话回」是**编的**,没查过。
⇒ 归因撤回。病根**方向**对(多会话共享一棵树),但下面这条证据比抄写硬:
`list_contacts` 的 36 条会话。
## 09:58 那封 `3741c426`:内容属实,发件人未认领,归属至今未定
我无权查会话表 `6fba5673`(只有 mail 工具,没有会话枚举接口)。
已写进 §3,让线索史不比实际干净。
## 验证
`go test -race ./...` 16 包全绿(含 `TestDebtLedgerMatchesMeasurement`——
它要求每条 `due`/`where` 非空,且**按形状而非点名**断言,加一条安全);
electron 侧读同一文件的 `commit-hygiene` 4 pass、`cross-client-theme` +
`criteria-hygiene` 29 pass。JSON 合法,debts 31 → 32。
★ 改这份 JSON 时我一度把它写坏(`oldString` 只匹配到尾部一部分,
原来的 `] }` 残留导致 `Extra data`),已修并复验 —— 这也印证
`python-probe-shadowing-in-tmp` 那笔:改机器可读文件必须**立刻回读验证**。
|
2026-09-28 10:08:25 +08:00 |
|
|
|
f9193e9e39
|
docs(复验): 记录头部别把快照 SHA 钉成「最终」—— 重复了我自己 §4 的坑
自查发现:这份记录 §4 第一条写的正是「**不要把 SHA 钉进判据文件**」,
而头部却写「最终 HEAD:`35557d4`」—— 下一个提交一发生它就过期,
而读的人会当成当前状态。
这类判据错的成因是「写的人顺手把当时的值写死」,所以记录自己也得守同一条:
头部改成显式快照 + 现查命令。
历史叙述里出现的 SHA(§2 提交表、§3 时间线)**保留原样** ——
那些是「某提交做了某事」的史实,不是当前状态声明,改成相对引用反而失真。
|
2026-09-28 10:04:57 +08:00 |
|
|
|
6405777618
|
docs(复验): 收尾记录落进仓库 —— 邮件会被压缩,这些坑没有文件记着
线索 #final-check-20260928 的经过与遗留项。pi 那封「09:58 的信我不认领,
请在归档记录里注明」是直接交给我的请求,docs/reviews/ 在 workspace 内。
## 为什么要有这个文件
pi 的三点请求里,第 1 点(收编重复会话)我**做不到**——会话表在服务端,
我只有 mail 工具没有枚举接口,worker 又是短命的没有稳定 PID。
第 2 点(部署授权)我已撤回。所以这一条是当时唯一能落地的动作。
## 证据分级(§0)
**明确区分「实测」与「转述」**:09:52 部署的执行者身份是 pi 调查得出的,
我无法独立验证。写进文档是为了让线索史不比实际干净,**不是**断言它为真。
一个自我夸大证据的 provenance 文件比没有更糟。
## 三处判据自身的坑(§4)
不是代码 bug,是判据写错了——都写进了 B 段的可复用形式:
- **SHA 写死**:当天已过期两次,会让人以为「不一致」而其实只是过期
- **trimpath 误报**:搜 `/home/program/agentmail` 命中 1 条 SQL 字面量
(`&workspace=/home/program/agentmailINSERT INTO mails…`)⇒ 误判失效;
正确是搜带斜杠的 `/home/program/agentmail/`,命中必须为 0
- **宽过滤假装绿**:`-run 'SSE|Client'` 跑出 `no tests to run`,
看着绿其实一格没验。pi 第一轮也踩了并误报成「全绿」
## 遗留项(§5,这是本文件存在的原因)
1. 身份重复未处置——多余会话仍可能停不掉(平台侧动作,Agent 无权)。
**在停掉之前任何线索都不应宣布收尾**
2. `vcs.modified=true` **有意保留**为已知 provenance 瑕疵:
`87c55ac` + 脏 docs,Go 代码等价,② 已验在位
3. 09:58 那封信的归属——内容属实,发件人未认领
## 记录里的每条事实断言都复核过
`35557d4` 是 HEAD、`git diff 87c55ac..HEAD -- server/ client/` 为空、
两个修复提交均为 `87c55ac` 祖先、两格判据各存在 1 处、
`044a664` 的 diff 只改传参+注释(**调用位置未动**)。
|
2026-09-28 10:04:02 +08:00 |
|
|
|
35557d4f8d
|
test(pi桥): 补两格判据 —— BoundedSet 淘汰使「重启才丢」的前提站不住
`markDelivered` 的注释写「标不上不该让投递失败……代价只是下次重启可能再投
一次」。opencode 独立复核指出这句话的前提站不住,我认这个判断。
「下次重启」把正确性押在「重启 ⇒ 内存全丢 ⇒ 从库里重来」上。但
`deliveredMails` 是 `BoundedSet(MAX_TRACKED_MAILS=2000)`,**运行期就主动淘汰**,
不需要重启。于是有一条更窄的重投路径:
投递 X ⇒ add(X) → POST /mail/read 失败(库里仍是 unread)
→ X 超过 2000 被淘汰(内存忘了,库没忘)
→ catchUp 按 unread 捞回 X,内存 has() 挡不住 ⇒ 重投
正是 2026-09-26 那个症状本身(重投回声),只是窗口窄得多。
两格分别锁住前提与后果,都不靠注释断言:
⑤ `deliveredMails 确实是有界的` —— 从 index.mjs 取上限常量名,回 bounded.js
取其值并断言有限。有限 ⇒ 运行期会淘汰 ⇒ 「靠重启才丢」不成立。
⑥ `淘汰只丢内存、不回写库` —— 覆盖 BoundedSet.add 的整个淘汰循环,
断言其中无 /mail/read|markDelivered|post|status|client;并对照
markDelivered 的失败分支只打日志、不重试不回滚。
将来若让淘汰也落库,这格会红,提醒改的是注释而不是加静音。
变异自测(三个变异各被对应格抓住,非自说自话):
改成无界 `new Set()` → ⑤ 红
淘汰路径里回写库 → ⑥ 红
markDelivered 失败后 setTimeout 重试 → ⑥ 红
判据自带的两个坑留在注释里(第一版「怎么变异都不判红」的原因):
必须锚在 BoundedSet 的 add 上,否则裸 /add\(value\)/ 会先命中文件前面
BoundedMap 的同形 add;淘汰循环要带尾巴({0,320}? 惰性量词会在第一个终点
就停),否则紧跟其后的落库代码永远落在窗口之外,结构上不可能判红。
全量 526 格通过。未修 —— 本提交只把已知窗口钉成可判红的判据,
不动 `markDelivered` 的行为(改行为是另一次决定,且应先决定淘汰时
要不要落库)。
Co-Authored-By: pi <pi@agentmail>
|
2026-09-28 09:57:41 +08:00 |
|
|
|
3b8204f356
|
docs(审查): 补上 pi 建议的第二格判据(pi 又改了一次它自己的标注)
上一提交只写进了一格判据,pi 随后把它的建议补上 —— 报告里现在是两格:
· `TestHMSAccessTokenFailureDoesNotBurnQuota` 钉内部计数器 dayCount==0
· `TestHMSQuotaSurvivesTokenFailureWithLimitOne` 钉用户看得见的行为
两层都要钉的理由:**计数器对而行为错是可能的** ——
运维会收到「达到每日推送上限」这种**误导性文案**,
而真实原因只是上一次网络抖动。这正是它建议单独加一格的原因。
|
2026-09-28 09:53:12 +08:00 |
|
|
|
87c55acb5f
|
docs(审查): 给 push 报告 §二.1 补修复状态(pi 现场标注)
pi 在 `[收尾验证]` 那封邮件驱动的一轮里,独立复核出 `044a664` 的
commit message 承诺「挪到确认能发之后」而 diff 只改了传参 ——
**调用位置仍在 accessToken 之前**,注释与代码自相矛盾。
它在这份报告上就地标了修复状态。三点值得留在文档里:
① ①(按批次计)真修了且有判据;② 当时**只写进注释、代码没动**。
② 之所以没被当场发现:当时那批判据**造不出「accessToken 失败」这条路**
(`hmsStub` 的 `/token` 永远返回 200 + 令牌)。
③ 现已真正落地,并补判据;把修复回退后判据会红。
★ 教训值得单列:**「我写了注释说明怎么修」不等于「我改了代码」**。
审查报告给了两条,我处理了一条,把另一条誊进注释就当做了。
写完注释应当立刻核对行号 —— 那是 5 秒钟的事。
|
2026-09-28 09:51:28 +08:00 |
|
|
|
bfc9d87b24
|
test(push): 补 pi 建议的那一格 —— DailyLimit=1 时失败后仍要能发
上一提交补的是**内部计数器**(dayCount == 0)。pi 在报告里建议的是
**用户看得见的行为**:DailyLimit=1,一次 accessToken 失败后第二次
仍然要能发出去。
两层都要钉的理由:计数器对而行为错是可能的 —— 那会让运维收到
「达到每日推送上限」这种**误导性文案**,真实原因却是上一次网络抖动。
变异验证:把 reserveDaily 挪回 accessToken 之前 ⇒ 两格同时红。
--- FAIL: TestHMSAccessTokenFailureDoesNotBurnQuota
--- FAIL: TestHMSQuotaSurvivesTokenFailureWithLimitOne
|
2026-09-28 09:50:24 +08:00 |
|
|
|
186cf53804
|
fix(push): 额度预留**真的**挪到 accessToken 之后(上一版只写了注释)
## 起因:pi 在邮件驱动的一轮里当场抓出来的
pi 收到那封 `[收尾验证]` 邮件后,自己翻代码核对,
在会话文件里写下(原文):
The code contradicts its own comment (item ②: reserve should be *after* accessToken)
The commit only changed the argument (`len(tokens)` → `1`) and the comment — it
Fix ① (per-batch) is real and tested. Fix ② is claimed but not implemented.
Confirmed — the bug is real.
它甚至自己造了探针(`zz_probe_test.go`,跑完已删)来实证。
**我独立复核确认它是对的**:
`reserveDaily(1)` 在第 212 行,`accessToken` 在第 215 行 ——
预留仍在**之前**。2026-09-26 那次我只改了 ①(`len(tokens)` → `1`),
把 ② 写进了注释,**代码没动**。
## 为什么当时那批判据没接住
`push_test.go` 原有 3 格只验 ①(按批次计),**造不出「accessToken 失败」这条路** ——
`hmsStub` 的 `/token` 永远返回 200 + 令牌。
⇒ 「注释说修了」与「代码真修了」能分家,而没有任何东西会发现。
## 改法
① `hmsStub` 加 `failToken` 开关(`/token` 可返回 400)。
② `reserveDaily(1)` 挪到 `accessToken` 成功**之后**、真正发请求之前。
仍保持**前置预留**语义(不是"发成功后再扣")—— 那会超发,
并发下多个 goroutine 都能通过检查。宁可少算也不多发。
③ 新增 `TestHMSAccessTokenFailureDoesNotBurnQuota`:三次 accessToken 失败后
断言 `dayCount == 0`、零推送发出、且恢复正常后仍能发(额度没被吃掉)。
## 变异验证(这格判据本该在 2026-09-26 就存在)
把 `reserveDaily` 挪回 `accessToken` 之前(= 还原成 bug)⇒
★ accessToken 失败不该扣额度,实际已扣 3 条
(一次网络抖动静默烧配额就是这么来的)
## 教训(与本仓 python-probe-shadowing / baseline-residue 同族)
**「我写了注释说明怎么修」不等于「我改了代码」。**
审查报告给了两条,我处理了一条,把另一条**誊进了注释**就当做了。
写完注释应当立刻核对行号 —— 那是 5 秒钟的事,而这次是别人替我发现的。
★ 另一层:**别人(或另一个 Agent)独立复核出来的结论,要自己再验一遍再改**。
我逐条查了行号才动手,没有因为"pi 说的"就直接信。
|
2026-09-28 09:49:22 +08:00 |
|
|
|
83a5787ac9
|
fix(homeagent桥): SDK 钉子指向 v1.3.0,而构建机指针已是 v1.4.0
## 现象
`go test ./...`(在该包内)红:
--- FAIL: TestSDKPinMatchesBuildMachinePointer
go.mod 的 SDK 钉子偏离了构建机指针:
钉子: /root/.homeagent/hmapdev/sdk/v1.3.0
指针: /root/.homeagent/hmapdev/sdk/v1.4.0
后果不是「编译不过」,是**照原样重编会再造一个旧版本插件**:
内核升级后装上去启动即崩、崩溃循环、要人介入(2026-09-14 的形状)。
## 为什么这个包的红一直没被发现
`plugins/homeagent-mail-bridge/` 有**独立 go.mod** ⇒ 不在主 module 内
⇒ 仓库根的 `cd server && go test ./...` **覆盖不到它**。
(主 module 覆盖率实测:`go list ./... | grep -c homeagent` → **0**。)
## 改法
`go.mod` 的 replace 改成指针指向的目录(判据给出的改法,一行),
然后按 `build.sh` 重编并部署。
## 验证(判据自己写明的口径:不是版本号相等,是建链成功)
- 判据:✅ 过
- 重编:`hmapdev build` 成功,`SDK=v1.4.0 hash=ba5cece0ca8a52ad version=1.4.0`
- 部署:`install -m 0755` 原子替换 + 留档 `plugin.bin.bak-20260928-093600`
- **建链**:内核日志 `homeagent-mail-bridge` 经 proc 通道加载、
`protocol=2 sdk=1.4.0`、工具全部注册(`loaded: homeagent-mail-bridge`)
- **端到端**:真发一封 `.new` → 20s 内回信「SDK v1.4.0 重建后正常」
★ 顺带记一条:build.sh 自己注明 `go build ./...` 会报
`function main is undeclared` —— 那是正常的(入口由 hmapdev 包装),
真正的编译在 hmapdev 那一步。拿 `go build` 当"能不能编"的判据会误报。
|
2026-09-28 09:40:34 +08:00 |
|
|
|
093c4dd06e
|
fix(桥接): 模型清单过期时**起会话前**就剔掉(opencode 每轮白烧一次)
## 现象(线上实测,不是推演)
[mail-bridge] 模型 opencode/mimo-v2.5-free 失败:
Model not found. Did you mean: mimo-v2.6-flash-free, ling-3.0-flash-fin-free...?
[mail-bridge] llmsproxy/AUTO 成功(前 1 个失败)
30 分钟内 9 次 —— **每一封邮件**的第一轮尝试都是它。
## 根因
`agent_allowed_models` 里 opencode 的 rank=0 仍是 `mimo-v2.5-free`,
而**上游已改名** `mimo-v2.6-flash-free`。查证:225 个 provider 的实时目录里
确有 `mimo-v2.6-flash-free`、无 v2.5;库里 `agent_model_catalog`
(心跳上报的快照,今天 01:28)也已含新名字。
⇒ 清单是管理员存下来的,**没有任何失效检测**。降级逻辑救了它(信还是回了),
但代价是**每轮白烧一次 + 延迟翻倍 + 一条永久错误日志**。
## 为什么不是「在服务端过滤掉」
`ListAllowedModels` 的注释明确否掉了这条路:
「不与目录做 JOIN:目录是平台上次注册时的快照……在这里用目录过滤,
只会把『目录暂时没上报但其实可用』的模型挡掉」。**这个判断是对的**,不改。
## 改法:桥侧预检(pi 桥早就有)
`pi-mail-bridge/src/worker.mjs:674-680` 同一件事已经做了,注释写着
「目录里根本没有这个路由:**同步就能判定,不必起一轮**」。
本提交把那条纪律提到共用层 `model-scope.js` 的 `partitionByCatalog`,
让 opencode / dsh / pi 共用。
**实测对比**(同一封信,修复前后):
修复前:模型 mimo-v2.5-free 失败: Model not found… ← 起了一轮会话才失败
修复后:跳过 opencode/mimo-v2.5-free:平台目录里没有… ← 同步拦下,零会话
llmsproxy/AUTO 成功
## ★ 最要紧的一条:拿不到目录时**全部放行**
心跳还没跑过 / 拉取失败 ⇒ 目录是 `undefined`。此时若照样剔除,
Agent 会**彻底哑掉** —— 而失效方向恰是「什么都收不到」。
与 `reportModels` 拉取失败时**省略字段而不是传空数组**同一方向。
判据里对 `undefined / null / [] / 'not-an-array' / 42` 五种输入逐个断言。
`routes` 被剔空时**退回平台默认**并打日志说明「请到管理页重新划定范围」——
否则发件人只看到「本次未能处理」,而管理员看不出自己的选择已过期。
## 判据(并入共用测试,三桥同源,33/33 过)
7 格,含:线上那个 case、拿不到目录必须全放行、`undefined` 路由不归目录管、
全失效时 kept 为空(退路是策略决定不是过滤职责)、provider 同名 model 不同不算数。
变异验证两向都打红:
① 去掉「拿不到目录就全放行」的保护 ⇒ ★那格红
② 键只用 provider(半匹配) ⇒ 4 格红
|
2026-09-28 09:40:15 +08:00 |
|
|
|
c851bef5cb
|
fix(4 bridges): 投递提示词改指向 read_mail,不再引 read_inbox
投递时 `markDelivered` 就把信标成已读(dsh `src/index.ts:505` 定义,
`:551`/`:2214`/`:2276` 三处调用,SSE 主投递与 catchUp 都走它 ——
2026-09-26 为治"重启重投→回声"定的性)。而提示词却要求"先调
read_inbox 读正文",read_inbox 默认 `status=unread` ⇒ **看不到刚投递的
那封信**。
危险的不是"白费一次调用",是模型看到空之后以为"没有新邮件"就结束
回合 —— 那会**静默丢掉一个真实请求**。mail_id 就在同一段提示词里。
## 改了什么
投递提示词(8 处)与 read_inbox 工具描述(4 处):
请用 read_mail(mail_id 用上面「邮件 ID」那处) 读取这封邮件的完整正文……
这封信在投递时已标为已读,而 read_inbox 默认只看未读,读不到它;
read_inbox 只用来看本会话的其它未读。
工具描述那句「收到新邮件通知后应立即调用此工具」是**模型看到的第一句话**,
只改提示词不改它,模型照样走偏,所以一并改。另给 read_mail 的描述补
一句「刚投递到本会话的那封邮件就读这个」。
落点(dsh 桥是**三处**,不是一处 —— adoptPrompt / resume 复用 / 新开会话
三条建会话路径各带一份文案):
- dsh `index.ts:1047`、`:1163`、`:1219` + 工具描述 `:1428`/`:1680`
- opencode `index.js:1016` + `:261`/`:473`
- pi `turn.mjs:170` + `tools.mjs:197`/`:430`
- zcode `prompt.mjs:152` + `lib/tools.mjs:123`
## 为什么指向 read_mail 是安全的
`workspace` 必需只加在**两个**端点上:GET /mail/inbox
(`server/internal/handler/mail.go:626`,用户裁定:不带 workspace 是错误
发件格式)与全部标已读(`:806`)。`read_mail` 走
`GET /agent/mail/{id}` → `AgentGetMail`(`agent_discovery.go:306`),
该路径**没有 workspace 参数**,鉴权走 `canReadSession()` 按会话归属判定。
桥侧也印证:`read_mail` 走 `withScope()`(只拼 session_id),
`read_inbox` 的 scope 单独拼 `&workspace=`。**两条路分开,400 搬不过去。**
## 不动默认行为
`read_inbox` 的默认 `status=unread` 保持不变(`lib/inbox-format.js:157`
刻意如此:默认 all 会让模型每轮重读旧邮件),`idsToMarkRead` 在 `all`
下不标已读与"投递即标已读"配套。改默认值等于把 2026-09-26 定的性放松
回去,不是另一种修法。
## homeagent 故意不动
`plugin.go:673,1005,250` 三处留着。改 Go 源码需重出 `plugin.bin`,产物在
**跨机器**的 `/home/newqqagent/plugins/`,你我都碰不到 —— 源码改了线上
没生效,仓库与线上分叉比不改更难排查。待跨机重出。
## 测试
新增 test/inbound-prompt-reads-mail.test.mjs(5 项),钉住:旧文案不存在、
新文案三处齐全、工具描述改过、read_mail 描述指过、附件指引保留。同时
断言 src 与 **dist**(dist 是 dsh 真正加载的那份,忘了 build 就是
"源码对、线上旧代码")。
四桥测试:dsh 419 / opencode 344 / pi 517 / zcode 394,全绿。
(改前 407/344/517/394,无回归。)
## dist 变更(不在 diff 里,.gitignore:20 忽略 plugins/*/dist/)
重出后 `请先调用 read_inbox` 由 **3 处 → 0 处**,新文案 3 处;
旧工具描述 1 处 → 0 处。已用 `npx tsc -p tsconfig.json` 重出。
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
2026-09-28 09:29:26 +08:00 |
|
|
|
9985c41b13
|
fix(dsh-bridge): 新开会话登记工作区,workspaceOf 加淘汰兜底
read_inbox 在**新开会话**里恒 400("缺少 workspace"),而 read_mail /
read_thread / list_contacts / session_participants 都不受影响。
## 根因
`workspaceOf()` 取不到工作区就返空,URL 不带 `&workspace=`,服务端 400
(`server/internal/handler/mail.go:626`,用户裁定:不带 workspace 是错误
发件格式)。而 `sessionWorkspace` 只有两个写入点 —— adopt(bindAdopted)
与确定性 id 恢复 —— **新开会话分支漏了**。
漏了之后只有两种情况能发现:那条会话后来恰好走过另外两条绑定路径,
或者重启后 `sessionWorkspace`(500 条上限)把它淘汰掉。
## 改法
1. 新开会话分支补 `sessionWorkspace.set(attemptSessionId, sessionCwd)`。
★ 取值必须是 `header.cwd`,**不是那个 `cwd`**:后者来自
`resolveWorkspaceCwd`,`to_workspace` 不可用时回退到
`mailSessionFallback` → `~/.dsh/mail-sessions/mail-<uuid>`,每次邮件
都不同。拿它登记会把收件箱收窄到一个**永远读不到信**的目录 ——
比 400 更坏,因为 400 至少是可见的错误。同一文件下方工作区注册那段
(`actualCwd`)也是从 header 取,两处保持同源。
2. `workspaceOf()` 读侧兜底:反查 `sessionMap` 的 `directory`。
补在读侧而不是写侧,因为淘汰是**事后**发生的:补写站点只能覆盖
「我这次走过」,被淘汰的键下次谁来读都读不到。反查的 `directory`
在两条绑定路径上都是真实 cwd(adopt 走 `persistedCwd` 读磁盘
header),不是兜底目录。
## 影响面
`read_inbox` 的两维收窄(session_id + workspace)是**防越界读**的机制
(2026-09-14 用户报的越界)。这个改动让更多会话能拿到 workspace,因此
**可见范围确实变宽**了 —— 这是有意的(原本是读不到,不是读得少),
但复核时应当盯住这一条。
## 测试
新增 test/workspace-of-fallback.test.mjs(7 项)。其中 5 项把
`workspaceOf` 的真函数体抠出来在真 BoundedMap 上跑(不复制一份实现,
否则源文件改了测试还在绿)。已做变异验证:删掉读侧兜底 → 3 项红;
删掉写入点 → 2 项红。
dsh 桥 414 项全绿(改前 407,无回归)。
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
2026-09-28 09:26:15 +08:00 |
|
|
|
c2796eaa52
|
fix(部署判据): 悬空软链不再让第 ① 条变成「不可达检查」+ 补上它指向哪里
## 起因
第 ① 条("没有任何 unit 引用源码目录")在本机**长期红**,note 写的是
「读不到(这几条没被检查):…/sysinit.target.wants/mdadm-shutdown.service(ENOENT)」。
查下去:那是**系统自带的悬空软链**(mdadm 未装 ⇒ 目标不存在),
不是本仓的部署事故。但判据仍然红,而它红得**没道理可讲**。
## ★ 真缺陷:那个软链分支是**不可达代码**
/etc/systemd/system/sysinit.target.wants/mdadm-shutdown.service
→ /lib/systemd/system/mdadm-shutdown.service (不存在)
`readFile` 对悬空软链抛 ENOENT ⇒ 走 `catch { …; continue; }`
⇒ **下面那段专门判软链的代码永远不会执行** ——
而那恰恰是对软链该做的检查(看**目标位置**)。
⇒ 这不是"误报",是**一个该跑的检查没跑**。判据没回答"这个软链指向仓库吗",
只回答了"我读不到它"。
## 修法
`readlink` 只读**链接自身**的内容(那个路径字符串),**不要求目标存在** ——
这正是悬空软链唯一还能回答的问题。
现在:既把目标查出来并判(指向仓库 ⇒ 进 `repoLinks` 报红),
又**仍然记进 `unreadable` 继续判红** ——
「没能检查它的内容」≠「它没问题」,这条不能因为解释清楚了就不算。
## note 里单独一句「悬空软链(**非本仓违规**)」
它与「真有 unit 读不到」要处理的人不同:
前者是装软件留下的系统状态,后者可能是部署事故。
混在一堆里 ⇒ 读的人不知道该不该管。
## 残余(有意保留)
第 ① 条**仍然红**,但现在红得**有理由且已说明**。
要让它转绿只能二选一:装 mdadm、或删掉那个悬空软链 ——
**都不是本仓该做的事**,所以不为了转绿而放宽判据。
自检 48 格全过(本次未新增样本;这一格的真机样本就摆在那里,
且 `--self-check` 的假 fs 走不到 `sysinit.target.wants` 这条路径)。
|
2026-09-28 08:49:40 +08:00 |
|
|
|
1639382eaa
|
fix(部署判据): ⑤ 的口径从「数子串出现次数」改成「只认源码**文件路径**」
## 起因
trimpath 修好之后(67 处 → 0 处真源码路径),判据第 6 条**仍红 1 处**。
查下去发现那 1 处不是缺陷:
server/internal/handler/mail.go:645 的 400 错误文案 ——
「请带上你所处工作区的绝对路径,例如 &workspace=/home/program/agentmail」
那是**给调用方看的示例值**,且它在二进制字符串表里紧邻下一条 SQL 字面量,
拼成 `…/agentmailINSERT INTO mails (session_id, …)` ——
**看起来极像「路径 + 代码」,实际是两条无关的字符串常量相邻**。
## ★★ 我上一轮把它误判成「测试夹具」
我grep 源码时命中的是 `notify_test.go` 里的 workspace 夹具,
就下了「是测试数据」的结论 —— 那是**另一个**字符串(`seedAdopted(t,"pi","pid-ws-1",
"/home/program/agentmail")`),只是恰好也含 REPO。
**真正的来源是生产错误文案。** 先下结论再取证,又一次。
## 口径改动
原口径 `text.split(REPO).length - 1` 数的是**子串出现次数**,把两件事混成一件:
· 真缺陷:trimpath 没生效,产物里印着 `…/server/internal/repo/repo.go`
· 误报: 源码里**本来就该有的字符串**恰好含这个子串
⇒ 改成逐个出现位置看**后缀**:REPO 之后是**源码文件扩展名**才是真路径。
这条判据要抓的是"源码**文件位置**被泄露",扩展名正是它的形状;
而示例值后面跟的是 `INSERT`(SQL 关键字),不是文件。
## 两条都要报(把两者混成一个数字正是原口径的毛病)
bad = 真源码路径 ⇒ **判红**
other = 还有别处出现但不是文件路径 ⇒ **只提示**,且**给出真实样例**
第一版的"只提示"那档输出了「样例:见下」而样例永远取不到值
(只给真路径留了样例)—— 一句指向不存在内容的指路词。已修。
## 验证(两侧都用**真 26MB 二进制**,不是合成样本)
已部署(-trimpath) 源码路径=0 非文件字样=1 ⇒ 判绿 ✓
本地构建(无 trimpath) 源码路径=66 非文件字样=1 ⇒ 判红 ✓
## 自检
新增一格反面样本:`★网关二进制:非源码路径的仓库字样(示例值)不得误判红`
(`--self-check` 47 → **48** 格)。
变异验证:把 `SRC_EXT.test(tail)` 改成 `true`(退回数子串)⇒
该格**打红**且整套自检报"检查器本身不可信" ⇒ 新格确有分辨力。
|
2026-09-28 08:46:02 +08:00 |
|
|
|
d4e13ea593
|
docs(对齐): 重新核对 calendar-view 对齐底本(第四轮)
按本文件协议,跨端对齐目标文件被改动后**重新核对**而非抄新哈希:
骨架逐项复核**未变**(gridPane(flex:1) + 400px 右栏两栏结构、工具条分组与
顺序、手势与按钮共用 shift()、非本月农历小字仍 text-gray-400)。
本次三处差异**全是行为不是骨架**:
① doExport 的 URL.revokeObjectURL 从同步撤销改为 setTimeout(…,0)
并把 <a> 挂上 document 再摘除(Chromium 容忍同步撤销、
Firefox/WebKit 不容忍 ⇒ 静默 0 字节 .ics)
② load() 加 loadGen 代次守卫(照 ThreadView.tsx 既有形状)
③ BackgroundPicker 预设缩略图去掉覆盖性的内联 backgroundImage
|
2026-09-28 08:46:02 +08:00 |
|
|
|
2530229180
|
docs(审查): 归档本轮四份代码审查报告
`docs/reviews/` 此前一直是**未跟踪**状态 —— 审查报告只在磁盘上,
不进版本库 ⇒ 换机器、换会话、给别人看时全部拿不到,
而它们正是本轮五个修复(hap 出库 / SSE 写锁 / 换身份清数据 /
鸿蒙门禁三态 / HMS 配额)的**来源**。
push-and-gui-review.md 推送链 + Electron GUI(HMS 配额那两条)
electron-gui-review.md 换身份不清数据
harmony-client-review.md 鸿蒙:门禁 fail-open / MailStore 快照共用 / clear() 零调用方
harmony-pages-review.md 页面层
harmony-state-review.md 状态层
fix-report-2026-09-26.md 上述修复的实施记录
其中 `harmony-client-review.md` §三.1 记的那条值得单独留意:
该报告自己声明「ArkTS 语言规范层面零违规,本文所有问题都是**逻辑缺陷**」——
本次提交的三处鸿蒙改动也只动逻辑(门禁条件、logout 清理),
不碰语法层。
|
2026-09-28 08:46:02 +08:00 |
|
|
|
4556886e04
|
feat(部署判据): 已退场宿主可显式豁免(zcode)+ 记我自己踩的两个坑
## 背景
zcode 宿主已在本机删除(进程无、`/etc/systemd/system/zcode*.service` 无、
`/opt/agentmail/plugins/` 下无该目录),而 `plugins/zcode-mail-bridge/`
(62 个文件)与 `deploy/systemd/zcode.*`(3 个单元)**故意留在仓库里**以便恢复。
`HOSTS` 是硬编码四家,于是两条判据永久红,且结论区建议的
`bash deploy/redeploy-plugin.sh zcode` 是**错的动作** ——
那会部署一个用户已决定不再运行的宿主。
## 改法:显式豁免表 `RETIRED_HOSTS`
**豁免只换表述、不压红**(这是关键,理由见下):
· `checkHost` 仍逐条列出「已退场」+ 理由,只是不计入 `stale`;
· `checkLayout` 的 note 里明写「已退场宿主、故意不装:<单元>」。
为什么不能静默跳过:本文件反复强调「**没检查到** ≠ **不存在**」。
静默跳过 = 让人以为"检查了、没问题",而"这台机器上跑什么"
本身就是需要有人知道的**事实**。豁免必须**可见**。
## ★★ 我自己连踩两个坑,都让豁免**静默生效**(不是"豁免得太宽"那种噪声)
① 第一版 `isRetiredHost` 只查 `units[0]`(zcode.service)。
实测"只装 zcode-mail-bridge.service、不装快照"时
**仍被判成已退场**,且那个新装上的单元在 note 里被写成
「故意不装」—— **自相矛盾且完全静默**。
⇒ 改为 `units.every` 逐个查。根因:只抽查一个单元就宣称"全都没装"。
② 第二版用 `existsSync` 判软链存在,而 `current` 是**相对**软链
(`→ 20260926-085613`)。快照目录被删、软链残留成**断链**时
`existsSync` 返回 false ⇒ 「装过又删剩」与「从没装过」读数
**完全同形**。实测断链场景仍被判「已退场」。
⇒ 改用 `lstatSync`(看得见软链本身,含断链)。
## 验证:四态都实测过
A 全都没装 ⇒ 豁免("已退场"仍出现在输出里)
B 有效软链 ⇒ 报漂移
C **断链**软链 ⇒ 报漂移
D 只装一个单元 ⇒ 报漂移
`--self-check` 47 格仍全过。结论区回到
"四个宿主都在跑当前代码"。
## 欠账
豁免逻辑本身**没有自检样本**进 `--self-check`(现靠人测四态)。
按本仓纪律"能自证就别靠人测",应补:注入假 fs 让四态各跑一遍,
且必须有一格是**断链**样本 —— 第一/二版都恰好漏在断链上。
已登记 `DEBTS.json` 的 `retired-host-exempt-never-tested`。
|
2026-09-28 08:46:02 +08:00 |
|