mirror of
https://github.com/anomalyco/opencode.git
synced 2026-07-23 18:55:37 -04:00
9e0d3976e1
Co-authored-by: Frank <frank@anoma.ly> Co-authored-by: Aarav Sareen <96787824+arvsrn@users.noreply.github.com> Co-authored-by: Brendan Allan <git@brendonovich.dev> Co-authored-by: opencode-agent[bot] <opencode-agent[bot]@users.noreply.github.com> Co-authored-by: Jack <jack@anoma.ly> Co-authored-by: Brendan Allan <14191578+Brendonovich@users.noreply.github.com> Co-authored-by: Shoubhit Dash <shoubhit2005@gmail.com> Co-authored-by: opencode-agent[bot] <219766164+opencode-agent[bot]@users.noreply.github.com> Co-authored-by: James Long <longster@gmail.com> Co-authored-by: Dustin Deus <deusdustin@gmail.com> Co-authored-by: starptech <starptech@starptechs-MBP.fritz.box> Co-authored-by: Luke Parker <10430890+Hona@users.noreply.github.com> Co-authored-by: 𝓛𝓲𝓽𝓽𝓵𝓮 𝓕𝓻𝓪𝓷𝓴 <little-frank@opencord.local> Co-authored-by: Dax <mail@thdxr.com> Co-authored-by: usrnk1 <7547651+usrnk1@users.noreply.github.com> Co-authored-by: Jay <53023+jayair@users.noreply.github.com> Co-authored-by: runvip <164729189+runvip@users.noreply.github.com> Co-authored-by: opencode <opencode@sst.dev> Co-authored-by: Julian Coy <julian@ex-machina.co> Co-authored-by: Vladimir Glafirov <vglafirov@gitlab.com> Co-authored-by: Adam <2363879+adamdotdevin@users.noreply.github.com> Co-authored-by: Kit Langton <kit.langton@gmail.com> Co-authored-by: Simon Klee <hello@simonklee.dk> Co-authored-by: Jay <air@live.ca> Co-authored-by: David Hill <1879069+iamdavidhill@users.noreply.github.com>
307 lines
11 KiB
TypeScript
307 lines
11 KiB
TypeScript
import type { Page } from "@playwright/test"
|
|
import { benchmark, benchmarkDiagnostics, expect } from "../benchmark"
|
|
import {
|
|
buildInitialStreamEvent,
|
|
buildStreamDeltaEvents,
|
|
setupTimelineBenchmark,
|
|
textPartID,
|
|
} from "./session-timeline-benchmark.fixture"
|
|
import { startTimelineProfile } from "./session-timeline-profile"
|
|
import { createReviewDiffs } from "./timeline-test-helpers"
|
|
import {
|
|
collectTimelineStreamMetrics,
|
|
installTimelineStreamProbe,
|
|
startTimelineStreamProbe,
|
|
} from "./session-timeline-stream-probe"
|
|
|
|
type TimelineStreamOptions = {
|
|
newLayoutDesigns?: boolean
|
|
reviewDiffs?: boolean
|
|
reviewPane?: boolean
|
|
}
|
|
|
|
type ReviewPaneSample = {
|
|
observedAtMs: number
|
|
panelVisible: boolean
|
|
header: string
|
|
diffViewers: number
|
|
diffLines: number
|
|
codeBlocks: number
|
|
ready: boolean
|
|
}
|
|
|
|
type ReviewPaneProbe = {
|
|
samples: ReviewPaneSample[]
|
|
start: () => void
|
|
stop: () => void
|
|
}
|
|
|
|
const reviewReadyStreak = 3
|
|
|
|
benchmark.describe("performance: session timeline streaming", () => {
|
|
benchmark("streams assistant text without remounting or oscillating", async ({ page, report }) => {
|
|
benchmark.setTimeout(Number(process.env.TIMELINE_COMPLETION_TIMEOUT_MS ?? 420_000) + 60_000)
|
|
const result = await runTimelineStreamBenchmark(page, {})
|
|
report(result.metrics, result.context)
|
|
})
|
|
|
|
benchmark("streams assistant text in v2 with review pane closed", async ({ page, report }) => {
|
|
benchmark.setTimeout(Number(process.env.TIMELINE_COMPLETION_TIMEOUT_MS ?? 420_000) + 60_000)
|
|
const result = await runTimelineStreamBenchmark(page, { newLayoutDesigns: true })
|
|
report(result.metrics, result.context)
|
|
})
|
|
|
|
benchmark("streams assistant text in v2 with review diffs and pane closed", async ({ page, report }) => {
|
|
benchmark.setTimeout(Number(process.env.TIMELINE_COMPLETION_TIMEOUT_MS ?? 420_000) + 60_000)
|
|
const result = await runTimelineStreamBenchmark(page, { newLayoutDesigns: true, reviewDiffs: true })
|
|
report(result.metrics, result.context)
|
|
})
|
|
|
|
benchmark("streams assistant text in v2 with review pane open", async ({ page, report }) => {
|
|
benchmark.setTimeout(Number(process.env.TIMELINE_COMPLETION_TIMEOUT_MS ?? 420_000) + 60_000)
|
|
const result = await runTimelineStreamBenchmark(page, { newLayoutDesigns: true, reviewPane: true })
|
|
report(result.metrics, result.context)
|
|
})
|
|
})
|
|
|
|
benchmark.describe("performance: review pane", () => {
|
|
benchmark("loads v2 review diffs and switches active files", async ({ page, report }) => {
|
|
benchmark.setTimeout(240_000)
|
|
const historyTurns = Number(process.env.REVIEW_PANE_HISTORY_TURNS ?? 72)
|
|
const diffs = createReviewDiffs()
|
|
const fixture = await setupTimelineBenchmark(page, {
|
|
historyTurns,
|
|
eventBatch: 1,
|
|
newLayoutDesigns: true,
|
|
vcsDiff: diffs,
|
|
})
|
|
|
|
fixture.transport.enqueue(buildInitialStreamEvent(1))
|
|
await expect(fixture.text).toBeVisible()
|
|
await expect(fixture.text).toContainText("Implementation plan")
|
|
await fixture.scrollToBottom()
|
|
await fixture.waitForStableGeometry()
|
|
|
|
const open = await measureReviewPaneLoad(page, diffs[0]!.file)
|
|
const switches = []
|
|
for (const diff of diffs.slice(1, 4)) switches.push(await measureReviewNextFile(page, diff.file))
|
|
|
|
report(
|
|
{
|
|
open,
|
|
switches,
|
|
},
|
|
{
|
|
historyTurns,
|
|
reviewDiffs: diffs.length,
|
|
},
|
|
)
|
|
})
|
|
})
|
|
|
|
async function runTimelineStreamBenchmark(page: Page, options: TimelineStreamOptions) {
|
|
const completionTimeoutMs = Number(process.env.TIMELINE_COMPLETION_TIMEOUT_MS ?? 420_000)
|
|
const cpuThrottle = Number(process.env.TIMELINE_CPU_THROTTLE ?? 30)
|
|
const deltaCount = Number(process.env.TIMELINE_DELTA_COUNT ?? 160)
|
|
const historyTurns = Number(process.env.TIMELINE_HISTORY_TURNS ?? 320)
|
|
const eventBatch = Number(process.env.TIMELINE_EVENT_BATCH ?? 1)
|
|
const minimal = process.env.TIMELINE_MINIMAL === "1"
|
|
const profileCPU = process.env.TIMELINE_CPU_PROFILE === "1"
|
|
const profileVisual = !minimal && profileCPU && process.env.TIMELINE_VISUAL_PROFILE !== "0"
|
|
const diffs = options.reviewDiffs || options.reviewPane ? createReviewDiffs() : undefined
|
|
const fixture = await setupTimelineBenchmark(page, {
|
|
historyTurns,
|
|
eventBatch,
|
|
newLayoutDesigns: options.newLayoutDesigns,
|
|
// Turn diffs exercise timeline data cost; the pane-open scenario serves the same
|
|
// diffs through the default git mode so it works across review implementations.
|
|
turnDiffs: options.reviewDiffs ? diffs : undefined,
|
|
vcsDiff: options.reviewPane ? diffs : undefined,
|
|
})
|
|
|
|
fixture.transport.enqueue(buildInitialStreamEvent(deltaCount))
|
|
const contentStart = performance.now()
|
|
await expect(fixture.text).toBeVisible()
|
|
await expect(fixture.text).toContainText("Implementation plan")
|
|
const initialContentObservedMs = performance.now() - contentStart
|
|
await fixture.scrollToBottom()
|
|
await fixture.waitForStableGeometry()
|
|
|
|
const reviewPane = options.reviewPane && diffs ? await measureReviewPaneLoad(page, diffs[0]!.file) : undefined
|
|
if (reviewPane) await fixture.waitForStableGeometry()
|
|
|
|
const profile = await startTimelineProfile(page, { cpuThrottle, profileCPU })
|
|
await installTimelineStreamProbe(page, { textPartID, finalIndex: deltaCount, profileVisual, minimal })
|
|
const deltas = buildStreamDeltaEvents(deltaCount)
|
|
await startTimelineStreamProbe(page)
|
|
fixture.transport.enqueue(deltas)
|
|
|
|
await page.waitForFunction(
|
|
(finalIndex) =>
|
|
(
|
|
window as Window & {
|
|
__timelineStreamBenchmark?: { applied: { index: number }[] }
|
|
}
|
|
).__timelineStreamBenchmark?.applied.some((value) => value.index === finalIndex),
|
|
deltaCount,
|
|
{ timeout: completionTimeoutMs },
|
|
)
|
|
await expect(fixture.text).toContainText("benchmark-complete")
|
|
await expect(fixture.text).toContainText("Streaming")
|
|
await fixture.waitForStableGeometry()
|
|
const metrics = await collectTimelineStreamMetrics(page, {
|
|
textPartID,
|
|
finalIndex: deltaCount,
|
|
navigations: benchmarkDiagnostics(page).navigations,
|
|
})
|
|
const delivered = deltas.length - fixture.transport.pendingCount()
|
|
await profile.stop()
|
|
|
|
const result = {
|
|
metrics: {
|
|
endToEndInitialContentObservedMs: initialContentObservedMs,
|
|
...metrics,
|
|
deliveredDeltas: delivered,
|
|
pendingDeltas: fixture.transport.pendingCount(),
|
|
reviewPane: reviewPane ?? null,
|
|
},
|
|
context: {
|
|
cpuThrottle,
|
|
profileCPU,
|
|
profileVisual,
|
|
minimal,
|
|
queuedDeltas: deltas.length,
|
|
historyTurns,
|
|
eventBatch,
|
|
newLayoutDesigns: options.newLayoutDesigns === true,
|
|
reviewPane: options.reviewPane === true ? "open" : "closed",
|
|
reviewDiffs: diffs?.length ?? 0,
|
|
},
|
|
}
|
|
|
|
await profile.reset()
|
|
return result
|
|
}
|
|
|
|
async function measureReviewPaneLoad(page: Page, file: string) {
|
|
// Default git mode reads the mocked /vcs/diff data, so opening the pane is enough
|
|
// and the flow works across review pane implementations.
|
|
await installReviewPaneProbe(page, { file })
|
|
await startReviewPaneProbe(page)
|
|
await page.getByRole("button", { name: "Toggle review" }).click()
|
|
await expect(page.locator("#review-panel")).toBeVisible()
|
|
return collectReviewPaneProbe(page)
|
|
}
|
|
|
|
async function measureReviewNextFile(page: Page, file: string) {
|
|
await installReviewPaneProbe(page, { file })
|
|
await startReviewPaneProbe(page)
|
|
await page.getByRole("button", { name: "Next file" }).click()
|
|
return collectReviewPaneProbe(page)
|
|
}
|
|
|
|
async function installReviewPaneProbe(page: Page, input: { file: string }) {
|
|
await page.evaluate((input) => {
|
|
const samples: ReviewPaneSample[] = []
|
|
const basename = input.file.split(/[\\/]/).at(-1) ?? input.file
|
|
let started: number | undefined
|
|
let running = true
|
|
|
|
const paneState = () => {
|
|
const panel = document.querySelector<HTMLElement>("#review-panel")
|
|
const review = panel?.querySelector<HTMLElement>('[data-component="session-review-v2"]')
|
|
const rect = (review ?? panel)?.getBoundingClientRect()
|
|
const text = panel?.textContent ?? ""
|
|
const previewHeader = panel?.querySelector<HTMLElement>(
|
|
'[data-slot="session-review-v2-file-header"]',
|
|
)?.textContent
|
|
const header = previewHeader ?? text
|
|
const viewers = panel ? [...panel.querySelectorAll<HTMLElement>('[data-component="file"][data-mode="diff"]')] : []
|
|
const codeBlocks = panel?.querySelectorAll("code").length ?? 0
|
|
const diffLines = viewers.reduce(
|
|
(sum, viewer) =>
|
|
sum +
|
|
(viewer.shadowRoot?.querySelectorAll("[data-line]").length ?? viewer.querySelectorAll("[data-line]").length),
|
|
0,
|
|
)
|
|
const panelVisible =
|
|
!!panel && panel.getAttribute("aria-hidden") !== "true" && !!rect && rect.width > 0 && rect.height > 0
|
|
return {
|
|
panelVisible,
|
|
header: header.slice(0, 500),
|
|
diffViewers: viewers.length,
|
|
diffLines,
|
|
codeBlocks,
|
|
ready:
|
|
panelVisible &&
|
|
header.includes(basename) &&
|
|
(viewers.length > 0 || text.includes("+3") || diffLines > 0 || codeBlocks > 0),
|
|
}
|
|
}
|
|
|
|
const sample = () => {
|
|
if (!running || started === undefined) return
|
|
requestAnimationFrame(() => {
|
|
setTimeout(() => {
|
|
if (!running || started === undefined) return
|
|
samples.push({ observedAtMs: performance.now() - started, ...paneState() })
|
|
if (performance.now() - started < 10_000) sample()
|
|
}, 0)
|
|
})
|
|
}
|
|
|
|
;(window as Window & { __reviewPaneProbe?: ReviewPaneProbe }).__reviewPaneProbe = {
|
|
samples,
|
|
start: () => {
|
|
started = performance.now()
|
|
performance.mark("opencode.review-pane.click")
|
|
sample()
|
|
},
|
|
stop: () => {
|
|
running = false
|
|
},
|
|
}
|
|
}, input)
|
|
}
|
|
|
|
async function startReviewPaneProbe(page: Page) {
|
|
await page.evaluate(() => {
|
|
;(window as Window & { __reviewPaneProbe?: ReviewPaneProbe }).__reviewPaneProbe!.start()
|
|
})
|
|
}
|
|
|
|
async function collectReviewPaneProbe(page: Page) {
|
|
await page.waitForFunction((streak) => {
|
|
const samples = (window as Window & { __reviewPaneProbe?: ReviewPaneProbe }).__reviewPaneProbe?.samples
|
|
if (!samples) return false
|
|
return samples.some((_, index) => {
|
|
const stable = samples.slice(index, index + streak)
|
|
return stable.length === streak && stable.every((sample) => sample.ready)
|
|
})
|
|
}, reviewReadyStreak)
|
|
|
|
const samples = await page.evaluate(() => {
|
|
const probe = (window as Window & { __reviewPaneProbe?: ReviewPaneProbe }).__reviewPaneProbe!
|
|
probe.stop()
|
|
return probe.samples
|
|
})
|
|
return { summary: summarizeReviewPaneSamples(samples), samples }
|
|
}
|
|
|
|
function summarizeReviewPaneSamples(samples: ReviewPaneSample[]) {
|
|
const firstReady = samples.find((sample) => sample.ready)
|
|
const stableIndex = samples.findIndex((_, index) => {
|
|
const stable = samples.slice(index, index + reviewReadyStreak)
|
|
return stable.length === reviewReadyStreak && stable.every((sample) => sample.ready)
|
|
})
|
|
return {
|
|
samples: samples.length,
|
|
firstReadyObservedMs: firstReady?.observedAtMs ?? null,
|
|
stableReadyObservedMs: stableIndex === -1 ? null : samples[stableIndex + reviewReadyStreak - 1]!.observedAtMs,
|
|
notReadySamples: samples.filter((sample) => !sample.ready).length,
|
|
maxDiffViewers: Math.max(0, ...samples.map((sample) => sample.diffViewers)),
|
|
maxDiffLines: Math.max(0, ...samples.map((sample) => sample.diffLines)),
|
|
maxCodeBlocks: Math.max(0, ...samples.map((sample) => sample.codeBlocks)),
|
|
}
|
|
}
|