Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Base automatically changed from
jk/phase0-xhtml-normalizer-and-list-tree
to
main
March 13, 2026 13:59
… identity helper를 추가합니다 - SidecarBlock에 reconstruction 필드 추가 (kind, old_plain_text, anchors, items) - ROUNDTRIP_SCHEMA_VERSION "2" → "3" 승격, v2 하위 호환 로드 유지 - build_sidecar가 block 타입별 reconstruction metadata를 자동 생성 - build_block_identity_index, find_block_by_identity: hash + line_range disambiguation - Phase 1 전용 테스트 25개 추가 (전체 845 pass) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
jk-kim0
force-pushed
the
jk/phase1-sidecar-schema-v3
branch
from
832048d to
65b8464
Compare
jk-kim0
added a commit
that referenced
this pull request
Mar 14, 2026
- SidecarBlock.to_dict() / from_dict() 메서드를 추가합니다 - build_sidecar_identity_index() / find_sidecar_block_by_identity() helper를 파일 상단으로 이동하고 API를 통일합니다 - _build_reconstruction_metadata()를 BlockMapping 기반으로 재작성합니다 (ordered list, child_xpaths 지원 추가) - load_sidecar() v2 하위 호환 제거 — v3 strict 검증으로 변경합니다 - test_reverse_sync_sidecar_v2.py 제거 및 v3 테스트를 갱신합니다 - 22개 testcase expected.roundtrip.json fixture를 갱신합니다 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This was referenced Mar 14, 2026
jk-kim0
added a commit
that referenced
this pull request
Mar 15, 2026
## Description PR #915(schema v3 기반 필드 추가)가 main에 머지된 이후의 증분 변경사항입니다. - `SidecarBlock.to_dict()` / `from_dict()` 메서드를 추가합니다 - `build_sidecar_identity_index()` / `find_sidecar_block_by_identity()` helper를 파일 상단으로 이동하고 API를 통일합니다 (기존 `build_block_identity_index` / `find_block_by_identity` 대체) - `_build_reconstruction_metadata()`를 `BlockMapping` 기반으로 재작성합니다 — ordered list, `child_xpaths` 지원 추가 - `load_sidecar()` v2 하위 호환 제거 — v3 strict 검증으로 변경합니다 - `test_reverse_sync_sidecar_v2.py` 제거, `test_reverse_sync_sidecar_v3.py` 갱신합니다 - 22개 `testcases/*/expected.roundtrip.json` fixture를 갱신합니다 ## Added/updated tests? - [x] Yes — `test_reverse_sync_sidecar_v3.py` 갱신, 22개 fixture 갱신 ## Additional notes - 이 PR은 PR #915 머지 이후 `origin/main` 위에 rebase된 단일 커밋입니다 Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
jk-kim0
added a commit
that referenced
this pull request
Mar 15, 2026
## Description PR #913 시점의 설계 초안을 2026-03-15 `main` (#917 머지 후) 기준으로 갱신합니다. **수정 파일:** `docs/plans/2026-03-13-reverse-sync-reconstruction-design.md` - 문서 헤더에 갱신일, 기준 커밋(`9e0d43b9`), 반영된 선행 PR(#913–#915, #917) 메타를 추가합니다 - 이미 `main`에 반영된 Phase 0/1 작업과 아직 남은 재구성 작업을 명확히 분리합니다 - Phase 1 후속(#917) 내용을 반영합니다: `strict v3` 전환, identity helper API 통일(`build_sidecar_identity_index` / `find_sidecar_block_by_identity`), `reconstruction` metadata 보강(`child_xpaths`, ordered list) - 현재 런타임 기본 경로(patch_builder → text_transfer)와 최종 목표(fragment reconstruction) 사이의 gap을 재정의합니다 - 남은 구현 단계(Phase 2–5)와 게이트를 현재 코드베이스 기준으로 재확정합니다 **신규 파일:** `docs/plans/2026-03-15-reverse-sync-reconstruction-cleanup-scope.md` - 기존 설계 문서에 포함되어 있던 legacy 코드 삭제 범위를 별도 문서로 분리합니다 - 삭제 시작 조건, 완전 삭제 대상, 부분 축소 대상, 유지 대상을 모듈 단위로 정리합니다 ## Added/updated tests? - [x] No — 문서 전용 변경입니다 ## Additional notes - 코드 변경 없음
3 tasks
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.
Description
SidecarBlock에reconstruction필드를 추가합니다 (kind, old_plain_text, anchors, items).ROUNDTRIP_SCHEMA_VERSION"2" → "3" 승격, v2 하위 호환 로드를 유지합니다.build_sidecar()가 block 타입별 reconstruction metadata를 자동 생성합니다.build_block_identity_index(),find_block_by_identity(): hash + line_range 기반 disambiguation helper를 추가합니다.Phase 1 게이트 충족 항목
Added/updated tests?
test_reverse_sync_sidecar_v3.py(25 tests),test_reverse_sync_sidecar_v2.py(v3 호환 수정)Additional notes
🤖 Generated with Claude Code