相对第一版 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(含失败关闭反证)。
68 lines
3.3 KiB
Python
68 lines
3.3 KiB
Python
from datetime import UTC, datetime
|
|
from uuid import uuid4
|
|
|
|
from sqlalchemy.ext.asyncio import AsyncSession
|
|
|
|
from app.core.contracts import RequestContext
|
|
from app.core.errors import (
|
|
FeedbackAlreadyExistsError,
|
|
GenericResourceNotFoundError,
|
|
)
|
|
from app.model.audit import InteractionAudit
|
|
from app.model.conversation import ConversationFeedback
|
|
from app.repository.conversation_repository import ConversationRepository
|
|
from app.service.authorization_service import AuthorizationService
|
|
|
|
|
|
class ConversationService:
|
|
def __init__(self, session: AsyncSession) -> None:
|
|
self.session = session
|
|
self.repository = ConversationRepository(session)
|
|
|
|
async def messages(
|
|
self, session_id: str, context: RequestContext, limit: int, before: int | None = None
|
|
) -> dict[str, object]:
|
|
"""消息列表投影;`before` 是文档 §3.8 的游标(记录 ID 边界),已由入口校验。"""
|
|
rows = await self.repository.messages(
|
|
session_id, int(context.user_id), limit, before=before
|
|
)
|
|
return {"data": [{"message_id": str(row.id), "role": row.role, "content": row.content,
|
|
"created_at": row.created_at.isoformat() + "Z"} for row in rows]}
|
|
|
|
async def feedback(
|
|
self, message_id: int, context: RequestContext, rating: int,
|
|
feedback_type: str | None, feedback_content: str | None,
|
|
) -> dict[str, object]:
|
|
await AuthorizationService.require(context, "conversation:feedback")
|
|
async with self.session.begin():
|
|
message = await self.repository.message(message_id, int(context.user_id))
|
|
if message is None:
|
|
raise GenericResourceNotFoundError("消息不存在")
|
|
if await self.repository.feedback(message_id, int(context.user_id)) is not None:
|
|
raise FeedbackAlreadyExistsError("消息已反馈")
|
|
now = datetime.now(UTC).replace(tzinfo=None)
|
|
feedback = ConversationFeedback(
|
|
feedback_no=f"fb-{uuid4().hex[:24]}", session_id=message.session_id,
|
|
message_id=message_id, customer_id=int(context.user_id), rating=rating,
|
|
feedback_type=feedback_type, feedback_content=feedback_content, status="open",
|
|
created_at=now, updated_at=now,
|
|
)
|
|
self.session.add(feedback)
|
|
self._audit(context, message.session_id, "conversation.feedback_created",
|
|
{"feedback_no": feedback.feedback_no})
|
|
return {"data": {"feedback_no": feedback.feedback_no, "status": feedback.status}}
|
|
|
|
# 转人工申请(POST /api/v1/conversations/{id}/handover-requests)的唯一实现
|
|
# 在 PublicPlatformService.write("handover", ...):它同一事务写工单 + Outbox
|
|
# 事件 + 审计。此处曾经的第二实现只写审计,会让转人工异步链路断掉,已删除。
|
|
|
|
def _audit(
|
|
self, context: RequestContext, session_id: str, action: str, detail: dict[str, object]
|
|
) -> None:
|
|
self.session.add(InteractionAudit(
|
|
actor_type="user", actor_id=int(context.user_id), session_id=session_id,
|
|
portal=context.portal, action_type=action,
|
|
detail={**detail, "trace_id": context.trace_id},
|
|
created_at=datetime.now(UTC).replace(tzinfo=None),
|
|
))
|