feat(blog): 커스텀 404 + 추천 글 자동화 + 글 공유 버튼 (#93, #89, #92)#116
Merged
Conversation
#93 커스텀 404 - app/not-found.tsx: MainClientLayout 재사용으로 헤더/푸터/검색 확보, 홈·전체 태그·최근 글 복구 동선 제공 - e2e/not-found: 헤더와 복구 링크 단언 추가 #89 추천 글 자동화 - libs/recommend-posts.ts: 태그 자카드 + 같은 시리즈 가중치로 상위 N 산출(현재 글 제외) - PostRecommendation이 현재 글을 받아 동적 추천(하드코딩 postIds 제거) - 죽은 Constants.recommendation 및 전용 테스트 제거(동적 추천으로 대체) - recommend-posts.test: 제외/순위/시리즈 가중치/limit/동점 커버 #92 글 공유 버튼 - share-buttons.tsx: 링크 복사 + X(트위터) 공유 + 모바일 navigator.share - post 하단(footer)에 canonical URL로 배치 - share-buttons.test: 복사/인텐트 URL/네이티브 공유 노출 조건 커버 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Owner
Author
🔍 코드 리뷰 (self-review)세 기능 모두 의도대로 동작하고 단위 21스위트/81개 + E2E 7개가 통과합니다. 반영할 미흡사항을 정리합니다. 1. (보통) 레이아웃 데이터 준비 로직 중복 — DRY
2. (낮음/의도적 차이) "인기글" → "최근 글"이슈 #93은 "인기글 링크"를 제안했지만, 조회수 등 인기도 데이터가 없어 최신 글 4개를 "최근 글"로 정직하게 표기했습니다. 인기도 집계가 생기기 전까지는 이 절충이 타당하다고 판단합니다(필요 시 후속). 3. (참고/미반영) 추천 sparse 케이스태그·시리즈 접점이 적은 글은 추천이 1~2개만 나올 수 있습니다(0개면 섹션 숨김). 무관한 추천을 넣는 것보다 적게 보여주는 편이 낫다고 보고 fallback은 두지 않았습니다. 향후 "최근 글로 채우기"는 별도 검토. 1번을 반영한 뒤 별도 코멘트로 정리하겠습니다. |
- libs/api/load-layout-data.ts: 시리즈/태그/글 준비 로직을 단일 출처로 추출 - (main)/layout, not-found가 공용 로더 사용(중복 findX/Mapper 제거) 레이아웃은 prev/next 계산만, not-found는 최근 글 정렬만 별도로 수행 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Owner
Author
✅ 리뷰 반영 완료 (
|
This was referenced Jul 1, 2026
Closed
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
세 개의 UX/기능 이슈를 한 PR로 묶어 처리합니다.
#93 커스텀 404 페이지
app/not-found.tsx신설.MainClientLayout을 재사용해 헤더(검색 포함)·푸터를 그대로 확보.#89 추천 글 자동화 (태그/시리즈 유사도)
libs/recommend-posts.ts: 태그 자카드 유사도 + 같은 시리즈 가중치로 상위 N 산출, 현재 글 제외, 접점 없는 글 제외, 동점은 최신 우선.PostRecommendation이 현재 글을 받아 동적 추천으로 전환(기존 하드코딩Constants.recommendation.postIds제거).constants.test.ts)를 정리하고, 동적 로직 단위 테스트로 대체.#92 글 공유 버튼
share-buttons.tsx: 링크 복사 / X(트위터) 공유 / 모바일navigator.share.siteConfig.url + route) 사용.테스트
recommend-posts(6),share-buttons(4) 추가 — 총 21 스위트 / 81개 통과CI=true프로덕션 빌드 양쪽 검증,lint0 /tscclean🤖 Generated with Claude Code