fix(privacy): close the remaining Codex P1 gaps after #522 merged - #537
Conversation
|
835ab90 to
0d062af
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 835ab90916
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
d0ef4b4 to
84faac8
Compare
84faac8 to
9426c27
Compare
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
Co-authored-by: Zack Jackson <ScriptedAlchemy@users.noreply.github.com>
9426c27 to
309f82a
Compare
fa9c631
into
codex/tracedecay-total-redesign-plan
Follow-up to the Codex P1 review on #522, which merged into
codex/tracedecay-total-redesign-planbefore all findings were fully settled. This PR closes the gaps that remain on the base.P1 dispositions on the merged base
memory_v2_assertion_payloads(and its FTS backing). This PR enforces it in the store: after a correction commits, every superseded payload row is re-evaluated under the one canonical sanitizer; rows the detector would refuse or rewrite are securely deleted in the same transaction (the FTS delete trigger scrubs the index), while clean history stays readable for as-of reads. A missing payload row on a superseded assertion is now an accepted replay state.facts_removedsum, zero secret-bearing payload rows at rest).DashboardApplicationRuntime::for_project_rootrebinds the daemon transport to the selected project's exact root andbuild_selected_project_statefails closed when rebinding is impossible, covered byselected_project_runtime_rebinds_to_the_selected_root. Verified present on the base; no further change is needed, so this PR carries none.Changes
crates/tracedecay-runtime-core/src/store/memory/crud/commands.rs— purge detector-flagged superseded payload rows inside the correction commit.crates/tracedecay-runtime-core/src/store/memory/crud/commit.rs— accept a purged superseded payload row as consistent during assertion replay verification.crates/tracedecay-runtime-core/src/store/memory/crud/{mod,project}.rs— expose the shared payload wire builder to the commit path.crates/tracedecay-runtime-core/src/store/memory/crud/tests.rs— pin that a clean superseded payload row survives an ordinary update.src/daemon/privacy_remediation.rs— batch-seeded over-capacity rescan fixture with tightened assertions.Verification
cargo test -p tracedecay-runtime-core --lib store::memory— store suite including the new retention test.cargo test -p tracedecay --lib daemon::privacy_remediation— both remediation journeys (quarantine + erase, over-capacity batching) in ~3.6 s.