Every push gets the tests back, and the empty tier goes - #33
Merged
Merged
Conversation
3lvis
marked this pull request as ready for review
September 17, 2026 18:52
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
CI has been manual-dispatch only, on the grounds that GitHub's macOS runners crash on the hostless capture tests. Dispatching it against the
xcode-27image settles it:Build + unit testspasses, 213 unit tests and all. That half of the pause has been obsolete since the image moved.The other half was never what the note said.
UI testsdid fail — after thirteen minutes, onDTServiceHubClient failed to bless service hub, with the runner crashing while bootstrapping:It never reached a test, because there are none to reach.
DemoUITestsis a target in the project with zero source files, which is exactly whatAGENTS.mdsays it should be — "throwaway probes only, empty at rest". The job was launching a test runner every time to execute an empty suite.The approach
pushandpull_requesttriggers are live again, and theui-testsjob is gone. A permanent UI test is already provided for —AGENTS.mdallows one for a workaround held against SwiftUI or UIKit, "only with teeth and a place in the merge gate" — and the job comes back with the first one that earns it. Until then, an empty tier gated nothing and cost thirteen minutes a run.actions/checkoutis pinned to the commitv5resolves to, matching Networking and SwiftSync.What this changes
maingoes from one required check to two:oidakeeps gating on Linux, andBuild + unit testsjoins it once this lands. Pinwheel stops being the only one of the three repositories resting entirely on the local gate.