Repository navigation
Gør det tydeligt, hvornår en PR kræver menneskelig godkendelse #88
Description
Behov
Ejeren savner et tydeligt flow, hvor AgentOps udvælger de PR’er, som kræver, at et menneske læser ændringen og tager stilling før merge. De hidtidige forløb har især været eksplicit godkendte opgaver; det gør det svært at se, hvordan nye produktvalg, ændret omfang eller væsentlig risiko bliver eskaleret.
Undersøg først, om hullet ligger i metoden eller i den synlige aflevering. #3 beskriver allerede admission og delivery som to menneskelige standardbeslutninger med mulighed for afgrænset delegation.
Ønsket resultat
- AgentOps gør det tydeligt, om en konkret PR kan afleveres under en gældende delegation, eller kræver en menneskelig beslutning. Grøn CI og en agents tekniske review er ikke i sig selv merge-tilladelse.
- Ved væsentlige nye produkt-/arkitekturvalg, ændret omfang, risiko eller uklar myndighed vurderer AgentOps behovet for menneskelig stillingtagen ud fra projektets faktiske mandat.
- Den udvalgte PR får en kort, forståelig aflevering: hvad ændres, hvorfor kræves mennesket, hvilken beslutning ønskes, og hvilke relevante ændringer/beviser skal læses. Bind beslutningen til den konkrete kandidat.
- Afventet eller afvist godkendelse holder den relevante merge/release tilbage. Tavshed tæller ikke som accept. Ændres kandidaten eller beslutningsgrundlaget, vurderes godkendelsen igen. Uafhængigt godkendt arbejde fortsætter.
- Bevar eksisterende afgrænsede delegationsaftaler og undgå rutinemæssig gen-godkendelse af ligetil arbejde. Brug eksisterende PR-/native beslutningsflader og den kanoniske AgentOps-guidance.
- Vis flowet på et almindeligt delegeret tilfælde, et tilfælde der kræver ejerbeslutning samt manglende/afvist svar og en ændret kandidat. Bevar uafhængig review og branch protection.
Afgrænsning og kø
Til den eksisterende Factory AgentOps’ normale triage og prioritering. Foreslå den mindste nyttige forbedring; en eksisterende regel kan være tilstrækkelig, hvis den bliver brugt og vist tydeligt. Ingen ny approval-service eller parallel runtime.
Relateret: #3 ejer den brede AgentOps-proof; #78 undersøger hurtigere review af små ændringer. Dette issue ejer det konkrete valg og den menneskelige PR-overlevering.
Activity
gustavonline commentedon Oct 28, 2026
Delivered as a scoped method improvement in PR #103, v0.1.26.
AgentOps policy now distinguishes delegated delivery from new owner decisions, gives a short candidate-bound PR handoff, treats silence/refusal as a hold on the affected work, and reconsiders changed candidates. Independent delegated work can continue. Green CI or a model review does not create merge authority.
Two fresh independent Opus 5.5 High/Fast off source reviewers accepted the repaired exact candidate. The objective reviewer rehearsed the four requested cases: ordinary delegated docs release, proposed private-data exposure, refused exposure with unrelated work, and a new production migration after approval of an earlier candidate. These are synthetic decision rehearsals, not four real PR demonstrations or a new approval-enforcement service.
Closing the method/handoff gap on that explicit basis. The actual missing native pending-approval wake/status and orphaned record remain open in #32; this closure does not claim unattended approval handling.