Skip to content

HTML 태그 포함 마크다운 콘텐츠의 줄바꿈 소실 버그 수정#654

Merged
HA-SEUNG-JEONG merged 2 commits into
developfrom
fix/markdown
May 20, 2026
Merged

HTML 태그 포함 마크다운 콘텐츠의 줄바꿈 소실 버그 수정#654
HA-SEUNG-JEONG merged 2 commits into
developfrom
fix/markdown

Conversation

@HA-SEUNG-JEONG

@HA-SEUNG-JEONG HA-SEUNG-JEONG commented May 20, 2026

Copy link
Copy Markdown
Contributor

🌱 연관된 이슈

☘️ 작업 내용

🍀 참고사항

🎨 디자인 비교 (UI 컴포넌트 PR에만 작성)

Figma 원본 Storybook 구현
- -
비교 결과
항목 상태 비고
색상
간격
타이포그래피
반경
트랜스폼

스크린샷 (선택)

Summary by CodeRabbit

릴리즈 노트

  • Refactor
    • 마크다운 콘텐츠 처리 로직을 개선하여, YouTube 링크 임베드 처리가 일관되게 적용되도록 변경되었습니다. 콘텐츠 보안 및 형식 표준화 절차가 강화되었습니다.

Review Change Stack

@HA-SEUNG-JEONG HA-SEUNG-JEONG self-assigned this May 20, 2026
@vercel

vercel Bot commented May 20, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
study-platform-client-dev Ready Ready Preview, Comment May 20, 2026 1:28pm

@coderabbitai

coderabbitai Bot commented May 20, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@HA-SEUNG-JEONG has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 20 minutes and 33 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: fad2732a-45a4-4454-ad02-1a6ac5d2ac91

📥 Commits

Reviewing files that changed from the base of the PR and between ae34165 and 23192b9.

📒 Files selected for processing (1)
  • src/components/common/ui/rich-text/markdown-content-core.tsx
📝 Walkthrough

Walkthrough

MarkdownContentCore가 입력 콘텐츠의 형식(HTML/마크다운) 판별을 제거하고 항상 동일한 파이프라인(marked.parsereplaceStandaloneYouTubeLinksWithEmbeds → DOMPurify sanitize)을 거치도록 리팩토링했습니다.

Changes

통합 마크다운 파싱 파이프라인

Layer / File(s) Summary
조건부 파싱 제거 및 통합 파이프라인 적용
src/components/common/ui/rich-text/markdown-content-core.tsx
isHtmlContent import 제거 및 marked.parse 호출을 조건부 로직에서 무조건 호출로 변경하여, 콘텐츠 형식 구분 없이 YouTube 링크 치환, 마크다운 파싱, sanitize, 후처리를 순차 적용합니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested labels

release:patch

Poem

🐰 이제 구분 없이 모두 한길로,
HTML과 마크다운 함께 흐르네.
조건은 사라지고 파이프라인 하나,
YouTube는 반짝이는 프레임으로,
깔끔한 렌더링의 새 시작! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 마크다운 콘텐츠의 줄바꿈 소실 버그 수정이라는 주요 변경사항을 명확하게 설명하고 있으며, 변경 내용과 직접적으로 관련되어 있습니다.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/markdown

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

Comment @coderabbitai help to get the list of available commands and usage tips.

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