binary-agents 1.1.5 → 1.3.0
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/README.md +9 -3
- package/agents/code-reviewer.md +84 -97
- package/agents/fundamentals-cohesion.md +293 -0
- package/agents/fundamentals-coupling.md +372 -0
- package/agents/fundamentals-predictability.md +287 -0
- package/agents/fundamentals-readability.md +561 -0
- package/agents/junior-checker.md +93 -445
- package/agents/maintainable-code-reviewer.md +30 -133
- package/agents/react-performance-optimizer.md +89 -295
- package/agents/react-principles-reviewer.md +174 -237
- package/agents/refactor-analyzer.md +74 -253
- package/agents/subagent-builder.md +69 -75
- package/commands/code-review.md +24 -12
- package/commands/design-to-code.md +26 -11
- package/commands/review-pr.md +26 -5
- package/docs/BUILDER_GUIDE.md +5 -2
- package/package.json +1 -1
- package/agents/fundamentals-code.md +0 -993
|
@@ -41,7 +41,7 @@ model: haiku | sonnet | opus
|
|
|
41
41
|
## Your Role
|
|
42
42
|
[호출 시 서브에이전트가 하는 일]
|
|
43
43
|
|
|
44
|
-
## 평가
|
|
44
|
+
## 평가 원칙
|
|
45
45
|
[무엇을 어떻게 분석하는지]
|
|
46
46
|
|
|
47
47
|
## 프로세스
|
|
@@ -51,7 +51,7 @@ model: haiku | sonnet | opus
|
|
|
51
51
|
[예상 출력 구조]
|
|
52
52
|
|
|
53
53
|
## 가이드라인
|
|
54
|
-
[베스트 프랙티스와
|
|
54
|
+
[베스트 프랙티스와 심각도 분류]
|
|
55
55
|
```
|
|
56
56
|
|
|
57
57
|
### 핵심 구성요소
|
|
@@ -67,7 +67,7 @@ model: haiku | sonnet | opus
|
|
|
67
67
|
- 자율 운영 지침
|
|
68
68
|
- 범위와 제한
|
|
69
69
|
|
|
70
|
-
3. **평가
|
|
70
|
+
3. **평가 원칙**
|
|
71
71
|
- 찾아야 할 것 (✅)
|
|
72
72
|
- 피해야 할 것 (❌)
|
|
73
73
|
- 업계 표준
|
|
@@ -84,7 +84,7 @@ model: haiku | sonnet | opus
|
|
|
84
84
|
- file:line 참조 포맷
|
|
85
85
|
|
|
86
86
|
6. **가이드라인**
|
|
87
|
-
-
|
|
87
|
+
- 심각도 분류 기준 (Critical / Recommended Improvements / Best Practices Found)
|
|
88
88
|
- 품질 표준
|
|
89
89
|
- 베스트 프랙티스
|
|
90
90
|
|
|
@@ -183,12 +183,12 @@ model: haiku
|
|
|
183
183
|
호출 시:
|
|
184
184
|
1. [특정 패턴]에 대해 코드베이스 스캔
|
|
185
185
|
2. [기준]에 대해 분석
|
|
186
|
-
3.
|
|
186
|
+
3. Critical / Recommended Improvements / Best Practices Found 분류
|
|
187
187
|
4. 종합 리포트 반환
|
|
188
188
|
|
|
189
|
-
## 평가
|
|
189
|
+
## 평가 원칙
|
|
190
190
|
|
|
191
|
-
### 1. [기준명]
|
|
191
|
+
### 1. [기준명]
|
|
192
192
|
|
|
193
193
|
**✅ 찾아야 할 것:**
|
|
194
194
|
- [좋은 패턴 1]
|
|
@@ -198,7 +198,7 @@ model: haiku
|
|
|
198
198
|
- [나쁜 패턴 1]
|
|
199
199
|
- [나쁜 패턴 2]
|
|
200
200
|
|
|
201
|
-
##
|
|
201
|
+
## 분석 프로세스
|
|
202
202
|
1. Glob: 관련 파일 찾기
|
|
203
203
|
2. Grep: 패턴 검색
|
|
204
204
|
3. Read: 상세 분석
|
|
@@ -243,11 +243,10 @@ model: sonnet
|
|
|
243
243
|
- 업계 표준과 비교
|
|
244
244
|
|
|
245
245
|
## Output Format
|
|
246
|
-
|
|
247
|
-
-
|
|
248
|
-
-
|
|
249
|
-
-
|
|
250
|
-
- 마이그레이션 가이드
|
|
246
|
+
통일된 공통 구조 사용:
|
|
247
|
+
- 발견 사항 요약
|
|
248
|
+
- Critical Issues / Recommended Improvements / Best Practices Found
|
|
249
|
+
- Metrics
|
|
251
250
|
```
|
|
252
251
|
|
|
253
252
|
### 3. 자동 수정 에이전트
|
|
@@ -281,6 +280,13 @@ model: sonnet
|
|
|
281
280
|
|
|
282
281
|
## 서브에이전트 생성 워크플로우
|
|
283
282
|
|
|
283
|
+
### 단계 0: 기존 에이전트 참고
|
|
284
|
+
|
|
285
|
+
**반드시 기존 에이전트들의 구조를 먼저 참고하세요:**
|
|
286
|
+
1. `Glob("agents/*.md")`로 기존 에이전트 목록 확인
|
|
287
|
+
2. 유사한 에이전트를 `Read`해서 구조, 평가 원칙, Output Format 패턴 파악
|
|
288
|
+
3. 기존 에이전트들의 통일된 컨벤션을 따라 새 에이전트 생성
|
|
289
|
+
|
|
284
290
|
### 단계 1: 요구사항 이해
|
|
285
291
|
|
|
286
292
|
사용자에게 질문:
|
|
@@ -311,7 +317,7 @@ WebSearch로 찾기:
|
|
|
311
317
|
- 도메인에 대한 업계 베스트 프랙티스
|
|
312
318
|
- 일반적인 안티패턴
|
|
313
319
|
- 평가 기준
|
|
314
|
-
-
|
|
320
|
+
- 심각도 분류 표준
|
|
315
321
|
|
|
316
322
|
**예시 검색:**
|
|
317
323
|
- "[기술] code review checklist"
|
|
@@ -324,9 +330,7 @@ WebSearch로 찾기:
|
|
|
324
330
|
1. **찾아야 할 것** (✅)
|
|
325
331
|
2. **피해야 할 것** (❌)
|
|
326
332
|
3. **업계 표준** (고급이면 🌐)
|
|
327
|
-
4.
|
|
328
|
-
|
|
329
|
-
중요도별 기준 가중치.
|
|
333
|
+
4. **심각도 분류** (Critical / Recommended / Best Practices)
|
|
330
334
|
|
|
331
335
|
### 단계 5: 프로세스 단계 생성
|
|
332
336
|
|
|
@@ -335,18 +339,17 @@ WebSearch로 찾기:
|
|
|
335
339
|
2. **검색**: Grep으로 패턴 검색
|
|
336
340
|
3. **분석**: 파일 상세 읽기
|
|
337
341
|
4. **리서치**: (고급이면) WebSearch/WebFetch
|
|
338
|
-
5.
|
|
342
|
+
5. **분류**: 각 기준 심각도 평가
|
|
339
343
|
6. **리포트**: 구조화된 출력 생성
|
|
340
344
|
|
|
341
345
|
### 단계 6: Output Format 설계
|
|
342
346
|
|
|
343
|
-
마크다운 템플릿
|
|
344
|
-
-
|
|
345
|
-
-
|
|
346
|
-
-
|
|
347
|
-
-
|
|
348
|
-
-
|
|
349
|
-
- (고급이면) 학습 리소스
|
|
347
|
+
통일된 마크다운 템플릿 생성 (기존 에이전트 참고):
|
|
348
|
+
- 발견 사항 요약 (Critical / Recommended Improvements / Best Practices Found)
|
|
349
|
+
- Critical Issues (즉시 수정) - 위반 원칙/파일/문제/현재 코드/수정 방법
|
|
350
|
+
- Recommended Improvements (권장 개선)
|
|
351
|
+
- Best Practices Found (잘하고 있음) - 원칙/파일/잘한 점/코드
|
|
352
|
+
- Metrics (수치 지표 테이블)
|
|
350
353
|
|
|
351
354
|
### 단계 7: 서브에이전트 파일 작성
|
|
352
355
|
|
|
@@ -396,8 +399,7 @@ Write 도구로 `.md` 파일 생성:
|
|
|
396
399
|
```markdown
|
|
397
400
|
## 추가 기준
|
|
398
401
|
|
|
399
|
-
### 팀 컨벤션
|
|
400
|
-
|
|
402
|
+
### 팀 컨벤션
|
|
401
403
|
**✅ 찾아야 할 것:**
|
|
402
404
|
- [회사 네이밍 컨벤션]
|
|
403
405
|
- [팀 폴더 구조]
|
|
@@ -414,8 +416,7 @@ Write 도구로 `.md` 파일 생성:
|
|
|
414
416
|
```markdown
|
|
415
417
|
## 도메인 요구사항
|
|
416
418
|
|
|
417
|
-
### 보안 & 컴플라이언스
|
|
418
|
-
|
|
419
|
+
### 보안 & 컴플라이언스
|
|
419
420
|
**✅ 필수:**
|
|
420
421
|
- [HIPAA 컴플라이언스 패턴]
|
|
421
422
|
- [PII 데이터 처리]
|
|
@@ -430,8 +431,7 @@ Write 도구로 `.md` 파일 생성:
|
|
|
430
431
|
성능 기준 추가:
|
|
431
432
|
|
|
432
433
|
```markdown
|
|
433
|
-
### 성능 지표
|
|
434
|
-
|
|
434
|
+
### 성능 지표
|
|
435
435
|
**✅ 찾아야 할 것:**
|
|
436
436
|
- 번들 크기 < [임계값]
|
|
437
437
|
- Lazy loading 구현
|
|
@@ -456,7 +456,7 @@ Write 도구로 `.md` 파일 생성:
|
|
|
456
456
|
|
|
457
457
|
2. **요구사항 이해**
|
|
458
458
|
- 무엇을 변경해야 하나?
|
|
459
|
-
- 기준 추가?
|
|
459
|
+
- 기준 추가? 심각도 분류 수정? 모델 변경?
|
|
460
460
|
|
|
461
461
|
3. **필요시 리서치**
|
|
462
462
|
- WebSearch로 새 패턴
|
|
@@ -476,8 +476,7 @@ Write 도구로 `.md` 파일 생성:
|
|
|
476
476
|
|
|
477
477
|
**1. 새 기준 추가:**
|
|
478
478
|
```markdown
|
|
479
|
-
### [새 기준]
|
|
480
|
-
|
|
479
|
+
### [새 기준]
|
|
481
480
|
**✅ 찾아야 할 것:**
|
|
482
481
|
- [패턴 1]
|
|
483
482
|
- [패턴 2]
|
|
@@ -492,9 +491,8 @@ Write 도구로 `.md` 파일 생성:
|
|
|
492
491
|
**2. 고급으로 업그레이드 (웹 리서치 추가):**
|
|
493
492
|
- 모델 변경: `haiku` → `sonnet`
|
|
494
493
|
- 도구 추가: `+ WebFetch, WebSearch`
|
|
495
|
-
- 🌐 웹 리서치 섹션 추가
|
|
496
|
-
-
|
|
497
|
-
- 업계 비교 섹션 추가
|
|
494
|
+
- 평가 원칙에 🌐 웹 리서치 섹션 추가
|
|
495
|
+
- Output Format은 공통 구조 유지
|
|
498
496
|
|
|
499
497
|
**3. 기술 스택 특화:**
|
|
500
498
|
- 설명 업데이트
|
|
@@ -531,18 +529,11 @@ Write 도구로 `.md` 파일 생성:
|
|
|
531
529
|
- [ ] 도구 사용 가이드
|
|
532
530
|
|
|
533
531
|
### 출력
|
|
534
|
-
- [ ]
|
|
535
|
-
- [ ]
|
|
532
|
+
- [ ] 통일된 공통 구조 (발견 사항 요약 → Critical → Recommended → Best Practices → Metrics)
|
|
533
|
+
- [ ] 이슈 라벨: 위반 원칙/파일/문제/현재 코드/수정 방법
|
|
536
534
|
- [ ] file:line 참조 포맷
|
|
537
535
|
- [ ] 코드 예시 포함
|
|
538
|
-
- [ ]
|
|
539
|
-
|
|
540
|
-
### 고급 기능 (해당시)
|
|
541
|
-
- [ ] 웹 리서치 가이드
|
|
542
|
-
- [ ] 업계 비교 섹션
|
|
543
|
-
- [ ] 학습 리소스 섹션
|
|
544
|
-
- [ ] 마이그레이션 가이드
|
|
545
|
-
- [ ] ROI 분석
|
|
536
|
+
- [ ] 기존 에이전트와 Output Format 일관성 확인
|
|
546
537
|
|
|
547
538
|
### 품질
|
|
548
539
|
- [ ] 오타나 문법 오류 없음
|
|
@@ -590,7 +581,7 @@ Write 도구로 `.md` 파일 생성:
|
|
|
590
581
|
|
|
591
582
|
5. **검증 및 제시:**
|
|
592
583
|
- 생성된 파일 보여주기
|
|
593
|
-
-
|
|
584
|
+
- Output Format 구조 설명
|
|
594
585
|
- 사용 예시 제공
|
|
595
586
|
|
|
596
587
|
### 예시 2: 기술 스택에 맞게 커스터마이징
|
|
@@ -623,40 +614,34 @@ Write 도구로 `.md` 파일 생성:
|
|
|
623
614
|
- Vue 특정 추가사항 강조
|
|
624
615
|
- 사용 가이드 제공
|
|
625
616
|
|
|
626
|
-
### 예시 3:
|
|
617
|
+
### 예시 3: 새 평가 기준 추가
|
|
627
618
|
|
|
628
|
-
**사용자:** "
|
|
619
|
+
**사용자:** "code-reviewer에 성능 관련 기준을 추가해줘"
|
|
629
620
|
|
|
630
621
|
**프로세스:**
|
|
631
622
|
1. **현재 버전 읽기:**
|
|
632
623
|
```typescript
|
|
633
|
-
Read("
|
|
624
|
+
Read("code-reviewer.md")
|
|
634
625
|
```
|
|
635
626
|
|
|
636
627
|
2. **변경사항 결정:**
|
|
637
|
-
-
|
|
638
|
-
- 도구: + WebFetch, WebSearch
|
|
639
|
-
- 기준에 🌐 섹션 추가
|
|
640
|
-
-
|
|
641
|
-
- 큐레이션된 학습 경로 추가
|
|
628
|
+
- 기존 평가 원칙에 성능 기준 추가
|
|
629
|
+
- 도구: + WebFetch, WebSearch (필요시)
|
|
630
|
+
- 기준에 🌐 웹 리서치 섹션 추가
|
|
631
|
+
- Output Format은 공통 구조 유지
|
|
642
632
|
|
|
643
|
-
3.
|
|
644
|
-
- WebSearch("best
|
|
645
|
-
-
|
|
633
|
+
3. **도메인 리서치:**
|
|
634
|
+
- WebSearch("React performance best practices [current year]")
|
|
635
|
+
- 기존 react-performance-optimizer.md 참고
|
|
646
636
|
|
|
647
637
|
4. **수정 적용:**
|
|
648
638
|
```typescript
|
|
649
|
-
Edit("
|
|
650
|
-
```
|
|
651
|
-
또는 새 고급 버전 생성:
|
|
652
|
-
```typescript
|
|
653
|
-
Write("advanced-junior-checker.md", [향상된 내용])
|
|
639
|
+
Edit("code-reviewer.md", [변경사항])
|
|
654
640
|
```
|
|
655
641
|
|
|
656
|
-
5. **변경사항
|
|
657
|
-
-
|
|
658
|
-
-
|
|
659
|
-
- 필요시 README 업데이트
|
|
642
|
+
5. **변경사항 검증:**
|
|
643
|
+
- 통일된 Output Format 유지 확인
|
|
644
|
+
- 심각도 분류 일관성 확인
|
|
660
645
|
|
|
661
646
|
---
|
|
662
647
|
|
|
@@ -693,7 +678,7 @@ Write 도구로 `.md` 파일 생성:
|
|
|
693
678
|
|
|
694
679
|
1. **과도하게 복잡하게 하지 말 것**
|
|
695
680
|
- 과도한 기준 피하기 (최대 7-8개)
|
|
696
|
-
-
|
|
681
|
+
- 심각도 분류 단순하게 유지
|
|
697
682
|
- 너무 많은 관심사 섞지 않기
|
|
698
683
|
|
|
699
684
|
2. **잘못된 모델 사용 금지**
|
|
@@ -725,7 +710,7 @@ Write 도구로 `.md` 파일 생성:
|
|
|
725
710
|
- ✅ 구체적 예시 포함
|
|
726
711
|
- ✅ 실행 가능한 출력 제공
|
|
727
712
|
- ✅ file:line 참조 사용
|
|
728
|
-
- ✅
|
|
713
|
+
- ✅ 통일된 Output Format 준수
|
|
729
714
|
- ✅ 너비와 깊이 균형
|
|
730
715
|
- ✅ 유지보수 및 업데이트 가능
|
|
731
716
|
|
|
@@ -760,7 +745,7 @@ model: haiku # 빠르고 적절함
|
|
|
760
745
|
|
|
761
746
|
### ❌ 너무 많은 기준
|
|
762
747
|
```markdown
|
|
763
|
-
## 평가
|
|
748
|
+
## 평가 원칙
|
|
764
749
|
### 1. 기준 A
|
|
765
750
|
### 2. 기준 B
|
|
766
751
|
### 3. 기준 C
|
|
@@ -781,10 +766,19 @@ model: haiku # 빠르고 적절함
|
|
|
781
766
|
## Output Format
|
|
782
767
|
```markdown
|
|
783
768
|
# 보안 분석
|
|
784
|
-
|
|
785
|
-
##
|
|
786
|
-
-
|
|
787
|
-
-
|
|
769
|
+
|
|
770
|
+
## 발견 사항 요약
|
|
771
|
+
- **Critical:** N개 (즉시 수정 필요)
|
|
772
|
+
- **Recommended Improvements:** M개 (권장 개선)
|
|
773
|
+
- **Best Practices Found:** P개 (잘하고 있음)
|
|
774
|
+
|
|
775
|
+
## Critical Issues (즉시 수정)
|
|
776
|
+
### 1. [Issue Name]
|
|
777
|
+
**위반 원칙:** [해당 원칙]
|
|
778
|
+
**파일:** [file:line]
|
|
779
|
+
**문제:** [설명]
|
|
780
|
+
**현재 코드:** [코드]
|
|
781
|
+
**수정 방법:** [코드]
|
|
788
782
|
```
|
|
789
783
|
|
|
790
784
|
---
|
package/commands/code-review.md
CHANGED
|
@@ -27,7 +27,10 @@ Task 도구를 통해 다음 전문 에이전트를 사용할 수 있습니다:
|
|
|
27
27
|
| `code-reviewer` | 아키텍처, 타입 안전성, 에러 처리, 테스트, 접근성, 보안 | opus |
|
|
28
28
|
| `refactor-analyzer` | 코드 중복, 복잡성, 추상화 기회, 코드 스멜 | opus |
|
|
29
29
|
| `junior-checker` | 주니어 개발자 관점 가독성, 네이밍, 복잡도 | opus |
|
|
30
|
-
| `fundamentals-
|
|
30
|
+
| `fundamentals-readability` | Toss Fundamentals - 가독성 (코드 분리, 추상화, 함수 쪼개기, 조건 네이밍, 매직 넘버, 시점 이동, 삼항 연산자, 비교 순서) | opus |
|
|
31
|
+
| `fundamentals-predictability` | Toss Fundamentals - 예측 가능성 (이름 충돌, 반환 타입 통일, 숨은 로직) | opus |
|
|
32
|
+
| `fundamentals-cohesion` | Toss Fundamentals - 응집도 (디렉토리 구조, 매직 넘버 관리, 폼 응집도) | opus |
|
|
33
|
+
| `fundamentals-coupling` | Toss Fundamentals - 결합도 (단일 책임, 중복 코드 허용, Props Drilling) | opus |
|
|
31
34
|
| `react-performance-optimizer` | React 리렌더, 메모이제이션, 훅 최적화 | opus |
|
|
32
35
|
| `react-principles-reviewer` | React 개발 원칙 (응집도/명시성, Props 관리, 네이밍, 부수효과, AsyncBoundary) | opus |
|
|
33
36
|
| `maintainable-code-reviewer` | 유지보수성 (UI-코드 1:1 대응, 분리의 4원칙, 추상화 원칙) | opus |
|
|
@@ -55,7 +58,7 @@ Skill은 사용자 설치에 따라 다르며 추가 리뷰 가이드라인/컨
|
|
|
55
58
|
|
|
56
59
|
| 옵션 | 이름 | 사용 에이전트 | 적합한 상황 |
|
|
57
60
|
|------|------|--------------|-------------|
|
|
58
|
-
| 1 | **전체 리뷰** | 모든
|
|
61
|
+
| 1 | **전체 리뷰** | 모든 10개 에이전트 병렬 실행 | 종합 코드 리뷰 (권장) |
|
|
59
62
|
| 2 | **커스텀** | 사용자가 직접 선택 | 특정 관점만 리뷰하고 싶을 때 |
|
|
60
63
|
|
|
61
64
|
3. **Skill 포함 여부 질문** (AskUserQuestion 사용)
|
|
@@ -99,7 +102,7 @@ AskUserQuestion으로 포함할 skill 여부 질문.
|
|
|
99
102
|
```
|
|
100
103
|
리뷰 실행 중...
|
|
101
104
|
├── code-reviewer: 완료
|
|
102
|
-
├── fundamentals-
|
|
105
|
+
├── fundamentals-readability: 실행 중...
|
|
103
106
|
├── refactor-analyzer: 완료
|
|
104
107
|
└── junior-checker: 대기 중
|
|
105
108
|
```
|
|
@@ -112,7 +115,7 @@ AskUserQuestion으로 포함할 skill 여부 질문.
|
|
|
112
115
|
## 요약
|
|
113
116
|
- **리뷰 대상:** [path]
|
|
114
117
|
- **실행된 에이전트:** [list]
|
|
115
|
-
- **총 발견 사항:** N개 (Critical: X,
|
|
118
|
+
- **총 발견 사항:** N개 (Critical: X, Recommended Improvements: Y, Best Practices Found: Z)
|
|
116
119
|
|
|
117
120
|
---
|
|
118
121
|
|
|
@@ -126,7 +129,7 @@ AskUserQuestion으로 포함할 skill 여부 질문.
|
|
|
126
129
|
|
|
127
130
|
---
|
|
128
131
|
|
|
129
|
-
##
|
|
132
|
+
## Recommended Improvements (권장 개선)
|
|
130
133
|
|
|
131
134
|
### 1. [Issue Title]
|
|
132
135
|
- **발견 에이전트:** [agent name]
|
|
@@ -136,7 +139,7 @@ AskUserQuestion으로 포함할 skill 여부 질문.
|
|
|
136
139
|
|
|
137
140
|
---
|
|
138
141
|
|
|
139
|
-
##
|
|
142
|
+
## Best Practices Found (잘하고 있음)
|
|
140
143
|
|
|
141
144
|
- [Good practice 1] - [file:line]
|
|
142
145
|
- [Good practice 2] - [file:line]
|
|
@@ -148,7 +151,7 @@ AskUserQuestion으로 포함할 skill 여부 질문.
|
|
|
148
151
|
### Code Reviewer
|
|
149
152
|
[발견사항 요약]
|
|
150
153
|
|
|
151
|
-
### Fundamentals
|
|
154
|
+
### Fundamentals (Readability / Predictability / Cohesion / Coupling)
|
|
152
155
|
[발견사항 요약]
|
|
153
156
|
|
|
154
157
|
### Refactor Analyzer
|
|
@@ -158,11 +161,14 @@ AskUserQuestion으로 포함할 skill 여부 질문.
|
|
|
158
161
|
|
|
159
162
|
---
|
|
160
163
|
|
|
161
|
-
##
|
|
164
|
+
## Metrics
|
|
162
165
|
|
|
163
|
-
|
|
164
|
-
|
|
165
|
-
|
|
166
|
+
| 지표 | 수치 |
|
|
167
|
+
|------|------|
|
|
168
|
+
| 총 파일 수 | X |
|
|
169
|
+
| Critical 이슈 | N |
|
|
170
|
+
| Recommended Improvements | M |
|
|
171
|
+
| Best Practices Found | P |
|
|
166
172
|
```
|
|
167
173
|
|
|
168
174
|
## 도구 사용 예시
|
|
@@ -175,7 +181,13 @@ AskUserQuestion으로 포함할 skill 여부 질문.
|
|
|
175
181
|
// 여러 에이전트를 병렬로 실행 (단일 메시지, 다중 Task 호출)
|
|
176
182
|
Task(code-reviewer): "src/components 코드를 리뷰하세요. 아키텍처, 타입 안전성, 에러 처리, 접근성, 보안에 집중하세요. file:line 참조와 함께 발견사항을 반환하세요."
|
|
177
183
|
|
|
178
|
-
Task(fundamentals-
|
|
184
|
+
Task(fundamentals-readability): "src/components를 Toss Fundamentals 가독성 원칙으로 분석하세요. 코드 분리, 추상화, 조건 네이밍, 매직 넘버, 시점 이동, 삼항 연산자를 확인하세요. file:line 참조와 함께 발견사항을 반환하세요."
|
|
185
|
+
|
|
186
|
+
Task(fundamentals-predictability): "src/components를 Toss Fundamentals 예측 가능성 원칙으로 분석하세요. 이름 충돌, 반환 타입 통일, 숨은 로직을 확인하세요. file:line 참조와 함께 발견사항을 반환하세요."
|
|
187
|
+
|
|
188
|
+
Task(fundamentals-cohesion): "src/components를 Toss Fundamentals 응집도 원칙으로 분석하세요. 디렉토리 구조, 매직 넘버 관리, 폼 응집도를 확인하세요. file:line 참조와 함께 발견사항을 반환하세요."
|
|
189
|
+
|
|
190
|
+
Task(fundamentals-coupling): "src/components를 Toss Fundamentals 결합도 원칙으로 분석하세요. 단일 책임, 중복 코드 허용, Props Drilling을 확인하세요. file:line 참조와 함께 발견사항을 반환하세요."
|
|
179
191
|
|
|
180
192
|
Task(refactor-analyzer): "src/components의 리팩토링 기회를 분석하세요. 코드 중복, 복잡성, 추상화 기회를 확인하세요. file:line 참조와 함께 발견사항을 반환하세요."
|
|
181
193
|
|
|
@@ -73,7 +73,10 @@ Task(Plan): "다음 설계를 구현하기 위한 계획을 수립하세요:
|
|
|
73
73
|
|
|
74
74
|
| 에이전트 | 검증 내용 |
|
|
75
75
|
|----------|----------|
|
|
76
|
-
| `fundamentals-
|
|
76
|
+
| `fundamentals-readability` | 가독성 관점에서 설계 검증 |
|
|
77
|
+
| `fundamentals-predictability` | 예측 가능성 관점에서 설계 검증 |
|
|
78
|
+
| `fundamentals-cohesion` | 응집도 관점에서 설계 검증 |
|
|
79
|
+
| `fundamentals-coupling` | 결합도 관점에서 설계 검증 |
|
|
77
80
|
| `refactor-analyzer` | 추상화 기회, 재사용 가능한 컴포넌트 식별 |
|
|
78
81
|
|
|
79
82
|
**조건부 에이전트 (React 프로젝트):**
|
|
@@ -93,10 +96,22 @@ Task(Plan): "다음 설계를 구현하기 위한 계획을 수립하세요:
|
|
|
93
96
|
**에이전트 실행 예시:**
|
|
94
97
|
|
|
95
98
|
```
|
|
96
|
-
// 필수 에이전트 (항상 실행)
|
|
97
|
-
Task(fundamentals-
|
|
99
|
+
// 필수 에이전트 (항상 실행 - 4개 병렬)
|
|
100
|
+
Task(fundamentals-readability): "다음 구현 계획을 가독성 원칙으로 검증하세요:
|
|
98
101
|
[Plan 결과]
|
|
99
|
-
|
|
102
|
+
코드 분리, 추상화, 함수 쪼개기, 조건 네이밍, 매직 넘버, 시점 이동 관점에서 개선점을 제안하세요."
|
|
103
|
+
|
|
104
|
+
Task(fundamentals-predictability): "다음 구현 계획을 예측 가능성 원칙으로 검증하세요:
|
|
105
|
+
[Plan 결과]
|
|
106
|
+
이름 충돌, 반환 타입 통일, 숨은 로직 관점에서 개선점을 제안하세요."
|
|
107
|
+
|
|
108
|
+
Task(fundamentals-cohesion): "다음 구현 계획을 응집도 원칙으로 검증하세요:
|
|
109
|
+
[Plan 결과]
|
|
110
|
+
디렉토리 구조, 매직 넘버 관리, 폼 응집도 관점에서 개선점을 제안하세요."
|
|
111
|
+
|
|
112
|
+
Task(fundamentals-coupling): "다음 구현 계획을 결합도 원칙으로 검증하세요:
|
|
113
|
+
[Plan 결과]
|
|
114
|
+
단일 책임, 중복 코드 허용, Props Drilling 관점에서 개선점을 제안하세요."
|
|
100
115
|
|
|
101
116
|
Task(refactor-analyzer): "다음 구현 계획에서 추상화 기회를 분석하세요:
|
|
102
117
|
[Plan 결과]
|
|
@@ -195,13 +210,13 @@ export function NewComponent({ ... }: Props) {
|
|
|
195
210
|
|
|
196
211
|
## 4. 설계 검증 결과
|
|
197
212
|
|
|
198
|
-
###
|
|
199
|
-
| 원칙 | 상태 | 피드백 |
|
|
200
|
-
|
|
201
|
-
| 가독성 | ✅/🟡/🔴 | [피드백] |
|
|
202
|
-
| 예측 가능성 | ✅/🟡/🔴 | [피드백] |
|
|
203
|
-
| 응집도 | ✅/🟡/🔴 | [피드백] |
|
|
204
|
-
| 결합도 | ✅/🟡/🔴 | [피드백] |
|
|
213
|
+
### Toss Fundamentals 검증
|
|
214
|
+
| 원칙 | 에이전트 | 상태 | 피드백 |
|
|
215
|
+
|------|----------|------|--------|
|
|
216
|
+
| 가독성 | fundamentals-readability | ✅/🟡/🔴 | [피드백] |
|
|
217
|
+
| 예측 가능성 | fundamentals-predictability | ✅/🟡/🔴 | [피드백] |
|
|
218
|
+
| 응집도 | fundamentals-cohesion | ✅/🟡/🔴 | [피드백] |
|
|
219
|
+
| 결합도 | fundamentals-coupling | ✅/🟡/🔴 | [피드백] |
|
|
205
220
|
|
|
206
221
|
### refactor-analyzer 검증
|
|
207
222
|
**추상화 기회:**
|
package/commands/review-pr.md
CHANGED
|
@@ -19,7 +19,10 @@ Task 도구를 통해 다음 전문 에이전트를 사용할 수 있습니다:
|
|
|
19
19
|
| Agent | 초점 영역 |
|
|
20
20
|
|-------|----------|
|
|
21
21
|
| `code-reviewer` | 아키텍처, 타입 안전성, 에러 처리, 테스트, 접근성, 보안 |
|
|
22
|
-
| `fundamentals-
|
|
22
|
+
| `fundamentals-readability` | Toss Fundamentals - 가독성 |
|
|
23
|
+
| `fundamentals-predictability` | Toss Fundamentals - 예측 가능성 |
|
|
24
|
+
| `fundamentals-cohesion` | Toss Fundamentals - 응집도 |
|
|
25
|
+
| `fundamentals-coupling` | Toss Fundamentals - 결합도 |
|
|
23
26
|
| `refactor-analyzer` | 코드 중복, 복잡성, 추상화 기회, 코드 스멜 |
|
|
24
27
|
| `junior-checker` | 주니어 개발자 관점 가독성, 네이밍, 복잡도 |
|
|
25
28
|
| `react-performance-optimizer` | React 리렌더, 메모이제이션, 훅 최적화 |
|
|
@@ -91,15 +94,33 @@ Skill은 사용자 설치에 따라 다르며 추가 리뷰 가이드라인/컨
|
|
|
91
94
|
- GitHub 리뷰 스타일로 file:line 참조와 함께 발견사항 반환
|
|
92
95
|
- 심각도를 Must Fix / Should Fix / Consider / Suggestion으로 구분"
|
|
93
96
|
|
|
94
|
-
Task(fundamentals-
|
|
97
|
+
Task(fundamentals-readability): "PR #<PR번호>를 가독성 원칙으로 분석해주세요.
|
|
98
|
+
[작업 순서]
|
|
99
|
+
1. `gh pr view <PR번호>`로 PR 정보 확인
|
|
100
|
+
2. `gh pr diff <PR번호>`로 전체 변경사항 확인
|
|
101
|
+
3. 필요 시 관련 파일 Read
|
|
102
|
+
[리뷰 관점] 코드 분리, 추상화, 함수 쪼개기, 조건 네이밍, 매직 넘버, 시점 이동, 삼항 연산자, 비교 순서"
|
|
95
103
|
|
|
104
|
+
Task(fundamentals-predictability): "PR #<PR번호>를 예측 가능성 원칙으로 분석해주세요.
|
|
96
105
|
[작업 순서]
|
|
97
106
|
1. `gh pr view <PR번호>`로 PR 정보 확인
|
|
98
107
|
2. `gh pr diff <PR번호>`로 전체 변경사항 확인
|
|
99
108
|
3. 필요 시 관련 파일 Read
|
|
109
|
+
[리뷰 관점] 이름 충돌, 반환 타입 통일, 숨은 로직"
|
|
100
110
|
|
|
101
|
-
|
|
102
|
-
|
|
111
|
+
Task(fundamentals-cohesion): "PR #<PR번호>를 응집도 원칙으로 분석해주세요.
|
|
112
|
+
[작업 순서]
|
|
113
|
+
1. `gh pr view <PR번호>`로 PR 정보 확인
|
|
114
|
+
2. `gh pr diff <PR번호>`로 전체 변경사항 확인
|
|
115
|
+
3. 필요 시 관련 파일 Read
|
|
116
|
+
[리뷰 관점] 디렉토리 구조, 매직 넘버 관리, 폼 응집도"
|
|
117
|
+
|
|
118
|
+
Task(fundamentals-coupling): "PR #<PR번호>를 결합도 원칙으로 분석해주세요.
|
|
119
|
+
[작업 순서]
|
|
120
|
+
1. `gh pr view <PR번호>`로 PR 정보 확인
|
|
121
|
+
2. `gh pr diff <PR번호>`로 전체 변경사항 확인
|
|
122
|
+
3. 필요 시 관련 파일 Read
|
|
123
|
+
[리뷰 관점] 단일 책임, 중복 코드 허용, Props Drilling"
|
|
103
124
|
|
|
104
125
|
Task(refactor-analyzer): "PR #<PR번호>에서 리팩토링 기회를 찾아주세요.
|
|
105
126
|
|
|
@@ -207,7 +228,7 @@ const user: User = response.data
|
|
|
207
228
|
|
|
208
229
|
## ✅ 잘한 점
|
|
209
230
|
|
|
210
|
-
- `src/hooks/useUser.ts` - 훅 분리가 잘 되어 있음 (fundamentals-
|
|
231
|
+
- `src/hooks/useUser.ts` - 훅 분리가 잘 되어 있음 (fundamentals-cohesion)
|
|
211
232
|
- `src/types/index.ts` - 타입 정의가 명확함 (code-reviewer)
|
|
212
233
|
|
|
213
234
|
---
|
package/docs/BUILDER_GUIDE.md
CHANGED
|
@@ -50,7 +50,10 @@ Builder가 자동으로:
|
|
|
50
50
|
| `code-reviewer` | 아키텍처, 타입, 에러 처리, 테스트, 접근성, 보안 |
|
|
51
51
|
| `refactor-analyzer` | 코드 중복, 복잡성, 추상화 기회 |
|
|
52
52
|
| `junior-checker` | 주니어 친화성 |
|
|
53
|
-
| `fundamentals-
|
|
53
|
+
| `fundamentals-readability` | Toss Fundamentals - 가독성 |
|
|
54
|
+
| `fundamentals-predictability` | Toss Fundamentals - 예측 가능성 |
|
|
55
|
+
| `fundamentals-cohesion` | Toss Fundamentals - 응집도 |
|
|
56
|
+
| `fundamentals-coupling` | Toss Fundamentals - 결합도 |
|
|
54
57
|
| `react-performance-optimizer` | React 성능 최적화 |
|
|
55
58
|
| `react-principles-reviewer` | React 개발 원칙 (응집도/명시성, Props 관리, 네이밍, 부수효과) |
|
|
56
59
|
| `maintainable-code-reviewer` | 유지보수성 (UI-코드 1:1 대응, 분리의 4원칙) |
|
|
@@ -60,7 +63,7 @@ Builder가 자동으로:
|
|
|
60
63
|
**프레임워크 특화:**
|
|
61
64
|
```bash
|
|
62
65
|
"code-reviewer를 Next.js 14 App Router에 맞게 수정해줘"
|
|
63
|
-
"fundamentals-
|
|
66
|
+
"fundamentals-readability를 Vue 3에 맞게 적용해줘"
|
|
64
67
|
```
|
|
65
68
|
|
|
66
69
|
**도메인 특화:**
|