game-fe-agent/skills/work-code-reviewer/SKILL.md

name, description

name	description
work-code-reviewer	현재 브랜치 변경 코드 또는 특정 파일을 받아 팀 공통 지침 기준으로 종합 리뷰를 수행합니다. 컨벤션, 로직, 보안, 성능, 접근성을 통합 검토하고 우선순위별 리포트를 생성합니다. 다음 상황에서 반드시 사용하세요: - "변경된 코드 리뷰해줘", "이 파일 코드 리뷰해줘" - PR 올리기 전 셀프 리뷰가 필요할 때 - 특정 파일의 전반적인 품질을 점검할 때

코드 리뷰어 (work-code-reviewer)

변경 코드 / 파일 → 컨벤션 · 로직 · 보안 · 성능 통합 리뷰 → 우선순위별 리포트.

언제 사용하는가

• MR 올리기 전 셀프 리뷰 / 사전 점검
• 특정 파일의 전반적인 코드 품질 검토
• 리뷰 요청 전 자가 진단

입력

• git diff 출력, 파일 경로 목록, 또는 코드 스니펫
• (선택) 리뷰 초점 (컨벤션 / 로직 / 보안 / 전체)

---

작업 순서

Phase 1: 검토 범위 파악

1. 대상 파일 또는 diff를 읽는다.
2. 파일 유형별로 적용할 검토 항목을 결정한다.
   • .vue → 컨벤션 + 컴포넌트 구조 + 접근성
   • .ts → 타입 안전성 + 로직
   • server/ → 보안 + 에러 핸들링
   • stores/ → 상태 관리 패턴
   • composables/ → 부수효과 최소화

Phase 2: 통합 체크리스트

코드 품질

• 함수/파일 단일 책임 원칙
• 중복 코드 없음 (재사용 가능 composable/유틸 추출 여부)
• 매직 넘버/문자열 → 상수 또는 i18n 키
• 불필요한 console.log 제거

타입 안전성

• any 타입 미사용
• 외부 API 응답 타입 정의
• 함수 파라미터/반환 타입 명시

에러 핸들링

• API 호출 try-catch 또는 onError 처리
• 에러 상태 UI 존재 여부
• 엣지 케이스 (null, undefined, 빈 배열) 처리

성능

• 불필요한 watch / watchEffect 없음
• computed 캐싱 활용
• 과도한 reactive 래핑 없음

보안

• v-html 입력 새니타이징
• 민감 정보 하드코딩 없음
• 클라이언트 노출 불필요한 데이터 없음

Vue 컨벤션 (.vue 파일)

verify-component-review 스킬의 체크리스트 전체 적용.

Phase 3: 리뷰 코멘트 작성

work-mr-reviewer의 코멘트 형식과 동일:

• 🚨 Critical / ⚠️ Warning / 💡 Nit 3단계 분류
• 파일명:라인번호 명시
• 수정 코드 예시 포함

Phase 4: 요약

개선 전/후 예상 품질 변화를 간단히 서술한다.

---

출력 형식

## 코드 리뷰: <파일명 또는 기능명>

### 검토 요약
- 파일: N개
- 주요 이슈: [컨벤션 | 로직 | 보안 | 성능]

### 🚨 Critical (N건)
[파일:라인] 문제 + 수정 코드

### ⚠️ Warning (N건)
[파일:라인] 문제 + 수정 방향

### 💡 Nit (N건)
[목록]

### ✅ 잘된 점
[1~3개]

### 개선 후 기대 효과
[한 줄 요약]