binary-agents 1.0.13 → 1.0.14

package/commands/pr.md

@@ -1,79 +1,79 @@
 ---
-description: Analyze branch differences, commits, and changed files to generate a pull request
+description: 브랜치 차이, 커밋, 변경 파일을 분석하여 Pull Request 자동 생성
 allowed-tools: Bash(git status:*), Bash(git diff:*), Bash(git log:*), Bash(git branch:*), Bash(git rev-parse:*), Bash(gh pr:*), Bash(gh auth:*)
 ---
 
-# Auto PR Generator
+# 자동 PR 생성기
 
-You are a pull request generator that analyzes the differences between the current branch and the main branch to create an appropriate PR.
+현재 브랜치와 main 브랜치의 차이를 분석하여 적절한 PR을 생성합니다.
 
-## Context Information
+## 컨텍스트 정보
 
-**Current branch:**
+**현재 브랜치:**
 !`git rev-parse --abbrev-ref HEAD`
 
-**Main branch (target):**
+**Main 브랜치 (타겟):**
 !`git symbolic-ref refs/remotes/origin/HEAD 2>/dev/null | sed 's@^refs/remotes/origin/@@' || echo "main"`
 
-**Commits in this branch (not in main):**
-!`git log --oneline main..HEAD 2>/dev/null || git log --oneline origin/main..HEAD 2>/dev/null || echo "No commits found"`
+**이 브랜치의 커밋 (main에 없는):**
+!`git log --oneline main..HEAD 2>/dev/null || git log --oneline origin/main..HEAD 2>/dev/null || echo "커밋을 찾을 수 없습니다"`
 
-**Changed files summary:**
-!`git diff --stat main..HEAD 2>/dev/null || git diff --stat origin/main..HEAD 2>/dev/null || echo "No changes found"`
+**변경 파일 요약:**
+!`git diff --stat main..HEAD 2>/dev/null || git diff --stat origin/main..HEAD 2>/dev/null || echo "변경사항을 찾을 수 없습니다"`
 
-**Detailed changes:**
-!`git diff main..HEAD --name-status 2>/dev/null || git diff origin/main..HEAD --name-status 2>/dev/null || echo "No changes found"`
+**상세 변경사항:**
+!`git diff main..HEAD --name-status 2>/dev/null || git diff origin/main..HEAD --name-status 2>/dev/null || echo "변경사항을 찾을 수 없습니다"`
 
-**Current git status:**
+**현재 git 상태:**
 !`git status --short`
 
-## Your Task
+## 작업 순서
 
-1. **Analyze the branch information**
-   - Identify the current branch name
-   - Confirm the target branch (usually main or master)
-   - Check if there are unpushed commits
+1. **브랜치 정보 분석**
+   - 현재 브랜치 이름 확인
+   - 타겟 브랜치 확인 (보통 main 또는 master)
+   - push되지 않은 커밋이 있는지 확인
 
-2. **Analyze the commits**
-   - Review all commits that will be included in the PR
-   - Understand the overall purpose of these changes
-   - Identify if this is a feature, bugfix, refactor, etc.
+2. **커밋 분석**
+   - PR에 포함될 모든 커밋 검토
+   - 변경사항의 전체적인 목적 파악
+   - feature, bugfix, refactor 등 유형 식별
 
-3. **Analyze the changed files**
-   - What files were modified/added/deleted?
-   - Which areas of the codebase are affected?
-   - Are there any breaking changes?
+3. **변경 파일 분석**
+   - 어떤 파일이 수정/추가/삭제되었는가?
+   - 코드베이스의 어느 영역이 영향받는가?
+   - breaking changes가 있는가?
 
-4. **Generate PR title and description**
-   - **Title**: Concise summary in Korean (following commit convention if exists)
-   - **Description**: Include the following sections:
-     - `## 변경 사항` (Summary of changes)
-     - `## 변경된 파일` (List of key changed files)
-     - `## 테스트` (How to test, if applicable)
-   - **Do NOT add "🤖 Generated with Claude Code" or any AI attribution footer**
+4. **PR 제목 및 설명 생성**
+   - **제목**: 한국어로 간결한 요약 (커밋 컨벤션이 있으면 따르기)
+   - **설명**: 다음 섹션 포함:
+     - `## 변경 사항` (변경사항 요약)
+     - `## 변경된 파일` (주요 변경 파일 목록)
+     - `## 테스트` (해당되는 경우 테스트 방법)
+   - **"Generated with Claude Code" 또는 AI attribution footer 추가 금지**
 
-5. **Create the PR**
-   - First, ensure the branch is pushed to remote
-   - Use `gh pr create` with the generated title and body
-   - If PR already exists, inform the user
+5. **PR 생성**
+   - 먼저 브랜치가 remote에 push되었는지 확인
+   - 생성된 title과 body로 `gh pr create` 사용
+   - PR이 이미 존재하면 사용자에게 알림
 
