Skip to content

[#202] refactor: isomorphic-dompurify로 마이그레이션#222

Merged
yujeong-jeon merged 1 commit into
mainfrom
feature/202
May 11, 2026
Merged

[#202] refactor: isomorphic-dompurify로 마이그레이션#222
yujeong-jeon merged 1 commit into
mainfrom
feature/202

Conversation

@yujeong-jeon

@yujeong-jeon yujeong-jeon commented May 11, 2026

Copy link
Copy Markdown
Contributor

Related Issue

Describe your changes

업스트림 메모리 누수 이슈(kkomelin/isomorphic-dompurify#368)를 우회하기 위해 도입했던 커스텀 DOMPurify 래퍼(jsdom/happy-dom/linkedom 중 선택 지원, LRU 캐시, recreate interval 등)를 제거하고, 누수가 해결된 isomorphic-dompurify를 직접 사용하도록 단순화합니다.

Breaking Changes

  • configureDOMPurify 함수 export 제거
  • SafeParseOptions.domPurifyOptions 옵션 제거
  • DOMWindow, DOMWindowFactory, DOMPurifyOptions 타입 export 제거
  • jsdom / happy-dom / linkedom 피어 의존성 제거 (별도 설치 불필요)

Request

@yujeong-jeon yujeong-jeon requested a review from a team as a code owner May 11, 2026 03:09

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Code review skipped — your organization's overage spend limit has been reached.

Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.

Once credits are available, push a new commit or reopen this pull request to trigger a review.

@yujeong-jeon yujeong-jeon self-assigned this May 11, 2026
@npayfebot

Copy link
Copy Markdown
Collaborator

✅ Changeset detected

Latest commit: 7b7de6c

@naverpay/safe-html-react-parser package have detected changes.

If no version change is needed, please add skip-detect-change to the label.

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@naverpay/safe-html-react-parser 💥 Major
powered by: naverpay changeset detect-add actions

@npayfebot

Copy link
Copy Markdown
Collaborator

NPM Packages

📦 @naverpay/safe-html-react-parser

Total Sizes: 1.93 kB

Total Changes: -2.63 kB (-58%) ✅ (Size Decreased)

File Status Previous Size Updated Size Changed
/dist/esm/index.mjs 🛠️ 712 B 683 B -29 B (-4%)
/dist/esm/utils/lru-cache.mjs 🔥 303 B - -303 B
/dist/esm/utils/dompurify.mjs 🛠️ 1.09 kB 132 B -954 B (-88%)
/dist/cjs/index.js 🛠️ 947 B 920 B -27 B (-3%)
/dist/cjs/utils/lru-cache.js 🔥 366 B - -366 B
/dist/cjs/utils/dompurify.js 🛠️ 1.15 kB 197 B -955 B (-83%)

🧩 Dependency Changes

Package Status Previous Version Updated Version Bundle Size (Max) Changed
isomorphic-dompurify@^3.12.0 - ^3.12.0 25.4 kB 25.4 kB
dompurify@^3.3.0 ^3.3.0 - -830 kB -830 kB

powered by: naverpay size-action

@yujeong-jeon yujeong-jeon merged commit 91e0f73 into main May 11, 2026
7 checks passed
@yujeong-jeon yujeong-jeon deleted the feature/202 branch May 11, 2026 03:15
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.

2 participants