gh workflow run, the one gh call site a workflow cannot declare a client for
@andres — run 504/505 closes the workflow-token endpoint/identity question: a step running as ${{ github.token }} received 204 and started a real workflow_dispatch run. That is valid and…
APPROVED on exact head 3bde48f24c1bfb8f424b07a3535b5eb29fbd5ac0.
Re-approved exact merged-up head 396744618f6ac0c51a694d40b34d1a269b0d1c3a.
@andres — the post-merge transition is accurate: !207 delivered the runbook, but the operator-owned half remains.
Current missing evidence/authority, directly from merged docs/RUNNER-PROBES.md…
@andres — reviewer verdict: the implementation/root cause is verified, but the literal live acceptance criterion is not yet met.
Evidence I accept:
- the old Forgejo path `repos/.../commits/<sha…
REQUEST CHANGES on exact head 37e31ffd85ae182819e53799cd68d62b45749d8f: two requested corrections are complete and the substantive never-silence coverage is now good, but the old vacuous assertion still remains.
Reviewed exact head 4e28d437d6f683df018b7e67c3b1e0c3395740e4 against #199 and the existing forge shim contracts.
REQUEST CHANGES on exact full head 935a813d75c289cd91ed386df891dab6e53f7a7c.
APPROVED on exact full head SHA 262705d3944d334ae8d24f34eb6b0ca0802727ea against current main@a13aa6d4b96d53fca56fe688940bb3d62b651d1c.
APPROVED on exact full head SHA 65cee3fdf9870cf03b6824c00cd54f9c8c482f06, freshly revalidated against current main@a13aa6d4b96d53fca56fe688940bb3d62b651d1c after #203 merged.
APPROVED on exact full head SHA 1bf7091d39e5be74ca8c74407769f37f7fa0899b.
Reviewer post-merge acceptance evidence:
- scheduled sweep #452 ran on exact merged `main@5c924294bfeced5be1f1913346a4722…
@andres: post-merge sweep #452 has now proved #210 live and moved it to post-merge, but #197’s governing task list still omits #210 even though #210 is Part of #197 and was the hard gate…
Re-review approved on exact merge head 65cee3fdf9870cf03b6824c00cd54f9c8c482f06 over current main@5c924294bfeced5be1f1913346a4722c7b9ceaa0.
Reviewer correction for review #1722: while that verdict was being submitted, the builder advanced #203 from e0cd0cb7a39b1cad17f0e86795291d0a7a01818d to merge head `1bf7091d39e5be74ca8c74407769f3…
Re-review approved on exact PR head e0cd0cb7a39b1cad17f0e86795291d0a7a01818d, integrated with current main@5c924294bfeced5be1f1913346a4722c7b9ceaa0.
Re-review approved on exact PR head e0cd0cb7a39b1cad17f0e86795291d0a7a01818d, integrated with current main@5c924294bfeced5be1f1913346a4722c7b9ceaa0.
Approved on exact head 368621dcea4e04191486b2140f27db02c0737178.
Request changes on exact head 6874e76c04a697ba1e1a1d8e11e3d9a03a24c6fb.
Approved on exact head fc24fa4b78daf6f7669a10f17ef6681b36248769.