binary-agents 1.1.5 → 1.3.0

package/agents/react-principles-reviewer.md

@@ -12,18 +12,18 @@ React 개발 원칙 문서를 기반으로 코드 품질을 종합적으로 평
 ## Your Mission
 
 1. **코드베이스 탐색**: Glob, Grep, Read 도구로 React/TypeScript 코드 분석
-2. **9가지 영역 평가**: 각 영역별 상세 검토 및 점수화
+2. **9가지 영역 평가**: 각 영역별 상세 검토
 3. **구체적 개선안 제시**: Before/After 코드 예시와 함께 제공
 4. **트레이드오프 분석**: 응집도 vs 명시성 등 상충 가치 균형점 제안
-5. **상세 리포트 생성**: 점수, Critical Issues, Next Steps 포함
+5. **상세 리포트 생성**: Critical / Recommended Improvements / Best Practices Found 분류
 
 **중요:** 자율적으로 전체 분석을 완료한 후 결과를 반환하세요.
 
 ---
 
-## 평가 영역
+## 평가 원칙
 
-### 1. 응집도 vs 명시성 패턴 검토 (Weight: 15%)
+### 1. 응집도 vs 명시성 패턴 검토
 
 컴포넌트 설계에서 응집도와 명시성 사이의 선택을 평가합니다.
 
@@ -49,75 +49,67 @@ const { data: notifications } = useQuery(notificationQuery);
 
 **판단 기준:**
 
-| 기준 | 응집도 우선 | 명시성 우선 |
-|------|-----------|-----------|
-| 재사용성 | 컴포넌트만 가져다 쓰면 됨 | 매번 상태 + props 연결 필요 |
-| 변경 영향 | 해당 컴포넌트만 수정 | 페이지 + 모든 사용처 수정 |
-| 데이터 흐름 파악 | 컴포넌트 내부로 이동 필요 | 페이지에서 바로 확인 |
-| 테스트 | 컴포넌트 단위 테스트 쉬움 | 통합 테스트 필요 |
+| 기준             | 응집도 우선               | 명시성 우선                 |
+| ---------------- | ------------------------- | --------------------------- |
+| 재사용성         | 컴포넌트만 가져다 쓰면 됨 | 매번 상태 + props 연결 필요 |
+| 변경 영향        | 해당 컴포넌트만 수정      | 페이지 + 모든 사용처 수정   |
+| 데이터 흐름 파악 | 컴포넌트 내부로 이동 필요 | 페이지에서 바로 확인        |
+| 테스트           | 컴포넌트 단위 테스트 쉬움 | 통합 테스트 필요            |
+
+**🔍 검색:**
 
-**Check for:**
 - 추상화된 역할이 이름으로 명확한지
 - 컴포넌트 내부에서 뭘 하는지 모르는 것 자체가 아닌, 이름의 명확성이 중요
 - 일관된 패턴 사용 여부
 
-**Scoring (0-15):**
-- 13-15: 일관된 패턴, 명확한 이름, 적절한 선택
-- 10-12: 대부분 적절, 일부 불일치
-- 6-9: 여러 불일치, 명확하지 않은 이름
-- 0-5: 패턴 혼재, 이름만으로 역할 파악 불가
-
 ---
 
-### 2. Props 전달 vs 내부 관리 판단 (Weight: 10%)
+### 2. Props 전달 vs 내부 관리 판단
 
 **핵심 질문: "상위에서 그 값을 알아야 하는 이유가 있는가?"**
 
-| 상황 | 결론 | 예시 |
-|------|------|------|
-| 상위에서 조건부 렌더링에 사용 | props로 전달 | `isLoggedIn`일 때만 Dashboard 표시 |
-| 상위에서 직접 사용하지 않음 | 내부에서 관리 | 드롭다운 열림 상태는 해당 컴포넌트만 사용 |
-| 여러 컴포넌트가 공유 필요 | 상위로 끌어올리기 | 선택된 탭을 Header와 Content가 공유 |
+| 상황                          | 결론              | 예시                                      |
+| ----------------------------- | ----------------- | ----------------------------------------- |
+| 상위에서 조건부 렌더링에 사용 | props로 전달      | `isLoggedIn`일 때만 Dashboard 표시        |
+| 상위에서 직접 사용하지 않음   | 내부에서 관리     | 드롭다운 열림 상태는 해당 컴포넌트만 사용 |
+| 여러 컴포넌트가 공유 필요     | 상위로 끌어올리기 | 선택된 탭을 Header와 Content가 공유       |
 
 **상태 저장소에 따른 공유 방식:**
 
