Move the benchmark comment rendering out of uefi-bench-pr-comment.yml and into the step that produces the results, so the comment workflow only downloads an artifact and posts it.
Why
The bencher output format is currently a contract between two places: whatever writes results-*.txt, and a sed/awk parser buried in workflow YAML. A format change breaks the second parser silently.
That script has also grown real logic — field sanitization, removed rows, unit-change detection — which belongs with the tool rather than the CI glue.
Raised by @rogurr in #209.
Acceptance criteria
Dependencies
Move the benchmark comment rendering out of
uefi-bench-pr-comment.ymland into the step that produces the results, so the comment workflow only downloads an artifact and posts it.Why
The bencher output format is currently a contract between two places: whatever writes
results-*.txt, and ased/awkparser buried in workflow YAML. A format change breaks the second parser silently.That script has also grown real logic — field sanitization,
removedrows, unit-change detection — which belongs with the tool rather than the CI glue.Raised by @rogurr in #209.
Acceptance criteria
uefi-bench-pr-comment.ymlcontains no bencher-format parsingnew/removed/unit changedrowsDependencies