feat(report): add agent markdown export view#2711
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b22f4ad539
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| if (reportViewMode !== 'markdown') { | ||
| return null; | ||
| } | ||
| return getReportMarkdownView(dump, reportToMarkdown); |
There was a problem hiding this comment.
Include every dump in report markdown export
When a report HTML contains multiple PlaywrightTasks groups, such as a merged Playwright report, dump is only the currently selected case from the store (initially the first case). The new Markdown View builds copy/download content from that single value, while the case selector is replaced by the markdown source in this mode, so exporting report.md silently omits all other cases in the same report. Build the markdown from dumps or preserve a way to select/aggregate cases in Markdown View.
Useful? React with 👍 / 👎.
b22f4ad to
f2dab8b
Compare
629a619 to
39ef9fb
Compare
Summary
Validation