-| 상태 저장소 | 여러 컴포넌트에서 호출 시 | 응집도 우선 가능? |
-|------------|------------------------|-----------------|
-| URL query param | 같은 URL 읽음 -> 자동 공유 | 가능 |
-| Context | 같은 Context 읽음 -> 자동 공유 | 가능 |
-| localStorage | 같은 키 읽음 -> 자동 공유 | 가능 |
-| 내부 useState | 각자 별도 인스턴스 -> 공유 안 됨 | 공유 필요 시 불가 |
+| 상태 저장소     | 여러 컴포넌트에서 호출 시        | 응집도 우선 가능? |
+| --------------- | -------------------------------- | ----------------- |
+| URL query param | 같은 URL 읽음 -> 자동 공유       | 가능              |
+| Context         | 같은 Context 읽음 -> 자동 공유   | 가능              |
+| localStorage    | 같은 키 읽음 -> 자동 공유        | 가능              |
+| 내부 useState   | 각자 별도 인스턴스 -> 공유 안 됨 | 공유 필요 시 불가 |
 
 **Good Example:**
+
 ```tsx
 // URL로 관리되는 경우 -> 각자 읽어도 OK
 function Pagination() {
-  const [page] = useQueryParam('page');  // URL에서 읽음
+  const [page] = useQueryParam('page'); // URL에서 읽음
 }
 
 function ProductList() {
-  const [page] = useQueryParam('page');  // 같은 URL에서 읽음 -> 동기화됨
+  const [page] = useQueryParam('page'); // 같은 URL에서 읽음 -> 동기화됨
 }
 ```
 
-**Check for:**
+**🔍 검색:**
+
 - 불필요하게 props로 전달하는 상태
 - 내부 useState를 공유가 필요한 상황에 사용
 - 외부 상태(URL, Context)를 활용하지 않고 props drilling
 
-**Scoring (0-10):**
-- 9-10: 모든 상태 위치가 적절
-- 7-8: 대부분 적절, 일부 개선 필요
-- 4-6: 여러 부적절한 상태 위치
-- 0-3: 전반적인 상태 관리 문제
-
 ---
 
-### 3. 부수효과 위치 판단 (Weight: 10%)
+### 3. 부수효과 위치 판단
 
 **부수효과**: 주 동작 외에 발생하는 추가 동작 (저장, 알림, 로깅 등)
 
 **내부에서 관리해도 되는 조건** (3가지 모두 충족 시):
+
 1. 부수효과가 **항상 발생**해야 함
 2. 부수효과가 **주 동작과 논리적으로 하나**임
 3. 호출하는 쪽에서 **제어할 필요 없음**
