blackroad-os/overpass
9
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(ci): Fix benchmark summary reporter after metric name format change (no-changelog)

Browse Source 
Metrics are now attached as metric:<name> directly (e.g. metric:throughput)
rather than metric:<scenario>-<name>. Remove the scenario-prefix stripping
in extractMetricSuffix so the reporter can match metric names to columns again.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Declan Carroll
2026-03-16 19:37:58 +00:00
parent 94761b88b7
commit b5eca50299
1 changed files with 8 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

21
packages/testing/playwright/reporters/benchmark-summary-reporter.ts
View File

@@ -76,11 +76,8 @@ function extractSuite(filePath: string): string {
return 'other';
}
function extractMetricSuffix(metricName: string, scenario: string): string | null {
if (metricName.startsWith(`${scenario}-`)) {
return metricName.slice(scenario.length + 1);
}
return null;
function extractMetricSuffix(metricName: string): string {
return metricName;
}
class BenchmarkSummaryReporter implements Reporter {
@@ -98,14 +95,12 @@ class BenchmarkSummaryReporter implements Reporter {
for (const attachment of metricAttachments) {
const fullName = attachment.name.replace('metric:', '');
const suffix = extractMetricSuffix(fullName, scenario);
if (suffix) {
try {
const data = JSON.parse(attachment.body?.toString() ?? '');
metrics.set(suffix, data.value);
} catch (error) {
console.warn(`[BenchmarkReporter] Malformed metric attachment "${fullName}":`, error);
}
const suffix = extractMetricSuffix(fullName);
try {
const data = JSON.parse(attachment.body?.toString() ?? '');
metrics.set(suffix, data.value);
} catch (error) {
console.warn(`[BenchmarkReporter] Malformed metric attachment "${fullName}":`, error);
}
}