90 lines
4.7 KiB
Markdown
90 lines
4.7 KiB
Markdown
|
# 第 10 轮工作计划（轻量合并版）：审批委托（delegation）
|
|||
|
|
|
|||
|
|
更新日期：2026-08-01
|
|||
|
|
|
|||
|
|
## 1. 本轮目标
|
|||
|
|
|
|||
|
|
新增审批委托：发起人在出卡时可指定被委托人（`delegate_user_id`），该确认卡的审批作用域允许 owner 或被委托人操作（P2 单重审批）；委托不改变 P3 双人职责分离语义。矩阵 P3「外部副作用二次确认」剩余项之一（审批委托）完成。
|
|||
|
|
|
|||
|
|
## 2. 背景和当前状态
|
|||
|
|
|
|||
|
|
- 当前已完成：审批意见（note）、批量审批、证据链强校验、双后端（文件/DB）持久化 record payload。
|
|||
|
|
- 当前缺口：`_matches_current_scope` 只认 owner（`ownerUserId == identity.user_id`），无委托能力；`stage_confirmation` 无 delegate 参数。
|
|||
|
|
- 本轮为什么现在做：矩阵 P3 剩余「委托/转派审批」的首项；纯本地可验证，双后端 record 以 payload 整存（新字段自然持久化）。
|
|||
|
|
- Workspace preflight：第 9 轮收口（462 passed）；服务 8003/5173 正常；共享脏工作区、无提交。
|
|||
|
|
- 方向分析：Q1 推荐选项 A（审批委托/转派-委托首切片），目标续跑轮授权执行。
|
|||
|
|
|
|||
|
|
## 3. 本轮工作方向
|
|||
|
|
|
|||
|
|
```text
|
|||
|
|
_matches_current_scope 只认 owner
|
|||
|
|
-> stage_confirmation 增加 delegate_user_id（写入 record.delegateUserId）
|
|||
|
|
-> _matches_current_scope：P2 允许 owner 或被委托人审批；P3 保持双人分离
|
|||
|
|
-> 黄金测试：delegate 可审批、非 delegate 不可、owner 仍可、P3 不受影响
|
|||
|
|
-> 矩阵 P3 注记「审批委托已完成」；CHANGELOG
|
|||
|
|
```
|
|||
|
|
|
|||
|
|
## 4. 已确认决策
|
|||
|
|
|
|||
|
|
任务重量：
|
|||
|
|
- 档位：轻型（harness + approval_store 作用域逻辑 + 测试）。
|
|||
|
|
- 规模依据：改动集中在 `stage_confirmation` 与 `_matches_current_scope`；双后端 record payload 整存无需迁移；风险 LOW。
|
|||
|
|
- 选择原因：P3 剩余项中最小的独立增量。
|
|||
|
|
|
|||
|
|
P0/P1 决策（按 Q1 推荐采纳）：
|
|||
|
|
- 决策 1（P0）：本轮切片 = 审批委托（选项 A 首项）。
|
|||
|
|
- 决策 2（P1）：不新建分支、不提交、不推送。
|
|||
|
|
- 决策 3（P1）：验证深度 = 聚焦委托测试 + 全量黄金回归。
|
|||
|
|
|
|||
|
|
默认假设：
|
|||
|
|
- 假设 1：委托仅对 P2 单重审批生效（P3 首次/二次批准仍强制双人分离，delegate 视为独立用户）。
|
|||
|
|
- 假设 2：delegate 不改变角色策略（仍需具备审批角色），只扩展作用域匹配。
|
|||
|
|
- 假设 3：`delegateUserId` 存入 record；双后端（文件 JSON / DB payload JSON 列）天然持久化，无需 schema 变更。
|
|||
|
|
- 假设 4：委托信息进入审批历史（history_event 保留 record 字段）。
|
|||
|
|
|
|||
|
|
未决但不阻塞：转派（transfer，将待办转移他人）、批量委托、委托撤销（后续轮）。
|
|||
|
|
|
|||
|
|
## 5. 范围
|
|||
|
|
|
|||
|
|
In scope:
|
|||
|
|
- `server/agent_core/harness.py`：`stage_confirmation` 增加 `delegate_user_id: int | None = None` 参数 → `record["delegateUserId"]`；`_matches_current_scope` P2 分支允许 `identity.user_id in (ownerUserId, delegateUserId)`。
|
|||
|
|
- `server/agent_core/approval_store.py` / `approval_db_store.py`：无需改动（record 整存）；核对 history_event 透传 delegate 字段（如有白名单投影需补）。
|
|||
|
|
- `tests/golden/test_approval_store.py`、`test_harness_p3.py` 或新 `test_approval_delegate.py`：delegate 可审批、非 delegate 拒绝、owner 仍可、P3 双人分离不受影响、双后端持久化。
|
|||
|
|
- 文档：`docs/architecture/harness.md`、`docs/product/plan-completion-matrix.md`（P3 注记）、`docs/CHANGELOG.md`。
|
|||
|
|
|
|||
|
|
Out of scope:
|
|||
|
|
- 转派、批量委托、委托撤销、MySQL 实机故障注入。
|
|||
|
|
- 提交/推送/合并/清理用户改动。
|
|||
|
|
|
|||
|
|
## 6. 成功标准
|
|||
|
|
|
|||
|
|
- 聚焦：`python -m pytest -q tests/golden/test_approval_delegate.py tests/golden/test_approval_store.py tests/golden/test_harness_p3.py -p no:cacheprovider` 通过。
|
|||
|
|
- 全量：`python -m pytest tests/golden -q -p no:cacheprovider`（固定 .venv）≥ 462。
|
|||
|
|
- ruff 干净；git diff --check 无空白错误。
|
|||
|
|
|
|||
|
|
## 7. 验证方式
|
|||
|
|
|
|||
|
|
- 委托作用域单测（owner/delegate/非委托三方）；P3 双人分离回归；全量黄金。
|
|||
|
|
|
|||
|
|
## 8. 关键风险
|
|||
|
|
|
|||
|
|
| 风险 | 影响 | 控制方式 |
|
|||
|
|
|---|---|---|
|
|||
|
|
| delegate 越权审批 | 安全回归 | 只扩展 P2 owner/delegate 匹配；P3 不变 |
|
|||
|
|
| 双后端字段漂移 | 委托失效 | record 整存 + 测试覆盖文件与 DB 后端 |
|
|||
|
|
| 角色策略绕过 | 越权 | delegate 仍须具备审批角色 |
|
|||
|
|
|
|||
|
|
## 9. 停止条件
|
|||
|
|
|
|||
|
|
- 全量黄金测试非本轮相关回归无法快速定位时暂停。
|
|||
|
|
- 任何提交/推送/合并/清理操作停下等待授权。
|
|||
|
|
|
|||
|
|
## 10. 本轮完成定义
|
|||
|
|
|
|||
|
|
- 实现、测试、文档完成；聚焦与全量通过；矩阵/CHANGELOG 回写；不提交。
|
|||
|
|
- 收口：报告主要结论、关键洞察、需要特别留意的地方。
|
|||
|
|
|
|||
|
|
## 11. 下一步
|
|||
|
|
|
|||
|
|
轻量合并版：主 agent 实现 + 验证 + 收口报告。
|