dev by slz6k · Pull Request #445 · Peeroreum/client

slz6k · 2026-05-15T07:07:05Z

resolved #443

Feat/#318

Feat/#330 답변 채택 확인 팝업

… pop(#333)

Feat/#333

Feat/#332

Feat/#336

feat: 화이트보드 다이얼로그 변경(#340)

Feat/#338

Feat/#337

- mypage_profile: Kakao FeedTemplate으로 프로필 공유 (peeroreum://profile/{nickname}) - MainActivity: getInitialNickname cold start / onDeepLinkProfile warm start 처리 - AndroidManifest: peeroreum://profile intent filter 추가 - main: extractNickname, onDeepLinkProfile 핸들러 / Android uriLinkStream 중복 제거 - pending_deep_link: profileNickname 필드 추가 - bottomNaviBar: profileNickname 딥링크 라우팅 추가

- wedu_room_info_sheet: showModalBottomSheet → Get.bottomSheet() 교체 (GetX Navigator 초기화 중 동일 GlobalKey<NavigatorState>가 Navigator와 _FocusInheritedScope에 중복 등록되는 문제 방지) - bottomNaviBar: wedu roomId 처리에 300ms delay 추가 - nested Scaffold 제거: mypage.dart, wedu_detail_screen.dart, mypage_profile.dart

- WeduRoomInfoSheet.onShare 콜백 타입을 VoidCallback → void Function(Rect?)으로 변경 - 공유 버튼을 Builder로 감싸 RenderBox 위치를 계산, 콜백에 Rect 전달 - wedu_home, wedu_search_result: sharePositionOrigin: shareRect 추가 - mypage_profile: GlobalKey(_shareButtonKey)로 공유 버튼 위치 추적 후 sharePositionOrigin 전달 - iOS에서 카카오톡 미설치 시 fallback Share.share() 호출 시 PlatformException 해결

- changeDetailWedu() 방장/일반 참여자 메뉴 모두에 공유하기 항목 추가 - 카카오톡 설치 시 FeedTemplate 공유, 미설치 시 Share.share() fallback - iOS sharePositionOrigin: dots 버튼(_dotsButtonKey)을 앵커로 사용 - 공유하기 탭 시 바텀시트 자동 닫힘(Get.back()) 후 공유 실행 - 일반 참여자 메뉴: GestureDetector → Column+TextButton으로 리팩터

Feat/#217

- SafeArea 제거 후 명시적 bottom 패딩 적용 - 공식: viewPadding.bottom > 20 ? viewPadding.bottom : 20 - iOS 홈 인디케이터(~34pt): viewPadding 값 사용 - Android 제스처 내비게이션(0): 20px 고정 - Android 3버튼 내비게이션(~48dp): viewPadding 값 사용

- shareDefault → shareCustom (templateId: 102956) - 같이방 상세에서 invitationUrl fetch 추가 - templateArgs: RoomName, ImageUrl(초대장 이미지), Link(딥링크) - wedu_detail_screen.dart, wedu_home.dart 동일 적용

- shareDefault → shareCustom (templateId: 102993) - templateArgs: UserName - 딥링크: 카카오 콘솔 템플릿에서 nickname=${UserName} 파라미터로 처리

- AppDelegate에 kakaoa://kakaolink 스킴 핸들러 추가 - 웜 스타트: application(_:open:options:)에서 MethodChannel로 전달 - 콜드 스타트: launchOptions URL 파싱 후 getInitialRoomId/getInitialNickname으로 응답 - main.dart: iOS 콜드 스타트 getInitialLink() 호출 추가

- AppDelegate: super.application() 호출 후 MethodChannel 설정하도록 순서 수정 (window 초기화 보장) - AppDelegate: FlutterViewController 캐스팅을 force cast에서 optional cast로 변경 (크래시 방지) - main.dart: iOS getInitialLink() 제거 — AppDelegate MethodChannel이 cold start 처리하므로 중복 방지 - main.dart: uriLinkStream에서 kakaoa:// 스킴 필터링 — AppDelegate MethodChannel과 중복 처리 방지

- fetchSocialLogin: http.get → http.post, email을 query param 대신 JSON body로 전송

- ApiClient.dart 신규 추가: Dio 싱글톤, 자동 토큰 갱신 인터셉터 (401 응답 시 /token/reissue 호출 후 원래 요청 재시도) - 기존 Authorization 헤더가 있을 경우 인터셉터가 덮어쓰지 않도록 처리 (비밀번호 재설정 등 특수 토큰 사용 케이스 대응) - main.dart에 ApiClient().init() 초기화 추가 - 소셜 로그인 GET → POST 변경, 응답에서 accessToken/refreshToken 등 정상 저장 - 아래 전체 화면의 직접 Dio/http 호출을 ApiClient()로 일괄 전환: iedu: iedu_create, iedu_home, iedu_detail, iedu_in, iedu_search_result wedu: wedu_home, wedu_create_screen, wedu_create_invitation, wedu_modify_screen, wedu_detail_screen, wedu_detail_calendar, wedu_in, wedu_search_result_screen, wedu_room_info_sheet, management_checklist_screen mypage: mypage, mypage_account, mypage_acount_ps, mypage_profile, mypage_scrap, follower_api, following_api ranking: ranking_api sign: signin_screen, signin_email_screen, signup_grade/subject/school/nickname/email, password_change (email, validate, new_password) 기타: NotificationApi, bottomNaviBar, alert_controller, VisitCount

- 카카오: loginWithKakaoTalk/Account() 반환 OAuthToken.accessToken 서버 전송 - 구글: GoogleSignInAuthentication.accessToken 서버 전송 - fetchSocialLogin 시그니처 변경: email → (provider, token, emailForSignup) - 404(신규 가입) 케이스는 SDK에서 가져온 이메일 그대로 사용 - Apple Sign-In 별도 구현 필요로 미지원 처리

Refactor/#443

github-actions · 2026-05-15T07:07:49Z

🤖 Claude Code Review

🔴 버그/보안 이슈

민감한 정보 노출: google-services.json 파일에 Firebase API 키와 OAuth 클라이언트 정보가 평문으로 저장되어 있습니다. 이 파일이 공개 저장소에 커밋되면 보안 위험이 있습니다.
환경 변수 검증 부재: claude_review.py에서 ANTHROPIC_API_KEY 환경 변수 존재 여부를 확인하지 않아 런타임 에러 가능성이 있습니다.
파일 인코딩 문제: Python 스크립트에서 파일 읽기/쓰기 시 인코딩을 명시하지 않아 한글 처리 시 문제가 발생할 수 있습니다.

🟡 개선 권장 사항

예외 처리 강화:
- Claude API 호출 실패 시 예외 처리 추가 필요
- Git diff 명령어 실패 시 처리 로직 필요
코드 정리:
- MainActivity.kt 파일이 잘린 상태로 보이므로 완전한 코드 확인 필요
- .gitignore에 .env 항목이 중복되어 있습니다
권한 최소화: GitHub Actions workflow에서 필요한 최소 권한만 부여하는 것이 좋습니다.
설정 값 외부화: 하드코딩된 값들(MAX_DIFF_CHARS, 모델명 등)을 환경 변수나 설정 파일로 분리 권장

🟢 잘된 점

자동화된 코드 리뷰 시스템: GitHub Actions를 통한 자동 코드 리뷰 구현이 잘되어 있습니다.
조건부 실행: PR과 코멘트 이벤트에 대한 적절한 조건 처리가 구현되어 있습니다.
Firebase 통합: Android 앱에 Firebase 서비스가 체계적으로 설정되어 있습니다.
딥링크 설정: 카카오 로그인과 앱 내 딥링크가 잘 구성되어 있습니다.
패키지명 변경: 예제에서 실제 앱으로 패키지명이 적절히 변경되었습니다.

우선순위 요약 테이블

우선순위	항목	분류	설명
🔥 높음	Firebase 설정 파일 보안	보안	`google-services.json` 보안 검토 필요
🔥 높음	환경 변수 검증	버그	API 키 존재 여부 확인 로직 추가
🟡 중간	예외 처리	안정성	API 호출 및 파일 처리 예외 처리 강화
🟡 중간	파일 인코딩	버그	UTF-8 인코딩 명시적 지정
🟢 낮음	코드 정리	품질	중복 제거 및 설정 외부화

이 PR은 자동화된 코드 리뷰 시스템과 Firebase 통합을 추가하는 의미있는 변경사항입니다. 다만 보안 관련 이슈들을 우선 해결하시기 바랍니다.

slz6k and others added 30 commits March 12, 2024 11:19

Merge pull request #326 from Peeroreum/feat/#318

f0ceab1

Feat/#318

delete unnecessary import

ac40927

feat: 답변 채택 확인 팝업 (#330)

5b87481

Merge pull request #331 from Peeroreum/feat/#330

7101e9e

Feat/#330 답변 채택 확인 팝업

feat: 같이해냄 달력화면에서 신고하기 화면 이동시 발생하는 에러 timeout 제거로 해결, 신고 완료후 잔여 다이얼로그…

d698421

… pop(#333)

feat: 같이해냄 상세 화면 신고, 삭제 바뀌어있는 거 다시 고침(#333)

754ee1c

feat: 내가해냄 비활성화된 채택하기 버튼 질문 작성자에게만 보이도록 수정(#333)

32b442c

Merge pull request #334 from Peeroreum/feat/#333

e0458c9

Feat/#333

core: update version (#332)

33b864e

fix: iOS Impeller 렌더링 에러

c1c1d31

feat: 마이페이지 메뉴 선택 시 효과 수정 (#332)

639495f

Merge pull request #335 from Peeroreum/feat/#332

218b4a8

Feat/#332

feat: 체크리스트 버튼 누르기만 해도 토스트 메세지 작동 수정(#336)

8c24781

feat: 내가해냄 질문/토스트메세지 마침표, 시간 전 띄어쓰기 추가 (#338)

1a6736f

feat: 신고하기 신고사유 높이 수정 (#338)

460dde5

feat: 상세 이미지 보기 화면 appbar 삭제 x 버튼 좌측으로, 페이지 인덱스 추가(#336)

ccf7456

feat: 학년, 과목 카테고리에서 전체 항목 제외 (#337)

526eb72

feat: 버튼 클릭 이벤트 색상 수정 (#338)

3a5ba1d

feat: 화이트보드 다이얼로그 변경(#340)

b41f3f0

Merge pull request #339 from Peeroreum/feat/#336

8f6d445

Feat/#336

Merge pull request #341 from Peeroreum/feat/#340

c3cd32c

feat: 화이트보드 다이얼로그 변경(#340)

feat: 버튼 선택 스플래시 색 없애기 (#337)

ceff295

design: 뒤로가기 아이콘 변경

969d8cf

feat: 내가해냄 질문결과 오름이 중앙에 배치

e61c9cc

feat: 같이해냄, 내가해냄 검색기록 터치범위 수정

2508304

Merge branch 'dev' into feat/#338

d9d5b5a

Merge pull request #342 from Peeroreum/feat/#338

586f8f8

Feat/#338

feat: 질문 작성 나가기 확인 팝업 (#337)

9885f4d

Merge pull request #344 from Peeroreum/feat/#337

ab6743c

Feat/#337

feat: 챌린지 0일차 아이콘 및 문구 변경 (#345)

720e7c9

slz6k and others added 28 commits April 29, 2026 15:49

chore: Firebase/FCM 초기 설정

60510a2

chore: app_links, kakao_flutter_sdk 의존성 추가

38c345f

feat: 카카오 딥링크 네이티브 설정 (Android MethodChannel, iOS URL Scheme)

63c7b0d

refactor: 같이방 참여 바텀시트를 WeduRoomInfoSheet 공용 위젯으로 추출

5d1e0c1

feat: 카카오 딥링크 → 같이방 참여 바텀시트 플로우 구현

ea36c46

Merge pull request #442 from Peeroreum/feat/#217

09f4fa4

Feat/#217

chore: 빌드번호 업데이트

a7808d8

feat: 카카오톡 공유 커스텀 템플릿 적용 - 프로필

b5db580

- shareDefault → shareCustom (templateId: 102993) - templateArgs: UserName - 딥링크: 카카오 콘솔 템플릿에서 nickname=${UserName} 파라미터로 처리

chore: 빌드 번호 수정

badc247

chore: 빌드 번호 업데이트

f5e6b7e

chore: 빌드 번호 업데이트

d80f441

fix: iOS 공유하기 동작 오류

b6e1ad4

fix: 대학 라벨 추가

88d8d1b

chore: 빌드 번호 업데이트

2de0082

refactor: 소셜 로그인 요청 GET → POST 변경

0f9fcfd

- fetchSocialLogin: http.get → http.post, email을 query param 대신 JSON body로 전송

fix: 소셜 로그인 404 처리 및 갤러리/카메라 취소 시 빈 요청 방지

0d1c84e

refactor: remove unnecessary braces

b0ba545

Merge pull request #444 from Peeroreum/refactor/#443

e5ac96e

Refactor/#443

slz6k merged commit 3c47bba into main May 15, 2026
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

dev#445

dev#445
slz6k merged 877 commits into
mainfrom
dev

slz6k commented May 15, 2026

Uh oh!

Uh oh!

github-actions Bot commented May 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

slz6k commented May 15, 2026

Uh oh!

Uh oh!

github-actions Bot commented May 15, 2026

🤖 Claude Code Review

🤖 Claude Code Review

🔴 버그/보안 이슈

🟡 개선 권장 사항

🟢 잘된 점

우선순위 요약 테이블

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants