feat: 서재 정렬 옵션값 추가

[doyeonk429]

🔗 관련 이슈

• Close [BOOK-380/feat] 내서재 조회 시 최신 업데이트 순으로 정렬 #236

📘 작업 유형

• ✨ Feature (기능 추가)
• 🐞 Bugfix (버그 수정)
• 🔧 Refactor (코드 리팩토링)
• ⚙️ Chore (환경 설정)
• 📝 Docs (문서 작성 및 수정)
• ✅ Test (기능 테스트)
• 🎨 style (코드 스타일 수정)

📙 작업 내역

• 서재 정렬이 변화되는 부분이 없어서 기존 Usecase 및 Repository에 별도로 sortOption 넣어서 호출하지 않았던 상황입니다. 그래서 옵션 값만 추가하고 별도의 수정이 없습니다. 추후 옵션 선택이 가능한 기능이 추가될 경우에 해당 파라미터도 api request 호출시에 전달할 수 있게 처리하면 됩니다.

🧪 테스트 내역

• 브라우저/기기에서 동작 확인
• 엣지 케이스 테스트 완료
• 기존 기능 영향 없음

✅ PR 체크리스트

• 커밋 메시지가 명확합니다
• PR 제목이 컨벤션에 맞습니다
• 관련 이슈 번호를 작성했습니다
• 기능이 정상적으로 작동합니다
• 불필요한 코드를 제거했습니다

Summary by CodeRabbit

• 신규 기능
  • 라이브러리 정렬에 업데이트 날짜 기준 정렬 옵션 추가 (오름차순, 내림차순)

[coderabbitai]

Walkthrough

LibrarySortType 열거형에 최신 업데이트 날짜 기준의 정렬 옵션 두 가지(오름차순, 내림차순)를 추가했습니다.

Changes

Cohort / File(s)	Summary
LibrarySortType 열거형 확장 src/Projects/BKDomain/Sources/VO/LibraryDetails/LibrarySortType.swift	업데이트 날짜 기준 정렬을 위한 두 가지 열거형 케이스 추가: updatedDateAsc("UPDATED_DATE_ASC")와 updatedDateDesc("UPDATED_DATE_DESC")

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• 특이사항: 단순 열거형 케이스 추가로 새로운 로직이나 복잡한 구조 변경이 없음

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Linked Issues Check	⚠️ Warning	연결된 이슈 #236의 주요 목표는 내서재 조회 시 최신 업데이트 순으로 정렬되도록 수정하는 것이며, 구체적인 작업은 "API request 수정"과 "UI 테스트"입니다. 그러나 현재 PR에서는 LibrarySortType에 옵션값만 추가되었을 뿐, API request 수정이나 기본 정렬 동작 변경은 이루어지지 않았습니다. PR 설명에서 sortOption 파라미터가 준비되었으며 미래의 UI 개선을 위한 것임을 명시하고 있으나, 이 PR만으로는 이슈의 코딩 요구사항인 "API request 수정"이 충분히 충족되지 않습니다.	이슈 #236의 핵심 요구사항을 완전히 충족하기 위해서는 UseCase 또는 Repository에서 기본 정렬 옵션을 최신 업데이트 순으로 설정하거나 API request를 수정하여 기본값으로 UPDATED_DATE_DESC를 사용하도록 변경해야 합니다. 현재 PR에서 옵션값이 추가되었다면, 이를 실제 API 요청에 적용하는 변경사항을 함께 포함하거나 명시적으로 별도 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 (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	PR 제목 "feat: 서재 정렬 옵션값 추가"는 changeset의 주요 변경사항을 정확하게 반영합니다. 실제 변경사항은 LibrarySortType 열거형에 updatedDateAsc와 updatedDateDesc 두 가지 새로운 케이스를 추가한 것이며, 제목은 이를 명확하고 간결하게 나타냅니다. 제목은 업데이트 날짜 기준 정렬 옵션이 추가되었다는 핵심 내용을 효과적으로 전달하고 있습니다.
Out of Scope Changes Check	✅ Passed	PR의 모든 변경사항은 LibrarySortType 열거형에 두 가지 새로운 정렬 옵션값(updatedDateAsc, updatedDateDesc)을 추가하는 것으로 제한됩니다. 이는 연결된 이슈 #236의 범위 내에 있으며, 서재 정렬 기능과 직접적으로 관련된 변경사항입니다. 다른 파일 수정이나 예상 범위 밖의 코드 변경은 없습니다.

✨ 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 BOOK-380-feature/#236

---

📜 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 5838da7 and dba0707.

📒 Files selected for processing (1)

• src/Projects/BKDomain/Sources/VO/LibraryDetails/LibrarySortType.swift (1 hunks)

⏰ 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 (1)

src/Projects/BKDomain/Sources/VO/LibraryDetails/LibrarySortType.swift (1)

10-11: 새로운 정렬 옵션의 API 명세 확인 및 기본값 설정 필요

새로운 enum 케이스는 올바르게 추가되었으나, 다음 사항들을 반드시 확인하고 처리해주세요:

1. 백엔드 API 명세 확인: "UPDATED_DATE_ASC"와 "UPDATED_DATE_DESC" 값이 실제로 백엔드 API에서 지원하는지 확인 필요
2. 이슈 [BOOK-380/feat] 내서재 조회 시 최신 업데이트 순으로 정렬 #236 요구사항 미충족: 현재 코드에서는 기본 정렬값이 설정되어 있지 않습니다. DefaultFetchMyLibraryUseCase와 DefaultMyLibrarySearchBookUseCase에서 MyLibraryParameters를 생성할 때 sortType: .updatedDateDesc 설정을 추가하여 기본값을 구성해주세요.

---

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.}