Skip to content

fix(docs): S-DOCS3-FIX page embed attrs 재로딩 시 유실 (parseHTML round-trip)#248

Merged
AngryJay91 merged 1 commit intomainfrom
feat/SID-E036-DOCS3-FIX-page-embed-title-persist
Apr 15, 2026
Merged

fix(docs): S-DOCS3-FIX page embed attrs 재로딩 시 유실 (parseHTML round-trip)#248
AngryJay91 merged 1 commit intomainfrom
feat/SID-E036-DOCS3-FIX-page-embed-title-persist

Conversation

@AngryJay91
Copy link
Copy Markdown
Contributor

RC

addAttributes에 per-attribute parseHTML 미지정 → markdown 형식 doc에서 turndown의 pageEmbed rule이 data-doc-id만 보존하고 docid attr은 제거. 재로딩 시 Tiptap 기본 parseHTMLdocid를 못 찾아 null → picker 초기 상태 복귀.

HTML 형식 doc은 docid attr이 raw HTML에 보존되어 정상 작동. markdown 형식 doc만 영향받는 경로.

변경 내용

addAttributes per-attribute parseHTML + renderHTML 추가

  • parseHTML: data-doc-id (markdown 경로) || docid (HTML 경로 하위호환) fallback
  • renderHTML: 각 attr을 data-* HTML 속성으로 일관 직렬화
  • 노드 레벨 renderHTML 단순화 (data-page-embed 추가만 담당)

AC 체크리스트

  • addAttributes per-attr parseHTMLdata-doc-id / docid fallback
  • addAttributes per-attr renderHTMLdata-doc-id 등 data-* 일관 직렬화
  • 노드 레벨 renderHTML 단순화
  • parseHTML round-trip 10케이스 테스트 신설 (markdown 경로, HTML 경로, null 케이스)
  • pnpm vitest run: 25/25 pass
  • pnpm type-check: FULL TURBO

🤖 Generated with Claude Code

…d-trip)

**RC:** addAttributes에 per-attribute parseHTML 미지정 → markdown 형식 doc에서
turndown의 pageEmbed rule이 data-doc-id 만 보존하고 legacy docid attr은 제거.
재로딩 시 Tiptap 기본 parseHTML이 docid를 못 찾아 null → picker 초기 상태 복귀.

**수정:**
- addAttributes에 per-attribute parseHTML + renderHTML 추가
  - parseHTML: data-doc-id (markdown 경로) || docid (HTML 경로 하위호환) fallback
  - renderHTML: 각 attr을 data-* HTML 속성으로 일관 직렬화
- 노드 레벨 renderHTML 단순화: HTMLAttributes에 data-page-embed 만 merge
  (per-attr renderHTML이 이미 data-doc-id 등 세팅)

**테스트 보강:** 15 → 25케이스
- parseHTML round-trip describe 신설 (10케이스)
  - markdown 경로 (data-doc-id only): docId/title/icon/slug 전원 복원 검증
  - HTML 경로 (legacy docid attr): 하위호환 검증
  - attr 누락 시 null 반환 검증
  - full round-trip 통합 케이스

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 15, 2026

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

Project Deployment Actions Updated (UTC)
sprintable Ready Ready Preview, Comment Apr 15, 2026 1:18pm

@AngryJay91 AngryJay91 merged commit 196a4e6 into main Apr 15, 2026
3 of 4 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