Skip to content

fix: 아티클 목록 정렬 기준을 게재일(publishedAt) DESC로 변경#114

Merged
yongseong123 merged 1 commit into
mainfrom
fix/article-sort-by-publishedAt
Jun 24, 2026
Merged

fix: 아티클 목록 정렬 기준을 게재일(publishedAt) DESC로 변경#114
yongseong123 merged 1 commit into
mainfrom
fix/article-sort-by-publishedAt

Conversation

@yongseong123

Copy link
Copy Markdown
Collaborator

Pull Request

작업 개요

  • 커뮤니티 아티클 목록이 게재일 오름차순으로 보이던 문제를 수정합니다.
  • 정렬 기준을 updatedAt에서 publishedAt(게재일)으로 변경하여 최신 아티클이 상단에 표시되도록 합니다.

변경 사항

  • TechArticleService.toSortedPageable(): 정렬 기준을 updatedAt DESCpublishedAt DESC 로 변경

관련 이슈

  • Closes #

원인 분석

아티클은 RSS 배치 크롤러로 수집되며, 같은 배치에서 수집된 아티클들은 updatedAt 값이 동일합니다.
이로 인해 updatedAt DESC 정렬 시 같은 날 수집된 아티클들의 순서가 보장되지 않아, 실제 게재일과 무관하게 오래된 아티클이 상위에 노출되는 현상이 발생했습니다.

publishedAt DESC로 변경하면 원문 블로그 게재일 기준으로 최신 아티클이 먼저 표시됩니다.
TechArticle 엔티티에 idx_tech_article_source_published_at 인덱스가 이미 존재하므로 성능 영향도 없습니다.

테스트

  • 로컬에서 기능 동작을 확인했습니다.
  • 빌드 또는 테스트가 통과했습니다.
  • 변경된 API의 성공, 실패 케이스를 확인했습니다.
  • 요청 값 검증과 예외 처리를 확인했습니다.

체크리스트

  • 불필요한 로그, 주석, 디버깅 코드를 제거했습니다.
  • API 명세 변경 사항을 문서화했거나 공유했습니다.
  • 인증, 인가, 입력 검증 영향을 확인했습니다.
  • DB 스키마, 마이그레이션, 초기 데이터 변경 여부를 확인했습니다.
  • 환경 변수나 배포 설정 변경이 있다면 문서화했습니다.

API 변경 사항

  • 변경된 엔드포인트: GET /api/articles
  • 요청 예시: GET /api/articles?page=0&size=5
  • 응답 예시: 동일 (정렬 순서만 변경됨 — publishedAt 내림차순)
  • 에러 응답: 변경 없음

리뷰 포인트

  • toSortedPageable() 1줄 변경으로, 리포지토리 메서드명(OrderByUpdatedAtDesc)과 실제 정렬 기준이 달라지는 점에 주목해주세요. 메서드명은 날짜 범위 필터 조건을 표현하는 역할이므로 기능 동작에 영향 없습니다. (Spring Data JPA에서 Pageable 정렬이 메서드명 정렬보다 우선 적용됩니다.)

배치 수집된 아티클은 updatedAt이 동일하여 게재일 순서가 보장되지 않았음.
publishedAt DESC 정렬로 최신 게재 아티클이 상단에 표시되도록 수정.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@yongseong123
yongseong123 merged commit 3856983 into main Jun 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant