refactor+test: E4 구조 편집 admission/self-verify DRY + 커버리지 마진#114
Merged
Conversation
delete/insert 에 중복됐던 입력 admission 게이트와 reverse-delta self-verify 를 공유 헬퍼(admit/self_verify)로 추출 — 코드 DRY + 중복 미커버 라인 제거. 안전망 fault-injection 테스트(self_verify 델타 불일치·admit round-trip 거부)로 이전 도달불가 방어 경로 커버. CLI/MCP FILE_WRITE_FAILED·batch json·insert admission 표면 테스트 보강. structural.rs 라인 커버 92.36→94.79%.
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.
E4(#112) 머지 후 워크스페이스 커버리지가 90% 문턱 바로 위에 걸려 Release PR(#113)의 Coverage 게이트가 linux 에서 미세하게 실패. E4 구조 편집기의 방어 코드(도달 불가한 안전망: SpanCountMismatch 등)가 커버리지를 눌러서.
이 PR:
결과: smithy structural.rs 라인 커버 92.36→94.79%, 워크스페이스 90.06%. 안전 코드는 커버리지 위해 제거하지 않음(원칙).