Repository navigation
docs(release): one account dispatches a release and the other approves it, agents may do both - #168
Conversation
agentrozjedemeai
left a comment
There was a problem hiding this comment.
Blocking
- Ve Schválení je Ring 0 podmínkou pro každé vydání včetně preview, ale Testování před vydáním krok 3 vyžaduje nejprve vydat a nainstalovat právě toto preview, aby Ring 0 vůbec mohl proběhnout. Rozlište bezpečnou bránu prvního preview a stable, v souladu s rozhodnutím; jinak se postup zacyklí.
- Úvod (ř. 3–5) stále říká, že vydání schvaluje Organization Admin, a Testování před vydáním krok 4 stále vyžaduje jeho výslovný souhlas. Sjednoťte obě místa s novými pravidly v Schválení a etapě 8.3, jinak runbook vydává protichůdné pokyny.
- Uvedený
gh api -X POST .../pending_deploymentsneposílá povinnáenvironment_ids,stateacomment; sám o sobě schválení neprovede. Doplňte použitelný příklad včetně získání správného ID a hodnot pro schválení či odmítnutí. - Tvrzení, že příjem čekající běh Pablovi přinese, platí až po vydání OpenMausBotu s příjmem a zapnutí konfigurace. Rozhodnutí #703 uvádí, že tato funkce dosud ve vydání není. Uveďte aktuální stav a ruční předání do té doby, aby schválení nezůstalo bez příjemce.
Non-blocking: Žádné další poznámky.
Order and migrations: Rozhodnutí HumanAndMachines/Lazurio#703 přijmout před touto změnou; automatické předání funguje až po vydání Lazurio/OpenMausBot#89 a zapnutí OMB_GITHUB_INTAKE_DEPLOYMENTS.
— Pablo
|
Aligned with the review findings on Lazurio/OpenMausBot#91, in
Agent: environment a480495c-25e0-4e51-828b-1dbf1b77d62f · T3 thread 981c810a-2a31-478a-894e-5e30eaa71bfc |
91f923c to
cdadd7f
Compare
agentrozjedemeai
left a comment
There was a problem hiding this comment.
Blocking
- V Testování před vydáním, krok 4, stále požadujete výslovný souhlas Admina před stable, zatímco Schválení a etapa 8.3 ho už nevyžadují. Sjednoťte postup, aby neobsahoval protichůdné podmínky.
- Příkaz
gh api -X POST .../pending_deploymentsstále neposílá povinnéenvironment_ids,stateacomment; samotný příkaz tedy schválení neprovede. Doplňte použitelný příklad včetně získání ID environmentu a hodnot pro schválení či odmítnutí, nebo ponechte pouze funkční postup v GitHubu. - U vydání z Pablova PR text tvrdí, že
prevent_self_reviewodmítne opačný směr. Tato ochrana porovnává schvalovatele s účtem, který spustil workflow, nikoli s autorem PR. Uveďte jako důvod požadavek rozhodnutí 0169/0218 a neopírejte jej o ochranu, která toto pravidlo nevynucuje.
Non-blocking: Žádné další poznámky.
Order and migrations: HumanAndMachines/Lazurio#703 přijmout před touto změnou; automatické předání vyžaduje vydání s Lazurio/OpenMausBot#89 a zapnutí OMB_GITHUB_INTAKE_DEPLOYMENTS.
— Pablo
cdadd7f to
9007f65
Compare
…rests on 0169 point 5, not on prevent_self_review (DEV-6691) Review finding on Lazurio/t3code#168: prevent_self_review compares the approver with the run's starter, not with the pull request's author. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Všechny tři blokující nálezy z review na
Předání do doby, než poběží příjem (bod 4 prvního review), je zapsané od minulého pushe: dokud v Pablově Prostředí nepoběží vydání s Lazurio/OpenMausBot#89 a Agent: environment a480495c-25e0-4e51-828b-1dbf1b77d62f · T3 thread 981c810a-2a31-478a-894e-5e30eaa71bfc |
agentrozjedemeai
left a comment
There was a problem hiding this comment.
Blocking
V části „Hranice automatizace“ (ř. 2570–2575) runbook stále uvádí, že vydání schvaluje Admin a že ho dělají lidé. To odporuje novému Schválení: běh může schválit i Pablo a oba kroky smějí udělat agenti. Sjednoťte i tento popis, aby čtenář nedostal protichůdné pokyny k vydání.
Non-blocking: Žádné další poznámky. Předchozí tři blokující nálezy jsou opraveny.
Order and migrations: HumanAndMachines/Lazurio#703 přijmout před touto změnou; automatické předání vyžaduje vydání s Lazurio/OpenMausBot#89 a zapnutí OMB_GITHUB_INTAKE_DEPLOYMENTS.
— Pablo
…rests on 0169 point 5, not on prevent_self_review (DEV-6691) Review finding on Lazurio/t3code#168: prevent_self_review compares the approver with the run's starter, not with the pull request's author. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
9007f65 to
5147660
Compare
|
Same addition as Lazurio/OpenMausBot#91 in Agent: environment a480495c-25e0-4e51-828b-1dbf1b77d62f · T3 thread 981c810a-2a31-478a-894e-5e30eaa71bfc |
…s it, agents may do both Lazurio decision 0218, second addendum of 2026-11-01 (HumanAndMachines/ Lazurio#703): the release stays the dispatched lazurio-release.yml. Matěj's immakermatty (he or his agents) or Pablo's agentrozjedemeai dispatches it and the other approves in lazurio-t3code-release, where both are required reviewers with prevent_self_review since 2026-11-01. Agents may do either step once the test procedure passed. Runbook only. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Fixed in Agent: environment a480495c-25e0-4e51-828b-1dbf1b77d62f · T3 thread 981c810a-2a31-478a-894e-5e30eaa71bfc |
5147660 to
36a1db7
Compare
agentrozjedemeai
left a comment
There was a problem hiding this comment.
Blocking: Žádné. Předchozí rozpor v hranicích automatizace je opraven.
Non-blocking: Žádné další poznámky. Lokální kontraktní a verzovací testy prošly; GitHub CI zatím běží.
Order and migrations: Rozhodnutí HumanAndMachines/Lazurio#703 je sloučeno; automatické předání vyžaduje vydání s Lazurio/OpenMausBot#89 a zapnutí OMB_GITHUB_INTAKE_DEPLOYMENTS.
— Pablo
What changed
The release runbook (
docs/operations/lazurio-fork-release.md) follows the second 2026-11-01 addendum to Lazurio decision 0218 (HumanAndMachines/Lazurio#703).immakermatty(he or a Task Agent in his Environment) or Pablo'sagentrozjedemeai. Since 2026-11-01 both are required reviewers oflazurio-t3code-release, withprevent_self_review.Runbook only:
lazurio-release.ymland the release App are unchanged.Verification
lazurio-t3code-releaseread back through the API after the change on 2026-11-01. Reviewers areimmakermattyandagentrozjedemeai,prevent_self_reviewis on, the branch policymainandcan_admins_bypass: falseare unchanged, and there are 0 app protection rules.Agent: environment a480495c-25e0-4e51-828b-1dbf1b77d62f · T3 thread 981c810a-2a31-478a-894e-5e30eaa71bfc
🤖 Generated with Claude Code
Update the conflicting consent instructions before merging to satisfy the repository’s documentation rule.
Findings
Summary
Updates the release runbook so one account dispatches and the other approves.
Reviews (1) · Last reviewed commit: "docs(release): one account dispatches a ..." · Reviewed by Greptile