Skip to content

Track PR#102 leftover items after merge #111

Description

@zts212653

概要

PR #102 已合入。当前仍有按 2026-02-27 二次 review 确认的剩余项,先记录后续收敛。

Blocking (P0/P1)

  • [P0] scope=session 在 dedup 场景下规则绑定错误

    • 证据链:approval 去重按 fingerprint 复用 pending,当前 grant/deny 的 session 规则仍绑定 request.session_id
    • 现象:A/B 两个 session 触发同 fingerprint,B 执行 grant(scope=session) 会把规则落到 A
    • 修复目标:引入 actor_session_id,scope=session 绑定执行动作来源 session
    • 回归:并发 A/B 同 fingerprint,B 授权后规则绑定 B
  • [P1] AFTER_TOOL.approved 语义污染

    • 现象:审批系统错误(fail)仍可被当成 approved
    • 证据链:网关将 deny 与 infra error 分开为不同 status;agent 侧仍以 status != not_allow 推导
    • 修复目标:由明确决策/错误字段驱动 approved,不再解析字符串状态做布尔映射
  • [P1] Transport 合同漂移(文档/代码/运行行为不一致)

    • 现象:文档仍宣称有 approval.resolved,但运行链路无发送 resolved 的落盘/广播;代码仍保留 resolved 分支与 helper
    • 修复目标:二选一且全链路一致:
      1. pending-only:删除 resolved 文档/类型/helper/adapter 分支;或
      2. pending+resolved:恢复并稳定发送 resolved

建议项

  • [P2] examples/05、06 的执行路径未传 transport,pending 无实时推送到 CLI
  • [P2] approvals action 响应 event_type 校验过宽,未知类型应 fail-fast

本次剩余修复分支

  • 已完成:在 approval_manager 中剥离 dare_runtime_context 与 RuntimeContextOverride 对去重、命令提取和 pending 持久化的影响
  • 对应提交:730ed39b
  • 新增回归测试:test_tool_approval_manager.py::test_deduplicate_ignores_runtime_context_internal_param

说明

本 issue 仅用作跟踪;各阻塞项按合并前后最小闭环分解推进。

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions