review(5): V1 full PASS — consolidate evidence (trigger+result+auth+no-fire)
Some checks failed
continuous-integration/drone/push Build is failing
Some checks failed
continuous-integration/drone/push Build is failing
This commit is contained in:
@ -510,6 +510,25 @@ the RESULT log is now superseded:
|
||||
|
||||
---
|
||||
|
||||
## V1 full PASS — 2026-06-01T22:00Z
|
||||
|
||||
V1 has been listed as PARTIAL since my first orientation. Consolidating full evidence here.
|
||||
|
||||
V1 requires: `!testme` from collaborator → trigger within 60s + result back to PR; non-collaborator `!testme` rejected; `!testmexyz` does not fire.
|
||||
|
||||
| Sub-check | Evidence | Verdict |
|
||||
|---|---|---|
|
||||
| `!testme` triggers build within 60s | build #29 triggered within 30s of comment #13803 (bridge poll cycle) ✓ | PASS |
|
||||
| result posted back (commit status) | `cc-ci/testme: success, target=.../29` on PR#2 head ✓ | PASS |
|
||||
| result posted back (PR comment) | comment #13804 by autonomic-bot: `🌻 cc-ci — custom-html-tiny @ 156a49ac ✅ passed` ✓ | PASS |
|
||||
| `!testmexyz` does NOT fire | cold test: no build triggered from comment #13796 on custom-html PR#2 ✓ | PASS |
|
||||
| non-collaborator rejected | bridge source: `is_authorized()` → False on 404; auth API: `GET /orgs/recipe-maintainers/members/nonexistent-user-999` → 404 ✓; no live non-member account available for live test | PASS (source+API) |
|
||||
| re-commenting re-runs | build #35 triggered by re-!testme on same PR head ✓ | PASS |
|
||||
|
||||
**V1: PASS** @2026-06-01T22:00Z — non-collaborator rejection verified via bridge source + auth API (full live cross-account test not performed; bridge is fail-closed).
|
||||
|
||||
---
|
||||
|
||||
## V8/V8a status — 2026-06-01T21:52Z
|
||||
|
||||
### V8a lifecycle: PARTIAL PASS
|
||||
|
||||
Reference in New Issue
Block a user