편집 프로토콜의 구현 간 동등성과 조합 안전성을 보장한다 - #715
Merged
Merged
Conversation
Owner
Author
merge 전 리뷰 수정·검증원래 완료 조건 2에 해당하는 배열 조상 move의 inverse 결함을 수정했습니다. 회귀 4개가 모두 실패함을 확인한 뒤 수정하여 통과했습니다.
사용자의 “리뷰 하고 merge”에 따라 원래 PR-open 종료 조건만 review·merge·merged workspace 정리로 확장됐고 npm 배포 제외는 유지됩니다. |
Owner
Author
최신 main과 조합 검증#716의 최종 CI 통과와 merge를 확인한 뒤, 그 merge commit
npm 배포 제외와 Core 공개 계약 보존은 유지합니다. |
Owner
Author
merge 확인PR #715을 최신 main과 조합한 검증 head
|
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
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.
Goal Anchor
Closes #713
결과
/editing연결을 추가했습니다. toolbar·키보드·history input·availability가 같은 owner를 사용하며 editor-authored target의 selection을 현재 문서로 복원합니다.보존한 의미와 제한
historyGroup으로 wire 기여를 합치지 않으며 external history의 ignore plan은 변경 전에 거절합니다.큰 snapshot의 조합 회귀
새 inverse의 정상 Core snapshot 소비 과정에서 반복 split/undo가 중단되는 Proxy reflection 예외가 드러났습니다. Core의 기존 shared-array owner에서 reflection 시에만 frozen dense snapshot을 materialize하도록 고쳤습니다. JSON clone이나 Editing에 소비자별 우회를 두지 않았습니다. indexed leaf 경로는 lazy로 유지하며 snapshot 재입력(64/512/4096 항목), reflection·immutability, 반복 structural undo 회귀로 검증했습니다.
검증
npm run build,npm run test:projects: 142 files, 1317 passed, 4 skipped.npm run standard:check,npm run workspace:check,npm run docs:evaluate: 통과.npm run site:verify:pages: build·site evaluation·HTTP smoke 통과.git diff --check: 통과.실제 Chrome와 unavailable 범위
기존 Chrome profile에서 작업트리의 실제 Rich Text 페이지를 열고 합성 beforeinput 입력 → 원격 삽입 → toolbar Undo → historyRedo → 합성 Meta+Z를 확인했습니다. 원격 문자열을 유지하며 내 입력과 caret offset을 복원했고 error/unhandledrejection 기록은 비었습니다. Document 페이지에서는 기본 UUID로 복제 후 move/undo의 ID 유지도 확인했습니다. Usage source에서 공개 connection과 ID API import를 확인했습니다.
macOS System Events 키 입력은 권한 오류 1002로 unavailable입니다. 새 browser profile이나 로그인 상태 변경 없이 진행했습니다. 전체 Playwright browser suite, OS native 키 입력, 실기기 IME 검증을 완료했다고 보고하지 않습니다. browser regression spec은 추가했으며 이번 실제 페이지 증거는 합성 이벤트 경로입니다.
Clean CI source 경로 보완
aa8a43ab에서 Site TypeScript paths에 새/editingsource entry를 등록했습니다. clean CI의 module-resolution 실패를 닫았으며 로컬 source typecheck·전체 build와 재실행 CI의 Verify site 단계가 통과했습니다. External kit은 최초 통과 후 동일 runtime의 재실행에서 keyboard focus 단계가 실패하여 재검증이 필요합니다. workflow 진행 중 개별 job 재실행 요청은 거절되었으며 이를 성공으로 보고하지 않습니다.승인된 merge 단계와 최신 리뷰
사용자가 “리뷰 하고 merge”를 승인하여 원래 PR-open 종료 조건만 리뷰·merge·merged 작업트리 정리로 확장했습니다. 원래 Goal Anchor는 이력으로 보존하며 npm 배포 제외는 유지합니다.
배열 조상으로 move한 값의 undo 결함을 추가 수정했습니다. 4개 회귀의 실패→통과, Editing 221 tests, 전체 1321 passed / 4 skipped, 유효한 move/copy 884개 복원을 확인했습니다. 새 head의 CI가 merge 기준입니다. 이전 head의 External kit 포커스 실패는 검사 변경 없이 재실행에서 통과했습니다.