lzf_0626
6516ccb385
相对第一版 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（含失败关闭反证）。
69 lines
3.3 KiB
Python
69 lines
3.3 KiB
Python
"""消息列表游标的真实数据库验证（文档 §3.8 游标分页、§16.1 非法游标 400）。
|
|
|
|
单元测试用替身证明了"校验 → 透传"这条链，但"游标真的改变返回哪一页"只有真实
|
|
MySQL 能证明：`before` 边界最终会变成一条 `id < :cursor` 的 SQL 条件，写错了（例如
|
|
方向反了、被当成 `id > :cursor`、或 Python 侧过滤丢掉跨页一致性）单元测试看不出来。
|
|
|
|
本用例只写自己的临时会话数据，结束时全部删除；不依赖 Worker，也不依赖常驻队列。
|
|
"""
|
|
|
|
from datetime import UTC, datetime
|
|
from uuid import uuid4
|
|
|
|
import httpx
|
|
import pytest
|
|
from sqlalchemy import delete
|
|
|
|
from app.api.dependencies.auth import build_request_context
|
|
from app.core.contracts import RequestContext
|
|
from app.infrastructure.db import SessionFactory
|
|
from app.main import create_app
|
|
from app.model.conversation import ConversationMessage
|
|
|
|
# 依赖真实 MySQL：必须打 integration marker，否则按 marker 过滤时会漏测这批用例。
|
|
pytestmark = pytest.mark.integration
|
|
|
|
CUSTOMER_ID = 9001 # tools/seed_test_rbac.py 造的数据
|
|
|
|
|
|
async def test_cursor_walks_pages_and_invalid_cursor_is_rejected() -> None:
|
|
session_id = f"cursor-{uuid4()}"
|
|
now = datetime.now(UTC).replace(tzinfo=None)
|
|
async with SessionFactory() as session, session.begin():
|
|
session.add_all([
|
|
ConversationMessage(session_id=session_id, customer_id=CUSTOMER_ID,
|
|
portal="customer_chat", role="user", content=f"m{index}",
|
|
created_at=now)
|
|
for index in range(3)
|
|
])
|
|
|
|
context = RequestContext(user_id=str(CUSTOMER_ID), trace_id=str(uuid4()),
|
|
roles=("customer",), permissions=())
|
|
application = create_app()
|
|
application.dependency_overrides[build_request_context] = lambda: context
|
|
path = f"/api/v1/conversations/{session_id}/messages"
|
|
try:
|
|
async with httpx.AsyncClient(transport=httpx.ASGITransport(app=application),
|
|
base_url="http://test") as client:
|
|
first = await client.get(path, params={"limit": 2})
|
|
assert first.status_code == 200
|
|
page = first.json()["data"]
|
|
assert [item["content"] for item in page] == ["m2", "m1"]
|
|
|
|
# 用上一页最后一条的 message_id 作游标：必须取到更旧的一页，而不是重复第一页。
|
|
second = await client.get(path, params={"limit": 2, "cursor": page[-1]["message_id"]})
|
|
assert [item["content"] for item in second.json()["data"]] == ["m0"]
|
|
|
|
# 走到头：再往前没有数据，返回空集合而不是报错、也不是回到第一页。
|
|
last = second.json()["data"][-1]["message_id"]
|
|
third = await client.get(path, params={"limit": 2, "cursor": last})
|
|
assert third.json()["data"] == []
|
|
|
|
invalid = await client.get(path, params={"cursor": "abc"})
|
|
assert invalid.status_code == 400
|
|
assert invalid.json()["error"]["code"] == "INVALID_CURSOR"
|
|
finally:
|
|
async with SessionFactory() as session, session.begin():
|
|
await session.execute(delete(ConversationMessage).where(
|
|
ConversationMessage.session_id == session_id))
|