Skip to content

Commit d679cb7

Browse files
committed
Match Pages refresh trigger to performance workflow name
1 parent 00567db commit d679cb7

3 files changed

Lines changed: 20 additions & 4 deletions

File tree

‎.github/workflows/pages.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ on:
1313
- experiments/Synapse.ReferenceBenchmarks/Program.cs
1414
workflow_dispatch:
1515
workflow_run:
16-
workflows: [verify, performance]
16+
workflows: [verify, performance-diagnostic]
1717
types: [completed]
1818

1919
permissions:

‎docs/Development/WebsiteMetricsEvidence-2026-10-03.md‎

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -82,3 +82,17 @@ because the dependency-repair authorization does not cover this feature. No
8282
commit or push occurred during that attempt. On 2026-10-03 the user explicitly
8383
authorized pushing these prepared changes to `main`; publication proceeds
8484
under that authorization.
85+
86+
## Workflow name regression after publication
87+
88+
Commit `00567dbba63f217ba8387829bb67131c877b5731` reached `origin/main` and
89+
created Pages run `37128235031`, verification run `37128235060`, and performance
90+
run `37128235064`. Run discovery exposed the actual performance workflow name
91+
`performance-diagnostic`, while the Pages subscription used `performance`.
92+
The improved source contract derives each upstream name from its real workflow
93+
file. It failed first (1/1, exit 2) before correcting the subscription.
94+
These run IDs establish dispatch, not completed test or deployment evidence.
95+
After the correction, Release test-project build passed with zero warnings
96+
or errors, the 19-case Website suite passed (exit 0, 13.971s), scoped format
97+
verification passed, and actionlint/diff checks passed. Sandbox IPC-denied
98+
attempts were rerun with required permissions and are not counted as passes.

‎tests/Synapse.IntegrationTests/Features/Website/MetricsPipelineTests.cs‎

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,19 +25,21 @@ public async Task MetricsWorkflowsConnectTrustedRunsAndImmutableJson()
2525
{
2626
var root = FindRepositoryRoot();
2727
var pages = await File.ReadAllTextAsync(Path.Combine(root, ".github/workflows/pages.yml"));
28+
var verify = await File.ReadAllTextAsync(Path.Combine(root, ".github/workflows/verify.yml"));
29+
var performance = await File.ReadAllTextAsync(Path.Combine(root, ".github/workflows/performance.yml"));
30+
var verifyName = verify.Split('\n').First(line => line.StartsWith("name:", StringComparison.Ordinal))[5..].Trim();
31+
var performanceName = performance.Split('\n').First(line => line.StartsWith("name:", StringComparison.Ordinal))[5..].Trim();
2832
await Assert.That(pages).Contains("workflow_run:");
29-
await Assert.That(pages).Contains("workflows: [verify, performance]");
33+
await Assert.That(pages).Contains($"workflows: [{verifyName}, {performanceName}]");
3034
await Assert.That(pages).Contains("github.event.workflow_run.head_branch == 'main'");
3135
await Assert.That(pages).Contains("github.event.workflow_run.event == 'push'");
3236
await Assert.That(pages).Contains("actions: read");
3337
await Assert.That(pages).Contains("collect-site-results.sh");
3438
await Assert.That(pages).Contains("site-data");
35-
var verify = await File.ReadAllTextAsync(Path.Combine(root, ".github/workflows/verify.yml"));
3639
await Assert.That(verify).Contains("--report-trx");
3740
await Assert.That(verify).Contains("--minimum-expected-tests 1");
3841
await Assert.That(verify).Contains("test-report");
3942
await Assert.That(verify).Contains("test-results-${{ matrix.runtime_identifier }}");
40-
var performance = await File.ReadAllTextAsync(Path.Combine(root, ".github/workflows/performance.yml"));
4143
await Assert.That(performance).Contains("--json \"${RUNNER_TEMP}/performance-results.json\"");
4244
var collector = await File.ReadAllTextAsync(Path.Combine(root, ".github/scripts/collect-site-results.sh"));
4345
await Assert.That(collector).Contains("run_attempt");

0 commit comments

Comments
 (0)