Repository navigation
Rust #325: re-run the fixtures when their other inputs change - #923
Conversation
The Shared Server fixtures path list missed five inputs its jobs read: activity result inspection fixtures, SDK signal readers, the conformance deadline and selected-run tests, the PHP capacity bindings and the v2 matched profile. A change to only those files skipped the parity matrix. A classifier test now checks each one. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
CI deliberately cancelled (November 1, 19:36 UTC). Actions is capped at about 8–12 concurrent jobs org-wide while a GitHub Support ticket is open, and this small trigger-list fix is lower priority than the release-blocking PRs. Re-run once concurrency recovers. |
|
Security bounded source review — exact head This change only expands the existing fail-closed fixture-input selector. The five added patterns cover the newly identified fixture, reader, conformance, capacity-adapter, and matched-profile inputs; they neither alter artifact identity/reuse nor grant permissions. The classifier still runs the full matrix when event identity, diff decoding, current-main verification, or qualification lookup cannot be established. The existing hosted result is not green qualification: its aggregate failure follows deliberately cancelled matrix jobs recorded on this PR. Rust Server still owns a fresh complete required-check run before merge. Non-blocking test-hardening note: the representative-path loop exercises four of the five added groups; |
Security's review of #923 noted the classifier test sampled four of the five added path groups. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Security focused rereview — exact head Relative to accepted Fresh hosted checks are queued and remain Rust Servers ordinary merge gate; this review does not claim them complete. |
|
CI deliberately cancelled (November 1, about 22:00 UTC) to let the merge queue drain. Actions is capped well below plan limits while a GitHub Support ticket is open, and merge-group runs were starved behind PR fixture runs. Re-run once the queue clears or concurrency recovers. |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Merged |
|
|
Security follow-up on current exact head |



The Shared Server fixtures workflow runs its matrix only when a file on its push path list changes; the
changesjob applies the same list to pull requests. A reference scan during today's CI measurement (#325 comment 6561901030) found five inputs the jobs read that the list misses, so a change to only those files skipped the parity matrix:tests/Fixtures/ActivityResultInspection/**tests/Fixtures/ServerParityReaders/**tests/Conformance/ServerParity*.mjsbenchmarks/capacity/v1/bindings/php/**benchmarks/server-port/v2/matched.jsonThis adds them.
ServerParityAggregateTestnow asserts that a change to one file from each set selects the matrix.bootstrap/app.phpstays off the list: PHP application changes skip the parity matrix by design today.Checks run: the seven unit tests that read
server-parity.ymlpass innode:22(140/140).🤖 Generated with Claude Code