fix(ui): reconcile trajectory ledger with current client contracts

This commit is contained in:
_Kerman
2026-07-28 10:23:44 +08:00
parent fb3e4effba
commit 92081094ae
12 changed files with 66 additions and 69 deletions

View File

@@ -127,21 +127,18 @@ describe('web e2e: navigation & panes over a rich seeded session', () => {
it.skipIf(MODE === 'record')('renders the trajectory ledger and opens its local record inspector', async () => {
onTestFailed(() => saveFailureShot(page, 'web-e2e-navigation-trajectory'))
await page.getByRole('tab', { name: 'Trajectory' }).click()
// Thick row rules mark the two turns; compact inline markers identify
// each step without introducing dedicated group rows.
// Turn rules partition the ledger without restoring a separate header row.
await expect.poll(() => page.locator('tr[data-turn-start="true"]').count(), { timeout: 15_000 }).toBe(2)
await expect.poll(() => page.locator('[aria-label^="Step "]').count(), { timeout: 10_000 }).toBe(3)
await expect.poll(() => page.getByRole('columnheader', { name: 'Tokens' }).count(), { timeout: 10_000 }).toBe(0)
await expect.poll(() => page.getByRole('columnheader', { name: '耗时' }).count(), { timeout: 10_000 }).toBe(0)
await expect.poll(() => page.getByRole('columnheader').count(), { timeout: 10_000 }).toBe(0)
await page.locator('tr[data-kind="tool"]').first().click()
await expect.poll(() => page.getByRole('complementary', { name: '记录详情' }).count(), { timeout: 10_000 }).toBe(1)
await page.getByRole('tab', { name: '输出' }).click()
await expect.poll(() => page.getByRole('complementary', { name: 'Event details' }).count(), { timeout: 10_000 }).toBe(1)
await page.getByRole('tab', { name: 'Result' }).click()
await expect.poll(() => page.getByText('NAVIGATION_OK', { exact: false }).count(), { timeout: 10_000 }).toBeGreaterThanOrEqual(1)
const snapshot = (await captureStableAria(page, '[class*="viewArea"]', scaffold.workspaceCwd))
.split(SEED_ID).join('{{seededId}}')
await compareOrRefreshGolden(TRAJECTORY_EXPECTED, snapshot, MODE)
await page.getByRole('complementary', { name: '记录详情' })
.getByRole('button', { name: '关闭详情' }).click()
await page.getByRole('complementary', { name: 'Event details' })
.getByRole('button', { name: 'Close details' }).click()
}, 60_000)
it.skipIf(MODE === 'record')('renders the waterfall tab with span stats and one lane per span', async () => {

View File

@@ -1,5 +1,6 @@
- text: bash
- button "关闭详情"
- text: Input
- text: Input json
- button "复制"
- code: "{ \"command\": \"echo NAVIGATION_OK\", \"description\": \"Print NAVIGATION_OK\" }"
- text: Output NAVIGATION_OK

View File

@@ -1,57 +1,50 @@
- toolbar "轨迹工具栏":
- text: 轨迹
- strong: "8"
- text: 条记录
- strong: "2"
- text: 轮
- strong: "3"
- text: 次工具
- button "收起记录"
- toolbar "Trajectory toolbar":
- text: Trajectory
- button "Collapse calls"
- button "Collapse turns"
- table:
- rowgroup:
- row "# 事件 内容":
- columnheader "#"
- columnheader "事件"
- columnheader "内容"
- rowgroup:
- 'row "记录 1USERNavScenario: first run bash to print exactly NAVIGATION_OK, then read nav-a.md and nav-b.md using two read calls in ONE assistant message, then reply with the single word FIRST_DONE and stop."':
- cell "#1"
- cell "USER"
- row "SYSTEM, Initial System Prompt":
- cell "SYSTEM"
- cell "Initial System Prompt"
- 'row "USER, NavScenario: first run bash to print exactly NAVIGATION_OK, then read nav-a.md and nav-b.md using two read calls in ONE assistant message, then reply with the single word FIRST_DONE and stop."':
- cell "Turn 1 USER"
- 'cell "NavScenario: first run bash to print exactly NAVIGATION_OK, then read nav-a.md and nav-b.md using two read calls in ONE assistant message, then reply with the single word FIRST_DONE and stop."'
- row "记录 2ASSISTANT请求调用 bash、read×2":
- cell "#2"
- cell "Step 1 ASSISTANT": S1 ASSISTANT
- cell "请求调用 bash、read×2"
- 'row "记录 3TOOLbash · {\"command\": \"echo NAVIGATION_OK\", \"description\": \"Print NAVIGATION_OK\"}" [selected]':
- cell "#3"
- 'row "Request 1, ASSISTANT, The user wants me to follow a specific navigation scenario. Let me: Run bash to print \"NAVIGATION_OK\" Read nav-a.md and nav-b.md in two read calls in ONE message Reply with \"FIRST_DONE\" Let me start with the bash command and the reads."':
- 'cell "Request #1 ASSISTANT"':
- 'button "Request #1"'
- text: ASSISTANT
- 'cell "The user wants me to follow a specific navigation scenario. Let me: Run bash to print \"NAVIGATION_OK\" Read nav-a.md and nav-b.md in two read calls in ONE message Reply with \"FIRST_DONE\" Let me start with the bash command and the reads."'
- 'row "TOOL, bash {\"command\": \"echo NAVIGATION_OK\", \"description\": \"Print NAVIGATION_OK\"}" [selected]':
- cell "TOOL"
- 'cell "bash · {\"command\": \"echo NAVIGATION_OK\", \"description\": \"Print NAVIGATION_OK\"}NAVIGATION_OK"'
- 'row "记录 4TOOLread · {\"file_path\": \"nav-a.md\"}"':
- cell "#4"
- 'cell "bash{\"command\": \"echo NAVIGATION_OK\", \"description\": \"Print NAVIGATION_OK\"}NAVIGATION_OK"'
- 'row "TOOL, read {\"file_path\": \"nav-a.md\"}"':
- cell "TOOL"
- 'cell "read · {\"file_path\": \"nav-a.md\"}<path>{{cwd}}/nav-a.md</path> <type>file</type> <content> 1: # alpha nav (End of file - total 1 lines) </content>"'
- 'row "记录 5TOOLread · {\"file_path\": \"nav-b.md\"}"':
- cell "#5"
- 'cell "read{\"file_path\": \"nav-a.md\"}<path>{{cwd}}/nav-a.md</path> <type>file</type> <content> 1: # alpha nav (End of file - total 1 lines) </content>"'
- 'row "TOOL, read {\"file_path\": \"nav-b.md\"}"':
- cell "TOOL"
- 'cell "read · {\"file_path\": \"nav-b.md\"}<path>{{cwd}}/nav-b.md</path> <type>file</type> <content> 1: # beta nav (End of file - total 1 lines) </content>"'
- row "记录 6ASSISTANTFIRST_DONE":
- cell "#6"
- cell "Step 2 ASSISTANT": S2 ASSISTANT
- 'cell "read{\"file_path\": \"nav-b.md\"}<path>{{cwd}}/nav-b.md</path> <type>file</type> <content> 1: # beta nav (End of file - total 1 lines) </content>"'
- row "Request 2, ASSISTANT, FIRST_DONE":
- 'cell "Request #2 ASSISTANT"':
- 'button "Request #2"'
- text: ASSISTANT
- cell "FIRST_DONE"
- 'row "记录 7USERReply in markdown with: a level-2 heading \"Navigation Summary\", a bulleted list of exactly two items, and a fenced code block containing echo WATERFALL. Then stop."':
- cell "#7"
- cell "USER"
- 'row "USER, Reply in markdown with: a level-2 heading \"Navigation Summary\", a bulleted list of exactly two items, and a fenced code block containing echo WATERFALL. Then stop."':
- cell "Turn 2 USER"
- 'cell "Reply in markdown with: a level-2 heading \"Navigation Summary\", a bulleted list of exactly two items, and a fenced code block containing echo WATERFALL. Then stop."'
- 'row "记录 8ASSISTANT## Navigation Summary - alpha nav - beta nav ``` echo WATERFALL ```"':
- cell "#8"
- cell "Step 1 ASSISTANT": S1 ASSISTANT
- 'cell "## Navigation Summary - alpha nav - beta nav ``` echo WATERFALL ```"'
- complementary "记录详情":
- text: "TOOL 记录 #3 Turn 1 · Step 1"
- button "关闭详情"
- tablist "记录详情":
- tab "概览"
- tab "输入"
- tab "输出" [selected]
- tab "计时"
- tabpanel "输出": NAVIGATION_OK
- row "Request 3, ASSISTANT, Navigation Summary alpha nav beta nav echo WATERFALL":
- 'cell "Request #3 ASSISTANT"':
- 'button "Request #3"'
- text: ASSISTANT
- cell "Navigation Summary alpha nav beta nav echo WATERFALL"
- complementary "Event details":
- separator "Resize event details"
- text: TOOL Turn 1 · Step 1
- button "Close details"
- tablist "Event details":
- tab "Summary"
- tab "Payload"
- tab "Result" [selected]
- tab "Schema"
- tab "Timing"
- tabpanel "Result": NAVIGATION_OK

View File

@@ -58,7 +58,6 @@ function materializeNode(
return {
kind: 'user', seq: event.seq, time: event.time,
content: event.data.content, source: event.data.source,
meta: event.data.meta,
}
case 'assistant/message':
return {
@@ -76,7 +75,6 @@ function materializeNode(
return {
kind: 'steering', seq: event.seq, time: event.time, turn: event.data.turn,
content: event.data.content, source: event.data.source,
meta: event.data.meta,
}
case 'tool/result': {
const call = callIndex.get(String(event.data.callId))

View File

@@ -955,7 +955,7 @@ function deriveModelRequests(events: readonly SessionEvent[]): readonly ModelReq
const key = modelRequestKey(sourceEvent.data.turn, sourceEvent.data.step)
const index = byStep.get(key)
const request = index === undefined ? undefined : requests[index]
if (request !== undefined && request.status === 'running') {
if (index !== undefined && request !== undefined && request.status === 'running') {
requests[index] = {
...request,
completedAt: sourceEvent.time,

View File

@@ -88,6 +88,7 @@ function mount(
if (key === 'conversation.session') {
return (
<ConversationSession
composer={null}
sessionId={SID}
SessionProvider={({ children }) => children(SID)}
useSession={useSession}

View File

@@ -72,7 +72,10 @@ function previewPrimitive(value: unknown): ReactNode {
if (typeof value === 'symbol') {
return <span className={css.otherValue}>{value.description ?? 'Symbol'}</span>
}
return <span className={css.otherValue}>{value.name || 'Function'}</span>
if (typeof value === 'function') {
return <span className={css.otherValue}>{value.name || 'Function'}</span>
}
return null
}
function previewValue(value: unknown, depth: number): ReactNode {

View File

@@ -1147,7 +1147,7 @@ function RequestTiming({
}: {
assistant: TableRecord | undefined
anchor: TableRecord | undefined
request?: TrajectoryRequestNumber
request: TrajectoryRequestNumber | undefined
}) {
if (assistant !== undefined) return <RecordTiming record={assistant} />
if (request?.startedAt !== undefined) {
@@ -1746,8 +1746,9 @@ export function TrajectoryTable({
onPointerDown={(event) => {
if (event.button !== 0) return
const details = event.currentTarget.parentElement
const split = details?.parentElement
if (details === null || split === null) return
if (details === null) return
const split = details.parentElement
if (split === null) return
const splitWidth = split.getBoundingClientRect().width
detailsResizeDrag.current = {
pointerId: event.pointerId,
@@ -1785,8 +1786,9 @@ export function TrajectoryTable({
onKeyDown={(event) => {
if (event.key !== 'ArrowLeft' && event.key !== 'ArrowRight') return
const details = event.currentTarget.parentElement
const split = details?.parentElement
if (details === null || split === null) return
if (details === null) return
const split = details.parentElement
if (split === null) return
const direction = event.key === 'ArrowLeft' ? 1 : -1
const currentDetailsWidth = details.getBoundingClientRect().width
const splitWidth = split.getBoundingClientRect().width

View File

@@ -331,7 +331,6 @@ export function deriveTrajectoryLayout(input: TrajectoryLayoutInput): readonly T
text: summarizeContent(node.content),
sourceSeq: node.seq,
messageSource: node.source,
...(node.meta === undefined ? {} : { messageMeta: node.meta }),
inputDetail: detailContent(node.content),
sourceBlocks: node.content.map(block => sourceBlock(block)),
timeSeconds: 0,

View File

@@ -130,6 +130,7 @@ function mount(slots: SlotsService, nodes: ConversationSnapshot['nodes'] = NODES
}) as unknown as ConversationSessionProps['renderSlot']
return render(
<ConversationSession
composer={null}
sessionId={SID}
SessionProvider={({ children }) => children(SID)}
useSession={useSession}

View File

@@ -20,6 +20,7 @@
"packages/client/*/tests/**/*.ts",
"packages/client/*/tests/**/*.tsx",
"packages/client/tsdown.client.ts",
"scripts/client-bundle-css.spec.ts",
"scripts/client-bundle-purity.spec.ts"
],
"references": [

View File

@@ -32,6 +32,7 @@
],
"exclude": [
"packages/client/**",
"scripts/client-bundle-css.spec.ts",
"scripts/client-bundle-purity.spec.ts"
],
"references": [