[FEAT] 기준정보 정렬

[yangjiseonn]

🙏 Question & PR point

• 기준정보 정렬

📬 Reference

Summary by CodeRabbit

개선 사항

• 검색 기능 강화

  • BOM 코드로 검색 가능
  • 자재 및 부품 검색 시 대소문자 구분 제거

• 정렬 순서 변경

  • BOM, 자재, 부품, 공정 목록을 코드 기준으로 오름차순 정렬하도록 개선
  • 검색 결과의 일관된 정렬 제공

[coderabbitai]

Walkthrough

여러 서비스의 검색 및 정렬 로직이 수정되었습니다. BOM, Part, Material, Process 서비스는 생성 시간 기반 정렬에서 코드 기반 오름차순 정렬로 변경되었고, 검색 기능에는 대소문자 무시 조건이 추가되었습니다. ItemService는 부품과 자재를 별도로 가져온 후 병합하는 방식으로 변경되었습니다.

Changes

Cohort / File(s)	변경 사항
BOM 검색 로직 src/main/java/com/sampoom/backend/api/bom/repository/BomRepository.java, src/main/java/com/sampoom/backend/api/bom/service/BomService.java	bomCode로 필터링하는 검색 조건 추가, 정렬 순서 변경 (createdAt DESC → bomCode ASC)
코드 기반 정렬 및 대소문자 무시 검색 src/main/java/com/sampoom/backend/api/part/service/PartService.java, src/main/java/com/sampoom/backend/api/material/service/MaterialService.java, src/main/java/com/sampoom/backend/api/process/service/ProcessService.java	정렬 기준을 코드 필드 오름차순으로 변경, 검색 시 대소문자 무시 적용
ItemService 통합 검색 전략 변경 src/main/java/com/sampoom/backend/api/item/service/ItemService.java	부품과 자재를 별도로 두 배 페이지 크기로 가져온 후 병합, 수동 페이지네이션 처리 방식 도입

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• ItemService의 ALL 검색 경로 변경 로직 검토 필요 (별도 페칭 및 병합 방식, totalElements 계산 로직)
• 각 서비스의 정렬 기준 변경이 기존 동작에 미치는 영향 확인
• 대소문자 무시 검색 구현의 데이터베이스 호환성 검증

Possibly related PRs

• [FEAT] BOM 상태, 복잡도 컬럼 추가 / BOM 검색 필터 기능, 부품 수정 API 수정 #26: BOM 검색 경로 수정 (BomRepository.findByFilters, BomService.searchBoms의 쿼리 필터 및 정렬 변경)
• Spm 411 #35: ItemService 검색/병합 로직 및 페이지네이션 수정 (부품과 자재 페칭, 병합, 페이지네이션 방식 변경)
• [FEAT] 공정 목록 검색/상세 기능 확장 (카테고리/그룹 표시, 검색 필터 추가) #36: ProcessService.search 메서드 수정 (정렬 기준 변경)

Suggested labels

ready-to-merge

Suggested reviewers

• Sangyoon98
• CHOOSLA
• Lee-Jong-Jin
• vivivim
• taemin3

Poem

🐰 코드 순서로 정렬하고,
대소문자 구분 없이 찾아내고,
부품과 자재 갈라 가져와
슬기롭게 하나로 모으는 마법!
검색도 빠르고 보기도 깔끔해~ ✨

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	PR 설명이 매우 불완전합니다. 필수 섹션인 'Related Issue'와 'Summary'가 누락되어 있고, 'Question & PR point'도 단순히 제목을 반복하고 있습니다.	최소한 'Related Issue' 번호와 'Summary' 섹션을 작성하여 변경사항의 목적, 이유, 영향 범위를 명확히 설명해야 합니다.
Docstring Coverage	⚠️ Warning	Docstring coverage is 62.50% 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
Title check	✅ Passed	PR 제목은 기준정보 정렬이라는 주요 변경사항을 명확하게 요약하고 있으며, 전반적인 변경 범위를 잘 나타냅니다.

✨ 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 SPM-454

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 99ce058 and 3fbd419.

📒 Files selected for processing (6)

• src/main/java/com/sampoom/backend/api/bom/repository/BomRepository.java (1 hunks)
• src/main/java/com/sampoom/backend/api/bom/service/BomService.java (2 hunks)
• src/main/java/com/sampoom/backend/api/item/service/ItemService.java (3 hunks)
• src/main/java/com/sampoom/backend/api/material/service/MaterialService.java (2 hunks)
• src/main/java/com/sampoom/backend/api/part/service/PartService.java (2 hunks)
• src/main/java/com/sampoom/backend/api/process/service/ProcessService.java (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: Analyze (java-kotlin)

---

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

[taemin3]

확인했습니다