/multi-perspective-code-review [대상]

# 예시
/multi-perspective-code-review                    # git diff 기준 변경 내용 전체
/multi-perspective-code-review payment_method     # 특정 feature 리뷰
/multi-perspective-code-review lib/features/auth  # 특정 경로 리뷰

# 변경 파일 목록 확인
git diff --name-only

# 변경 상세 내용 확인
git diff

## 코드 리뷰 결과

### 요약
- 리뷰 파일: {N}개
- 발견 이슈: Critical {N} / High {N} / Medium {N} / Low {N}

### Critical Issues
[이슈 목록]

### High Issues
[이슈 목록]

### 관점별 상세 분석

#### 1. UI/UX
[분석 결과]

#### 2. 성능
[분석 결과]

...

### 잘된 점
[긍정적 피드백]

### 권장 액션
1. [즉시 수정할 항목]
2. [개선할 항목]

// 로딩 상태 처리 확인
state.when(
  loading: () => LoadingIndicator(),  // 있는지?
  error: (e, st) => ErrorWidget(e),    // 있는지?
  data: (data) => DataWidget(data),
);

// 빈 상태 처리
if (items.isEmpty) {
  return EmptyStateWidget();  // 있는지?
}

// const 위젯 활용
const SizedBox(height: 16),  // const 붙었는지?

// ListView 최적화
ListView.builder(  // .builder 사용하는지?
  itemCount: items.length,
  itemBuilder: (context, index) => ItemWidget(items[index]),
);

// dispose에서 리소스 정리
@override
void dispose() {
  _controller.dispose();  // 정리하는지?
  _subscription?.cancel();
  super.dispose();
}

// Repository를 통한 데이터 접근
class TransactionNotifier {
  final TransactionRepository _repository;  // Repository 사용하는지?

  // 직접 Supabase 호출 금지
  // final supabase = Supabase.instance.client;
}

// Feature-first 구조
lib/features/{feature}/
  domain/entities/     # Entity 정의
  data/
    models/          # Model (JSON 변환)
    repositories/    # Repository 구현
  presentation/
    providers/       # 상태 관리
    pages/           # 화면
    widgets/         # 위젯

// 올바른 문자열
final message = '저장되었습니다';

// 틀린 문자열
// final message = "저장되었습니다";

// 올바른 주석
// 사용자가 이미 로그인되어 있으면 홈으로 이동

// 틀린 주석 (이모티콘 금지)
// 사용자 로그인 체크

// 매직 넘버 제거
const maxRetryCount = 3;  // 올바름
// if (retry < 3) {}      // 틀림

// Null 안전 처리
final amount = transaction.amount ?? 0;

// 빈 리스트 체크
if (transactions.isEmpty) {
  return EmptyState();
}

// 중복 요청 방지
if (_isLoading) return;
_isLoading = true;
try {
  await doSomething();
} finally {
  _isLoading = false;
}

// Race Condition 방지 (SafeNotifier 사용)
class MyNotifier extends SafeNotifier<Data> {
  Future<void> load() async {
    final result = await safeAsync(() => repository.fetch());
    if (result == null) return;  // disposed 체크
    safeUpdateState(AsyncValue.data(result));
  }
}

// RLS 정책 확인
// Supabase 마이그레이션에서 RLS 활성화 확인

// 민감 정보 로깅 금지
// debugPrint('Token: $accessToken');  // 금지
// debugPrint('Login successful');     // 허용

// 입력값 검증
if (amount <= 0) {
  throw ValidationException('금액은 0보다 커야 합니다');
}

// 권한 검증
if (member.role != 'owner' && member.role != 'admin') {
  throw UnauthorizedException('권한이 없습니다');
}

// SafeNotifier + rethrow 패턴
class MyNotifier extends SafeNotifier<Data> {
  Future<void> action() async {
    state = const AsyncValue.loading();
    try {
      final result = await safeAsync(() => repository.action());
      if (result == null) return;
      safeUpdateState(AsyncValue.data(result));
    } catch (e, st) {
      safeUpdateState(AsyncValue.error(e, st));
      rethrow;  // 필수!
    }
  }
}

## 다각도 코드 리뷰 결과

### 요약
| 관점 | Critical | High | Medium | Low |
|------|----------|------|--------|-----|
| UI/UX | 0 | 1 | 2 | 0 |
| 성능 | 0 | 0 | 1 | 1 |
| 디자인 | 0 | 0 | 0 | 0 |
| 코드 품질 | 0 | 1 | 0 | 2 |
| 엣지케이스 | 1 | 0 | 1 | 0 |
| 보안 | 0 | 0 | 0 | 0 |

### Critical Issues (즉시 수정)
1. **[엣지케이스]** `pending_transaction_card.dart:45`
   - 문제: 금액이 null일 때 크래시 발생
   - 해결: null 체크 추가 `amount ?? 0`

### High Issues (수정 권장)
1. **[UI/UX]** `auto_save_settings_page.dart:120`
   - 문제: 저장 버튼 클릭 시 로딩 표시 없음
   - 해결: 버튼에 CircularProgressIndicator 추가

2. **[코드 품질]** `sms_parsing_service.dart:78`
   - 문제: rethrow 누락으로 에러가 UI까지 전파 안 됨
   - 해결: catch 블록에 rethrow 추가

### 잘된 점
- SafeNotifier 패턴 일관되게 적용
- 한글 주석으로 가독성 좋음
- Repository 패턴 잘 준수

### 권장 액션
1. [ ] Critical 이슈 1건 즉시 수정
2. [ ] High 이슈 2건 수정 권장
3. [ ] Medium 이슈는 다음 스프린트에서 개선