fix(ops OPS57): simplify — veto de draft vale em conversão mid-run; CLI valida base main; docs sem mecanismo fantasma #52

Merged

fsolla merged 1 commit from OPS57-guard-forgejo-pr-automerge-alargar-gatilho-do-workflow into main

2026-08-18 15:24:21 +00:00

fsolla commented

2026-08-18 15:19:42 +00:00

Owner

Follow-up do OPS57 (#44, PR #48 mergeada): fixes pontuais do /simplify (3 reviewers) na mesma entrega.

waitForChecks/autoMerge (forgejo-api.mjs): param quando o PR vira draft no meio do wait (antes: merge POST → 405 → re-throw ruidoso na conversão mid-run, contradizendo o "veto nunca tocado" documentado). Agora o veto vale literalmente a qualquer momento; spec nova em forgejoApi.unit.spec.ts.
CLI (forgejo-pr-automerge.mjs): valida base.ref === 'main' (invocação manual com PR errado); mensagem distingue "fechado sem merge" de "voltou a draft — skip"; early-exit no branch de draft.
Comentário do fork guard no workflow: documenta por que o guard de same-repo não pode descer pro CLI — em PR de fork o checkout já roda o script do fork (RCE com o token do Actions); o guard tem que estar no if: pré-checkout.
Docs sem o mecanismo fantasma gh pr ready/gh pr merge --auto --rebase (era a era GitHub; OPS50 trocou por CLI plain-Node): .agents/rules/agent-pr-workflow.mdc (heading "Safety net (workflow)" + mecanismo real) e docs/AGENT-OPS.md (tabela).
Changelog da entrega atualizado: validação ao vivo observada (o Forgejo roda o workflow do head da PR — a PR #48 foi mergeada pelo safety net alargado no primeiro uso real) e fixes listados.

Validação: 2319 unit (1 spec nova) + lint/typecheck/format/knip/cycles verdes; gate:ci mirror passou no push. Esta PR exercita o safety net de novo (workflow do head), agora com o veto mid-run.

Follow-up do OPS57 (#44, PR #48 mergeada): fixes pontuais do /simplify (3 reviewers) na mesma entrega. - `waitForChecks`/`autoMerge` (forgejo-api.mjs): param quando o PR vira **draft no meio do wait** (antes: merge POST → 405 → re-throw ruidoso na conversão mid-run, contradizendo o "veto nunca tocado" documentado). Agora o veto vale literalmente a qualquer momento; spec nova em `forgejoApi.unit.spec.ts`. - CLI (`forgejo-pr-automerge.mjs`): valida `base.ref === 'main'` (invocação manual com PR errado); mensagem distingue "fechado sem merge" de "voltou a draft — skip"; early-exit no branch de draft. - Comentário do fork guard no workflow: documenta **por que o guard de same-repo não pode descer pro CLI** — em PR de fork o checkout já roda o script do fork (RCE com o token do Actions); o guard tem que estar no `if:` pré-checkout. - Docs sem o mecanismo fantasma `gh pr ready`/`gh pr merge --auto --rebase` (era a era GitHub; OPS50 trocou por CLI plain-Node): `.agents/rules/agent-pr-workflow.mdc` (heading "Safety net (workflow)" + mecanismo real) e `docs/AGENT-OPS.md` (tabela). - Changelog da entrega atualizado: validação ao vivo observada (o Forgejo roda o workflow do head da PR — a PR #48 foi mergeada pelo safety net alargado no primeiro uso real) e fixes listados. Validação: 2319 unit (1 spec nova) + lint/typecheck/format/knip/cycles verdes; gate:ci mirror passou no push. Esta PR exercita o safety net de novo (workflow do head), agora com o veto mid-run.

fsolla added 1 commit 2026-08-18 15:19:42 +00:00