ci(workflows): hosted Check and standards-graph on main only; delete Coherence (#236 C1–C3) - #240
Merged
Merged
Conversation
D1 (2026-09-01, owner decision): "main only + one gate". Check and Validate standards graph now trigger on push:[main] plus workflow_dispatch, never pull_request or testing. Coherence is deleted: tests/coherence carries neither the performance nor compatibility marker, so it was already collected by Check's ordinary pytest step (-m "not performance and not compatibility"), making the standalone workflow pure duplication. Updates the two live-workflow reconstruction tests (test_standards_graph_workflow.py, test_action_runtime_versions.py) to match. Refs #236
…on tests Completes the D1 trigger collapse: push:[main] + workflow_dispatch on Check and Validate standards graph, pull_request and testing dropped. Updates the two reconstruction tests to the new contract and records the change in CHANGELOG.md [Unreleased]. Refs #236
D1 (#236) deleted coherence.yml, the only workflow that ever invoked pytest with a tests/ path argument, leaving test_ci_phase_discovery_is_not_vacuous's path-scoped-branch assertion with no live example to check. Factor the per-file parsing out of pytest_phases() into pytest_phases_in_document() and feed the test a minimal synthetic workflow document written to tmp_path instead of depending on a real repository workflow. Refs #236
chrisdpurcell
marked this pull request as ready for review
September 1, 2026 20:34
This was referenced Sep 1, 2026
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
1 participant
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.
Summary
Owner decision D1/D5 (2026-09-01): hosted
Checkruns only onpushtomainandworkflow_dispatch; theCoherenceworkflow is deleted (itstests/coherenceselection was already inside Check's ordinary step);Validate standards graphlikewise runs onmainand dispatch only. The seven cheap workflows keep their pull-request triggers. The CI-phase guard test that used Coherence as its only path-scoped witness now uses a synthetic workflow document.Governing work
Supporting: #236
Acceptance coverage
check.ymltriggerspush: [main]+workflow_dispatch; one Check run per release train.coherence.ymldeleted;tests/coherencecarries neitherperformancenorcompatibilitymarker sopytest -m "not performance and not compatibility"in Check selects it.validate-standards-graph.ymlno longer fires ontestingor PRs; reconstruction tests updated.Verification
YAML parse of every workflow;
tests/test_standards_graph_workflow.py+tests/test_action_runtime_versions.py(24 passed);tests/mcp_server/integration/test_registry_invariants.py(3 passed); ruff on touched tests.