ci: isolate Flow PRs and pin trusted runner execution - #35
Conversation
athena-omt
left a comment
There was a problem hiding this comment.
Independent cross-model review published by Athena automation, separately from source author under common control.
APPROVE exact head7970371ceb2c46d6e459f268d44497a30da07689 base165ff16e2cb3cbc176a107eb1c1f99f4f925438d.
Actual reviewer: official Claude Pro CLI Sonnet4.6 with Anthropic Haiku helper, no OpenAI fallback. Source authored by OpenAI. One bounded fresh review; remaining subscription quota unknown.
Full effective source diff plus proposed narrowing/controller scripts reviewed. Focused tests3/3, seven bypass-relevant mutation cases, aggregate failure/cancel/skip controls, full-file digest matching and actionlint pass independently. Parent88test/extended evidence and actual hosted/native runs recorded separately. No blocking findings. Entire trusted callee and caller digest-bound; PR execution hosted; only immutable callee reachable through restricted group. Callee's own predicate and server group restrictions enforce boundary, not mutable caller conditions alone.
Independent review seals REVIEW.md and review.json under reports/claude-review-flow35-20260915/. Actual provider receipt, executed tool evidence, reviewed policy script hashes and live isolation verification under reports/flow-isolation-20260915/. Runtime proof and normal required CI/codeowner gates remain parent-owned. No real fork trial is claimed by a same-repository negative-ref control.
Summary
Governing Issue
Refs #34
Validation
Bootstrap Governance
Flow Contract
Flow Merge Readiness
Merge Automation
Notes
Immutable callee source ref
pheidon/flow-trusted-sourcemust be retained after merge. No spending, new worker count, releases, or host admission expansion.