refactor(converter): FC의 연속 공백 정규화를 제거합니다#980
Merged
Conversation
HTML과 Markdown 모두 연속 공백을 렌더링 시 단일 공백으로 표시하므로, FC에서 `re.sub(r'\s+', ' ', text)` 정규화는 불필요합니다. 이 정규화는 XHTML 원본의 공백 정보를 손실시켜, reverse-sync 파이프라인에서 `collapse_ws` 보상 로직과 gap 공백 축소 등 불필요한 복잡도를 유발합니다. - `navigable_string_as_markdown()`에서 연속 공백 정규화 제거 - 7개 테스트케이스의 expected.mdx 및 sidecar 업데이트 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
FC의 연속 공백 정규화(`re.sub(r'\s+', ' ')`) 제거 후 convert_all.py 재실행으로 XHTML 원본의 이중 공백이 MDX에 그대로 보존됩니다. - 52개 한국어 MDX 파일의 공백 변경 (99줄) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
FC 공백 정규화 제거로 MDX에 이중 공백이 보존되어 렌더링 결과도 변경됩니다. 7개 테스트케이스의 expected.html을 재생성합니다. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
jk-kim0
added a commit
that referenced
this pull request
Apr 4, 2026
FC가 이중 공백을 보존하므로 original.mdx, improved.mdx의
`{{..}}` 뒤 공백을 1→2로 맞춥니다.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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
1 participant
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.
Summary
navigable_string_as_markdown()에서re.sub(r'\s+', ' ', text)연속 공백 정규화를 제거합니다collapse_ws보상 로직과 gap 공백 축소 등 불필요한 복잡도를 유발합니다 (PR [debate: failed] fix(reverse_sync): code span 뒤 공백이 소실되는 문제를 수정합니다 #979 참조)변경 내용
bin/converter/context.py: 연속 공백 정규화 2줄 제거expected.mdx및expected.roundtrip.json업데이트이력
이 정규화는 2025-07-25 PR #34 (
a0da79dd)에서as_markdown()함수 추출 시 방어적으로 추가된 것으로, 특정 버그 수정이 아닌 HTML 텍스트 노드의 일반적인 공백 정규화 목적이었습니다.Test plan
🤖 Generated with Claude Code