review

review 개요

review가 하는 일

review는 코드가 머지되기 전에 기준 브랜치와의 diff를 확인하는 사전 랜딩 PR 리뷰 skill입니다. 단순한 범용 프롬프트보다 더 구체적인 리뷰가 필요할 때 쓰도록 설계되어 있으며, SQL 안전성, 신뢰 경계 위반, 조건부 부작용, 셸 인젝션, enum 완결성처럼 실제 구조적 위험에 초점을 맞춥니다.

누가 사용해야 하나요

코드를 랜딩하기 직전이거나, “이 PR 리뷰해 주세요” 요청에 답해야 하거나, 여러 repo에서 일관된 review for PR Review 워크플로를 만들고 싶을 때 review skill을 사용하세요. 특히 실제 결함을 빠르게 잡아내야 하고, 잡음이 많은 스타일 중심 코멘트는 피하고 싶은 agent에게 유용합니다.

무엇이 다른가요

이 skill은 리뷰 순서와 심각도에 대해 꽤 의견이 분명합니다. 먼저 치명적인 안전성 검사를 하고, 그다음 낮은 심각도의 이슈를 다루며, 테스트·보안·성능·유지보수성 같은 영역에 대해서는 전문 리뷰 경로도 지원합니다. 그래서 넓게 “diff를 훑어보라”는 프롬프트보다 review guide가 훨씬 실행 가능한 결과를 줍니다.

적합한 경우와 한계

명확한 git diff와 의미 있는 base branch가 있는 코드 변경에 가장 강합니다. 반대로 고수준 설계 비평, 제품 카피 수정, 또는 사용자가 칭찬이나 요약만 원하는 경우에는 효용이 떨어집니다. 고치기 중심의 리뷰가 아니라 가벼운 의견만 필요하다면, 일반 프롬프트로도 충분할 수 있습니다.

review skill 사용 방법

설치하고 트리거하기

npx skills add garrytan/gstack --skill review로 설치하세요. 이 skill은 “review this PR”, “code review”, “check my diff”, “pre-landing review” 같은 표현에서 트리거되도록 설계되어 있으므로, 가능한 한 변경 내용과 비교 기준을 입력에 함께 적어야 합니다.

skill이 볼 실제 리뷰 대상을 주기

좋은 review usage는 모호한 요청이 아니라 diff를 인식할 수 있는 요청에서 시작합니다. repo, branch, 의도를 함께 적는 입력이 좋습니다.

• “origin/main 대비 이 PR을 리뷰해 주세요. 안전성, merge risk, 그리고 랜딩을 막을 만한 요소를 중심으로 봐 주세요.”
• “packages/billing의 diff를 검토해서 SQL injection, race condition, API contract break를 찾아 주세요.”

먼저 읽어야 할 파일

실무적인 review install 검증을 하려면 먼저 SKILL.md를 보고, 그다음 checklist.md, design-checklist.md, greptile-triage.md를 읽으세요. 워크플로를 조정해야 한다면 TODOS-format.md와 specialists/ 폴더도 확인해서 어떤 이슈가 서브에이전트로 보내지고, 어떤 이슈가 메인 체크리스트에서 처리되는지 이해하는 것이 좋습니다.

skill이 기대하는 워크플로를 따르기

이 repo는 두 단계 리뷰를 중심으로 구성되어 있습니다. 먼저 치명적인 안전성 검사를 하고, 그다음 정보성 이슈를 봅니다. 강한 프롬프트는 단순히 “버그를 찾아줘”가 아니라, 파일/라인 참조와 수정 방향이 포함된 실행 가능한 출력을 요구해야 합니다. 변경 영역을 알고 있다면 함께 적어 주세요. 그러면 skill이 관련 specialist 경로를 우선순위로 둘 수 있습니다.

review skill FAQ

review가 일반 프롬프트보다 더 좋은가요?

반복 가능한 PR 리뷰 동작이 필요하다면 그렇습니다. 일반 프롬프트는 범위 규칙, 에스컬레이션 순서, specialist 라우팅을 놓칠 수 있습니다. review skill은 이런 요소를 워크플로에 미리 담아 두기 때문에, 무엇을 먼저 봐야 할지 덜 헷갈립니다.

review skill은 초보자도 쓰기 쉬운가요?

브랜치나 PR을 설명할 수만 있다면 대체로 그렇습니다. 초보자는 안내된 체크리스트의 도움을 받지만, 그래도 실제 diff와 충분한 컨텍스트는 제공해야 의도된 동작과 회귀를 구분할 수 있습니다.

언제 review를 쓰지 말아야 하나요?

코드가 없는 질문, 브레인스토밍, 또는 구체적인 변경 집합이 없는 추상적인 아키텍처 논의에는 쓰지 마세요. 또한 빠른 sanity check만 필요하고 라인 단위 결과가 필요하지 않다면 이 skill은 과할 수 있습니다.

review for PR Review에서는 무엇을 기대해야 하나요?

심각도 우선순위가 반영된, 간결하고 수정 중심적인 코멘트를 기대하세요. 이 skill은 repo 전체를 다시 설명하거나 축하성 요약을 만드는 용도가 아니라, 랜딩을 막을 문제를 드러내도록 설계되어 있습니다.

review skill 개선 방법

입력을 더 날카롭게 만들기

가장 강한 review skill 입력은 기준 브랜치, 위험한 하위 시스템, 그리고 원하는 기준을 함께 적습니다. “내 PR 좀 리뷰해줘” 대신 “feature/payment-retry를 origin/main과 비교해서 리뷰해 주세요. 데이터 안전성, idempotency, 클라이언트가 깨질 수 있는 동작 변경을 우선 봐 주세요.”처럼 쓰는 것이 좋습니다.

판단을 바꾸는 컨텍스트를 함께 주기

변경이 의도적으로 위험한 것이라면 그 사실을 밝히세요. 프로젝트 정책상 허용되는 패턴이 있다면 미리 적어 두세요. 그러면 false positive를 줄이고, 이미 정해진 결정을 다시 다투는 대신 실제 tradeoff에 리뷰가 집중됩니다.

필요한 출력 형식을 요청하기

랜딩 관점의 출력을 원한다면 file:line, 심각도, 구체적인 수정안을 포함해 달라고 요청하세요. blocker만 원한다면 그렇게 말하세요. specialist coverage가 필요하다면 그것도 명시해야 리뷰가 첫 번째 패스에서 멈추지 않습니다.

첫 번째 패스 뒤에 다시 반복하기

첫 리뷰로 명백한 문제를 먼저 수정한 뒤, 업데이트된 diff에 대해 review를 다시 실행하세요. 품질 향상은 대체로 범위 명확화, 비교 브랜치 정밀화, 그리고 가장 중요하게 보는 위험 영역을 정확히 입력하는 데서 가장 크게 나옵니다.