mirror of
https://github.com/ThearchyHelios/AgentLab.git
synced 2026-11-15 02:21:58 +00:00
cc608f56ec
「跑完了」和「答得对」之间一直没有任何一层。库里 264 条 succeeded 里, 14 条带着 warn、4 条 agent 步数用满,而交到用户手上的答案对此只字不提。 最糟的一次(run dd9927e6)交出去的干脆是「本轮只执行了第一个工具」—— 一句内部管道文案冒充结论。 出具体系(issuance.py)本来就是干这个的,但它挂在输出节点的可选契约上, 还要配口径卡提供受控指标集;问数据页生成的图基本都是 input → agent → output, 走不到那条路。所以补一层轻的。 **干净的运行一次模型都不调。** 先用规则扫一遍事件流(engine/review.py:scan), 没扫出东西就到此为止。实测 15ms 返回。这是整层设计的前提,测试里用 「monkeypatch 一个会抛异常的模型,断言它没被碰过」来守。 **分类挂在 code 上,不是中文消息串。** 12 处 warn 发射点都带上 code。 这样「流式失败,回退非流式」能归成 benign 直接跳过——它对答案内容零影响, 为它叫一次模型等于每次网络抖一下就多花一笔;而「步数用满」归 broken。 原来这两者在事件流里长得一模一样。工具报错之后同一个工具又跑成了, 算 degraded 不算 broken——否则参数自愈每修一次就报一次「答案不可信」。 **复核不许现编数字。** 它手上只有原答案和一份异常清单,没有任何原始数据, 写出来的新数字只能是凭空来的。直接复用 issuance.trace_numbers,把原答案的 数字当受控指标集,对不上就整条退回原答案、只留说明。硬拦截,不是提示。 **规则管重试资格,模型只有否决权。** 一开始写成两边取并,实测 loop_limit 时模型判 retry=true——而重跑一遍循环还是会撞上限,白烧一次建图加一次跑图。 反过来之后,「检索什么都没查到」这类也不会被模型强推重跑。 **重试带着失败的那张图,不是推倒重来。** 原来传 base_graph=null,模型只看到 「原问题 + 一句失败原因」,很可能原样再建出同一张坏图。而且它连自己上次建了 什么都看不到——会话历史按 status != running 过滤,正在重试的这一轮恰好被 排除在外。现在交回去走后端的改图分支,调大步数、改正参数就是改一个字段。 上限一次。 **答案等复核完成再显示。** 端到端抓到的:答案先摆出来,十秒后说明才追上去—— 这十秒里用户读的是一个没人核对过的结论,正是要修的那件事本身。改成落库立刻 (防关标签页丢结果)、显示等复核,并给复核加 25s 上限。说明排在成果上方, broken 用报警色、degraded 用普通边框——混成一个样子用户就学会了一概忽略。 复核结论跟着轮次落库,刷新后还在;也进下一轮的对话历史,否则模型会拿一个 已知不可信的结论当事实继续往下推。 验证:后端 242 passed(新增 test_review.py 25 条);tsc 干净; check-decode / check-stream / check-ui 全过,check-stream 新增 9 条渲染断言。 两条端到端都在真浏览器 + 真模型上跑通:检索降级那次说明正确排在答案上方; 重试那次建图请求 2 次、第二次 base_graph 带着失败的 3 个节点、第一版没漏出去。 复核结果不是事件流,decode.ts 翻译不到它,所以渲染测试放在 check-stream。
207 lines
10 KiB
JavaScript
207 lines
10 KiB
JavaScript
// AssistantStream 的渲染回归检查。
|
|
//
|
|
// 解码器的检查(check-decode.mjs)守的是"翻译对不对",这里守的是"画出来
|
|
// 对不对"——两者能各自通过而合起来是坏的:Step 里字段都对,组件却把它渲染成
|
|
// 一屏转义 JSON,或者在 360px 窄栏里把页面撑得横向滚动。
|
|
//
|
|
// 用真浏览器跑真组件,数据还是 fixtures.json 里那批真实事件。
|
|
// 跑之前前端得起着:./scripts/dev.sh
|
|
import { chromium } from '../frontend/node_modules/playwright-core/index.mjs'
|
|
|
|
const WEB = process.env.AGENTLAB_WEB ?? 'http://localhost:5273'
|
|
// 不用 playwright install:它既下不动也会动到已有缓存。系统 Chrome 就够了。
|
|
const CHROME = process.env.CHROME_PATH
|
|
?? '/Applications/Google Chrome.app/Contents/MacOS/Google Chrome'
|
|
const CASES = ['db', 'think', 'human', 'issue', 'failed', 'loop_approve', 'supervisor']
|
|
|
|
let failed = 0
|
|
const check = (name, cond, detail = '') => {
|
|
console.log(` ${cond ? '✓' : '✗'} ${name}${detail ? ` — ${detail}` : ''}`)
|
|
if (!cond) failed++
|
|
}
|
|
|
|
const browser = await chromium.launch({ executablePath: CHROME })
|
|
|
|
for (const kind of CASES) {
|
|
console.log(`\n=== ${kind} ===`)
|
|
const page = await browser.newPage({ viewport: { width: 1280, height: 900 } })
|
|
const errors = []
|
|
page.on('pageerror', (e) => errors.push(e.message))
|
|
page.on('console', (m) => { if (m.type() === 'error') errors.push(m.text()) })
|
|
await page.goto(`${WEB}/preview.html?case=${kind}`, { waitUntil: 'networkidle' })
|
|
await page.waitForTimeout(300)
|
|
|
|
const text = await page.locator('body').innerText()
|
|
// 404 是 favicon,和组件无关
|
|
const real = errors.filter((e) => !e.includes('404'))
|
|
check('没有运行时报错', real.length === 0, real.join(' | '))
|
|
check('有内容渲染出来', text.length > 40, `${text.length} 字`)
|
|
check('没有把转义 JSON 直接吐出来', !text.includes('\\"columns\\"'))
|
|
check('没有 0ms 噪音', !/\b0ms\b/.test(text))
|
|
check('没有裸事件名', !/\b(node|run|llm|tool)\.(started|finished|start|end)\b/.test(text))
|
|
|
|
// 360px 窄栏是画布右栏的真实宽度。这里溢出,右栏就会横向滚动
|
|
const overflow = await page.evaluate(() => {
|
|
const el = document.documentElement
|
|
return el.scrollWidth - el.clientWidth
|
|
})
|
|
check('页面不横向溢出', overflow <= 0, `${overflow}px`)
|
|
|
|
await page.close()
|
|
}
|
|
|
|
console.log('\n=== 展开交互 ===')
|
|
{
|
|
const page = await browser.newPage({ viewport: { width: 1280, height: 900 } })
|
|
await page.goto(`${WEB}/preview.html?case=db`, { waitUntil: 'networkidle' })
|
|
await page.waitForTimeout(300)
|
|
const before = await page.locator('body').innerText()
|
|
check('收起时不显示 SQL', !before.includes('SELECT'))
|
|
await page.locator('button', { hasText: '在 bi 上查询数据' }).first().click()
|
|
await page.waitForTimeout(250)
|
|
const after = await page.locator('body').innerText()
|
|
check('展开后能看到 SQL 原文', after.includes('SELECT') && after.includes('ANALYTICS'))
|
|
check('展开后能看到结果表', after.includes('factory_code'))
|
|
await page.close()
|
|
}
|
|
|
|
console.log('\n=== Markdown 渲染 ===')
|
|
{
|
|
// 用库里导出的**真实输出**,不是编的样本:模型写什么才是要渲染的,
|
|
// 按 CommonMark 规范挑测例只会测到用不上的角落
|
|
const page = await browser.newPage({ viewport: { width: 1100, height: 900 } })
|
|
const errors = []
|
|
page.on('pageerror', (e) => errors.push(e.message))
|
|
await page.goto(`${WEB}/preview.html?md=1`, { waitUntil: 'networkidle' })
|
|
await page.waitForTimeout(600)
|
|
check('没有运行时报错', errors.length === 0, errors.join(' | '))
|
|
|
|
const text = await page.locator('body').innerText()
|
|
// 「1+1 等于 **2**」原样带着星号显示——这就是改之前的样子
|
|
const stray = [...text.matchAll(/\*\*[^*\n]{1,40}\*\*|(?<!\w)`[^`\n]{1,40}`/g)].map((m) => m[0])
|
|
check('没有残留的 markdown 标记', stray.length === 0, stray.slice(0, 4).join(' '))
|
|
|
|
const el = await page.evaluate(() => ({
|
|
strong: document.querySelectorAll('strong').length,
|
|
code: document.querySelectorAll('code').length,
|
|
li: document.querySelectorAll('li').length,
|
|
table: document.querySelectorAll('table').length,
|
|
quote: document.querySelectorAll('blockquote').length,
|
|
// 模型很爱写 **`table_name`**。粗体在扫描顺序上先命中,强调内部不再
|
|
// 解析的话,那对反引号会原样显示出来
|
|
codeInStrong: document.querySelectorAll('strong code').length,
|
|
// 模型输出不可信,链接必须挡住 javascript:
|
|
badHref: [...document.querySelectorAll('a')]
|
|
.filter((a) => !/^(https?:|mailto:)/i.test(a.getAttribute('href') || '')).length,
|
|
// 只数 markdown 渲染出来的,不数整页——dev server 自己就注入 HMR 脚本
|
|
rawHtml: [...document.querySelectorAll('main, .space-y-2')]
|
|
.reduce((n, el) => n + el.querySelectorAll('script,iframe,object,embed,img').length, 0),
|
|
goodHref: [...document.querySelectorAll('a')]
|
|
.filter((a) => /^https?:/i.test(a.getAttribute('href') || '')).length,
|
|
pwned: window.__pwned === 1,
|
|
}))
|
|
check('粗体渲染出来了', el.strong > 0, `${el.strong} 处`)
|
|
check('行内代码渲染出来了', el.code > 0, `${el.code} 处`)
|
|
check('列表渲染出来了', el.li > 0, `${el.li} 条`)
|
|
check('表格渲染出来了', el.table > 0, `${el.table} 张`)
|
|
check('粗体里的代码也解析', el.codeInStrong > 0, `${el.codeInStrong} 处`)
|
|
// 样本里混了一段构造的恶意输入(真实输出不会自带攻击,但模型输出是不可信
|
|
// 内容,渲染层要么天生免疫,要么就是个 XSS 口子)
|
|
check('javascript: 链接没被渲染成可点的', el.badHref === 0, `${el.badHref} 个`)
|
|
check('正常链接照常可点', el.goodHref > 0, `${el.goodHref} 个`)
|
|
check('原始 HTML 没有变成真标签', el.rawHtml === 0, `${el.rawHtml} 个`)
|
|
check('注入的脚本没有执行', el.pwned === false)
|
|
check('原始 HTML 当文本显示出来了', text.includes('<script>'))
|
|
await page.close()
|
|
}
|
|
|
|
console.log('\n=== 长报告的折叠 ===')
|
|
{
|
|
// 折叠以前是按字符硬切的,切点落进表格中间:前面几行渲染成表格、最后半行
|
|
// 留成原始的 `| 1 | ThearchyHelios | …`。用户看到的是一份被咬掉一口的报告。
|
|
const page = await browser.newPage({ viewport: { width: 1280, height: 900 } })
|
|
await page.goto(`${WEB}/preview.html?long=1`, { waitUntil: 'networkidle' })
|
|
await page.waitForTimeout(300)
|
|
|
|
const folded = await page.locator('body').innerText()
|
|
check('折叠时说破了"后面还有"', folded.includes('后面还有'))
|
|
check('给得出展开入口', folded.includes('展开全部'))
|
|
// 关键一条:折叠后不能留下没渲染成表格的原始 markdown 行
|
|
const rawRow = folded.split('\n').find((l) => /^\s*\|.*\|/.test(l))
|
|
check('没有半截的原始表格行', !rawRow, rawRow ? `残留:${rawRow.slice(0, 48)}` : '')
|
|
check('开头的正文还在', folded.includes('总结'))
|
|
|
|
await page.locator('button', { hasText: '展开全部' }).first().click()
|
|
await page.waitForTimeout(250)
|
|
const full = await page.locator('body').innerText()
|
|
check('展开后拿得到结尾', full.includes('身兼管理员与商家双重身份'))
|
|
check('展开后表格渲染完整', full.includes('user_39'))
|
|
await page.close()
|
|
}
|
|
|
|
console.log('\n=== 没查库的那一轮 ===')
|
|
{
|
|
// 「涉及数据必须真查」是这条路径上最硬的约定,放开直接回答之后,用户得能
|
|
// 一眼分清哪些结论背后真的动了库
|
|
const page = await browser.newPage({ viewport: { width: 1280, height: 900 } })
|
|
await page.goto(`${WEB}/preview.html?noquery=1`, { waitUntil: 'networkidle' })
|
|
await page.waitForTimeout(300)
|
|
const body = await page.locator('body').innerText()
|
|
check('说破了这一条没有查库', body.includes('没有查库'))
|
|
check('声明排在结论前面',
|
|
body.indexOf('没有查库') < body.indexOf('role.level'),
|
|
`声明@${body.indexOf('没有查库')} 结论@${body.indexOf('role.level')}`)
|
|
check('答案本身照常渲染', body.includes('口径'))
|
|
await page.close()
|
|
}
|
|
|
|
console.log('\n=== 复核说明 ===')
|
|
{
|
|
// 「跑完了」和「答得对」是两回事。复核说明必须排在成果**上方**——排在下面
|
|
// 等于让人读完整个结论、信了,才知道它是在什么条件下得出的
|
|
const page = await browser.newPage({ viewport: { width: 1280, height: 900 } })
|
|
await page.goto(`${WEB}/preview.html?review=1`, { waitUntil: 'networkidle' })
|
|
await page.waitForTimeout(300)
|
|
const body = await page.locator('body').innerText()
|
|
|
|
check('说破了取数没跑通', body.includes('取数没跑通'))
|
|
check('说明排在结论前面',
|
|
body.indexOf('取数没跑通') < body.indexOf('大约 5 个'),
|
|
`说明@${body.indexOf('取数没跑通')} 结论@${body.indexOf('大约 5 个')}`)
|
|
check('答案本身照常渲染', body.includes('大约 5 个'))
|
|
check('异常清单默认收起', !body.includes('no such table'))
|
|
check('数得出有几处异常', body.includes('检测到 2 处异常'))
|
|
|
|
// broken 用报警色,degraded 用普通边框。混成一个样子,用户就学会了一概忽略
|
|
const colors = await page.evaluate(() => {
|
|
const err = getComputedStyle(document.documentElement).getPropertyValue('--err').trim()
|
|
const hit = [...document.querySelectorAll('div[style*="border-color"]')]
|
|
.map((el) => getComputedStyle(el).borderTopColor)
|
|
return { err, hit }
|
|
})
|
|
check('两档复核长得不一样', new Set(colors.hit).size > 1, colors.hit.join(' / '))
|
|
|
|
// 改写是有损的,原件得留着能对照
|
|
check('改写前的原文可以展开', body.includes('看改写前的原文'))
|
|
check('原文默认不占地方', !body.includes('平台管理员可以修改他人权限。\n'))
|
|
|
|
const overflow = await page.evaluate(() =>
|
|
document.documentElement.scrollWidth - document.documentElement.clientWidth)
|
|
check('页面不横向溢出', overflow <= 0, `${overflow}px`)
|
|
await page.close()
|
|
}
|
|
|
|
console.log('\n=== 空态 ===')
|
|
{
|
|
const page = await browser.newPage({ viewport: { width: 1280, height: 900 } })
|
|
await page.goto(`${WEB}/preview.html?case=__none__`, { waitUntil: 'networkidle' })
|
|
await page.waitForTimeout(250)
|
|
const text = await page.locator('body').innerText()
|
|
check('没有内容时给的是空态而不是白屏', text.includes('问点什么'))
|
|
await page.close()
|
|
}
|
|
|
|
await browser.close()
|
|
console.log(failed ? `\n✗ ${failed} 项未通过` : '\n✓ 助手流渲染全部通过')
|
|
process.exit(failed ? 1 : 0)
|