feat: add PostgreSQL recovery rehearsal - #18
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 |
|
@opencode-agent Review only the exact current head 37382cf against base 80379bc for PR #18. Do not update branches, approve, merge, or bypass protection. Check the forward migration, SQLx embedded-migration behavior, disposable PostgreSQL rehearsal cleanup/connection safety, and whether the documented evidence stays distinct from production-readiness claims. |
|
@opencode-agent Review only the exact current head f0a5e55 on PR #18 against base 80379bc. The head adds the required PostgreSQL smoke assertion for audit_event_record_tenant_correlation_idx. Do not update branches, approve, merge, or bypass protection; re-check migration lock/SQLx behavior, rehearsal cleanup/connection safety, and evidence boundaries. |
Summary
Real local evidence
0002_audit_correlation_index.sqlapplied successfullyVerification
shellcheck scripts/postgres_recovery_rehearsal.shPGHOST=127.0.0.1 PGPORT=5432 PGUSER=$(id -un) scripts/postgres_recovery_rehearsal.shcargo +1.97.1 fmt --all -- --checkcargo +1.97.1 test --workspace --all-targets --lockedcargo +1.97.1 clippy --workspace --all-targets --locked -- -D warningsRUSTDOCFLAGS="-D warnings" cargo +1.97.1 doc --workspace --no-deps --lockedactionlint, docs contract, andgit diff --checkGate
This PR is intentionally stacked on PR #17 exact head
80379bc. It remains subject to an independent current-head review, required Checks, and the protected merge rules. No self-approval, merge, or protection bypass is requested.