🔀 :: (#129) - 다이얼로그 중복, 위젯 로컬 상태, pass-through 계층 정리와 CI·문서 정비

[aiden30015]

💡 개요

코드 리뷰에서 나온 중복/과설계를 정리하고, CI와 네이밍 규칙 문서를 손봤습니다.
커밋 3개로 나눠져 있어 순서대로 보시면 됩니다. 동작 변경은 없습니다.

🔗 관련 이슈

Closes #129

📃 작업내용

1. CI (e0bf2b8)

• flutter analyze에서 --no-fatal-warnings --no-fatal-infos 제거.
  어떤 지적도 CI를 실패시키지 못하는 상태였고, 실제로 info 1건이 통과 중이었습니다.
• 생성 파일(*.g.dart, *.freezed.dart)이 이미 커밋돼 있어 CI의 build_runner 재생성 단계 삭제.
• 위 게이트에 걸리던 trailing comma 1건 수정.

2. 코드 리팩터링 (6976768) — 17파일 / +100 −523

다이얼로그 5개 → 1개
banned_outing / banned_outing_release 는 기본 문구를 빼면 동일했고,
force_outing / forced_outing_release 도 마찬가지였습니다.
showOutingActionDialog 하나로 합치고 redContent / content2 는 선택 인자로 뺐습니다.

• 배색(취소 파랑 / 확인 빨강)과 문구는 기존 디자인 그대로입니다.
• 함수별 기본값에 숨어 있던 confirmText 는 호출부 5곳에서 명시하도록 바꿨습니다.

admin_outing_state_screen.dart 1187 → 1082줄
위젯 로컬 상태를 Object() 식별자 + NotifierProvider.autoDispose.family 로
관리하던 곳 3군데를 setState 로 내렸습니다. 바텀시트 안에서만 사는 상태들입니다.

• 부모가 새 props를 내려줄 때의 리셋을 didUpdateWidget 으로 명시했습니다.
  기존에는 family 키가 바뀌면서 암묵적으로 일어나던 동작입니다.
• await 이후 상태 갱신과 스낵바 노출에 mounted 가드를 넣었습니다.
  _updateUiState 는 finally 에서도 불려서, 시트가 닫힌 뒤 도착하면 터질 수 있었습니다.
• ref 를 쓰지 않게 된 두 위젯은 StatefulWidget 으로 바꿨습니다.

pass-through 계층 제거

• SettingsService / Impl 삭제 → SettingsStorage 직접 호출.
  theme_provider 와 settings_viewmodel 은 이미 직접 부르고 있어서
  같은 저장소로 가는 경로가 두 개였습니다.
• NotificationRepository / Impl 삭제 → usecase가 NotificationRemoteDataSource 를 직접 받습니다.
• PermissionService 는 유지했습니다. 구현에 실제 로직(denied → request →
  permanentlyDenied → 설정 열기)이 있고 플랫폼 API라 테스트에서 갈아끼울 값어치가 있습니다.

3. 문서 (21518b8)

docs/naming-conventions.md 에 상태 배치 기준을 추가했습니다.
기존에는 "viewmodels 디렉터리 금지"라는 이름 규칙만 있고 판단 기준이 없었습니다.

• "이 상태가 위젯보다 오래 사는가" 를 기준으로 provider / setState / 만들지 않음 3분류 정리
• 전달만 하는 저장소·서비스 계층도 두지 않는다는 조항 추가
• 디렉터리 일괄 리네임은 하지 않고, 정리 대상만 표로 남겼습니다

4. Git 컨벤션 문서화 (3bd0b9c)

문서에 디렉터리·파일 규칙만 있어서 Git 쪽 규칙은 기존 히스토리를 보고 추측해야 했습니다.
실제로 쓰이고 있는 형태를 그대로 옮겼습니다.

• 브랜치: <타입>/#<이슈번호>-<영문 설명>, 타입은 브랜치가 하는 일에 맞춘다
• 커밋: :gitmoji: :: <한글 설명>, 콜론 형식으로 통일
• PR 제목: 🔀 :: (#<이슈번호>) - <한글 설명>, 이모지는 변경 성격과 무관하게 항상 🔀

🔍 테스트 방법

• flutter analyze → 0건
• flutter test → 140개 전부 통과
• 화면 확인이 필요한 곳
  • 학생회 > 외출 현황 > 학생 편집: 강제외출 / 복귀 / 외출금지 / 외출금지 해제 다이얼로그 문구와 버튼 색
  • 같은 화면의 필터 바텀시트: 칩 선택 유지, 필터 초기화
  • 외출 현황 > 강제 복귀 다이얼로그
  • 마이페이지 > 푸시 알림 / 카메라 자동 실행 토글

🙋‍♂️ 질문사항

• 다이얼로그 버튼 배색은 의도된 디자인이라 하셔서 그대로 뒀습니다.
  다만 GomsDialog.confirm 은 비-destructive일 때 취소가 빨강 / 확인이 파랑으로
  나머지 다이얼로그와 반대입니다. 이것도 의도된 것인지 확인 부탁드립니다.
  (로그아웃, 후기 삭제 등 4곳에서 쓰입니다. 이번 PR 범위에서는 건드리지 않았습니다.)
• viewmodels 디렉터리 11개는 이번에 옮기지 않았습니다. 문서의 정리 대상 표에만 남겼습니다.

✅ 체크리스트

• 코드가 정상적으로 컴파일되고 실행되는지 확인했습니다.
• 불필요한 코드가 없는지 확인했습니다.
• 코드 스타일 가이드를 준수했습니다.
• 기존 기능이 정상적으로 작동하는지 확인했습니다.

🤖 Generated with Claude Code