jianf/MailUI4Agents
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(prune)!: 构建暂存那段清理**一直是空转的**（ls -1t 对目录打 路径: 头）+ del() 失败分支补齐覆盖

Browse Source 
## 那个真 bug：`ls -1t <多个目录>` 不打裸名字

追 pi 的 shim 建议时撞出来的，与 shim 无关 —— 是我为了给它造样本才发现的：

    ls -1t /tmp/agentmail-gateway-build-*     # 这些是**目录**（mktemp -d 造的）
    /tmp/…-20260101-000000:
    agentmail-gateway
    /tmp/…-20260105-000000:
    agentmail-gateway

`ls -1t` 收到**多个目录参数**时会列出**每个目录的内容**并打 `路径:` 头 ——
于是 `mapfile` 拿到的全是 `…000000:` / `agentmail-gateway` 这类行，都不是文件名
⇒ `basename | grep -oE '[0-9]{8}-[0-9]{6}'` 取不到时间戳 ⇒ 每条都判
"文件名无时间戳，判定不了" ⇒ **一个都不删**。
也就是说本段注释里写的那个问题（"每次部署留下一个 24MB"）**从来没被清理过**。
修法：`ls -1dt`（`-d` 让目录自身作为条目，不打头）。

**为什么一直没人发现**：自检夹具用 `: >` 造的是**普通文件**，而生产是**有内容的目录**。
夹具形状与生产不一致 ⇒ 夹具自己认了错形状，而判据 197 行又只按**文件名**判
"窗口内的还在、窗口外的不在"，于是判据也认了。这是 docs 第 13 条那一族。
→ 夹具已改成真目录 + 里面放 `agentmail-gateway`；**改完立刻变红**（"干净样本：/tmp 构建暂存
只留窗口内那 1 份"失败），证明夹具现在真的有分辨力，然后加 `-d` 转绿。

顺带确认：其余三处 `ls -1t`（`agentmail-gateway.bak-*`、`pre-deploy-*.db`、`pre-prune-*.db`）
glob 到的是**文件**，不受影响；插件快照那处（第 294 行）本来就已经写了 `-1dt`。
生产现场实测：`/tmp` 下确实还躺着 1 份 24MB 暂存没被收掉。

## `del()` 的失败分支：采纳 pi 的"让 rm 自己失败"

他指出的第三条路（我原先只想到 immutable 与注入点）是对的：本机以 root 跑、权限拦不住；
`unshare -r` 被拒（`/proc/self/uid_map: Permission denied`）；tmpfs 无 `chattr +i`。
**改机器的权限**不如**让 rm 失败**。

实现上走了 `RM="${RM:-rm}"` 而不是 PATH shim，理由：`in_use` 会把命令行里含该路径的进程
判成"在用"，而自检必须把路径写在命令行上 —— 实测评 PATH shim 时确实被 `in_use` 挡掉、
`del()` 根本没被调用（那次"测试通过"是假的）。`$RM` 默认就是 `rm`，生产行为逐字不变。

自检新增 4 条（并通过变异确认有区分力：去掉失败判定 ⇒ 强断言变红）：
退出码 2、必须打出 `[FAIL] 删不掉`、不许出现收尾汇总、那条路径必须还在。

★ 变异还暴露出一条**弱断言**：单看"退出码 = 2"在变异后**照样通过**（脚本别处也有退 2 的路径）
—— 已在注释里注明它弱、区分力来自另两条，没有让它冒充证据。

★ 顺手修掉一处自指的措辞：我原先在报错里抄了收尾汇总的原话（"已删除 N 项"），
于是 `grep -c '已删除'` 命中**这句报错自己** ⇒ "有没有虚报成功"这个检查把自己的措辞
当成了证据。改写成不含该字面量的说法（与 `grep -c 用例名` 是同一族：判据锚在元文本上）。

## pi 的另两点

