feat: add tenant audit provenance events - #13
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
seonghobae
left a comment
There was a problem hiding this comment.
Current-head review request for 4fafb5d. Please independently verify the append-only audit trigger, tenant RLS behavior under the application role, transaction atomicity for assessment/completion/credential lifecycle events, and the exact-head quality assertions. This is a COMMENT only; no approval or protected-merge bypass is requested.
seonghobae
left a comment
There was a problem hiding this comment.
Current-head review request for e31ed03. Re-verify the exact-head diff, required Checks, tenant/security boundaries, and any valid review findings after the shared Rust component-install fix. COMMENT only; no approval or protected-merge bypass is requested.
Summary
audit_event_recordwith opaque service actor defaults, correlation UUID, source authority/version, event identity, and SHA-256 event digestEvidence
cargo test --workspace --all-targets --lockedcargo clippy --workspace --all-targets --locked -- -D warningsRUSTDOCFLAGS=-Dwarnings cargo doc --workspace --no-deps --lockedcargo build --bin lms_api --lockedcargo fmt --all -- --checkactionlintgit diff --checkGate
This is stacked on PR #12 and remains unmerged product evidence until an independent current-head review and required Checks pass. Merge only after re-fetching exact head
4fafb5dand confirming protected rules.