[devin-ai-integration]

Summary

深度测试（soak 三次崩溃复现）发现的持久化层缺陷修复：state.json 既当「索引」又当「数据」，且 flush_state 的快照语义错误。

• S1 竞态：flush_state 原实现要求 await 后 _pending_state 全局为空，并发 save() 落在 await 之后会虚假抛 RuntimeError 拖垮整个 Registry。改为三代际计数（_save_gen/_pending_gen/_flushed_gen），只等调用时刻已发起的 save 落盘。
• 降级语义：flush_state 永不 raise——最多 3 次尝试后置 _state_error + warning，快照累积性保证下次 save() 自动重试（H-08）。
• S2 负载：_state_task_row 瘦身（message≤2000、result_text≤6000，全文仍在 result/transcript 工件）；RESIDENT_SESSION_KEEP=50 按 last_active 强卸最老空闲常驻会话——soak 实测 idle_unload_sec=0 下 ~970 会话 → ~20GB 子进程 RSS 是 OOM 主因。
• H-01：request_id 幂等绑定持久化到 state.json requests 键（sha256 指纹、上限 1000、owner_pid 戳使 _merge_owned 可删自身行），重启后重放返回 reused: true。
• B7：wait_task 终态返回前 flush_state()；返回体新增 persisted: bool 告知落盘是否真实完成。

测试：+8 用例，全套 630 passed / 1 skipped；repro_flush_race（bursty save 下原代码 3s 必中）现 0/76000。

Link to Devin session: https://app.devin.ai/sessions/b270264aa3634bc5934cad9bcec9f9ae
Open in Devin Desktop: https://app.devin.ai/desktop/session/b270264aa3634bc5934cad9bcec9f9ae?variant=devin
Requested by: @FeiZhuLulu

---

[devin-ai-integration]

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

• Disable automatic comment, CI, and merge conflict monitoring

[devin-ai-integration]

Devin Review found 1 new potential issue.

3 flags not posted on this PR by your GitHub settings — view them in Devin Review. (Configure)

[cursor]

代际 flush_state 与 request_id 持久化整体方向正确；发现 1 个高置信问题：wait_task 的 persisted 会在终态 save() 之前误报 True。

  

_{Sent by Cursor Automation: GitHub PR 自动审核}

[cursor]

严重程度：高

问题： persisted 只保证「当前 _save_gen 已落盘」，并不保证「这份终态快照」已落盘。wait_task 在 task.status 已是终态时会跳过 _done.wait()，立刻 flush_state()。而 _run_task 是先改 status，再 await（finally 里的 await watch、merge_files_changed；stall 路径里更是先标 failed 再 await adapter.cancel），最后才 _done.set() + save()。窗口内上一档 running 快照往往已经 flushed，于是 persisted=True 配上内存里的终态。

触发条件： 协调端按文档短轮询 wait_task（例如 Cursor timeout_sec≈30）。上一轮 event.wait() 超时返回时，任务已在 stall/cancel/收尾中变成终态但尚未 save()。下一轮看到终态、跳过 wait、把旧代际当成已持久化。stall 取消可维持到 STALL_CANCEL_GRACE_SEC（15s），窗口足够跨过一次短轮询。

影响： 调用方以为终态已在 state.json。若此时进程退出，重启会把磁盘上的 running 收成 failed/bridge_restarted，stop_reason、error 与刚返回的 stalled/cancelled 不一致；request_id 重放也会绑到这条被改写的记录，而不是调用方以为已落盘的终态。

建议改法： 以 _done 为准，而不是内存 status：event 未 set 就继续 wait（超时仍返回 timed_out）。_run_task 里 save() 与 _done.set() 之间没有 await，等到 event 后再 flush_state() 才对应终态代际。更稳的是把终态 save() 挪到任何收尾 await 之前，并让 flush_state 等到那一档 _save_gen。