[Feat] 제보하기 카테고리 추천루틴 필터링 제거

[choijungp]

🌁 Background

진심 진심 진심으로 하나의 PR로 빼기 민망할 정도로 짧은 ............. 수정사항입니다 ㅠㅠ

기존에는 '제보하기' 기능이 없었기 때문에 추천 루틴 중에 '나가봐요_제보하기' 카테고리는 필터링하여 보여주지 않기로 되어 있었는데요 !!
현재는 제보하기 기능이 추가되었으므로 '나가봐요_제보하기' 카테고리의 추천 루틴들도 보여줄 수 있도록 하였습니다 !!

👩‍💻 Contents

• 나가봐요_제보하기 추천 루틴 보여주기

📝 Review Note

1. 나가봐요_제보하기 추천 루틴 중 '밤 산책하며 노후 가로등 찾아보기'

나가봐요_제보하기 추천 루틴 중 '밤 산책하며 노후 가로등 찾아보기' 루틴이 있는데요 ..
서버 DB에 저장된 루틴이 대부분 동일하지만 description이 다르게 3개가 있어유 ....

그래서 클라에서 볼 때에는 똑같은 루틴 3개가 보이는 현상입니다 !!! 암튼 클라 잘못 아니라는 뜻 !!!
서버에게 말해두었고 아마 PM한테도 전달되었을 듯 .. (전달되었길 🙏)

2. 그 외의 작업들

카톡으로 나눴던 남은 작업들은 [Feat] 제보 완료화면 ViewModel 추가 #74 요기에 작업해두었습니다 !!!
제보하기 작업이라 뭔가 다시 나누기 애매했다랄까요 .. ㅠㅠ

그래서 해당 PR description도 일부 추가해두었습니다 !!!

감사함니다 !!!

Summary by CodeRabbit

릴리스 노트

• 버그 수정
  • 앱 버전 비교 로직이 개선되어 업데이트 확인 알림이 더 정확하게 표시됩니다.
  • 루틴 카테고리 데이터 정규화로 카테고리 처리가 안정화되었습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

개요

두 파일에서 버전 처리 및 카테고리 정규화 로직이 변경되었습니다. RecommendedRoutineDTO에서는 카테고리 매핑 시 .outdoorReport를 .outdoor로 변환하는 단계가 추가되었으며, HomeViewModel에서는 버전 비교 로직이 문자열 기반에서 정수 기반으로 변경되고 비교 연산자가 수정되었습니다.

변경 사항

코호트 / 파일(들)	변경 요약
카테고리 정규화 Projects/DataSource/Sources/DTO/RecommendedRoutineDTO.swift	카테고리 매핑 후 .outdoorReport를 .outdoor로 변환하는 정규화 단계 추가
버전 비교 로직 업데이트 Projects/Presentation/Sources/Home/ViewModel/HomeViewModel.swift	문자열 기반 메이저 버전 파싱을 정수 기반으로 변경하고, 버전 비교 연산자를 부등호(!=)에서 미만(\<)으로 수정하여 App Store 업데이트 트리거 조건 변경

예상 코드 리뷰 소요 시간

🎯 2 (단순) | ⏱️ ~12분

• 주의 필요 영역:
  • HomeViewModel.swift의 버전 비교 로직 변경: 기존 부등호 조건에서 미만 조건으로의 변경이 의도한 동작인지 확인 필요
  • 정수 기반 파싱으로 변경되면서 예상치 못한 부작용이 없는지 검토 필요 (예: 버전 문자열 형식 변화)

시

🐰 카테고리는 깔끔하게,
야외는 야외답게,
버전 체크는 더 똑똑하게,
정수로 비교하니 명확하네!
업데이트 조건도 한층 견고해졌네요~ ✨

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Title check	⚠️ Warning	제목이 PR의 주요 변경사항과 일치하지 않습니다. PR은 제보 카테고리 필터링 제거 외에도 버전 비교 로직 변경이 포함되어 있습니다.	제목을 수정하여 포함된 모든 주요 변경사항을 반영하거나, PR 설명을 더 명확하게 업데이트하세요.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/recommendation-routine-report

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 885a272 and 1abc266.

📒 Files selected for processing (2)

• Projects/DataSource/Sources/DTO/RecommendedRoutineDTO.swift (1 hunks)
• Projects/Presentation/Sources/Home/ViewModel/HomeViewModel.swift (1 hunks)

🧰 Additional context used

🧠 Learnings (1)

📓 Common learnings

Learnt from: choijungp
Repo: YAPP-Github/Bitnagil-iOS PR: 30
File: Projects/DataSource/Sources/NetworkService/Plugin/RefreshTokenPlugin.swift:42-43
Timestamp: 2025-07-30T03:56:18.617Z
Learning: choijungp는 현재 테스트 단계에서 RefreshTokenPlugin의 토큰 갱신 로직 디버깅을 위해 액세스 토큰과 리프레시 토큰의 실제 값을 로그에 기록하는 것을 선호함.

🧬 Code graph analysis (1)

Projects/Presentation/Sources/Home/ViewModel/HomeViewModel.swift (1)

Projects/DataSource/Sources/Repository/AppConfigRepository.swift (1)

• fetchAppVersion (13-21)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: build

🔇 Additional comments (2)

Projects/Presentation/Sources/Home/ViewModel/HomeViewModel.swift (1)

279-286: 버전 비교 로직 개선 승인

버전 비교 로직이 개선되었습니다:

• 변수명 변경 (major → majorVersion): 가독성 향상
• 비교 연산자 변경 (!= → <): 현재 버전이 앱스토어 버전보다 낮을 때만 업데이트 유도

기존의 불일치 비교(!=)는 개발/베타 버전처럼 현재 버전이 더 높은 경우에도 업데이트를 유도했을 수 있습니다. 새로운 로직은 실제로 구버전을 사용하는 경우에만 업데이트를 유도하므로 더 정확합니다.

Projects/DataSource/Sources/DTO/RecommendedRoutineDTO.swift (1)

33-35: 카테고리 정규화 로직 검증 완료

.outdoorReport를 .outdoor로 변환하는 정규화 로직은 의도된 설계입니다.

확인 결과:

• RoutineCategoryType 열거형에 .outdoorReport("OUTING_REPORT")와 .outdoor("OUTING") 모두 정의됨
• RoutineCategoryView에서 .outdoorReport를 명시적으로 필터링하여 사용자가 선택할 수 없도록 제한
• 두 카테고리가 동일한 아이콘과 배경색 사용 (.outdoor와 .outdoorReport 같은 처리)

의도된 동작: DTO 레이어에서 서버의 outdoorReport 루틴을 일반 outdoor로 정규화하여, 사용자 인터페이스에서는 구분 없이 일관되게 표시하되, 백엔드에서는 정보 추적 가능하게 함. 현재 구현이 올바릅니다.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}