Sam Valladares c70404b8d1 fix(blackbox): C1 unconditional quarantine release + C2 trace destructive writes ... Two more review findings — both real, both blockers — plus stale-doc cleanup. C1: the B1 release used reverse_suppression(subject_id, labile_hours), which REFUSES once the 24h active-forgetting labile window has passed. So a Memory PR reviewed late could be marked "promoted" while its memory stayed suppressed. Approving a quarantined write is an explicit reviewer decision and must release the memory regardless of elapsed time. New SqliteMemoryStore::release_quarantine fully clears the suppression (count→0, suppressed_at→NULL) with NO time-window limit; the PR handler now uses it. Proven: a test backdates suppressed_at to +100h, shows reverse_suppression refuses, and release_quarantine still releases. C2: memory(action="purge"|"delete") returns `action` + nodeId but those labels weren't in is_write_decision, so destructive removal bypassed the memory.write trace and the PR gate. Added purge/purged/delete/deleted/forget/forgotten. Proven live: purging a node now records a second memory.write event ({"decision":"purge"}) under the run. Docs: REVIEW.md de-staled — removed the fixed 140b15f diff-stat / "3 commits" prose (it moved with each fix), listed all commits, added C1/C2 to the findings table, updated the test count. Gates: 1002 lib tests pass (+3 new regressions), clippy -D warnings clean, dashboard check + build clean. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>		2026-06-24 16:05:35 -05:00
..
benches	feat(v2.0.5): Intentional Amnesia — active forgetting via top-down inhibitory control	2026-04-14 17:30:30 -05:00
src	fix(blackbox): C1 unconditional quarantine release + C2 trace destructive writes	2026-06-24 16:05:35 -05:00
Cargo.toml	chore(release): v2.1.27 — External-Source Connectors	2026-06-19 11:10:54 -05:00