Skip to content

Commit f8f78e8

Browse files
committed
chore(gates): retitle the depgraph gate as the report's model tests
The Layering Guard step no longer claims to agree the report with the gate; it runs the depgraph model and blast-radius tests, which the gate manifest requires a registered check to own.
1 parent 6324121 commit f8f78e8

2 files changed

Lines changed: 6 additions & 7 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -115,15 +115,14 @@ jobs:
115115
uses: ./.github/actions/run-gate
116116
with: { gate: layering }
117117

118-
- name: Check the depgraph report agrees with the gate
119-
# scripts/depgraph reads the same model as the gate, so its inversion count must
120-
# reproduce TYPE_INVERSION_BASELINE. Free two-sources check: if the tree changes
121-
# and only one side is updated, this fails and names the difference. Runs beside
122-
# the layering gate so the two can never be green independently.
118+
# Model tests for the dependency-graph report and its blast-radius query. The report
119+
# reads the gate's model (scripts/layering/model.ts) and applies the gate's own R6
120+
# counting rule, so it is not a second measurement of TYPE_INVERSION_BASELINE.
121+
- name: Check the depgraph report model
123122
uses: ./.github/actions/run-gate
124123
with: { gate: depgraph }
125124

126-
# Tests for the TMPDIR redirection itself, hidden the same way as the check above.
125+
# Tests for the TMPDIR redirection itself.
127126
#
128127
# Deliberately NOT in Coverage next to `check:tmpdir-leaks`, where the subject matter
129128
# would put it: vitest-tmpdir-global-setup.test.ts proves the lifecycle by spawning a

‎scripts/check-affected/checks.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ export const CHECK_CATALOG: readonly CheckSpec[] = [
9595
gate('affected-selector', 'Affected-check selector model', 'check:affected:test'),
9696
gate('gate-manifest', 'Gate manifest — every gate owned and wired', 'check:gate-manifest'),
9797
gate('gate-manifest-model', 'Gate manifest model', 'check:gate-manifest:test'),
98-
gate('depgraph', 'Dependency graph report agrees with the gate', 'depgraph:test'),
98+
gate('depgraph', 'Dependency graph report model', 'depgraph:test'),
9999
gate('tmpdir-leaks', 'Leaked test tmpdir detector', 'check:tmpdir-leaks'),
100100
gate('tmpdir-leaks-model', 'TMPDIR redirection model', 'check:tmpdir-leaks:test'),
101101
gate('coverage-model', 'Changed-line coverage model', 'check:coverage-changed:test'),

0 commit comments

Comments
 (0)