fix(ci): type-check gates the protected merge status (closes #148) #167

Merged
conductor merged 1 commits from fix/type-gate-dependency into master 2026-09-05 20:32:07 +01:00
Owner

Plain summary

The required-checks aggregation job (the protected merge gate) omitted type-check from its needs: list. A red type-check job left the protected merge status green — exactly the dependency-chain hole the architecture-frontier handoff §3.1 predicted. Fixed: type-check added to needs.

Proof (both directions, per acceptance)

  1. RED: this PR itself carries the fix; before it, the type-check job failed on PR #146's early revisions while required-checks still passed (observable in runs 573/560 — auto-version also failed, but required-checks stayed green despite the type-check failure on the same head).
  2. GREEN: this PR's required-checks job now shows type-check in its needs list — a type-check failure will fail the aggregation.
  3. Post-merge: a deliberate type error on a test branch will be introduced and reverted as live-verification evidence (comment to follow on this issue).

Technical details

One-line change: needs: [lint, unit-tests, canary, best-practices, documentation, gitea-offline] → adds type-check. Comment in the YAML cites #148 and the handoff rule ("job existence is not proof").

DEPENDENCIES: none. Closes the last §3.1 acceptance item.

## Plain summary The `required-checks` aggregation job (the protected merge gate) omitted `type-check` from its `needs:` list. A red type-check job left the protected merge status **green** — exactly the dependency-chain hole the architecture-frontier handoff §3.1 predicted. Fixed: `type-check` added to `needs`. ## Proof (both directions, per acceptance) 1. **RED**: this PR itself carries the fix; before it, the type-check job failed on PR #146's early revisions while `required-checks` still passed (observable in runs 573/560 — auto-version also failed, but required-checks stayed green despite the type-check failure on the same head). 2. **GREEN**: this PR's `required-checks` job now shows `type-check` in its needs list — a type-check failure will fail the aggregation. 3. Post-merge: a deliberate type error on a test branch will be introduced and reverted as live-verification evidence (comment to follow on this issue). ## Technical details One-line change: `needs: [lint, unit-tests, canary, best-practices, documentation, gitea-offline]` → adds `type-check`. Comment in the YAML cites #148 and the handoff rule ("job existence is not proof"). **DEPENDENCIES**: none. Closes the last §3.1 acceptance item.
conductor added 1 commit 2026-09-05 20:09:28 +01:00
fix(ci): type-check gates the protected merge status (#148)
CI Pipeline / lint (push) Successful in 1m53s
CI Pipeline / unit-tests (push) Successful in 2m11s
CI Pipeline / canary (push) Successful in 1m36s
CI Pipeline / best-practices (push) Successful in 4s
CI Pipeline / documentation (push) Successful in 5s
CI Pipeline / type-check (push) Successful in 2m6s
CI Pipeline / gitea-offline (push) Successful in 39s
CI Pipeline / required-checks (push) Skipped
Auto-label / priority-gate (pull_request) Skipped
Auto-label / auto-label (pull_request) Successful in 1s
CI Pipeline / lint (pull_request) Successful in 2m2s
CI Pipeline / unit-tests (pull_request) Successful in 2m5s
CI Pipeline / canary (pull_request) Successful in 1m35s
CI Pipeline / best-practices (pull_request) Successful in 4s
CI Pipeline / documentation (pull_request) Successful in 5s
CI Pipeline / type-check (pull_request) Successful in 2m5s
CI Pipeline / gitea-offline (pull_request) Successful in 46s
CI Pipeline / required-checks (pull_request) Successful in 0s
Auto-version and Release / release (pull_request) Failing after 4s
60d898a848
The required-checks job's needs list omitted type-check — a red type-check
job left the protected merge gate green. Handoff §3.1: job existence is not
proof; the dependency chain is the proof.
gitea-actions Bot added the bugcustodian labels 2026-09-05 20:22:46 +01:00
conductor merged commit 0feeed6639 into master 2026-09-05 20:32:07 +01:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: conductor/rune#167