Skip to content

feat(blog): 최신글 통합 피드에서 특정 시리즈 제외 설정 메커니즘 (#25)#125

Merged
Malloc72P merged 1 commit into
mainfrom
claude/issue-25-exclude-series
Jul 1, 2026
Merged

feat(blog): 최신글 통합 피드에서 특정 시리즈 제외 설정 메커니즘 (#25)#125
Malloc72P merged 1 commit into
mainfrom
claude/issue-25-exclude-series

Conversation

@Malloc72P

Copy link
Copy Markdown
Owner

#25 검토 결과 "제외할 시리즈"가 구체적으로 정해지지 않아, 설정 메커니즘만 구현합니다(사용자 확인 완료). 지금은 빈 배열이라 동작 변화 없음 — 나중에 시리즈 ID만 추가하면 바로 동작합니다.

구현

  • Constants.series.excludedFromLatestIds: string[](빈 배열) 추가
  • findPostsseriesId === 'latest'(즉 /posts/latest 페이지)일 때만 제외 목록을 존중
  • 랜딩 페이지의 시리즈별 루프에도 동일 제외 적용(랜딩의 "최신글" 통합 뷰)

범위 밖(의도적으로 미적용)

  • 개별 시리즈 페이지(/posts/{id}): 제외 목록에 있어도 그 시리즈 자체 페이지는 그대로 보임
  • 태그 페이지·사이트맵·seriesId 없는 전체 글 목록(레이아웃의 prev/next 계산 포함): 영향 없음

seriesId가 없는 호출까지 필터링하면 사이트맵 누락·prev/next 네비게이션 깨짐 등 부작용이 생기므로, "최신글 통합 피드"에만 범위를 좁혔습니다.

검증

  • find-posts.test: 최신글 제외 / 개별 시리즈 미영향 / 전체 목록 미영향 3케이스 추가
  • 단위 84개 / E2E 8개(프로덕션 빌드, 동작 변화 없음 확인) / lint 0 / tsc clean

🤖 Generated with Claude Code

- Constants.series.excludedFromLatestIds(빈 배열)를 find-posts가 존중하도록 구현
- 적용 범위: /posts/latest(seriesId='latest') + 랜딩 페이지 시리즈 루프만.
  개별 시리즈 페이지·태그·사이트맵·전체 글 목록(prev/next 계산 포함)은 영향받지 않음
- 지금은 빈 배열이라 동작 변화 없음. 제외하고 싶은 시리즈가 생기면 ID만 추가하면 즉시 동작
- find-posts.test: 최신글 제외/개별 시리즈 미영향/전체 목록 미영향 3케이스 추가

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@vercel

vercel Bot commented Jul 1, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
blog Ignored Ignored Jul 1, 2026 12:51am

@Malloc72P

Copy link
Copy Markdown
Owner Author

🔍 코드 리뷰 (self-review)

범위가 명확한 설정 메커니즘이라 발견한 미흡점은 크지 않습니다.

검토한 것

  • app/(main)/page.tsx의 랜딩 루프 제외 vs find-posts.tsseriesId==='latest' 제외가 이중 관리 아닌가? — 둘 다 필요합니다. 랜딩은 시리즈별로 개별 findPosts({seriesId: series.id})를 호출해 조합하는 구조라 seriesId==='latest' 분기를 안 타고, /posts/latest는 반대로 seriesId==='latest'로 직접 호출됩니다. 두 경로가 서로 다른 코드 경로라 각각 필터링이 필요합니다(중복 로직 아님, 진입점이 다름).
  • 빈 배열 기본값이 실제로 무해한지 — 테스트(seriesId 없이 호출하면 영향 없음)와 프로덕션 E2E로 확인. 회귀 없음.
  • 네이밍: excludedFromLatestIds가 두 진입점(랜딩+/posts/latest) 모두를 가리키기에 정확한 이름인지 — "latest 통합 피드"라는 개념이 두 곳에서 동일하므로 적절하다고 판단.

수정할 미흡사항은 없습니다.

@Malloc72P Malloc72P merged commit 9c8c657 into main Jul 1, 2026
3 checks 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