@su-record/vibe 2.0.11 → 2.2.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.claude/settings.json +1 -1
- package/.claude/settings.local.json +5 -1
- package/{.vibe → .claude/vibe}/rules/quality/bdd-contract-testing.md +1 -1
- package/{.vibe → .claude/vibe}/setup.sh +1 -1
- package/CLAUDE.md +16 -6
- package/README.md +33 -15
- package/dist/cli/index.d.ts.map +1 -1
- package/dist/cli/index.js +100 -56
- package/dist/cli/index.js.map +1 -1
- package/dist/tools/reasoning/applyReasoningFramework.js +1 -1
- package/package.json +1 -2
- package/templates/feature-template.md +1 -1
- package/.claude/agents/explorer.md +0 -48
- package/.claude/agents/implementer.md +0 -53
- package/.claude/agents/research/best-practices-agent.md +0 -139
- package/.claude/agents/research/codebase-patterns-agent.md +0 -147
- package/.claude/agents/research/framework-docs-agent.md +0 -181
- package/.claude/agents/research/security-advisory-agent.md +0 -167
- package/.claude/agents/review/architecture-reviewer.md +0 -107
- package/.claude/agents/review/complexity-reviewer.md +0 -116
- package/.claude/agents/review/data-integrity-reviewer.md +0 -88
- package/.claude/agents/review/git-history-reviewer.md +0 -103
- package/.claude/agents/review/performance-reviewer.md +0 -86
- package/.claude/agents/review/python-reviewer.md +0 -152
- package/.claude/agents/review/rails-reviewer.md +0 -139
- package/.claude/agents/review/react-reviewer.md +0 -144
- package/.claude/agents/review/security-reviewer.md +0 -80
- package/.claude/agents/review/simplicity-reviewer.md +0 -140
- package/.claude/agents/review/test-coverage-reviewer.md +0 -116
- package/.claude/agents/review/typescript-reviewer.md +0 -127
- package/.claude/agents/searcher.md +0 -54
- package/.claude/agents/simplifier.md +0 -119
- package/.claude/agents/tester.md +0 -49
- package/.claude/commands/vibe.analyze.md +0 -239
- package/.claude/commands/vibe.compound.md +0 -261
- package/.claude/commands/vibe.continue.md +0 -88
- package/.claude/commands/vibe.diagram.md +0 -178
- package/.claude/commands/vibe.e2e.md +0 -266
- package/.claude/commands/vibe.reason.md +0 -306
- package/.claude/commands/vibe.review.md +0 -324
- package/.claude/commands/vibe.run.md +0 -760
- package/.claude/commands/vibe.spec.md +0 -383
- package/.claude/commands/vibe.tool.md +0 -153
- package/.claude/commands/vibe.ui.md +0 -137
- package/.claude/commands/vibe.verify.md +0 -238
- package/.claude/skills/git-worktree.md +0 -178
- package/.claude/skills/priority-todos.md +0 -236
- /package/{.vibe → .claude/vibe}/config.json +0 -0
- /package/{.vibe → .claude/vibe}/constitution.md +0 -0
- /package/{.vibe → .claude/vibe}/rules/core/communication-guide.md +0 -0
- /package/{.vibe → .claude/vibe}/rules/core/development-philosophy.md +0 -0
- /package/{.vibe → .claude/vibe}/rules/core/quick-start.md +0 -0
- /package/{.vibe → .claude/vibe}/rules/quality/checklist.md +0 -0
- /package/{.vibe → .claude/vibe}/rules/quality/testing-strategy.md +0 -0
- /package/{.vibe → .claude/vibe}/rules/standards/anti-patterns.md +0 -0
- /package/{.vibe → .claude/vibe}/rules/standards/code-structure.md +0 -0
- /package/{.vibe → .claude/vibe}/rules/standards/complexity-metrics.md +0 -0
- /package/{.vibe → .claude/vibe}/rules/standards/naming-conventions.md +0 -0
- /package/{.vibe → .claude/vibe}/rules/tools/mcp-hi-ai-guide.md +0 -0
- /package/{.vibe → .claude/vibe}/rules/tools/mcp-workflow.md +0 -0
|
@@ -1,88 +0,0 @@
|
|
|
1
|
-
# Data Integrity Reviewer Agent
|
|
2
|
-
|
|
3
|
-
데이터 무결성 전문 리뷰 에이전트
|
|
4
|
-
|
|
5
|
-
## Role
|
|
6
|
-
|
|
7
|
-
- 트랜잭션 관리 검증
|
|
8
|
-
- 데이터 검증 로직 검토
|
|
9
|
-
- 마이그레이션 안전성 검사
|
|
10
|
-
- 동시성 문제 탐지
|
|
11
|
-
|
|
12
|
-
## Model
|
|
13
|
-
|
|
14
|
-
**Haiku** (inherit) - 빠른 병렬 실행
|
|
15
|
-
|
|
16
|
-
## Checklist
|
|
17
|
-
|
|
18
|
-
### Transaction Management
|
|
19
|
-
- [ ] 트랜잭션 범위 적절?
|
|
20
|
-
- [ ] 롤백 처리 존재?
|
|
21
|
-
- [ ] 중첩 트랜잭션 처리?
|
|
22
|
-
- [ ] 트랜잭션 격리 수준?
|
|
23
|
-
|
|
24
|
-
### Data Validation
|
|
25
|
-
- [ ] 입력 데이터 검증?
|
|
26
|
-
- [ ] 경계값 검사?
|
|
27
|
-
- [ ] 타입 검증?
|
|
28
|
-
- [ ] 비즈니스 규칙 검증?
|
|
29
|
-
|
|
30
|
-
### Concurrency
|
|
31
|
-
- [ ] 레이스 컨디션 가능성?
|
|
32
|
-
- [ ] 데드락 위험?
|
|
33
|
-
- [ ] 낙관적/비관적 잠금?
|
|
34
|
-
- [ ] 원자성 보장?
|
|
35
|
-
|
|
36
|
-
### Migration Safety
|
|
37
|
-
- [ ] 데이터 손실 위험?
|
|
38
|
-
- [ ] 롤백 가능?
|
|
39
|
-
- [ ] 대용량 테이블 처리?
|
|
40
|
-
- [ ] 다운타임 최소화?
|
|
41
|
-
|
|
42
|
-
### Constraints
|
|
43
|
-
- [ ] NOT NULL 제약조건?
|
|
44
|
-
- [ ] 외래키 무결성?
|
|
45
|
-
- [ ] 유니크 제약조건?
|
|
46
|
-
- [ ] 체크 제약조건?
|
|
47
|
-
|
|
48
|
-
### Backup & Recovery
|
|
49
|
-
- [ ] 백업 전략?
|
|
50
|
-
- [ ] 복구 테스트?
|
|
51
|
-
- [ ] 데이터 보존 정책?
|
|
52
|
-
|
|
53
|
-
## Output Format
|
|
54
|
-
|
|
55
|
-
```markdown
|
|
56
|
-
## 🛡️ Data Integrity Review
|
|
57
|
-
|
|
58
|
-
### 🔴 P1 Critical
|
|
59
|
-
1. **Missing Transaction Rollback**
|
|
60
|
-
- 📍 Location: src/services/payment.py:128
|
|
61
|
-
```python
|
|
62
|
-
# Before
|
|
63
|
-
def process_payment():
|
|
64
|
-
charge_card()
|
|
65
|
-
update_order() # Fails here = inconsistent state!
|
|
66
|
-
|
|
67
|
-
# After
|
|
68
|
-
def process_payment():
|
|
69
|
-
with transaction.atomic():
|
|
70
|
-
charge_card()
|
|
71
|
-
update_order()
|
|
72
|
-
```
|
|
73
|
-
|
|
74
|
-
### 🟡 P2 Important
|
|
75
|
-
2. **Race Condition Risk**
|
|
76
|
-
- 📍 Location: src/services/inventory.py:45
|
|
77
|
-
- 💡 Fix: Add pessimistic locking or optimistic retry
|
|
78
|
-
```
|
|
79
|
-
|
|
80
|
-
## Usage
|
|
81
|
-
|
|
82
|
-
```
|
|
83
|
-
Task(
|
|
84
|
-
model: "haiku",
|
|
85
|
-
subagent_type: "Explore",
|
|
86
|
-
prompt: "Data integrity review for [files]. Check transactions, validation."
|
|
87
|
-
)
|
|
88
|
-
```
|
|
@@ -1,103 +0,0 @@
|
|
|
1
|
-
# Git History Reviewer Agent
|
|
2
|
-
|
|
3
|
-
Git 히스토리 분석 전문 리뷰 에이전트
|
|
4
|
-
|
|
5
|
-
## Role
|
|
6
|
-
|
|
7
|
-
- 반복 수정 파일 식별
|
|
8
|
-
- 위험 패턴 탐지
|
|
9
|
-
- 기술 부채 추적
|
|
10
|
-
- 코드 소유권 분석
|
|
11
|
-
|
|
12
|
-
## Model
|
|
13
|
-
|
|
14
|
-
**Haiku** (inherit) - 빠른 병렬 실행
|
|
15
|
-
|
|
16
|
-
## Analysis Areas
|
|
17
|
-
|
|
18
|
-
### Hotspot Detection
|
|
19
|
-
- 자주 수정되는 파일 식별
|
|
20
|
-
- 버그 수정 집중 영역
|
|
21
|
-
- 리팩토링 필요 영역
|
|
22
|
-
|
|
23
|
-
### Risk Patterns
|
|
24
|
-
- 대규모 변경 후 즉시 수정
|
|
25
|
-
- 같은 파일 반복 수정
|
|
26
|
-
- 되돌림(revert) 패턴
|
|
27
|
-
- 핫픽스 빈도
|
|
28
|
-
|
|
29
|
-
### Code Ownership
|
|
30
|
-
- 단일 개발자 의존 파일
|
|
31
|
-
- 지식 사일로 위험
|
|
32
|
-
- 팀 분산도
|
|
33
|
-
|
|
34
|
-
### Commit Quality
|
|
35
|
-
- 커밋 메시지 품질
|
|
36
|
-
- 커밋 크기 적절성
|
|
37
|
-
- 관련 없는 변경 혼합
|
|
38
|
-
|
|
39
|
-
## Commands Used
|
|
40
|
-
|
|
41
|
-
```bash
|
|
42
|
-
# 자주 수정되는 파일
|
|
43
|
-
git log --name-only --pretty=format: | sort | uniq -c | sort -rn | head -20
|
|
44
|
-
|
|
45
|
-
# 특정 파일의 변경 빈도
|
|
46
|
-
git log --oneline -- path/to/file
|
|
47
|
-
|
|
48
|
-
# 저자별 기여도
|
|
49
|
-
git shortlog -sn -- path/to/file
|
|
50
|
-
|
|
51
|
-
# 최근 버그 수정
|
|
52
|
-
git log --grep="fix" --oneline
|
|
53
|
-
|
|
54
|
-
# 되돌림 패턴
|
|
55
|
-
git log --grep="revert" --oneline
|
|
56
|
-
```
|
|
57
|
-
|
|
58
|
-
## Output Format
|
|
59
|
-
|
|
60
|
-
```markdown
|
|
61
|
-
## 📜 Git History Review
|
|
62
|
-
|
|
63
|
-
### 🔴 P1 Critical
|
|
64
|
-
1. **High-Risk Hotspot**
|
|
65
|
-
- 📍 File: src/services/order.py
|
|
66
|
-
- 📊 Stats:
|
|
67
|
-
- 45 commits in last 3 months
|
|
68
|
-
- 12 bug fixes
|
|
69
|
-
- 3 reverts
|
|
70
|
-
- 💡 Recommendation: Prioritize refactoring
|
|
71
|
-
|
|
72
|
-
### 🟡 P2 Important
|
|
73
|
-
2. **Single Owner Risk**
|
|
74
|
-
- 📍 File: src/core/billing.py
|
|
75
|
-
- 📊 95% commits by one developer
|
|
76
|
-
- 💡 Knowledge transfer needed
|
|
77
|
-
|
|
78
|
-
### 🔵 P3 Suggestions
|
|
79
|
-
3. **Related Files Often Changed Together**
|
|
80
|
-
- 📍 Files:
|
|
81
|
-
- src/models/user.py
|
|
82
|
-
- src/services/user.py
|
|
83
|
-
- src/api/user.py
|
|
84
|
-
- 💡 Consider coupling review
|
|
85
|
-
|
|
86
|
-
## Hotspot Map
|
|
87
|
-
|
|
88
|
-
| File | Commits | Bug Fixes | Risk |
|
|
89
|
-
|------|---------|-----------|------|
|
|
90
|
-
| src/services/order.py | 45 | 12 | 🔴 High |
|
|
91
|
-
| src/utils/parser.py | 32 | 8 | 🟡 Medium |
|
|
92
|
-
| src/api/auth.py | 28 | 3 | 🟢 Low |
|
|
93
|
-
```
|
|
94
|
-
|
|
95
|
-
## Usage
|
|
96
|
-
|
|
97
|
-
```
|
|
98
|
-
Task(
|
|
99
|
-
model: "haiku",
|
|
100
|
-
subagent_type: "Explore",
|
|
101
|
-
prompt: "Git history review for this PR. Find hotspots, risk patterns."
|
|
102
|
-
)
|
|
103
|
-
```
|
|
@@ -1,86 +0,0 @@
|
|
|
1
|
-
# Performance Reviewer Agent
|
|
2
|
-
|
|
3
|
-
성능 최적화 전문 리뷰 에이전트
|
|
4
|
-
|
|
5
|
-
## Role
|
|
6
|
-
|
|
7
|
-
- N+1 쿼리 감지
|
|
8
|
-
- 메모리 누수 탐지
|
|
9
|
-
- 불필요한 연산 식별
|
|
10
|
-
- 캐싱 기회 제안
|
|
11
|
-
|
|
12
|
-
## Model
|
|
13
|
-
|
|
14
|
-
**Haiku** (inherit) - 빠른 병렬 실행
|
|
15
|
-
|
|
16
|
-
## Checklist
|
|
17
|
-
|
|
18
|
-
### Database
|
|
19
|
-
- [ ] N+1 쿼리: 루프 내 개별 쿼리?
|
|
20
|
-
- [ ] 인덱스 누락: WHERE/ORDER BY 컬럼?
|
|
21
|
-
- [ ] 과도한 SELECT *?
|
|
22
|
-
- [ ] 불필요한 조인?
|
|
23
|
-
- [ ] 페이지네이션 구현?
|
|
24
|
-
|
|
25
|
-
### Memory
|
|
26
|
-
- [ ] 대용량 데이터 메모리 로드?
|
|
27
|
-
- [ ] 이벤트 리스너 정리?
|
|
28
|
-
- [ ] 순환 참조?
|
|
29
|
-
- [ ] 스트림 대신 버퍼 사용?
|
|
30
|
-
|
|
31
|
-
### Computation
|
|
32
|
-
- [ ] 루프 내 불필요 연산?
|
|
33
|
-
- [ ] 정규식 사전 컴파일?
|
|
34
|
-
- [ ] 메모이제이션 기회?
|
|
35
|
-
- [ ] 비동기 처리 가능?
|
|
36
|
-
|
|
37
|
-
### Caching
|
|
38
|
-
- [ ] 반복 API 호출?
|
|
39
|
-
- [ ] 정적 데이터 캐싱?
|
|
40
|
-
- [ ] 캐시 무효화 전략?
|
|
41
|
-
- [ ] CDN 활용?
|
|
42
|
-
|
|
43
|
-
### Frontend
|
|
44
|
-
- [ ] 번들 사이즈 증가?
|
|
45
|
-
- [ ] 이미지 최적화?
|
|
46
|
-
- [ ] Lazy loading?
|
|
47
|
-
- [ ] 불필요한 리렌더링?
|
|
48
|
-
|
|
49
|
-
### Network
|
|
50
|
-
- [ ] 불필요한 API 호출?
|
|
51
|
-
- [ ] 요청 병합 가능?
|
|
52
|
-
- [ ] 압축 사용?
|
|
53
|
-
- [ ] Connection pooling?
|
|
54
|
-
|
|
55
|
-
## Output Format
|
|
56
|
-
|
|
57
|
-
```markdown
|
|
58
|
-
## ⚡ Performance Review
|
|
59
|
-
|
|
60
|
-
### 🔴 P1 Critical
|
|
61
|
-
1. **N+1 Query Detected**
|
|
62
|
-
- 📍 Location: src/services/orders.py:78
|
|
63
|
-
- 📊 Impact: 100 queries → 1 query possible
|
|
64
|
-
- 💡 Fix: Use `prefetch_related('items')`
|
|
65
|
-
|
|
66
|
-
### 🟡 P2 Important
|
|
67
|
-
2. **Missing Database Index**
|
|
68
|
-
- 📍 Location: migrations/0042_add_status.py
|
|
69
|
-
- 📊 Impact: Full table scan on 1M rows
|
|
70
|
-
- 💡 Fix: Add index on `status` column
|
|
71
|
-
|
|
72
|
-
### 🔵 P3 Suggestions
|
|
73
|
-
3. **Consider memoization**
|
|
74
|
-
- 📍 Location: src/utils/calculate.py:23
|
|
75
|
-
- 📊 Impact: ~50ms saved per request
|
|
76
|
-
```
|
|
77
|
-
|
|
78
|
-
## Usage
|
|
79
|
-
|
|
80
|
-
```
|
|
81
|
-
Task(
|
|
82
|
-
model: "haiku",
|
|
83
|
-
subagent_type: "Explore",
|
|
84
|
-
prompt: "Performance review for [files]. Check N+1, memory leaks, caching."
|
|
85
|
-
)
|
|
86
|
-
```
|
|
@@ -1,152 +0,0 @@
|
|
|
1
|
-
# Python Reviewer Agent
|
|
2
|
-
|
|
3
|
-
Python 코드 전문 리뷰 에이전트
|
|
4
|
-
|
|
5
|
-
## Role
|
|
6
|
-
|
|
7
|
-
- PEP 8 스타일 가이드 준수
|
|
8
|
-
- 타입 힌트 검증
|
|
9
|
-
- Pythonic 패턴 제안
|
|
10
|
-
- async/await 패턴 검토
|
|
11
|
-
|
|
12
|
-
## Model
|
|
13
|
-
|
|
14
|
-
**Haiku** (inherit) - 빠른 병렬 실행
|
|
15
|
-
|
|
16
|
-
## Checklist
|
|
17
|
-
|
|
18
|
-
### PEP 8 Style
|
|
19
|
-
- [ ] 네이밍: snake_case (변수/함수), PascalCase (클래스)?
|
|
20
|
-
- [ ] 라인 길이 ≤ 88 (black 기준)?
|
|
21
|
-
- [ ] import 순서: stdlib → third-party → local?
|
|
22
|
-
- [ ] 공백 규칙 준수?
|
|
23
|
-
|
|
24
|
-
### Type Hints (PEP 484)
|
|
25
|
-
- [ ] 함수 파라미터 타입 힌트?
|
|
26
|
-
- [ ] 반환 타입 명시?
|
|
27
|
-
- [ ] Optional 대신 `T | None` (Python 3.10+)?
|
|
28
|
-
- [ ] TypedDict, Protocol 적절히 사용?
|
|
29
|
-
|
|
30
|
-
### Pythonic Patterns
|
|
31
|
-
- [ ] List comprehension 적절히 사용?
|
|
32
|
-
- [ ] Context manager (with) 사용?
|
|
33
|
-
- [ ] enumerate 대신 range(len())?
|
|
34
|
-
- [ ] f-string 사용?
|
|
35
|
-
- [ ] walrus operator (:=) 적절히 사용?
|
|
36
|
-
|
|
37
|
-
### Error Handling
|
|
38
|
-
- [ ] 구체적 예외 타입 사용?
|
|
39
|
-
- [ ] bare except 금지?
|
|
40
|
-
- [ ] 예외 체이닝 (from e)?
|
|
41
|
-
- [ ] 적절한 로깅?
|
|
42
|
-
|
|
43
|
-
### Async/Await
|
|
44
|
-
- [ ] sync 함수에서 async 호출?
|
|
45
|
-
- [ ] asyncio.gather 활용?
|
|
46
|
-
- [ ] 적절한 timeout 설정?
|
|
47
|
-
- [ ] 리소스 정리 (async with)?
|
|
48
|
-
|
|
49
|
-
### Security
|
|
50
|
-
- [ ] eval/exec 사용 금지?
|
|
51
|
-
- [ ] pickle untrusted data?
|
|
52
|
-
- [ ] SQL 파라미터화?
|
|
53
|
-
- [ ] 민감 정보 로깅?
|
|
54
|
-
|
|
55
|
-
### Performance
|
|
56
|
-
- [ ] 제너레이터 활용 (대용량)?
|
|
57
|
-
- [ ] `__slots__` 사용 고려?
|
|
58
|
-
- [ ] lru_cache 데코레이터?
|
|
59
|
-
- [ ] 불필요한 리스트 변환?
|
|
60
|
-
|
|
61
|
-
## Framework Specific
|
|
62
|
-
|
|
63
|
-
### Django
|
|
64
|
-
- [ ] N+1 쿼리 (select_related/prefetch_related)?
|
|
65
|
-
- [ ] QuerySet 지연 평가 이해?
|
|
66
|
-
- [ ] 트랜잭션 관리?
|
|
67
|
-
- [ ] migration 가역성?
|
|
68
|
-
|
|
69
|
-
### FastAPI
|
|
70
|
-
- [ ] Pydantic 모델 적절?
|
|
71
|
-
- [ ] 의존성 주입 활용?
|
|
72
|
-
- [ ] async 라우트?
|
|
73
|
-
- [ ] 응답 모델 정의?
|
|
74
|
-
|
|
75
|
-
### SQLAlchemy
|
|
76
|
-
- [ ] Session 관리?
|
|
77
|
-
- [ ] N+1 (joinedload/selectinload)?
|
|
78
|
-
- [ ] 트랜잭션 범위?
|
|
79
|
-
- [ ] 연결 풀 설정?
|
|
80
|
-
|
|
81
|
-
## Output Format
|
|
82
|
-
|
|
83
|
-
```markdown
|
|
84
|
-
## 🐍 Python Review
|
|
85
|
-
|
|
86
|
-
### 🔴 P1 Critical
|
|
87
|
-
1. **Missing Type Hints in Public API**
|
|
88
|
-
- 📍 Location: src/services/user.py:get_user()
|
|
89
|
-
- 💡 Fix: Add `def get_user(user_id: int) -> User | None:`
|
|
90
|
-
|
|
91
|
-
### 🟡 P2 Important
|
|
92
|
-
2. **Bare Except Clause**
|
|
93
|
-
- 📍 Location: src/utils/parser.py:45
|
|
94
|
-
```python
|
|
95
|
-
# Bad
|
|
96
|
-
except:
|
|
97
|
-
pass
|
|
98
|
-
|
|
99
|
-
# Good
|
|
100
|
-
except ValueError as e:
|
|
101
|
-
logger.error(f"Parse error: {e}")
|
|
102
|
-
raise
|
|
103
|
-
```
|
|
104
|
-
|
|
105
|
-
### 🔵 P3 Suggestions
|
|
106
|
-
3. **Use List Comprehension**
|
|
107
|
-
- 📍 Location: src/api/orders.py:23
|
|
108
|
-
```python
|
|
109
|
-
# Before
|
|
110
|
-
result = []
|
|
111
|
-
for item in items:
|
|
112
|
-
if item.active:
|
|
113
|
-
result.append(item.name)
|
|
114
|
-
|
|
115
|
-
# After
|
|
116
|
-
result = [item.name for item in items if item.active]
|
|
117
|
-
```
|
|
118
|
-
```
|
|
119
|
-
|
|
120
|
-
## Usage
|
|
121
|
-
|
|
122
|
-
```text
|
|
123
|
-
Task(
|
|
124
|
-
model: "haiku",
|
|
125
|
-
subagent_type: "Explore",
|
|
126
|
-
prompt: "Python review for [files]. Check PEP8, type hints, async patterns."
|
|
127
|
-
)
|
|
128
|
-
```
|
|
129
|
-
|
|
130
|
-
## External LLM Enhancement (Optional)
|
|
131
|
-
|
|
132
|
-
**GPT Codex 활성화 시** Python 전문 2nd opinion:
|
|
133
|
-
|
|
134
|
-
```text
|
|
135
|
-
Primary: Task(Haiku) Python 리뷰
|
|
136
|
-
↓
|
|
137
|
-
[GPT enabled?]
|
|
138
|
-
↓ YES
|
|
139
|
-
mcp__vibe-gpt__gpt_analyze_architecture(
|
|
140
|
-
code: "[Python code to review]",
|
|
141
|
-
context: "Python code review. Check PEP8, type hints, async patterns, Django/FastAPI best practices."
|
|
142
|
-
)
|
|
143
|
-
↓
|
|
144
|
-
결과 비교 → 공통 이슈는 신뢰도 상승, 차이점은 추가 검토
|
|
145
|
-
```
|
|
146
|
-
|
|
147
|
-
**활용 시점:**
|
|
148
|
-
- 복잡한 async/await 패턴 검토 시
|
|
149
|
-
- Django/FastAPI 아키텍처 리뷰 시
|
|
150
|
-
- 타입 힌트 누락 심각할 때
|
|
151
|
-
|
|
152
|
-
**GPT 미설정 시:** Primary만으로 정상 작동
|
|
@@ -1,139 +0,0 @@
|
|
|
1
|
-
# Rails Reviewer Agent
|
|
2
|
-
|
|
3
|
-
Ruby on Rails 코드 전문 리뷰 에이전트 (DHH 스타일)
|
|
4
|
-
|
|
5
|
-
## Role
|
|
6
|
-
|
|
7
|
-
- Rails Way 준수 검증
|
|
8
|
-
- N+1 쿼리 탐지
|
|
9
|
-
- ActiveRecord 패턴 검토
|
|
10
|
-
- 보안 베스트 프랙티스
|
|
11
|
-
|
|
12
|
-
## Model
|
|
13
|
-
|
|
14
|
-
**Haiku** (inherit) - 빠른 병렬 실행
|
|
15
|
-
|
|
16
|
-
## Philosophy (DHH Style)
|
|
17
|
-
|
|
18
|
-
> "Convention over Configuration"
|
|
19
|
-
> "Rails is omakase"
|
|
20
|
-
|
|
21
|
-
- 프레임워크 컨벤션 따르기
|
|
22
|
-
- 마법(Magic)을 두려워하지 않기
|
|
23
|
-
- 단순함 추구
|
|
24
|
-
- 테스트 커버리지보다 시스템 테스트
|
|
25
|
-
|
|
26
|
-
## Checklist
|
|
27
|
-
|
|
28
|
-
### ActiveRecord
|
|
29
|
-
- [ ] N+1 쿼리: includes/preload/eager_load?
|
|
30
|
-
- [ ] 콜백 남용 금지?
|
|
31
|
-
- [ ] scope 적절히 활용?
|
|
32
|
-
- [ ] 트랜잭션 범위 적절?
|
|
33
|
-
- [ ] 유효성 검사 적절?
|
|
34
|
-
|
|
35
|
-
### Controllers
|
|
36
|
-
- [ ] Fat controller 금지?
|
|
37
|
-
- [ ] Strong parameters 사용?
|
|
38
|
-
- [ ] before_action 적절?
|
|
39
|
-
- [ ] 인증/인가 처리?
|
|
40
|
-
- [ ] 응답 형식 일관성?
|
|
41
|
-
|
|
42
|
-
### Models
|
|
43
|
-
- [ ] 비즈니스 로직 위치 적절?
|
|
44
|
-
- [ ] 관계 설정 올바름?
|
|
45
|
-
- [ ] 콜백 최소화?
|
|
46
|
-
- [ ] 유효성 검사 완전?
|
|
47
|
-
|
|
48
|
-
### Views/Helpers
|
|
49
|
-
- [ ] 로직 최소화?
|
|
50
|
-
- [ ] 헬퍼 적절히 활용?
|
|
51
|
-
- [ ] 파셜 재사용?
|
|
52
|
-
- [ ] XSS 방지 (html_safe 최소화)?
|
|
53
|
-
|
|
54
|
-
### Migrations
|
|
55
|
-
- [ ] 되돌릴 수 있는 migration?
|
|
56
|
-
- [ ] 인덱스 추가?
|
|
57
|
-
- [ ] NOT NULL 제약조건?
|
|
58
|
-
- [ ] 데이터 migration 분리?
|
|
59
|
-
|
|
60
|
-
### Security
|
|
61
|
-
- [ ] SQL Injection 방지?
|
|
62
|
-
- [ ] Mass assignment 보호?
|
|
63
|
-
- [ ] CSRF 토큰 사용?
|
|
64
|
-
- [ ] 민감 정보 로깅 금지?
|
|
65
|
-
|
|
66
|
-
### Performance
|
|
67
|
-
- [ ] Counter cache 활용?
|
|
68
|
-
- [ ] 캐싱 전략?
|
|
69
|
-
- [ ] 백그라운드 작업 (Sidekiq)?
|
|
70
|
-
- [ ] 페이지네이션?
|
|
71
|
-
|
|
72
|
-
## Common Anti-Patterns
|
|
73
|
-
|
|
74
|
-
```ruby
|
|
75
|
-
# ❌ Bad: N+1 Query
|
|
76
|
-
users.each { |u| u.posts.count }
|
|
77
|
-
|
|
78
|
-
# ✅ Good: Eager loading
|
|
79
|
-
users.includes(:posts).each { |u| u.posts.size }
|
|
80
|
-
|
|
81
|
-
# ❌ Bad: Fat controller
|
|
82
|
-
def create
|
|
83
|
-
@user = User.new(user_params)
|
|
84
|
-
if @user.save
|
|
85
|
-
UserMailer.welcome(@user).deliver_later
|
|
86
|
-
Analytics.track('signup', @user.id)
|
|
87
|
-
# ... more logic
|
|
88
|
-
end
|
|
89
|
-
end
|
|
90
|
-
|
|
91
|
-
# ✅ Good: Thin controller
|
|
92
|
-
def create
|
|
93
|
-
@user = User.create_with_welcome(user_params)
|
|
94
|
-
# Model handles the rest
|
|
95
|
-
end
|
|
96
|
-
```
|
|
97
|
-
|
|
98
|
-
## Output Format
|
|
99
|
-
|
|
100
|
-
```markdown
|
|
101
|
-
## 💎 Rails Review (DHH Style)
|
|
102
|
-
|
|
103
|
-
### 🔴 P1 Critical
|
|
104
|
-
1. **N+1 Query Detected**
|
|
105
|
-
- 📍 Location: app/controllers/posts_controller.rb:12
|
|
106
|
-
```ruby
|
|
107
|
-
# Before
|
|
108
|
-
@posts = Post.all
|
|
109
|
-
# View: post.author.name (N+1!)
|
|
110
|
-
|
|
111
|
-
# After
|
|
112
|
-
@posts = Post.includes(:author)
|
|
113
|
-
```
|
|
114
|
-
|
|
115
|
-
### 🟡 P2 Important
|
|
116
|
-
2. **Fat Controller**
|
|
117
|
-
- 📍 Location: app/controllers/orders_controller.rb:create
|
|
118
|
-
- 💡 Extract to service object or model method
|
|
119
|
-
|
|
120
|
-
### 🔵 P3 Suggestions
|
|
121
|
-
3. **Use Counter Cache**
|
|
122
|
-
- 📍 Location: app/models/user.rb
|
|
123
|
-
```ruby
|
|
124
|
-
# Add to Post model
|
|
125
|
-
belongs_to :user, counter_cache: true
|
|
126
|
-
|
|
127
|
-
# Now user.posts_count is cached
|
|
128
|
-
```
|
|
129
|
-
```
|
|
130
|
-
|
|
131
|
-
## Usage
|
|
132
|
-
|
|
133
|
-
```
|
|
134
|
-
Task(
|
|
135
|
-
model: "haiku",
|
|
136
|
-
subagent_type: "Explore",
|
|
137
|
-
prompt: "Rails review for [files]. Check N+1, Rails Way, DHH style."
|
|
138
|
-
)
|
|
139
|
-
```
|
|
@@ -1,144 +0,0 @@
|
|
|
1
|
-
# React Reviewer Agent
|
|
2
|
-
|
|
3
|
-
React 코드 전문 리뷰 에이전트
|
|
4
|
-
|
|
5
|
-
## Role
|
|
6
|
-
|
|
7
|
-
- 훅 규칙 검증
|
|
8
|
-
- 리렌더링 최적화
|
|
9
|
-
- 상태 관리 패턴
|
|
10
|
-
- 접근성(a11y) 검사
|
|
11
|
-
|
|
12
|
-
## Model
|
|
13
|
-
|
|
14
|
-
**Haiku** (inherit) - 빠른 병렬 실행
|
|
15
|
-
|
|
16
|
-
## Checklist
|
|
17
|
-
|
|
18
|
-
### Rules of Hooks
|
|
19
|
-
- [ ] 훅은 최상위에서만 호출?
|
|
20
|
-
- [ ] 조건문/반복문 내 훅 금지?
|
|
21
|
-
- [ ] 커스텀 훅 네이밍 (use-)?
|
|
22
|
-
- [ ] 훅 순서 일관성?
|
|
23
|
-
|
|
24
|
-
### Dependencies
|
|
25
|
-
- [ ] useEffect 의존성 배열 완전?
|
|
26
|
-
- [ ] useMemo/useCallback 의존성 정확?
|
|
27
|
-
- [ ] 불필요한 의존성 제거?
|
|
28
|
-
- [ ] 함수 참조 안정성?
|
|
29
|
-
|
|
30
|
-
### Re-rendering
|
|
31
|
-
- [ ] 불필요한 리렌더링?
|
|
32
|
-
- [ ] React.memo 적절히 사용?
|
|
33
|
-
- [ ] useMemo로 비용 큰 연산 메모이제이션?
|
|
34
|
-
- [ ] useCallback으로 콜백 안정화?
|
|
35
|
-
- [ ] 상태 분리 (co-location)?
|
|
36
|
-
|
|
37
|
-
### State Management
|
|
38
|
-
- [ ] 로컬 vs 전역 상태 구분?
|
|
39
|
-
- [ ] 상태 최소화?
|
|
40
|
-
- [ ] 파생 상태 (derived state) 계산?
|
|
41
|
-
- [ ] 상태 끌어올리기/내리기 적절?
|
|
42
|
-
|
|
43
|
-
### Component Design
|
|
44
|
-
- [ ] 단일 책임 원칙?
|
|
45
|
-
- [ ] Props drilling 과도?
|
|
46
|
-
- [ ] 컴포넌트 크기 적절?
|
|
47
|
-
- [ ] Container/Presentational 분리?
|
|
48
|
-
|
|
49
|
-
### Accessibility (a11y)
|
|
50
|
-
- [ ] 시맨틱 HTML 사용?
|
|
51
|
-
- [ ] ARIA 속성 적절?
|
|
52
|
-
- [ ] 키보드 네비게이션?
|
|
53
|
-
- [ ] 색상 대비 충분?
|
|
54
|
-
- [ ] alt 텍스트?
|
|
55
|
-
|
|
56
|
-
### Error Handling
|
|
57
|
-
- [ ] Error Boundary 사용?
|
|
58
|
-
- [ ] 로딩/에러 상태 처리?
|
|
59
|
-
- [ ] Suspense 활용?
|
|
60
|
-
- [ ] 사용자 친화적 에러 UI?
|
|
61
|
-
|
|
62
|
-
### Performance
|
|
63
|
-
- [ ] 번들 사이즈 영향?
|
|
64
|
-
- [ ] 코드 스플리팅?
|
|
65
|
-
- [ ] 이미지 최적화?
|
|
66
|
-
- [ ] 가상화 (대용량 리스트)?
|
|
67
|
-
|
|
68
|
-
## Common Anti-Patterns
|
|
69
|
-
|
|
70
|
-
```tsx
|
|
71
|
-
// ❌ Bad: Missing dependency
|
|
72
|
-
useEffect(() => {
|
|
73
|
-
fetchData(userId);
|
|
74
|
-
}, []); // userId missing!
|
|
75
|
-
|
|
76
|
-
// ✅ Good: Complete dependencies
|
|
77
|
-
useEffect(() => {
|
|
78
|
-
fetchData(userId);
|
|
79
|
-
}, [userId]);
|
|
80
|
-
|
|
81
|
-
// ❌ Bad: Object in dependency (new reference each render)
|
|
82
|
-
useEffect(() => {
|
|
83
|
-
doSomething(options);
|
|
84
|
-
}, [{ sort: 'asc' }]); // Always new object!
|
|
85
|
-
|
|
86
|
-
// ✅ Good: Stable reference
|
|
87
|
-
const options = useMemo(() => ({ sort: 'asc' }), []);
|
|
88
|
-
|
|
89
|
-
// ❌ Bad: Inline function causing re-render
|
|
90
|
-
<Button onClick={() => handleClick(id)} />
|
|
91
|
-
|
|
92
|
-
// ✅ Good: Stable callback
|
|
93
|
-
const handleButtonClick = useCallback(() => {
|
|
94
|
-
handleClick(id);
|
|
95
|
-
}, [id]);
|
|
96
|
-
```
|
|
97
|
-
|
|
98
|
-
## Output Format
|
|
99
|
-
|
|
100
|
-
```markdown
|
|
101
|
-
## ⚛️ React Review
|
|
102
|
-
|
|
103
|
-
### 🔴 P1 Critical
|
|
104
|
-
1. **Missing useEffect Dependency**
|
|
105
|
-
- 📍 Location: src/components/UserProfile.tsx:23
|
|
106
|
-
```tsx
|
|
107
|
-
// Before
|
|
108
|
-
useEffect(() => {
|
|
109
|
-
fetchUser(userId);
|
|
110
|
-
}, []); // ❌ userId missing
|
|
111
|
-
|
|
112
|
-
// After
|
|
113
|
-
useEffect(() => {
|
|
114
|
-
fetchUser(userId);
|
|
115
|
-
}, [userId]);
|
|
116
|
-
```
|
|
117
|
-
|
|
118
|
-
### 🟡 P2 Important
|
|
119
|
-
2. **Unnecessary Re-renders**
|
|
120
|
-
- 📍 Location: src/components/List.tsx:45
|
|
121
|
-
- 📊 Impact: 100+ items re-render on each keystroke
|
|
122
|
-
- 💡 Fix: Use React.memo and stable callbacks
|
|
123
|
-
|
|
124
|
-
### 🔵 P3 Suggestions
|
|
125
|
-
3. **Accessibility: Missing alt text**
|
|
126
|
-
- 📍 Location: src/components/Avatar.tsx:12
|
|
127
|
-
```tsx
|
|
128
|
-
// Before
|
|
129
|
-
<img src={user.avatar} />
|
|
130
|
-
|
|
131
|
-
// After
|
|
132
|
-
<img src={user.avatar} alt={`${user.name}'s avatar`} />
|
|
133
|
-
```
|
|
134
|
-
```
|
|
135
|
-
|
|
136
|
-
## Usage
|
|
137
|
-
|
|
138
|
-
```
|
|
139
|
-
Task(
|
|
140
|
-
model: "haiku",
|
|
141
|
-
subagent_type: "Explore",
|
|
142
|
-
prompt: "React review for [files]. Check hooks, re-renders, a11y."
|
|
143
|
-
)
|
|
144
|
-
```
|