-## Output Format
+## 출력 형식
 
-First, briefly explain:
-- Current branch and target branch
-- Number of commits to be included
-- Summary of changes
+먼저 간략히 설명:
+- 현재 브랜치와 타겟 브랜치
+- 포함될 커밋 수
+- 변경사항 요약
 
-Then ask for confirmation before creating the PR, showing the proposed title and description.
+그 다음 PR 생성 전 확인 요청, 제안된 title과 description 표시.
 
-After confirmation, execute:
+확인 후 실행:
 ```bash
 gh pr create --title "title" --body "body"
 ```
 
-## Error Handling
+## 에러 처리
 
-- If not logged into GitHub CLI: Guide user to run `gh auth login`
-- If branch not pushed: Offer to push with `git push -u origin <branch>`
-- If PR already exists: Show the existing PR URL
-- If on main branch: Inform user to create a feature branch first
+- GitHub CLI 로그인 안됨: `gh auth login` 실행 안내
+- 브랜치 미푸시: `git push -u origin <branch>`로 push 제안
+- PR 이미 존재: 기존 PR URL 표시
+- main 브랜치에서 실행: feature 브랜치 먼저 생성하라고 안내

package/commands/review-pr.md

@@ -0,0 +1,231 @@
+---
+description: PR 링크를 받아 변경사항을 분석하고 GitHub 스타일로 라인별 코드 리뷰 제공
+allowed-tools: Task, Skill, Bash(gh pr:*), Bash(gh api:*), Read, Glob, Grep
+---
+
+# PR 코드 리뷰어
+
+GitHub PR 링크를 받아 변경된 파일들을 분석하고, 실제 GitHub 리뷰처럼 라인별 코멘트를 제공합니다.
+
+## 컨텍스트 정보
+
+**PR 링크 또는 번호:**
+!`echo "${1:-}"`
+
+## 사용 가능한 리뷰 에이전트
+
+Task 도구를 통해 다음 전문 에이전트를 사용할 수 있습니다:
+
+| Agent | 초점 영역 |
+|-------|----------|
+| `code-reviewer` | 아키텍처, 타입 안전성, 에러 처리, 테스트, 접근성, 보안 |
+| `fundamentals-code` | Toss Frontend Fundamentals 기반 (가독성, 예측 가능성, 응집도, 결합도) |
+| `refactor-analyzer` | 코드 중복, 복잡성, 추상화 기회, 코드 스멜 |
+| `junior-checker` | 주니어 개발자 관점 가독성, 네이밍, 복잡도 |
+| `react-performance-optimizer` | React 리렌더, 메모이제이션, 훅 최적화 |
+
+## 사용 가능한 Skill
+
+Skill은 사용자 설치에 따라 다르며 추가 리뷰 가이드라인/컨텍스트를 제공합니다:
+- `vercel-react-best-practices` - Vercel의 React/Next.js 최적화 가이드라인
+- 커스텀 팀 코딩 표준
+- 프레임워크별 best practices
+
+**참고:** Skill은 Skill 도구로 로드되며 직접 분석이 아닌 리뷰 컨텍스트를 제공합니다.
+
+## 작업 순서
+
+1. **PR 정보 가져오기**
+
+   ```bash
+   # PR 정보
+   gh pr view <PR번호> --json title,body,author,baseRefName,headRefName,changedFiles,additions,deletions
+
+   # 변경된 파일 목록
+   gh pr diff <PR번호> --name-only
+
+   # 변경 내용 (diff)
+   gh pr diff <PR번호>
+   ```
+
+2. **사용자에게 리뷰 유형 질문** (AskUserQuestion 사용)
+
+   옵션 제시:
+
+   | 옵션 | 이름 | 사용 에이전트 | 적합한 상황 |
+   |------|------|--------------|-------------|
+   | 1 | **전체 리뷰** | 모든 5개 에이전트 병렬 실행 | 종합 코드 리뷰 (권장) |
+   | 2 | **커스텀** | 사용자가 직접 선택 | 특정 관점만 리뷰하고 싶을 때 |
+
+3. **Skill 포함 여부 질문** (AskUserQuestion 사용)
+
+   리뷰 유형 선택 후:
+   > "포함할 skill이 있나요? (예: `vercel-react-best-practices`, 팀 코딩 가이드 등)"
+
+   옵션:
+   - **없음** - skill 없이 에이전트만 실행
+   - **있음** - skill 이름 입력받아서 로드
+
+   사용자가 skill 이름 제공 시:
+   - 에이전트 실행 전 `Skill(<skill-name>)`으로 각 skill 로드
+   - Skill은 리뷰를 위한 추가 컨텍스트/가이드라인 제공
+
+4. **선택된 에이전트 병렬 실행**
+
+   Task 도구로 에이전트를 **동시에** 실행. 각 에이전트에게 PR diff를 전달:
+
+   ```
+   Task(code-reviewer): "다음 PR의 변경사항을 리뷰해주세요.
+
+   [PR 정보]
+   - 제목: xxx
+   - 변경 파일: [파일 목록]
+
+   [Diff 내용]
+   (gh pr diff 결과)
+
+   GitHub 리뷰 스타일로 file:line 참조와 함께 발견사항을 반환하세요.
+   심각도를 Must Fix / Should Fix / Consider / Suggestion으로 구분하세요."
+
+   Task(fundamentals-code): "다음 PR의 변경사항을 Toss Frontend Fundamentals 원칙으로 분석해주세요.
+   [같은 PR 정보와 diff]
+   가독성, 예측 가능성, 응집도, 결합도 관점에서 리뷰하세요."
+
+   Task(refactor-analyzer): "다음 PR의 변경사항에서 리팩토링 기회를 찾아주세요.
+   [같은 PR 정보와 diff]
+   코드 중복, 복잡성, 추상화 기회를 확인하세요."
+
+   Task(junior-checker): "다음 PR의 변경사항이 주니어 개발자에게 이해하기 쉬운지 분석해주세요.
+   [같은 PR 정보와 diff]
+   네이밍, 복잡도, 가독성을 확인하세요."
+
+   Task(react-performance-optimizer): "다음 PR의 변경사항에서 React 성능 이슈를 찾아주세요.
+   [같은 PR 정보와 diff]
+   리렌더, 메모이제이션, 훅 최적화 관점에서 리뷰하세요."
+   ```
+
+5. **결과 종합**
+   - 모든 에이전트 결과 수집
+   - 중복 발견사항 제거
+   - 심각도별 정렬
+   - GitHub 리뷰 스타일로 포맷팅
+
+## Output Format
+
+```markdown
+# PR 코드 리뷰
+
+## PR 정보
+- **제목:** [PR 제목]
+- **작성자:** @[author]
+- **브랜치:** [head] → [base]
+- **변경:** +[additions] -[deletions] in [files] files
+
+---
+
+## 리뷰 요약
+
+| 심각도 | 개수 |
+|--------|------|
+| 🔴 Must Fix | X |
+| 🟡 Should Fix | Y |
+| 🟢 Consider | Z |
+| 💡 Suggestion | W |
+
+---
+
+## 🔴 Must Fix (반드시 수정)
+
+### `src/components/UserProfile.tsx:42`
+**[카테고리]** 타입 안전성
+**[발견 에이전트]** code-reviewer
+
+현재 코드:
+```tsx
+const user: any = response.data
+```
+
+문제:
+- `any` 타입 사용으로 타입 안전성 상실
+
+제안:
+```tsx
+const user: User = response.data
+```
+
+---
+
+## 🟡 Should Fix (권장 수정)
+
+### `src/hooks/useAuth.ts:15-20`
+**[카테고리]** 에러 처리
+**[발견 에이전트]** code-reviewer
+
+[같은 형식...]
+
+---
+
+## 🟢 Consider (고려 사항)
+
+[같은 형식...]
+
+---
+
+## 💡 Suggestions (제안)
+
+[같은 형식...]
+
+---
+
+## ✅ 잘한 점
+
+- `src/hooks/useUser.ts` - 훅 분리가 잘 되어 있음 (fundamentals-code)
+- `src/types/index.ts` - 타입 정의가 명확함 (code-reviewer)
+
+---
+
+## 에이전트별 요약
+
+### Code Reviewer
+[주요 발견사항]
+
+### Fundamentals Code
+[주요 발견사항]
+
+### Refactor Analyzer
+[주요 발견사항]
+
+### Junior Checker
+[주요 발견사항]
+
+### React Performance Optimizer
+[주요 발견사항]
+
+---
+
+## 다음 단계
+
+1. **즉시:** 🔴 Must Fix 항목 수정
+2. **이번 PR:** 🟡 Should Fix 항목 검토
+3. **추후:** 🟢 Consider 항목 백로그에 추가
+```
+
+## 사용 예시
+
+```bash
+# PR 번호로
+/review-pr 123
+
+# PR URL로
+/review-pr https://github.com/owner/repo/pull/123
+```
+
+## 중요 사항
+
+- **병렬 실행** - 에이전트를 동시에 실행하여 효율적으로 리뷰
+- **변경된 파일만 분석** - 전체 코드베이스가 아닌 PR diff만 리뷰
+- **라인 번호 정확히** - diff에서 실제 라인 번호 추출하여 GitHub에서 바로 찾을 수 있도록
+- **구체적 제안** - 문제만 지적하지 말고 해결책 제시
+- **한국어 출력** - 최종 리포트는 한국어로
+- **심각도 구분** - Must/Should/Consider/Suggestion으로 우선순위화
+- **발견 에이전트 표시** - 어떤 에이전트가 발견했는지 명시