feat(ci): add pull-request gate for director merges #423

Merged
coilysiren merged 1 commit from issue-421 into main 2026-07-10 10:15:01 +00:00
Member

closes #421

ward.workflow: pull-requests-and-merge

closes #421 ward.workflow: pull-requests-and-merge
feat(ci): add pull-request gate for director merges
Some checks failed
ci / gate (pull_request) Failing after 7s
a693bbc0df
closes #421
Author
Member

verdict: block
confidence: high
summary: The PR creates the expected ci / gate (pull_request) context, but the new gate is currently red and the director still sees it as an unset required context. That means it does not yet satisfy the merge-gate issue contract.
findings:

  • severity: high
    file: .forgejo/workflows/ci.yml
    line: 16
    problem: The newly added ci / gate (pull_request) status for head a693bbc0df9e63d6a73951279dce2b9310666d74 is failure in Forgejo Actions.
    impact: warded director merge --repo coilyco-flight-deck/agentic-os --dry-run --limit 100 skips PR #423 with has required status context ci / gate (pull_request)=, so the PR does not yet create a green director-readable gate.
    conclusion: Review blocks PR #423. The workflow file is the right general shape, but the live PR status is failing and the director dry-run still refuses the PR, so the issue is not landed until the gate runs green and the director reads it as mergeable.
verdict: block confidence: high summary: The PR creates the expected `ci / gate (pull_request)` context, but the new gate is currently red and the director still sees it as an unset required context. That means it does not yet satisfy the merge-gate issue contract. findings: - severity: high file: .forgejo/workflows/ci.yml line: 16 problem: The newly added `ci / gate (pull_request)` status for head `a693bbc0df9e63d6a73951279dce2b9310666d74` is `failure` in Forgejo Actions. impact: `warded director merge --repo coilyco-flight-deck/agentic-os --dry-run --limit 100` skips PR #423 with `has required status context ci / gate (pull_request)=`, so the PR does not yet create a green director-readable gate. conclusion: Review blocks PR #423. The workflow file is the right general shape, but the live PR status is failing and the director dry-run still refuses the PR, so the issue is not landed until the gate runs green and the director reads it as mergeable.
coilyco-ops force-pushed issue-421 from a693bbc0df
Some checks failed
ci / gate (pull_request) Failing after 7s
to 29d53be09e
Some checks failed
ci / gate (pull_request) Failing after 6s
2026-07-10 09:40:57 +00:00
Compare
Author
Member

verdict: block
confidence: high
summary: The update now seeds WARD_CONFIG_REF from github.sha, but the live pull-request gate is still red and the director still reads the required context as unset. The PR still does not satisfy the issue contract.
findings:

  • severity: high
    file: .forgejo/workflows/ci.yml
    line: 20
    problem: The current head 29d53be09e30298aac1a87ceb9f7d2d1b0653885 still has ci / gate (pull_request) = failure in Forgejo, with description Failing after 6s.
    impact: warded director merge --repo coilyco-flight-deck/agentic-os --dry-run --limit 100 still skips PR #423 with has required status context ci / gate (pull_request)=, so this is not a working director merge gate yet.
    conclusion: Review still blocks PR #423. The config-ref adjustment is directionally plausible, but the live PR status remains failing and the director dry-run still refuses the PR.
verdict: block confidence: high summary: The update now seeds `WARD_CONFIG_REF` from `github.sha`, but the live pull-request gate is still red and the director still reads the required context as unset. The PR still does not satisfy the issue contract. findings: - severity: high file: .forgejo/workflows/ci.yml line: 20 problem: The current head `29d53be09e30298aac1a87ceb9f7d2d1b0653885` still has `ci / gate (pull_request)` = `failure` in Forgejo, with description `Failing after 6s`. impact: `warded director merge --repo coilyco-flight-deck/agentic-os --dry-run --limit 100` still skips PR #423 with `has required status context ci / gate (pull_request)=`, so this is not a working director merge gate yet. conclusion: Review still blocks PR #423. The config-ref adjustment is directionally plausible, but the live PR status remains failing and the director dry-run still refuses the PR.
coilyco-ops force-pushed issue-421 from 29d53be09e
Some checks failed
ci / gate (pull_request) Failing after 6s
to 4a689cae78
Some checks failed
ci / gate (pull_request) Failing after 5m15s
2026-07-10 09:44:12 +00:00
Compare
coilyco-ops force-pushed issue-421 from 4a689cae78
Some checks failed
ci / gate (pull_request) Failing after 5m15s
to 3d50ba28b4
All checks were successful
ci / gate (pull_request) Successful in 11s
2026-07-10 09:51:40 +00:00
Compare
Author
Member

verdict: pass
confidence: medium
summary: PR #423 adds a live .forgejo/workflows/ci.yml pull-request workflow with the expected ci / gate context, documents the context, and has focused tests that pin the workflow/docs contract. Forgejo status history for head 3d50ba28b4f83661775320f7e9f13d644c620f70 reports ci / gate (pull_request) with status=success; the director still prints an empty value because ward#994 is reading the wrong status field.
findings: []
conclusion: The PR satisfies the code-side part of agentic-os#421: it creates a pull_request CI gate and names the branch-protection context. Remaining lane blockers are outside this PR: ward#994 must teach the director to read Forgejo's populated status field, and infrastructure#532 now tracks converging required status contexts for PR-gated repos.

verdict: pass confidence: medium summary: PR #423 adds a live `.forgejo/workflows/ci.yml` pull-request workflow with the expected `ci / gate` context, documents the context, and has focused tests that pin the workflow/docs contract. Forgejo status history for head `3d50ba28b4f83661775320f7e9f13d644c620f70` reports `ci / gate (pull_request)` with `status=success`; the director still prints an empty value because ward#994 is reading the wrong status field. findings: [] conclusion: The PR satisfies the code-side part of agentic-os#421: it creates a pull_request CI gate and names the branch-protection context. Remaining lane blockers are outside this PR: ward#994 must teach the director to read Forgejo's populated `status` field, and infrastructure#532 now tracks converging required status contexts for PR-gated repos.
Sign in to join this conversation.
No description provided.