相对第一版 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
5.9 KiB
Python
124 lines
5.9 KiB
Python
from datetime import UTC, datetime
|
||
from uuid import uuid4
|
||
|
||
import pytest
|
||
from sqlalchemy import delete, select
|
||
|
||
from app.infrastructure.db import SessionFactory
|
||
from app.model.audit import InteractionAudit
|
||
from app.model.configuration import ConfigRelease
|
||
from app.model.platform import DomainEventOutbox
|
||
from app.service.config_release_service import ConfigReleaseError, ConfigReleaseService
|
||
|
||
|
||
@pytest.mark.integration
|
||
@pytest.mark.asyncio
|
||
async def test_config_release_lifecycle_is_versioned_and_audited() -> None:
|
||
now = datetime.now(UTC).replace(tzinfo=None)
|
||
release_no = f"it-{uuid4().hex}"
|
||
release_id = 0
|
||
rollback_id = 0
|
||
# 本测试会激活临时版本,把平台当前生效的版本顶成 superseded。先记下它是谁,
|
||
# 清理时恢复——否则平台会静默停在"零个 active 版本"(见 finally 的说明)。
|
||
async with SessionFactory() as session:
|
||
original_active_id = await session.scalar(
|
||
select(ConfigRelease.id).where(ConfigRelease.status == "active")
|
||
)
|
||
try:
|
||
async with SessionFactory() as session:
|
||
release = ConfigRelease(
|
||
release_no=release_no,
|
||
title="integration",
|
||
change_summary="integration",
|
||
status="draft",
|
||
created_by=101,
|
||
created_at=now,
|
||
updated_at=now,
|
||
)
|
||
session.add(release)
|
||
await session.flush()
|
||
release_id = release.id
|
||
service = ConfigReleaseService(session)
|
||
# 草稿不能直接审核:审核节点不可跳过。
|
||
with pytest.raises(ConfigReleaseError, match="not pending review"):
|
||
await service.approve(release_id, 101)
|
||
await service.submit_for_review(release_id, 101)
|
||
# 单人复核:创建人 101 自己审核通过。库侧的
|
||
# chk_config_release_separation 已由迁移 20260910_drop_review_separation 撤下,
|
||
# 因此自审与多人复核一样**如实写入** reviewer_id(列含义就是"审核人")。
|
||
approved = await service.approve(release_id, 101)
|
||
assert approved.reviewer_id == 101
|
||
assert approved.reviewed_at is not None
|
||
await service.activate(release_id, 101)
|
||
await session.commit()
|
||
|
||
async with SessionFactory() as session:
|
||
service = ConfigReleaseService(session)
|
||
rollback = await service.rollback(release_id, 101)
|
||
rollback_id = rollback.id
|
||
await session.commit()
|
||
|
||
async with SessionFactory() as session:
|
||
release = await session.get(ConfigRelease, release_id)
|
||
rollback = await session.get(ConfigRelease, rollback_id)
|
||
events = list(
|
||
await session.scalars(
|
||
select(DomainEventOutbox).where(
|
||
DomainEventOutbox.event_type == "config.cache_invalidate_requested",
|
||
DomainEventOutbox.aggregate_id.in_([str(release_id), str(rollback_id)]),
|
||
)
|
||
)
|
||
)
|
||
audits = list(
|
||
await session.scalars(
|
||
select(InteractionAudit).where(
|
||
InteractionAudit.action_type.in_(
|
||
[
|
||
"config.submit_review",
|
||
"config.approve",
|
||
"config.activate",
|
||
"config.rollback",
|
||
]
|
||
),
|
||
InteractionAudit.detail["release_id"].in_([release_id, rollback_id]),
|
||
)
|
||
)
|
||
)
|
||
assert release is not None and release.status == "superseded"
|
||
assert rollback is not None and rollback.status == "active"
|
||
assert rollback.rollback_of_release_id == release_id
|
||
assert len(events) == 2
|
||
assert len(audits) == 4
|
||
finally:
|
||
async with SessionFactory() as session:
|
||
await session.execute(
|
||
delete(InteractionAudit).where(
|
||
InteractionAudit.detail["release_id"].in_([release_id, rollback_id])
|
||
)
|
||
)
|
||
await session.execute(
|
||
delete(DomainEventOutbox).where(
|
||
DomainEventOutbox.aggregate_id.in_([str(release_id), str(rollback_id)])
|
||
)
|
||
)
|
||
await session.execute(
|
||
delete(ConfigRelease).where(ConfigRelease.release_no.like("it-%"))
|
||
)
|
||
# 回滚记录的 release_no 是 rollback-*,不在 it-* 前缀内。不删会留下一个
|
||
# 没有配置项的 active 版本,污染后续脚本读取的"当前生效版本"。
|
||
await session.execute(
|
||
delete(ConfigRelease).where(ConfigRelease.rollback_of_release_id == release_id)
|
||
)
|
||
# 删除临时版本只是清理的一半。本测试期间的激活已经把**平台原有的** active 版本
|
||
# 顶成了 superseded(activate 顶替当前 active),只删自己的版本会让平台静默停在
|
||
# "零个 active 版本":工具白名单随之变成空集,所有 Agent 工具按 fail-closed 被拒,
|
||
# 而这种故障没有任何报错,只在真正调用工具时才暴露(曾实际发生过)。因此删完后
|
||
# 必须把原先生效的版本恢复为 active——此时临时版本已删除,不会撞唯一键
|
||
# uk_config_release_active_one。
|
||
if original_active_id is not None:
|
||
original = await session.get(ConfigRelease, original_active_id)
|
||
if original is not None and original.status == "superseded":
|
||
original.status = "active"
|
||
original.updated_at = datetime.now(UTC).replace(tzinfo=None)
|
||
await session.commit()
|