[Priority: Low] [Reliability] 评估 SessionManager 空闲 session lock 的清理策略

[liujuanjuan1984]

背景

#312 原始提出的“内存活跃 Session 实例缺乏上限控制”在当前主干上已不成立：

• SessionManager 已通过 SessionStateRepository 管理 session binding / owner state
• memory repository 中的 sessions 与 session_owners 已采用带 ttl + maxsize 的 _TTLCache
• 已有测试验证 owner cache 为有界缓存

因此，不需要再引入新的 A2A_MAX_ACTIVE_SESSIONS_IN_MEMORY 或额外的 LRU 驱逐层。

剩余问题

当前更窄但仍值得评估的残余是：SessionManager._session_locks 按 session_id 懒创建后，缺少显式清理路径。若服务长时间运行且遇到大量不同 session id，lock 字典可能持续累计。

这个问题与 session binding / owner cache 是否有界不同，应单独跟踪，不与 #312 混在一起。

建议方向

1. 审查 session lock 的生命周期是否需要显式回收。
2. 若需要，采用最小机制清理空闲 lock，避免引入新的全局配置复杂度。
3. 保持现有 session binding / owner repository 语义不变，不重复设计已有 TTL/maxsize 边界。

回归测试建议

• 构造大量不同 session_id 的锁获取路径，验证清理策略不会破坏并发串行语义。
• 验证长期运行后空闲 lock 不会无界累计。

Related #312

[liujuanjuan1984]

基于当前主干（已包含 #343 等后续改动）重新评估，结论如下。

1. 需求是否仍合理 / 是否仍有效

• #326 仍然合理，且问题依然存在。
• 当前 SessionManager._session_locks 仍然是按 session_id 懒创建后放入普通 dict[str, asyncio.Lock]，只有创建路径，没有回收路径。
• 这与 #312 已经关闭的“session binding / owner cache 需要 LRU 驱逐”不是同一个问题；后者已经通过 SessionStateRepository + _TTLCache 收敛，而 lock 字典仍可能在长生命周期进程中随不同 session_id 累积。

2. 当前 issue 中的实施方向是否符合最新最佳实践

• 大方向仍对，但实现建议需要收敛。
• 我不建议再加新的全局配置、TTL 或显式 sweep 任务；这会把一个很窄的对象生命周期问题升级成新的配置面。
• 更符合当前代码的最小方案是：把 _session_locks 从普通 dict 收敛为 weakref.WeakValueDictionary[str, asyncio.Lock]。
• 我已在当前环境验证 asyncio.Lock 可被弱引用，因此这个方案可行。
• 这个方案的好处是：
  • 不改变现有 get_session_lock() 的对外语义。
  • 不需要依赖私有 waiter 字段或手写 sweep。
  • 当没有执行路径再持有该 lock 时，字典项可自然回收。
  • 与当前“最小机制、避免新配置复杂度”的 issue 目标一致。

3. 建议补充到实施方案中的实现要点

• 变更文件主要是：
  • src/opencode_a2a/execution/session_manager.py
  • tests/execution/test_cancellation.py
  • 建议新增专门针对 lock 生命周期的测试文件，例如 tests/execution/test_session_lock_lifecycle.py
• 建议测试覆盖：
  • 相同 session_id 仍复用同一把 lock，不破坏串行语义。
  • 请求结束后，空闲 lock 不再被 manager 强持有。
  • 取消路径里预先注入 lock 的回归用例仍成立。

4. 是否有高度相关的 open issues 建议一并开发

• 当前没有建议并入同分支一起做的 open issue。
• #333 与 session lifecycle 相邻，但范围明显更大，不建议与 #326 合并实现。
• #312 是已关闭背景 issue，可在 PR/提交说明里引用，但不应作为并行开发项。

[liujuanjuan1984]

已在当前分支完成 #326 实现，并通过全量回归。

本次实现：

• 将 SessionManager._session_locks 从普通 dict 收敛为 weakref.WeakValueDictionary[str, asyncio.Lock]
• 保持 get_session_lock() 的现有复用语义不变，但不再由 manager 对空闲 lock 施加强引用
• 新增 tests/execution/test_session_lock_lifecycle.py，覆盖：
  • 同一 session_id 的活跃 lock 仍然复用
  • 空闲 lock 在无外部强引用后可自动回收

验证：

• ./scripts/doctor.sh
• 结果：439 个测试通过，coverage 91.48%

分支与提交：

• Branch: issue-326-session-lock-lifecycle
• Commit: 406284f use weak session lock registry