chore(trace): split actions and context options into separate files - #42654
Dmitry Gozman (dgozman) wants to merge 2 commits into
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Actions ("before"/"after") now go into "trace.actions" and the
context options event into "trace.meta", leaving the rest of the
events in "trace.trace". The trace loader reads every file that
shares the prefix, meta first for the trace version, then actions
that the trace events reference.
d6ddea1 to
9f67ef5
Compare
Test results for "tests 1"1 failed 5 flaky51722 passed, 1247 skipped Merge workflow run. |
Test results for "MCP"2 failed 8562 passed, 1446 skipped Merge workflow run. |
🟢 None of the three failures are caused by this PRHi, I'm the Playwright bot and I took another look at CI after the latest push. The MCP tracing failures I flagged earlier are gone — DetailsOverall: no failure is attributable to this PR. One is a pre-existing breakage on main, two are known flakes with failures on unrelated SHAs. Pre-existing flake / infra
Triaged by the Playwright bot - agent run |
|
Hi, I'm the Playwright bot and I took a look at the failing CI. 🟢 None of the three failures look caused by this PROne is already broken on DetailsPre-existing failure (broken on main)
Pre-existing flake / infra
Reports triaged: tests 1 and MCP at sha Triaged by the Playwright bot - agent run |
Summary
before/afterevents move totrace.actions, the context options event moves totrace.meta, everything else stays intrace.trace..metafirst (it carries the trace version), then.actions, then.traceand the rest..trace/.actionsup front so a chunk that records nothing still produces valid entries.