refactor(chat): split 10070-line monolith into 8 service submodules (Phase1) [v2026.07.19.1]
This commit is contained in:
1 parent
4167e23e50
commit
9fb9d19cda
11 files changed
+9797
-10099
No files matched your search
@@ -3,20 +3,21 @@
|
||||
# 版本号格式: vYYYY.MM.DD.N (年.月.日.当日递增序号,从 1 开始)
|
||||
# 环境标签: prod = 正式版 | test = 测试版
|
||||
|
||||
VERSION=v2026.07.18.5
|
||||
VERSION=v2026.07.19.1
|
||||
ENV=prod
|
||||
DATE=2026-07-18
|
||||
DATE=2026-07-19
|
||||
BASE_IMAGE=qihu-aiflow:with-deps
|
||||
BACKUP_IMAGE=qihu-aiflow:backup-20260718-005
|
||||
BACKUP_IMAGE=qihu-aiflow:backup-20260719-001
|
||||
|
||||
## 本版本变更 (v2026.07.18.5 / prod)
|
||||
- Phase0 收官:新增「最小 CI 门禁」
|
||||
- scripts/ci_gate.py:统一测试网关,容器内跑后端 pytest + 前端 vitest,自动判定本地 docker 或经 ssh_helper 远程执行,任一失败即非零退出阻断发布
|
||||
- scripts/git-hooks/pre-push:纯 bash 的 git pre-push 钩子,push 前在容器内跑两套测试,失败即阻断 push(已安装到 .git/hooks)
|
||||
- .gitea/workflows/ci.yml:Gitea Actions 最小 CI 模板(当前无 act_runner,处于 dormant;钩子已先行拦截坏推送)
|
||||
- 测试安全网现状:后端 9 passed(smoke/market/sms + R1/R2/R3),前端 4 passed(datetime 时区格式化)
|
||||
## 本版本变更 (v2026.07.19.1 / prod)
|
||||
- Phase1 chat.py 巨石拆分收官:10070 行单体拆分为 chat_{utils,media,agents,skill,mcp,context,knowledge,service}.py 共 8 个子模块,controller 层 chat.py 降至 1352 行(<3000 目标达成)
|
||||
- 路由处理器(/conversations、/messages、/media/relay、/models 等全部 10 个路由)保留在 chat.py,业务/服务函数迁至子模块并通过 import 再导出
|
||||
- 关键修复:此前拆分误删了全部 @router 装饰器导致路由为 0(重启即全 404),本版本以 AST 比对原始提交恢复装饰器,并补回 chat_service._send_message_core 导入
|
||||
- 测试安全网守住 R1/R2/R3 不回退:后端 9 passed(smoke/market/sms + R1/R2/R3),前端 4 passed(datetime 时区格式化);ci_gate 全绿
|
||||
- 部署前已按 SOP 备份 rollback 点 qihu-aiflow:backup-20260719-001(旧 chat.py 10070 行)
|
||||
|
||||
## 历史
|
||||
- v2026.07.18.5 (prod):Phase0 收官 — 最小 CI 门禁(ci_gate.py + pre-push 钩子 + 休眠 Gitea workflow)。后端 9 passed,前端 4 passed。
|
||||
- v2026.07.18.4 (prod):前端 vitest 基建落地(4 passed,datetime 时区格式化)。
|
||||
- v2026.07.18.3 (prod):回归用例 R1/R2/R3(9 passed)— 微信Bot截断、技能/MCP覆盖规则、媒体生成超时回写。
|
||||
- v2026.07.18.2 (prod):后端 pytest 基建落地(6 passed)— 冒烟/市场14类/短信宝MD5。
|
||||
|
||||
+1383
-10070
File diff suppressed because it is too large.
Load diff
@@ -0,0 +1,943 @@
|
||||
# Auto-extracted from chat.py during Phase 1 refactor (Strangler Fig).
|
||||
# Source of truth: scripts/phase1_extract.py. Do not hand-merge back.
|
||||
|
||||
from fastapi import APIRouter, Depends, HTTPException, Query, Response
|
||||
from sqlalchemy.orm import Session
|
||||
from sqlalchemy import and_, desc, asc, func
|
||||
from pydantic import BaseModel, Field
|
||||
from typing import Optional
|
||||
import uuid
|
||||
from datetime import datetime, timezone, timedelta
|
||||
import re
|
||||
import json
|
||||
import asyncio
|
||||
import math
|
||||
import httpx
|
||||
import os
|
||||
import logging
|
||||
from ..database import get_db, SessionLocal
|
||||
from ..models import (
|
||||
|
||||
|
||||
|
||||
User, Conversation, Message, AIModel, ModelProvider, TokenUsageLog,
|
||||
|
||||
|
||||
|
||||
KnowledgeBase, KnowledgeDocument, KnowledgeDocumentChunk, EvolutionSkill,
|
||||
|
||||
|
||||
|
||||
EvolutionTrajectory, MarketApp, AppPurchase, AppInstallation,
|
||||
|
||||
|
||||
|
||||
MCPServer,
|
||||
|
||||
|
||||
|
||||
gen_uuid
|
||||
|
||||
|
||||
|
||||
)
|
||||
from ..auth import get_current_user
|
||||
from ..platform_knowledge import get_platform_knowledge
|
||||
from ..mcp_client import list_mcp_tools, call_mcp_tool
|
||||
from ..middleware.log_service import LogService
|
||||
from ..capability_registry import resolve_auto_model
|
||||
from app.api.chat_utils import (_extract_text_from_json, _proxy_for_url)
|
||||
|
||||
def _normalize_dify_chat_url(endpoint: str) -> str:
|
||||
|
||||
|
||||
|
||||
"""把用户填写的 Dify 地址归一化为 /v1/chat-messages 端点。与 market.py 保持一致。"""
|
||||
|
||||
|
||||
|
||||
ep = (endpoint or "").rstrip("/")
|
||||
|
||||
|
||||
|
||||
if ep.endswith("/chat-messages"):
|
||||
|
||||
|
||||
|
||||
return ep
|
||||
|
||||
|
||||
|
||||
if ep.endswith("/v1"):
|
||||
|
||||
|
||||
|
||||
return ep + "/chat-messages"
|
||||
|
||||
|
||||
|
||||
return ep + "/v1/chat-messages"
|
||||
|
||||
async def _call_dify_agent(endpoint: str, api_key: str, query: str,
|
||||
|
||||
|
||||
|
||||
user_id: str, ext_conv_id: str | None) -> tuple:
|
||||
|
||||
|
||||
|
||||
"""调用 Dify(新版 Agent) /v1/chat-messages,流式(SSE)解析最终回答。
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
新版 Agent API 仅支持 response_mode=streaming(blocking 会 400)。事件流中
|
||||
|
||||
|
||||
|
||||
message / agent_message 事件携带增量 answer,累加即为最终回答;每个事件都带
|
||||
|
||||
|
||||
|
||||
conversation_id,用于维持多轮上下文。返回 (answer, new_conversation_id)。
|
||||
|
||||
|
||||
|
||||
"""
|
||||
|
||||
|
||||
|
||||
url = _normalize_dify_chat_url(endpoint)
|
||||
|
||||
|
||||
|
||||
headers = {"Authorization": f"Bearer {api_key}", "Content-Type": "application/json"}
|
||||
|
||||
|
||||
|
||||
payload = {
|
||||
|
||||
|
||||
|
||||
"inputs": {},
|
||||
|
||||
|
||||
|
||||
"query": query,
|
||||
|
||||
|
||||
|
||||
"user": (user_id or "web-user")[:64],
|
||||
|
||||
|
||||
|
||||
"response_mode": "streaming",
|
||||
|
||||
|
||||
|
||||
"conversation_id": ext_conv_id or "",
|
||||
|
||||
|
||||
|
||||
}
|
||||
|
||||
|
||||
|
||||
answer_parts = []
|
||||
|
||||
|
||||
|
||||
new_conv_id = ext_conv_id
|
||||
|
||||
|
||||
|
||||
err_msg = None
|
||||
|
||||
|
||||
|
||||
wf_outputs = None
|
||||
|
||||
|
||||
|
||||
# httpx 默认 follow_redirects=False;stream POST 若遇 301/302 会直接返回该状态码。
|
||||
|
||||
|
||||
|
||||
# 且 httpx 自动跟随会把 POST 降级为 GET(对 /v1/chat-messages 变 405)。
|
||||
|
||||
|
||||
|
||||
# 因此手动跟随 301/302/307/308,保持 POST 与请求体不变(最多 4 跳)。
|
||||
|
||||
|
||||
|
||||
async with httpx.AsyncClient(timeout=180, proxy=_proxy_for_url(url), trust_env=False,
|
||||
|
||||
|
||||
|
||||
follow_redirects=False) as client:
|
||||
|
||||
|
||||
|
||||
cur_url = url
|
||||
|
||||
|
||||
|
||||
for _redir in range(4):
|
||||
|
||||
|
||||
|
||||
async with client.stream("POST", cur_url, headers=headers, json=payload) as resp:
|
||||
|
||||
|
||||
|
||||
if resp.status_code in (301, 302, 307, 308):
|
||||
|
||||
|
||||
|
||||
loc = resp.headers.get("location") or resp.headers.get("Location")
|
||||
|
||||
|
||||
|
||||
await resp.aclose()
|
||||
|
||||
|
||||
|
||||
if not loc:
|
||||
|
||||
|
||||
|
||||
raise RuntimeError(f"Dify HTTP {resp.status_code}: 缺少重定向地址")
|
||||
|
||||
|
||||
|
||||
# 处理相对地址
|
||||
|
||||
|
||||
|
||||
if loc.startswith("/"):
|
||||
|
||||
|
||||
|
||||
from urllib.parse import urlsplit, urlunsplit
|
||||
|
||||
|
||||
|
||||
p = urlsplit(cur_url)
|
||||
|
||||
|
||||
|
||||
loc = urlunsplit((p.scheme, p.netloc, loc, "", ""))
|
||||
|
||||
|
||||
|
||||
cur_url = loc
|
||||
|
||||
|
||||
|
||||
continue
|
||||
|
||||
|
||||
|
||||
if resp.status_code != 200:
|
||||
|
||||
|
||||
|
||||
body = (await resp.aread()).decode("utf-8", "replace")
|
||||
|
||||
|
||||
|
||||
raise RuntimeError(f"Dify HTTP {resp.status_code}: {body[:300]}")
|
||||
|
||||
|
||||
|
||||
async for line in resp.aiter_lines():
|
||||
|
||||
|
||||
|
||||
if not line:
|
||||
|
||||
|
||||
|
||||
continue
|
||||
|
||||
|
||||
|
||||
line = line.strip()
|
||||
|
||||
|
||||
|
||||
if not line.startswith("data:"):
|
||||
|
||||
|
||||
|
||||
continue
|
||||
|
||||
|
||||
|
||||
data_str = line[5:].strip()
|
||||
|
||||
|
||||
|
||||
if not data_str or data_str == "[DONE]":
|
||||
|
||||
|
||||
|
||||
continue
|
||||
|
||||
|
||||
|
||||
try:
|
||||
|
||||
|
||||
|
||||
evt = json.loads(data_str)
|
||||
|
||||
|
||||
|
||||
except Exception:
|
||||
|
||||
|
||||
|
||||
continue
|
||||
|
||||
|
||||
|
||||
ev = evt.get("event")
|
||||
|
||||
|
||||
|
||||
if evt.get("conversation_id"):
|
||||
|
||||
|
||||
|
||||
new_conv_id = evt["conversation_id"]
|
||||
|
||||
|
||||
|
||||
data = evt.get("data") or {}
|
||||
|
||||
|
||||
|
||||
# 兼容三类 Dify 应用:
|
||||
|
||||
|
||||
|
||||
# 1) Chat/Agent 应用: message / agent_message 事件带增量 answer
|
||||
|
||||
|
||||
|
||||
# 2) Chatflow/Workflow 应用: text_chunk 事件带 data.text;
|
||||
|
||||
|
||||
|
||||
# 结束时 workflow_finished 带 status / outputs / error
|
||||
|
||||
|
||||
|
||||
if ev in ("message", "agent_message"):
|
||||
|
||||
|
||||
|
||||
answer_parts.append(evt.get("answer", "") or "")
|
||||
|
||||
|
||||
|
||||
elif ev == "message_replace":
|
||||
|
||||
|
||||
|
||||
# 内容审查替换:以替换后的完整 answer 为准
|
||||
|
||||
|
||||
|
||||
answer_parts = [evt.get("answer", "") or ""]
|
||||
|
||||
|
||||
|
||||
elif ev == "text_chunk":
|
||||
|
||||
|
||||
|
||||
answer_parts.append((data.get("text") or ""))
|
||||
|
||||
|
||||
|
||||
elif ev == "node_finished":
|
||||
|
||||
|
||||
|
||||
# answer / end 节点的 outputs 可能含最终文本
|
||||
|
||||
|
||||
|
||||
if data.get("node_type") in ("answer", "end") and data.get("outputs"):
|
||||
|
||||
|
||||
|
||||
wf_outputs = data.get("outputs")
|
||||
|
||||
|
||||
|
||||
if data.get("status") == "failed" and data.get("error"):
|
||||
|
||||
|
||||
|
||||
err_msg = str(data.get("error"))
|
||||
|
||||
|
||||
|
||||
elif ev == "workflow_finished":
|
||||
|
||||
|
||||
|
||||
if data.get("status") == "failed":
|
||||
|
||||
|
||||
|
||||
err_msg = str(data.get("error") or "Dify 工作流执行失败")
|
||||
|
||||
|
||||
|
||||
elif data.get("outputs"):
|
||||
|
||||
|
||||
|
||||
wf_outputs = data.get("outputs")
|
||||
|
||||
|
||||
|
||||
elif ev == "error":
|
||||
|
||||
|
||||
|
||||
err_msg = evt.get("message") or evt.get("code") or "Dify 流式返回错误"
|
||||
|
||||
|
||||
|
||||
break
|
||||
|
||||
|
||||
|
||||
else:
|
||||
|
||||
|
||||
|
||||
raise RuntimeError("Dify 重定向次数过多(>4),请直接填写重定向后的地址")
|
||||
|
||||
|
||||
|
||||
answer = "".join(answer_parts).strip()
|
||||
|
||||
|
||||
|
||||
# 流式无增量文本时,尝试从 workflow 结束 outputs 兜底提取
|
||||
|
||||
|
||||
|
||||
if not answer and wf_outputs:
|
||||
|
||||
|
||||
|
||||
answer = _extract_text_from_json(wf_outputs).strip()
|
||||
|
||||
|
||||
|
||||
if not answer and err_msg:
|
||||
|
||||
|
||||
|
||||
raise RuntimeError(err_msg)
|
||||
|
||||
|
||||
|
||||
return answer, new_conv_id
|
||||
|
||||
async def _call_n8n_agent(endpoint: str, api_key: str, query: str,
|
||||
|
||||
|
||||
|
||||
user_id: str, history: list) -> tuple:
|
||||
|
||||
|
||||
|
||||
"""调用 n8n 生产 Webhook (POST)。webhook 用 Respond to Webhook 节点返回 JSON,
|
||||
|
||||
|
||||
|
||||
尽力从中抽取文本回答。n8n webhook 无平台级会话态,多轮靠我们下发 history。
|
||||
|
||||
|
||||
|
||||
返回 (answer, None)。"""
|
||||
|
||||
|
||||
|
||||
url = (endpoint or "").strip()
|
||||
|
||||
|
||||
|
||||
headers = {"Content-Type": "application/json"}
|
||||
|
||||
|
||||
|
||||
if api_key:
|
||||
|
||||
|
||||
|
||||
# n8n Header Auth 自定义头名不固定,同时下发常见几种以提高兼容性
|
||||
|
||||
|
||||
|
||||
headers["Authorization"] = f"Bearer {api_key}"
|
||||
|
||||
|
||||
|
||||
headers["X-N8N-API-KEY"] = api_key
|
||||
|
||||
|
||||
|
||||
# 携带多种字段别名,兼容 n8n AI Agent(chatInput) 与自定义工作流
|
||||
|
||||
|
||||
|
||||
payload = {
|
||||
|
||||
|
||||
|
||||
"query": query,
|
||||
|
||||
|
||||
|
||||
"chatInput": query,
|
||||
|
||||
|
||||
|
||||
"message": query,
|
||||
|
||||
|
||||
|
||||
"input": query,
|
||||
|
||||
|
||||
|
||||
"user": (user_id or "web-user")[:64],
|
||||
|
||||
|
||||
|
||||
"history": history or [],
|
||||
|
||||
|
||||
|
||||
}
|
||||
|
||||
|
||||
|
||||
async with httpx.AsyncClient(timeout=180, proxy=_proxy_for_url(url), trust_env=False) as client:
|
||||
|
||||
|
||||
|
||||
resp = await client.post(url, headers=headers, json=payload)
|
||||
|
||||
|
||||
|
||||
if resp.status_code not in (200, 201):
|
||||
|
||||
|
||||
|
||||
raise RuntimeError(f"n8n Webhook HTTP {resp.status_code}: {resp.text[:300]}")
|
||||
|
||||
|
||||
|
||||
ctype = resp.headers.get("content-type", "")
|
||||
|
||||
|
||||
|
||||
if "application/json" in ctype:
|
||||
|
||||
|
||||
|
||||
try:
|
||||
|
||||
|
||||
|
||||
answer = _extract_text_from_json(resp.json())
|
||||
|
||||
|
||||
|
||||
except Exception:
|
||||
|
||||
|
||||
|
||||
answer = resp.text
|
||||
|
||||
|
||||
|
||||
else:
|
||||
|
||||
|
||||
|
||||
answer = resp.text
|
||||
|
||||
|
||||
|
||||
return (answer or "").strip(), None
|
||||
|
||||
async def _call_coze_agent(endpoint: str, api_key: str, query: str,
|
||||
|
||||
|
||||
|
||||
user_id: str, ext_conv_id: str | None) -> tuple:
|
||||
|
||||
|
||||
|
||||
"""调用扣子(Coze) v3 对话 API:create → 轮询 retrieve → 拉取 message list。
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
约定 agent_endpoint 存放 bot_id(纯 id),或形如
|
||||
|
||||
|
||||
|
||||
https://api.coze.cn/v3/chat?bot_id=xxx 的完整地址。默认基址 api.coze.cn。
|
||||
|
||||
|
||||
|
||||
返回 (answer, conversation_id)。
|
||||
|
||||
|
||||
|
||||
"""
|
||||
|
||||
|
||||
|
||||
ep = (endpoint or "").strip()
|
||||
|
||||
|
||||
|
||||
from urllib.parse import urlparse, parse_qs
|
||||
|
||||
|
||||
|
||||
bot_id = None
|
||||
|
||||
|
||||
|
||||
base = "https://api.coze.cn"
|
||||
|
||||
|
||||
|
||||
if ep.startswith("http"):
|
||||
|
||||
|
||||
|
||||
pr = urlparse(ep)
|
||||
|
||||
|
||||
|
||||
base = f"{pr.scheme}://{pr.netloc}"
|
||||
|
||||
|
||||
|
||||
qs = parse_qs(pr.query)
|
||||
|
||||
|
||||
|
||||
bot_id = (qs.get("bot_id") or [None])[0]
|
||||
|
||||
|
||||
|
||||
if not bot_id and "/bot/" in pr.path:
|
||||
|
||||
|
||||
|
||||
bot_id = pr.path.rstrip("/").split("/bot/")[-1]
|
||||
|
||||
|
||||
|
||||
else:
|
||||
|
||||
|
||||
|
||||
bot_id = ep # 纯 bot_id
|
||||
|
||||
|
||||
|
||||
if not bot_id:
|
||||
|
||||
|
||||
|
||||
raise RuntimeError("Coze 接入缺少 bot_id(请在 Endpoint 填入 bot_id 或带 ?bot_id= 的地址)")
|
||||
|
||||
|
||||
|
||||
headers = {"Authorization": f"Bearer {api_key}", "Content-Type": "application/json"}
|
||||
|
||||
|
||||
|
||||
create_url = f"{base}/v3/chat"
|
||||
|
||||
|
||||
|
||||
if ext_conv_id:
|
||||
|
||||
|
||||
|
||||
create_url += f"?conversation_id={ext_conv_id}"
|
||||
|
||||
|
||||
|
||||
payload = {
|
||||
|
||||
|
||||
|
||||
"bot_id": bot_id,
|
||||
|
||||
|
||||
|
||||
"user_id": (user_id or "web-user")[:64],
|
||||
|
||||
|
||||
|
||||
"stream": False,
|
||||
|
||||
|
||||
|
||||
"auto_save_history": True,
|
||||
|
||||
|
||||
|
||||
"additional_messages": [
|
||||
|
||||
|
||||
|
||||
{"role": "user", "content": query, "content_type": "text"}
|
||||
|
||||
|
||||
|
||||
],
|
||||
|
||||
|
||||
|
||||
}
|
||||
|
||||
|
||||
|
||||
async with httpx.AsyncClient(timeout=180, proxy=_proxy_for_url(create_url), trust_env=False) as client:
|
||||
|
||||
|
||||
|
||||
r = await client.post(create_url, headers=headers, json=payload)
|
||||
|
||||
|
||||
|
||||
if r.status_code != 200:
|
||||
|
||||
|
||||
|
||||
raise RuntimeError(f"Coze create HTTP {r.status_code}: {r.text[:300]}")
|
||||
|
||||
|
||||
|
||||
cd = r.json()
|
||||
|
||||
|
||||
|
||||
if cd.get("code") not in (0, None):
|
||||
|
||||
|
||||
|
||||
raise RuntimeError(f"Coze 错误: {cd.get('msg') or cd}")
|
||||
|
||||
|
||||
|
||||
chat = cd.get("data") or {}
|
||||
|
||||
|
||||
|
||||
chat_id = chat.get("id")
|
||||
|
||||
|
||||
|
||||
conv_id2 = chat.get("conversation_id") or ext_conv_id
|
||||
|
||||
|
||||
|
||||
if not chat_id or not conv_id2:
|
||||
|
||||
|
||||
|
||||
raise RuntimeError(f"Coze 未返回 chat_id/conversation_id: {str(cd)[:200]}")
|
||||
|
||||
|
||||
|
||||
# 轮询直到完成(最多约 3 分钟)
|
||||
|
||||
|
||||
|
||||
status = chat.get("status")
|
||||
|
||||
|
||||
|
||||
for _ in range(60):
|
||||
|
||||
|
||||
|
||||
if status == "completed":
|
||||
|
||||
|
||||
|
||||
break
|
||||
|
||||
|
||||
|
||||
if status in ("failed", "requires_action", "canceled"):
|
||||
|
||||
|
||||
|
||||
raise RuntimeError(f"Coze 对话状态 {status}")
|
||||
|
||||
|
||||
|
||||
await asyncio.sleep(3)
|
||||
|
||||
|
||||
|
||||
rr = await client.get(
|
||||
|
||||
|
||||
|
||||
f"{base}/v3/chat/retrieve?chat_id={chat_id}&conversation_id={conv_id2}",
|
||||
|
||||
|
||||
|
||||
headers=headers,
|
||||
|
||||
|
||||
|
||||
)
|
||||
|
||||
|
||||
|
||||
if rr.status_code == 200:
|
||||
|
||||
|
||||
|
||||
status = ((rr.json() or {}).get("data") or {}).get("status")
|
||||
|
||||
|
||||
|
||||
# 拉取消息列表,取 type=answer 的内容
|
||||
|
||||
|
||||
|
||||
rm = await client.get(
|
||||
|
||||
|
||||
|
||||
f"{base}/v3/chat/message/list?chat_id={chat_id}&conversation_id={conv_id2}",
|
||||
|
||||
|
||||
|
||||
headers=headers,
|
||||
|
||||
|
||||
|
||||
)
|
||||
|
||||
|
||||
|
||||
answer = ""
|
||||
|
||||
|
||||
|
||||
if rm.status_code == 200:
|
||||
|
||||
|
||||
|
||||
for m in ((rm.json() or {}).get("data") or []):
|
||||
|
||||
|
||||
|
||||
if m.get("type") == "answer" and m.get("role") == "assistant":
|
||||
|
||||
|
||||
|
||||
answer = m.get("content", "") or answer
|
||||
|
||||
|
||||
|
||||
return (answer or "").strip(), conv_id2
|
||||
|
||||
async def _call_external_agent(db, agent: dict, query: str, user_id: str,
|
||||
|
||||
|
||||
|
||||
ext_conversation_id: str | None,
|
||||
|
||||
|
||||
|
||||
history: list) -> tuple:
|
||||
|
||||
|
||||
|
||||
"""按平台分发到对应外部 Agent 远程调用。返回 (answer, new_ext_conversation_id)。"""
|
||||
|
||||
|
||||
|
||||
platform = (agent.get("platform") or "dify").lower()
|
||||
|
||||
|
||||
|
||||
endpoint = agent.get("agent_endpoint")
|
||||
|
||||
|
||||
|
||||
api_key = agent.get("agent_api_key") or ""
|
||||
|
||||
|
||||
|
||||
if not endpoint:
|
||||
|
||||
|
||||
|
||||
raise RuntimeError("该 Agent 未配置远程调用地址(agent_endpoint)")
|
||||
|
||||
|
||||
|
||||
if platform == "dify":
|
||||
|
||||
|
||||
|
||||
return await _call_dify_agent(endpoint, api_key, query, user_id, ext_conversation_id)
|
||||
|
||||
|
||||
|
||||
if platform == "n8n":
|
||||
|
||||
|
||||
|
||||
return await _call_n8n_agent(endpoint, api_key, query, user_id, history)
|
||||
|
||||
|
||||
|
||||
if platform == "coze":
|
||||
|
||||
|
||||
|
||||
return await _call_coze_agent(endpoint, api_key, query, user_id, ext_conversation_id)
|
||||
|
||||
|
||||
|
||||
# 未知平台:默认按 Dify 协议尝试
|
||||
|
||||
|
||||
|
||||
return await _call_dify_agent(endpoint, api_key, query, user_id, ext_conversation_id)
|
||||
File diff suppressed because it is too large.
Load diff
@@ -0,0 +1,319 @@
|
||||
# Auto-extracted from chat.py during Phase 1 refactor (Strangler Fig).
|
||||
# Source of truth: scripts/phase1_extract.py. Do not hand-merge back.
|
||||
|
||||
from fastapi import APIRouter, Depends, HTTPException, Query, Response
|
||||
from sqlalchemy.orm import Session
|
||||
from sqlalchemy import and_, desc, asc, func
|
||||
from pydantic import BaseModel, Field
|
||||
from typing import Optional
|
||||
import uuid
|
||||
from datetime import datetime, timezone, timedelta
|
||||
import re
|
||||
import json
|
||||
import asyncio
|
||||
import math
|
||||
import httpx
|
||||
import os
|
||||
import logging
|
||||
from ..database import get_db, SessionLocal
|
||||
from ..models import (
|
||||
|
||||
|
||||
|
||||
User, Conversation, Message, AIModel, ModelProvider, TokenUsageLog,
|
||||
|
||||
|
||||
|
||||
KnowledgeBase, KnowledgeDocument, KnowledgeDocumentChunk, EvolutionSkill,
|
||||
|
||||
|
||||
|
||||
EvolutionTrajectory, MarketApp, AppPurchase, AppInstallation,
|
||||
|
||||
|
||||
|
||||
MCPServer,
|
||||
|
||||
|
||||
|
||||
gen_uuid
|
||||
|
||||
|
||||
|
||||
)
|
||||
from ..auth import get_current_user
|
||||
from ..platform_knowledge import get_platform_knowledge
|
||||
from ..mcp_client import list_mcp_tools, call_mcp_tool
|
||||
from ..middleware.log_service import LogService
|
||||
from ..capability_registry import resolve_auto_model
|
||||
from app.api.chat_utils import (cst_now)
|
||||
|
||||
def _build_knowledge_graph(db: Session, kb: KnowledgeBase) -> dict:
|
||||
|
||||
|
||||
|
||||
"""从知识库文档构建知识图谱节点和边,返回 graph_data。"""
|
||||
|
||||
|
||||
|
||||
documents = db.query(KnowledgeDocument).filter(
|
||||
|
||||
|
||||
|
||||
and_(
|
||||
|
||||
|
||||
|
||||
KnowledgeDocument.knowledge_base_id == kb.id,
|
||||
|
||||
|
||||
|
||||
KnowledgeDocument.status == "completed",
|
||||
|
||||
|
||||
|
||||
)
|
||||
|
||||
|
||||
|
||||
).all()
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
graph_nodes = []
|
||||
|
||||
|
||||
|
||||
graph_edges = []
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
doc_entity_types = [
|
||||
|
||||
|
||||
|
||||
{"type": "concept", "label": "核心概念"},
|
||||
|
||||
|
||||
|
||||
{"type": "entity", "label": "关键实体"},
|
||||
|
||||
|
||||
|
||||
{"type": "attribute", "label": "属性特征"},
|
||||
|
||||
|
||||
|
||||
{"type": "relation", "label": "关联关系"},
|
||||
|
||||
|
||||
|
||||
]
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
for idx, doc in enumerate(documents):
|
||||
|
||||
|
||||
|
||||
doc_node_id = f"doc-{doc.id[:8]}"
|
||||
|
||||
|
||||
|
||||
graph_nodes.append({
|
||||
|
||||
|
||||
|
||||
"id": doc_node_id, "label": doc.filename, "type": "document", "size": 18, "group": "document",
|
||||
|
||||
|
||||
|
||||
})
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
filename_base = doc.filename.rsplit(".", 1)[0]
|
||||
|
||||
|
||||
|
||||
words = filename_base.replace("_", " ").replace("-", " ").split()
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
for wi, word in enumerate(words):
|
||||
|
||||
|
||||
|
||||
if len(word) >= 2:
|
||||
|
||||
|
||||
|
||||
entity_type = doc_entity_types[wi % len(doc_entity_types)]
|
||||
|
||||
|
||||
|
||||
node_id = f"ent-{doc.id[:8]}-{wi}"
|
||||
|
||||
|
||||
|
||||
graph_nodes.append({
|
||||
|
||||
|
||||
|
||||
"id": node_id, "label": word, "type": entity_type["type"],
|
||||
|
||||
|
||||
|
||||
"size": 10 + (wi % 8), "group": entity_type["type"],
|
||||
|
||||
|
||||
|
||||
})
|
||||
|
||||
|
||||
|
||||
graph_edges.append({
|
||||
|
||||
|
||||
|
||||
"source": doc_node_id, "target": node_id,
|
||||
|
||||
|
||||
|
||||
"label": entity_type["label"], "weight": 0.5 + (0.5 / (wi + 1)),
|
||||
|
||||
|
||||
|
||||
})
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
for i in range(len(documents)):
|
||||
|
||||
|
||||
|
||||
for j in range(i + 1, len(documents)):
|
||||
|
||||
|
||||
|
||||
fi = documents[i].filename.lower().rsplit(".", 1)[0]
|
||||
|
||||
|
||||
|
||||
fj = documents[j].filename.lower().rsplit(".", 1)[0]
|
||||
|
||||
|
||||
|
||||
words_i = set(fi.replace("_", " ").replace("-", " ").split())
|
||||
|
||||
|
||||
|
||||
words_j = set(fj.replace("_", " ").replace("-", " ").split())
|
||||
|
||||
|
||||
|
||||
common = words_i & words_j
|
||||
|
||||
|
||||
|
||||
if common and len(common) > 0:
|
||||
|
||||
|
||||
|
||||
graph_edges.append({
|
||||
|
||||
|
||||
|
||||
"source": f"doc-{documents[i].id[:8]}",
|
||||
|
||||
|
||||
|
||||
"target": f"doc-{documents[j].id[:8]}",
|
||||
|
||||
|
||||
|
||||
"label": f"共有关键词: {', '.join(list(common)[:3])}",
|
||||
|
||||
|
||||
|
||||
"weight": min(0.9, 0.3 * len(common)),
|
||||
|
||||
|
||||
|
||||
})
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
graph_data = {
|
||||
|
||||
|
||||
|
||||
"nodes": graph_nodes,
|
||||
|
||||
|
||||
|
||||
"edges": graph_edges,
|
||||
|
||||
|
||||
|
||||
"stats": {
|
||||
|
||||
|
||||
|
||||
"total_nodes": len(graph_nodes), "total_edges": len(graph_edges),
|
||||
|
||||
|
||||
|
||||
"document_count": len(documents),
|
||||
|
||||
|
||||
|
||||
},
|
||||
|
||||
|
||||
|
||||
}
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
kb.knowledge_graph_count = len(graph_nodes)
|
||||
|
||||
|
||||
|
||||
kb.updated_at = cst_now()
|
||||
|
||||
|
||||
|
||||
return graph_data
|
||||
@@ -0,0 +1,240 @@
|
||||
# Auto-extracted from chat.py during Phase 1 refactor (Strangler Fig).
|
||||
# Source of truth: scripts/phase1_extract.py. Do not hand-merge back.
|
||||
|
||||
from fastapi import APIRouter, Depends, HTTPException, Query, Response
|
||||
from sqlalchemy.orm import Session
|
||||
from sqlalchemy import and_, desc, asc, func
|
||||
from pydantic import BaseModel, Field
|
||||
from typing import Optional
|
||||
import uuid
|
||||
from datetime import datetime, timezone, timedelta
|
||||
import re
|
||||
import json
|
||||
import asyncio
|
||||
import math
|
||||
import httpx
|
||||
import os
|
||||
import logging
|
||||
from ..database import get_db, SessionLocal
|
||||
from ..models import (
|
||||
|
||||
|
||||
|
||||
User, Conversation, Message, AIModel, ModelProvider, TokenUsageLog,
|
||||
|
||||
|
||||
|
||||
KnowledgeBase, KnowledgeDocument, KnowledgeDocumentChunk, EvolutionSkill,
|
||||
|
||||
|
||||
|
||||
EvolutionTrajectory, MarketApp, AppPurchase, AppInstallation,
|
||||
|
||||
|
||||
|
||||
MCPServer,
|
||||
|
||||
|
||||
|
||||
gen_uuid
|
||||
|
||||
|
||||
|
||||
)
|
||||
from ..auth import get_current_user
|
||||
from ..platform_knowledge import get_platform_knowledge
|
||||
from ..mcp_client import list_mcp_tools, call_mcp_tool
|
||||
from ..middleware.log_service import LogService
|
||||
from ..capability_registry import resolve_auto_model
|
||||
|
||||
def _resolve_market_mcp(db: Session, mcp_ids: list, user_id: str) -> list:
|
||||
|
||||
|
||||
|
||||
"""
|
||||
|
||||
|
||||
|
||||
加载用户选择的 MCP 应用配置(MarketApp.type == 'mcp')。
|
||||
|
||||
|
||||
|
||||
返回: [{id, name, transport, endpoint, description}]
|
||||
|
||||
|
||||
|
||||
"""
|
||||
|
||||
|
||||
|
||||
if not mcp_ids:
|
||||
|
||||
|
||||
|
||||
return []
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
mcps = db.query(MarketApp).filter(
|
||||
|
||||
|
||||
|
||||
MarketApp.id.in_(mcp_ids),
|
||||
|
||||
|
||||
|
||||
MarketApp.type == "mcp"
|
||||
|
||||
|
||||
|
||||
).all()
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
results = []
|
||||
|
||||
|
||||
|
||||
for m in mcps:
|
||||
|
||||
|
||||
|
||||
config = {}
|
||||
|
||||
|
||||
|
||||
try:
|
||||
|
||||
|
||||
|
||||
if m.skill_content:
|
||||
|
||||
|
||||
|
||||
config = json.loads(m.skill_content)
|
||||
|
||||
|
||||
|
||||
except Exception:
|
||||
|
||||
|
||||
|
||||
config = {}
|
||||
|
||||
|
||||
|
||||
results.append({
|
||||
|
||||
|
||||
|
||||
"id": m.id,
|
||||
|
||||
|
||||
|
||||
"name": m.name,
|
||||
|
||||
|
||||
|
||||
"transport": m.mcp_transport or config.get("transport", "stdio"),
|
||||
|
||||
|
||||
|
||||
"endpoint": m.agent_endpoint or config.get("endpoint", "") or config.get("url", ""),
|
||||
|
||||
|
||||
|
||||
"description": m.description or config.get("description", ""),
|
||||
|
||||
|
||||
|
||||
})
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
return results
|
||||
|
||||
def _resolve_active_mcp_servers(db: Session, mcp_ids: list) -> list:
|
||||
|
||||
"""
|
||||
|
||||
解析用户选中的 MCP 服务连接信息(来自 mcp_servers 表,import 与后台添加通用)。
|
||||
|
||||
返回: [{id, name, description, transport, endpoint, url, headers}]
|
||||
|
||||
"""
|
||||
|
||||
if not mcp_ids:
|
||||
|
||||
return []
|
||||
|
||||
servers = db.query(MCPServer).filter(
|
||||
|
||||
MCPServer.id.in_(mcp_ids),
|
||||
|
||||
MCPServer.is_enabled == True,
|
||||
|
||||
).all()
|
||||
|
||||
out = []
|
||||
|
||||
for s in servers:
|
||||
|
||||
try:
|
||||
|
||||
headers = json.loads(s.headers) if isinstance(s.headers, str) else (s.headers or {})
|
||||
|
||||
except Exception:
|
||||
|
||||
headers = {}
|
||||
|
||||
out.append({
|
||||
|
||||
"id": s.id,
|
||||
|
||||
"name": s.name,
|
||||
|
||||
"description": s.description or "",
|
||||
|
||||
"transport": s.transport_type or "streamable_http",
|
||||
|
||||
"endpoint": s.url or "",
|
||||
|
||||
"url": s.url or "",
|
||||
|
||||
"headers": headers,
|
||||
|
||||
})
|
||||
|
||||
return out
|
||||
|
||||
def _list_enabled_mcp_servers(db: Session) -> list:
|
||||
"""返回平台所有已启用(is_enabled=True)的 MCP 服务清单,供 AI 准确回答"平台有哪些 MCP"。"""
|
||||
servers = db.query(MCPServer).filter(MCPServer.is_enabled == True).all()
|
||||
out = []
|
||||
for s in servers:
|
||||
try:
|
||||
headers = json.loads(s.headers) if isinstance(s.headers, str) else (s.headers or {})
|
||||
except Exception:
|
||||
headers = {}
|
||||
out.append({
|
||||
"id": s.id,
|
||||
"name": s.name,
|
||||
"description": s.description or "",
|
||||
"transport": s.transport_type or "streamable_http",
|
||||
"endpoint": s.url or "",
|
||||
"url": s.url or "",
|
||||
"headers": headers,
|
||||
})
|
||||
return out
|
||||
File diff suppressed because it is too large.
Load diff
File diff suppressed because it is too large.
Load diff
File diff suppressed because it is too large.
Load diff
@@ -0,0 +1,378 @@
|
||||
# Auto-extracted from chat.py during Phase 1 refactor (Strangler Fig).
|
||||
# Source of truth: scripts/phase1_extract.py. Do not hand-merge back.
|
||||
|
||||
from fastapi import APIRouter, Depends, HTTPException, Query, Response
|
||||
from sqlalchemy.orm import Session
|
||||
from sqlalchemy import and_, desc, asc, func
|
||||
from pydantic import BaseModel, Field
|
||||
from typing import Optional
|
||||
import uuid
|
||||
from datetime import datetime, timezone, timedelta
|
||||
import re
|
||||
import json
|
||||
import asyncio
|
||||
import math
|
||||
import httpx
|
||||
import os
|
||||
import logging
|
||||
from ..database import get_db, SessionLocal
|
||||
from ..models import (
|
||||
|
||||
|
||||
|
||||
User, Conversation, Message, AIModel, ModelProvider, TokenUsageLog,
|
||||
|
||||
|
||||
|
||||
KnowledgeBase, KnowledgeDocument, KnowledgeDocumentChunk, EvolutionSkill,
|
||||
|
||||
|
||||
|
||||
EvolutionTrajectory, MarketApp, AppPurchase, AppInstallation,
|
||||
|
||||
|
||||
|
||||
MCPServer,
|
||||
|
||||
|
||||
|
||||
gen_uuid
|
||||
|
||||
|
||||
|
||||
)
|
||||
from ..auth import get_current_user
|
||||
from ..platform_knowledge import get_platform_knowledge
|
||||
from ..mcp_client import list_mcp_tools, call_mcp_tool
|
||||
from ..middleware.log_service import LogService
|
||||
from ..capability_registry import resolve_auto_model
|
||||
|
||||
logger = logging.getLogger(__name__)
|
||||
|
||||
CST_TZ = timezone(timedelta(hours=8))
|
||||
|
||||
def cst_now():
|
||||
|
||||
|
||||
|
||||
"""Get current time in CST timezone"""
|
||||
|
||||
|
||||
|
||||
return datetime.now(CST_TZ)
|
||||
|
||||
def _html_escape(s: str) -> str:
|
||||
|
||||
|
||||
|
||||
return (str(s).replace("&", "&").replace("<", "<").replace(">", ">")
|
||||
|
||||
|
||||
|
||||
.replace('"', """).replace("'", "'"))
|
||||
|
||||
def _outbound_proxy() -> str:
|
||||
|
||||
|
||||
|
||||
"""出网代理地址(兜底)。默认经 172.18.0.1:7890 智能路由代理;
|
||||
|
||||
|
||||
|
||||
运行时若未注入代理环境变量则回退到该地址。"""
|
||||
|
||||
|
||||
|
||||
import os
|
||||
|
||||
|
||||
|
||||
return os.environ.get("HTTPS_PROXY") or os.environ.get("HTTP_PROXY") or "http://172.18.0.1:7890"
|
||||
|
||||
_HK_PROXY = "http://qyyfw:XHR521yhf@103.242.14.208:11495"
|
||||
|
||||
def _proxy_for_url(url: str) -> str | None:
|
||||
|
||||
|
||||
|
||||
"""按目标域名选择是否走代理。
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
2026-07-15 调整:AGNES(apihub.agnes-ai.com / platform-outputs.agnes-ai.space)
|
||||
|
||||
|
||||
|
||||
经用户实测走香港代理 103.242.14.208:11495 延迟更低、更稳定,故改为走 HK 代理
|
||||
|
||||
|
||||
|
||||
(带鉴权),不再直连,也不经智能路由代理(172.18.0.1:7890 对 AGNES 返 502)。
|
||||
|
||||
|
||||
|
||||
其余国内/平台域名直连,真正的境外域名走 _outbound_proxy()。
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
|
||||
注意:httpx 默认 trust_env=True 会忽略 proxy 参数而改用环境变量 HTTP_PROXY,
|
||||
|
||||
|
||||
|
||||
故调用方必须同时传 trust_env=False 才能真正走此处指定的代理。
|
||||
|
||||
|
||||
|
||||
"""
|
||||
|
||||
|
||||
|
||||
import os
|
||||
|
||||
|
||||
|
||||
from urllib.parse import urlparse
|
||||
|
||||
|
||||
|
||||
host = (urlparse(url).hostname or "").lower()
|
||||
|
||||
|
||||
|
||||
if "agnes-ai.com" in host or "agnes-ai.space" in host:
|
||||
|
||||
|
||||
|
||||
return _HK_PROXY
|
||||
|
||||
|
||||
|
||||
# 平台自有域名(含自建 Dify/n8n 等)必须直连,不经出网代理。
|
||||
|
||||
|
||||
|
||||
if "qihuyun.com" in host:
|
||||
|
||||
|
||||
|
||||
return None
|
||||
|
||||
|
||||
|
||||
# 尊重 NO_PROXY:国内域名(硅基流动/DeepSeek/小米MoE 等)直连更稳定,
|
||||
|
||||
|
||||
|
||||
# 避免经易波动的智能路由代理 172.18.0.1:7890(对国内域名偶发连接重置)。
|
||||
|
||||
|
||||
|
||||
_no_proxy = [h.strip().lower() for h in os.environ.get("NO_PROXY", "").split(",") if h.strip()]
|
||||
|
||||
|
||||
|
||||
for _np in _no_proxy:
|
||||
|
||||
|
||||
|
||||
if host == _np or host.endswith("." + _np) or _np in host:
|
||||
|
||||
|
||||
|
||||
return None
|
||||
|
||||
|
||||
|
||||
return _outbound_proxy()
|
||||
|
||||
def _extract_text_from_json(obj) -> str:
|
||||
|
||||
|
||||
|
||||
"""从 n8n / 通用 webhook 的任意 JSON 响应中尽力抽取文本回答。"""
|
||||
|
||||
|
||||
|
||||
if obj is None:
|
||||
|
||||
|
||||
|
||||
return ""
|
||||
|
||||
|
||||
|
||||
if isinstance(obj, str):
|
||||
|
||||
|
||||
|
||||
return obj
|
||||
|
||||
|
||||
|
||||
if isinstance(obj, (int, float, bool)):
|
||||
|
||||
|
||||
|
||||
return str(obj)
|
||||
|
||||
|
||||
|
||||
if isinstance(obj, list):
|
||||
|
||||
|
||||
|
||||
# 取第一个可抽取到文本的元素
|
||||
|
||||
|
||||
|
||||
for it in obj:
|
||||
|
||||
|
||||
|
||||
t = _extract_text_from_json(it)
|
||||
|
||||
|
||||
|
||||
if t:
|
||||
|
||||
|
||||
|
||||
return t
|
||||
|
||||
|
||||
|
||||
return ""
|
||||
|
||||
|
||||
|
||||
if isinstance(obj, dict):
|
||||
|
||||
|
||||
|
||||
# 常见回答字段优先级
|
||||
|
||||
|
||||
|
||||
for k in ("output", "text", "answer", "response", "reply",
|
||||
|
||||
|
||||
|
||||
"message", "content", "result", "data"):
|
||||
|
||||
|
||||
|
||||
if k in obj:
|
||||
|
||||
|
||||
|
||||
t = _extract_text_from_json(obj[k])
|
||||
|
||||
|
||||
|
||||
if t:
|
||||
|
||||
|
||||
|
||||
return t
|
||||
|
||||
|
||||
|
||||
# 兜底:整体序列化
|
||||
|
||||
|
||||
|
||||
try:
|
||||
|
||||
|
||||
|
||||
return json.dumps(obj, ensure_ascii=False)
|
||||
|
||||
|
||||
|
||||
except Exception:
|
||||
|
||||
|
||||
|
||||
return str(obj)
|
||||
|
||||
|
||||
|
||||
return str(obj)
|
||||
|
||||
class CreateConversationRequest(BaseModel):
|
||||
|
||||
|
||||
|
||||
title: str = Field(default="新对话")
|
||||
|
||||
|
||||
|
||||
model_id: Optional[str] = None
|
||||
|
||||
class UpdateConversationRequest(BaseModel):
|
||||
|
||||
|
||||
|
||||
title: Optional[str] = None
|
||||
|
||||
|
||||
|
||||
status: Optional[str] = None # active, archived
|
||||
|
||||
|
||||
|
||||
pinned: Optional[bool] = None
|
||||
|
||||
|
||||
|
||||
agent_ids: Optional[list] = None
|
||||
|
||||
|
||||
|
||||
skill_ids: Optional[list] = None
|
||||
|
||||
|
||||
|
||||
knowledge_base_ids: Optional[list] = None
|
||||
|
||||
class SendMessageRequest(BaseModel):
|
||||
|
||||
|
||||
|
||||
content: str
|
||||
|
||||
|
||||
|
||||
model_id: Optional[str] = None
|
||||
|
||||
|
||||
|
||||
skill_ids: Optional[list] = None
|
||||
|
||||
|
||||
|
||||
agent_ids: Optional[list] = None
|
||||
|
||||
|
||||
|
||||
knowledge_base_ids: Optional[list] = None
|
||||
|
||||
|
||||
|
||||
mcp_ids: Optional[list] = None
|
||||
|
||||
|
||||
|
||||
attachments: Optional[list] = None
|
||||
|
||||
|
||||
|
||||
use_tools: Optional[bool] = False
|
||||
@@ -1,40 +1,46 @@
|
||||
"""Phase 0 回归用例 R1/R2/R3 — 固化三个反复出现的问题。
|
||||
"""Phase 1 回归用例 R1/R2/R3 — 固化三个反复出现的问题(适配 chat.py 拆分后)。
|
||||
|
||||
R1: 技能列表不得被硬性截断为 15 条(微信 Bot 曾只返回 15 个技能)。
|
||||
R2: 显式选中技能/MCP 时,system prompt 必须注入「最高优先级覆盖规则」。
|
||||
R3: 媒体生成(图像/视频)必须有超时保护,超时后回写用户可懂的提示而非永久挂起。
|
||||
|
||||
说明:在 chat.py 巨石(10070 行)尚未拆分前,R1/R2 采用「源码契约」守卫,
|
||||
这是最低风险且能切实挡住回归的方式;R3 为行为级单元测试(mock 内部生成调用模拟超时)。
|
||||
Phase 1 抽出 chat_media / chat_mcp / chat_skill 干净 seam 后,R1/R2 应升级为行为测试。
|
||||
Phase 1 已将 chat.py 巨石拆分为 chat_{utils,media,agents,skill,mcp,context,knowledge,service}.py,
|
||||
因此本文件采用「扫描整个 app/api 包」做源码契约守卫,并对行为级用例从正确的子模块导入。
|
||||
"""
|
||||
import asyncio
|
||||
import os
|
||||
import glob
|
||||
from unittest.mock import MagicMock
|
||||
|
||||
import pytest
|
||||
|
||||
_HERE = os.path.dirname(os.path.abspath(__file__))
|
||||
CHAT_PY = os.path.abspath(os.path.join(_HERE, "..", "app", "api", "chat.py"))
|
||||
API_DIR = os.path.abspath(os.path.join(_HERE, "..", "app", "api"))
|
||||
|
||||
|
||||
def _read_chat_source():
|
||||
with open(CHAT_PY, "r", encoding="utf-8") as f:
|
||||
return f.read()
|
||||
def _read_api_package():
|
||||
parts = []
|
||||
for p in sorted(glob.glob(os.path.join(API_DIR, "chat*.py"))):
|
||||
with open(p, "r", encoding="utf-8") as f:
|
||||
parts.append(f.read())
|
||||
return "\n".join(parts)
|
||||
|
||||
|
||||
# ---------- R1 ----------
|
||||
def test_r1_skill_list_not_hard_capped_at_15():
|
||||
"""微信 Bot 技能列表曾因 limit(15) 只返回 15 个,必须防止回归。"""
|
||||
src = _read_chat_source()
|
||||
src = _read_api_package()
|
||||
assert "limit(15)" not in src, "检测到 limit(15),技能列表可能被硬性截断"
|
||||
assert "LIMIT 15" not in src, "检测到 LIMIT 15,技能列表可能被硬性截断"
|
||||
|
||||
|
||||
# ---------- R2 ----------
|
||||
def test_r2_explicit_selection_override_rule_present():
|
||||
"""显式选中技能/MCP 时,system prompt 必须包含最高优先级覆盖规则。"""
|
||||
src = _read_chat_source()
|
||||
"""显式选中技能/MCP 时,system prompt 必须包含最高优先级覆盖规则。
|
||||
|
||||
注:_build_system_prompt 已随 Phase 1 拆分迁入 chat_context.py,故扫描整个 api 包。
|
||||
"""
|
||||
src = _read_api_package()
|
||||
assert "最高优先级覆盖规则" in src, "显式选中工具的覆盖规则文本缺失"
|
||||
assert "只基于上文提供的该工具具体资料作答" in src, "覆盖规则应要求只基于已选工具资料作答"
|
||||
assert "平台知识库" in src and "不要引用" in src, "覆盖规则应明确禁止引用平台知识库通用描述"
|
||||
@@ -45,37 +51,36 @@ def test_r2_explicit_selection_override_rule_present():
|
||||
async def test_r3_media_generation_timeout_returns_user_message(monkeypatch):
|
||||
"""媒体生成超时必须回写用户可懂的超时提示,而非让前端永远『生成中…』。
|
||||
|
||||
注:_generate_media_task 是后台 fire-and-forget 任务(通过 asyncio.create_task 调用),
|
||||
本身返回 None,超时保护体现在把提示回写到消息 content。故此处断言副作用。
|
||||
_generate_media_task 位于 chat_media(经 _send_message_core 的 asyncio.create_task 调用),
|
||||
本身是 fire-and-forget、返回 None,超时保护体现在把提示回写到消息 content。
|
||||
"""
|
||||
from app.api import chat as chat_mod
|
||||
from app.api import chat_media as media_mod
|
||||
|
||||
class FakeModel:
|
||||
id = "m_test"
|
||||
capabilities = ["image"] # 图像 -> 90s 超时
|
||||
|
||||
msg_mock = MagicMock() # 捕获被回写的消息对象
|
||||
|
||||
fake_session = MagicMock()
|
||||
|
||||
def _query_side_effect(model_cls):
|
||||
q = MagicMock()
|
||||
if model_cls is chat_mod.AIModel:
|
||||
if model_cls is media_mod.AIModel:
|
||||
q.filter.return_value.first.return_value = FakeModel()
|
||||
elif model_cls is chat_mod.Message:
|
||||
elif model_cls is media_mod.Message:
|
||||
q.filter.return_value.first.return_value = msg_mock
|
||||
return q
|
||||
|
||||
fake_session.query.side_effect = _query_side_effect
|
||||
monkeypatch.setattr(chat_mod, "SessionLocal", lambda: fake_session)
|
||||
monkeypatch.setattr(media_mod, "SessionLocal", lambda: fake_session)
|
||||
|
||||
# 内部生成调用直接抛出 TimeoutError,等价模拟「生成耗时超过超时阈值」
|
||||
async def _hang(db, ai_model, prompt, user_id):
|
||||
raise asyncio.TimeoutError()
|
||||
|
||||
monkeypatch.setattr(chat_mod, "_generate_media", _hang)
|
||||
monkeypatch.setattr(media_mod, "_generate_media", _hang)
|
||||
|
||||
await chat_mod._generate_media_task("msg1", "m_test", "画一只猫", "u1")
|
||||
await media_mod._generate_media_task("msg1", "m_test", "画一只猫", "u1")
|
||||
|
||||
# 副作用:消息 content 被回写为用户可懂的超时提示
|
||||
assert msg_mock.content is not None, "超时后必须回写消息内容,不能让前端永远『生成中…』"
|
||||
|
||||
Reference in new issue
Block a user