Preserve exact receipts for audio confirmation retries - #2888
Chris0Jeky merged 3 commits into
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
One bounded independent Luna review found no direct HIGH/CRITICAL defect in the confirmation receipt diff. All four request fields plus schema participate in the immutable fingerprint; ownership precedes comparison and exact repeats avoid new writes. The nullable migration and exports agree. Runtimec22960732 is covered by10APItests and fullbackend9,223pass/34skip; additive migration applied to isolatedSQLite. Subsequent commits only record validation. No new review loop requested. |
27aa36d
into
codex/2808-overhaul-followthrough
1 participant
Confirmed audio answers now retain a fingerprint of the exact confirmation request: recording revision, deck revision, written representation and answer status. An identical retry returns the existing receipt. Changing any field returns409 before creating another memory or representation. Older confirmations retain read access but reject unprovable write retries.
The nullable receipt is persisted by an additive migration and retained by both account export routes. Continues #2808 and the confirmed MEDIUM follow-up3975276160 on #2866. Stacked on #2886; no automatic transcription or board mutation is added.
Validation:10realAPItests passed, including exact/mismatched retries, unchanged representation count, exports and legacy-null behavior. Fullbackend9,223passed/34existing skips. The complete migration chain applied successfully to an isolated emptySQLite database. Docs links/governance/diff checks passed. One bounded independent Luna review found no HIGH/CRITICAL defect. No frontend behavior changed and no workingdatabase was migrated.
Exact-head hosted qualification remains required. Existing OUTSTANDING_TASKS.md owner decisions and broader audio/provider/device/restore/release acceptance remain open. Evidence: docs/product/WORKSPACE_OVERHAUL_VALIDATION.md and C:/Taskdeck-overhaul/evidence/audio-confirmation-*.