review

Visão geral da skill de review

O que a skill de review faz

review é uma skill de review de PR pré-merge para verificar diffs em relação à branch base antes de o código ser incorporado. Ela foi pensada para revisores que precisam de mais do que um prompt genérico: o foco está em riscos estruturais concretos, como segurança em SQL, violações de trust boundary, efeitos colaterais condicionais, injeção via shell e completude de enums.

Quem deve usar

Use a skill de review quando você estiver prestes a subir código, responder a um “review this PR” ou quiser um fluxo consistente de review for PR Review entre repositórios. Ela é especialmente útil para agentes que precisam apontar defeitos reais com rapidez e evitar comentários barulhentos focados só em estilo.

Por que ela é diferente

A skill é opinativa quanto à ordem da revisão e à severidade. Ela prioriza primeiro as checagens críticas de segurança, depois os problemas de menor severidade, e oferece caminhos de review especializados para áreas como testes, segurança, performance e manutenibilidade. Isso torna o review guide mais acionável do que um prompt amplo de “varra o diff”.

Aderência e limitações

Ela funciona melhor para mudanças de código com um diff git claro e uma branch base relevante. É menos útil para críticas de design em alto nível, ajustes de texto de produto ou casos em que a pessoa só quer elogio ou um resumo. Se você precisa de uma opinião leve, em vez de uma revisão orientada a correções, um prompt genérico pode ser suficiente.

Como usar a skill de review

Instale e acione

Instale com npx skills add garrytan/gstack --skill review. A skill foi projetada para disparar em frases como “review this PR”, “code review”, “check my diff” e “pre-landing review”, então o input deve nomear a mudança e o ponto de comparação sempre que possível.

Dê à skill um alvo real de review

O melhor review usage começa com um pedido ciente do diff, não com uma solicitação vaga. Bons inputs incluem o repositório, a branch e a intenção:

• “Review this PR against origin/main; focus on safety, merge risk, and anything that would block landing.”
• “Review the diff in packages/billing for SQL injection, race conditions, and API contract breaks.”

Leia estes arquivos primeiro

Para validar na prática o review install, comece por SKILL.md, depois checklist.md, design-checklist.md e greptile-triage.md. Se você estiver adaptando o fluxo, também vale inspecionar TODOS-format.md e a pasta specialists/ para entender quais issues são encaminhadas para subagents e quais ficam na checklist principal.

Use o fluxo que a skill espera

O repositório é estruturado em uma revisão em duas passadas: primeiro as checagens críticas de segurança, depois os issues informativos. Um prompt forte deve pedir saída acionável, com referências de arquivo/linha e correções, e não apenas “encontre bugs”. Se você souber a área da mudança, inclua isso para que a skill priorize o caminho especializado mais relevante.

FAQ da skill de review

A skill de review é melhor do que um prompt normal?

Sim, quando você precisa de um comportamento de review de PR repetível. Um prompt normal pode deixar passar regras de escopo, ordem de escalonamento ou roteamento para especialistas. A skill de review incorpora isso ao fluxo, o que reduz a adivinhação sobre o que inspecionar primeiro.

A skill de review é amigável para iniciantes?

Na maior parte, sim, desde que a pessoa consiga descrever a branch ou o PR. Iniciantes se beneficiam da checklist guiada, mas ainda precisam fornecer um diff real e contexto suficiente para distinguir comportamento intencional de regressões.

Quando não devo usar review?

Não use para perguntas sem código, brainstorming ou discussões abstratas de arquitetura sem um conjunto concreto de mudanças. Também não é a opção certa se você só quer uma checagem rápida de sanidade e não precisa de achados em nível de linha.

O que devo esperar de review for PR Review?

Espere comentários curtos, orientados a correção, com priorização por severidade. A skill foi desenhada para expor bloqueadores de merge, e não para repetir o repositório inteiro nem produzir um resumo comemorativo.

Como melhorar a skill de review

Faça inputs mais precisos

Os inputs mais fortes de review skill nomeiam a branch base, o subsistema de risco e o padrão desejado. Em vez de “review my PR”, use “review feature/payment-retry against origin/main; prioritize data safety, idempotency, and any behavior change that could break clients.”

Inclua o contexto que muda o julgamento

Se a mudança é intencionalmente arriscada, diga isso. Se um padrão é permitido pela política do projeto, mencione logo de início. Isso reduz falsos positivos e ajuda a revisão a focar nas trocas reais, em vez de reabrir decisões já conhecidas.

Peça o formato de que você precisa

Se você quer uma saída voltada para landing, solicite achados com file:line, severidade e correções concretas. Se quiser apenas bloqueadores, diga isso. Se quiser cobertura especializada, peça explicitamente para a review não parar na primeira passada.

Itere depois da primeira passada

Use a primeira review para corrigir os problemas óbvios e, depois, rode review de novo no diff atualizado. Os maiores ganhos de qualidade normalmente vêm de esclarecer o escopo, apertar a branch de comparação e alimentar a skill com a área exata de risco que mais importa para você.