· `diffSummary` 带 `ctx` 时**会读文件**（判 `scripts.test` 要读两侧原文），不带是纯内存比较
  —— 已写进函数头，免得以后有人当纯函数用而在大树上意外吃到 I/O。
· "自检样本不独立"的三种形态（位置选择器 / 共享夹具状态泄漏 / 探针无分辨力）
  **合成 docs 第 13 条**（修法同一个：显式命名 + 显式复位），并把上面"夹具形状必须与生产
  一致"作为配套一条写进同一条 —— 今天的真 bug 正是它。

验证：prune 自检 22/22、干跑 exit 0；npm test exit 0；drift 自检 35/0；check-shared-libs exit 0。
This commit is contained in:
JianFeeeee
2026-09-14 20:53:54 +08:00
parent 7eec311756
commit 743e397916
3 changed files with 99 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
docs/DEV-TOOLING.md
View File

@ -201,13 +201,20 @@ bash deploy/prune-deploy-artifacts.sh --self-check # 判据自检（16 项，
`set -e` ⇒ **脚本当场中止**：只报第一个分叉文件，后续对象与收尾汇总都不打印。
退出码**恰好还是 1**（判定是对的），所以光量退出码看不见它 ——
这正是"判定对、证据被截断"。修法 `|| true`，并在注释里写明它不是风格而是判据。
13. **锚点必须与结论一一对应 —— 连"文件名"都会骗你。** 实例（2026-09-14）：
我写了个探针判"通知脚本内容与仓库不一致 ⇒ 必须红"，探针按**文件名**取文件
（`p.endsWith('service-failure-notify.mjs')`），可两个路径的 basename **相同**
（`deploy/service-failure-notify.mjs` vs `/opt/agentmail/bin/service-failure-notify.mjs`）
⇒ 两侧读到同一个串、探针**自己没分辨力**，样本"通过"得毫无意义。
与 `grep -c 用例名` 那个假数、与"名字出现在 `# Subtest:` 头"是同一族但不同形：
**锚在"名字"而没有锚在"哪一侧"**。
13. **自检样本必须"独立"：每条样本都要说清它锚在哪一条检查、用的是哪一棵树。**
这一族我 2026-09-14 一天里踩了三次，合起来记比拆成三条好，因为**修法是同一个**
（显式命名 + 显式复位）—— 三者都是"断言的语义被当前状态悄悄改掉"：
| 形态 | 实例 |
|---|---|
| **位置选择器** | 自检里 `bad[0]` / `badBak[0]`：前面插一条新检查之后，锚点指的就不是原来那条了 |
| **共享夹具状态泄漏** | 新补的四条豁免样本第一版**全红**：它们继承了两棵树里既有的改动（`extra.mjs`、只存在于 `a` 的 `test/`），红的理由根本不是要测的那件事 |
| **探针自身没有分辨力** | E 那次的探针按**文件名**判两侧，而两条路径 basename 相同（`deploy/service-failure-notify.mjs` vs `/opt/agentmail/bin/…`）⇒ 两侧读到同一个串、样本"通过"得毫无意义。与 `grep -c 用例名` 那个假数、"名字出现在 `# Subtest:` 头"同族不同形：**锚在"名字"而没有锚在"哪一侧"** |
配套一条（同一天踩到）：**自检的夹具形状必须与生产形状一致**，否则夹具会把真 bug 藏住 ——
`prune-deploy-artifacts.sh` 的构建暂存自检用 `: >` 造**普通文件**，而生产是
**有内容的目录**；正是这个差异让"`ls -1t` 对多个目录打 `路径:` 头 ⇒ 那段清理一直空转"
这个 bug 藏了很久（判据自己在用文件名判"删了没有"，夹具认了错形状，于是它也认了）。
14. **退出码也有量纲。** `node deploy/check-deploy-drift.mjs --self-check` 的退出码
**不是**"自检的结论"：自检本体 28/28 全过，但同一个进程接着跑了宿主判据、
于是整体 exit 1。报"自检失败"就是把两个量纲混成一个。