From 042a11428e54adc9a59821d149be5e6a5e1a84b7 Mon Sep 17 00:00:00 2001 From: Filipe Moreira Date: Fri, 25 Sep 2026 16:30:46 -0300 Subject: [PATCH 1/3] ci: tailor Pullfrog PR reviews to Memory Bee --- .github/pullfrog/review.md | 18 ++++++++++++++++++ CONTRIBUTING.md | 17 +++++++++++++++++ docs/EXECUTION.md | 14 ++++++++++++++ docs/MANUAL_TESTS.md | 15 +++++++++++++++ 4 files changed, 64 insertions(+) create mode 100644 .github/pullfrog/review.md diff --git a/.github/pullfrog/review.md b/.github/pullfrog/review.md new file mode 100644 index 0000000..c63d434 --- /dev/null +++ b/.github/pullfrog/review.md @@ -0,0 +1,18 @@ +Revise o diff do PR como revisor de código. Leia AGENTS.md e os contratos/ADRs +afetados. Responda em português do Brasil, preservando nomes de código e comandos. + +Priorize achados reproduzíveis nas linhas alteradas, com impacto, condição que +dispara o problema e correção sugerida. Confira especialmente: + +- Leitores e exportadores: ordem, proveniência, omissões, limites, privacidade e + exportação básica sem chamada a modelo, mesmo com a origem indisponível. +- Pacotes e Git: hashes, caminhos seguros, base/checkout exatos, alterações locais + explícitas e ausência de publicação implícita de código. +- Workspace/TUI/PTY: permissões negadas por padrão em falhas, isolamento de + projeto/perfil, interrupção, retomada e restauração do terminal. +- Testes e documentação: casos de falha relevantes; distinguir fixture sintética, + ensaio humano e prova com agente real. Não tratar teste automatizado como + validação manual ou funcionalidade planejada como entregue. + +Evite sugestões de estilo, refatorações fora do escopo e riscos hipotéticos sem +um caminho concreto de falha. Se não houver achados acionáveis, diga isso. diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index ccb21bf..c04cebf 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -64,6 +64,23 @@ mantenedor, podem ocorrer depois sem impedir implementações testadas automatic O corpo deve conter problema e resultado, escopo, validação, revisão e limitações relevantes. Preferir uma explicação curta e concreta. Manter título e descrição atualizados com a implementação final. +### Revisão automática com Pullfrog + +O Pullfrog revisa PRs novos e novos commits. O texto versionado em +[`.github/pullfrog/review.md`](.github/pullfrog/review.md) é a fonte das instruções +de revisão configuradas em `prompts.review` no repositório do Pullfrog. Após +alterá-lo e integrar o PR, sincronizar com: + +```sh +npx --yes pullfrog@latest config set prompts.review --file .github/pullfrog/review.md --repo lippdev/memory-bee +npx --yes pullfrog@latest config get prompts.review --repo lippdev/memory-bee +``` + +O Pullfrog é uma revisão automatizada; seu parecer não substitui autorrevisão, +testes, ensaios humanos nem aprovação independente. O workflow gerado em +`.github/workflows/pullfrog.yml` é administrado pelo serviço; ajustar as +instruções pela configuração acima, sem editar os gatilhos no YAML. + ## Dados e exportações Não versionar conversas reais, credenciais ou pacotes de usuários. Usar fixtures sintéticas. Exportações devem ficar fora do repositório ou em `exports/`, ignorado pelo Git. Uma exportação de contexto não autoriza commit, push ou publicação do código do usuário. diff --git a/docs/EXECUTION.md b/docs/EXECUTION.md index ee2a045..fc6e6ae 100644 --- a/docs/EXECUTION.md +++ b/docs/EXECUTION.md @@ -34,6 +34,20 @@ - Próximo recorte de implementação: #29, seguido da prova de permissões #30; os ensaios P0 #25–#26 podem avançar em paralelo quando o mantenedor os fizer. +## Revisão automática de PRs — 2026-09-25 + +- Pullfrog já estava ativo para PRs novos e novos commits; configuração inicial + tinha `prompts.review` vazio, aprovação automática e auto-merge desligados. +- Instruções específicas de revisão foram versionadas em + [`.github/pullfrog/review.md`](../.github/pullfrog/review.md). O procedimento + de sincronização está no [guia de contribuição](../CONTRIBUTING.md), e o + ensaio humano permanece pendente no item 53 do [roteiro](MANUAL_TESTS.md). +- `npx --yes pullfrog@latest config set prompts.review --file + .github/pullfrog/review.md --repo lippdev/memory-bee` retornou `updated`; `get` + devolveu o texto completo. Validar o primeiro parecer no PR e a revisão + incremental em PR futuro; isso não altera o status das etapas do produto. + Autorrevisão documental; sem revisão independente registrada nesta entrega. + ## Revisão de sequência — orientação do mantenedor Em 2026-09-24, o mantenedor pediu continuidade das implementações e deixou os diff --git a/docs/MANUAL_TESTS.md b/docs/MANUAL_TESTS.md index efebef4..ea93180 100644 --- a/docs/MANUAL_TESTS.md +++ b/docs/MANUAL_TESTS.md @@ -901,3 +901,18 @@ ensaio humano. Exportação integrada e perfis reais ainda pendentes. | Item novo | Estado | Evidência manual | |---|---|---| | 52: UI Bee sobre Claude real oculto, ferramentas, permissões e retomada | Pendente | — | + +## 53. Revisão do Pullfrog em PR (pendente) + +Após integrar `.github/pullfrog/review.md` e sincronizar `prompts.review` pelo +comando em `CONTRIBUTING.md`, abrir um PR pequeno com uma mudança revisável. +Conferir no PR que o check `pullfrog` termina e que há parecer em português, +com referência a um risco concreto quando existir. Enviar um novo commit ao PR +e conferir a revisão incremental. Testar um PR de rascunho: a revisão automática +deve aguardar a marcação como pronto. Um parecer sem achados não prova que todos +os comportamentos foram testados. Anotar links dos PRs/runs e qualquer falha; +não contar esta seção como executada pelo resultado de CI deste ajuste. + +| Item novo | Estado | Evidência manual | +|---|---|---| +| 53: revisão inicial, incremental e rascunho | Pendente | — | From e542f2aa17cb81b2cfb109560d29743d943233e0 Mon Sep 17 00:00:00 2001 From: Filipe Moreira Date: Fri, 25 Sep 2026 16:36:04 -0300 Subject: [PATCH 2/3] docs: clarify Pullfrog prompt synchronization --- CONTRIBUTING.md | 6 +++--- docs/EXECUTION.md | 8 +++++--- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index c04cebf..576f1f7 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -67,9 +67,9 @@ O corpo deve conter problema e resultado, escopo, validação, revisão e limita ### Revisão automática com Pullfrog O Pullfrog revisa PRs novos e novos commits. O texto versionado em -[`.github/pullfrog/review.md`](.github/pullfrog/review.md) é a fonte das instruções -de revisão configuradas em `prompts.review` no repositório do Pullfrog. Após -alterá-lo e integrar o PR, sincronizar com: +[`.github/pullfrog/review.md`](.github/pullfrog/review.md) é a cópia versionada das +instruções de `prompts.review`. O workflow não lê esse arquivo automaticamente. +Após alterá-lo e integrar o PR, sincronizar a configuração externa com: ```sh npx --yes pullfrog@latest config set prompts.review --file .github/pullfrog/review.md --repo lippdev/memory-bee diff --git a/docs/EXECUTION.md b/docs/EXECUTION.md index fc6e6ae..bfd74a1 100644 --- a/docs/EXECUTION.md +++ b/docs/EXECUTION.md @@ -44,9 +44,11 @@ ensaio humano permanece pendente no item 53 do [roteiro](MANUAL_TESTS.md). - `npx --yes pullfrog@latest config set prompts.review --file .github/pullfrog/review.md --repo lippdev/memory-bee` retornou `updated`; `get` - devolveu o texto completo. Validar o primeiro parecer no PR e a revisão - incremental em PR futuro; isso não altera o status das etapas do produto. - Autorrevisão documental; sem revisão independente registrada nesta entrega. + devolveu o texto completo. O primeiro parecer no PR #49 veio em português e + apontou ambiguidade na explicação da sincronização; o texto foi corrigido. + A revisão incremental e o ensaio do item 53 seguem pendentes; isso não altera + o status das etapas do produto. Autorrevisão documental; Pullfrog é revisor + automatizado, não revisão humana independente. ## Revisão de sequência — orientação do mantenedor From 3f52161f5f89447339363542e9cc6c628ec817ae Mon Sep 17 00:00:00 2001 From: Filipe Moreira Date: Fri, 25 Sep 2026 16:43:24 -0300 Subject: [PATCH 3/3] docs: record Pullfrog review and macOS CI flake --- docs/EXECUTION.md | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/docs/EXECUTION.md b/docs/EXECUTION.md index bfd74a1..c0ac4e8 100644 --- a/docs/EXECUTION.md +++ b/docs/EXECUTION.md @@ -46,9 +46,14 @@ .github/pullfrog/review.md --repo lippdev/memory-bee` retornou `updated`; `get` devolveu o texto completo. O primeiro parecer no PR #49 veio em português e apontou ambiguidade na explicação da sincronização; o texto foi corrigido. - A revisão incremental e o ensaio do item 53 seguem pendentes; isso não altera - o status das etapas do produto. Autorrevisão documental; Pullfrog é revisor - automatizado, não revisão humana independente. + A revisão incremental no commit `e542f2a` confirmou a correção sem novos + achados. CI Linux passou; o primeiro job macOS falhou por timeout no teste + sintético de PTY e passou no rerun do mesmo commit. A intermitência foi + registrada na [issue #50](https://github.com/lippdev/memory-bee/issues/50). + O ensaio de rascunho do item 53 segue pendente. Isso não altera o status das + etapas do produto. + Autorrevisão documental; Pullfrog é revisor automatizado, não revisão humana + independente. ## Revisão de sequência — orientação do mantenedor