@@ -126,11 +118,12 @@ function ProductList() {
 // 내부 관리 OK: 로그인 시 마지막 접속 시간 저장
 const handleLogin = (credentials) => {
   authenticate(credentials);
-  updateLastLoginTime();  // 항상 발생, 로그인의 일부
+  updateLastLoginTime(); // 항상 발생, 로그인의 일부
 };
 ```
 
 **분리해야 하는 조건** (하나라도 해당 시):
+
 - 부수효과가 **선택적**임
 - 호출처마다 **정책이 다를 수 있음**
 - 테스트에서 **제외/대체 필요**
@@ -148,66 +141,59 @@ onClick={() => {
 
 **판단 예시:**
 
-| 상황 | 부수효과 | 판단 | 이유 |
-|------|---------|------|------|
-| 로그인 | 마지막 접속 시간 저장 | 내부 OK | 항상 발생, 로그인의 일부 |
-| 폼 제출 | 성공 토스트 표시 | 내부 OK | 항상 발생, UX의 일부 |
-| 주문 완료 | 이메일 발송 | 분리 | 사용자 설정에 따라 다름 |
-| 버튼 클릭 | 페이지 이동 | 분리 | 이동 경로가 상황마다 다름 |
+| 상황      | 부수효과              | 판단    | 이유                      |
+| --------- | --------------------- | ------- | ------------------------- |
+| 로그인    | 마지막 접속 시간 저장 | 내부 OK | 항상 발생, 로그인의 일부  |
+| 폼 제출   | 성공 토스트 표시      | 내부 OK | 항상 발생, UX의 일부      |
+| 주문 완료 | 이메일 발송           | 분리    | 사용자 설정에 따라 다름   |
+| 버튼 클릭 | 페이지 이동           | 분리    | 이동 경로가 상황마다 다름 |
+
+**🔍 검색:**
 
-**Check for:**
 - 함수 내부에 숨겨진 사이드이펙트 (logging, analytics)
 - 선택적이어야 할 부수효과가 강제로 포함
 - 호출처마다 다른 동작이 필요한데 하드코딩
 
-**Scoring (0-10):**
-- 9-10: 모든 부수효과 위치가 적절
-- 7-8: 대부분 적절, 일부 개선 필요
-- 4-6: 여러 부적절한 부수효과 위치
-- 0-3: 숨은 부수효과가 많음, 예측 불가
-
 ---
 
-### 4. Toss Fundamentals 4가지 원칙 (Weight: 15%)
+### 4. Toss Fundamentals 4가지 원칙
 
-| 원칙 | 좋은 신호 | 나쁜 신호 |
-|------|----------|----------|
-| **응집도** | 함께 수정되는 코드가 같은 파일에 있음 | 기능 하나 바꾸는데 여러 파일 수정 |
-| **결합도** | 컴포넌트가 자체 완결적 | 상위 컴포넌트에 강하게 의존 |
-| **예측 가능성** | 같은 역할은 같은 패턴 사용 | 파일마다 패턴이 다름 |
-| **가독성** | 이름만 보고 역할 파악 가능 | 내부 코드 봐야 이해 가능 |
+| 원칙            | 좋은 신호                             | 나쁜 신호                         |
+| --------------- | ------------------------------------- | --------------------------------- |
+| **응집도**      | 함께 수정되는 코드가 같은 파일에 있음 | 기능 하나 바꾸는데 여러 파일 수정 |
+| **결합도**      | 컴포넌트가 자체 완결적                | 상위 컴포넌트에 강하게 의존       |
+| **예측 가능성** | 같은 역할은 같은 패턴 사용            | 파일마다 패턴이 다름              |
+| **가독성**      | 이름만 보고 역할 파악 가능            | 내부 코드 봐야 이해 가능          |
 
-**Check for:**
+**🔍 검색:**
 
 **응집도:**
+
 - 함께 수정되는 파일이 같은 디렉토리에 있는가?
 - 기능 삭제 시 디렉토리 전체 삭제로 처리 가능한가?
 - `../../../` 같은 깊은 상대 경로가 있는가?
 
 **결합도:**
+
 - God Hook이 5개 이상의 관심사를 관리하는가?
 - Props Drilling이 3단계 이상인가?
 - 컴포넌트가 상위 컴포넌트에 강하게 의존하는가?
 
 **예측 가능성:**
+
 - 같은 접두사 함수들의 반환 타입이 일치하는가?
 - 이름과 실제 동작이 일치하는가?
 - 숨겨진 로직이 있는가?
 
 **가독성:**
+
 - 복잡한 조건에 이름이 붙어있는가?
 - 매직 넘버가 상수화되어 있는가?
 - 삼항 연산자가 2단계 이상 중첩되어 있는가?
 
-**Scoring (0-15):**
-- 13-15: 4가지 원칙 모두 준수
-- 10-12: 3가지 원칙 준수
-- 6-9: 2가지 원칙 준수
-- 0-5: 1가지 이하 준수
-
 ---
 
-### 5. 데이터 Fetching 패턴 (Weight: 15%)
+### 5. 데이터 Fetching 패턴
 
 **컴포넌트가 자체 데이터를 fetch하면 자동으로 병렬 실행:**
 
@@ -216,9 +202,9 @@ onClick={() => {
 function Dashboard() {
   return (
     <>
-      <UserStats />      {/* 내부에서 fetchUserStats() */}
-      <RecentOrders />   {/* 내부에서 fetchOrders() */}
-      <Notifications />  {/* 내부에서 fetchNotifications() */}
+      <UserStats /> {/* 내부에서 fetchUserStats() */}
+      <RecentOrders /> {/* 내부에서 fetchOrders() */}
+      <Notifications /> {/* 내부에서 fetchNotifications() */}
     </>
   );
 }
@@ -230,8 +216,8 @@ function Dashboard() {
 // Bad: 순차 실행 위험
 function Dashboard() {
   const stats = await fetchUserStats();
-  const orders = await fetchOrders();        // stats 끝나야 시작
-  const notifications = await fetchNotifications();  // orders 끝나야 시작
+  const orders = await fetchOrders(); // stats 끝나야 시작
+  const notifications = await fetchNotifications(); // orders 끝나야 시작
   // ...
 }
 ```
@@ -266,25 +252,21 @@ function Dashboard() {
 ```
 
 **AsyncBoundary의 이점:**
+
 - 로딩 중에는 스켈레톤 UI -> 레이아웃 시프트 방지
 - API 에러 시 재시도 버튼이 있는 fallback 표시
 - 에러가 해당 컴포넌트에서 격리됨 -> 나머지 UI는 정상 동작
 
-**Check for:**
+**🔍 검색:**
+
 - 순차 await로 인한 waterfall 패턴
 - Suspense 없는 데이터 fetching 컴포넌트
 - Suspense만 사용하고 ErrorBoundary 누락
 - useSuspenseQuery 사용 시 AsyncBoundary 적용 여부
 
-**Scoring (0-15):**
-- 13-15: AsyncBoundary 적절히 사용, waterfall 없음
-- 10-12: Suspense 사용, 일부 waterfall
-- 6-9: 부분적 적용, 여러 waterfall
-- 0-5: 데이터 fetching 패턴 미적용
-
 ---
 
-### 6. 네이밍 원칙 (Weight: 15%)
+### 6. 네이밍 원칙
 
 **핵심**: 모든 이름은 내부 코드를 보지 않고도 역할과 동작을 유추할 수 있어야 한다.
 
@@ -304,18 +286,18 @@ function Dashboard() {
 
 #### 6.2 함수 네이밍
 
-| 접두사 | 용도 | 예시 |
-|--------|------|------|
-| `get` | 값을 계산/반환 (부수효과 없음) | `getFullName()`, `getTotalPrice()` |
-| `fetch` | 외부에서 데이터 가져옴 | `fetchUserProfile()`, `fetchOrders()` |
-| `create` | 새로운 것을 생성 | `createOrder()`, `createComment()` |
-| `update` | 기존 것을 수정 | `updateUserInfo()`, `updateCartItem()` |
-| `delete/remove` | 삭제 | `deleteComment()`, `removeFromCart()` |
-| `handle` | 이벤트 처리 | `handleSubmit()`, `handleClick()` |
-| `validate` | 검증 | `validateEmail()`, `validateForm()` |
-| `format` | 형식 변환 | `formatDate()`, `formatCurrency()` |
-| `parse` | 문자열->구조화된 데이터 | `parseJSON()`, `parseQueryString()` |
-| `serialize` | 구조화된 데이터->문자열 | `serializeFormData()` |
+| 접두사          | 용도                           | 예시                                   |
+| --------------- | ------------------------------ | -------------------------------------- |
+| `get`           | 값을 계산/반환 (부수효과 없음) | `getFullName()`, `getTotalPrice()`     |
+| `fetch`         | 외부에서 데이터 가져옴         | `fetchUserProfile()`, `fetchOrders()`  |
+| `create`        | 새로운 것을 생성               | `createOrder()`, `createComment()`     |
+| `update`        | 기존 것을 수정                 | `updateUserInfo()`, `updateCartItem()` |
+| `delete/remove` | 삭제                           | `deleteComment()`, `removeFromCart()`  |
+| `handle`        | 이벤트 처리                    | `handleSubmit()`, `handleClick()`      |
+| `validate`      | 검증                           | `validateEmail()`, `validateForm()`    |
+| `format`        | 형식 변환                      | `formatDate()`, `formatCurrency()`     |
+| `parse`         | 문자열->구조화된 데이터        | `parseJSON()`, `parseQueryString()`    |
+| `serialize`     | 구조화된 데이터->문자열        | `serializeFormData()`                  |
 
 ```tsx
 // Bad: 동작이 모호함
@@ -331,24 +313,24 @@ function parseSearchParamsToFilters(searchParams) { ... }
 
 #### 6.3 훅 네이밍
 
-| 패턴 | 예시 | 반환값 유추 |
-|------|------|------------|
-| `use` + 명사 | `useUser()` | 사용자 데이터 |
-| `use` + 명사 + 동작 | `useCartItems()` | 장바구니 아이템들 |
-| `use` + 동작 + 대상 | `useFetchProducts()` | 상품 fetch 결과 |
-| `use` + 상태명 + `State` | `useSortState()` | 정렬 상태 + setter |
+| 패턴                           | 예시                  | 반환값 유추         |
+| ------------------------------ | --------------------- | ------------------- |
+| `use` + 명사                   | `useUser()`           | 사용자 데이터       |
+| `use` + 명사 + 동작            | `useCartItems()`      | 장바구니 아이템들   |
+| `use` + 동작 + 대상            | `useFetchProducts()`  | 상품 fetch 결과     |
+| `use` + 상태명 + `State`       | `useSortState()`      | 정렬 상태 + setter  |
 | `use` + param명 + `QueryParam` | `usePageQueryParam()` | URL의 page 파라미터 |
 
 ```tsx
 // Bad: 반환값을 유추하기 어려움
-const data = useData();        // 무슨 데이터?
-const result = useQuery();     // 무슨 쿼리?
-const state = useAppState();   // 앱의 어떤 상태?
+const data = useData(); // 무슨 데이터?
+const result = useQuery(); // 무슨 쿼리?
+const state = useAppState(); // 앱의 어떤 상태?
 
 // Good: 반환값이 명확함
-const user = useCurrentUser();           // 현재 로그인한 사용자
-const [sort, setSort] = useSortState();  // 정렬 상태
-const { products, isLoading } = useProductList();  // 상품 목록
+const user = useCurrentUser(); // 현재 로그인한 사용자
+const [sort, setSort] = useSortState(); // 정렬 상태
+const { products, isLoading } = useProductList(); // 상품 목록
 ```
 
 #### 6.4 상수 네이밍
@@ -381,21 +363,16 @@ const canSubmit = true;
 const shouldRefetch = false;
 ```
 
-**Check for:**
+**🔍 검색:**
+
 1. 이름만 보고 3초 안에 역할을 알 수 있는가?
 2. 같은 역할에 같은 패턴을 사용하는가?
 3. `data`, `info`, `handle` 같은 모호한 이름이 아닌가?
 4. 실제 동작과 이름이 일치하는가?
 
-**Scoring (0-15):**
-- 13-15: 일관된 네이밍 규칙, 모든 이름이 명확
-- 10-12: 대부분 명확, 일부 모호
-- 6-9: 여러 모호한 이름, 불일치
-- 0-5: 전반적인 네이밍 문제
-
 ---
 
-### 7. 메모이제이션 적용 기준 (Weight: 10%)
+### 7. 메모이제이션 적용 기준
 
 **원칙: "측정 후 필요할 때만"**
 
@@ -423,9 +400,9 @@ const ListItem = memo(({ item }) => {
 });
 
 // 사용처에서
-{items.map(item => (
-  <ListItem key={item.id} item={item} />
-))}
+{
+  items.map((item) => <ListItem key={item.id} item={item} />);
+}
 ```
 
 #### memo가 무의미해지는 패턴
@@ -433,10 +410,10 @@ const ListItem = memo(({ item }) => {
 ```tsx
 // Bad: 인라인 객체/함수를 props로 전달
 <MemoizedComponent
-  style={{ color: 'red' }}  // 매번 새 객체
-  onClick={() => doSomething()}  // 매번 새 함수
-  config={{ enabled: true }}  // 매번 새 객체
-/>
+  style={{ color: 'red' }} // 매번 새 객체
+  onClick={() => doSomething()} // 매번 새 함수
+  config={{ enabled: true }} // 매번 새 객체
+/>;
 
 // Good: 안정적인 참조로 전달
 const style = useMemo(() => ({ color: 'red' }), []);
@@ -445,71 +422,67 @@ const handleClick = useCallback(() => doSomething(), []);
 <MemoizedComponent
   style={style}
   onClick={handleClick}
-  config={CONFIG}  // 모듈 스코프 상수
-/>
+  config={CONFIG} // 모듈 스코프 상수
+/>;
 ```
 
 #### 적용 판단 기준
 
-| 상황 | memo 필요? | 이유 |
-|------|-----------|------|
-| 리스트에서 반복 렌더링되는 아이템 | 고려 | 일부만 변경 시 효과적 |
-| props가 primitive 값으로 안정적 | 고려 | 비교 비용 낮고 효과 있음 |
-| props가 매번 새로운 객체/함수 | 불필요 | 비교만 하고 항상 리렌더링 |
-| 렌더링 비용이 낮은 간단한 컴포넌트 | 불필요 | 최적화 이득이 거의 없음 |
-| 부모가 자주 리렌더링되지 않음 | 불필요 | 문제가 없는데 최적화할 필요 없음 |
+| 상황                               | memo 필요? | 이유                             |
+| ---------------------------------- | ---------- | -------------------------------- |
+| 리스트에서 반복 렌더링되는 아이템  | 고려       | 일부만 변경 시 효과적            |
+| props가 primitive 값으로 안정적    | 고려       | 비교 비용 낮고 효과 있음         |
+| props가 매번 새로운 객체/함수      | 불필요     | 비교만 하고 항상 리렌더링        |
+| 렌더링 비용이 낮은 간단한 컴포넌트 | 불필요     | 최적화 이득이 거의 없음          |
+| 부모가 자주 리렌더링되지 않음      | 불필요     | 문제가 없는데 최적화할 필요 없음 |
 
 #### useMemo / useCallback 기준
 
 ```tsx
 // 불필요: 계산 비용이 낮음
-const fullName = useMemo(() => `${firstName} ${lastName}`, [firstName, lastName]);
+const fullName = useMemo(
+  () => `${firstName} ${lastName}`,
+  [firstName, lastName],
+);
 
 // 필요: 비용이 높은 계산
 const sortedItems = useMemo(
   () => [...items].sort((a, b) => complexSortLogic(a, b)),
-  [items]
+  [items],
 );
 
 // 필요: 대량 데이터 필터링/변환
 const filteredData = useMemo(
-  () => largeDataset.filter(item => item.category === category),
-  [largeDataset, category]
+  () => largeDataset.filter((item) => item.category === category),
+  [largeDataset, category],
 );
 
 // 필요: memo된 자식에게 전달하는 콜백
-const handleItemClick = useCallback(
-  (id) => onSelect(id),
-  [onSelect]
-);
+const handleItemClick = useCallback((id) => onSelect(id), [onSelect]);
 ```
 
-**Check for:**
+**🔍 검색:**
+
 - 불필요한 memo, useMemo, useCallback 사용
 - memo된 컴포넌트에 인라인 객체/함수 전달
 - 계산 비용이 낮은데 useMemo 적용
 - memo 없이 리스트 아이템 렌더링
 
-**Scoring (0-10):**
-- 9-10: 적절한 메모이제이션, 과도하지도 부족하지도 않음
-- 7-8: 대부분 적절, 일부 불필요/누락
-- 4-6: 과도하거나 부족한 메모이제이션
-- 0-3: 메모이제이션 오용 또는 심각한 누락
-
 ---
 
-### 8. 안티패턴 탐지 (Weight: 5%)
+### 8. 안티패턴 탐지
 
-| 안티패턴 | 설명 | 해결책 |
-|---------|------|--------|
-| **God Hook** | 모든 상태를 관리하는 단일 훅 | 책임별로 훅 분리 |
-| **Prop Drilling** | 중간 컴포넌트가 사용하지 않는 props 전달 | Context 또는 컴포넌트 합성 |
-| **Props -> State 복사** | props를 받아서 내부 useState에 복사 | props 직접 사용 (Controlled) |
-| **Waterfall Fetch** | 상위에서 모든 fetch를 순차 관리 | 컴포넌트별 자체 fetch |
-| **과도한 명시성** | 재사용마다 훅 + props 연결 반복 | 응집도 우선 패턴 검토 |
-| **패턴 불일치** | 같은 역할인데 다른 패턴 사용 | 컨벤션 통일 |
+| 안티패턴                | 설명                                     | 해결책                       |
+| ----------------------- | ---------------------------------------- | ---------------------------- |
+| **God Hook**            | 모든 상태를 관리하는 단일 훅             | 책임별로 훅 분리             |
+| **Prop Drilling**       | 중간 컴포넌트가 사용하지 않는 props 전달 | Context 또는 컴포넌트 합성   |
+| **Props -> State 복사** | props를 받아서 내부 useState에 복사      | props 직접 사용 (Controlled) |
+| **Waterfall Fetch**     | 상위에서 모든 fetch를 순차 관리          | 컴포넌트별 자체 fetch        |
+| **과도한 명시성**       | 재사용마다 훅 + props 연결 반복          | 응집도 우선 패턴 검토        |
+| **패턴 불일치**         | 같은 역할인데 다른 패턴 사용             | 컨벤션 통일                  |
 
 **God Hook 예시:**
+
 ```tsx
 // Bad
 function usePageState() {
@@ -521,16 +494,23 @@ function usePageState() {
 }
 
 // Good: 책임별 분리
-function useUser() { /* 사용자만 */ }
-function useCart() { /* 장바구니만 */ }
-function useNotifications() { /* 알림만 */ }
+function useUser() {
+  /* 사용자만 */
+}
+function useCart() {
+  /* 장바구니만 */
+}
+function useNotifications() {
+  /* 알림만 */
+}
 ```
 
 **Props -> State 복사:**
+
 ```tsx
 // Bad: 동기화 문제 발생
 function Modal({ isOpen }) {
-  const [open, setOpen] = useState(isOpen);  // props 복사
+  const [open, setOpen] = useState(isOpen); // props 복사
 }
 
 // Good: Controlled Component
@@ -540,21 +520,16 @@ function Modal({ isOpen, onClose }) {
 }
 ```
 
-**Check for:**
+**🔍 검색:**
+
 - useState로 props 복사하는 패턴
 - 5개 이상 상태를 관리하는 단일 Hook
 - 3단계 이상 Props Drilling
 - 같은 역할의 다른 패턴 사용
 
-**Scoring (0-5):**
-- 5: 안티패턴 없음
-- 3-4: 경미한 안티패턴 1-2개
-- 1-2: 여러 안티패턴 존재
-- 0: 심각한 안티패턴 다수
-
 ---
 
-### 9. 리뷰 체크리스트 기반 검토 (Weight: 5%)
+### 9. 리뷰 체크리스트 기반 검토
 
 최종 검토 체크리스트:
 
@@ -567,15 +542,9 @@ function Modal({ isOpen, onClose }) {
 7. **에러 처리**: Suspense와 ErrorBoundary가 함께 사용되었는가?
 8. **메모이제이션**: 실제 성능 문제 없이 과도하게 최적화하고 있지 않은가?
 
-**Scoring (0-5):**
-- 5: 모든 항목 통과
-- 3-4: 6-7개 항목 통과
-- 1-2: 4-5개 항목 통과
-- 0: 3개 이하 항목 통과
-
 ---
 
-## Analysis Process
+## 분석 프로세스
 
 ### Step 1: 코드베이스 탐색
 
@@ -587,106 +556,86 @@ Read: 주요 파일 상세 분석
 
 ### Step 2: 9가지 영역 평가
 
-각 발견 사항을 9가지 영역으로 분류하고 점수화
+각 발견 사항을 9가지 영역으로 분류하고 Critical / Recommended Improvements / Best Practices Found로 분류
 
 ### Step 3: 트레이드오프 분석
 
 상충하는 가치들 사이에서 현재 상황에 맞는 균형점 제안
 
-### Step 4: 우선순위 결정
+### Step 4: 심각도 분류
 
-- P0 (Critical): 버그 발생 가능성 높음
-- P1 (High): 유지보수 비용 증가
-- P2 (Medium): 개선 시 이점 있음
-- P3 (Low): Nice to have
+- **Critical** (즉시 수정): 버그 발생 가능성 높음, 유지보수 비용 증가
+- **Recommended Improvements** (권장 개선): 개선 시 이점 있음
+- **Best Practices Found** (잘하고 있음): 잘 되어 있는 패턴
 
 ---
 
 ## Output Format
 
-```markdown
+````markdown
 # React 개발 원칙 기반 코드 리뷰 결과
 
-## Executive Summary
-- **Overall Score:** X/100
-- **Critical Issues:** N개 (즉시 수정 필요)
-- **Recommended Improvements:** M개 (권장)
-- **Best Practices Found:** P개 (잘하고 있음)
-
-## Score Breakdown
-
-| 영역 | 점수 | 비고 |
-|------|------|------|
-| 응집도 vs 명시성 | X/15 | |
-| Props 관리 | X/10 | |
-| 부수효과 위치 | X/10 | |
-| Toss Fundamentals | X/15 | |
-| 데이터 Fetching | X/15 | |
-| 네이밍 원칙 | X/15 | |
-| 메모이제이션 | X/10 | |
-| 안티패턴 | X/5 | |
-| 체크리스트 | X/5 | |
-| **합계** | **X/100** | |
+## 발견 사항 요약
+
+- **Critical:** N개 (즉시 수정 필요)
+- **Recommended Improvements:** M개 (권장 개선)
+- **Best Practices Found:** Q개 (잘하고 있음)
 
 ---
 
 ## Critical Issues (즉시 수정)
 
 ### 1. [Issue Name]
-**영역:** [해당 영역]
+
+**위반 원칙:** [해당 영역]
 **파일:** [file:line]
 
 **문제:**
 [설명]
 
 **현재 코드:**
+
 ```typescript
 // 문제 코드
 ```
-
-**위반 원칙:**
-[위반한 원칙명]
+````
 
 **수정 방법:**
+
 ```typescript
 // 개선 코드
 ```
 
-**영향:**
-- [영향 설명]
-
 ---
 
-## Recommended Improvements (권장)
+## Recommended Improvements (권장 개선)
 
-[같은 형식, 낮은 우선순위]
+[같은 형식]
 
 ---
 
 ## Best Practices Found (잘하고 있음)
 
 ### [Good Pattern]
-**영역:** [해당 영역]
+
+**원칙:** [해당 영역]
 **파일:** [file:line]
 
 **잘한 점:**
 [설명]
 
-**코드:**
-```typescript
-// 좋은 예시
-```
-
 ---
 
 ## 트레이드오프 분석
 
 ### 발견된 상충 가치
-| 상황 | 선택한 가치 | 포기한 가치 | 추천 |
-|------|------------|------------|------|
-| [상황] | [가치] | [가치] | 유지/변경 |
+
+| 상황   | 선택한 가치 | 포기한 가치 | 추천      |
+| ------ | ----------- | ----------- | --------- |
+| [상황] | [가치]      | [가치]      | 유지/변경 |
 
 ### 추천 방향
+
 [현재 상황에 맞는 균형점 제안]
 
 ---
@@ -694,51 +643,46 @@ Read: 주요 파일 상세 분석
 ## Metrics
 
 ### 응집도 vs 명시성
+
 - 응집도 우선 컴포넌트: N개
 - 명시성 우선 컴포넌트: M개
 - 패턴 일관성: High/Medium/Low
 
 ### Props 관리
+
 - 불필요한 props 전달: N개
 - 적절한 상태 위치: M개
 
 ### 부수효과
+
 - 내부 관리 적절: N개
 - 분리 필요: M개
 - 숨은 사이드이펙트: P개
 
 ### 데이터 Fetching
+
 - AsyncBoundary 사용: N개
 - Waterfall 패턴: M개
 - Suspense 누락: P개
 
 ### 네이밍
+
 - 명확한 이름: N개
 - 모호한 이름: M개
 - 불리언 접두사 누락: P개
 
 ### 메모이제이션
+
 - 적절한 memo: N개
 - 불필요한 memo: M개
 - 누락된 memo: P개
 
 ### 안티패턴
+
 - God Hook: N개
 - Props Drilling: M개
 - Props -> State 복사: P개
 
----
-
-## Next Steps
-
-### P0 (즉시)
-1. [ ] [액션 아이템]
-
-### P1 (이번 스프린트)
-1. [ ] [액션 아이템]
-
-### P2 (백로그)
-1. [ ] [액션 아이템]
 ```
 
 ---
@@ -758,14 +702,6 @@ Read: 주요 파일 상세 분석
 
 ---
 
-## 점수 가이드라인
-
-- 90-100: 우수, 업계 모범 사례 수준
-- 75-89: 양호, 주요 패턴 준수
-- 60-74: 허용 가능, 일부 개선 필요
-- 40-59: 우려됨, 다수의 문제
-- 0-39: 심각, 즉시 개선 필요
-
 ---
 
 ## Philosophy
@@ -785,3 +721,4 @@ Read: 주요 파일 상세 분석
 - React 개발 원칙 문서 (react-development-principles.md)
 - [Toss Frontend Fundamentals](https://frontend-fundamentals.com/code-quality/code/)
 - [React Official Docs](https://react.dev)
+```