[refactor] 검색 범위를 동아리명, 분과태그, 지원정보태그로 변경한다 #930
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning
|
| Cohort / File(s) | Change Summary |
|---|---|
검색 조건 변경 backend/src/main/java/moadong/club/repository/ClubSearchRepository.java |
searchClubsByKeyword 메서드의 키워드 매칭 집합 수정: category 필드에 대한 정규식(case-insensitive) 매칭 추가, recruitmentInformation.introduction 및 recruitmentInformation.description 매칭 제거. recruitmentInformation.tags 매칭은 유지. |
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~10 minutes
- 주의 검토 포인트:
- 정규식 패턴 및 case-insensitive 옵션 확인
- 기존 aggregation/단계와의 의도된 동작(순서, 필터링 영향) 검증
- 테스트 케이스(검색 결과)의 업데이트 필요 여부 확인
Possibly related issues
- [refactor] MOA-422 검색 범위를 동아리명, 분과태그로 변경한다 #929: 검색 범위 변경 요구사항 (동아리명, 분과태그 기반으로 조정)
Suggested reviewers
- yw6938
- Zepelown
- lepitaaar
Pre-merge checks and finishing touches
❌ Failed checks (1 warning)
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Docstring Coverage | 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 (4 passed)
| Check name | Status | Explanation |
|---|---|---|
| Title check | ✅ Passed | PR 제목은 검색 범위를 동아리명, 분과태그, 지원정보태그로 변경한다는 주요 변경 내용을 명확하게 요약하고 있습니다. |
| Linked Issues check | ✅ Passed | 코드 변경사항이 MOA-422 이슈의 목표(검색 범위를 동아리명과 분과 태그(category)로 변경, 지원정보 태그 유지)를 완벽히 충족합니다. |
| Out of Scope Changes check | ✅ Passed | 변경사항이 ClubSearchRepository.java의 검색 키워드 매칭 로직 수정으로 제한되어 있으며, 모두 MOA-422 이슈의 범위 내 변경입니다. |
| 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
refactor/#929-search-range-MOA-422
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.
Comment @coderabbitai help to get the list of available commands and usage tips.
Test Results75 tests 72 ✅ 19s ⏱️ Results for commit 9a4ec5d. ♻️ This comment has been updated with latest results. |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
Disabled knowledge base sources:
- Jira integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
backend/src/main/java/moadong/club/repository/ClubSearchRepository.java(1 hunks)
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: Due-IT
Repo: Moadong/moadong PR: 329
File: backend/src/main/java/moadong/club/controller/ClubSearchController.java:0-0
Timestamp: 2025-05-01T08:32:09.398Z
Learning: ClubSearchController의 searchClubsByKeyword 메서드와 ClubSearchService의 searchClubsByKeyword 메서드 사이에 파라미터 순서 일치가 중요하다. 현재 컨트롤러의 매개변수 순서는 'keyword, recruitmentStatus, category, division'인 반면, 서비스 메서드의 순서는 'keyword, recruitmentStatus, division, category'이다. 컨트롤러에서 서비스 호출 시 'keyword, recruitmentStatus, division, category' 순서로 전달하면 category와 division 값이 바뀌어 검색 결과가 잘못될 수 있다.
📚 Learning: 2025-05-01T08:32:09.398Z
Learnt from: Due-IT
Repo: Moadong/moadong PR: 329
File: backend/src/main/java/moadong/club/controller/ClubSearchController.java:0-0
Timestamp: 2025-05-01T08:32:09.398Z
Learning: ClubSearchController의 searchClubsByKeyword 메서드와 ClubSearchService의 searchClubsByKeyword 메서드 사이에 파라미터 순서 일치가 중요하다. 현재 컨트롤러의 매개변수 순서는 'keyword, recruitmentStatus, category, division'인 반면, 서비스 메서드의 순서는 'keyword, recruitmentStatus, division, category'이다. 컨트롤러에서 서비스 호출 시 'keyword, recruitmentStatus, division, category' 순서로 전달하면 category와 division 값이 바뀌어 검색 결과가 잘못될 수 있다.
Applied to files:
backend/src/main/java/moadong/club/repository/ClubSearchRepository.java
📚 Learning: 2025-08-25T14:43:52.320Z
Learnt from: lepitaaar
Repo: Moadong/moadong PR: 703
File: backend/src/main/java/moadong/club/controller/ClubApplyController.java:84-84
Timestamp: 2025-08-25T14:43:52.320Z
Learning: In the Moadong codebase, questionId and clubId are equivalent identifiers that represent the same entity. The ClubApplicationRepository.findAllByIdInAndQuestionId method correctly uses clubId as the questionId parameter for filtering club applications.
Applied to files:
backend/src/main/java/moadong/club/repository/ClubSearchRepository.java
Zepelown
left a comment
Zepelown
left a comment
There was a problem hiding this comment.
혹시 어쩌다가 검색 범위를 변경하게 되었는지 알 수 있을까요?
소개글과 상세설명에 실제 동아리와 무관한 키워드가 포함될 경우, 이로 인해 검색 범위를 동아리명 + 분과 태그로 제한하도록 개선하게 되었습니다. |
lepitaaar
left a comment
lepitaaar
left a comment
There was a problem hiding this comment.
검색 범위 변경하신거 좋네요 기존에 너무 과한범위로 검색했던거같긴 합니다. 수고하셨어요~~
alsdddk
force-pushed
the
refactor/#929-search-range-MOA-422
branch
from
c4173b4 to
9a4ec5d
Compare
#️⃣연관된 이슈
#929
📝작업 내용
작업 배경
소개글과 상세설명에 실제 동아리와 무관한 키워드가 포함될 경우,
검색 결과에 관련 없는 동아리까지 무분별하게 노출되는 문제가 있었습니다.
이로 인해 검색 범위를 아래와 같이 개선하게 되었습니다.
기존 검색범위
변경된 검색범위
중점적으로 리뷰받고 싶은 부분(선택)
논의하고 싶은 부분(선택)
🫡 참고사항
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.