@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.
Files changed (60) hide show
  1. package/.claude/settings.json +1 -1
  2. package/.claude/settings.local.json +5 -1
  3. package/{.vibe → .claude/vibe}/rules/quality/bdd-contract-testing.md +1 -1
  4. package/{.vibe → .claude/vibe}/setup.sh +1 -1
  5. package/CLAUDE.md +16 -6
  6. package/README.md +33 -15
  7. package/dist/cli/index.d.ts.map +1 -1
  8. package/dist/cli/index.js +100 -56
  9. package/dist/cli/index.js.map +1 -1
  10. package/dist/tools/reasoning/applyReasoningFramework.js +1 -1
  11. package/package.json +1 -2
  12. package/templates/feature-template.md +1 -1
  13. package/.claude/agents/explorer.md +0 -48
  14. package/.claude/agents/implementer.md +0 -53
  15. package/.claude/agents/research/best-practices-agent.md +0 -139
  16. package/.claude/agents/research/codebase-patterns-agent.md +0 -147
  17. package/.claude/agents/research/framework-docs-agent.md +0 -181
  18. package/.claude/agents/research/security-advisory-agent.md +0 -167
  19. package/.claude/agents/review/architecture-reviewer.md +0 -107
  20. package/.claude/agents/review/complexity-reviewer.md +0 -116
  21. package/.claude/agents/review/data-integrity-reviewer.md +0 -88
  22. package/.claude/agents/review/git-history-reviewer.md +0 -103
  23. package/.claude/agents/review/performance-reviewer.md +0 -86
  24. package/.claude/agents/review/python-reviewer.md +0 -152
  25. package/.claude/agents/review/rails-reviewer.md +0 -139
  26. package/.claude/agents/review/react-reviewer.md +0 -144
  27. package/.claude/agents/review/security-reviewer.md +0 -80
  28. package/.claude/agents/review/simplicity-reviewer.md +0 -140
  29. package/.claude/agents/review/test-coverage-reviewer.md +0 -116
  30. package/.claude/agents/review/typescript-reviewer.md +0 -127
  31. package/.claude/agents/searcher.md +0 -54
  32. package/.claude/agents/simplifier.md +0 -119
  33. package/.claude/agents/tester.md +0 -49
  34. package/.claude/commands/vibe.analyze.md +0 -239
  35. package/.claude/commands/vibe.compound.md +0 -261
  36. package/.claude/commands/vibe.continue.md +0 -88
  37. package/.claude/commands/vibe.diagram.md +0 -178
  38. package/.claude/commands/vibe.e2e.md +0 -266
  39. package/.claude/commands/vibe.reason.md +0 -306
  40. package/.claude/commands/vibe.review.md +0 -324
  41. package/.claude/commands/vibe.run.md +0 -760
  42. package/.claude/commands/vibe.spec.md +0 -383
  43. package/.claude/commands/vibe.tool.md +0 -153
  44. package/.claude/commands/vibe.ui.md +0 -137
  45. package/.claude/commands/vibe.verify.md +0 -238
  46. package/.claude/skills/git-worktree.md +0 -178
  47. package/.claude/skills/priority-todos.md +0 -236
  48. /package/{.vibe → .claude/vibe}/config.json +0 -0
  49. /package/{.vibe → .claude/vibe}/constitution.md +0 -0
  50. /package/{.vibe → .claude/vibe}/rules/core/communication-guide.md +0 -0
  51. /package/{.vibe → .claude/vibe}/rules/core/development-philosophy.md +0 -0
  52. /package/{.vibe → .claude/vibe}/rules/core/quick-start.md +0 -0
  53. /package/{.vibe → .claude/vibe}/rules/quality/checklist.md +0 -0
  54. /package/{.vibe → .claude/vibe}/rules/quality/testing-strategy.md +0 -0
  55. /package/{.vibe → .claude/vibe}/rules/standards/anti-patterns.md +0 -0
  56. /package/{.vibe → .claude/vibe}/rules/standards/code-structure.md +0 -0
  57. /package/{.vibe → .claude/vibe}/rules/standards/complexity-metrics.md +0 -0
  58. /package/{.vibe → .claude/vibe}/rules/standards/naming-conventions.md +0 -0
  59. /package/{.vibe → .claude/vibe}/rules/tools/mcp-hi-ai-guide.md +0 -0
  60. /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
- ```