相对第一版 46fc976 的完整变更。组员迁移对照表见 docs/20。
一、对外契约对齐 docs/05(破坏性,共 4 处,组员需按 docs/20 调整)
1) 配置发布端点改为文档规定的复数资源名:submit→validations、
approve→reviews(需 body decision)、activate→activations、
rollback→rollbacks;第一版这 4 个动词式路径 docs/05 从未定义过。
2) 错误码由 8 个笼统码改为 15 个具体语义码(FORBIDDEN→AGENT_PERMISSION_DENIED、
UNAUTHORIZED→AUTHENTICATION_REQUIRED、CONFLICT→RESOURCE_VERSION_CONFLICT、
RESOURCE_NOT_FOUND→RUN_NOT_FOUND/SESSION_NOT_FOUND 等),
输入类错误状态码 400→422。
3) POST /api/v1/agent-runs 与 GET /api/v1/agent-runs/{run_id} 统一为
{data, meta} 信封(data 内字段名与语义未变)。
4) 错误响应体统一为 {error:{code,message,retryable,field_errors}, meta:{trace_id}},
不再返回 FastAPI 默认的 {"detail": ...}。
二、数据库基线与约束
新增 39 张表的基线迁移(链根)与联合唯一键纠偏(4 张表、删 8 增 4,幂等收敛);
撤下 config_release 的双人复核 CHECK(应用层已允许自审,审核节点保留,
自审如实写入 reviewer_id);记忆 active key 生成列与唯一键;
activate 开始记录 supersedes_release_id 使版本链可追溯。
docs/00 基线未修改,未重命名或删除任何表与字段。
三、修复会静默出错或无报错的缺陷
- 跑完集成测试后平台会静默失去生效配置:清理只删自己创建的版本,却没有恢复被它
顶成 superseded 的原生效版本,且审计一并删除因而完全无痕,表现为所有工具被拒
但没有任何报错。已修清理逻辑并加恢复。
- Worker 单轮异常导致进程退出;记忆抽取调用方的“事务已开始”异常;
召回缓存丢失 degraded 标记;连接时区未生效导致 created_at/updated_at 差 8 小时;
.env 与 os.getenv 密钥来源分裂导致“没有可用的已批准模型端点”。
- 记忆信号识别漏判与跨键误命中;SSE 未带 Accept 的协商行为。
四、功能补齐
记忆链路 P1/P2/P3(抽取、受控词表、召回与缓存、生命周期级联及投影事件)、
fin_* 场内交易只读 ORM 层、agent_intent_config 状态流转并在运行期真正生效、
限流(Redis 固定窗口、故障一律放行)、游标校验、trace_id 中间件、
示例业务 Agent fund_query_demo 与一键端到端验证脚本,以及审计/指纹/迁移状态工具。
五、文档与验证
新增 docs/19(业务 Agent 接入实操)、docs/20(第一版迁移指南)与 docs/evidence 证据;
docs/01/02/06/08/09/17 同步实现现状。
验证结果:ruff 通过、mypy 103 文件无错、unit+contract 447 passed、
integration 29 passed、acceptance_check --production 7 PASS、
demo_agent_e2e 9/9 PASS(含失败关闭反证)。
124 lines
3.9 KiB
Python
124 lines
3.9 KiB
Python
"""`AuthorizationService.require` 契约测试(替身 session,不连数据库)。
|
||
|
||
它是所有受保护操作的公共闸门,覆盖四点:
|
||
1. 有权限时**直接返回**,不写审计、更不碰数据库——用"创建会话即报错"的替身证明;
|
||
2. 拒绝时**开新事务**写 `permission.denied` 审计,并抛 `ForbiddenAgentError`;
|
||
3. `admin=True` 时除了权限码还必须具备 admin/super_admin 角色;
|
||
4. 审计里必须留下 permission 与 trace_id,供事后追责。
|
||
"""
|
||
|
||
from typing import Any
|
||
|
||
import pytest
|
||
|
||
from app.core.contracts import RequestContext
|
||
from app.core.errors import ForbiddenAgentError
|
||
from app.service.authorization_service import AuthorizationService
|
||
|
||
BASE = RequestContext(user_id="9001", trace_id="trace-9")
|
||
|
||
|
||
def context_with(*permissions: str, roles: tuple[str, ...] = ()) -> RequestContext:
|
||
return RequestContext(
|
||
user_id="9001", trace_id="trace-9", permissions=tuple(permissions), roles=roles
|
||
)
|
||
|
||
|
||
class _AsyncContext:
|
||
async def __aenter__(self) -> None:
|
||
return None
|
||
|
||
async def __aexit__(self, *exc: object) -> bool:
|
||
return False
|
||
|
||
|
||
class FakeSession:
|
||
def __init__(self, sink: list[Any]) -> None:
|
||
self.sink = sink
|
||
|
||
def add(self, value: Any) -> None:
|
||
self.sink.append(value)
|
||
|
||
def begin(self) -> Any:
|
||
return _AsyncContext()
|
||
|
||
|
||
def patch_session(monkeypatch: pytest.MonkeyPatch, sink: list[Any]) -> None:
|
||
class FakeFactory:
|
||
async def __aenter__(self) -> FakeSession:
|
||
return FakeSession(sink)
|
||
|
||
async def __aexit__(self, *exc: object) -> bool:
|
||
return False
|
||
|
||
monkeypatch.setattr("app.service.authorization_service.SessionFactory", FakeFactory)
|
||
|
||
|
||
def explode_on_session() -> None:
|
||
raise AssertionError("有权限时不应创建数据库会话")
|
||
|
||
|
||
async def test_allowed_permission_returns_without_database_access(
|
||
monkeypatch: pytest.MonkeyPatch,
|
||
) -> None:
|
||
monkeypatch.setattr("app.service.authorization_service.SessionFactory", explode_on_session)
|
||
|
||
await AuthorizationService.require(
|
||
context_with("conversation:feedback"), "conversation:feedback"
|
||
)
|
||
|
||
|
||
async def test_denied_permission_writes_audit_and_raises(
|
||
monkeypatch: pytest.MonkeyPatch,
|
||
) -> None:
|
||
sink: list[Any] = []
|
||
patch_session(monkeypatch, sink)
|
||
|
||
with pytest.raises(ForbiddenAgentError):
|
||
await AuthorizationService.require(context_with("agent:run"), "config:write")
|
||
|
||
assert len(sink) == 1
|
||
audit = sink[0]
|
||
assert audit.action_type == "permission.denied"
|
||
assert audit.actor_id == 9001
|
||
assert audit.actor_type == "user"
|
||
assert audit.detail == {"permission": "config:write", "trace_id": "trace-9"}
|
||
|
||
|
||
async def test_admin_flag_requires_admin_role_even_with_permission(
|
||
monkeypatch: pytest.MonkeyPatch,
|
||
) -> None:
|
||
"""权限码齐备但角色不是 admin:必须拒绝,否则普通角色能进管理面。"""
|
||
sink: list[Any] = []
|
||
patch_session(monkeypatch, sink)
|
||
|
||
with pytest.raises(ForbiddenAgentError):
|
||
await AuthorizationService.require(
|
||
context_with("config:read", roles=("customer",)), "config:read", admin=True
|
||
)
|
||
|
||
assert len(sink) == 1
|
||
|
||
|
||
async def test_admin_flag_passes_for_admin_role(monkeypatch: pytest.MonkeyPatch) -> None:
|
||
monkeypatch.setattr("app.service.authorization_service.SessionFactory", explode_on_session)
|
||
|
||
await AuthorizationService.require(
|
||
context_with("config:read", roles=("admin",)), "config:read", admin=True
|
||
)
|
||
|
||
|
||
async def test_admin_flag_still_requires_the_permission_code(
|
||
monkeypatch: pytest.MonkeyPatch,
|
||
) -> None:
|
||
"""反之亦然:是 admin 角色但没有该权限码,同样拒绝。"""
|
||
sink: list[Any] = []
|
||
patch_session(monkeypatch, sink)
|
||
|
||
with pytest.raises(ForbiddenAgentError):
|
||
await AuthorizationService.require(
|
||
context_with(roles=("admin",)), "config:read", admin=True
|
||
)
|
||
|
||
assert len(sink) == 1
|