[code-review] swarm ShutdownTeammate deletes m.results, contradicting #1814 get-result-after-shutdown semantics kept by emit path

[topcheer]

文件: internal/swarm/manager.go L533-535（delete(m.results, tmID)）vs L719-735（GetTeammateResult，#1814 修复）与 L797-799（emit 注释）

问题描述
ShutdownTeammate 无条件 delete(m.results, tmID)。但 #1814 修复专门让 stored-result 在 teammate 移除后仍可取（GetTeammateResult 把 stored-result 检查移到存在性门禁之前，注释明说"先 shutdown 后取结果"是最常见顺序）；emit teammate_shutdown case 注释 "Keep the last result available even after shutdown" 与该 delete 直接矛盾。#2121 guard（teammateGoverned）使 shutdown 后无任何路径再写回——delete 是终态性数据丢失。两个 issue 修复互相打架（#1633 修 quota 泄漏时顺带删掉了 #1814 刻意保留的数据）。

触发场景
leader 按 #1814 描述的顺序：teammate 完成任务（结果已入 m.results）→ ShutdownTeammate 释放配额槽 → GetTeammateResult → 返回 ("", false)，最终产出永久丢失。

修复建议
ShutdownTeammate 保留 m.results[tmID]（与 emit 注释一致）；results 条目由 team 生命周期（DeleteTeam/GetTeamResults）或容量上限统一回收。

严重程度: medium（明确现实序列的数据丢失）

来源: cron-1 深度审查，初审 sa-133 + 独立复核 sa-134 双重确认 2>&1 | tail -1 && gh issue create --title "[code-review] swarm SendToTeammate/GetTeamResults use team pointer after releasing m.mu: TOCTOU vs DeleteTeam/ShutdownTeammate silently drops messages" --body "文件: internal/swarm/manager.go L609-621（SendToTeammate）、L740-747（GetTeamResults）

问题描述
m.mu 下取 team 指针后解锁再操作。与 DeleteTeam/ShutdownTeammate 并发时：消息可投进已被 cancel 的 teammate Inbox，runner 已退出不消费，SendToTeammate 返回 nil 静默成功。对比 ShutdownTeammate 自身（L520-524 在 tm.mu 下同步 cancel+移除）与 #2121 对 SpawnTeammate 的同类修复，修复模式已存在而 Send 路径未用。

触发场景
leader 并发 SendToTeammate(team, tm, msg) 与 DeleteTeam(team)：Send 在 L616 解锁后被调度让出；DeleteTeam 完成移除+cancel；Send 恢复后消息投递成功返回 nil，实际无人消费。

修复建议
投递操作纳入 team 生命周期临界区（team.mu 下校验 teammate 存活 + 投递原子化），或对已 cancel 的 teammate inbox 投递返回错误。

严重程度: medium（并发窗口窄，但后果为静默消息丢失 + 假送达）

来源: cron-1 深度审查，初审 sa-133 + 独立复核 sa-134 双重确认（候选 3 metadata alias 经复核否定未立案）"

[topcheer]

已复核并修复（分支 fix/issue-2787，待 PR 窗口）。

根因：#1633 修 quota 泄漏时在 ShutdownTeammate 无条件 delete(m.results, tmID)，与 #1814 专门保留的"先关后取"语义（GetTeammateResult 把 stored-result 检查提到存在性门禁之前，注释明载此为最常见顺序）及 emit teammate_shutdown 分支注释 "Keep the last result available even after shutdown" 直接矛盾——#2121 late-emit guard 使 shutdown 后无任何路径写回，delete 即终态数据丢失。既有 #1814 钉死测试（TestGetTeammateResultAfterShutdown）正是靠手动绕过 ShutdownTeammate 建模"关后态"才得以成立，侧面印证真实路径矛盾。

改动：ShutdownTeammate 不再删本 teammate 的 result；改为有界保留——同临界区顺带修剪"teammate 已不属于任何活 team"的陈旧条目（此前多次 shutdown 周期从未被收集的残留），#1633 泄漏保持修复（映射上界=活 teammates+刚关的 1 条），DeleteTeam 两阶段全量回收不变。锁序 m.mu→team.mu.RLock 与 DeleteTeam 同序；字段声明与 emit 两处过时注释同步更新。

验证：新增 zz_issue2787_test.go 双探针（真实 ShutdownTeammate 路径后 GetTeammateResult 必须取到结果——修复前 ("",false) FAIL 钉根因；陈旧 ghost 条目被修剪+新关 teammate 结果保留）。internal/swarm 全包测试绿 + -race 定向（ShutdownTeammate/GetTeammateResult/DeleteTeam 族）绿 + build/gofmt 净。