confluence-mdx: reverse-sync 설계 문서를 4차 갱신합니다 (기준선 교정 및 Phase 5 계획 재정의)#938
Merged
confluence-mdx: reverse-sync 설계 문서를 4차 갱신합니다 (기준선 교정 및 Phase 5 계획 재정의)#938
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
jk-kim0
force-pushed
the
jk/docs-reverse-sync-phase5-plan
branch
from
66d63dc to
756ffec
Compare
jk-kim0
changed the title
1 task
리뷰 의견을 반영하여 설계 기준선과 Phase 5 계획을 수정합니다. - pages.yaml 실제 수치로 교정: pass 29 → 31, fail 13 → 11 - 런타임 경로 다이어그램에 transfer_text_changes() 3개 호출 지점 명시 - §4.1: text_transfer 잔존 경로를 3개 지점으로 구체화하고 PR #942 현황 반영 - §4.2: Axis 2 범위를 CLI record_mapping() 중복 제거로 한정 (build_patches 시그니처의 mappings/xpath_to_mapping 전체 제거는 대체 경로 확정 후 별도 진행) - §6 Axis 1: 목표를 'text_transfer 호출 지점 전수 제거'로 재정의 (routing 변경 + 3개 fallback 제거), 완료 기준에 pass 31, transfer_text_changes 0개 추가 - §6 게이트 표: unsupported skip vs fail-closed 구분 명확화 - §8: text_transfer.py 삭제 조건을 '3개 지점 전부 제거 후'로 수정 - §10: 4차 갱신 기준 상태로 업데이트, fail failure_type 표 11건으로 교정 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Axis 1: clean container 전환 시 reconstruct_container_fragment()가 outer wrapper를 "이미 보존함"이라는 잘못된 전제를 교정하고, _apply_outer_wrapper_template() 확장 방안 추가 - Axis 2: mappings=[] 전달이라는 모호한 표현을 "내부 자동 구축"으로 명확화하고, mapping.original.yaml artifact 처분 방침 신규 섹션 추가 Co-Authored-By: Atlas <atlas@jk.agent>
리뷰 피드백 반영은 같은 PR 내 커밋이므로 갱신 차수를 올릴 이유가 없습니다. Co-Authored-By: Atlas <atlas@jk.agent>
- SidecarBlock에 BlockMapping 필드(block_id, xhtml_plain_text 등)가 없으므로 roundtrip_sidecar만으로 mappings 복원이 불가함을 명시 - 데이터 소스를 "build_patches() 내부 record_mapping() 호출"로 확정 - 전환 순서를 구체화: page_xhtml 파라미터 추가 → CLI 중복 호출 제거 - mapping.original.yaml artifact는 build_patches() 반환 타입 확장으로 유지 Co-Authored-By: Atlas <atlas@jk.agent>
jk-kim0
changed the title
build_patches() 내부에서 record_mapping()을 호출해도 build_sidecar()의 별도 호출은 남으므로 runtime 중복이 실제로 제거되지 않습니다. Axis 2의 실제 목표는 CLI의 배선 책임을 build_patches() 내부로 이전하는 것이므로 문서 전반의 목표 문구를 이에 맞춰 교정합니다. Co-Authored-By: Atlas <atlas@jk.agent>
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
docs/plans/2026-03-13-reverse-sync-reconstruction-design.md)를 2026-03-19 기준으로 갱신합니다.주요 변경 사항
transfer_text_changes()3개 호출 지점 전수 제거. clean container 전환 시reconstruct_container_fragment()확장 필요(_apply_outer_wrapper_template())record_mapping()책임을build_patches()내부로 캡슐화. 데이터 소스는 내부record_mapping()호출로 확정. runtime 중복 제거(build_sidecar()와의 이중 호출)는 SidecarBlock 스키마 확장 후 별도 단계.mapping.original.yamlartifact는 반환 타입 확장으로 유지text_transfer.py,table_patcher.py및 관련 테스트 삭제리뷰 피드백 반영
reconstruct_container_fragment()가 outer wrapper를 "이미 보존함"이라는 잘못된 전제 교정mappings=[]전달이라는 모호한 표현을 "내부record_mapping()호출"로 명확화SidecarBlock에BlockMapping필드가 없어 sidecar만으로 복원 불가함을 명시, 데이터 소스 확정build_sidecar()의 별도 호출이 남으므로 runtime 중복은 해소되지 않음을 명시mapping.original.yamlartifact 처분 방침 추가Added/updated tests?
설계 문서 갱신이며 코드 변경 없음Additional notes
docs/plans/2026-03-15-reverse-sync-reconstruction-cleanup-scope.md