fix(have:review-cycle): operational discipline — Copilot non-optional, loop convergence, blocker docs by willgriffin · Pull Request #5 · happyvertical/have-config

fix(review-cycle): make Copilot CLI step non-optional + document auth/policy blockers

willgriffin

committed

fix(review-cycle): add hard rules for loop convergence

willgriffin

committed

fix(review-cycle): exit on P3/nit-only — don't loop on trivia

willgriffin

committed

fix(review-cycle): clarify P3 triage — fix inline, record, or follow up

willgriffin

committed

fix(review-cycle): address pr-review round-1 findings on this PR

willgriffin

committed

fix(review-cycle): round-2 — correct Copilot --allow-tool syntax + Status contract

willgriffin

committed

fix(review-cycle): round-3 — workflow contract repairs

willgriffin

committed

fix(review-cycle,ship): round-4 — accepted-P2 stop condition + ship handles partial

willgriffin

committed

fix(review-cycle,ship): round-5 — remove leaked transcripts, fix

willgriffin

committed

fix(review-cycle,ship): round-6 — security gap + workflow contract fixes

willgriffin

committed

fix(review-cycle): round-7 — fully scope Copilot path access + narrow gitignore

willgriffin

committed

revert(review-cycle): walk back wrong-threat-model security overspec

willgriffin

committed

docs(review-cycle,ship): rename codex/claude/copilot → *-cli everywhere

willgriffin

committed

fix(ci): allow multi-scope and slash in commit scope + escape commit msg in error output

willgriffin

committed

fix(review-cycle): one reviewer clean ≠ ensemble convergence

willgriffin

committed

fix(review-cycle,ci): round-1 — close 3 ensemble findings on PR #5

willgriffin

committed

fix(review-cycle,ship): round-2 — close 3 ensemble findings + 1 self-review polish

willgriffin

committed

fix(review-cycle,ship,ci): round-3 — close 6 of 8 ensemble findings (4 reviewers)

willgriffin

committed

fix(review-cycle,ship,ci): round-4 — 4 ensemble findings (4 reviewers, no rejections)

willgriffin

committed

fix(review-cycle,ship): round-5 — 5 of 6 ensemble findings (4 reviewers)

willgriffin

committed

fix(review-cycle,ship): round-6 — 7 ensemble findings (4 reviewers, no rejections)

willgriffin

committed

fix(review-cycle): round-7 — 6 ensemble findings, slot model + bias caveat

willgriffin

committed

fix(review-cycle,ship): round-8 — 6 ensemble findings (1 medium real bug, 5 low polish)

willgriffin

committed

fix(review-cycle,ship): round-9 — 4 ensemble findings, 1 reject (4 reviewers)

willgriffin

committed

fix(review-cycle): round-10 — delete WIP recipe entirely (5 reviewers converged)

willgriffin

committed

fix(review-cycle): round-11 — fix shell injection in snapshot recipe (4 reviewers)

willgriffin

committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(have:review-cycle): operational discipline — Copilot non-optional, loop convergence, blocker docs#5

fix(have:review-cycle): operational discipline — Copilot non-optional, loop convergence, blocker docs#5
willgriffin merged 26 commits into
mainfrom
fix/review-cycle-copilot-required

Commits on May 23, 2026

fix(review-cycle): make Copilot CLI step non-optional + document auth/policy blockers

fix(review-cycle): add hard rules for loop convergence

fix(review-cycle): exit on P3/nit-only — don't loop on trivia

fix(review-cycle): clarify P3 triage — fix inline, record, or follow up

fix(review-cycle): address pr-review round-1 findings on this PR

fix(review-cycle): round-2 — correct Copilot --allow-tool syntax + Status contract

fix(review-cycle): round-3 — workflow contract repairs

fix(review-cycle,ship): round-4 — accepted-P2 stop condition + ship handles partial

fix(review-cycle,ship): round-5 — remove leaked transcripts, fix

fix(review-cycle,ship): round-6 — security gap + workflow contract fixes

fix(review-cycle): round-7 — fully scope Copilot path access + narrow gitignore

revert(review-cycle): walk back wrong-threat-model security overspec

docs(review-cycle,ship): rename codex/claude/copilot → *-cli everywhere

fix(ci): allow multi-scope and slash in commit scope + escape commit msg in error output

fix(review-cycle): one reviewer clean ≠ ensemble convergence

fix(review-cycle,ci): round-1 — close 3 ensemble findings on PR #5

fix(review-cycle,ship): round-2 — close 3 ensemble findings + 1 self-review polish

fix(review-cycle,ship,ci): round-3 — close 6 of 8 ensemble findings (4 reviewers)

fix(review-cycle,ship,ci): round-4 — 4 ensemble findings (4 reviewers, no rejections)

fix(review-cycle,ship): round-5 — 5 of 6 ensemble findings (4 reviewers)

fix(review-cycle,ship): round-6 — 7 ensemble findings (4 reviewers, no rejections)

fix(review-cycle): round-7 — 6 ensemble findings, slot model + bias caveat

fix(review-cycle,ship): round-8 — 6 ensemble findings (1 medium real bug, 5 low polish)

fix(review-cycle,ship): round-9 — 4 ensemble findings, 1 reject (4 reviewers)

fix(review-cycle): round-10 — delete WIP recipe entirely (5 reviewers converged)

fix(review-cycle): round-11 — fix shell injection in snapshot recipe (4 reviewers)