binary-agents 1.3.1 → 1.3.2
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 +2 -0
- package/commands/fundamentals-review.md +170 -0
- package/package.json +1 -1
package/README.md
CHANGED
|
@@ -61,6 +61,7 @@ npx binary-agents list
|
|
|
61
61
|
| `/pr` | 브랜치 변경사항 분석 후 PR 자동 생성 |
|
|
62
62
|
| `/review-pr` | PR 링크를 받아 변경사항 분석 후 GitHub 스타일 라인별 코드 리뷰 |
|
|
63
63
|
| `/code-review` | 여러 에이전트를 병렬 실행하여 종합 코드 리뷰 |
|
|
64
|
+
| `/fundamentals-review` | Toss Fundamentals 4개 + React 상태관리 집중 리뷰 |
|
|
64
65
|
| `/design-to-code` | 설계/요구사항을 분석하여 구현 계획 생성 |
|
|
65
66
|
| `/figma-check` | Figma MCP로 디자인 정보를 가져와 구현 코드와 비교 분석 |
|
|
66
67
|
|
|
@@ -179,6 +180,7 @@ binary-agents/
|
|
|
179
180
|
│ ├── pr.md
|
|
180
181
|
│ ├── review-pr.md
|
|
181
182
|
│ ├── code-review.md
|
|
183
|
+
│ ├── fundamentals-review.md
|
|
182
184
|
│ ├── design-to-code.md
|
|
183
185
|
│ └── figma-check.md
|
|
184
186
|
├── bin/ # CLI 실행 파일
|
|
@@ -0,0 +1,170 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Toss Fundamentals 4개 에이전트 + React 상태관리 리뷰를 병렬 실행
|
|
3
|
+
allowed-tools: Task, Read, Glob, Grep
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Fundamentals + 상태관리 리뷰
|
|
7
|
+
|
|
8
|
+
Toss Frontend Fundamentals 4개 원칙(가독성, 예측 가능성, 응집도, 결합도)과 React 상태관리 패턴을 병렬로 분석하는 집중 리뷰 커맨드입니다.
|
|
9
|
+
|
|
10
|
+
## 컨텍스트 정보
|
|
11
|
+
|
|
12
|
+
**리뷰 대상 경로:**
|
|
13
|
+
!`echo "${1:-.}"`
|
|
14
|
+
|
|
15
|
+
**현재 git 상태:**
|
|
16
|
+
!`git status --short | head -20`
|
|
17
|
+
|
|
18
|
+
**React 프로젝트 감지:**
|
|
19
|
+
!`if [ -f "package.json" ] && grep -q '"react"' package.json 2>/dev/null; then echo "✅ React 프로젝트"; else echo "❌ React 프로젝트 아님"; fi`
|
|
20
|
+
|
|
21
|
+
## 사용 에이전트
|
|
22
|
+
|
|
23
|
+
| Agent | 초점 영역 | 실행 조건 |
|
|
24
|
+
|-------|----------|----------|
|
|
25
|
+
| `fundamentals-readability` | 가독성 (코드 분리, 추상화, 함수 쪼개기, 조건 네이밍, 매직 넘버, 시점 이동, 삼항 연산자, 비교 순서) | 항상 |
|
|
26
|
+
| `fundamentals-predictability` | 예측 가능성 (이름 충돌, 반환 타입 통일, 숨은 로직) | 항상 |
|
|
27
|
+
| `fundamentals-cohesion` | 응집도 (디렉토리 구조, 매직 넘버 관리, 폼 응집도) | 항상 |
|
|
28
|
+
| `fundamentals-coupling` | 결합도 (단일 책임, 중복 코드 허용, Props Drilling) | 항상 |
|
|
29
|
+
| `react-state-reviewer` | 상태관리 (상태 성질 기반 배치, Context 오용, 5단계 에스컬레이터) | React 프로젝트 |
|
|
30
|
+
|
|
31
|
+
## 작업 순서
|
|
32
|
+
|
|
33
|
+
1. **리뷰 범위 결정**
|
|
34
|
+
- **경로가 제공된 경우:** 해당 경로의 모든 파일을 리뷰
|
|
35
|
+
- **경로가 없는 경우:** 현재 작업 디렉토리 전체를 리뷰
|
|
36
|
+
|
|
37
|
+
> **중요:** git 변경사항에 제한하지 마세요. 대상 경로의 모든 파일을 리뷰합니다.
|
|
38
|
+
|
|
39
|
+
2. **React 프로젝트 여부 확인**
|
|
40
|
+
|
|
41
|
+
다음 중 하나라도 해당되면 React 프로젝트:
|
|
42
|
+
- package.json에 `react` 또는 `next` 의존성 존재
|
|
43
|
+
- `.tsx` 또는 `.jsx` 파일 존재
|
|
44
|
+
|
|
45
|
+
3. **에이전트 병렬 실행**
|
|
46
|
+
|
|
47
|
+
Fundamentals 4개는 항상 실행. React 프로젝트면 react-state-reviewer도 추가.
|
|
48
|
+
|
|
49
|
+
```
|
|
50
|
+
// 항상 실행 (4개 병렬)
|
|
51
|
+
Task(fundamentals-readability): "[path]를 가독성 원칙으로 분석하세요. 코드 분리, 추상화, 함수 쪼개기, 조건 네이밍, 매직 넘버, 시점 이동, 삼항 연산자, 비교 순서를 확인하세요. file:line 참조와 함께 발견사항을 반환하세요."
|
|
52
|
+
|
|
53
|
+
Task(fundamentals-predictability): "[path]를 예측 가능성 원칙으로 분석하세요. 이름 충돌, 반환 타입 통일, 숨은 로직을 확인하세요. file:line 참조와 함께 발견사항을 반환하세요."
|
|
54
|
+
|
|
55
|
+
Task(fundamentals-cohesion): "[path]를 응집도 원칙으로 분석하세요. 디렉토리 구조, 매직 넘버 관리, 폼 응집도를 확인하세요. file:line 참조와 함께 발견사항을 반환하세요."
|
|
56
|
+
|
|
57
|
+
Task(fundamentals-coupling): "[path]를 결합도 원칙으로 분석하세요. 단일 책임, 중복 코드 허용, Props Drilling을 확인하세요. file:line 참조와 함께 발견사항을 반환하세요."
|
|
58
|
+
|
|
59
|
+
// React 프로젝트면 추가
|
|
60
|
+
Task(react-state-reviewer): "[path]의 상태관리 패턴을 리뷰하세요. 상태 성질(수명/범위/빈도) 기반 도구 적절성, Context 오용, 5단계 에스컬레이터 준수, 서버/클라이언트 상태 분리를 확인하세요. file:line 참조와 함께 발견사항을 반환하세요."
|
|
61
|
+
```
|
|
62
|
+
|
|
63
|
+
4. **결과 집계 및 종합**
|
|
64
|
+
- 모든 에이전트 완료 대기
|
|
65
|
+
- 발견사항을 통합 리포트로 결합
|
|
66
|
+
- 중복 발견사항 제거
|
|
67
|
+
- 심각도와 영향도로 우선순위 정렬
|
|
68
|
+
|
|
69
|
+
## 출력 형식
|
|
70
|
+
|
|
71
|
+
```markdown
|
|
72
|
+
# Fundamentals + 상태관리 리뷰 결과
|
|
73
|
+
|
|
74
|
+
## 요약
|
|
75
|
+
- **리뷰 대상:** [path]
|
|
76
|
+
- **실행된 에이전트:** [list]
|
|
77
|
+
- **총 발견 사항:** N개 (Critical: X, Recommended Improvements: Y, Best Practices Found: Z)
|
|
78
|
+
|
|
79
|
+
---
|
|
80
|
+
|
|
81
|
+
## Critical Issues (즉시 수정 필요)
|
|
82
|
+
|
|
83
|
+
### 1. [Issue Title]
|
|
84
|
+
- **발견 에이전트:** [agent name]
|
|
85
|
+
- **위반 원칙:** [principle]
|
|
86
|
+
- **위치:** [file:line]
|
|
87
|
+
- **문제:** [description]
|
|
88
|
+
- **현재 코드:**
|
|
89
|
+
```typescript
|
|
90
|
+
// 문제 코드
|
|
91
|
+
```
|
|
92
|
+
- **수정 방법:**
|
|
93
|
+
```typescript
|
|
94
|
+
// 개선 코드
|
|
95
|
+
```
|
|
96
|
+
|
|
97
|
+
---
|
|
98
|
+
|
|
99
|
+
## Recommended Improvements (권장 개선)
|
|
100
|
+
|
|
101
|
+
[같은 형식]
|
|
102
|
+
|
|
103
|
+
---
|
|
104
|
+
|
|
105
|
+
## Best Practices Found (잘하고 있음)
|
|
106
|
+
|
|
107
|
+
- [Good practice 1] - [file:line] (에이전트)
|
|
108
|
+
- [Good practice 2] - [file:line] (에이전트)
|
|
109
|
+
|
|
110
|
+
---
|
|
111
|
+
|
|
112
|
+
## 에이전트별 상세 결과
|
|
113
|
+
|
|
114
|
+
### Fundamentals - 가독성
|
|
115
|
+
[발견사항 요약]
|
|
116
|
+
|
|
117
|
+
### Fundamentals - 예측 가능성
|
|
118
|
+
[발견사항 요약]
|
|
119
|
+
|
|
120
|
+
### Fundamentals - 응집도
|
|
121
|
+
[발견사항 요약]
|
|
122
|
+
|
|
123
|
+
### Fundamentals - 결합도
|
|
124
|
+
[발견사항 요약]
|
|
125
|
+
|
|
126
|
+
### React 상태관리 (React 프로젝트만)
|
|
127
|
+
[발견사항 요약]
|
|
128
|
+
|
|
129
|
+
---
|
|
130
|
+
|
|
131
|
+
## Metrics
|
|
132
|
+
|
|
133
|
+
### 원칙별 현황
|
|
134
|
+
|
|
135
|
+
| 원칙 | Critical | Recommended | Best Practices |
|
|
136
|
+
|------|----------|-------------|----------------|
|
|
137
|
+
| 가독성 | N | N | N |
|
|
138
|
+
| 예측 가능성 | N | N | N |
|
|
139
|
+
| 응집도 | N | N | N |
|
|
140
|
+
| 결합도 | N | N | N |
|
|
141
|
+
| 상태관리 | N | N | N |
|
|
142
|
+
|
|
143
|
+
### 상태 배치 현황 (React 프로젝트만)
|
|
144
|
+
|
|
145
|
+
| 카테고리 | 개수 | 적절 | 부적절 |
|
|
146
|
+
|----------|------|------|--------|
|
|
147
|
+
| useState | N | N | N |
|
|
148
|
+
| Context API | N | N | N |
|
|
149
|
+
| 전역 스토어 | N | N | N |
|
|
150
|
+
| 서버 상태 | N | N | N |
|
|
151
|
+
```
|
|
152
|
+
|
|
153
|
+
## 사용 예시
|
|
154
|
+
|
|
155
|
+
```bash
|
|
156
|
+
# 전체 프로젝트 리뷰
|
|
157
|
+
/fundamentals-review
|
|
158
|
+
|
|
159
|
+
# 특정 경로 리뷰
|
|
160
|
+
/fundamentals-review src/components
|
|
161
|
+
```
|
|
162
|
+
|
|
163
|
+
## 중요 사항
|
|
164
|
+
|
|
165
|
+
- **병렬 실행** - 5개 에이전트를 동시에 실행하여 빠르게 분석
|
|
166
|
+
- **React 자동 감지** - React 프로젝트가 아니면 상태관리 리뷰 생략
|
|
167
|
+
- **file:line 참조 보존** - 실행 가능한 피드백을 위해 필수
|
|
168
|
+
- **한국어 출력** - 최종 리포트는 한국어로
|
|
169
|
+
- **AI attribution 금지** - "Generated by AI" footer 추가 금지
|
|
170
|
+
- **결과 파일 저장** - 최종 리포트를 `fundamentals-review.md` 파일로 저장 (이미 존재하면 `---` 구분선과 함께 하단에 추가)
|