feat(document-records): add governed HR document evidence - #98
feat(document-records): add governed HR document evidence#98seonghobae wants to merge 23 commits into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (10)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughDocumentRecordEvidence 패키지를 추가했습니다. 최소화된 HR 문서 메타데이터와 무결성 digest를 검증합니다. 관련 ADR, 참조 문서, 추적성 문서 및 설치 휠 품질 검증 워크플로를 추가했습니다. ChangesDocument Record Evidence
Estimated code review effort: 4 (Complex) | ~45 minutes Merge Risk: ⚪ Minimal · up to The PR adds a constrained, evidence-only HR document boundary with successful exact-head validation and no actionable merge-blocking risk remaining beyond normal independent review and repository protection checks. Sequence Diagram(s)sequenceDiagram
participant Caller
participant build_document_record_evidence
participant DocumentRecordEvidence
Caller->>build_document_record_evidence: 키워드 인자 전달
build_document_record_evidence->>DocumentRecordEvidence: 객체 생성 및 입력 검증
Caller->>DocumentRecordEvidence: canonical_json 호출
DocumentRecordEvidence-->>Caller: 검증된 canonical JSON 반환
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Current authority — 2026-09-06
Current exact head is
6a9f3e214079e2b46bba9776a862f194b899f0e4. Protected truth isdevelop@eb9757f8649aaad026a9865508d9aad50c1a7a4f; this branch still retains the olderdevelop@9e3e4847510e1e612b48474ba42b177b8ed824dfsnapshot. Live state is open · Draft · non-mergeable. The document-record evidence delta remains valid and is not a Close candidate.The previous PR prose stopped at predecessor
59b809bead617d9045357396df684991548bdc30. Intervening ordinary successor6a9f3e214079e2b46bba9776a862f194b899f0e4is a real production+test trust-boundary repair:_validate_received_at(...)now requiresrecorded_atitself to be an exact built-in UTCdatetimebefore chronology/canonical-seal processing, andtest_rejects_forged_recorded_time_before_seal_verificationproves a caller-defineddatetimesubtype cannot spoof the original serialized time while retaining a different underlying instant. This successor is adopted as current branch truth; it is not treated as race/noise.Exact-head historical evidence on
6a9f3e...is locally strong: Document Record Evidence Quality, Foundation CI, Recovery, SAST, Security Scan, coverage source/evidence, Dependency Review, and applicable PostgreSQL integrity contracts completed successfully. The exact-head central OpenCode, Noema, and Strix checks are terminal failures, so this historical old-base head is not merge-passing. No predecessor result is promoted over those failures or over the current protected-parent gap.Before Ready, non-force adopt current protected
develop, preserve #161 repository-workflow consolidation and the complete document-evidence trust-boundary delta, resolve semantic/provenance conflicts, reseal deterministic artifacts from the final resolved bytes, then reacquire 100% owned coverage, PostgreSQL evidence where applicable, every then-current central security/review gate, resolved conversations, and a qualifying independent approval on the reconciled exact head. Do not self-approve, use administrator bypass, force-push, create no-op retriggers, weaken gates, or copy foreign-owner source.