session-timeline-benchmark.spec.ts 11 KB

123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306
  1. import type { Page } from "@playwright/test"
  2. import { benchmark, benchmarkDiagnostics, expect } from "../benchmark"
  3. import {
  4. buildInitialStreamEvent,
  5. buildStreamDeltaEvents,
  6. setupTimelineBenchmark,
  7. textPartID,
  8. } from "./session-timeline-benchmark.fixture"
  9. import { startTimelineProfile } from "./session-timeline-profile"
  10. import { createReviewDiffs } from "./timeline-test-helpers"
  11. import {
  12. collectTimelineStreamMetrics,
  13. installTimelineStreamProbe,
  14. startTimelineStreamProbe,
  15. } from "./session-timeline-stream-probe"
  16. type TimelineStreamOptions = {
  17. newLayoutDesigns?: boolean
  18. reviewDiffs?: boolean
  19. reviewPane?: boolean
  20. }
  21. type ReviewPaneSample = {
  22. observedAtMs: number
  23. panelVisible: boolean
  24. header: string
  25. diffViewers: number
  26. diffLines: number
  27. codeBlocks: number
  28. ready: boolean
  29. }
  30. type ReviewPaneProbe = {
  31. samples: ReviewPaneSample[]
  32. start: () => void
  33. stop: () => void
  34. }
  35. const reviewReadyStreak = 3
  36. benchmark.describe("performance: session timeline streaming", () => {
  37. benchmark("streams assistant text without remounting or oscillating", async ({ page, report }) => {
  38. benchmark.setTimeout(Number(process.env.TIMELINE_COMPLETION_TIMEOUT_MS ?? 420_000) + 60_000)
  39. const result = await runTimelineStreamBenchmark(page, {})
  40. report(result.metrics, result.context)
  41. })
  42. benchmark("streams assistant text in v2 with review pane closed", async ({ page, report }) => {
  43. benchmark.setTimeout(Number(process.env.TIMELINE_COMPLETION_TIMEOUT_MS ?? 420_000) + 60_000)
  44. const result = await runTimelineStreamBenchmark(page, { newLayoutDesigns: true })
  45. report(result.metrics, result.context)
  46. })
  47. benchmark("streams assistant text in v2 with review diffs and pane closed", async ({ page, report }) => {
  48. benchmark.setTimeout(Number(process.env.TIMELINE_COMPLETION_TIMEOUT_MS ?? 420_000) + 60_000)
  49. const result = await runTimelineStreamBenchmark(page, { newLayoutDesigns: true, reviewDiffs: true })
  50. report(result.metrics, result.context)
  51. })
  52. benchmark("streams assistant text in v2 with review pane open", async ({ page, report }) => {
  53. benchmark.setTimeout(Number(process.env.TIMELINE_COMPLETION_TIMEOUT_MS ?? 420_000) + 60_000)
  54. const result = await runTimelineStreamBenchmark(page, { newLayoutDesigns: true, reviewPane: true })
  55. report(result.metrics, result.context)
  56. })
  57. })
  58. benchmark.describe("performance: review pane", () => {
  59. benchmark("loads v2 review diffs and switches active files", async ({ page, report }) => {
  60. benchmark.setTimeout(240_000)
  61. const historyTurns = Number(process.env.REVIEW_PANE_HISTORY_TURNS ?? 72)
  62. const diffs = createReviewDiffs()
  63. const fixture = await setupTimelineBenchmark(page, {
  64. historyTurns,
  65. eventBatch: 1,
  66. newLayoutDesigns: true,
  67. vcsDiff: diffs,
  68. })
  69. fixture.transport.enqueue(buildInitialStreamEvent(1))
  70. await expect(fixture.text).toBeVisible()
  71. await expect(fixture.text).toContainText("Implementation plan")
  72. await fixture.scrollToBottom()
  73. await fixture.waitForStableGeometry()
  74. const open = await measureReviewPaneLoad(page, diffs[0]!.file)
  75. const switches = []
  76. for (const diff of diffs.slice(1, 4)) switches.push(await measureReviewNextFile(page, diff.file))
  77. report(
  78. {
  79. open,
  80. switches,
  81. },
  82. {
  83. historyTurns,
  84. reviewDiffs: diffs.length,
  85. },
  86. )
  87. })
  88. })
  89. async function runTimelineStreamBenchmark(page: Page, options: TimelineStreamOptions) {
  90. const completionTimeoutMs = Number(process.env.TIMELINE_COMPLETION_TIMEOUT_MS ?? 420_000)
  91. const cpuThrottle = Number(process.env.TIMELINE_CPU_THROTTLE ?? 30)
  92. const deltaCount = Number(process.env.TIMELINE_DELTA_COUNT ?? 160)
  93. const historyTurns = Number(process.env.TIMELINE_HISTORY_TURNS ?? 320)
  94. const eventBatch = Number(process.env.TIMELINE_EVENT_BATCH ?? 1)
  95. const minimal = process.env.TIMELINE_MINIMAL === "1"
  96. const profileCPU = process.env.TIMELINE_CPU_PROFILE === "1"
  97. const profileVisual = !minimal && profileCPU && process.env.TIMELINE_VISUAL_PROFILE !== "0"
  98. const diffs = options.reviewDiffs || options.reviewPane ? createReviewDiffs() : undefined
  99. const fixture = await setupTimelineBenchmark(page, {
  100. historyTurns,
  101. eventBatch,
  102. newLayoutDesigns: options.newLayoutDesigns,
  103. // Turn diffs exercise timeline data cost; the pane-open scenario serves the same
  104. // diffs through the default git mode so it works across review implementations.
  105. turnDiffs: options.reviewDiffs ? diffs : undefined,
  106. vcsDiff: options.reviewPane ? diffs : undefined,
  107. })
  108. fixture.transport.enqueue(buildInitialStreamEvent(deltaCount))
  109. const contentStart = performance.now()
  110. await expect(fixture.text).toBeVisible()
  111. await expect(fixture.text).toContainText("Implementation plan")
  112. const initialContentObservedMs = performance.now() - contentStart
  113. await fixture.scrollToBottom()
  114. await fixture.waitForStableGeometry()
  115. const reviewPane = options.reviewPane && diffs ? await measureReviewPaneLoad(page, diffs[0]!.file) : undefined
  116. if (reviewPane) await fixture.waitForStableGeometry()
  117. const profile = await startTimelineProfile(page, { cpuThrottle, profileCPU })
  118. await installTimelineStreamProbe(page, { textPartID, finalIndex: deltaCount, profileVisual, minimal })
  119. const deltas = buildStreamDeltaEvents(deltaCount)
  120. await startTimelineStreamProbe(page)
  121. fixture.transport.enqueue(deltas)
  122. await page.waitForFunction(
  123. (finalIndex) =>
  124. (
  125. window as Window & {
  126. __timelineStreamBenchmark?: { applied: { index: number }[] }
  127. }
  128. ).__timelineStreamBenchmark?.applied.some((value) => value.index === finalIndex),
  129. deltaCount,
  130. { timeout: completionTimeoutMs },
  131. )
  132. await expect(fixture.text).toContainText("benchmark-complete")
  133. await expect(fixture.text).toContainText("Streaming")
  134. await fixture.waitForStableGeometry()
  135. const metrics = await collectTimelineStreamMetrics(page, {
  136. textPartID,
  137. finalIndex: deltaCount,
  138. navigations: benchmarkDiagnostics(page).navigations,
  139. })
  140. const delivered = deltas.length - fixture.transport.pendingCount()
  141. await profile.stop()
  142. const result = {
  143. metrics: {
  144. endToEndInitialContentObservedMs: initialContentObservedMs,
  145. ...metrics,
  146. deliveredDeltas: delivered,
  147. pendingDeltas: fixture.transport.pendingCount(),
  148. reviewPane: reviewPane ?? null,
  149. },
  150. context: {
  151. cpuThrottle,
  152. profileCPU,
  153. profileVisual,
  154. minimal,
  155. queuedDeltas: deltas.length,
  156. historyTurns,
  157. eventBatch,
  158. newLayoutDesigns: options.newLayoutDesigns === true,
  159. reviewPane: options.reviewPane === true ? "open" : "closed",
  160. reviewDiffs: diffs?.length ?? 0,
  161. },
  162. }
  163. await profile.reset()
  164. return result
  165. }
  166. async function measureReviewPaneLoad(page: Page, file: string) {
  167. // Default git mode reads the mocked /vcs/diff data, so opening the pane is enough
  168. // and the flow works across review pane implementations.
  169. await installReviewPaneProbe(page, { file })
  170. await startReviewPaneProbe(page)
  171. await page.getByRole("button", { name: "Toggle review" }).click()
  172. await expect(page.locator("#review-panel")).toBeVisible()
  173. return collectReviewPaneProbe(page)
  174. }
  175. async function measureReviewNextFile(page: Page, file: string) {
  176. await installReviewPaneProbe(page, { file })
  177. await startReviewPaneProbe(page)
  178. await page.getByRole("button", { name: "Next file" }).click()
  179. return collectReviewPaneProbe(page)
  180. }
  181. async function installReviewPaneProbe(page: Page, input: { file: string }) {
  182. await page.evaluate((input) => {
  183. const samples: ReviewPaneSample[] = []
  184. const basename = input.file.split(/[\\/]/).at(-1) ?? input.file
  185. let started: number | undefined
  186. let running = true
  187. const paneState = () => {
  188. const panel = document.querySelector<HTMLElement>("#review-panel")
  189. const review = panel?.querySelector<HTMLElement>('[data-component="session-review-v2"]')
  190. const rect = (review ?? panel)?.getBoundingClientRect()
  191. const text = panel?.textContent ?? ""
  192. const previewHeader = panel?.querySelector<HTMLElement>(
  193. '[data-slot="session-review-v2-file-header"]',
  194. )?.textContent
  195. const header = previewHeader ?? text
  196. const viewers = panel ? [...panel.querySelectorAll<HTMLElement>('[data-component="file"][data-mode="diff"]')] : []
  197. const codeBlocks = panel?.querySelectorAll("code").length ?? 0
  198. const diffLines = viewers.reduce(
  199. (sum, viewer) =>
  200. sum +
  201. (viewer.shadowRoot?.querySelectorAll("[data-line]").length ?? viewer.querySelectorAll("[data-line]").length),
  202. 0,
  203. )
  204. const panelVisible =
  205. !!panel && panel.getAttribute("aria-hidden") !== "true" && !!rect && rect.width > 0 && rect.height > 0
  206. return {
  207. panelVisible,
  208. header: header.slice(0, 500),
  209. diffViewers: viewers.length,
  210. diffLines,
  211. codeBlocks,
  212. ready:
  213. panelVisible &&
  214. header.includes(basename) &&
  215. (viewers.length > 0 || text.includes("+3") || diffLines > 0 || codeBlocks > 0),
  216. }
  217. }
  218. const sample = () => {
  219. if (!running || started === undefined) return
  220. requestAnimationFrame(() => {
  221. setTimeout(() => {
  222. if (!running || started === undefined) return
  223. samples.push({ observedAtMs: performance.now() - started, ...paneState() })
  224. if (performance.now() - started < 10_000) sample()
  225. }, 0)
  226. })
  227. }
  228. ;(window as Window & { __reviewPaneProbe?: ReviewPaneProbe }).__reviewPaneProbe = {
  229. samples,
  230. start: () => {
  231. started = performance.now()
  232. performance.mark("opencode.review-pane.click")
  233. sample()
  234. },
  235. stop: () => {
  236. running = false
  237. },
  238. }
  239. }, input)
  240. }
  241. async function startReviewPaneProbe(page: Page) {
  242. await page.evaluate(() => {
  243. ;(window as Window & { __reviewPaneProbe?: ReviewPaneProbe }).__reviewPaneProbe!.start()
  244. })
  245. }
  246. async function collectReviewPaneProbe(page: Page) {
  247. await page.waitForFunction((streak) => {
  248. const samples = (window as Window & { __reviewPaneProbe?: ReviewPaneProbe }).__reviewPaneProbe?.samples
  249. if (!samples) return false
  250. return samples.some((_, index) => {
  251. const stable = samples.slice(index, index + streak)
  252. return stable.length === streak && stable.every((sample) => sample.ready)
  253. })
  254. }, reviewReadyStreak)
  255. const samples = await page.evaluate(() => {
  256. const probe = (window as Window & { __reviewPaneProbe?: ReviewPaneProbe }).__reviewPaneProbe!
  257. probe.stop()
  258. return probe.samples
  259. })
  260. return { summary: summarizeReviewPaneSamples(samples), samples }
  261. }
  262. function summarizeReviewPaneSamples(samples: ReviewPaneSample[]) {
  263. const firstReady = samples.find((sample) => sample.ready)
  264. const stableIndex = samples.findIndex((_, index) => {
  265. const stable = samples.slice(index, index + reviewReadyStreak)
  266. return stable.length === reviewReadyStreak && stable.every((sample) => sample.ready)
  267. })
  268. return {
  269. samples: samples.length,
  270. firstReadyObservedMs: firstReady?.observedAtMs ?? null,
  271. stableReadyObservedMs: stableIndex === -1 ? null : samples[stableIndex + reviewReadyStreak - 1]!.observedAtMs,
  272. notReadySamples: samples.filter((sample) => !sample.ready).length,
  273. maxDiffViewers: Math.max(0, ...samples.map((sample) => sample.diffViewers)),
  274. maxDiffLines: Math.max(0, ...samples.map((sample) => sample.diffLines)),
  275. maxCodeBlocks: Math.max(0, ...samples.map((sample) => sample.codeBlocks)),
  276. }
  277. }