Conversation
FEAT: 아티클 메타데이터 저장 기능 구현
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThis PR introduces article metadata extraction and storage functionality. It adds JSoup-based HTML parsing to extract article titles and thumbnails from URLs, stores these in the Article domain model, uploads thumbnails to S3, and provides async batch migration for enriching existing articles with metadata. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant Controller as ArticleControllerV3
participant Usecase as CreateArticleUsecase
participant Adapter as ArticleMetadataAdapter
participant S3 as S3StorageService
participant Repo as ArticleRepository
participant DB as Database
Client->>Controller: POST /api/v3/articles
Controller->>Usecase: createArticleV3(user, command)
Usecase->>Adapter: extractMetadata(url)
Adapter->>Adapter: parseHTML with Jsoup
Adapter->>Adapter: extractTitle & ogImage
alt Image exists
Adapter->>S3: uploadArticleThumbnail(image)
S3->>S3: hashUrl, checkExists
S3->>Repo: upload to S3
S3-->>Adapter: thumbnail S3 URL
else No image
Adapter->>Adapter: use DEFAULT_THUMBNAIL
end
Adapter-->>Usecase: ArticleMetadataResponse
Usecase->>Repo: save Article with metadata
Repo->>DB: INSERT article
DB-->>Repo: success
Repo-->>Usecase: saved Article
Usecase-->>Controller: success
Controller-->>Client: ResponseDto<Void>
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Possibly related issues
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
✨ Finishing touches
🧪 Generate unit tests (beta)
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 |
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.
🚀 PR 요약
목적이 무엇인가요? - 지우고 작성
✨ PR 상세 내용
어떤 부분이 어떻게 변경이 되었나요? - 지우고 작성
🚨 주의 사항
주의할 부분이 무엇인가요? - 지우고 작성
✅ 체크 리스트
Summary by CodeRabbit
Release Notes