Skip to content

ci: tailor Pullfrog PR reviews to Memory Bee - #49

Merged
lippdev merged 3 commits into
mainfrom
codex/pullfrog-pr-review
Sep 25, 2026
Merged

lippdev merged 3 commits into
mainfrom
codex/pullfrog-pr-review

Conversation

@lippdev

@lippdev lippdev commented Sep 25, 2026 •

Copy link
Copy Markdown
Owner

Problema e resultado

Pullfrog já revisava PRs, mas prompts.review estava vazio. Configurei instruções curtas para os invariantes de leitura, exportação, Git, privacidade e permissões do workspace, com achados acionáveis em português.

Escopo

  • .github/pullfrog/review.md versiona o texto sincronizado na configuração externa; o workflow não lê esse arquivo automaticamente.
  • CONTRIBUTING.md registra como sincronizar e distingue revisão automática de revisão independente.
  • docs/EXECUTION.md registra estado e evidência; docs/MANUAL_TESTS.md acrescenta o cenário 53.
  • Mantidos os gatilhos existentes, aprovação automática e auto-merge desligados.

Validação e revisão

  • npx --yes pullfrog@latest config set prompts.review --file .github/pullfrog/review.md --repo lippdev/memory-bee: updated; get devolveu o texto completo.
  • Links locais novos conferidos; git diff --check passou.
  • Pullfrog revisou o primeiro commit em português, apontou ambiguidade de sincronização, corrigida no commit seguinte; a revisão incremental não trouxe novos achados. Autorrevisão do diff; revisão automatizada, sem revisão humana independente.
  • CI do commit e542f2a: Linux passou; macOS falhou uma vez por timeout no teste sintético de PTY e passou no rerun do mesmo commit. Intermitência registrada na [P1][CI/Workspace] Investigar timeout intermitente no teste Claude PTY oculto em macOS #50. Checks do commit final devem passar antes de integrar.
  • Ensaio de rascunho no cenário manual 53 segue pendente.

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ℹ️ Nenhum problema crítico; há uma sugestão menor inline.

Reviewed changes

Este parecer cobre o commit 042a114, que adiciona instruções versionadas para as revisões Pullfrog, documenta a sincronização de prompts.review e registra a evidência e o ensaio manual pendente.

  • Instruções de revisão — adiciona critérios específicos para leitores, exportadores, Git, privacidade, workspace e validação.
  • Procedimento operacional — documenta como sincronizar a configuração externa do Pullfrog após alterar o arquivo versionado.
  • Rastreamento e validação — registra a configuração aplicada e adiciona o cenário manual para revisão inicial, incremental e de rascunho.

Pullfrog  | Fix all ➔ | Fix 👍s ➔ | View workflow run | Using GPT Luna | 𝕏

Comment thread CONTRIBUTING.md Outdated

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Nenhum problema novo encontrado.

Reviewed changes

Esta revisão incremental cobre as alterações desde o parecer Pullfrog anterior, com foco no esclarecimento da sincronização externa de prompts.review e no registro correspondente.

  • Sincronização documentada — esclareceu que .github/pullfrog/review.md é uma cópia versionada e que o workflow não a lê automaticamente.
  • Evidência atualizada — registrou que a ambiguidade apontada no primeiro parecer foi corrigida, mantendo o ensaio manual 53 explicitamente pendente.

Pullfrog  | View workflow run | Using GPT Luna | 𝕏

@lippdev
lippdev merged commit 531f20b into main Sep 25, 2026
3 checks passed
@lippdev
lippdev deleted the codex/pullfrog-pr-review branch September 25, 2026 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant