Repository navigation
fix(sync): P2 3건 + 실사용 시험 UX 묶음 수정 - #12
Merged
Merged
Conversation
- sync status 텍스트에서 기기 JSON 원문 대신 레이블·이상 표시만 출력(--json 은 그대로) - sync add·resolve·review 성공 시 한 줄 확인, 종료 코드 8이면 남은 일 안내 - pkcheck 종료 2(에이전트 없음)면 원인과 해결 방법을 안내 Refs DCO-40 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- 수락 중인 후보를 축출에서 보호(pin)하고, 축출된 후보의 vid 바인딩을 정리 - 거절한 제안 기록에 30일 슬라이딩 만료(rejected_until, 구 상태 파일과 호환) - 걸린 후보가 없는 unbound 바인딩을 참조 기준으로 정리 - legacy 로 받아들인 값을 이 기기 변경으로 기록해 범위 기기에 전송 - 같은 기기의 나중 변경이 이어받은 보류분은 퇴역시키고 review 에서 제외 Refs DCO-40 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- 후속 변경이 먼저 와 즉시 퇴역한 보류분의 Rejected receipt 를 Held 로 덮지 않음 - accept() 결과를 Applied/Conflict/Unsettled/Full 로 나눠 review 문구를 실제 결과에 맞춤 - resolve_value() 가 용량 부족이면 false 를 돌려 성공 문구 대신 저장 공간 안내 Refs DCO-40 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AcJNLuNvFR9uo8NsqLVCj6
Refs DCO-40 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AcJNLuNvFR9uo8NsqLVCj6
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.
요약
v0.3.0
kc sync의 남은 P2 3건과 실사용 시험(맥 2대 + 리눅스 1대, 실제 중계 서버)에서 나온 UX 결함을 묶어 고친다.UX (927a42d)
kc sync status텍스트 출력이 기기 정보를 JSON 원문으로 찍던 것을 레이블과 이상 표시만 보이게 바꿨다.--json은 그대로.sync add·resolve·review가 성공해도 아무것도 출력하지 않던 것을 한 줄 확인으로 바꿨다. 종료 코드 8이면 남은 일을 알린다.상태 기계 (b08b4fe)
rejected_until에 두고 기존rejected_offers형식은 유지해, v0.3.0 도 새 상태 파일을 읽는다.리뷰 후속 (4a25d52)
검증
cargo test --workspace: 496 통과, 실패 0cargo clippy --workspace --all-targets -- -D warnings: 경고 없음 /cargo fmt --check: 차이 없음이번에 하지 않은 것
known256 상한 trim 의 슬롯 누수, 거절된 제안의 송신 측 재전송(프로토콜 변경 필요).Refs DCO-40
🤖 Generated with Claude Code
https://claude.ai/code/session_01AcJNLuNvFR9uo8NsqLVCj6